diff options
author | Andrew Cooper <andrew.cooper3@citrix.com> | 2011-08-19 09:58:22 +0100 |
---|---|---|
committer | Andrew Cooper <andrew.cooper3@citrix.com> | 2011-08-19 09:58:22 +0100 |
commit | b5a0e71dd79e976796098f415e30dfcb276de9f8 (patch) | |
tree | cfa0cfb876be7f477549fdebaeaa3857337de515 /xen/arch/x86/crash.c | |
parent | b57bd6d2732468362c1dd3be0b26ccf5f4afbf0d (diff) | |
download | xen-b5a0e71dd79e976796098f415e30dfcb276de9f8.tar.gz xen-b5a0e71dd79e976796098f415e30dfcb276de9f8.tar.bz2 xen-b5a0e71dd79e976796098f415e30dfcb276de9f8.zip |
x86/KEXEC: disable hpet legacy broadcasts earlier
On x2apic machines which booted in xapic mode,
hpet_disable_legacy_broadcast() sends an event check IPI to all online
processors. This leads to a protection fault as the genapic blindly
pokes x2apic MSRs while the local apic is in xapic mode.
One option is to change genapic when we shut down the local apic, but
there are still problems with trying to IPI processors in the online
processor map which are actually sitting in NMI loops
Another option is to have each CPU take itself out of the online CPU
map during the NMI shootdown.
Realistically however, disabling hpet legacy broadcasts earlier in the
kexec path is the easiest fix to the problem.
Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com>
Diffstat (limited to 'xen/arch/x86/crash.c')
-rw-r--r-- | xen/arch/x86/crash.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/xen/arch/x86/crash.c b/xen/arch/x86/crash.c index 392c8d2395..2636950dbc 100644 --- a/xen/arch/x86/crash.c +++ b/xen/arch/x86/crash.c @@ -29,6 +29,7 @@ #include <asm/io_apic.h> #include <xen/iommu.h> #include <xen/pci.h> +#include <asm/hpet.h> static atomic_t waiting_for_crash_ipi; static unsigned int crashing_cpu; @@ -61,6 +62,9 @@ static void nmi_shootdown_cpus(void) local_irq_disable(); + if ( hpet_broadcast_is_available() ) + hpet_disable_legacy_broadcast(); + crashing_cpu = smp_processor_id(); local_irq_count(crashing_cpu) = 0; |