diff options
author | Richard Purdie <richard.purdie@linuxfoundation.org> | 2014-02-08 20:56:46 +0000 |
---|---|---|
committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2014-02-09 09:40:00 +0000 |
commit | 4a85e2698e824fceeecaf4fb98bf6c7047f6ae93 (patch) | |
tree | 915427aa64f9c03b50f697544936d2510db967b7 /meta/recipes-devtools | |
parent | c7ee4de62bc1542450e2048f87822788d85b1308 (diff) | |
download | poky-4a85e2698e824fceeecaf4fb98bf6c7047f6ae93.tar.gz |
binutils: Add backports to resolve systemd (from git) breakage
Backport the patches for PR2404 and PR16476 from binutils to
resolve failures when compiling systemd from git.
(From OE-Core rev: 25e376f6c8c66cca95ae7fa7f512b7dadfdd6140)
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'meta/recipes-devtools')
-rw-r--r-- | meta/recipes-devtools/binutils/binutils-2.24.inc | 2 | ||||
-rw-r--r-- | meta/recipes-devtools/binutils/binutils/fix-pr16476.patch | 220 | ||||
-rw-r--r-- | meta/recipes-devtools/binutils/binutils/fix-pr2404.patch | 76 |
3 files changed, 298 insertions, 0 deletions
diff --git a/meta/recipes-devtools/binutils/binutils-2.24.inc b/meta/recipes-devtools/binutils/binutils-2.24.inc index 86ff9999c0..42d4ede229 100644 --- a/meta/recipes-devtools/binutils/binutils-2.24.inc +++ b/meta/recipes-devtools/binutils/binutils-2.24.inc | |||
@@ -26,6 +26,8 @@ SRC_URI = "\ | |||
26 | file://mips64-default-ld-emulation.patch \ | 26 | file://mips64-default-ld-emulation.patch \ |
27 | file://binutils-xlp-support.patch \ | 27 | file://binutils-xlp-support.patch \ |
28 | file://fix-pr15815.patch \ | 28 | file://fix-pr15815.patch \ |
29 | file://fix-pr2404.patch \ | ||
30 | file://fix-pr16476.patch \ | ||
29 | " | 31 | " |
30 | 32 | ||
31 | SRC_URI[md5sum] = "e0f71a7b2ddab0f8612336ac81d9636b" | 33 | SRC_URI[md5sum] = "e0f71a7b2ddab0f8612336ac81d9636b" |
diff --git a/meta/recipes-devtools/binutils/binutils/fix-pr16476.patch b/meta/recipes-devtools/binutils/binutils/fix-pr16476.patch new file mode 100644 index 0000000000..65d62bcde4 --- /dev/null +++ b/meta/recipes-devtools/binutils/binutils/fix-pr16476.patch | |||
@@ -0,0 +1,220 @@ | |||
1 | From: H.J. Lu <hjl.tools@gmail.com> | ||
2 | Date: Tue, 21 Jan 2014 23:42:43 +0000 (-0800) | ||
3 | Subject: Check incompatible existing default symbol definition | ||
4 | X-Git-Tag: hjl/linux/release/2.24.51.0.3~1^2~8^2~16 | ||
5 | X-Git-Url: https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;a=commitdiff_plain;h=4584ec12076e088cf36965b88ef8710ca85491f9 | ||
6 | |||
7 | Check incompatible existing default symbol definition | ||
8 | |||
9 | After resolving a versioned reference, foo@VER1, to a default versioned | ||
10 | definition, foo@@VER1, from a shared object, we also merge it with | ||
11 | the existing regular default symbol definition, foo. When foo is IFUNC | ||
12 | and foo@@VER1 aren't, we will merge 2 incompatible definitions. This | ||
13 | patch avoids merging foo@@VER1 definition with foo definition if | ||
14 | one is IFUNC and the other isn't. | ||
15 | |||
16 | Upstream-Status: Backport | ||
17 | --- | ||
18 | |||
19 | #diff --git a/bfd/ChangeLog b/bfd/ChangeLog | ||
20 | #index 5923bc3..c70a7db 100644 | ||
21 | #--- a/bfd/ChangeLog | ||
22 | #+++ b/bfd/ChangeLog | ||
23 | #@@ -1,5 +1,13 @@ | ||
24 | # 2014-01-21 H.J. Lu <hongjiu.lu@intel.com> | ||
25 | # | ||
26 | #+ PR ld/16467 | ||
27 | #+ * elflink.c (_bfd_elf_merge_symbol): When types of the existing | ||
28 | #+ regular default symbol definition and the versioned dynamic | ||
29 | #+ symbol definition mismatch, skip the default symbol definition | ||
30 | #+ if one of them is IFUNC. | ||
31 | #+ | ||
32 | #+2014-01-21 H.J. Lu <hongjiu.lu@intel.com> | ||
33 | #+ | ||
34 | # PR ld/2404 | ||
35 | # * elflink.c (_bfd_elf_merge_symbol): Don't check info->shared, | ||
36 | # info->export_dynamic nor h->ref_dynamic for type mismatch when | ||
37 | diff --git a/bfd/elflink.c b/bfd/elflink.c | ||
38 | index d0006da..792e14e 100644 | ||
39 | --- a/bfd/elflink.c | ||
40 | +++ b/bfd/elflink.c | ||
41 | @@ -1092,11 +1092,14 @@ _bfd_elf_merge_symbol (bfd *abfd, | ||
42 | && newdyn | ||
43 | && newdef | ||
44 | && !olddyn | ||
45 | - && (olddef || h->root.type == bfd_link_hash_common) | ||
46 | - && ELF_ST_TYPE (sym->st_info) != h->type | ||
47 | - && ELF_ST_TYPE (sym->st_info) != STT_NOTYPE | ||
48 | - && h->type != STT_NOTYPE | ||
49 | - && !(newfunc && oldfunc)) | ||
50 | + && (((olddef || h->root.type == bfd_link_hash_common) | ||
51 | + && ELF_ST_TYPE (sym->st_info) != h->type | ||
52 | + && ELF_ST_TYPE (sym->st_info) != STT_NOTYPE | ||
53 | + && h->type != STT_NOTYPE | ||
54 | + && !(newfunc && oldfunc)) | ||
55 | + || (olddef | ||
56 | + && ((h->type == STT_GNU_IFUNC) | ||
57 | + != (ELF_ST_TYPE (sym->st_info) == STT_GNU_IFUNC))))) | ||
58 | { | ||
59 | *skip = TRUE; | ||
60 | return TRUE; | ||
61 | #diff --git a/ld/testsuite/ChangeLog b/ld/testsuite/ChangeLog | ||
62 | #index ac65a3a..a092428 100644 | ||
63 | #--- a/ld/testsuite/ChangeLog | ||
64 | #+++ b/ld/testsuite/ChangeLog | ||
65 | #@@ -1,5 +1,19 @@ | ||
66 | # 2014-01-21 H.J. Lu <hongjiu.lu@intel.com> | ||
67 | # | ||
68 | #+ PR ld/16467 | ||
69 | #+ * ld-ifunc/dummy.c: New file. | ||
70 | #+ * ld-ifunc/pr16467.out: Likewise. | ||
71 | ##+ * ld-ifunc/pr16467a.c: Likewise. | ||
72 | #+ * ld-ifunc/pr16467a.map: Likewise. | ||
73 | #+ * ld-ifunc/pr16467b.c: Likewise. | ||
74 | #+ * ld-ifunc/pr16467b.map: Likewise. | ||
75 | #+ * ld-ifunc/pr16467c.c: Likewise. | ||
76 | #+ | ||
77 | #+ * ld-ifunc/ifunc.exp (run_cc_link_tests): New. | ||
78 | #+ (run_ld_link_exec_tests): Run pr16467. | ||
79 | #+ | ||
80 | #+2014-01-21 H.J. Lu <hongjiu.lu@intel.com> | ||
81 | #+ | ||
82 | # PR ld/2404 | ||
83 | # * ld-elf/shared.exp: Add a PIE test for PR ld/2404. | ||
84 | # | ||
85 | diff --git a/ld/testsuite/ld-ifunc/dummy.c b/ld/testsuite/ld-ifunc/dummy.c | ||
86 | new file mode 100644 | ||
87 | index 0000000..5c03287 | ||
88 | --- /dev/null | ||
89 | +++ b/ld/testsuite/ld-ifunc/dummy.c | ||
90 | @@ -0,0 +1 @@ | ||
91 | +/* An empty file. */ | ||
92 | diff --git a/ld/testsuite/ld-ifunc/ifunc.exp b/ld/testsuite/ld-ifunc/ifunc.exp | ||
93 | index fb106c6..d7ff445 100644 | ||
94 | --- a/ld/testsuite/ld-ifunc/ifunc.exp | ||
95 | +++ b/ld/testsuite/ld-ifunc/ifunc.exp | ||
96 | @@ -349,6 +349,42 @@ if { $verbose < 1 } { | ||
97 | remote_file host delete "tmpdir/static_nonifunc_prog" | ||
98 | } | ||
99 | |||
100 | +run_cc_link_tests [list \ | ||
101 | + [list \ | ||
102 | + "Build libpr16467a.so" \ | ||
103 | + "-shared -Wl,--version-script=pr16467a.map" \ | ||
104 | + "-fPIC" \ | ||
105 | + { pr16467a.c } \ | ||
106 | + {} \ | ||
107 | + "libpr16467a.so" \ | ||
108 | + ] \ | ||
109 | + [list \ | ||
110 | + "Build libpr16467b.a" \ | ||
111 | + "" \ | ||
112 | + "-fPIC" \ | ||
113 | + { pr16467b.c } \ | ||
114 | + {} \ | ||
115 | + "libpr16467b.a" \ | ||
116 | + ] \ | ||
117 | + [list \ | ||
118 | + "Build libpr16467b.so" \ | ||
119 | + "-shared tmpdir/pr16467b.o tmpdir/libpr16467a.so \ | ||
120 | + -Wl,--version-script=pr16467b.map" \ | ||
121 | + "-fPIC" \ | ||
122 | + { dummy.c } \ | ||
123 | + {} \ | ||
124 | + "libpr16467b.so" \ | ||
125 | + ] \ | ||
126 | + [list \ | ||
127 | + "Build libpr16467c.a" \ | ||
128 | + "" \ | ||
129 | + "" \ | ||
130 | + { pr16467c.c } \ | ||
131 | + {} \ | ||
132 | + "libpr16467c.a" \ | ||
133 | + ] \ | ||
134 | +] | ||
135 | + | ||
136 | run_ld_link_exec_tests [] [list \ | ||
137 | [list \ | ||
138 | "Common symbol override ifunc test 1a" \ | ||
139 | @@ -368,6 +404,15 @@ run_ld_link_exec_tests [] [list \ | ||
140 | "ifunc-common-1.out" \ | ||
141 | "-g" \ | ||
142 | ] \ | ||
143 | + [list \ | ||
144 | + "Run pr16467" \ | ||
145 | + "tmpdir/pr16467c.o tmpdir/libpr16467b.so tmpdir/libpr16467a.so" \ | ||
146 | + "" \ | ||
147 | + { dummy.c } \ | ||
148 | + "pr16467" \ | ||
149 | + "pr16467.out" \ | ||
150 | + "" \ | ||
151 | + ] \ | ||
152 | ] | ||
153 | |||
154 | set test_list [lsort [glob -nocomplain $srcdir/$subdir/*.d]] | ||
155 | diff --git a/ld/testsuite/ld-ifunc/pr16467.out b/ld/testsuite/ld-ifunc/pr16467.out | ||
156 | new file mode 100644 | ||
157 | index 0000000..d86bac9 | ||
158 | --- /dev/null | ||
159 | +++ b/ld/testsuite/ld-ifunc/pr16467.out | ||
160 | @@ -0,0 +1 @@ | ||
161 | +OK | ||
162 | diff --git a/ld/testsuite/ld-ifunc/pr16467a.c b/ld/testsuite/ld-ifunc/pr16467a.c | ||
163 | new file mode 100644 | ||
164 | index 0000000..ae3f084 | ||
165 | --- /dev/null | ||
166 | +++ b/ld/testsuite/ld-ifunc/pr16467a.c | ||
167 | @@ -0,0 +1,5 @@ | ||
168 | +const char * | ||
169 | +sd_get_seats(void) | ||
170 | +{ | ||
171 | + return "OK"; | ||
172 | +} | ||
173 | diff --git a/ld/testsuite/ld-ifunc/pr16467a.map b/ld/testsuite/ld-ifunc/pr16467a.map | ||
174 | new file mode 100644 | ||
175 | index 0000000..d677f37 | ||
176 | --- /dev/null | ||
177 | +++ b/ld/testsuite/ld-ifunc/pr16467a.map | ||
178 | @@ -0,0 +1,4 @@ | ||
179 | +LIBSYSTEMD_209 { | ||
180 | +global: | ||
181 | + sd_get_seats; | ||
182 | +}; | ||
183 | diff --git a/ld/testsuite/ld-ifunc/pr16467b.c b/ld/testsuite/ld-ifunc/pr16467b.c | ||
184 | new file mode 100644 | ||
185 | index 0000000..264f6cf | ||
186 | --- /dev/null | ||
187 | +++ b/ld/testsuite/ld-ifunc/pr16467b.c | ||
188 | @@ -0,0 +1,7 @@ | ||
189 | +void new_sd_get_seats(void); | ||
190 | +__asm__(".symver new_sd_get_seats,sd_get_seats@LIBSYSTEMD_209"); | ||
191 | +void (*resolve_sd_get_seats(void)) (void) __asm__ ("sd_get_seats"); | ||
192 | +void (*resolve_sd_get_seats(void)) (void) { | ||
193 | + return new_sd_get_seats; | ||
194 | +} | ||
195 | +__asm__(".type sd_get_seats, %gnu_indirect_function"); | ||
196 | diff --git a/ld/testsuite/ld-ifunc/pr16467b.map b/ld/testsuite/ld-ifunc/pr16467b.map | ||
197 | new file mode 100644 | ||
198 | index 0000000..1f263de | ||
199 | --- /dev/null | ||
200 | +++ b/ld/testsuite/ld-ifunc/pr16467b.map | ||
201 | @@ -0,0 +1,4 @@ | ||
202 | +LIBSYSTEMD_208 { | ||
203 | +global: | ||
204 | + sd_get_seats; | ||
205 | +}; | ||
206 | diff --git a/ld/testsuite/ld-ifunc/pr16467c.c b/ld/testsuite/ld-ifunc/pr16467c.c | ||
207 | new file mode 100644 | ||
208 | index 0000000..e2a901c | ||
209 | --- /dev/null | ||
210 | +++ b/ld/testsuite/ld-ifunc/pr16467c.c | ||
211 | @@ -0,0 +1,9 @@ | ||
212 | +#include <stdio.h> | ||
213 | +const char* sd_get_seats(void); | ||
214 | + | ||
215 | +int | ||
216 | +main (int argc, char **argv) | ||
217 | +{ | ||
218 | + printf("%s\n", sd_get_seats()); | ||
219 | + return 0; | ||
220 | +} | ||
diff --git a/meta/recipes-devtools/binutils/binutils/fix-pr2404.patch b/meta/recipes-devtools/binutils/binutils/fix-pr2404.patch new file mode 100644 index 0000000000..41f35046fa --- /dev/null +++ b/meta/recipes-devtools/binutils/binutils/fix-pr2404.patch | |||
@@ -0,0 +1,76 @@ | |||
1 | From: H.J. Lu <hjl.tools@gmail.com> | ||
2 | Date: Tue, 21 Jan 2014 13:33:48 +0000 (-0800) | ||
3 | Subject: Don't check shared/export_dynamic/ref_dynamic for type mismatch | ||
4 | X-Git-Tag: hjl/linux/release/2.24.51.0.3~1^2~8^2~22 | ||
5 | X-Git-Url: https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;a=commitdiff_plain;h=22ef172a21663abb477d72284b4d59c8eabcbb82 | ||
6 | |||
7 | Don't check shared/export_dynamic/ref_dynamic for type mismatch | ||
8 | |||
9 | There is nothing linker can do when a type mismatched default definition | ||
10 | are made dynamic by info->shared, info->export_dynamic or h->ref_dynamic. | ||
11 | But we do want to avoid exporting it when building PIE. Let's remove | ||
12 | those checks. | ||
13 | |||
14 | bfd/ | ||
15 | |||
16 | PR ld/2404 | ||
17 | * elflink.c (_bfd_elf_merge_symbol): Don't check info->shared, | ||
18 | info->export_dynamic, nor !h->ref_dynamic for type mismatch when | ||
19 | adding the default version. | ||
20 | |||
21 | ld/testsuite/ | ||
22 | |||
23 | PR ld/2404 | ||
24 | * ld-elf/shared.exp: Add a PIE test for PR ld/2404. | ||
25 | |||
26 | Upstream-Status: Backport | ||
27 | --- | ||
28 | |||
29 | Index: binutils-2.24/bfd/elflink.c | ||
30 | =================================================================== | ||
31 | --- binutils-2.24.orig/bfd/elflink.c 2014-02-08 13:20:08.628378267 +0000 | ||
32 | +++ binutils-2.24/bfd/elflink.c 2014-02-08 13:20:08.624378267 +0000 | ||
33 | @@ -1090,9 +1090,6 @@ | ||
34 | the type of existing regular definition mismatch. We only do it | ||
35 | if the existing regular definition won't be dynamic. */ | ||
36 | if (pold_alignment == NULL | ||
37 | - && !info->shared | ||
38 | - && !info->export_dynamic | ||
39 | - && !h->ref_dynamic | ||
40 | && newdyn | ||
41 | && newdef | ||
42 | && !olddyn | ||
43 | Index: binutils-2.24/ld/testsuite/ld-elf/shared.exp | ||
44 | =================================================================== | ||
45 | --- binutils-2.24.orig/ld/testsuite/ld-elf/shared.exp 2014-02-08 13:20:08.628378267 +0000 | ||
46 | +++ binutils-2.24/ld/testsuite/ld-elf/shared.exp 2014-02-08 13:20:08.624378267 +0000 | ||
47 | @@ -432,3 +432,29 @@ | ||
48 | |||
49 | run_cc_link_tests $build_cxx_tests | ||
50 | run_ld_link_exec_tests [] $run_cxx_tests | ||
51 | + | ||
52 | +if { [istarget *-*-linux*] | ||
53 | + || [istarget *-*-nacl*] | ||
54 | + || [istarget *-*-gnu*] } { | ||
55 | + run_cc_link_tests [list \ | ||
56 | + [list \ | ||
57 | + "Build libpr2404b.a with PIE" \ | ||
58 | + "" \ | ||
59 | + "-fPIE" \ | ||
60 | + { pr2404b.c } \ | ||
61 | + {} \ | ||
62 | + "libpr2404b.a" \ | ||
63 | + ] \ | ||
64 | + ] | ||
65 | + run_ld_link_exec_tests [] [list \ | ||
66 | + [list \ | ||
67 | + "Run pr2404 with PIE" \ | ||
68 | + "-pie tmpdir/pr2404b.o tmpdir/libpr2404a.so" \ | ||
69 | + "" \ | ||
70 | + { dummy.c } \ | ||
71 | + "pr2404pie" \ | ||
72 | + "pr2404.out" \ | ||
73 | + "-fPIE" \ | ||
74 | + ] \ | ||
75 | + ] | ||
76 | +} | ||