diff options
author | Andrew Cooper <andrew.cooper3@citrix.com> | 2012-06-11 15:12:50 +0100 |
---|---|---|
committer | Andrew Cooper <andrew.cooper3@citrix.com> | 2012-06-11 15:12:50 +0100 |
commit | 035a14476e384494a9c70139dd1747d6e9fd448e (patch) | |
tree | fd89b687109c46e2d1fdb438958e056406857f44 /xen/arch/x86/traps.c | |
parent | 073d4837444dc3b7480a463945317c2b6c9110c5 (diff) | |
download | xen-035a14476e384494a9c70139dd1747d6e9fd448e.tar.gz xen-035a14476e384494a9c70139dd1747d6e9fd448e.tar.bz2 xen-035a14476e384494a9c70139dd1747d6e9fd448e.zip |
x86/nmi: Fix deadlock in unknown_nmi_error()
Additionally, correct the text description to reflect what is being
done, and make use of fatal_trap() in preference to kexec_crash() in
case an unknown NMI occurs before a kdump kernel has been loaded.
Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com>
Committed-by: Keir Fraser <keir@xen.org>
Diffstat (limited to 'xen/arch/x86/traps.c')
-rw-r--r-- | xen/arch/x86/traps.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/xen/arch/x86/traps.c b/xen/arch/x86/traps.c index 03fd696505..d4fdf64e1d 100644 --- a/xen/arch/x86/traps.c +++ b/xen/arch/x86/traps.c @@ -3293,7 +3293,7 @@ static void io_check_error(struct cpu_user_regs *regs) outb((inb(0x61) & 0x07) | 0x00, 0x61); /* enable IOCK */ } -static void unknown_nmi_error(unsigned char reason) +static void unknown_nmi_error(struct cpu_user_regs *regs, unsigned char reason) { switch ( opt_nmi[0] ) { @@ -3302,10 +3302,10 @@ static void unknown_nmi_error(unsigned char reason) case 'i': /* 'ignore' */ break; default: /* 'fatal' */ + console_force_unlock(); printk("Uhhuh. NMI received for unknown reason %02x.\n", reason); - printk("Dazed and confused, but trying to continue\n"); printk("Do you have a strange power saving mode enabled?\n"); - kexec_crash(); + fatal_trap(TRAP_nmi, regs); } } @@ -3338,7 +3338,7 @@ void do_nmi(struct cpu_user_regs *regs) else if ( reason & 0x40 ) io_check_error(regs); else if ( !nmi_watchdog ) - unknown_nmi_error((unsigned char)(reason&0xff)); + unknown_nmi_error(regs, (unsigned char)(reason&0xff)); } } |