about summary refs log tree commit diff
path: root/src/ldso/dynlink.c
diff options
context:
space:
mode:
authorSzabolcs Nagy <nsz@port70.net>2015-03-11 12:48:12 +0000
committerRich Felker <dalias@aerifal.cx>2015-03-11 18:53:48 -0400
commit204a69d2d917f413eacb086020b63d6dc6da0672 (patch)
treeffa1ecefe51649524b192d013f1d5337da160e39 /src/ldso/dynlink.c
parenta46677af1863f3d050f216a43dfdf4818210c266 (diff)
downloadmusl-204a69d2d917f413eacb086020b63d6dc6da0672.tar.gz
musl-204a69d2d917f413eacb086020b63d6dc6da0672.tar.xz
musl-204a69d2d917f413eacb086020b63d6dc6da0672.zip
copy the dtv pointer to the end of the pthread struct for TLS_ABOVE_TP archs
There are two main abi variants for thread local storage layout:

 (1) TLS is above the thread pointer at a fixed offset and the pthread
 struct is below that. So the end of the struct is at known offset.

 (2) the thread pointer points to the pthread struct and TLS starts
 below it. So the start of the struct is at known (zero) offset.

Assembly code for the dynamic TLSDESC callback needs to access the
dynamic thread vector (dtv) pointer which is currently at the front
of the pthread struct. So in case of (1) the asm code needs to hard
code the offset from the end of the struct which can easily break if
the struct changes.

This commit adds a copy of the dtv at the end of the struct. New members
must not be added after dtv_copy, only before it. The size of the struct
is increased a bit, but there is opportunity for size optimizations.
Diffstat (limited to 'src/ldso/dynlink.c')
-rw-r--r--src/ldso/dynlink.c6
1 files changed, 3 insertions, 3 deletions
diff --git a/src/ldso/dynlink.c b/src/ldso/dynlink.c
index ca101993..b0d5ff46 100644
--- a/src/ldso/dynlink.c
+++ b/src/ldso/dynlink.c
@@ -1015,7 +1015,7 @@ void *__copy_tls(unsigned char *mem)
 	dtv[0] = (void *)tls_cnt;
 	if (!tls_cnt) {
 		td = (void *)(dtv+1);
-		td->dtv = dtv;
+		td->dtv = td->dtv_copy = dtv;
 		return td;
 	}
 
@@ -1041,7 +1041,7 @@ void *__copy_tls(unsigned char *mem)
 		memcpy(dtv[p->tls_id], p->tls_image, p->tls_len);
 	}
 #endif
-	td->dtv = dtv;
+	td->dtv = td->dtv_copy = dtv;
 	return td;
 }
 
@@ -1071,7 +1071,7 @@ void *__tls_get_new(size_t *v)
 		memcpy(newdtv, self->dtv,
 			((size_t)self->dtv[0]+1) * sizeof(void *));
 		newdtv[0] = (void *)v[0];
-		self->dtv = newdtv;
+		self->dtv = self->dtv_copy = newdtv;
 	}
 
 	/* Get new TLS memory from all new DSOs up to the requested one */