From: Paul Mundt <lethal@linux-sh.org>
To: Dave Hansen <haveblue@us.ibm.com>
Cc: linux-arch@vger.kernel.org
Subject: Re: [RFC][PATCH] unify all architecture PAGE_SIZE definitions
Date: Fri, 25 Aug 2006 14:31:27 +0900 [thread overview]
Message-ID: <20060825053127.GA9066@localhost.hsdv.com> (raw)
In-Reply-To: <20060824234445.6B519D52@localhost.localdomain>
On Thu, Aug 24, 2006 at 04:44:45PM -0700, Dave Hansen wrote:
> This patch unifies all of those definitions. It defines PAGE_SIZE in
> a single header which gets its definitions from Kconfig. The new
> Kconfig options mirror what used to be done with #ifdefs and
> arch-specific Kconfig options. The new Kconfig menu eliminates
> the need for parisc, ia64, and sparc64 to have their own "choice"
> menus for selecting page size. The help text has been adapted from
> these three architectures, but is now more generic.
>
I like the idea of asm-generic/page_size.h (though why not just use
asm-generic/page.h?) and ripping that out from the architecture headers,
though the mm/Kconfig bits look like something best left in the
architecture Kconfigs.
PAGE_SHIFT is really the only thing of interest, and that can be handled
without the CONFIG_PAGE_SIZE mess. The architectures that really care
about it can use their current hacks or just check PAGE_SHIFT.
next prev parent reply other threads:[~2006-08-25 5:31 UTC|newest]
Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-08-24 23:44 [RFC][PATCH] unify all architecture PAGE_SIZE definitions Dave Hansen
2006-08-24 23:58 ` William Lee Irwin III
2006-08-25 0:07 ` Dave Hansen
2006-08-25 0:14 ` William Lee Irwin III
2006-08-25 0:23 ` Dave Hansen
2006-08-25 0:32 ` William Lee Irwin III
2006-08-25 8:12 ` Geert Uytterhoeven
2006-08-25 5:31 ` Paul Mundt [this message]
2006-08-25 15:03 ` Dave Hansen
2006-08-26 1:17 ` Paul Mundt
2006-08-25 21:04 ` Dave Hansen
2006-08-25 8:11 ` Geert Uytterhoeven
2006-08-25 14:55 ` Dave Hansen
2006-08-25 15:07 ` Geert Uytterhoeven
2006-08-25 17:16 ` David Howells
2006-08-25 17:31 ` Dave Hansen
2006-09-11 0:51 ` Jeff Dike
2006-09-11 17:33 ` Dave Hansen
2006-09-11 18:43 ` Dave Hansen
2006-09-11 23:40 ` Jeff Dike
2006-09-11 13:34 ` Ralf Baechle
2006-09-11 17:43 ` Dave Hansen
[not found] <BFECAF9E178F144FAEF2BF4CE739C66804016FA5@exmail1.se.axis.com>
2006-08-25 5:04 ` Mikael Starvik
-- strict thread matches above, loose matches on Subject: below --
2006-08-25 16:06 Luck, Tony
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=20060825053127.GA9066@localhost.hsdv.com \
--to=lethal@linux-sh.org \
--cc=haveblue@us.ibm.com \
--cc=linux-arch@vger.kernel.org \
/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