diff options
author | Adhemerval Zanella <adhemerval.zanella@linaro.org> | 2017-01-13 15:24:53 -0200 |
---|---|---|
committer | Adhemerval Zanella <adhemerval.zanella@linaro.org> | 2017-02-06 10:21:55 -0200 |
commit | 841a67a0ade3ea9e3d10c044524a199fa608cc7e (patch) | |
tree | 7521fd6869a9b3c167d0a4d9db272f8a09171301 /sysdeps/unix/sysv/linux/posix_fadvise.c | |
parent | 58557c229319a3b8d2eefdb62e7df95089eabe37 (diff) | |
download | glibc-841a67a0ade3ea9e3d10c044524a199fa608cc7e.tar.gz glibc-841a67a0ade3ea9e3d10c044524a199fa608cc7e.tar.xz glibc-841a67a0ade3ea9e3d10c044524a199fa608cc7e.zip |
Consolidate arm and mips posix_fadvise implementations
As noted by c1f0601389db64d9, previous posix_fadvise consolidation broke on mips o32. As stated in commit message, MIPS o32 only defines __NR_fadvise64 and it is behaves like __NR_fadvise64_64. This patches consolidates both ARM and mips o32 version by fixing the ARM used option (__NR_fadvise64_64 withouth the alignment required by abi) and added another option, __ASSUME_FADVISE64_AS_64_64, which is used on mips o32. When this option is used, posix_fadvise will use __NR_fadvise64_64 behavior (by defining or not __ASSUME_FADVISE64_64_6ARG). For mips, if __NR_fadvise64_64 is not defined, __NR_fadvise will be used. I also updated the posix_fadvise comments to explain better the different kernel abi used in the supported architectures. I checked with a mips o32 and verified that posix_fadvise.o is indeed using 7 argument syscall with the expected argument position. I also checked on i686-linux-gnu and arm-gnu-eabihf. * sysdeps/unix/sysv/linux/arm/posix_fadvise.c: Remove file. * sysdeps/unix/sysv/linux/mips/mips32/posix_fadvise.c: Likewise. * sysdeps/unix/sysv/linux/mips/kernel-features.h (__ASSUME_FADVISE64_AS_64_64): Define. * sysdeps/unix/sysv/linux/posix_fadvise.c [__NR_fadvise64]: Add !defined __ASSUME_FADVISE64_AS_64_64 to use syscall issue. [!__NR_fadvise64 && __ASSUME_FADVISE64_64_6ARG]: Remove __ALIGNMENT_ARG usage. [!__NR_fadvise64 && !__ASSUME_FADVISE64_64_6ARG]: Define __NR_fadvise64_64 if it is not defined.
Diffstat (limited to 'sysdeps/unix/sysv/linux/posix_fadvise.c')
-rw-r--r-- | sysdeps/unix/sysv/linux/posix_fadvise.c | 23 |
1 files changed, 15 insertions, 8 deletions
diff --git a/sysdeps/unix/sysv/linux/posix_fadvise.c b/sysdeps/unix/sysv/linux/posix_fadvise.c index 15d72d4c03..222ac978e2 100644 --- a/sysdeps/unix/sysv/linux/posix_fadvise.c +++ b/sysdeps/unix/sysv/linux/posix_fadvise.c @@ -24,12 +24,16 @@ #ifndef __OFF_T_MATCHES_OFF64_T -/* Both arm and powerpc implements fadvise64_64 with last 'advise' argument - just after 'fd' to avoid the requirement of implementing 7-arg syscalls. - ARM also defines __NR_fadvise64_64 as __NR_arm_fadvise64_64. +/* Default implementation will use __NR_fadvise64 with expected argument + positions (for instance i386 and powerpc32 that uses __ALIGNMENT_ARG). - tile requires __ASSUME_ALIGNED_REGISTER_PAIRS but implements the 32-bit - fadvise64_64 without the padding 0 after fd. + Second option will be used by arm which define __NR_arm_fadvise64_64 + (redefined to __NR_fadvise64_64 in kernel-features.h) that behaves as + __NR_fadvise64_64 (without the aligment argument required for the ABI). + + Third option will be used by both tile 32-bits and mips o32. Tile + will set __ASSUME_FADVISE64_64_NO_ALIGN to issue a 6 argument syscall, + while mips will use a 7 argument one with __NR_fadvise64. s390 implements fadvice64_64 using a specific struct with arguments packed inside. This is the only implementation handled in arch-specific @@ -39,15 +43,14 @@ int posix_fadvise (int fd, off_t offset, off_t len, int advise) { INTERNAL_SYSCALL_DECL (err); -# ifdef __NR_fadvise64 +# if defined (__NR_fadvise64) && !defined (__ASSUME_FADVISE64_AS_64_64) int ret = INTERNAL_SYSCALL_CALL (fadvise64, err, fd, __ALIGNMENT_ARG SYSCALL_LL (offset), len, advise); # else # ifdef __ASSUME_FADVISE64_64_6ARG int ret = INTERNAL_SYSCALL_CALL (fadvise64_64, err, fd, advise, - __ALIGNMENT_ARG SYSCALL_LL (offset), - SYSCALL_LL (len)); + SYSCALL_LL (offset), SYSCALL_LL (len)); # else # ifdef __ASSUME_FADVISE64_64_NO_ALIGN @@ -55,6 +58,10 @@ posix_fadvise (int fd, off_t offset, off_t len, int advise) # define __ALIGNMENT_ARG # endif +# ifndef __NR_fadvise64_64 +# define __NR_fadvise64_64 __NR_fadvise64 +# endif + int ret = INTERNAL_SYSCALL_CALL (fadvise64_64, err, fd, __ALIGNMENT_ARG SYSCALL_LL (offset), SYSCALL_LL (len), advise); |