about summary refs log tree commit diff
diff options
context:
space:
mode:
authorCarlos O'Donell <carlos@redhat.com>2013-03-01 16:18:08 -0500
committerCarlos O'Donell <carlos@redhat.com>2013-03-01 16:18:08 -0500
commit4e9b5995775c99e029ed4003aaddb88bfbf8e2cd (patch)
tree971f5f28317f29e5c3547bffff8c8bde28ba71e8
parentace4acc8ace692f64051594afe47efb1135b3c29 (diff)
downloadglibc-4e9b5995775c99e029ed4003aaddb88bfbf8e2cd.tar.gz
glibc-4e9b5995775c99e029ed4003aaddb88bfbf8e2cd.tar.xz
glibc-4e9b5995775c99e029ed4003aaddb88bfbf8e2cd.zip
Revert GLIBC_PTHREAD_DEFAULT_STACKSIZE changes.
This reverts the change that allows the POSIX Thread default stack size
to be changed by the environment variable
GLIBC_PTHREAD_DEFAULT_STACKSIZE. It has been requested that more
discussion happen before this change goes into 2.18.
-rw-r--r--ChangeLog5
-rw-r--r--NEWS4
-rw-r--r--csu/libc-start.c4
-rw-r--r--csu/libc-tls.c2
-rw-r--r--nptl/ChangeLog8
-rw-r--r--nptl/Makefile5
-rw-r--r--nptl/nptl-init.c74
-rw-r--r--nptl/tst-pthread-stack-env.c77
8 files changed, 39 insertions, 140 deletions
diff --git a/ChangeLog b/ChangeLog
index 864832330c..46edfc0bea 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,8 @@
+2013-03-01  Carlos O'Donell  <carlos@redhat.com>
+
+	* csu/libc-start.c (__pthread_initialize_minimal): Revert last change.
+	* csu/libc-tls.c (__pthread_initialize_minimal): Likewise.
+
 2013-03-01  Siddhesh Poyarekar  <siddhesh@redhat.com>
 
 	* NEWS: Mention libm performance improvements and non-x86 PI
diff --git a/NEWS b/NEWS
index 313ccd78d9..c5e38b3f62 100644
--- a/NEWS
+++ b/NEWS
@@ -17,10 +17,6 @@ Version 2.18
   and program exit.  This needs compiler support for offloading C++11
   destructor calls to glibc.
 
-* Add support for setting thread stack sizes from the program environment,
-  independently of the process stack size using the
-  GLIBC_PTHREAD_DEFAULT_STACKSIZE environment variable.
-
 * Improved worst case performance of libm functions with double inputs and
   output.
 
diff --git a/csu/libc-start.c b/csu/libc-start.c
index f53f71c975..9c4c01d9fd 100644
--- a/csu/libc-start.c
+++ b/csu/libc-start.c
@@ -30,7 +30,7 @@ extern int __libc_multiple_libcs;
 #include <tls.h>
 #ifndef SHARED
 # include <dl-osinfo.h>
-extern void __pthread_initialize_minimal (int, char **, char **);
+extern void __pthread_initialize_minimal (void);
 # ifndef THREAD_SET_STACK_GUARD
 /* Only exported for architectures that don't store the stack guard canary
    in thread local area.  */
@@ -167,7 +167,7 @@ LIBC_START_MAIN (int (*main) (int, char **, char ** MAIN_AUXVEC_DECL),
   /* Initialize the thread library at least a bit since the libgcc
      functions are using thread functions if these are available and
      we need to setup errno.  */
-  __pthread_initialize_minimal (argc, argv, __environ);
+  __pthread_initialize_minimal ();
 
   /* Set up the stack checker's canary.  */
   uintptr_t stack_chk_guard = _dl_setup_stack_chk_guard (_dl_random);
diff --git a/csu/libc-tls.c b/csu/libc-tls.c
index 4c20bb5531..5fa39eb8d1 100644
--- a/csu/libc-tls.c
+++ b/csu/libc-tls.c
@@ -243,7 +243,7 @@ _dl_tls_setup (void)
    not used.  */
 void
 __attribute__ ((weak))
-__pthread_initialize_minimal (int argc, char **argv, char **envp)
+__pthread_initialize_minimal (void)
 {
   __libc_setup_tls (TLS_INIT_TCB_SIZE, TLS_INIT_TCB_ALIGN);
 }
diff --git a/nptl/ChangeLog b/nptl/ChangeLog
index 4beb77c43e..2374bbfd17 100644
--- a/nptl/ChangeLog
+++ b/nptl/ChangeLog
@@ -1,3 +1,11 @@
+2013-03-01  Carlos O'Donell  <carlos@redhat.com>
+
+	* Makefile (tests): Revert last change.
+	(tst-pthread-stack-env-ENV): Likewise.
+	* nptl-init.c (set_default_stacksize): Likewise.
+	(__pthread_initialize_minimal_internal): Likewise.
+	* tst-pthread-stack-env.c: Likewise.
+
 2013-03-01  Siddhesh Poyarekar  <siddhesh@redhat.com>
 
 	* tst-oddstacklimit.c: Include stdlib.h.
diff --git a/nptl/Makefile b/nptl/Makefile
index e7cfe8bcd4..6af4b37af4 100644
--- a/nptl/Makefile
+++ b/nptl/Makefile
@@ -251,8 +251,7 @@ tests = tst-typesizes \
 	tst-exec1 tst-exec2 tst-exec3 tst-exec4 \
 	tst-exit1 tst-exit2 tst-exit3 \
 	tst-stdio1 tst-stdio2 \
-	tst-stack1 tst-stack2 tst-stack3 \
-	tst-pthread-getattr tst-pthread-stack-env \
+	tst-stack1 tst-stack2 tst-stack3 tst-pthread-getattr \
 	tst-unload \
 	tst-dlsym1 \
 	tst-sysconf \
@@ -442,8 +441,6 @@ tst-cancel7-ARGS = --command "exec $(host-test-program-cmd)"
 tst-cancelx7-ARGS = $(tst-cancel7-ARGS)
 tst-umask1-ARGS = $(objpfx)tst-umask1.temp
 
-tst-pthread-stack-env-ENV = GLIBC_PTHREAD_DEFAULT_STACKSIZE=1048576
-
 $(objpfx)tst-atfork2: $(libdl) $(shared-thread-library)
 LDFLAGS-tst-atfork2 = -rdynamic
 tst-atfork2-ENV = MALLOC_TRACE=$(objpfx)tst-atfork2.mtrace
diff --git a/nptl/nptl-init.c b/nptl/nptl-init.c
index 50148224e8..19e6616420 100644
--- a/nptl/nptl-init.c
+++ b/nptl/nptl-init.c
@@ -276,26 +276,8 @@ extern void **__libc_dl_error_tsd (void) __attribute__ ((const));
 /* This can be set by the debugger before initialization is complete.  */
 static bool __nptl_initial_report_events __attribute_used__;
 
-static void
-set_default_stacksize (size_t stacksize)
-{
-  if (stacksize < PTHREAD_STACK_MIN)
-    stacksize = PTHREAD_STACK_MIN;
-
-  /* Make sure it meets the minimum size that allocate_stack
-     (allocatestack.c) will demand, which depends on the page size.  */
-  const uintptr_t pagesz = GLRO(dl_pagesize);
-  const size_t minstack = pagesz + __static_tls_size + MINIMAL_REST_STACK;
-
-  if (stacksize < minstack)
-    stacksize = minstack;
-
-  /* Round the resource limit up to page size.  */
-  stacksize = (stacksize + pagesz - 1) & -pagesz;
-  __default_stacksize = stacksize;
-}
 void
-__pthread_initialize_minimal_internal (int argc, char **argv, char **envp)
+__pthread_initialize_minimal_internal (void)
 {
 #ifndef SHARED
   /* Unlike in the dynamically linked case the dynamic linker has not
@@ -419,41 +401,29 @@ __pthread_initialize_minimal_internal (int argc, char **argv, char **envp)
 
   __static_tls_size = roundup (__static_tls_size, static_tls_align);
 
-  /* Initialize the environment.  libc.so gets initialized after us due to a
-     circular dependency and hence __environ is not available otherwise.  */
-    __environ = envp;
-
-#ifndef SHARED
-    __libc_init_secure ();
-#endif
-
-  size_t stacksize = 0;
-  char *envval = __libc_secure_getenv ("GLIBC_PTHREAD_DEFAULT_STACKSIZE");
-
-  if (__glibc_unlikely (envval != NULL && envval[0] != '\0'))
-    {
-      char *env_conv = envval;
-      size_t ret = strtoul (envval, &env_conv, 0);
+  /* Determine the default allowed stack size.  This is the size used
+     in case the user does not specify one.  */
+  struct rlimit limit;
+  if (getrlimit (RLIMIT_STACK, &limit) != 0
+      || limit.rlim_cur == RLIM_INFINITY)
+    /* The system limit is not usable.  Use an architecture-specific
+       default.  */
+    limit.rlim_cur = ARCH_STACK_DEFAULT_SIZE;
+  else if (limit.rlim_cur < PTHREAD_STACK_MIN)
+    /* The system limit is unusably small.
+       Use the minimal size acceptable.  */
+    limit.rlim_cur = PTHREAD_STACK_MIN;
 
-      if (*env_conv == '\0' && env_conv != envval)
-	stacksize = ret;
-    }
-
-  if (stacksize == 0)
-    {
-      /* Determine the default allowed stack size.  This is the size used
-	 in case the user does not specify one.  */
-      struct rlimit limit;
-      if (getrlimit (RLIMIT_STACK, &limit) != 0
-	  || limit.rlim_cur == RLIM_INFINITY)
-	/* The system limit is not usable.  Use an architecture-specific
-	   default.  */
-	stacksize = ARCH_STACK_DEFAULT_SIZE;
-      else
-	stacksize = limit.rlim_cur;
-    }
+  /* Make sure it meets the minimum size that allocate_stack
+     (allocatestack.c) will demand, which depends on the page size.  */
+  const uintptr_t pagesz = GLRO(dl_pagesize);
+  const size_t minstack = pagesz + __static_tls_size + MINIMAL_REST_STACK;
+  if (limit.rlim_cur < minstack)
+    limit.rlim_cur = minstack;
 
-  set_default_stacksize (stacksize);
+  /* Round the resource limit up to page size.  */
+  limit.rlim_cur = (limit.rlim_cur + pagesz - 1) & -pagesz;
+  __default_stacksize = limit.rlim_cur;
 
 #ifdef SHARED
   /* Transfer the old value from the dynamic linker's internal location.  */
diff --git a/nptl/tst-pthread-stack-env.c b/nptl/tst-pthread-stack-env.c
deleted file mode 100644
index 09f03044c0..0000000000
--- a/nptl/tst-pthread-stack-env.c
+++ /dev/null
@@ -1,77 +0,0 @@
-/* Verify that pthreads uses the default thread stack size set with the
-   GLIBC_PTHREAD_DEFAULT_STACKSIZE environment variable.
-   Copyright (C) 2013 Free Software Foundation, Inc.
-   This file is part of the GNU C Library.
-
-   The GNU C Library is free software; you can redistribute it and/or
-   modify it under the terms of the GNU Lesser General Public
-   License as published by the Free Software Foundation; either
-   version 2.1 of the License, or (at your option) any later version.
-
-   The GNU C Library is distributed in the hope that it will be useful,
-   but WITHOUT ANY WARRANTY; without even the implied warranty of
-   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
-   Lesser General Public License for more details.
-
-   You should have received a copy of the GNU Lesser General Public
-   License along with the GNU C Library; if not, see
-   <http://www.gnu.org/licenses/>.  */
-
-#include <pthread.h>
-#include <stdio.h>
-#include <string.h>
-
-/* It is possible that the default stack size somehow ends up being 1MB, thus
-   giving a false positive.  The ideal way to test this would be to get the
-   current stacksize fork a process with the default stack size set to
-   something different to the current stack size and verify in the child
-   process that the environment variable worked.  */
-#define STACKSIZE 1024 * 1024L
-
-void *
-thr (void *u)
-{
-  size_t stacksize, guardsize;
-  pthread_attr_t attr;
-  pthread_getattr_np (pthread_self (), &attr);
-
-  pthread_attr_getstacksize (&attr, &stacksize);
-  pthread_attr_getguardsize (&attr, &guardsize);
-
-  /* FIXME once guardsize is excluded from stacksize.  */
-  if (stacksize - guardsize != STACKSIZE)
-    {
-      printf ("Stack size is %zu, should be %zu\n", stacksize - guardsize,
-	      STACKSIZE);
-      return (void *) 1;
-    }
-
-  return NULL;
-}
-
-int
-do_test (int argc, char **argv)
-{
-  pthread_t t;
-  void *thr_ret;
-  int ret;
-
-  if ((ret = pthread_create (&t, NULL, thr, NULL)) != 0)
-    {
-      printf ("thread create failed: %s\n", strerror (ret));
-      return 1;
-    }
-
-  if ((ret = pthread_join (t, &thr_ret)) != 0)
-    {
-      printf ("join failed: %s\n", strerror (ret));
-      return 1;
-    }
-
-  if (thr_ret != NULL && thr_ret != PTHREAD_CANCELED)
-    return 1;
-
-  return 0;
-}
-
-#include "../test-skeleton.c"