From 74385da564a92fc441a7d7f3ed7a4295b74bfdbf Mon Sep 17 00:00:00 2001 From: Joseph Myers Date: Wed, 25 Jun 2014 20:29:45 +0000 Subject: Remove __ASSUME_O_CLOEXEC / O_CLOEXEC conditionals in sysdeps/unix/sysv/linux/. This patch removes conditionals on __ASSUME_O_CLOEXEC, and on O_CLOEXEC being defined, in sysdeps/unix/sysv/linux/, now that O_CLOEXEC support can be unconditionally assumed. The patch is conservative in what it changes and further followup cleanups may be possible. It may be possible to remove dl-opendir.c, but the patch does not do so, just removing a redundant undefine and redefine of __ASSUME_O_CLOEXEC. Also, __ASSUME_O_CLOEXEC is defined unconditionally for Hurd as well as Linux. Thus, if we decide that O_CLOEXEC support is a required feature of any glibc port, we could remove __ASSUME_O_CLOEXEC and all conditionals on it throughout glibc, rather than just cleaning up sysdeps/unix/sysv/linux/. Tested x86_64 that the disassembly of installed shared libraries is unchanged by this patch. * sysdeps/unix/sysv/linux/dl-opendir.c (__ASSUME_O_CLOEXEC): Do not undefine and redefine. * sysdeps/unix/sysv/linux/getsysstats.c (__get_nprocs) [O_CLOEXEC]: Make code unconditional. (__get_nprocs) [!O_CLOEXEC]: Remove conditional code. * sysdeps/unix/sysv/linux/shm_open.c: Do not include . [O_CLOEXEC && !__ASSUME_O_CLOEXEC] (have_o_cloexec): Remove conditional variable definition. (shm_open) [O_CLOEXEC]: Make code unconditional. (shm_open) [!O_CLOEXEC || !__ASSUME_O_CLOEXEC]: Remove conditional code. --- ChangeLog | 13 +++++++++++++ 1 file changed, 13 insertions(+) (limited to 'ChangeLog') diff --git a/ChangeLog b/ChangeLog index 2727f7fe6d..a8350cc5f0 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,5 +1,18 @@ 2014-06-25 Joseph Myers + * sysdeps/unix/sysv/linux/dl-opendir.c (__ASSUME_O_CLOEXEC): Do + not undefine and redefine. + * sysdeps/unix/sysv/linux/getsysstats.c (__get_nprocs) + [O_CLOEXEC]: Make code unconditional. + (__get_nprocs) [!O_CLOEXEC]: Remove conditional code. + * sysdeps/unix/sysv/linux/shm_open.c: Do not include + . + [O_CLOEXEC && !__ASSUME_O_CLOEXEC] (have_o_cloexec): Remove + conditional variable definition. + (shm_open) [O_CLOEXEC]: Make code unconditional. + (shm_open) [!O_CLOEXEC || !__ASSUME_O_CLOEXEC]: Remove conditional + code. + * configure.ac (USE_REGPARMS): Don't define here. * configure: Regenerated. * sysdeps/i386/configure.ac (USE_REGPARMS): Define here. -- cgit v1.2.3