This is the mail archive of the libc-hacker@sourceware.org mailing list for the glibc project.
Note that libc-hacker is a closed list. You may look at the archives of this list, but subscription and posting are not open.
Index Nav: | [Date Index] [Subject Index] [Author Index] [Thread Index] | |
---|---|---|
Message Nav: | [Date Prev] [Date Next] | [Thread Prev] [Thread Next] |
Other format: | [Raw text] |
Hi! I've noticed that sync_file_range is a stub on ppc/ppc64. The kernel on these arches provides sync_file_range2 syscall with swapped parameters. The following completely untested patch ought to fix this. BTW, on all arches, the glibc prototypes for sync_file_range (as well as sync_file_range.c) use from and to argument names, while kernel uses offset and nbytes. As the kernel uses endbyte = offset + nbytes, I'd say naming the arguments this way is very confusing. As this is a second busted syscall wrapper for ppc/ppc64 discovered in the last fortnight, maybe it would be best if somebody went through all the recently added syscalls (last 6 years or so) and verified the wrappers on ppc/ppc64 match the kernel ABI. 2009-11-16 Jakub Jelinek <jakub@redhat.com> * sysdeps/unix/sysv/linux/sync_file_range.c (sync_file_range): Implement using sync_file_range2 syscall if __NR_sync_file_range2 is defined. * sysdeps/unix/sysv/linux/powerpc/powerpc64/sync_file_range.c: New file. --- libc/sysdeps/unix/sysv/linux/sync_file_range.c.jj 2009-05-16 19:23:46.000000000 +0200 +++ libc/sysdeps/unix/sysv/linux/sync_file_range.c 2009-11-16 10:11:22.000000000 +0100 @@ -1,5 +1,5 @@ /* Selective file content synch'ing. - Copyright (C) 2006, 2007 Free Software Foundation, Inc. + Copyright (C) 2006, 2007, 2009 Free Software Foundation, Inc. This file is part of the GNU C Library. The GNU C Library is free software; you can redistribute it and/or @@ -34,6 +34,14 @@ sync_file_range (int fd, __off64_t from, __LONG_LONG_PAIR ((long) (to >> 32), (long) to), flags); } +#elif defined __NR_sync_file_range2 +int +sync_file_range (int fd, __off64_t from, __off64_t to, unsigned int flags) +{ + return INLINE_SYSCALL (sync_file_range2, 6, fd, flags, + __LONG_LONG_PAIR ((long) (from >> 32), (long) from), + __LONG_LONG_PAIR ((long) (to >> 32), (long) to)); +} #else int sync_file_range (int fd, __off64_t from, __off64_t to, unsigned int flags) --- libc/sysdeps/unix/sysv/linux/powerpc/powerpc64/sync_file_range.c.jj 2009-11-16 10:15:24.000000000 +0100 +++ libc/sysdeps/unix/sysv/linux/powerpc/powerpc64/sync_file_range.c 2009-11-16 10:16:04.000000000 +0100 @@ -0,0 +1,44 @@ +/* Selective file content synch'ing. + Copyright (C) 2006, 2007, 2009 Free Software Foundation, Inc. + This file is part of the GNU C Library. + + The GNU C Library is free software; you can redistribute it and/or + modify it under the terms of the GNU Lesser General Public + License as published by the Free Software Foundation; either + version 2.1 of the License, or (at your option) any later version. + + The GNU C Library is distributed in the hope that it will be useful, + but WITHOUT ANY WARRANTY; without even the implied warranty of + MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU + Lesser General Public License for more details. + + You should have received a copy of the GNU Lesser General Public + License along with the GNU C Library; if not, write to the Free + Software Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA + 02111-1307 USA. */ + +#include <errno.h> +#include <fcntl.h> +#include <sys/types.h> + +#include <sysdep.h> +#include <sys/syscall.h> + + +#if defined __NR_sync_file_range2 +int +sync_file_range (int fd, __off64_t from, __off64_t to, unsigned int flags) +{ + return INLINE_SYSCALL (sync_file_range2, 4, fd, flags, from, to); +} +#else +int +sync_file_range (int fd, __off64_t from, __off64_t to, unsigned int flags) +{ + __set_errno (ENOSYS); + return -1; +} +stub_warning (sync_file_range) + +# include <stub-tag.h> +#endif Jakub
Index Nav: | [Date Index] [Subject Index] [Author Index] [Thread Index] | |
---|---|---|
Message Nav: | [Date Prev] [Date Next] | [Thread Prev] [Thread Next] |