diff options
| author | Alexandre Oliva <aoliva@redhat.com> | 2017-02-03 20:35:16 -0500 |
|---|---|---|
| committer | Carlos O'Donell <carlos@systemhalted.org> | 2017-02-03 21:34:14 -0500 |
| commit | d675eaf7d99096a952c1d140abfed82c939fb259 (patch) | |
| tree | aa4d46c8e5d704b3a8dfa8b38765e2710adccff2 /nptl | |
| parent | 33d7e138ca63b4091bba840c609a95cdcc56c10e (diff) | |
| download | glibc-d675eaf7d99096a952c1d140abfed82c939fb259.tar.gz | |
Bug 20915: Do not initialize DTV of other threads.
In _dl_nothread_init_static_tls() and init_one_static_tls() we must not
touch the DTV of other threads since we do not have ownership of them.
The DTV need not be initialized at this point anyway since only LD/GD
accesses will use them. If LD/GD accesses occur they will take care to
initialize their own thread's DTV.
Concurrency comments were removed from the patch since they need to be
reworked along with a full description of DTV ownership and when it is
or is not safe to modify these structures.
Alexandre Oliva's original patch and discussion:
https://sourceware.org/ml/libc-alpha/2016-09/msg00512.html
Diffstat (limited to 'nptl')
| -rw-r--r-- | nptl/allocatestack.c | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/nptl/allocatestack.c b/nptl/allocatestack.c index 6402ea43f2..8a228ab254 100644 --- a/nptl/allocatestack.c +++ b/nptl/allocatestack.c @@ -1191,11 +1191,6 @@ init_one_static_tls (struct pthread *curp, struct link_map *map) # error "Either TLS_TCB_AT_TP or TLS_DTV_AT_TP must be defined" # endif - /* Fill in the DTV slot so that a later LD/GD access will find it. */ - dtv_t *dtv = GET_DTV (TLS_TPADJ (curp)); - dtv[map->l_tls_modid].pointer.to_free = NULL; - dtv[map->l_tls_modid].pointer.val = dest; - /* Initialize the memory. */ memset (__mempcpy (dest, map->l_tls_initimage, map->l_tls_initimage_size), '\0', map->l_tls_blocksize - map->l_tls_initimage_size); |