Date: Mon, 22 Dec 2008 14:34:57 +0900
To: jeremy@goop.org
Cc: beckyb@kernel.crashing.org, mingo@elte.hu, fujita.tomonori@lab.ntt.co.jp,
       linux-kernel@vger.kernel.org, ian.campbell@citrix.com,
       jbeulich@novell.com, joerg.roedel@amd.com, benh@kernel.crashing.org
Subject: Re: [PATCH 06/11] swiotlb: Store phys address in io_tlb_orig_addr
 array
From: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>
In-Reply-To: <494BDC6E.8050509@goop.org>
References: <20081218210231.GB24271@elte.hu>
	<1229663480-10757-7-git-send-email-beckyb@kernel.crashing.org>
	<494BDC6E.8050509@goop.org>
Mime-Version: 1.0
Content-Type: Text/Plain; charset=us-ascii
Content-Transfer-Encoding: 7bit
Message-Id: <20081222143507R.fujita.tomonori@lab.ntt.co.jp>
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 826
Lines: 16

On Fri, 19 Dec 2008 09:39:58 -0800
Jeremy Fitzhardinge <jeremy@goop.org> wrote:

> Becky Bruce wrote:
> > When we enable swiotlb for platforms that support HIGHMEM, we
> > can no longer store the virtual address of the original dma
> > buffer, because that buffer might not have a permament mapping.
> > Change the iotlb code to instead store the physical address of
> > the original buffer.
> >   
> 
> Hm, yes, I think using a phys_addr_t may end up being cleaner than using 
> struct page *+offset.

Surely, cleaner and simpler. Can you please rework [PATCH 07/11] in
that way?
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/