about summary refs log tree commit diff
path: root/nptl/forward.c
diff options
context:
space:
mode:
authorFlorian Weimer <fweimer@redhat.com>2017-12-20 11:40:36 +0100
committerFlorian Weimer <fweimer@redhat.com>2017-12-20 11:42:04 +0100
commit8a26ad2ab7396bc94fd3cd7b25cba433908d30e1 (patch)
tree3e87bff8b59daaa5c7564532cc48c8d53ee5f7db /nptl/forward.c
parent5069ff32842c60c55f8b573ee66fe43f9ec364af (diff)
downloadglibc-8a26ad2ab7396bc94fd3cd7b25cba433908d30e1.tar.gz
glibc-8a26ad2ab7396bc94fd3cd7b25cba433908d30e1.tar.xz
glibc-8a26ad2ab7396bc94fd3cd7b25cba433908d30e1.zip
nptl: Implement pthread_self in libc.so [BZ #22635]
All binaries use TLS and thus need a properly set up TCB, so we can
simply return its address directly, instead of forwarding to the
libpthread implementation from libc.

For versioned symbols, the dynamic linker checks that the soname matches
the name supplied by the link editor, so a compatibility symbol in
libpthread is needed.

To avoid linking against the libpthread function in all cases, we would
have to bump the symbol version of libpthread in libc.so and supply a
compat symbol.  This commit does not do that because the function
implementation is so small, so the overhead by two active copies of the
same function might well be smaller than the increase in symbol table
size.
Diffstat (limited to 'nptl/forward.c')
-rw-r--r--nptl/forward.c4
1 files changed, 0 insertions, 4 deletions
diff --git a/nptl/forward.c b/nptl/forward.c
index ac96765f29..8abbccdf5e 100644
--- a/nptl/forward.c
+++ b/nptl/forward.c
@@ -193,10 +193,6 @@ FORWARD (pthread_mutex_lock, (pthread_mutex_t *mutex), (mutex), 0)
 
 FORWARD (pthread_mutex_unlock, (pthread_mutex_t *mutex), (mutex), 0)
 
-
-FORWARD2 (pthread_self, pthread_t, (void), (), return 0)
-
-
 FORWARD (__pthread_setcancelstate, (int state, int *oldstate),
 	 (state, oldstate), 0)
 strong_alias (__pthread_setcancelstate, pthread_setcancelstate)