powerpc/64s: sreset panic if there is no debugger or crash dump handlers
authorNicholas Piggin <npiggin@gmail.com>
Mon, 26 Mar 2018 15:01:16 +0000 (01:01 +1000)
committerMichael Ellerman <mpe@ellerman.id.au>
Sat, 31 Mar 2018 13:47:46 +0000 (00:47 +1100)
system_reset_exception does most of its own crash handling now,
invoking the debugger or crash dumps if they are registered. If not,
then it goes through to die() to print stack traces, and then is
supposed to panic (according to comments).

However after die() prints oopses, it does its own handling which
doesn't allow system_reset_exception to panic (e.g., it may just
kill the current process). This patch causes sreset exceptions to
return from die after it prints messages but before acting.

This also stops die from invoking the debugger on 0x100 crashes.
system_reset_exception similarly calls the debugger. It had been
thought this was harmless (because if the debugger was disabled,
neither call would fire, and if it was enabled the first call
would return). However in some cases like xmon 'X' command, the
debugger returns 0, which currently causes it to be entered
again (first in system_reset_exception, then in die), which is
confusing.

Signed-off-by: Nicholas Piggin <npiggin@gmail.com>
Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
arch/powerpc/kernel/traps.c

index f200bfd98b17e6b4dcfcc0a863896cbe882954fe..2c1a1d24f0abb4f5e395d66fdb3fb483a5ef1b0b 100644 (file)
@@ -208,6 +208,12 @@ static void oops_end(unsigned long flags, struct pt_regs *regs,
        }
        raw_local_irq_restore(flags);
 
+       /*
+        * system_reset_excption handles debugger, crash dump, panic, for 0x100
+        */
+       if (TRAP(regs) == 0x100)
+               return;
+
        crash_fadump(regs, "die oops");
 
        if (kexec_should_crash(current))
@@ -272,8 +278,13 @@ void die(const char *str, struct pt_regs *regs, long err)
 {
        unsigned long flags;
 
-       if (debugger(regs))
-               return;
+       /*
+        * system_reset_excption handles debugger, crash dump, panic, for 0x100
+        */
+       if (TRAP(regs) != 0x100) {
+               if (debugger(regs))
+                       return;
+       }
 
        flags = oops_begin(regs);
        if (__die(str, regs, err))