powerpc/tm: update comment about interrupt re-entrancy
authorNicholas Piggin <npiggin@gmail.com>
Fri, 28 Jun 2019 05:33:32 +0000 (15:33 +1000)
committerMichael Ellerman <mpe@ellerman.id.au>
Tue, 2 Jul 2019 11:39:49 +0000 (21:39 +1000)
Since the system reset interrupt began to use its own stack, and
machine check interrupts have done so for some time, r1 can be
changed without clearing MSR[RI], provided no other interrupts
(including SLB misses) are taken.

MSR[RI] does have to be cleared when using SCRATCH0, however.

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

index 9fabdce255cdbb6c286de7d05af98e72a6b64015..6ba0fdd1e7f81531d26d46c98b734afc1e89b776 100644 (file)
@@ -148,7 +148,7 @@ _GLOBAL(tm_reclaim)
        /* Stash the stack pointer away for use after reclaim */
        std     r1, PACAR1(r13)
 
-       /* Clear MSR RI since we are about to change r1, EE is already off. */
+       /* Clear MSR RI since we are about to use SCRATCH0, EE is already off */
        li      r5, 0
        mtmsrd  r5, 1
 
@@ -474,7 +474,7 @@ restore_gprs:
 
        REST_GPR(7, r7)
 
-       /* Clear MSR RI since we are about to change r1. EE is already off */
+       /* Clear MSR RI since we are about to use SCRATCH0. EE is already off */
        li      r5, 0
        mtmsrd  r5, 1