sparc: no need to sign-extend in sync_file_range() wrapper
authorAl Viro <viro@zeniv.linux.org.uk>
Sat, 2 Mar 2013 15:38:22 +0000 (10:38 -0500)
committerAl Viro <viro@zeniv.linux.org.uk>
Mon, 4 Mar 2013 04:00:40 +0000 (23:00 -0500)
the first argument will be sign-extended by sys_sync_file_range()
SYSCALL_DEFINE-generate wrapper; the last argument is unsigned int,
so the same wrapper will will truncate it anyway.

Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
arch/sparc/kernel/sys32.S
arch/sparc/kernel/sys_sparc32.c

index 0dbc2d6..2e680b5 100644 (file)
@@ -48,7 +48,6 @@ SIGN3(sys32_futex, compat_sys_futex, %o1, %o2, %o5)
 SIGN1(sys32_recvfrom, compat_sys_recvfrom, %o0)
 SIGN1(sys32_recvmsg, compat_sys_recvmsg, %o0)
 SIGN1(sys32_sendmsg, compat_sys_sendmsg, %o0)
-SIGN2(sys32_sync_file_range, compat_sync_file_range, %o0, %o5)
 
        .globl          sys32_mmap2
 sys32_mmap2:
index d546188..3d0ddbc 100644 (file)
@@ -238,7 +238,7 @@ long compat_sys_fadvise64_64(int fd,
                                advice);
 }
 
-long compat_sync_file_range(int fd, unsigned long off_high, unsigned long off_low, unsigned long nb_high, unsigned long nb_low, int flags)
+long sys32_sync_file_range(unsigned int fd, unsigned long off_high, unsigned long off_low, unsigned long nb_high, unsigned long nb_low, unsigned int flags)
 {
        return sys_sync_file_range(fd,
                                   (off_high << 32) | off_low,