diff options
author | Roland McGrath <roland@gnu.org> | 2002-12-07 21:55:56 +0000 |
---|---|---|
committer | Roland McGrath <roland@gnu.org> | 2002-12-07 21:55:56 +0000 |
commit | 82412d5402cbd1f606773e4c865857978f4e7a33 (patch) | |
tree | 33379f766b41151158151ede031cd36f1c7f137d /sysdeps | |
parent | cc1b826866c65da5f902e5e037acccc6e4940e05 (diff) | |
download | glibc-82412d5402cbd1f606773e4c865857978f4e7a33.tar.gz glibc-82412d5402cbd1f606773e4c865857978f4e7a33.tar.xz glibc-82412d5402cbd1f606773e4c865857978f4e7a33.zip |
* 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>
Diffstat (limited to 'sysdeps')
-rw-r--r-- | sysdeps/generic/libc-tls.c | 16 |
1 files changed, 10 insertions, 6 deletions
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 <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 |