From owner-cvs-all@FreeBSD.ORG Wed Jul 2 07:28:33 2003 Return-Path: Delivered-To: cvs-all@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 87E9037B401; Wed, 2 Jul 2003 07:28:33 -0700 (PDT) Received: from mailhub.fokus.fraunhofer.de (mailhub.fokus.fraunhofer.de [193.174.154.14]) by mx1.FreeBSD.org (Postfix) with ESMTP id 747C14400B; Wed, 2 Jul 2003 07:28:31 -0700 (PDT) (envelope-from brandt@fokus.fraunhofer.de) Received: from beagle (beagle [193.175.132.100])h62ESUQ07591; Wed, 2 Jul 2003 16:28:30 +0200 (MEST) Date: Wed, 2 Jul 2003 16:28:30 +0200 (CEST) From: Harti Brandt To: Maxime Henrion In-Reply-To: <20030702140500.GJ42121@elvis.mu.org> Message-ID: <20030702162128.S30117@beagle.fokus.fraunhofer.de> References: <200307021353.h62Drfvp083130@repoman.freebsd.org> <20030702140500.GJ42121@elvis.mu.org> MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII cc: cvs-src@freebsd.org cc: src-committers@freebsd.org cc: cvs-all@freebsd.org Subject: Re: cvs commit: src/sys/dev/fatm if_fatm.c X-BeenThere: cvs-all@freebsd.org X-Mailman-Version: 2.1.1 Precedence: list List-Id: CVS commit messages for the entire tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 02 Jul 2003 14:28:34 -0000 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@fokus.fraunhofer.de, harti@freebsd.org