From: Renlin Li Date: Sat, 3 Feb 2018 13:18:17 +0000 (+0000) Subject: [PR22764][LD][AARCH64]Allow R_AARCH64_ABS16 and R_AARCH64_ABS32 against absolution... X-Git-Tag: users/ARM/embedded-binutils-2_30-branch-2018q2~79 X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=b01452b1d44a586f4ecf5cf02ffc0643e4962324;p=thirdparty%2Fbinutils-gdb.git [PR22764][LD][AARCH64]Allow R_AARCH64_ABS16 and R_AARCH64_ABS32 against absolution symbol or undefine symbol in shared object. backport from mainline bfd/ 2018-02-05 Renlin Li PR ld/22764 * elfnn-aarch64.c (elfNN_aarch64_check_relocs): Relax the R_AARCH64_ABS32 and R_AARCH64_ABS16 for absolute symbol. Apply the check for writeable section as well. ld/ 2018-02-05 Renlin Li PR ld/22764 * testsuite/ld-aarch64/emit-relocs-258.s: Define symbol as an address. * testsuite/ld-aarch64/emit-relocs-259.s: Likewise. * testsuite/ld-aarch64/aarch64-elf.exp: Run new test. * testsuite/ld-aarch64/pr22764.s: New. * testsuite/ld-aarch64/pr22764.d: New. --- diff --git a/bfd/ChangeLog b/bfd/ChangeLog index 92e26a8aa23..9fe447a29e5 100644 --- a/bfd/ChangeLog +++ b/bfd/ChangeLog @@ -1,3 +1,10 @@ +2018-02-05 Renlin Li + + PR ld/22764 + * elfnn-aarch64.c (elfNN_aarch64_check_relocs): Relax the + R_AARCH64_ABS32 and R_AARCH64_ABS16 for absolute symbol. Apply the + check for writeable section as well. + 2018-01-27 Nick Clifton Back to development. diff --git a/bfd/elfnn-aarch64.c b/bfd/elfnn-aarch64.c index d5711e0eb1d..973188220ba 100644 --- a/bfd/elfnn-aarch64.c +++ b/bfd/elfnn-aarch64.c @@ -7074,10 +7074,19 @@ elfNN_aarch64_check_relocs (bfd *abfd, struct bfd_link_info *info, #if ARCH_SIZE == 64 case BFD_RELOC_AARCH64_32: #endif - if (bfd_link_pic (info) - && (sec->flags & SEC_ALLOC) != 0 - && (sec->flags & SEC_READONLY) != 0) + if (bfd_link_pic (info) && (sec->flags & SEC_ALLOC) != 0) { + if (h != NULL + /* This is an absolute symbol. It represents a value instead + of an address. */ + && ((h->root.type == bfd_link_hash_defined + && bfd_is_abs_section (h->root.u.def.section)) + /* This is an undefined symbol. */ + || h->root.type == bfd_link_hash_undefined)) + break; + + /* For local symbols, defined global symbols in a non-ABS section, + it is assumed that the value is an address. */ int howto_index = bfd_r_type - BFD_RELOC_AARCH64_RELOC_START; _bfd_error_handler /* xgettext:c-format */ diff --git a/ld/ChangeLog b/ld/ChangeLog index ddc1d88f3c1..e871cef7c07 100644 --- a/ld/ChangeLog +++ b/ld/ChangeLog @@ -1,3 +1,11 @@ +2018-02-05 Renlin Li + + PR ld/22764 + * testsuite/ld-aarch64/emit-relocs-258.s: Define symbol as an address. + * testsuite/ld-aarch64/emit-relocs-259.s: Likewise. + * testsuite/ld-aarch64/pr22764.s: New. + * testsuite/ld-aarch64/pr22764.d: New. + 2018-02-05 Nick Clifton * po/pt_BR.po: Updated Brazilian Portuguese translation. diff --git a/ld/testsuite/ld-aarch64/aarch64-elf.exp b/ld/testsuite/ld-aarch64/aarch64-elf.exp index f31089361b8..d766f3736bc 100644 --- a/ld/testsuite/ld-aarch64/aarch64-elf.exp +++ b/ld/testsuite/ld-aarch64/aarch64-elf.exp @@ -275,6 +275,7 @@ run_dump_test "pr17415" run_dump_test_lp64 "tprel_g2_overflow" run_dump_test "tprel_add_lo12_overflow" run_dump_test "protected-data" +run_dump_test_lp64 "pr22764" # ifunc tests run_dump_test "ifunc-1" diff --git a/ld/testsuite/ld-aarch64/emit-relocs-258.s b/ld/testsuite/ld-aarch64/emit-relocs-258.s index f7247762431..87bb657c5d9 100644 --- a/ld/testsuite/ld-aarch64/emit-relocs-258.s +++ b/ld/testsuite/ld-aarch64/emit-relocs-258.s @@ -1,5 +1,6 @@ +.global dummy .text - +dummy: ldr x0, .L1 .L1: diff --git a/ld/testsuite/ld-aarch64/emit-relocs-259.s b/ld/testsuite/ld-aarch64/emit-relocs-259.s index 7e1ba3ceb40..0977c9d869f 100644 --- a/ld/testsuite/ld-aarch64/emit-relocs-259.s +++ b/ld/testsuite/ld-aarch64/emit-relocs-259.s @@ -1,5 +1,6 @@ +.global dummy .text - +dummy: ldr x0, .L1 .L1: diff --git a/ld/testsuite/ld-aarch64/pr22764.d b/ld/testsuite/ld-aarch64/pr22764.d new file mode 100644 index 00000000000..997519f4699 --- /dev/null +++ b/ld/testsuite/ld-aarch64/pr22764.d @@ -0,0 +1,18 @@ +#source: pr22764.s +#ld: -shared -T relocs.ld -defsym sym_abs1=0x1 -defsym sym_abs2=0x2 -defsym sym_abs3=0x3 -e0 --emit-relocs +#notarget: aarch64_be-*-* +#objdump: -dr +#... + +Disassembly of section \.text: + +0000000000010000 \<\.text\>: + 10000: d503201f nop + ... + 10004: R_AARCH64_ABS64 sym_abs1 + 1000c: 00000002 \.word 0x00000002 + 1000c: R_AARCH64_ABS32 sym_abs2 + 10010: 0003 \.short 0x0003 + 10010: R_AARCH64_ABS16 sym_abs3 + 10012: 0000 \.short 0x0000 + 10014: d503201f nop diff --git a/ld/testsuite/ld-aarch64/pr22764.s b/ld/testsuite/ld-aarch64/pr22764.s new file mode 100644 index 00000000000..25e36b4a129 --- /dev/null +++ b/ld/testsuite/ld-aarch64/pr22764.s @@ -0,0 +1,6 @@ + .text + nop + .xword sym_abs1 + .word sym_abs2 + .short sym_abs3 + nop