diff options
author | Alexander van Heukelum <heukelum@fastmail.fm> | 2008-10-22 10:00:08 (GMT) |
---|---|---|
committer | Ingo Molnar <mingo@elte.hu> | 2008-10-22 12:00:22 (GMT) |
commit | b4b8f87bf4958cbad620654efc0882ac46c19846 (patch) | |
tree | ac94939d6bc8e75e212225da99ccbf8a48e7c30e /arch/x86/kernel/dumpstack_32.c | |
parent | cf52ebedba77ee494b495dedd3a1f55944611275 (diff) | |
download | linux-b4b8f87bf4958cbad620654efc0882ac46c19846.tar.xz |
i386, dumpstack: move crash_kexec before bust_spinlocks(0) in oops_end
crash_kexec should not be called with console_sem held. Move
the call before bust_spinlocks(0) in oops_end to avoid the
problem.
Signed-off-by: Alexander van Heukelum <heukelum@fastmail.fm>
Acked-by: "Neil Horman" <nhorman@tuxdriver.com>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Diffstat (limited to 'arch/x86/kernel/dumpstack_32.c')
-rw-r--r-- | arch/x86/kernel/dumpstack_32.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/arch/x86/kernel/dumpstack_32.c b/arch/x86/kernel/dumpstack_32.c index b361475..5493d31 100644 --- a/arch/x86/kernel/dumpstack_32.c +++ b/arch/x86/kernel/dumpstack_32.c @@ -309,6 +309,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); + bust_spinlocks(0); die_owner = -1; add_taint(TAINT_DIE); @@ -318,8 +321,6 @@ void __kprobes oops_end(unsigned long flags, struct pt_regs *regs, int signr) if (!regs) return; - if (kexec_should_crash(current)) - crash_kexec(regs); if (in_interrupt()) panic("Fatal exception in interrupt"); if (panic_on_oops) |