Merge tag 'for-3.14' of git://openrisc.net/~jonas/linux

Pull OpenRISC updates from Jonas Bonn:
 "The interesting change here is a rework of the OpenRISC signal
  handling to make it more like other architectures in the hopes that
  this makes it easier for others to comment on and understand.  This
  rework fixes some real bugs, like the fact that syscall restart did
  not work reliably"

* tag 'for-3.14' of git://openrisc.net/~jonas/linux:
  openrisc: Use get_signal() signal_setup_done()
  openrisc: Rework signal handling

2 files changed, 149 insertions, 143 deletions

diff --git a/arch/openrisc/kernel/entry.S b/arch/openrisc/kernel/entry.S
index d8a455ede5a7..fec8bf97d806 100644
--- a/arch/openrisc/kernel/entry.S
+++ b/arch/openrisc/kernel/entry.S
@@ -853,37 +853,44 @@ UNHANDLED_EXCEPTION(_vector_0x1f00,0x1f00)
 
 /* ========================================================[ return ] === */
 
+_resume_userspace:
+        DISABLE_INTERRUPTS(r3,r4)
+        l.lwz   r4,TI_FLAGS(r10)
+        l.andi  r13,r4,_TIF_WORK_MASK
+        l.sfeqi r13,0
+        l.bf    _restore_all
+         l.nop
+
 _work_pending:
-        /*
-         * if (current_thread_info->flags & _TIF_NEED_RESCHED)
-         *     schedule();
-         */
-        l.lwz   r5,TI_FLAGS(r10)
-        l.andi  r3,r5,_TIF_NEED_RESCHED
-        l.sfnei r3,0
-        l.bnf   _work_notifysig
+        l.lwz   r5,PT_ORIG_GPR11(r1)
+        l.sfltsi r5,0
+        l.bnf   1f
          l.nop
-        l.jal   schedule
+        l.andi  r5,r5,0
+1:
+        l.jal   do_work_pending
+         l.ori  r3,r1,0                 /* pt_regs */
+
+        l.sfeqi r11,0
+        l.bf    _restore_all
          l.nop
-        l.j     _resume_userspace
+        l.sfltsi r11,0
+        l.bnf   1f
          l.nop
-
-/* Handle pending signals and notify-resume requests.
- * do_notify_resume must be passed the latest pushed pt_regs, not
- * necessarily the "userspace" ones.  Also, pt_regs->syscallno
- * must be set so that the syscall restart functionality works.
- */
-_work_notifysig:
-        l.jal   do_notify_resume
-         l.ori  r3,r1,0           /* pt_regs */
-
-_resume_userspace:
-        DISABLE_INTERRUPTS(r3,r4)
-        l.lwz   r3,TI_FLAGS(r10)
-        l.andi  r3,r3,_TIF_WORK_MASK
-        l.sfnei r3,0
-        l.bf    _work_pending
+        l.and   r11,r11,r0
+        l.ori   r11,r11,__NR_restart_syscall
+        l.j     _syscall_check_trace_enter
          l.nop
+1:
+        l.lwz   r11,PT_ORIG_GPR11(r1)
+        /* Restore arg registers */
+        l.lwz   r3,PT_GPR3(r1)
+        l.lwz   r4,PT_GPR4(r1)
+        l.lwz   r5,PT_GPR5(r1)
+        l.lwz   r6,PT_GPR6(r1)
+        l.lwz   r7,PT_GPR7(r1)
+        l.j     _syscall_check_trace_enter
+         l.lwz  r8,PT_GPR8(r1)
 
 _restore_all:
         RESTORE_ALL
diff --git a/arch/openrisc/kernel/signal.c b/arch/openrisc/kernel/signal.c
index ae167f7e081a..66775bc07a8e 100644
--- a/arch/openrisc/kernel/signal.c
+++ b/arch/openrisc/kernel/signal.c
@@ -28,24 +28,24 @@
 #include <linux/tracehook.h>
 
 #include <asm/processor.h>
+#include <asm/syscall.h>
 #include <asm/ucontext.h>
 #include <asm/uaccess.h>
 
 #define DEBUG_SIG 0
 
 struct rt_sigframe {
-        struct siginfo *pinfo;
-        void *puc;
         struct siginfo info;
         struct ucontext uc;
         unsigned char retcode[16];      /* trampoline code */
 };
 
-static int restore_sigcontext(struct pt_regs *regs, struct sigcontext *sc)
+static int restore_sigcontext(struct pt_regs *regs,
+                              struct sigcontext __user *sc)
 {
-        unsigned int err = 0;
+        int err = 0;
 
-        /* Alwys make any pending restarted system call return -EINTR */
+        /* Always make any pending restarted system calls return -EINTR */
         current_thread_info()->restart_block.fn = do_no_restart_syscall;
 
         /*
@@ -53,25 +53,21 @@ static int restore_sigcontext(struct pt_regs *regs, struct sigcontext *sc)
          * (sc is already checked for VERIFY_READ since the sigframe was
          *  checked in sys_sigreturn previously)
          */
-        if (__copy_from_user(regs, sc->regs.gpr, 32 * sizeof(unsigned long)))
-                goto badframe;
-        if (__copy_from_user(&regs->pc, &sc->regs.pc, sizeof(unsigned long)))
-                goto badframe;
-        if (__copy_from_user(&regs->sr, &sc->regs.sr, sizeof(unsigned long)))
-                goto badframe;
+        err |= __copy_from_user(regs, sc->regs.gpr, 32 * sizeof(unsigned long));
+        err |= __copy_from_user(&regs->pc, &sc->regs.pc, sizeof(unsigned long));
+        err |= __copy_from_user(&regs->sr, &sc->regs.sr, sizeof(unsigned long));
 
         /* make sure the SM-bit is cleared so user-mode cannot fool us */
         regs->sr &= ~SPR_SR_SM;
 
+        regs->orig_gpr11 = -1;  /* Avoid syscall restart checks */
+
         /* TODO: the other ports use regs->orig_XX to disable syscall checks
          * after this completes, but we don't use that mechanism. maybe we can
          * use it now ?
          */
 
         return err;
-
-badframe:
-        return 1;
 }
 
 asmlinkage long _sys_rt_sigreturn(struct pt_regs *regs)
@@ -111,21 +107,18 @@ badframe:
  * Set up a signal frame.
  */
 
-static int setup_sigcontext(struct sigcontext *sc, struct pt_regs *regs,
-                            unsigned long mask)
+static int setup_sigcontext(struct pt_regs *regs, struct sigcontext __user *sc)
 {
         int err = 0;
 
         /* copy the regs */
-
+        /* There should be no need to save callee-saved registers here...
+         * ...but we save them anyway.  Revisit this
+         */
         err |= __copy_to_user(sc->regs.gpr, regs, 32 * sizeof(unsigned long));
         err |= __copy_to_user(&sc->regs.pc, &regs->pc, sizeof(unsigned long));
         err |= __copy_to_user(&sc->regs.sr, &regs->sr, sizeof(unsigned long));
 
-        /* then some other stuff */
-
-        err |= __put_user(mask, &sc->oldmask);
-
         return err;
 }
 
@@ -173,55 +166,53 @@ static inline void __user *get_sigframe(struct k_sigaction *ka,
  * trampoline which performs the syscall sigreturn, or a provided
  * user-mode trampoline.
  */
-static int setup_rt_frame(int sig, struct k_sigaction *ka, siginfo_t *info,
+static int setup_rt_frame(struct ksignal *ksig, sig