diff options
author | Roland McGrath <roland@gnu.org> | 2002-08-05 01:28:17 +0000 |
---|---|---|
committer | Roland McGrath <roland@gnu.org> | 2002-08-05 01:28:17 +0000 |
commit | a4dda453b876ec04b144067c0fe54c7c715f7320 (patch) | |
tree | 73bbb2287f347cf5342f4356390ef3bfb038fad9 | |
parent | a816b435dd0b8dde7dc2f8d995bd290998801317 (diff) | |
download | glibc-a4dda453b876ec04b144067c0fe54c7c715f7320.tar.gz glibc-a4dda453b876ec04b144067c0fe54c7c715f7320.tar.xz glibc-a4dda453b876ec04b144067c0fe54c7c715f7320.zip |
* sysdeps/generic/dl-tls.c (_dl_allocate_tls_storage): Allocate the
TCB with __libc_memalign instead of mmap. (_dl_deallocate_tls): Free it with free instad of munmap.
-rw-r--r-- | ChangeLog | 4 | ||||
-rw-r--r-- | sysdeps/generic/dl-tls.c | 21 |
2 files changed, 10 insertions, 15 deletions
diff --git a/ChangeLog b/ChangeLog index 3bd844d17b..70a211281d 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,5 +1,9 @@ 2002-08-04 Roland McGrath <roland@redhat.com> + * sysdeps/generic/dl-tls.c (_dl_allocate_tls_storage): Allocate the + TCB with __libc_memalign instead of mmap. + (_dl_deallocate_tls): Free it with free instad of munmap. + * Makerules (cpp-srcs-left): When setting this to run cppflags-iterator.mk, must append .c to $(tests) and $(xtests) words. Combine the two loops into one on the concatenated list, diff --git a/sysdeps/generic/dl-tls.c b/sysdeps/generic/dl-tls.c index e4c36fb0ab..2d211fb662 100644 --- a/sysdeps/generic/dl-tls.c +++ b/sysdeps/generic/dl-tls.c @@ -205,25 +205,16 @@ _dl_allocate_tls_storage (void) size_t dtv_length; /* Allocate a correctly aligned chunk of memory. */ - /* XXX For now */ - assert (GL(dl_tls_static_align) <= GL(dl_pagesize)); -# ifdef MAP_ANON -# define _dl_zerofd (-1) -# else -# define _dl_zerofd GL(dl_zerofd) - if ((dl_zerofd) == -1) - GL(dl_zerofd) = _dl_sysdep_open_zero_fill (); -# define MAP_ANON 0 -# endif - result = __mmap (0, GL(dl_tls_static_size) ?: 1, PROT_READ|PROT_WRITE, - MAP_ANON|MAP_PRIVATE, _dl_zerofd, 0); + result = __libc_memalign (GL(dl_tls_static_align), GL(dl_tls_static_size)); + if (__builtin_expect (result == NULL, 0)) + return result; /* We allocate a few more elements in the dtv than are needed for the initial set of modules. This should avoid in most cases expansions of the dtv. */ dtv_length = GL(dl_tls_max_dtv_idx) + DTV_SURPLUS; dtv = (dtv_t *) malloc ((dtv_length + 2) * sizeof (dtv_t)); - if (result != MAP_FAILED && dtv != NULL) + if (dtv != NULL) { # if TLS_TCB_AT_TP /* The TCB follows the TLS blocks. */ @@ -241,7 +232,7 @@ _dl_allocate_tls_storage (void) /* Add the dtv to the thread data structures. */ INSTALL_DTV (result, dtv); } - else if (result != NULL) + else { free (result); result = NULL; @@ -335,7 +326,7 @@ _dl_deallocate_tls (void *tcb) /* The array starts with dtv[-1]. */ free (dtv - 1); - munmap (tcb, GL(dl_tls_static_size)); + free (tcb); } |