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

Harti Brandt brandt at fokus.fraunhofer.de
Wed Jul 2 07:28:34 PDT 2003


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...

MH>I'll document the BUS_DMA_NOWAIT flag for bus_dmamap_load() in bus_dma.9
MH>soon.

That is nice to hear. Someone with enough bus_dma- and man- knowledge
could also have a look at docs/53271 and docs/53751.

harti
-- 
harti brandt,
http://www.fokus.fraunhofer.de/research/cc/cats/employees/hartmut.brandt/private
brandt at fokus.fraunhofer.de, harti at freebsd.org


More information about the cvs-src mailing list