powerpc/xmon: Fix disabling tracing while in xmon
authorNaveen N. Rao <naveen.n.rao@linux.vnet.ibm.com>
Thu, 27 Jun 2019 09:59:40 +0000 (15:29 +0530)
committerMichael Ellerman <mpe@ellerman.id.au>
Mon, 1 Jul 2019 06:26:55 +0000 (16:26 +1000)
Commit ed49f7fd6438d ("powerpc/xmon: Disable tracing when entering
xmon") added code to disable recording trace entries while in xmon. The
commit introduced a variable 'tracing_enabled' to record if tracing was
enabled on xmon entry, and used this to conditionally enable tracing
during exit from xmon.

However, we are not checking the value of 'fromipi' variable in
xmon_core() when setting 'tracing_enabled'. Due to this, when secondary
cpus enter xmon, they will see tracing as being disabled already and
tracing won't be re-enabled on exit. Fix the same.

Fixes: ed49f7fd6438d ("powerpc/xmon: Disable tracing when entering xmon")
Signed-off-by: Naveen N. Rao <naveen.n.rao@linux.vnet.ibm.com>
Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
arch/powerpc/xmon/xmon.c

index 1b0149b2bb6c9236493a3c0eb2260fc6aeba6c8b..f1c4e1601b9d3000303bef12f6dee65d09f7fd43 100644 (file)
@@ -469,8 +469,10 @@ static int xmon_core(struct pt_regs *regs, int fromipi)
        local_irq_save(flags);
        hard_irq_disable();
 
-       tracing_enabled = tracing_is_on();
-       tracing_off();
+       if (!fromipi) {
+               tracing_enabled = tracing_is_on();
+               tracing_off();
+       }
 
        bp = in_breakpoint_table(regs->nip, &offset);
        if (bp != NULL) {