]> git.ipfire.org Git - thirdparty/binutils-gdb.git/commit
bfd: Fix Solaris/x86 ELF_MAXPAGESIZE
authorRainer Orth <ro@CeBiTec.Uni-Bielefeld.DE>
Tue, 19 Aug 2025 10:52:41 +0000 (12:52 +0200)
committerRainer Orth <ro@CeBiTec.Uni-Bielefeld.DE>
Tue, 19 Aug 2025 10:52:41 +0000 (12:52 +0200)
commite067b9d0277039d04698099dc6f42d10c4a2bcb5
treedef064dae63d9aa51b3010928d835e58c14c6186
parent09292f4ae2ccb46130652f6b310ee7a5227326d3
bfd: Fix Solaris/x86 ELF_MAXPAGESIZE

I noticed that the alignment of the .text and .data sections on
Solaris/x86 doesn't match what /bin/ld does: gld uses the original i386
psABI default of 0x1000, while Solaris has moved to larger values as can
be seen both in the Oracle Solaris 11.4 Linkers and Libraries Guide,
ch. 15, Program Loading and Dynamic Linking, p. 15-6 and the system
headers (<sys/elf_{i386,amd64}.h>) that have

while the Solaris/SPARC values are already correct.

To fix this, on i386 it's sufficient to redefine ELF_MAXPAGESIZE.  On
x86_64, unlike i386, ELF_COMMONPAGESIZE is hardcoded as 0x1000, the
default, so setting ELF_MAXPAGESIZE has no effect on ELF_P_ALIGN.
Setting ELF_COMMONPAGESIZE to ELF_MAXPAGESIZE, too, fixes that and
brings both target in sync.  ELF_MACHINE_CODE is just set to the original
value again, so it's removed.

Tested on {i386,amd64}-pc-solaris2.11, {i686,x86_64}-pc-linux-gnu, and
amd64-pc-freebsd14.0.

2025-07-29  Rainer Orth  <ro@CeBiTec.Uni-Bielefeld.DE>

bfd:
* elf32-i386.c <elf32-i386-sol2> (ELF_MAXPAGESIZE): Redefine.
<elf32-iamcu> (ELF_MAXPAGESIZE): Restore previous value.
* elf64-x86-64.c (ELF_COMMONPAGESIZE): Define as ELF_MAXPAGESIZE.
<elf64-x86-64-sol2> (ELF_MAXPAGESIZE): Redefine
<elf32-x86-64> (ELF_MAXPAGESIZE): Restore previous value.
(ELF_MACHINE_CODE): Remove.
bfd/elf32-i386.c
bfd/elf64-x86-64.c