summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorRich Felker <dalias@aerifal.cx>2013-06-03 16:35:59 -0400
committerRich Felker <dalias@aerifal.cx>2013-06-03 16:35:59 -0400
commit44b4d09fc0626541ca12cf96f65adc21ab1fc413 (patch)
tree5ef32eaf4dc0e507a69fb9bb4cca14d8b2cab35a
parentd9265653552a2b667bf3e84407a06edaa7e9f74f (diff)
downloadmusl-44b4d09fc0626541ca12cf96f65adc21ab1fc413.tar.gz
ensure that thread dtv pointer is never null to optimize __tls_get_addr
-rw-r--r--src/ldso/dynlink.c8
-rw-r--r--src/thread/pthread_self.c2
2 files changed, 6 insertions, 4 deletions
diff --git a/src/ldso/dynlink.c b/src/ldso/dynlink.c
index dec95111..ee5ec302 100644
--- a/src/ldso/dynlink.c
+++ b/src/ldso/dynlink.c
@@ -740,13 +740,13 @@ void *__copy_tls(unsigned char *mem)
void *__tls_get_addr(size_t *v)
{
pthread_t self = __pthread_self();
- if (self->dtv && v[0]<=(size_t)self->dtv[0] && self->dtv[v[0]])
+ if (v[0]<=(size_t)self->dtv[0] && self->dtv[v[0]])
return (char *)self->dtv[v[0]]+v[1];
/* Block signals to make accessing new TLS async-signal-safe */
sigset_t set;
pthread_sigmask(SIG_BLOCK, SIGALL_SET, &set);
- if (self->dtv && v[0]<=(size_t)self->dtv[0] && self->dtv[v[0]]) {
+ if (v[0]<=(size_t)self->dtv[0] && self->dtv[v[0]]) {
pthread_sigmask(SIG_SETMASK, &set, 0);
return (char *)self->dtv[v[0]]+v[1];
}
@@ -759,10 +759,10 @@ void *__tls_get_addr(size_t *v)
for (p=head; p->tls_id != v[0]; p=p->next);
/* Get new DTV space from new DSO if needed */
- if (!self->dtv || v[0] > (size_t)self->dtv[0]) {
+ if (v[0] > (size_t)self->dtv[0]) {
void **newdtv = p->new_dtv +
(v[0]+1)*sizeof(void *)*a_fetch_add(&p->new_dtv_idx,1);
- if (self->dtv) memcpy(newdtv, self->dtv,
+ memcpy(newdtv, self->dtv,
((size_t)self->dtv[0]+1) * sizeof(void *));
newdtv[0] = (void *)v[0];
self->dtv = newdtv;
diff --git a/src/thread/pthread_self.c b/src/thread/pthread_self.c
index c50a2fb5..aed4b5f1 100644
--- a/src/thread/pthread_self.c
+++ b/src/thread/pthread_self.c
@@ -17,6 +17,8 @@ static int init_main_thread()
main_thread->self = main_thread;
main_thread->tid = main_thread->pid =
__syscall(SYS_set_tid_address, &main_thread->tid);
+ if (!main_thread->dtv)
+ main_thread->dtv = (void *)dummy;
libc.main_thread = main_thread;
return 0;
}