]> git.ipfire.org Git - thirdparty/binutils-gdb.git/commit
objdump, readelf: apply relocations before textual dump users/ibhagat/try-pr32589
authorIndu Bhagat <indu.bhagat@oracle.com>
Tue, 4 Feb 2025 19:58:11 +0000 (11:58 -0800)
committerIndu Bhagat <indu.bhagat@oracle.com>
Thu, 6 Feb 2025 15:34:09 +0000 (07:34 -0800)
commit4e2182df14602eb9c65b1c1fb4685da8dc1ec9e7
tree216714125ca619cddc6a1109413e7f7a3f4e2542
parent078d38682291b06c44a3e091ad87c302d9385eb4
objdump, readelf: apply relocations before textual dump

PR libsframe/32589 - function start address is zero in SFrame section dump

Currently, readelf and objdump display SFrame section in object file
with function start addresses of each function as 0.  This makes it
difficult to correlate SFrame stack trace information with the
individual functions in the object file.

Use the dump_dwarf () interface to dump SFrame section.  The current
infrastructure (for DWARF debug sections) already supports relocating
the section contents before dumping, so lets use that.

Even after the section contents are relocated, there is need to fixup
the function start address, which is what sframe_fde_tbl_reloc_fixup ()
is about.

As a side effect, objdump now adds two new ways of dumping SFrame sections:
  - objdump -WS <obj>
  - objdump --dwarf=sframe
We do not publicize these options.  The lone advertised user interfacing
option (in --help) remains:
  - objdump --sframe

Furthermore, we continue to keep the same error messaging as earlier (by
folding the check for section into the new display_sframe_section ()
function):

$ objdump --sframe=sframe bubble_sort.o
...
No sframe section present

$ objdump --sframe=.sfram bubble_sort.o
...
No .sfram section present

Note the new API dump_sframe_reloc (). This new API is exposed because
objdump / readelf will need it.  Since this addition is backwards
compatible, update libtool-version with age+1 and revision+1.

TBD:
- Add tests where the fixup function is exercised. No explicit tests
for that yet on x86_64. Although the AArch64 one sort of suffices ?

binutils/
        * dwarf.c (display_sframe): New definition.
        (dwarf_select_sections_all): Enable SFrame section too.
        (struct dwarf_section_display): Add entry for SFrame section.
        * dwarf.h (enum dwarf_section_display_enum): Add enumerator for
SFrame.
        * objdump.c (dump_section_sframe): Remove.
        (dump_sframe_section): Add new definition.
        (dump_bfd): Use dump_sframe_section.
        * binutils/readelf.c (dump_section_as_sframe): Remove.

gas/testsuite/
        * gas/cfi-sframe/cfi-sframe-aarch64-pac-ab-key-1.d:
binutils/dwarf.c
binutils/dwarf.h
binutils/objdump.c
binutils/readelf.c
gas/testsuite/gas/cfi-sframe/cfi-sframe-aarch64-pac-ab-key-1.d