[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <Pine.LNX.4.64.0704201207120.21342@schroedinger.engr.sgi.com>
Date: Fri, 20 Apr 2007 12:10:43 -0700 (PDT)
From: Christoph Lameter <clameter@....com>
To: Dave Kleikamp <shaggy@...ux.vnet.ibm.com>
cc: Mel Gorman <mel@...net.ie>, linux-kernel@...r.kernel.org,
Peter Zijlstra <a.p.zijlstra@...llo.nl>,
Nick Piggin <nickpiggin@...oo.com.au>, Andi Kleen <ak@...e.de>,
Paul Jackson <pj@....com>, Dave Chinner <dgc@....com>
Subject: Re: [RFC 4/8] Enhance fallback functions in libs to support higher
order pages
On Fri, 20 Apr 2007, Dave Kleikamp wrote:
> On Fri, 2007-04-20 at 12:05 +0100, Mel Gorman wrote:
>
> > comments about missing page_cache_size() covered elsewhere. However, I
> > note that Dave Kleikamp might be interested in this changing of
> > page_cache_size() from the perspective of page cache tails. I've added
> > him to the cc so he can take a quick look.
>
> Yeah. I'm working on patches for storing file tails in buffers
> allocated from the slab cache, and the tail will be represented by a
> fake struct page. (This is primarily for kernels with a larger page
> size). So my version of page_cache_size(page) may return a different
> size for different pages belonging to the same mapping. I'm in the
> midst of cleaning up the patches and plan to post them to linux-mm by
> Monday.
I am not sure what the point of that patchset would be in this context
given that this is about support for arbitrary page sizes. If the
filesystem wants it then it can reduce the page size for small files.
Different page sizes for one mapping may introduce high complexity into a
filesystem.
And we can already represent different page size. We have compound pages
support in the kernel.
page_cache_size(page) in terms of current code in mm is
PAGE_SIZE << compound_order(page)
-
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