From mboxrd@z Thu Jan 1 00:00:00 1970 From: "John W. Linville" Date: Fri, 23 Sep 2005 18:22:35 +0000 Subject: Re: [patch 2.6.13 0/6] swiotlb maintenance and x86_64 dma_sync_single_range_for_{cpu,device} Message-Id: <20050923182233.GB6576@tuxdriver.com> List-Id: References: <20050922204155.GA5400@infradead.org> In-Reply-To: <20050922204155.GA5400@infradead.org> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Christoph Hellwig , "Luck, Tony" , linux-kernel@vger.kernel.org, discuss@x86-64.org, linux-ia64@vger.kernel.org, ak@suse.de, "Mallick, Asit K" On Thu, Sep 22, 2005 at 09:41:55PM +0100, Christoph Hellwig wrote: > On Thu, Sep 22, 2005 at 01:37:46PM -0700, Luck, Tony wrote: > > >Conduct some maintenance of the swiotlb code: > > > > > > -- Move the code from arch/ia64/lib to lib > > > > I agree that this code needs to move up out of arch/ia64, it is messy > > that x86_64 needs to reach over and grab this from arch/ia64. > > > > But is "lib" really the right place for it to move to? Perhaps > > a more logical place might be "drivers/pci/swiotlb/" since this > > code is tightly coupled to pci? > > It should just go away once the GFP_DMA32 code is merged. Is that the plan? I suppose it makes sense. So, move it to driver/pci/swiotlb.c? Or just leave it where it is? Either way, I'll redo the other patches to reflect the correct location. John -- John W. Linville linville@tuxdriver.com