Re: [-next Nov 17] s390 build break(arch/s390/kernel/compat_wrapper.S)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, 2009-11-17 at 16:50 +0100, Heiko Carstens wrote:
> On Tue, Nov 17, 2009 at 10:23:56AM -0500, Eric Paris wrote:
> > On Tue, 2009-11-17 at 14:55 +0100, Heiko Carstens wrote:
> > 
> > > Yes, also some places should have used lgfr instead of llgfr for proper sign
> > > extension. But please, just drop the s390 bits from your patch.
> > > Its easier and less painful for us to do it ourselves instead of reviewing
> > > and fixing these things. (No offence intended!).
> > 
> > dropped and won't show up in -next tomorrow.
> > 
> > This what I thought it should be and would love to read if you say it's
> > right....
> 
> Yes, it is except for one thing:
> 
> > diff --git a/arch/s390/include/asm/unistd.h b/arch/s390/include/asm/unistd.h
> > index cb5232d..8962161 100644
> > --- a/arch/s390/include/asm/unistd.h
> > +++ b/arch/s390/include/asm/unistd.h
> > @@ -269,7 +269,9 @@
> >  #define	__NR_pwritev		329
> >  #define __NR_rt_tgsigqueueinfo	330
> >  #define __NR_perf_event_open	331
> > -#define NR_syscalls 332
> > +#define __NR_fanotify_init	332
> > +#define __NR_fanotify_mark	333
> > +#define NR_syscalls 334
> > 
> >  /* 
> >   * There are some system calls that are not present on 64 bit, some
> > diff --git a/arch/s390/kernel/compat_wrapper.S b/arch/s390/kernel/compat_wrapper.S
> > index cbd9901..c81723d 100644
> > --- a/arch/s390/kernel/compat_wrapper.S
> > +++ b/arch/s390/kernel/compat_wrapper.S
> > @@ -1855,3 +1855,20 @@ sys32_execve_wrapper:
> >  	llgtr	%r3,%r3			# compat_uptr_t *
> >  	llgtr	%r4,%r4			# compat_uptr_t *
> >  	jg	sys32_execve		# branch to system call
> > +
> > +	.globl	sys_fanotify_init_wrapper
> > +sys_fanotify_init_wrapper:
> > +	llgfr	%r2,%r2			# unsigned int
> > +	llgfr	%r3,%r3			# unsigned int
> > +	llgfr	%r4,%r4			# unsigned int
> > +	jg	sys_fanotify_init	# branch to system call
> 
> The third argument an int and hence it should have been lgfr ;)

It should have been an unsigned int, just noticed it today (because of
this discussion) and fixed it.  You'll see in a patch which does so in
-next tomorrow.

I'll let this settle a bit and come back to linux-s390 in a week or two
to ask you guys to write/review some hooks!

-Eric

--
To unsubscribe from this list: send the line "unsubscribe linux-next" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Kernel]     [Linux USB Development]     [Yosemite News]     [Linux SCSI]

  Powered by Linux