All of lore.kernel.org
 help / color / mirror / Atom feed
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.

  parent reply	other threads:[~2006-08-25  5:31 UTC|newest]

Thread overview: 30+ 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
  -- strict thread matches above, loose matches on Subject: below --
2006-08-25 16:06 Luck, Tony
     [not found] <BFECAF9E178F144FAEF2BF4CE739C66804016FA5@exmail1.se.axis.com>
2006-08-25  5:04 ` Mikael Starvik
2006-08-24 23:44 Dave Hansen
2006-08-24 23:44 ` Dave Hansen
2006-08-25 15:42 ` Christoph Lameter
2006-08-25 15:42   ` Christoph Lameter
2006-08-25 15:51   ` Dave Hansen
2006-08-25 15:51     ` Dave Hansen

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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.