* [RESEND PATCH 0/1] RISC-V: Update 'Zihintpause' extension version @ 2022-07-03 10:31 Tsukasa OI 2022-07-03 10:31 ` [RESEND PATCH 1/1] RISC-V: Update Zihintpause " Tsukasa OI 0 siblings, 1 reply; 5+ messages in thread From: Tsukasa OI @ 2022-07-03 10:31 UTC (permalink / raw) To: Nelson Chu, Kito Cheng, Palmer Dabbelt, Tsukasa OI; +Cc: binutils Hi RISC-V folks, `Zihintpause' extension actually has version 2.0 (not 1.0). This patch will fix the version value. ORIG: <https://sourceware.org/pipermail/binutils/2022-May/120978.html> PING: <https://sourceware.org/pipermail/binutils/2022-May/120978.html> Some discussion record: <https://sourceware.org/pipermail/binutils/2022-February/119587.html> Tracker on GitHub: <https://github.com/a4lg/binutils-gdb/wiki/riscv_zihintpause_ver> My copyright assignment is complete and I request review for upstream merge. To me, 1. This one and 2. Zhinx fix patchset (to make it to GNU Binutils 2.39 release) <https://sourceware.org/pipermail/binutils/2022-June/121416.html> <https://github.com/a4lg/binutils-gdb/wiki/riscv_zhinx_quick_fix> are two important patchsets intended for GNU Binutils 2.39 (to minimize confusion and compatibility problems as possible). I can say that all my other submissions can be postponed for Binutils 2.40 or later. I understand that committers are busy right now but I think I could minimize the patchsets to review until 2.39 branch date. I am so sorry to rush you. Thanks, Tsukasa Tsukasa OI (1): RISC-V: Update Zihintpause extension version bfd/elfxx-riscv.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) base-commit: dcc9b683dc7d6b0c8e64bfa5e11cf0830387bb31 -- 2.25.1 ^ permalink raw reply [flat|nested] 5+ messages in thread
* [RESEND PATCH 1/1] RISC-V: Update Zihintpause extension version 2022-07-03 10:31 [RESEND PATCH 0/1] RISC-V: Update 'Zihintpause' extension version Tsukasa OI @ 2022-07-03 10:31 ` Tsukasa OI 2022-07-04 1:37 ` Nelson Chu 2022-07-04 6:26 ` Jan Beulich 0 siblings, 2 replies; 5+ messages in thread From: Tsukasa OI @ 2022-07-03 10:31 UTC (permalink / raw) To: Nelson Chu, Kito Cheng, Palmer Dabbelt, Tsukasa OI; +Cc: binutils Because ratified Zihintpause extension has a version number of 2.0 (not 1.0), we should update the number. bfd/ChangeLog: * elfxx-riscv.c (riscv_supported_std_z_ext): Update version number of Zihintpause extension. --- bfd/elfxx-riscv.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/bfd/elfxx-riscv.c b/bfd/elfxx-riscv.c index bf7dc20e892..c846d6baaa2 100644 --- a/bfd/elfxx-riscv.c +++ b/bfd/elfxx-riscv.c @@ -1185,7 +1185,7 @@ static struct riscv_supported_ext riscv_supported_std_z_ext[] = {"zicsr", ISA_SPEC_CLASS_20190608, 2, 0, 0 }, {"zifencei", ISA_SPEC_CLASS_20191213, 2, 0, 0 }, {"zifencei", ISA_SPEC_CLASS_20190608, 2, 0, 0 }, - {"zihintpause", ISA_SPEC_CLASS_DRAFT, 1, 0, 0 }, + {"zihintpause", ISA_SPEC_CLASS_DRAFT, 2, 0, 0 }, {"zfh", ISA_SPEC_CLASS_DRAFT, 1, 0, 0 }, {"zfinx", ISA_SPEC_CLASS_DRAFT, 1, 0, 0 }, {"zdinx", ISA_SPEC_CLASS_DRAFT, 1, 0, 0 }, -- 2.25.1 ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [RESEND PATCH 1/1] RISC-V: Update Zihintpause extension version 2022-07-03 10:31 ` [RESEND PATCH 1/1] RISC-V: Update Zihintpause " Tsukasa OI @ 2022-07-04 1:37 ` Nelson Chu 2022-07-04 6:26 ` Jan Beulich 1 sibling, 0 replies; 5+ messages in thread From: Nelson Chu @ 2022-07-04 1:37 UTC (permalink / raw) To: Tsukasa OI; +Cc: Kito Cheng, Palmer Dabbelt, Binutils OK, thanks, committed. Nelson On Sun, Jul 3, 2022 at 6:31 PM Tsukasa OI <research_trasio@irq.a4lg.com> wrote: > > Because ratified Zihintpause extension has a version number of 2.0 > (not 1.0), we should update the number. > > bfd/ChangeLog: > > * elfxx-riscv.c (riscv_supported_std_z_ext): Update version > number of Zihintpause extension. > --- > bfd/elfxx-riscv.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/bfd/elfxx-riscv.c b/bfd/elfxx-riscv.c > index bf7dc20e892..c846d6baaa2 100644 > --- a/bfd/elfxx-riscv.c > +++ b/bfd/elfxx-riscv.c > @@ -1185,7 +1185,7 @@ static struct riscv_supported_ext riscv_supported_std_z_ext[] = > {"zicsr", ISA_SPEC_CLASS_20190608, 2, 0, 0 }, > {"zifencei", ISA_SPEC_CLASS_20191213, 2, 0, 0 }, > {"zifencei", ISA_SPEC_CLASS_20190608, 2, 0, 0 }, > - {"zihintpause", ISA_SPEC_CLASS_DRAFT, 1, 0, 0 }, > + {"zihintpause", ISA_SPEC_CLASS_DRAFT, 2, 0, 0 }, > {"zfh", ISA_SPEC_CLASS_DRAFT, 1, 0, 0 }, > {"zfinx", ISA_SPEC_CLASS_DRAFT, 1, 0, 0 }, > {"zdinx", ISA_SPEC_CLASS_DRAFT, 1, 0, 0 }, > -- > 2.25.1 > ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [RESEND PATCH 1/1] RISC-V: Update Zihintpause extension version 2022-07-03 10:31 ` [RESEND PATCH 1/1] RISC-V: Update Zihintpause " Tsukasa OI 2022-07-04 1:37 ` Nelson Chu @ 2022-07-04 6:26 ` Jan Beulich 2022-07-05 10:40 ` Kito Cheng 1 sibling, 1 reply; 5+ messages in thread From: Jan Beulich @ 2022-07-04 6:26 UTC (permalink / raw) To: Tsukasa OI, Nelson Chu; +Cc: binutils, Kito Cheng, Palmer Dabbelt On 03.07.2022 12:31, Tsukasa OI via Binutils wrote: > Because ratified Zihintpause extension has a version number of 2.0 > (not 1.0), we should update the number. > > bfd/ChangeLog: > > * elfxx-riscv.c (riscv_supported_std_z_ext): Update version > number of Zihintpause extension. > --- > bfd/elfxx-riscv.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/bfd/elfxx-riscv.c b/bfd/elfxx-riscv.c > index bf7dc20e892..c846d6baaa2 100644 > --- a/bfd/elfxx-riscv.c > +++ b/bfd/elfxx-riscv.c > @@ -1185,7 +1185,7 @@ static struct riscv_supported_ext riscv_supported_std_z_ext[] = > {"zicsr", ISA_SPEC_CLASS_20190608, 2, 0, 0 }, > {"zifencei", ISA_SPEC_CLASS_20191213, 2, 0, 0 }, > {"zifencei", ISA_SPEC_CLASS_20190608, 2, 0, 0 }, > - {"zihintpause", ISA_SPEC_CLASS_DRAFT, 1, 0, 0 }, > + {"zihintpause", ISA_SPEC_CLASS_DRAFT, 2, 0, 0 }, > {"zfh", ISA_SPEC_CLASS_DRAFT, 1, 0, 0 }, > {"zfinx", ISA_SPEC_CLASS_DRAFT, 1, 0, 0 }, > {"zdinx", ISA_SPEC_CLASS_DRAFT, 1, 0, 0 }, Just for my own education: Isn't the general versioning scheme using 2.x only for non-draft functionality? Jan ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [RESEND PATCH 1/1] RISC-V: Update Zihintpause extension version 2022-07-04 6:26 ` Jan Beulich @ 2022-07-05 10:40 ` Kito Cheng 0 siblings, 0 replies; 5+ messages in thread From: Kito Cheng @ 2022-07-05 10:40 UTC (permalink / raw) To: Jan Beulich; +Cc: Tsukasa OI, Nelson Chu, Kito Cheng, Binutils > Just for my own education: Isn't the general versioning scheme using > 2.x only for non-draft functionality? Version scheme has *slightly* changed, RISC-V uses 2.0 as raritfed version number, and 2.x for non-draft functionality change, and now RISC-V uses 1.0 as a ratified version. More discussion you can found here: https://sourceware.org/pipermail/binutils/2022-February/119587.html But I must say I still can't figure out what happened on zihintpause after reading the thread... ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2022-07-05 10:40 UTC | newest] Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2022-07-03 10:31 [RESEND PATCH 0/1] RISC-V: Update 'Zihintpause' extension version Tsukasa OI 2022-07-03 10:31 ` [RESEND PATCH 1/1] RISC-V: Update Zihintpause " Tsukasa OI 2022-07-04 1:37 ` Nelson Chu 2022-07-04 6:26 ` Jan Beulich 2022-07-05 10:40 ` Kito Cheng
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).