[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <200807110021.29392.nickpiggin@yahoo.com.au>
Date: Fri, 11 Jul 2008 00:21:28 +1000
From: Nick Piggin <nickpiggin@...oo.com.au>
To: Jack Steiner <steiner@....com>
Cc: Hugh Dickins <hugh@...itas.com>,
Christoph Hellwig <hch@...radead.org>, cl@...ux-foundation.org,
akpm@...l.org, linux-kernel@...r.kernel.org, mingo@...e.hu,
tglx@...utronix.de, holt@....com, andrea@...ranet.com,
linux-mm@...ck.org
Subject: Re: [patch 12/13] GRU Driver V3 - export is_uv_system(), zap_page_range() & follow_page()
On Thursday 10 July 2008 23:29, Jack Steiner wrote:
> On Thu, Jul 10, 2008 at 05:31:54PM +1000, Nick Piggin wrote:
> > On Thursday 10 July 2008 05:11, Jack Steiner wrote:
> > > I'll post the new GRU patch in a few minutes.
> >
> > It looks broken to me. How does it determine whether it has a
> > normal page or not?
>
> Right. Hugepages are not currently supported by the GRU. There is code that
> I know is missing/broken in this path. I'm trying to get the core driver
> accepted, then I'll get the portion dealing with hugepages working.
Oh, I meant "normal" pages as in vm_normal_page(), or is there some
other reason this codepath is exempt from them?
Using gup.c code I don't think will prevent your driver from getting
accepted. Conversely, I would not like the open coded page table walk
to go upstream...
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists