diff options
author | Khem Raj <raj.khem@gmail.com> | 2021-12-01 22:19:33 -0800 |
---|---|---|
committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2021-12-03 23:37:16 +0000 |
commit | b1984dd2f5096fca7b7a8b93a45143825b168bda (patch) | |
tree | 94ec1caabf7ee74975e503413c3973e939d76312 /meta | |
parent | 5c8c4dd925bbb9d29014ffd717ffe69ebb3553e3 (diff) | |
download | poky-b1984dd2f5096fca7b7a8b93a45143825b168bda.tar.gz |
glibc: Drop patch to support/workaround prelinked apps on armv5
The usecase explained in bug #1443 works fine now a days on qemuarmv5,
tested by using lltng-ust and explicitly linking in liburcu-bp.so as
well, since its no more a direct dependency of liblttng-ust.so.1
Given that usecase works, unbolt this fix now.
(From OE-Core rev: 088cf642e4a58fd50f93b22b4fdf5a2f25e1ed53)
Signed-off-by: Khem Raj <raj.khem@gmail.com>
Cc: Mark Hatle <mark.hatle@kernel.crashing.org>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'meta')
-rw-r--r-- | meta/recipes-core/glibc/glibc/0012-Quote-from-bug-1443-which-explains-what-the-patch-do.patch | 58 | ||||
-rw-r--r-- | meta/recipes-core/glibc/glibc_2.34.bb | 1 |
2 files changed, 0 insertions, 59 deletions
diff --git a/meta/recipes-core/glibc/glibc/0012-Quote-from-bug-1443-which-explains-what-the-patch-do.patch b/meta/recipes-core/glibc/glibc/0012-Quote-from-bug-1443-which-explains-what-the-patch-do.patch deleted file mode 100644 index 07d4411c61..0000000000 --- a/meta/recipes-core/glibc/glibc/0012-Quote-from-bug-1443-which-explains-what-the-patch-do.patch +++ /dev/null | |||
@@ -1,58 +0,0 @@ | |||
1 | From add514edf4299d1bf540d85d0aa0bd5fe0d46b78 Mon Sep 17 00:00:00 2001 | ||
2 | From: Khem Raj <raj.khem@gmail.com> | ||
3 | Date: Wed, 18 Mar 2015 00:20:09 +0000 | ||
4 | Subject: [PATCH] Quote from bug 1443 which explains what the patch does : | ||
5 | |||
6 | We build some random program and link it with -lust. When we run it, | ||
7 | it dies with a SIGSEGV before reaching main(). | ||
8 | |||
9 | Libust.so depends on liburcu-bp.so from the usermode-rcu package. | ||
10 | Although libust.so is not prelinked, liburcu-bp.so IS prelinked; this | ||
11 | is critical. | ||
12 | |||
13 | Libust.so uses a TLS / __thread variable that is defined in liburcu- | ||
14 | bp.so. There are special ARM-specific relocation types that allow two | ||
15 | shared libraries to share thread-specific data. This is critical too. | ||
16 | |||
17 | One more critical issue: although liburcu-bp.so is prelinked, we can't | ||
18 | load it at its prelinked address, because we also link against | ||
19 | librt.so, and librt.so uses that address. | ||
20 | |||
21 | The dynamic linker is forced to relink liburcu-bp.so at a different | ||
22 | address. In the course of relinking, it processes the special ARM | ||
23 | relocation record mentioned above. The prelinker has already filled | ||
24 | in the information, which is a short offset into a table of thread- | ||
25 | specific data that is allocated per-thread for each library that uses | ||
26 | TLS. Because the normal behavior of a relocation is to add the symbol | ||
27 | value to an addend stored at the address being relocated, we end up | ||
28 | adding the short offset to itself, doubling it. | ||
29 | |||
30 | Now we have an awkward situation. The libust.so library doesn't know | ||
31 | about the addend, so its TLS data for this element is correct. The | ||
32 | liburcu-bp.so library has a different offset for the element. When we | ||
33 | go to initialize the element for the first time in liburcu-bp.so, we | ||
34 | write the address of the result at the doubled (broken) offset. | ||
35 | Later, when we refer to the address from libust.so, we check the value | ||
36 | at the correct offset, but it's NULL, so we eat hot SIGSEGV. | ||
37 | |||
38 | Upstream-Status: Pending | ||
39 | |||
40 | Signed-off-by: Andrei Dinu <andrei.adrianx.dinu@intel.com> | ||
41 | Signed-off-by: Khem Raj <raj.khem@gmail.com> | ||
42 | --- | ||
43 | sysdeps/arm/dl-machine.h | 2 +- | ||
44 | 1 file changed, 1 insertion(+), 1 deletion(-) | ||
45 | |||
46 | diff --git a/sysdeps/arm/dl-machine.h b/sysdeps/arm/dl-machine.h | ||
47 | index ff5e09e207..d68bfe5cbe 100644 | ||
48 | --- a/sysdeps/arm/dl-machine.h | ||
49 | +++ b/sysdeps/arm/dl-machine.h | ||
50 | @@ -510,7 +510,7 @@ elf_machine_rel (struct link_map *map, const Elf32_Rel *reloc, | ||
51 | |||
52 | case R_ARM_TLS_DTPOFF32: | ||
53 | if (sym != NULL) | ||
54 | - *reloc_addr += sym->st_value; | ||
55 | + *reloc_addr = sym->st_value; | ||
56 | break; | ||
57 | |||
58 | case R_ARM_TLS_TPOFF32: | ||
diff --git a/meta/recipes-core/glibc/glibc_2.34.bb b/meta/recipes-core/glibc/glibc_2.34.bb index 7efc1ec1ef..5529dc5aa3 100644 --- a/meta/recipes-core/glibc/glibc_2.34.bb +++ b/meta/recipes-core/glibc/glibc_2.34.bb | |||
@@ -37,7 +37,6 @@ SRC_URI = "${GLIBC_GIT_URI};branch=${SRCBRANCH};name=glibc \ | |||
37 | file://0009-fsl-e500-e5500-e6500-603e-fsqrt-implementation.patch \ | 37 | file://0009-fsl-e500-e5500-e6500-603e-fsqrt-implementation.patch \ |
38 | file://0010-ppc-sqrt-Fix-undefined-reference-to-__sqrt_finite.patch \ | 38 | file://0010-ppc-sqrt-Fix-undefined-reference-to-__sqrt_finite.patch \ |
39 | file://0011-__ieee754_sqrt-f-are-now-inline-functions-and-call-o.patch \ | 39 | file://0011-__ieee754_sqrt-f-are-now-inline-functions-and-call-o.patch \ |
40 | file://0012-Quote-from-bug-1443-which-explains-what-the-patch-do.patch \ | ||
41 | file://0013-eglibc-run-libm-err-tab.pl-with-specific-dirs-in-S.patch \ | 40 | file://0013-eglibc-run-libm-err-tab.pl-with-specific-dirs-in-S.patch \ |
42 | file://0014-__ieee754_sqrt-f-are-now-inline-functions-and-call-o.patch \ | 41 | file://0014-__ieee754_sqrt-f-are-now-inline-functions-and-call-o.patch \ |
43 | file://0015-sysdeps-gnu-configure.ac-handle-correctly-libc_cv_ro.patch \ | 42 | file://0015-sysdeps-gnu-configure.ac-handle-correctly-libc_cv_ro.patch \ |