Re: [patch] bootmem: use MAX_DMA_ADDRESS instead of LOW32LIMIT

From: Randy.Dunlap
Date: Mon Aug 07 2006 - 01:22:55 EST


On Sat, 29 Jul 2006 05:55:23 +0200 Ingo Molnar wrote:

>
> * Heiko Carstens <heiko.carstens@xxxxxxxxxx> wrote:
>
> > Hm... how about this one then:
> >
> > From: Heiko Carstens <heiko.carstens@xxxxxxxxxx>
> >
> > Introduce ARCH_LOW_ADDRESS_LIMIT which can be set per architecture to
> > override the 4GB default limit used by the bootmem allocater within
> > __alloc_bootmem_low() and __alloc_bootmem_low_node().
> > E.g. s390 needs a 2GB limit instead of 4GB.
> >
> > Cc: Ingo Molnar <mingo@xxxxxxx>
>
> Acked-by: Ingo Molnar <mingo@xxxxxxx>
>
> (although you might get some flak about using an ARCH* define. I'm not
> sure what the current upstream policy is - using an #ifndef default
> value is the most compact hence sanest thing to do, still it's sometimes
> being frowned upon in favor of sprinkling the default value into every
> architecture's processor.h. Putting the value into a Kconfig and
> combining it with #ifndef might be better.)

(sorry for the delay, too much travel/conferences)

I agree with your ordering. Linus wrote about the current
ARCH_HAS* (and HAVE_ARCH* I suppose):
"WE SHOULD GET RID OF ARCH_HAS_XYZZY. It's a disease."

I have patches for some of these that I will post soon (prob.
Monday), converting several ARCH_HAS* to CONFIG_ namespace.

---
~Randy
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/