[PATCH v7] MIPS: Sync elf.h from binutils
Maciej W. Rozycki
macro@orcam.me.uk
Mon Aug 21 11:14:47 GMT 2023
On Mon, 21 Aug 2023, Ying Huang wrote:
> Can you help commit these two patchs?
To move forward you need to put the findings from the discussion in the
change description, especially given the difficulty to figure out what the
correct naming convention for the ELF file header flags is supposed to be.
This is so that the next time this information is needed it is readily
available rather than requiring chasing in the archives and/or depending
on the presence of people who may still remember it as on this occasion.
I think it will help if you split the glibc patch into self-contained
functional subsets:
1. File header flags -- need documentation, as noted above.
2. SHT_MIPS_ABIFLAGS section type -- obviously correct, no need to
elaborate on it.
3. Relocation types -- obviously correct, no need to elaborate on it.
4. GNU attribute stuff -- technically Tag_GNU_MIPS_ABI_FP along with the
enumeration it comes with is a bug fix for commit 0bd956720c45 ("Add
support for MIPS O32 FPXX and .MIPS.abiflags") which, conservatively,
only added definitions actually used by the dynamic loader itself,
despite that the header is a part of our public API for the MIPS psABI.
I think that either said definitions ought to have been provided
internally only or the whole of the API should have been exported at
once (this applies to #2 above too).
Then the MSA stuff and Val_GNU_MIPS_ABI_FP_NAN2008 as later additions
are distinct changes each. I think it is worth mentioning in the
inline comment that the latter attribute value has been superseded by
EF_MIPS_NAN2008.
I'd be tempted to say these would best be three distint changes then,
but maybe other people would consider it overly pedantic.
Then the binutils change needs to document the switch from E_* to EF_*
definitions accordingly.
Maciej
More information about the Binutils
mailing list