Re: kbuild: fixing the select problem

[James Bottomley <James.Bottomley@HansenPartnership.com>]

On Thu, 2010-05-06 at 13:59 -0700, Randy Dunlap wrote:
> On Thu, 06 May 2010 15:48:35 -0500 James Bottomley wrote:
> 
> > On Thu, 2010-05-06 at 09:52 -0500, James Bottomley wrote:
> > > The list is
> > > 
> > > USB_ARCH_HAS_HCD has 4 defaults
> > > DEFCONFIG_LIST has 5 defaults
> > > MAC80211_RC_DEFAULT has 2 defaults
> > > X86_L1_CACHE_SHIFT has 2 defaults
> > > SPLIT_PTLOCK_CPUS has 2 defaults
> > > X86_MINIMUM_CPU_FAMILY has 3 defaults
> > > DEFAULT_TCP_CONG has 2 defaults
> > > DEFCONFIG_LIST has 5 defaults
> > > USB_ARCH_HAS_HCD has 4 defaults
> > > X86_L1_CACHE_SHIFT has 2 defaults
> > > X86_MINIMUM_CPU_FAMILY has 3 defaults
> > > SPLIT_PTLOCK_CPUS has 2 defaults
> > > DEFAULT_TCP_CONG has 2 defaults
> > > MAC80211_RC_DEFAULT has 2 defaults
> > 
> > Here's a patch that alters the default processing to the needed form and
> > checks the old vs new values.  I've been running randconfigs but I can't
> > get the warning to trip ... have at it.
> > 
> 
> I don't see the warning, but I took a problem config from linux-next 2010-0506
> (today) [attached].  In it, CONFIG_TCG_BIOS_LOG=y and that selects ACPI,
> so now ACPI is enabled, but ACPI depends on PM and PCI, but they are still not
> selected...
> 
> or is this only step 1 of a multi-step implementation?

Actually, it's not any step.  It's just validation of the contention
that I can alter the default parser from taking the first valid default
to calculating the or of the defaults without changing the current
kbuild behaviour.

The patch to fix kbuild goes on top of this change minus the warnings
(when I actually code it up).

James