From e6f526809dbca3e772147b7bdedc3ebca8aff30e Mon Sep 17 00:00:00 2001 From: Roland McGrath Date: Mon, 12 Aug 2002 06:25:47 +0000 Subject: [PATCH] * elf/tst-tlsmod4.c (in_dso): Insert a random library call before use of the TLS macros, otherwise the compiler might not have initialized the PIC register yet when we use the PLT via asm. * elf/tst-tlsmod3.c (in_dso2): Likewise. * elf/tst-tlsmod2.c (in_dso): Likewise. --- ChangeLog | 6 ++++++ elf/tst-tlsmod2.c | 6 +++++- elf/tst-tlsmod3.c | 9 +++++++-- elf/tst-tlsmod4.c | 6 +++++- linuxthreads/ChangeLog | 10 ++++++++++ linuxthreads/manager.c | 2 +- linuxthreads/pthread.c | 2 +- 7 files changed, 35 insertions(+), 6 deletions(-) diff --git a/ChangeLog b/ChangeLog index 670cd29..da29fd2 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,5 +1,11 @@ 2002-08-11 Roland McGrath + * elf/tst-tlsmod4.c (in_dso): Insert a random library call before use + of the TLS macros, otherwise the compiler might not have initialized + the PIC register yet when we use the PLT via asm. + * elf/tst-tlsmod3.c (in_dso2): Likewise. + * elf/tst-tlsmod2.c (in_dso): Likewise. + * sunrpc/svc_authux.c (_svcauth_unix): Remove spurious printf (ugh!). * sysdeps/i386/bits/byteswap.h (__bswap_16, __bswap_32, __bswap_64): diff --git a/elf/tst-tlsmod2.c b/elf/tst-tlsmod2.c index 6aec812..1a4c73b 100644 --- a/elf/tst-tlsmod2.c +++ b/elf/tst-tlsmod2.c @@ -12,9 +12,13 @@ COMMON_INT_DEF(foo); int in_dso (int n, int *caller_foop) { - int *foop = TLS_GD (foo); + int *foop; int result = 0; + puts ("foo"); /* Make sure PLT is used before macros. */ + + foop = TLS_GD (foo); + if (caller_foop != NULL && foop != caller_foop) { printf ("callers address of foo differs: %p vs %p\n", caller_foop, foop); diff --git a/elf/tst-tlsmod3.c b/elf/tst-tlsmod3.c index 087c11b..6b7fbcc 100644 --- a/elf/tst-tlsmod3.c +++ b/elf/tst-tlsmod3.c @@ -15,10 +15,15 @@ COMMON_INT_DEF(comm_n); int in_dso2 (void) { - int *foop = TLS_GD (foo); + int *foop; int result = 0; static int n; - int *np = TLS_GD (comm_n); + int *np; + + puts ("foo"); /* Make sure PLT is used before macros. */ + + foop = TLS_GD (foo); + np = TLS_GD (comm_n); if (n != *np) { diff --git a/elf/tst-tlsmod4.c b/elf/tst-tlsmod4.c index d40b3fd..c536303 100644 --- a/elf/tst-tlsmod4.c +++ b/elf/tst-tlsmod4.c @@ -12,9 +12,13 @@ COMMON_INT_DEF(baz); int in_dso (int n, int *caller_bazp) { - int *bazp = TLS_GD (baz); + int *bazp; int result = 0; + puts ("foo"); /* Make sure PLT is used before macros. */ + + bazp = TLS_GD (baz); + if (caller_bazp != NULL && bazp != caller_bazp) { printf ("callers address of baz differs: %p vs %p\n", caller_bazp, bazp); diff --git a/linuxthreads/ChangeLog b/linuxthreads/ChangeLog index a6a2e9b..67be250 100644 --- a/linuxthreads/ChangeLog +++ b/linuxthreads/ChangeLog @@ -1,3 +1,13 @@ +2002-08-11 Roland McGrath + + * pthread.c (__pthread_initialize_manager): Initialize + p_header.data.tcb field of manager thread's descriptor. + (__pthread_initialize_minimal): Don't initialize p_header.data.self + field, already done by TLS_INIT_TP. + + * manager.c (pthread_handle_create): Move p_header field initializers + together. + 2002-08-08 Ulrich Drepper * sysdeps/i386/tls.h (TLS_DO_SET_THREAD_AREA): Removed. diff --git a/linuxthreads/manager.c b/linuxthreads/manager.c index 8ad9eb5..655c7d6 100644 --- a/linuxthreads/manager.c +++ b/linuxthreads/manager.c @@ -644,6 +644,7 @@ static int pthread_handle_create(pthread_t *thread, const pthread_attr_t *attr, /* Initialize the thread descriptor. Elements which have to be initialized to zero already have this value. */ new_thread->p_header.data.tcb = new_thread; + new_thread->p_header.data.self = new_thread; new_thread->p_tid = new_thread_id; new_thread->p_lock = &(__pthread_handles[sseg].h_lock); new_thread->p_cancelstate = PTHREAD_CANCEL_ENABLE; @@ -655,7 +656,6 @@ static int pthread_handle_create(pthread_t *thread, const pthread_attr_t *attr, #endif new_thread->p_guardaddr = guardaddr; new_thread->p_guardsize = guardsize; - new_thread->p_header.data.self = new_thread; new_thread->p_nr = sseg; new_thread->p_inheritsched = attr ? attr->__inheritsched : 0; /* Initialize the thread handle */ diff --git a/linuxthreads/pthread.c b/linuxthreads/pthread.c index cc4d26d..c43176c 100644 --- a/linuxthreads/pthread.c +++ b/linuxthreads/pthread.c @@ -429,7 +429,6 @@ __pthread_initialize_minimal(void) part of the TLS allocation. We have to initialize the data structure by hand. This initialization must mirror the struct definition above. */ - self->p_header.data.self = self; self->p_nextlive = self->p_prevlive = self; self->p_tid = PTHREAD_THREADS_MAX; self->p_lock = &__pthread_handles[0].h_lock; @@ -633,6 +632,7 @@ int __pthread_initialize_manager(void) } /* Initialize the descriptor. */ + tcb->p_header.data.tcb = tcb; tcb->p_header.data.self = tcb; tcb->p_lock = &__pthread_handles[1].h_lock; # ifndef HAVE___THREAD -- 2.7.4