2002-06-05 04:55:50

by Keith Owens

[permalink] [raw]
Subject: 2.5.20 config forward references to CONFIG_X86_LOCAL_APIC

arch/i386/config.in tests CONFIG_X86_LOCAL_APIC before it is defined.
dep_bool 'check for P4 thermal throttling interrupt.' CONFIG_X86_MCE_P4THERMAL $CONFIG_X86_MCE $CONFIG_X86_LOCAL_APIC

arch/i386/config.in also includes drivers/acpi/Config.in which tests
CONFIG_X86_LOCAL_APIC before this variable is defined via CONFIG_SMP.

The forward references result in incorrect configurations when
switching config from one cpu type to another or from SMP to UP.


2002-06-05 08:36:05

by Zwane Mwaikambo

[permalink] [raw]
Subject: Re: 2.5.20 config forward references to CONFIG_X86_LOCAL_APIC

Hi Keith,

On Wed, 5 Jun 2002, Keith Owens wrote:

> arch/i386/config.in tests CONFIG_X86_LOCAL_APIC before it is defined.
> dep_bool 'check for P4 thermal throttling interrupt.' CONFIG_X86_MCE_P4THERMAL $CONFIG_X86_MCE $CONFIG_X86_LOCAL_APIC
>
> arch/i386/config.in also includes drivers/acpi/Config.in which tests
> CONFIG_X86_LOCAL_APIC before this variable is defined via CONFIG_SMP.
>
> The forward references result in incorrect configurations when
> switching config from one cpu type to another or from SMP to UP.

We could move the conditional to preprocessor time by wrapping certain
bits in #ifdef (urgh), what really is the more elegant way of doing it?

Regards,
Zwane Mwaikambo
--
http://function.linuxpower.ca



2002-06-05 10:14:27

by Dave Jones

[permalink] [raw]
Subject: Re: 2.5.20 config forward references to CONFIG_X86_LOCAL_APIC

On Wed, Jun 05, 2002 at 10:07:15AM +0200, Zwane Mwaikambo wrote:
> > The forward references result in incorrect configurations when
> > switching config from one cpu type to another or from SMP to UP.
> We could move the conditional to preprocessor time by wrapping certain
> bits in #ifdef (urgh), what really is the more elegant way of doing it?

Doing the CONFIG_X86_LOCAL_APIC definition earlier.

Dave.

--
| Dave Jones. http://www.codemonkey.org.uk
| SuSE Labs

2002-06-05 10:25:14

by Zwane Mwaikambo

[permalink] [raw]
Subject: Re: 2.5.20 config forward references to CONFIG_X86_LOCAL_APIC

On Wed, 5 Jun 2002, Dave Jones wrote:

> On Wed, Jun 05, 2002 at 10:07:15AM +0200, Zwane Mwaikambo wrote:
> > > The forward references result in incorrect configurations when
> > > switching config from one cpu type to another or from SMP to UP.
> > We could move the conditional to preprocessor time by wrapping certain
> > bits in #ifdef (urgh), what really is the more elegant way of doing it?
>
> Doing the CONFIG_X86_LOCAL_APIC definition earlier.

... so simple it hurts ...

--
http://function.linuxpower.ca