diff --git a/ChangeLog b/ChangeLog index ed98408880..467b3b5172 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,9 +1,21 @@ +2002-12-07 Roland McGrath + + * 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 * sysdeps/generic/libc-tls.c: Include . Reported by Art Haas . -2002-12-07 Kenneth W. Chen . +2002-12-07 Kenneth W. Chen * 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 ea325ec183..ce1f9d5f07 100644 --- a/sysdeps/generic/libc-tls.c +++ b/sysdeps/generic/libc-tls.c @@ -21,8 +21,10 @@ #include #include #include +#include #include + #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