powerpc: Disable interrupts early in Program Check
authorBenjamin Herrenschmidt <benh@kernel.crashing.org>
Mon, 13 Feb 2012 20:42:18 +0000 (20:42 +0000)
committerBenjamin Herrenschmidt <benh@kernel.crashing.org>
Thu, 16 Feb 2012 05:15:10 +0000 (16:15 +1100)
Program Check exceptions are the result of WARNs, BUGs, some
type of breakpoints, kprobe, and other illegal instructions.

We want interrupts (and thus preemption) to remain disabled
while doing the initial stage of testing the reason and
branching off to a debugger or kprobe, so we are still on
the original CPU which makes debugging easier in various cases.

This is how the code was intended, hence the local_irq_enable()
right in the middle of program_check_exception().

However, the assembly exception prologue for that exception was
incorrectly marked as enabling interrupts, which defeats that
(and records a redundant enable with lockdep).

Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
arch/powerpc/kernel/exceptions-64s.S

index d4be7bb3dbdfdffd5d62d76aa96860392040a305..3844ca7c509924b82769703c2167278958d136cc 100644 (file)
@@ -775,7 +775,7 @@ program_check_common:
        EXCEPTION_PROLOG_COMMON(0x700, PACA_EXGEN)
        bl      .save_nvgprs
        addi    r3,r1,STACK_FRAME_OVERHEAD
-       ENABLE_INTS
+       DISABLE_INTS
        bl      .program_check_exception
        b       .ret_from_except