* sysdeps/generic/libc-tls.c (__libc_setup_tls): Reintroduce changes

	from 2002-12-06, but remove unnecessary memset call without removing
	necessary copy of initialization image.

	* Makerules ($(common-objpfx)format.lds): New target.
	(common-generated): Add it.
	($(inst_libdir)/libc.so): Depend on that, and cat it into the result.
	* scripts/output-format.sed: New file.
	* Makefile (distribute): Add it.

2002-12-07  Kenneth W. Chen  <kenneth.w.chen@intel.com>

diff --git a/ChangeLog b/ChangeLog
index ed984088803b250f6502c56ce7ea133fb4def94f..467b3b5172de15c058df6c12949fe05553982f6d 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,9 +1,21 @@
+2002-12-07  Roland McGrath  <roland@redhat.com>
+
+       * sysdeps/generic/libc-tls.c (__libc_setup_tls): Reintroduce changes
+       from 2002-12-06, but remove unnecessary memset call without removing
+       necessary copy of initialization image.
+
+       * Makerules ($(common-objpfx)format.lds): New target.
+       (common-generated): Add it.
+       ($(inst_libdir)/libc.so): Depend on that, and cat it into the result.
+       * scripts/output-format.sed: New file.
+       * Makefile (distribute): Add it.
+
 2002-12-07  Ulrich Drepper  <drepper@redhat.com>
 
        * sysdeps/generic/libc-tls.c: Include <sys/param.h>.
        Reported by Art Haas <ahaas@airmail.net>.
 
-2002-12-07  Kenneth W. Chen  <kenneth.w.chen@intel.com>.
+2002-12-07  Kenneth W. Chen  <kenneth.w.chen@intel.com>
 
        * sysdeps/ia64/strncpy.S (strncpy): Clear ar.ec.  Fix .recovery4.
 

diff --git a/sysdeps/generic/libc-tls.c b/sysdeps/generic/libc-tls.c
index ea325ec1834d8f3dea13ed0e7bac2b401d0f2e3e..ce1f9d5f077489381a34eb87526b204d02c5e0ef 100644 (file)
--- a/sysdeps/generic/libc-tls.c
+++ b/sysdeps/generic/libc-tls.c
@@ -21,8 +21,10 @@
 #include <ldsodefs.h>
 #include <tls.h>
 #include <unistd.h>
+#include <stdio.h>
 #include <sys/param.h>
 
+
 #ifdef SHARED
  #error makefile bug, this file is for static only
 #endif
@@ -169,8 +171,8 @@ __libc_setup_tls (size_t tcbsize, size_t tcbalign)
   /* Initialize the TLS block.  */
 # if TLS_TCB_AT_TP
   static_dtv[2].pointer = ((char *) tlsblock + tcb_offset
-                          - roundup (memsz, align));
-  static_map.l_tls_offset = roundup (memsz, align);
+                          - roundup (memsz, align ?: 1));
+  static_map.l_tls_offset = roundup (memsz, align ?: 1);
 # elif TLS_DTV_AT_TP
   tcb_offset = roundup (tcbsize, align);
   static_dtv[2].pointer = (char *) tlsblock + tcb_offset;
@@ -178,8 +180,8 @@ __libc_setup_tls (size_t tcbsize, size_t tcbalign)
 # else
 #  error "Either TLS_TCB_AT_TP or TLS_DTV_AT_TP must be defined"
 # endif
-  memset (__mempcpy (static_dtv[2].pointer, initimage, filesz),
-         '\0', memsz - filesz);
+  /* sbrk gives us zero'd memory, so we don't need to clear the remainder.  */
+  memcpy (static_dtv[2].pointer, initimage, filesz);
 
   /* Install the pointer to the dtv.  */
 
@@ -187,13 +189,15 @@ __libc_setup_tls (size_t tcbsize, size_t tcbalign)
 # if TLS_TCB_AT_TP
   INSTALL_DTV ((char *) tlsblock + tcb_offset, static_dtv);
 
-  TLS_INIT_TP ((char *) tlsblock + tcb_offset, 0);
+  const char *lossage = TLS_INIT_TP ((char *) tlsblock + tcb_offset, 0);
 # elif TLS_DTV_AT_TP
   INSTALL_DTV (tlsblock, static_dtv);
-  TLS_INIT_TP (tlsblock, 0);
+  const char *lossage = TLS_INIT_TP (tlsblock, 0);
 # else
 #  error "Either TLS_TCB_AT_TP or TLS_DTV_AT_TP must be defined"
 # endif
+  if (__builtin_expect (lossage != NULL, 0))
+    __libc_fatal (lossage);
 
   /* We have to create a fake link map which normally would be created
      by the dynamic linker.  It just has to have enough information to