diff options
| author | ryanarn <ryanarn@igoo.rchland.ibm.com> | 2011-07-01 09:42:21 -0500 |
|---|---|---|
| committer | Ryan S. Arnold <rsa@us.ibm.com> | 2011-07-01 09:44:59 -0500 |
| commit | 86a00204ac4492964771c1f1a10b0dec1444ba31 (patch) | |
| tree | dab8bfed88199e5cdb014b4715876babfcf5fe46 | |
| parent | 51f3cf36c1fc25ce2f2d932e087f29e8db19354c (diff) | |
| download | glibc-86a00204ac4492964771c1f1a10b0dec1444ba31.zip glibc-86a00204ac4492964771c1f1a10b0dec1444ba31.tar.gz glibc-86a00204ac4492964771c1f1a10b0dec1444ba31.tar.bz2 | |
Remove pthread_attr_setstacksize from __aio_create_helper_thread.
(cherry picked from commit 2c3d9ba9a332cecb009b97ebac5bb78a13118c7c)
| -rw-r--r-- | ChangeLog | 8 | ||||
| -rw-r--r-- | nptl/sysdeps/unix/sysv/linux/aio_misc.h | 4 |
2 files changed, 10 insertions, 2 deletions
@@ -1,3 +1,11 @@ +2011-06-30 Ryan S. Arnold <rsa@linux.vnet.ibm.com> + + * nptl/sysdeps/unix/sysv/linux/aio_misc.h + (__aio_create_helper_thread): Remove call to + pthread_attr_setstacksize () which set the stacksize to + PTHREAD_STACK_MIN. This is too small if large amounts of TLS are in + use. Removing this call effectively sets stacksize to 'default'. + 2011-06-28 Adhemerval Zanella <azanella@linux.vnet.ibm.com> * sysdeps/powerpc/fpu/e_hypot.c: optimizations for POWER5 and POWER6. diff --git a/nptl/sysdeps/unix/sysv/linux/aio_misc.h b/nptl/sysdeps/unix/sysv/linux/aio_misc.h index 406d96e..2bf7c17 100644 --- a/nptl/sysdeps/unix/sysv/linux/aio_misc.h +++ b/nptl/sysdeps/unix/sysv/linux/aio_misc.h @@ -46,8 +46,8 @@ __aio_create_helper_thread (pthread_t *threadp, void *(*tf) (void *), pthread_attr_init (&attr); pthread_attr_setdetachstate (&attr, PTHREAD_CREATE_DETACHED); - /* The helper thread needs only very little resources. */ - (void) pthread_attr_setstacksize (&attr, PTHREAD_STACK_MIN); + /* Don't set the stacksize to PTHREAD_STACK_MIN. Let it default, otherwise + the stack may be too small if TLS is in use. */ /* Block all signals in the helper thread. To do this thoroughly we temporarily have to block all signals here. */ |