lkml.org 
[lkml]   [2002]   [Nov]   [7]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH][2.5-AC] Forced enable/disable local APIC
On Thu, 7 Nov 2002, Mikael Pettersson wrote:

> Zwane Mwaikambo writes:
> > +int enable_local_apic_flag __initdata = 0; /* 0=probe, 1=force, 2=disable e.g. DMI */
> ...
> > + if (enable_local_apic_flag == 1)
> > + goto force_apic;
> >
> > switch (boot_cpu_data.x86_vendor) {
> > case X86_VENDOR_AMD:
> > @@ -642,6 +661,7 @@
> > goto no_apic;
> > }
> >
> > +force_apic:
> > if (!cpu_has_apic) {
> > /*
> > * Some BIOSes disable the local APIC in the
>
> Of what use is the force case? If someone boots with "lapic" on a CPU
> where the APIC feature bit is off, then the code will rdmsr/wrmsr on
> APICBASE, even though we (the kernel) haven't verified that the CPU
> actually has that MSR. This is doubleplusungood.

The assumption is a user knows better what he wants to achieve. But I'm
not sure we really need it here. A DMI override with the APICBASE
availability test in place should suffice -- I'd remove the "goto".

--
+ Maciej W. Rozycki, Technical University of Gdansk, Poland +
+--------------------------------------------------------------+
+ e-mail: macro@ds2.pg.gda.pl, PGP key available +

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2005-03-22 13:30    [W:0.044 / U:7.248 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site