alpha: get rid of pointless insn in ret_from_kernel_thread
authorAl Viro <viro@zeniv.linux.org.uk>
Mon, 19 Mar 2018 18:29:17 +0000 (14:29 -0400)
committerAl Viro <viro@zeniv.linux.org.uk>
Wed, 28 Mar 2018 12:58:27 +0000 (08:58 -0400)
It used to clear a3, so that signal handling on
return to userland would've passed zero r0 to do_work_pending(),
preventing the syscall restart logics from triggering.

It had been pointless all along, since we only go there
after successful do_execve().  Which does clear regs->r0 on alpha,
preventing the syscall restart logics just fine, no extra help
needed.  Good thing, that, since back in 2012 do_work_pending()
has lost the second argument, shifting the registers used to pass
that thing from a3 to a2.  Commit that had done that adjusted the
entry.S code accordingly, but missed that one.

As the result, we were left with useless insn in
ret_from_kernel_thread and confusing comment to go with it.
Get rid of both...

Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
arch/alpha/kernel/entry.S

index d92abb01c24951915583a0d281b5e394e5de935b..c64806a2daf549fe012aef2848a77ae57982063f 100644 (file)
@@ -785,7 +785,6 @@ ret_from_kernel_thread:
        mov     $9, $27
        mov     $10, $16
        jsr     $26, ($9)
-       mov     $31, $19                /* to disable syscall restarts */
        br      $31, ret_to_user
 .end ret_from_kernel_thread