parisc: Fix comments and enable interrupts later
authorJohn David Anglin <dave.anglin@bell.net>
Fri, 2 Oct 2020 18:03:27 +0000 (20:03 +0200)
committerHelge Deller <deller@gmx.de>
Thu, 15 Oct 2020 06:10:38 +0000 (08:10 +0200)
Correct the comments: The jump is forwards, not backwards.
Enable the interrupts after %r29 (reference param area) was loaded.

Signed-off-by: John David Anglin <dave.anglin@bell.net>
Signed-off-by: Helge Deller <deller@gmx.de>
arch/parisc/kernel/entry.S

index 519f9056fd00b1e0afd7b14be828bb0e3f1bd1f4..f6f28e41bb5e082c0a014e034d01ab9e46ee64c9 100644 (file)
@@ -899,20 +899,20 @@ intr_check_sig:
         * Only do signals if we are returning to user space
         */
        LDREG   PT_IASQ0(%r16), %r20
-       cmpib,COND(=),n LINUX_GATEWAY_SPACE, %r20, intr_restore /* backward */
+       cmpib,COND(=),n LINUX_GATEWAY_SPACE, %r20, intr_restore /* forward */
        LDREG   PT_IASQ1(%r16), %r20
-       cmpib,COND(=),n LINUX_GATEWAY_SPACE, %r20, intr_restore /* backward */
-
-       /* NOTE: We need to enable interrupts if we have to deliver
-        * signals. We used to do this earlier but it caused kernel
-        * stack overflows. */
-       ssm     PSW_SM_I, %r0
+       cmpib,COND(=),n LINUX_GATEWAY_SPACE, %r20, intr_restore /* forward */
 
        copy    %r0, %r25                       /* long in_syscall = 0 */
 #ifdef CONFIG_64BIT
        ldo     -16(%r30),%r29                  /* Reference param save area */
 #endif
 
+       /* NOTE: We need to enable interrupts if we have to deliver
+        * signals. We used to do this earlier but it caused kernel
+        * stack overflows. */
+       ssm     PSW_SM_I, %r0
+
        BL      do_notify_resume,%r2
        copy    %r16, %r26                      /* struct pt_regs *regs */