On Mon, Jun 22, 2020 at 08:35:16PM -0700, David Miller wrote: > From: Christian Brauner <christian.brauner@xxxxxxxxxx> > 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