diff mbox

[3/4] Use PTHREAD_DESTRUCTOR_ITERATIONS

Message ID 1411122007-1461-4-git-send-email-siddhesh@redhat.com
State New
Headers show

Commit Message

Siddhesh Poyarekar Sept. 19, 2014, 10:20 a.m. UTC
The _POSIX_THREAD_DESTRUCTOR_ITERATIONS variable is not implemented by
getconf, so to get both getconf and sysconf on the same page, make the
latter use PTHREAD_DESTRUCTOR_ITERATIONS.

The generated code is unchanged on x86_64.

Siddhesh

	* posix/conf.list (SYSCONF:POSIX): Remove
	THREAD_DESTRUCTOR_ITERATIONS.
	* sysdeps/posix/sysconf.c (__sysconf): Use
	PTHREAD_DESTRUCTOR_ITERATIONS.
---
 posix/conf.list         | 1 -
 sysdeps/posix/sysconf.c | 4 ++--
 2 files changed, 2 insertions(+), 3 deletions(-)

Comments

Roland McGrath Dec. 18, 2014, 12:44 a.m. UTC | #1
Is there any reason not to just add the _POSIX_* name to getconf?
Siddhesh Poyarekar Dec. 30, 2014, 3:10 a.m. UTC | #2
On Wed, Dec 17, 2014 at 04:44:15PM -0800, Roland McGrath wrote:
> Is there any reason not to just add the _POSIX_* name to getconf?

There isn't.  I've dropped this patch.  When I move the variable list
from getconf.c to posix-conf-vars.list, the _POSIX_* name will
automatically get added.

Siddhesh
diff mbox

Patch

diff --git a/posix/conf.list b/posix/conf.list
index 6c870af..07607b7 100644
--- a/posix/conf.list
+++ b/posix/conf.list
@@ -100,7 +100,6 @@  SYSCONF:POSIX {
   USER_GROUPS_R
   VERSION
 # Additional variables not in getconf.
-  THREAD_DESTRUCTOR_ITERATIONS
   IPV6
   RAW_SOCKETS
 }
diff --git a/sysdeps/posix/sysconf.c b/sysdeps/posix/sysconf.c
index f5b65bf..c7b748b 100644
--- a/sysdeps/posix/sysconf.c
+++ b/sysdeps/posix/sysconf.c
@@ -555,8 +555,8 @@  __sysconf (name)
 #endif
 
     case _SC_THREAD_DESTRUCTOR_ITERATIONS:
-#if CONF_IS_DEFINED_SET (_POSIX_THREAD_DESTRUCTOR_ITERATIONS)
-      return _POSIX_THREAD_DESTRUCTOR_ITERATIONS;
+#if PTHREAD_DESTRUCTOR_ITERATIONS
+      return PTHREAD_DESTRUCTOR_ITERATIONS;
 #else
       return -1;
 #endif