diff options
| author | Alan Modra <amodra@gmail.com> | 2020-06-09 09:32:10 +0930 |
|---|---|---|
| committer | Alan Modra <amodra@gmail.com> | 2020-06-09 09:37:23 +0930 |
| commit | efb2a7b412c2c78eaf6d3b63f153a749fcde292c (patch) | |
| tree | 9520362ef89c24e92b7c7a545778f700065aef45 /bfd/ChangeLog | |
| parent | c3cd66019e6ce9234909ea18331049eda56d76c2 (diff) | |
| download | gdb-efb2a7b412c2c78eaf6d3b63f153a749fcde292c.zip gdb-efb2a7b412c2c78eaf6d3b63f153a749fcde292c.tar.gz gdb-efb2a7b412c2c78eaf6d3b63f153a749fcde292c.tar.bz2 | |
PowerPC64: Downgrade ifunc with textrel error to a warning
For ppc64 I set flags when recording the dynamic relocation rather
than when allocating space. That allows you to distinguish three
cases:
1) The dynamic ifunc relocation is in an executable and will always be
to an ifunc resolver in the executable.
2) The dynamic ifunc relocation is in a shared library which provides
an ifunc resolver, but that may be overridden at runtime to use a
resolver in another binary.
3) The dynamic ifunc relocation is not to a locally defined ifunc
resolver.
Case (3) won't cause a segfault trying to run resolver code that is
non-exec on older glibc.
I made case (1) an error for ppc64, but since newer glibc ld.so does
allow running ifunc resolvers when segments are writable I suppose I
should downgrade that to a warning like case (2).
* elf64-ppc.c (struct ppc_link_hash_table): Delete
maybe_local_ifunc_resolver field.
(build_global_entry_stubs_and_plt): Set local_ifunc_resolver in
cases where maybe_local_ifunc_resolver was set.
(ppc64_elf_relocate_section): Likewise.
(ppc64_elf_finish_dynamic_sections): Downgrade ifunc with textrel
error to a warning.
Diffstat (limited to 'bfd/ChangeLog')
| -rw-r--r-- | bfd/ChangeLog | 10 |
1 files changed, 10 insertions, 0 deletions
diff --git a/bfd/ChangeLog b/bfd/ChangeLog index 7765dd3..146045b 100644 --- a/bfd/ChangeLog +++ b/bfd/ChangeLog @@ -1,3 +1,13 @@ +2020-06-09 Alan Modra <amodra@gmail.com> + + * elf64-ppc.c (struct ppc_link_hash_table): Delete + maybe_local_ifunc_resolver field. + (build_global_entry_stubs_and_plt): Set local_ifunc_resolver in + cases where maybe_local_ifunc_resolver was set. + (ppc64_elf_relocate_section): Likewise. + (ppc64_elf_finish_dynamic_sections): Downgrade ifunc with textrel + error to a warning. + 2020-06-08 H.J. Lu <hongjiu.lu@intel.com> * elf-bfd.h (elf_link_hash_entry): Add tlsdesc_plt and |