x86_64, dumpstack: move kexec_crash from __die to oops_end
authorAlexander van Heukelum <heukelum@fastmail.fm>
Wed, 22 Oct 2008 10:00:10 +0000 (12:00 +0200)
committerIngo Molnar <mingo@elte.hu>
Wed, 22 Oct 2008 12:00:23 +0000 (14:00 +0200)
oops_end is preceded by either a call to __die, or a conditional
call to crash_kexec. Move the conditional call to crash_kexec
from the end of __die to the start of oops_end and remove
the superfluous call to crash_kexec in die_nmi.

Signed-off-by: Alexander van Heukelum <heukelum@fastmail.fm>
Acked-by: Neil Horman <nhorman@tuxdriver.com>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
arch/x86/kernel/dumpstack_64.c

index ffefea611ba38221b18d154e2c19f25dab729af2..57ce11b895cef66809ddd21e6d299adaddc9f11d 100644 (file)
@@ -458,6 +458,9 @@ unsigned __kprobes long oops_begin(void)
 
 void __kprobes oops_end(unsigned long flags, struct pt_regs *regs, int signr)
 {
+       if (regs && kexec_should_crash(current))
+               crash_kexec(regs);
+
        die_owner = -1;
        bust_spinlocks(0);
        die_nest_count--;
@@ -501,8 +504,6 @@ int __kprobes __die(const char *str, struct pt_regs *regs, long err)
        printk(KERN_ALERT "RIP ");
        printk_address(regs->ip, 1);
        printk(" RSP <%016lx>\n", regs->sp);
-       if (kexec_should_crash(current))
-               crash_kexec(regs);
        return 0;
 }
 
@@ -536,11 +537,9 @@ die_nmi(char *str, struct pt_regs *regs, int do_panic)
        printk(" on CPU%d, ip %08lx, registers:\n",
                smp_processor_id(), regs->ip);
        show_registers(regs);
-       if (kexec_should_crash(current))
-               crash_kexec(regs);
+       oops_end(flags, regs, 0);
        if (do_panic || panic_on_oops)
                panic("Non maskable interrupt");
-       oops_end(flags, regs, 0);
        nmi_exit();
        local_irq_enable();
        do_exit(SIGBUS);