Browse Source

sparc: no need to sign-extend in sync_file_range() wrapper

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>
Al Viro 12 years ago
parent
commit
2ae80c43d4
2 changed files with 1 additions and 2 deletions
  1. 0 1
      arch/sparc/kernel/sys32.S
  2. 1 1
      arch/sparc/kernel/sys_sparc32.c

+ 0 - 1
arch/sparc/kernel/sys32.S

@@ -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:

+ 1 - 1
arch/sparc/kernel/sys_sparc32.c

@@ -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,