Commit cbf2829b authored by Bryan O'Donoghue's avatar Bryan O'Donoghue Committed by H. Peter Anvin

x86, apic: APIC code touches invalid MSR on P5 class machines

Current APIC code assumes MSR_IA32_APICBASE is present for all systems.
Pentium Classic P5 and friends didn't have this MSR. MSR_IA32_APICBASE
was introduced as an architectural MSR by Intel @ P6.

Code paths that can touch this MSR invalidly are when vendor == Intel &&
cpu-family == 5 and APIC bit is set in CPUID - or when you simply pass
lapic on the kernel command line, on a P5.

The below patch stops Linux incorrectly interfering with the
MSR_IA32_APICBASE for P5 class machines. Other code paths exist that
touch the MSR - however those paths are not currently reachable for a
conformant P5.
Signed-off-by: default avatarBryan O'Donoghue <bryan.odonoghue@linux.intel.com>
Link: http://lkml.kernel.org/r/4F8EEDD3.1080404@linux.intel.comSigned-off-by: default avatarH. Peter Anvin <hpa@linux.intel.com>
Cc: <stable@vger.kernel.org>
parent 089f9fba
...@@ -1637,9 +1637,11 @@ static int __init apic_verify(void) ...@@ -1637,9 +1637,11 @@ static int __init apic_verify(void)
mp_lapic_addr = APIC_DEFAULT_PHYS_BASE; mp_lapic_addr = APIC_DEFAULT_PHYS_BASE;
/* The BIOS may have set up the APIC at some other address */ /* The BIOS may have set up the APIC at some other address */
rdmsr(MSR_IA32_APICBASE, l, h); if (boot_cpu_data.x86 >= 6) {
if (l & MSR_IA32_APICBASE_ENABLE) rdmsr(MSR_IA32_APICBASE, l, h);
mp_lapic_addr = l & MSR_IA32_APICBASE_BASE; if (l & MSR_IA32_APICBASE_ENABLE)
mp_lapic_addr = l & MSR_IA32_APICBASE_BASE;
}
pr_info("Found and enabled local APIC!\n"); pr_info("Found and enabled local APIC!\n");
return 0; return 0;
...@@ -1657,13 +1659,15 @@ int __init apic_force_enable(unsigned long addr) ...@@ -1657,13 +1659,15 @@ int __init apic_force_enable(unsigned long addr)
* MSR. This can only be done in software for Intel P6 or later * MSR. This can only be done in software for Intel P6 or later
* and AMD K7 (Model > 1) or later. * and AMD K7 (Model > 1) or later.
*/ */
rdmsr(MSR_IA32_APICBASE, l, h); if (boot_cpu_data.x86 >= 6) {
if (!(l & MSR_IA32_APICBASE_ENABLE)) { rdmsr(MSR_IA32_APICBASE, l, h);
pr_info("Local APIC disabled by BIOS -- reenabling.\n"); if (!(l & MSR_IA32_APICBASE_ENABLE)) {
l &= ~MSR_IA32_APICBASE_BASE; pr_info("Local APIC disabled by BIOS -- reenabling.\n");
l |= MSR_IA32_APICBASE_ENABLE | addr; l &= ~MSR_IA32_APICBASE_BASE;
wrmsr(MSR_IA32_APICBASE, l, h); l |= MSR_IA32_APICBASE_ENABLE | addr;
enabled_via_apicbase = 1; wrmsr(MSR_IA32_APICBASE, l, h);
enabled_via_apicbase = 1;
}
} }
return apic_verify(); return apic_verify();
} }
...@@ -2209,10 +2213,12 @@ static void lapic_resume(void) ...@@ -2209,10 +2213,12 @@ static void lapic_resume(void)
* FIXME! This will be wrong if we ever support suspend on * FIXME! This will be wrong if we ever support suspend on
* SMP! We'll need to do this as part of the CPU restore! * SMP! We'll need to do this as part of the CPU restore!
*/ */
rdmsr(MSR_IA32_APICBASE, l, h); if (boot_cpu_data.x86 >= 6) {
l &= ~MSR_IA32_APICBASE_BASE; rdmsr(MSR_IA32_APICBASE, l, h);
l |= MSR_IA32_APICBASE_ENABLE | mp_lapic_addr; l &= ~MSR_IA32_APICBASE_BASE;
wrmsr(MSR_IA32_APICBASE, l, h); l |= MSR_IA32_APICBASE_ENABLE | mp_lapic_addr;
wrmsr(MSR_IA32_APICBASE, l, h);
}
} }
maxlvt = lapic_get_maxlvt(); maxlvt = lapic_get_maxlvt();
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment