diff options
| author | Tamar Christina <tamar.christina@arm.com> | 2020-04-01 10:47:18 +0100 |
|---|---|---|
| committer | Tamar Christina <tamar.christina@arm.com> | 2020-04-01 10:52:33 +0100 |
| commit | a7618269b727da9cf56727c22cb538ff5f0e4d25 (patch) | |
| tree | 87615d7d18619600f5bfc60c2b3c924ac0ccdb76 /ld | |
| parent | 15ccbdd717530f81f545a716f0df1de62aee1157 (diff) | |
| download | binutils-a7618269b727da9cf56727c22cb538ff5f0e4d25.zip binutils-a7618269b727da9cf56727c22cb538ff5f0e4d25.tar.gz binutils-a7618269b727da9cf56727c22cb538ff5f0e4d25.tar.bz2 | |
Arm: Fix LSB of GOT for Thumb2 only PLT.
When you have a Thumb only PLT then the address in the GOT for PLT0 needs to
have the Thumb bit set since the instruction used in PLTn to get there is
`ldr.w pc` which is an inter-working instruction:
the PLT sequence in question is
00000120 <foo@plt>:
120: f240 0c98 movw ip, #152 ; 0x98
124: f2c0 0c01 movt ip, #1
128: 44fc add ip, pc
12a: f8dc f000 ldr.w pc, [ip]
12e: e7fc b.n 12a <foo@plt+0xa>
Disassembly of section .text:
00000130 <bar>:
130: b580 push {r7, lr}
132: af00 add r7, sp, #0
134: f7ff fff4 bl 120 <foo@plt>
and previously the linker would generate
Hex dump of section '.got':
...
0x000101b8 40010100 00000000 00000000 10010000 @...............
Which would make it jump and transition out of thumb mode and crash since you
only have thumb mode on such cores.
Now it correctly generates
Hex dump of section '.got':
...
0x000101b8 40010100 00000000 00000000 11010000 @...............
Thanks to Amol for testing patch and to rgujju for reporting it.
bfd/ChangeLog:
PR ld/16017
* elf32-arm.c (elf32_arm_populate_plt_entry): Set LSB of the PLT0
address in the GOT if in thumb only mode.
ld/ChangeLog:
PR ld/16017
* testsuite/ld-arm/arm-elf.exp (thumb-plt-got): New.
* testsuite/ld-arm/thumb-plt-got.d: New test.
Diffstat (limited to 'ld')
| -rw-r--r-- | ld/ChangeLog | 6 | ||||
| -rw-r--r-- | ld/testsuite/ld-arm/arm-elf.exp | 1 | ||||
| -rw-r--r-- | ld/testsuite/ld-arm/thumb-plt-got.d | 14 |
3 files changed, 21 insertions, 0 deletions
diff --git a/ld/ChangeLog b/ld/ChangeLog index ef4045a..16ffc30 100644 --- a/ld/ChangeLog +++ b/ld/ChangeLog @@ -1,5 +1,11 @@ 2020-04-01 Tamar Christina <tamar.christina@arm.com> + PR ld/16017 + * testsuite/ld-arm/arm-elf.exp (thumb-plt-got): New. + * testsuite/ld-arm/thumb-plt-got.d: New test. + +2020-04-01 Tamar Christina <tamar.christina@arm.com> + * testsuite/ld-arm/arm-elf.exp (thumb-plt): New. * testsuite/ld-arm/thumb-plt.d: New test. * testsuite/ld-arm/thumb-plt.s: New test. diff --git a/ld/testsuite/ld-arm/arm-elf.exp b/ld/testsuite/ld-arm/arm-elf.exp index 99a3139..59e68de 100644 --- a/ld/testsuite/ld-arm/arm-elf.exp +++ b/ld/testsuite/ld-arm/arm-elf.exp @@ -1270,3 +1270,4 @@ run_dump_test "non-contiguous-arm5" run_dump_test "non-contiguous-arm6" run_dump_test "thumb-plt" +run_dump_test "thumb-plt-got" diff --git a/ld/testsuite/ld-arm/thumb-plt-got.d b/ld/testsuite/ld-arm/thumb-plt-got.d new file mode 100644 index 0000000..e65aba9 --- /dev/null +++ b/ld/testsuite/ld-arm/thumb-plt-got.d @@ -0,0 +1,14 @@ +#source: thumb-plt.s +#name: Thumb only PLT and GOT LSB Symbol +#ld: -shared -e0 +#readelf: -rx .got +#skip: *-*-pe *-*-wince *-*-vxworks armeb-*-* *-*-gnueabihf + +Relocation section '.rel.plt' at offset 0x108 contains 1 entry: + Offset Info Type Sym.Value Sym. Name +000101c4 00000116 R_ARM_JUMP_SLOT 00000000 foo + +Hex dump of section '.got': + NOTE: This section has relocations against it, but these have NOT been applied to this dump. + 0x000101b8 40010100 00000000 00000000 11010000 @............... + |