From: David Howells <dhowells@redhat.com>
To: Andrew Morton <akpm@osdl.org>
Cc: torvalds@osdl.org, wli@holomorphy.com, hch@infradead.org,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH] Compound page overhaul
Date: Tue, 23 Nov 2004 17:48:18 +0000 [thread overview]
Message-ID: <16887.1101232098@redhat.com> (raw)
In-Reply-To: <20041123085608.3c30aa34.akpm@osdl.org>
> So why did you create a "Compound page overhaul" in the first place? Was it
> not to address some insufficiency for !MMU?
Not entirely. Part of it was to improve for !MMU use, and part of it was it
looked like I could improve it in general both by making it more readable and
by things such as making page->private available on the head page.
Linus suggested adding a CONFIG_COMPOUND_PAGE or something similar. By making
half of the compound page stuff mandatory I could also get rid of some
#ifdefs[*] for what appears to be a small overhead when allocating high-order
pages when HUGETLBFS is not defined by making use of the fact that we'd be
tickling the cache over the secondary page structures anyway.
[*] People seem to want to give me the impression that #ifdefs are evil and
should all be buried at least 10 feet down:-)
This in turn provides a way to simplify a number of other things, such as the
"free_pages" functions.
There should be no overhead on single page handling when
ENHANCED_COMPOUND_PAGES is not set. If it is set, then the overhead is pretty
much the same as for hugetlbfs being compiled in now.
> The current compound page logic should handle that quite happily, no?
The current compound page implementation takes page->private away. What I've
done gives it back, currently at the cost of one page flag bit, but there are
ways around even that.
David
next prev parent reply other threads:[~2004-11-23 19:11 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2004-11-22 13:27 [PATCH] Compound page overhaul David Howells
2004-11-22 14:41 ` William Lee Irwin III
2004-11-22 16:07 ` David Howells
2004-11-22 16:34 ` William Lee Irwin III
2004-11-22 23:54 ` Andrew Morton
2004-11-23 9:18 ` David Howells
2004-11-23 16:11 ` Andrew Morton
2004-11-23 16:48 ` David Howells
2004-11-23 16:56 ` Andrew Morton
2004-11-23 17:48 ` David Howells [this message]
2004-11-23 17:10 ` William Lee Irwin III
2004-11-23 17:24 ` David Howells
2004-11-23 17:46 ` William Lee Irwin III
2004-11-23 17:51 ` David Howells
2004-11-24 14:22 ` Greg Ungerer
2004-11-24 18:03 ` David Howells
2004-11-25 3:37 ` Greg Ungerer
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=16887.1101232098@redhat.com \
--to=dhowells@redhat.com \
--cc=akpm@osdl.org \
--cc=hch@infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=torvalds@osdl.org \
--cc=wli@holomorphy.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox