about summary refs log tree commit diff
diff options
context:
space:
mode:
authorRoland McGrath <roland@gnu.org>2002-08-05 01:28:17 +0000
committerRoland McGrath <roland@gnu.org>2002-08-05 01:28:17 +0000
commita4dda453b876ec04b144067c0fe54c7c715f7320 (patch)
tree73bbb2287f347cf5342f4356390ef3bfb038fad9
parenta816b435dd0b8dde7dc2f8d995bd290998801317 (diff)
downloadglibc-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--ChangeLog4
-rw-r--r--sysdeps/generic/dl-tls.c21
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);
 }