Re: [PATCH] Don't override CONFIG_64BIT for ARCH={i386,x86_64}builds

From: Ingo Molnar
Date: Thu Feb 12 2009 - 02:27:18 EST



* Ingo Molnar <mingo@xxxxxxx> wrote:

>
> * David Woodhouse <dwmw2@xxxxxxxxxxxxx> wrote:
>
> > It is a steaming great pain in the arse when the value of CONFIG_64BIT
> > explicitly written in my .config file is overridden by the value of
> > $ARCH inferred from the environment.
> >
> > If I have a 32-bit .config and I happen to build it without remembering
> > to put 'ARCH=i386' on the make command line, it shouldn't force
> > CONFIG_64BIT=y and reconfigure.
> >
> > This patch should fix that, while still allowing defconfig to work as it
> > currently does for both 32-bit and 64-bit environments, and still giving
> > the default value of CONFIG_64BIT according to $ARCH. It's just that it
> > won't _force_ a change to CONFIG_64BIT when it's set to something other
> > than what $ARCH would imply.
> >
> > Signed-off-by: David Woodhouse <David.Woodhouse@xxxxxxxxx>
>
> Applied to tip:x86/ugent, thanks David!

i removed it again, because it regresses randconfig behavior:

titan:~/tip> grep X86_32 .config; make ARCH=x86_64 randconfig; grep X86_32 .config
# CONFIG_X86_32 is not set
scripts/kconfig/conf -r arch/x86/Kconfig
#
# configuration written to .config
#
CONFIG_X86_32=y
CONFIG_X86_32_SMP=y
CONFIG_X86_32_LAZY_GS=y
CONFIG_X86_32_ALWAYS_ON=y

I dont mind the change you are after, but randconfig should work as
expected too: if ARCH=x86_64 is passed it should generate a 64-bit
randconfig, not a 50% 32-bit one.

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