On Wed, Feb 08, 2006 at 02:24:51PM -0800, Chen, Kenneth W wrote:
> Adrian Bunk wrote on Wednesday, February 08, 2006 1:38 PM
> > > Not really, it helps a bit by selecting some things we know we need
> > > for all GENERIC builds. True we can't make it bullet proof, but whats
> > > there is better than removing it.
> >
> > Like the bug of allowing the illegal configuration NUMA=y, FLATMEM=y?
>
>
> You can't even compile a kernel with that combination ...
> Just about every arch except ia64 turns off ARCH_FLATMEM_ENABLE if NUMA=y.
> ia64 can just do the same thing. Instead of mucking around with select,
> fix the bug at its source. The real culprit is in mm/Kconfig, it shouldn't
> enable ARCH_FLATMEM_ENABLE if NUMA=y.
No, the bug is exactly the part of arch/ia64/Kconfig you are patching,
because mm/Kconfig simply relies on architectures setting the right
dependencies for ARCH_FLATMEM_ENABLE.
> Fix ARCH_FLATMEM_ENABLE dependency in ia64 arch.
>
> Signed-off-by: Ken Chen <[email protected]>
>
> --- ./arch/ia64/Kconfig.orig 2006-02-08 14:57:40.597354431 -0800
> +++ ./arch/ia64/Kconfig 2006-02-08 15:04:15.552427718 -0800
> @@ -298,7 +298,8 @@ config ARCH_DISCONTIGMEM_ENABLE
> See <file:Documentation/vm/numa> for more.
>
> config ARCH_FLATMEM_ENABLE
> - def_bool y
> + depends on !NUMA
> + def_bool y if !NUMA
>...
Only one of the two NUMA dependencies is required.
cu
Adrian
--
"Is there not promise of rain?" Ling Tan asked suddenly out
of the darkness. There had been need of rain for many days.
"Only a promise," Lao Er said.
Pearl S. Buck - Dragon Seed
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[Index of Archives]
[Kernel Newbies]
[Netfilter]
[Bugtraq]
[Photo]
[Stuff]
[Gimp]
[Yosemite News]
[MIPS Linux]
[ARM Linux]
[Linux Security]
[Linux RAID]
[Video 4 Linux]
[Linux for the blind]
[Linux Resources]
