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-s390" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html