Re: [Bugme-new] [Bug 2019] New: Bug from the mmsubsystem involving X (fwd)

From: Dave Hansen
Date: Fri Feb 06 2004 - 12:25:08 EST


On Fri, 2004-02-06 at 07:49, Martin J. Bligh wrote:
> >> +#ifdef CONFIG_NUMA
> >> + #ifdef CONFIG_X86_NUMAQ
> >> + #include <asm/numaq.h>
> >> + #else /* summit or generic arch */
> >> + #include <asm/srat.h>
> >> + #endif
> >> +#else /* !CONFIG_NUMA */
> >> + #define get_memcfg_numa get_memcfg_numa_flat
> >> + #define get_zholes_size(n) (0)
> >> +#endif /* CONFIG_NUMA */
> >
> > We ran into a bug with #ifdefs like this before. It was fixed in some
> > of the code that you're trying to remove.
>
> What bug?

With a regular PC config, plus CONFIG_NUMA turned on:
CC arch/i386/kernel/process.o
In file included from include/asm/mmzone.h:17,
from include/linux/mmzone.h:318,
from include/linux/gfp.h:4,
from include/linux/slab.h:15,
from include/linux/percpu.h:4,
from include/linux/sched.h:31,
from include/linux/module.h:10,
from init/do_mounts.c:1:
include/asm/srat.h:31: #error CONFIG_ACPI_SRAT not defined, and srat.h
header has been included
In file included from include/asm/mmzone.h:17,
from include/linux/mmzone.h:318,
from include/linux/gfp.h:4,
from include/linux/slab.h:15,
from include/linux/percpu.h:4,
from include/linux/rcupdate.h:42,
from include/linux/dcache.h:10,
from include/linux/fs.h:17,
from init/do_mounts_initrd.c:3:

I can post the config if you like. You were the one who made me go fix
it in the first place. That's why I added that #error. :)

--dave

-
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/