From owner-cvs-src@FreeBSD.ORG Tue May 27 11:31:25 2003 Return-Path: Delivered-To: cvs-src@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 5D0EA37B405 for ; Tue, 27 May 2003 11:31:25 -0700 (PDT) Received: from rootlabs.com (root.org [67.118.192.226]) by mx1.FreeBSD.org (Postfix) with SMTP id 1D29243FA3 for ; Tue, 27 May 2003 11:31:24 -0700 (PDT) (envelope-from nate@rootlabs.com) Received: (qmail 6285 invoked by uid 1000); 27 May 2003 18:31:24 -0000 Date: Tue, 27 May 2003 11:31:24 -0700 (PDT) From: Nate Lawson To: Scott Long In-Reply-To: <3ED2E303.7070304@btc.adaptec.com> Message-ID: <20030527112937.K6264@root.org> References: <20030526040103.EF60537B401@hub.freebsd.org> <20030526201453.V5016@root.org> <3ED2E303.7070304@btc.adaptec.com> 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/alpha/alpha busdma_machdep.csrc/sys/alpha/include bus.h src/sys/amd64/amd64 busdma_machdep.c src/sys/amd64/include bus_dma.h src/sys/i386/i386 busdma_machdep X-BeenThere: cvs-src@freebsd.org X-Mailman-Version: 2.1.1 Precedence: list List-Id: CVS commit messages for the src tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 27 May 2003 18:31:26 -0000 On Mon, 26 May 2003, Scott Long wrote: > Nate Lawson wrote: > >> De-orbit bus_dmamem_alloc_size(). It's a hack and was never used anyways. > >> No need for it to pollute the 5.x API any further. > > > > Do you plan to address this in the future? I believe it was useful for a > > few drivers. I seem to recall you mentioning a more general approach for > > the future. > > The better approach is to have bus_dmamap_load() respect the maxsegs > field of the tag and coelesce the segments by whatever means it has > available to it (bcopy, GART, etc). Yes, that was it. Do you have any plans to implement this? > Note the the bus_dmamem_alloc_size stuff went into the tree in Jan but > was never once used. I thought it would be useful for some existing uses of contigmalloc() but didn't get around to converting them since you said it was temporary anyway. This would help with the busdma conversion effort. -Nate