On Mon, Jun 22, 2020 at 08:35:16PM -0700, David Miller wrote:
> From: Christian Brauner <[email protected]>
> Date: Tue, 23 Jun 2020 01:43:11 +0200
> 
> > diff --git a/arch/sparc/kernel/syscalls.S b/arch/sparc/kernel/syscalls.S
> > index db42b4fb3708..192f3a28a2b7 100644
> > --- a/arch/sparc/kernel/syscalls.S
> > +++ b/arch/sparc/kernel/syscalls.S
> > @@ -86,19 +86,22 @@ sys32_rt_sigreturn:
> >      * during system calls...
> >      */
> >     .align  32
> > -sys_vfork: /* Under Linux, vfork and fork are just special cases of clone. 
> > */
> > -   sethi   %hi(0x4000 | 0x0100 | SIGCHLD), %o0
> > -   or      %o0, %lo(0x4000 | 0x0100 | SIGCHLD), %o0
> > -   ba,pt   %xcc, sys_clone
> > +sys_vfork:
> > +   flushw
> > +   ba,pt   %xcc, sparc_vfork
> > +   add     %sp, PTREGS_OFF, %o0
> 
> Please indent branch delay slot instructions with one extra space, as
> was done in the code you are changing.
> 
> > +   ba,pt   %xcc, sparc_fork
> > +   add     %sp, PTREGS_OFF, %o0
> 
> Likewise.
> 
> > +   ba,pt   %xcc, sparc_clone
> > +   add     %sp, PTREGS_OFF, %o0
> 
> Likewise.

Fixed, thanks for taking a look again! This somehow slipped past me when
applying. 

Christian

Reply via email to