cvs commit: src/sys/dev/fatm if_fatm.c

[Maxime Henrion]

Harti Brandt wrote:
> On Wed, 2 Jul 2003, Maxime Henrion wrote:
> 
> MH>Hartmut Brandt wrote:
> MH>> harti       2003/07/02 06:53:41 PDT
> MH>>
> MH>>   FreeBSD src repository
> MH>>
> MH>>   Modified files:
> MH>>     sys/dev/fatm         if_fatm.c
> MH>>   Log:
> MH>>   Make the bus_dma_tag_create use NULL for the lock arguments. We are
> MH>>   careful to call all map_load calls with BUS_DMA_NOWAIT because we
> MH>>   really don't want some PDUs to wait while others go out - ATM guarantees
> MH>>   the ordering of cells and also of PDUs (within one VC, that is). With
> MH>>   BUS_DMA_NOWAIT bus_dmamap_load should never return EINPROGRESS.
> MH>
> MH>FWIW, this is true since yesterday only, because I updated all the other
> MH>busdma backends which use bounce pages to behave this way.  Previously,
> MH>this was only true for x86 and amd64.
> 
> When locking through all the bus_dma implementations I always wonder
> whether it is possible to factor out the common code (except for sparc64).
> They all lock the same, just with different small bugs...

Yes, I'm going to factor out as much code as I can from the busdma API,
and this is why I sync'd all the backends with each other.

Cheers,
Maxime