From: "Maciej W. Rozycki" <macro@orcam.me.uk>
To: Ying Huang <ying.huang@oss.cipunited.com>
Cc: libc-alpha <libc-alpha@sourceware.org>,
yunqiang.su@oss.cipunited.com,
binutils <binutils@sourceware.org>
Subject: Re: [PATCH v7] MIPS: Sync elf.h from binutils
Date: Mon, 21 Aug 2023 12:14:47 +0100 (BST) [thread overview]
Message-ID: <alpine.DEB.2.21.2308211123450.49340@angie.orcam.me.uk> (raw)
In-Reply-To: <8725cca2-4c4d-f32d-fc8c-3c6c194c481c@oss.cipunited.com>
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
next prev parent reply other threads:[~2023-08-21 11:14 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <20230807020524.2031213-1-ying.huang@oss.cipunited.com>
[not found] ` <87leemerwd.fsf@oldenburg.str.redhat.com>
[not found] ` <4d3d1518-364c-8374-9d3d-86d1afe25d62@oss.cipunited.com>
[not found] ` <87r0odeust.fsf@oldenburg.str.redhat.com>
[not found] ` <3d7f6d51-7d7f-cc7c-31b9-744dc06010fa@oss.cipunited.com>
[not found] ` <87cyzxfugb.fsf@oldenburg.str.redhat.com>
[not found] ` <alpine.DEB.2.21.2308091654110.25915@angie.orcam.me.uk>
[not found] ` <87il9oywuh.fsf@oldenburg.str.redhat.com>
2023-08-11 17:43 ` Maciej W. Rozycki
2023-08-16 6:15 ` Ying Huang
2023-08-18 10:10 ` 黄莺
2023-08-21 6:27 ` Ying Huang
2023-08-21 11:14 ` Maciej W. Rozycki [this message]
2023-08-22 8:55 ` Ying Huang
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=alpine.DEB.2.21.2308211123450.49340@angie.orcam.me.uk \
--to=macro@orcam.me.uk \
--cc=binutils@sourceware.org \
--cc=libc-alpha@sourceware.org \
--cc=ying.huang@oss.cipunited.com \
--cc=yunqiang.su@oss.cipunited.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).