diff options
author | Roland McGrath <roland@gnu.org> | 2002-10-24 19:14:59 +0000 |
---|---|---|
committer | Roland McGrath <roland@gnu.org> | 2002-10-24 19:14:59 +0000 |
commit | 5bbcba0db58cef39ec4b582cd665ccb019dc6763 (patch) | |
tree | 5e18843757e510eddaef04aa3213bcb351681319 | |
parent | 369b849f1a382df2c1ee5d3a997bf318950cf5ab (diff) | |
download | glibc-5bbcba0db58cef39ec4b582cd665ccb019dc6763.tar.gz |
* elf/dl-misc.c: Include <sysdep.h>.
(_dl_debug_vdprintf): Only take dl_load_lock if not _dl_starting_up.
* sysdeps/generic/ldsodefs.h (_dl_starting_up): Declare it here.
* sysdeps/unix/sysv/linux/init-first.c: Not here.
* sysdeps/powerpc/elf/libc-start.c: Or here.
* sysdeps/unix/sysv/aix/libc-start.c: Or here.
* sysdeps/unix/sysv/aix/start-libc.c: Or here.
* sysdeps/unix/sysv/aix/init-first.c: Or here.
* sysdeps/generic/libc-start.c: Or here.
* sysdeps/unix/sysv/linux/init-first.c (init): Protect _dl_starting_up
access with [! SHARED].
* sysdeps/unix/sysv/aix/init-first.c (init): Likewise.
-rw-r--r-- | ChangeLog | 14 | ||||
-rw-r--r-- | elf/dl-misc.c | 18 | ||||
-rw-r--r-- | sysdeps/generic/ldsodefs.h | 7 | ||||
-rw-r--r-- | sysdeps/generic/libc-start.c | 2 | ||||
-rw-r--r-- | sysdeps/powerpc/elf/libc-start.c | 3 | ||||
-rw-r--r-- | sysdeps/unix/sysv/aix/init-first.c | 5 | ||||
-rw-r--r-- | sysdeps/unix/sysv/aix/libc-start.c | 2 | ||||
-rw-r--r-- | sysdeps/unix/sysv/aix/start-libc.c | 2 | ||||
-rw-r--r-- | sysdeps/unix/sysv/linux/init-first.c | 6 |
9 files changed, 39 insertions, 20 deletions
@@ -1,5 +1,8 @@ 2002-10-24 Jakub Jelinek <jakub@redhat.com> + * elf/dl-misc.c: Include <sysdep.h>. + (_dl_debug_vdprintf): Only take dl_load_lock if not _dl_starting_up. + * sysdeps/unix/sysv/linux/s390/s390-32/sysdep.h (INTERNAL_SYSCALL, INTERNAL_SYSCALL_ERROR_P, INTERNAL_SYSCALL_ERRNO): New macros. (INLINE_SYSCALL): Use that. @@ -27,6 +30,17 @@ 2002-10-24 Roland McGrath <roland@redhat.com> + * sysdeps/generic/ldsodefs.h (_dl_starting_up): Declare it here. + * sysdeps/unix/sysv/linux/init-first.c: Not here. + * sysdeps/powerpc/elf/libc-start.c: Or here. + * sysdeps/unix/sysv/aix/libc-start.c: Or here. + * sysdeps/unix/sysv/aix/start-libc.c: Or here. + * sysdeps/unix/sysv/aix/init-first.c: Or here. + * sysdeps/generic/libc-start.c: Or here. + * sysdeps/unix/sysv/linux/init-first.c (init): Protect _dl_starting_up + access with [! SHARED]. + * sysdeps/unix/sysv/aix/init-first.c (init): Likewise. + * libio/bug-wfflush.c: New file. * libio/Makefile (tests): Add bug-wfflush. diff --git a/elf/dl-misc.c b/elf/dl-misc.c index f80472485d..0fbfcde004 100644 --- a/elf/dl-misc.c +++ b/elf/dl-misc.c @@ -30,6 +30,7 @@ #include <sys/param.h> #include <sys/stat.h> #include <sys/uio.h> +#include <sysdep.h> #include <stdio-common/_itoa.h> #include <bits/libc-lock.h> @@ -81,7 +82,7 @@ _dl_sysdep_read_whole_file (const char *file, size_t *sizep, int prot) } -/* Bare-bone printf implementation. This function only knows about +/* Bare-bones printf implementation. This function only knows about the formats and flags needed and can handle only up to 64 stripes in the output. */ static void @@ -250,10 +251,17 @@ _dl_debug_vdprintf (int fd, int tag_p, const char *fmt, va_list arg) INTERNAL_SYSCALL (writev, 3, fd, iov, niov); #elif RTLD_PRIVATE_ERRNO /* We have to take this lock just to be sure we don't clobber the private - errno when it's being used by another thread that cares about it. */ - __libc_lock_lock_recursive (GL(dl_load_lock)); - __writev (fd, iov, niov); - __libc_lock_unlock_recursive (GL(dl_load_lock)); + errno when it's being used by another thread that cares about it. + Yet we must be sure not to try calling the lock functions before + the thread library is fully initialized. */ + if (__builtin_expect (INTUSE (_dl_starting_up), 0)) + __writev (fd, iov, niov); + else + { + __libc_lock_lock_recursive (GL(dl_load_lock)); + __writev (fd, iov, niov); + __libc_lock_unlock_recursive (GL(dl_load_lock)); + } #else __writev (fd, iov, niov); #endif diff --git a/sysdeps/generic/ldsodefs.h b/sysdeps/generic/ldsodefs.h index 36582a7463..a8f006122b 100644 --- a/sysdeps/generic/ldsodefs.h +++ b/sysdeps/generic/ldsodefs.h @@ -416,6 +416,13 @@ extern const char _dl_out_of_memory[]; extern const char _dl_out_of_memory_internal[] attribute_hidden; #endif +#ifndef SHARED +/* Flag set at startup and cleared when the last initializer has run. */ +extern int _dl_starting_up; +weak_extern (_dl_starting_up) +#elif defined IS_IN_rtld +extern int _dl_starting_up_internal attribute_hidden; +#endif /* OS-dependent function to open the zero-fill device. */ extern int _dl_sysdep_open_zero_fill (void); /* dl-sysdep.c */ diff --git a/sysdeps/generic/libc-start.c b/sysdeps/generic/libc-start.c index 3c7f2fa639..806ed789cc 100644 --- a/sysdeps/generic/libc-start.c +++ b/sysdeps/generic/libc-start.c @@ -24,8 +24,6 @@ extern void __libc_init_first (int argc, char **argv, char **envp); -extern int _dl_starting_up; -weak_extern (_dl_starting_up) extern int __libc_multiple_libcs; extern void *__libc_stack_end; diff --git a/sysdeps/powerpc/elf/libc-start.c b/sysdeps/powerpc/elf/libc-start.c index 6751b9ea06..4878979f43 100644 --- a/sysdeps/powerpc/elf/libc-start.c +++ b/sysdeps/powerpc/elf/libc-start.c @@ -24,9 +24,6 @@ extern void __libc_init_first (int argc, char **argv, char **envp); -extern int _dl_starting_up; -weak_extern (_dl_starting_up) - extern int __cache_line_size; weak_extern (__cache_line_size) diff --git a/sysdeps/unix/sysv/aix/init-first.c b/sysdeps/unix/sysv/aix/init-first.c index 4c6768b8ad..8272a62824 100644 --- a/sysdeps/unix/sysv/aix/init-first.c +++ b/sysdeps/unix/sysv/aix/init-first.c @@ -37,9 +37,6 @@ extern void __libc_init (int, char **, char **); /* The function is called from assembly stubs the compiler can't see. */ static void init (int, char **, char **) __attribute__ ((unused)); -extern int _dl_starting_up; -weak_extern (_dl_starting_up) - /* Set nonzero if we have to be prepared for more then one libc being used in the process. Safe assumption if initializer never runs. */ int __libc_multiple_libcs attribute_hidden = 1; @@ -60,10 +57,12 @@ init (int argc, char **argv, char **envp) If the address would be taken inside the expression the optimizer would try to be too smart and throws it away. Grrr. */ +#ifndef SHARED /* XXX disable dl for now int *dummy_addr = &_dl_starting_up; __libc_multiple_libcs = dummy_addr && !_dl_starting_up; */ +#endif /* Save the command-line arguments. */ __libc_argc = argc; diff --git a/sysdeps/unix/sysv/aix/libc-start.c b/sysdeps/unix/sysv/aix/libc-start.c index 757b34975c..813e28d09b 100644 --- a/sysdeps/unix/sysv/aix/libc-start.c +++ b/sysdeps/unix/sysv/aix/libc-start.c @@ -31,8 +31,6 @@ typedef unsigned char uchar; extern void __libc_init_first (int argc, char **argv, char **envp); /* XXX disable for now -extern int _dl_starting_up; -weak_extern (_dl_starting_up) extern int __libc_multiple_libcs; */ /* XXX normally defined in generic/dl-sydep.c, hack it into existance diff --git a/sysdeps/unix/sysv/aix/start-libc.c b/sysdeps/unix/sysv/aix/start-libc.c index 303caa7078..0cbe79d0b1 100644 --- a/sysdeps/unix/sysv/aix/start-libc.c +++ b/sysdeps/unix/sysv/aix/start-libc.c @@ -31,8 +31,6 @@ typedef unsigned char uchar; extern void __libc_init_first (int argc, char **argv, char **envp); /* XXX disable for now -extern int _dl_starting_up; -weak_extern (_dl_starting_up) extern int __libc_multiple_libcs; */ /* XXX normally defined in generic/dl-sydep.c, hack it into existance diff --git a/sysdeps/unix/sysv/linux/init-first.c b/sysdeps/unix/sysv/linux/init-first.c index e644b29eda..a2905f9392 100644 --- a/sysdeps/unix/sysv/linux/init-first.c +++ b/sysdeps/unix/sysv/linux/init-first.c @@ -35,9 +35,6 @@ /* The function is called from assembly stubs the compiler can't see. */ static void init (int, char **, char **) __attribute__ ((unused)); -extern int _dl_starting_up; -weak_extern (_dl_starting_up) - /* Set nonzero if we have to be prepared for more then one libc being used in the process. Safe assumption if initializer never runs. */ int __libc_multiple_libcs attribute_hidden = 1; @@ -54,12 +51,15 @@ init (int argc, char **argv, char **envp) #ifdef USE_NONOPTION_FLAGS extern void __getopt_clean_environment (char **); #endif + +#ifndef SHARED /* The next variable is only here to work around a bug in gcc <= 2.7.2.2. If the address would be taken inside the expression the optimizer would try to be too smart and throws it away. Grrr. */ int *dummy_addr = &_dl_starting_up; __libc_multiple_libcs = dummy_addr && !_dl_starting_up; +#endif /* Make sure we don't initialize twice. */ if (!__libc_multiple_libcs) |