r5843 - in glibc-package/branches/eglibc-2.18/debian: . patches patches/s390
Author: aurel32
Date: 2013-12-01 12:44:17 +0000 (Sun, 01 Dec 2013)
New Revision: 5843
Added:
glibc-package/branches/eglibc-2.18/debian/patches/s390/cvs-s390-tls-got-pointer.diff
Modified:
glibc-package/branches/eglibc-2.18/debian/changelog
glibc-package/branches/eglibc-2.18/debian/patches/series
Log:
patches/s390/cvs-s390-tls-got-pointer.diff: new patch from upstream to fix
the TLS GOT pointer setup on s390.
Modified: glibc-package/branches/eglibc-2.18/debian/changelog
===================================================================
--- glibc-package/branches/eglibc-2.18/debian/changelog 2013-12-01 12:43:29 UTC (rev 5842)
+++ glibc-package/branches/eglibc-2.18/debian/changelog 2013-12-01 12:44:17 UTC (rev 5843)
@@ -164,6 +164,8 @@
patches/ia64/cvs-ia64-DL_CALL_DT_INIT_FINI.diff: new patches from upstream
to fix a segmentation fault in ld.so on hppa and ia64.
* testsuite-checking/expected-results-sparc*-linux-gnu-*: cleanup.
+ * patches/s390/cvs-s390-tls-got-pointer.diff: new patch from upstream to fix
+ the TLS GOT pointer setup on s390.
-- Adam Conrad <adconrad@ubuntu.com> Sat, 09 Nov 2013 12:48:20 +0100
Added: glibc-package/branches/eglibc-2.18/debian/patches/s390/cvs-s390-tls-got-pointer.diff
===================================================================
--- glibc-package/branches/eglibc-2.18/debian/patches/s390/cvs-s390-tls-got-pointer.diff (rev 0)
+++ glibc-package/branches/eglibc-2.18/debian/patches/s390/cvs-s390-tls-got-pointer.diff 2013-12-01 12:44:17 UTC (rev 5843)
@@ -0,0 +1,135 @@
+2013-11-29 Siddhesh Poyarekar <siddhesh@redhat.com>
+ Andreas Krebbel <Andreas.Krebbel@de.ibm.com>
+
+ [BZ #16214]
+ * sysdeps/s390/dl-tls.h (__TLS_GET_ADDR): Invoke
+ __tls_get_addr_internal instead of __tls_get_offset in order to
+ avoid GOT pointer dependency. Make rtld export
+ __tls_get_addr_internal@@GLIBC_PRIVATE while still hiding
+ __tls_get_addr since we are a __tls_get_offset platform.
+ * sysdeps/s390/s390-64/tls-macros.h (TLS_IE PIC): Don't rely on
+ GOT pointer being set up before.
+ * sysdeps/s390/s390-32/tls-macros.h (TLS_IE PIC): Likewise.
+
+diff --git a/sysdeps/s390/Versions b/sysdeps/s390/Versions
+index e18617c..baf9842 100644
+--- a/sysdeps/s390/Versions
++++ b/sysdeps/s390/Versions
+@@ -3,4 +3,8 @@ ld {
+ # runtime interface to TLS
+ __tls_get_offset;
+ }
++ GLIBC_PRIVATE {
++ # Exported by ld used by libc.
++ __tls_get_addr_internal;
++ }
+ }
+diff --git a/sysdeps/s390/dl-tls.h b/sysdeps/s390/dl-tls.h
+index 68a5af4..52192a2 100644
+--- a/sysdeps/s390/dl-tls.h
++++ b/sysdeps/s390/dl-tls.h
+@@ -26,11 +26,26 @@ typedef struct
+
+
+ #ifdef SHARED
+-/* This is the prototype for the GNU version. */
+-extern void *__tls_get_addr (tls_index *ti) attribute_hidden;
++
+ extern unsigned long __tls_get_offset (unsigned long got_offset);
+
+ # ifdef IS_IN_rtld
++
++# include <shlib-compat.h>
++
++extern void *__tls_get_addr (tls_index *ti) attribute_hidden;
++/* Make a temporary alias of __tls_get_addr to remove the hidden
++ attribute. Then export __tls_get_addr as __tls_get_addr_internal
++ for use from libc. We do not want to export __tls_get_addr, but we
++ do need to use it from libc when looking up the address of a TLS
++ variable. We don't use __tls_get_offset because it requires r12 to
++ be setup and that might not always be true. Either way it's more
++ optimal to use __tls_get_addr directly (that's what
++ __tls_get_offset does anyways). */
++strong_alias (__tls_get_addr, __tls_get_addr_internal_tmp);
++versioned_symbol (ld, __tls_get_addr_internal_tmp,
++ __tls_get_addr_internal, GLIBC_PRIVATE);
++
+ /* The special thing about the s390 TLS ABI is that we do not have the
+ standard __tls_get_addr function but the __tls_get_offset function
+ which differs in two important aspects:
+@@ -63,15 +78,21 @@ __tls_get_offset:\n\
+ 1: .long __tls_get_addr - 0b\n\
+ ");
+ # endif
+-# endif
++# else /* IS_IN_rtld */
++extern void *__tls_get_addr_internal (tls_index *ti);
++# endif /* !IS_IN_rtld */
+
+ # define GET_ADDR_OFFSET \
+ (ti->ti_offset - (unsigned long) __builtin_thread_pointer ())
+
+-# define __TLS_GET_ADDR(__ti) \
+- ({ extern char _GLOBAL_OFFSET_TABLE_[] attribute_hidden; \
+- (void *) __tls_get_offset ((char *) (__ti) - _GLOBAL_OFFSET_TABLE_) \
+- + (unsigned long) __builtin_thread_pointer (); })
++/* Use the privately exported __tls_get_addr_internal instead of
++ __tls_get_offset in order to avoid the __tls_get_offset special
++ linkage requiring the GOT pointer to be set up in r12. The
++ compiler will take care of setting up r12 only if itself issued the
++ __tls_get_offset call. */
++# define __TLS_GET_ADDR(__ti) \
++ ({ (void *) __tls_get_addr_internal ((char *) (__ti)) \
++ + (unsigned long) __builtin_thread_pointer (); })
+
+ #endif
+
+diff --git a/sysdeps/s390/s390-32/tls-macros.h b/sysdeps/s390/s390-32/tls-macros.h
+index 8a0ad58..a592d81 100644
+--- a/sysdeps/s390/s390-32/tls-macros.h
++++ b/sysdeps/s390/s390-32/tls-macros.h
+@@ -8,12 +8,15 @@
+
+ #ifdef PIC
+ # define TLS_IE(x) \
+- ({ unsigned long __offset; \
++ ({ unsigned long __offset, __got; \
+ asm ("bras %0,1f\n" \
+- "0:\t.long " #x "@gotntpoff\n" \
+- "1:\tl %0,0(%0)\n\t" \
+- "l %0,0(%0,%%r12):tls_load:" #x \
+- : "=&a" (__offset) : : "cc" ); \
++ "0:\t.long _GLOBAL_OFFSET_TABLE_-0b\n\t" \
++ ".long " #x "@gotntpoff\n" \
++ "1:\tl %1,0(%0)\n\t" \
++ "la %1,0(%1,%0)\n\t" \
++ "l %0,4(%0)\n\t" \
++ "l %0,0(%0,%1):tls_load:" #x "\n" \
++ : "=&a" (__offset), "=&a" (__got) : : "cc" ); \
+ (int *) (__builtin_thread_pointer() + __offset); })
+ #else
+ # define TLS_IE(x) \
+diff --git a/sysdeps/s390/s390-64/tls-macros.h b/sysdeps/s390/s390-64/tls-macros.h
+index be8aa6c..3c59436 100644
+--- a/sysdeps/s390/s390-64/tls-macros.h
++++ b/sysdeps/s390/s390-64/tls-macros.h
+@@ -8,12 +8,13 @@
+
+ #ifdef PIC
+ # define TLS_IE(x) \
+- ({ unsigned long __offset; \
+- asm ("bras %0,1f\n" \
+- "0:\t.quad " #x "@gotntpoff\n" \
+- "1:\tlg %0,0(%0)\n\t" \
+- "lg %0,0(%0,%%r12):tls_load:" #x \
+- : "=&a" (__offset) : : "cc" ); \
++ ({ unsigned long __offset, __got; \
++ asm ("bras %0,0f\n\t" \
++ ".quad " #x "@gotntpoff\n" \
++ "0:\tlarl %1,_GLOBAL_OFFSET_TABLE_\n\t" \
++ "lg %0,0(%0)\n\t" \
++ "lg %0,0(%0,%1):tls_load:" #x "\n" \
++ : "=&a" (__offset), "=&a" (__got) : : "cc" ); \
+ (int *) (__builtin_thread_pointer() + __offset); })
+ #else
+ # define TLS_IE(x) \
Modified: glibc-package/branches/eglibc-2.18/debian/patches/series
===================================================================
--- glibc-package/branches/eglibc-2.18/debian/patches/series 2013-12-01 12:43:29 UTC (rev 5842)
+++ glibc-package/branches/eglibc-2.18/debian/patches/series 2013-12-01 12:44:17 UTC (rev 5843)
@@ -181,6 +181,7 @@
powerpc/local-math-logb.diff
s390/submitted-nexttowardf.diff
+s390/cvs-s390-tls-got-pointer.diff
sparc/local-fork.diff
sparc/local-sparcv9-target.diff
Reply to: