* [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).