diff options
author | Joseph Myers <joseph@codesourcery.com> | 2014-06-25 11:36:10 +0000 |
---|---|---|
committer | Joseph Myers <joseph@codesourcery.com> | 2014-06-25 11:36:10 +0000 |
commit | cecf2ed43a4c2eede0b58740f16e47f31e7586a4 (patch) | |
tree | b15e9cdb4bff094dae290c581c0cf9750f106e9c | |
parent | a638de828d877376fb238efc8b0bc9220dd2601e (diff) | |
download | glibc-cecf2ed43a4c2eede0b58740f16e47f31e7586a4.tar glibc-cecf2ed43a4c2eede0b58740f16e47f31e7586a4.tar.gz glibc-cecf2ed43a4c2eede0b58740f16e47f31e7586a4.tar.bz2 glibc-cecf2ed43a4c2eede0b58740f16e47f31e7586a4.zip |
Remove __ASSUME_UTIMENSAT.
This patch removes the __ASSUME_UTIMENSAT macro, now it can be
unconditionally assumed to be true.
This shows that the only live uses of __ASSUME_UTIMES are in utimes.c
and they are only live for hppa. I intend a followup patch to make
__ASSUME_UTIMES into an hppa-specific macro (not used or defined
outside sysdeps/unix/sysv/linux/hppa/).
Tested x86_64 that the disassembly of installed shared libraries is
unchanged by this patch.
* sysdeps/unix/sysv/linux/kernel-features.h (__ASSUME_UTIMENSAT):
Remove macro.
* sysdeps/unix/sysv/linux/futimes.c: Do not include
<kernel-features.h>.
[__NR_utimensat && !__ASSUME_UTIMENSAT] (miss_utimensat): Remove
conditional variable definition.
(__futimes): Update comment.
(__futimes) [__ASSUME_UTIMENSAT]: Make code unconditional.
(__futimes) [!__ASSUME_UTIMENSAT]: Remove conditional code.
-rw-r--r-- | ChangeLog | 10 | ||||
-rw-r--r-- | sysdeps/unix/sysv/linux/futimes.c | 85 | ||||
-rw-r--r-- | sysdeps/unix/sysv/linux/kernel-features.h | 4 |
3 files changed, 11 insertions, 88 deletions
@@ -1,5 +1,15 @@ 2014-06-25 Joseph Myers <joseph@codesourcery.com> + * sysdeps/unix/sysv/linux/kernel-features.h (__ASSUME_UTIMENSAT): + Remove macro. + * sysdeps/unix/sysv/linux/futimes.c: Do not include + <kernel-features.h>. + [__NR_utimensat && !__ASSUME_UTIMENSAT] (miss_utimensat): Remove + conditional variable definition. + (__futimes): Update comment. + (__futimes) [__ASSUME_UTIMENSAT]: Make code unconditional. + (__futimes) [!__ASSUME_UTIMENSAT]: Remove conditional code. + [BZ #16560] * sysdeps/ieee754/dbl-64/e_exp10.c (__ieee754_exp10): Return 1 for arguments close to 0. diff --git a/sysdeps/unix/sysv/linux/futimes.c b/sysdeps/unix/sysv/linux/futimes.c index 1b1f005073..522dfe7a70 100644 --- a/sysdeps/unix/sysv/linux/futimes.c +++ b/sysdeps/unix/sysv/linux/futimes.c @@ -25,19 +25,12 @@ #include <_itoa.h> #include <fcntl.h> -#include <kernel-features.h> - - -#if defined __NR_utimensat && !defined __ASSUME_UTIMENSAT -static int miss_utimensat; -#endif /* Change the access time of the file associated with FD to TVP[0] and the modification time of FILE to TVP[1]. Starting with 2.6.22 the Linux kernel has the utimensat syscall which - can be used to implement futimes. Earlier kernels have no futimes() - syscall so we use the /proc filesystem. */ + can be used to implement futimes. */ int __futimes (int fd, const struct timeval tvp[2]) { @@ -56,82 +49,6 @@ __futimes (int fd, const struct timeval tvp[2]) TIMEVAL_TO_TIMESPEC (&tvp[1], &ts[1]); } -#ifdef __ASSUME_UTIMENSAT return INLINE_SYSCALL (utimensat, 4, fd, NULL, tvp ? &ts : NULL, 0); -#else - int result; -# ifdef __NR_utimensat - if (!__builtin_expect (miss_utimensat, 0)) - { - result = INLINE_SYSCALL (utimensat, 4, fd, NULL, tvp ? &ts : NULL, 0); - if (__builtin_expect (result, 0) != -1 || errno != ENOSYS) - return result; - - miss_utimensat = 1; - } -# endif - - static const char selffd[] = "/proc/self/fd/"; - char fname[sizeof (selffd) + 3 * sizeof (int)]; - fname[sizeof (fname) - 1] = '\0'; - char *cp = _itoa_word ((unsigned int) fd, fname + sizeof (fname) - 1, 10, 0); - cp = memcpy (cp - sizeof (selffd) + 1, selffd, sizeof (selffd) - 1); - -# ifdef __NR_utimes - result = INLINE_SYSCALL (utimes, 2, cp, tvp); -# ifndef __ASSUME_UTIMES - if (result == -1 && errno == ENOSYS) -# endif -# endif - { - /* The utimes() syscall does not exist or is not available in the - used kernel. Use utime(). For this we have to convert to the - data format utime() expects. */ -# ifndef __ASSUME_UTIMES - struct utimbuf buf; - struct utimbuf *times; - - if (tvp != NULL) - { - times = &buf; - buf.actime = tvp[0].tv_sec; - buf.modtime = tvp[1].tv_sec; - } - else - times = NULL; - - result = INLINE_SYSCALL (utime, 2, cp, times); -# endif - } - - if (result == -1) - /* Check for errors that result from failing to find /proc. - This means we can't do futimes at all, so return ENOSYS - rather than some confusing error. */ - switch (errno) - { - case EACCES: - if (tvp == NULL) /* Could be a path problem or a file problem. */ - break; - /*FALLTHROUGH*/ - case ELOOP: - case ENAMETOOLONG: - case ENOTDIR: - __set_errno (ENOSYS); - break; - - case ENOENT: - /* Validate the file descriptor by letting fcntl set errno to - EBADF if it's bogus. Otherwise it's a /proc issue. */ -# if !defined __NR_fcntl && defined __NR_fcntl64 -# define __NR_fcntl __NR_fcntl64 -# endif - if (INLINE_SYSCALL (fcntl, 3, fd, F_GETFD, 0) != -1) - __set_errno (ENOSYS); - break; - } - - return result; -#endif } weak_alias (__futimes, futimes) diff --git a/sysdeps/unix/sysv/linux/kernel-features.h b/sysdeps/unix/sysv/linux/kernel-features.h index 015b1c75bc..3c8ea50f79 100644 --- a/sysdeps/unix/sysv/linux/kernel-features.h +++ b/sysdeps/unix/sysv/linux/kernel-features.h @@ -100,10 +100,6 @@ # define __ASSUME_FUTEX_LOCK_PI 1 #endif -/* Support for utimensat syscall was added in 2.6.22, on SH - only after 2.6.22-rc1. */ -#define __ASSUME_UTIMENSAT 1 - /* Support for private futexes was added in 2.6.22. */ #define __ASSUME_PRIVATE_FUTEX 1 |