From 841a67a0ade3ea9e3d10c044524a199fa608cc7e Mon Sep 17 00:00:00 2001 From: Adhemerval Zanella Date: Fri, 13 Jan 2017 15:24:53 -0200 Subject: 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. --- ChangeLog | 13 +++++++++++++ 1 file changed, 13 insertions(+) (limited to 'ChangeLog') diff --git a/ChangeLog b/ChangeLog index 6037fef737..d4888b4d74 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,16 @@ +2017-02-06 Adhemerval Zanella + + * 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. + 2017-02-05 Siddhesh Poyarekar * version.h (RELEASE): Set to "development" -- cgit v1.2.3