From 238d60ac9b0fb8262b18a44a23d14de4b59f2f23 Mon Sep 17 00:00:00 2001 From: Joseph Myers Date: Mon, 21 Mar 2016 16:30:05 +0000 Subject: Remove __ASSUME_SIGNALFD4. Current Linux kernel version requirements mean the signalfd4 syscall can always be assumed to be available. This patch removes __ASSUME_SIGNALFD4 and associated conditionals. Tested for x86_64 and x86 that installed stripped shared libraries are unchanged by the patch. * sysdeps/unix/sysv/linux/kernel-features.h (__ASSUME_SIGNALFD4): Remove macro. * sysdeps/unix/sysv/linux/signalfd.c: Do not include . (signalfd) [__NR_signalfd4]: Make code unconditional. (signalfd) [!__ASSUME_SIGNALFD4]: Remove conditional code. --- ChangeLog | 9 +++++++++ sysdeps/unix/sysv/linux/kernel-features.h | 1 - sysdeps/unix/sysv/linux/signalfd.c | 26 +------------------------- 3 files changed, 10 insertions(+), 26 deletions(-) diff --git a/ChangeLog b/ChangeLog index ffaf973d14..52ec51044e 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,12 @@ +2016-03-21 Joseph Myers + + * sysdeps/unix/sysv/linux/kernel-features.h (__ASSUME_SIGNALFD4): + Remove macro. + * sysdeps/unix/sysv/linux/signalfd.c: Do not include + . + (signalfd) [__NR_signalfd4]: Make code unconditional. + (signalfd) [!__ASSUME_SIGNALFD4]: Remove conditional code. + 2016-03-21 Adhemerval Zanella * sysdeps/unix/sysv/linux/spawni.c (__spawnix): Fix implict checks diff --git a/sysdeps/unix/sysv/linux/kernel-features.h b/sysdeps/unix/sysv/linux/kernel-features.h index 34a7bd23a2..28195e5629 100644 --- a/sysdeps/unix/sysv/linux/kernel-features.h +++ b/sysdeps/unix/sysv/linux/kernel-features.h @@ -83,7 +83,6 @@ 2.6.27. */ #define __ASSUME_IN_NONBLOCK 1 #define __ASSUME_PIPE2 1 -#define __ASSUME_SIGNALFD4 1 #define __ASSUME_DUP3 1 /* Support for accept4 functionality was added in 2.6.28, but for some diff --git a/sysdeps/unix/sysv/linux/signalfd.c b/sysdeps/unix/sysv/linux/signalfd.c index ba64dad801..99ecf6e0d7 100644 --- a/sysdeps/unix/sysv/linux/signalfd.c +++ b/sysdeps/unix/sysv/linux/signalfd.c @@ -19,34 +19,10 @@ #include #include #include -#include int signalfd (int fd, const sigset_t *mask, int flags) { -#ifdef __NR_signalfd4 - int res = INLINE_SYSCALL (signalfd4, 4, fd, mask, _NSIG / 8, flags); -# ifndef __ASSUME_SIGNALFD4 - if (res != -1 || errno != ENOSYS) -# endif - return res; -#endif - -#ifndef __ASSUME_SIGNALFD4 - /* The old system call has no flag parameter which is bad. So we have - to wait until we have to support to pass additional values to the - kernel (sys_indirect) before implementing setting flags like - O_NONBLOCK etc. */ - if (flags != 0) - return INLINE_SYSCALL_ERROR_RETURN_VALUE (EINVAL); - -# ifdef __NR_signalfd - return INLINE_SYSCALL (signalfd, 3, fd, mask, _NSIG / 8); -# else - return INLINE_SYSCALL_ERROR_RETURN_VALUE (ENOSYS); -# endif -#elif !defined __NR_signalfd4 -# error "__ASSUME_SIGNALFD4 defined but not __NR_signalfd4" -#endif + return INLINE_SYSCALL (signalfd4, 4, fd, mask, _NSIG / 8, flags); } -- cgit 1.4.1