From: Tsukasa OI <research_trasio@irq.a4lg.com>
To: Tsukasa OI <research_trasio@irq.a4lg.com>,
Alistair Francis <alistair23@gmail.com>,
Frank Chang <frank.chang@sifive.com>,
Weiwei Li <liweiwei@iscas.ac.cn>
Cc: qemu-devel@nongnu.org, qemu-riscv@nongnu.org
Subject: [PATCH v2 0/2] target/riscv: ISA string conversion fix and enhancement
Date: Tue, 10 May 2022 20:29:06 +0900 [thread overview]
Message-ID: <cover.1652181972.git.research_trasio@irq.a4lg.com> (raw)
Hello,
There is two issues related to RISC-V ISA extension string
I want to be fixed before QEMU 7.1 release.
This is PATCH v2.
For details, see cover letter of PATCH v1.
<https://lists.gnu.org/archive/html/qemu-devel/2022-04/msg03817.html>
PATCH v1 -> PATCH v2:
I decided to "move" Zhinx*, not to remove them tentativelly.
Because, although I disagree with Weiwei's opinion (on canonical
ordering, Z* multi-letter extensions are ordered by second character;
that should reflect closely related single-letter extension but that
didn't happened on Zhinx*), ordering of "H" is reviewed and going to be
right after "V".
I considered this is safe enough (despite its "unratified" status):
1. Zhinx and Zhinxmin are minor extensions
2. Extension ordering is going to be not that important in the future
(by upcoming rules) but we have reasons to make extension ordering
canonical as possible for toolchain compatibility (for now).
Tsukasa OI (2):
target/riscv: Move Zhinx* extensions on ISA string
target/riscv: Add short-isa-string option
target/riscv/cpu.c | 9 ++++++---
target/riscv/cpu.h | 2 ++
2 files changed, 8 insertions(+), 3 deletions(-)
base-commit: 178bacb66d98d9ee7a702b9f2a4dfcd88b72a9ab
--
2.34.1
next reply other threads:[~2022-05-10 11:31 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-05-10 11:29 Tsukasa OI [this message]
2022-05-10 11:29 ` [PATCH v2 1/2] target/riscv: Move Zhinx* extensions on ISA string Tsukasa OI
2022-05-16 23:19 ` Alistair Francis
2022-05-10 11:29 ` [PATCH v2 2/2] target/riscv: Add short-isa-string option Tsukasa OI
2022-05-16 23:20 ` Alistair Francis
2022-05-16 23:27 ` [PATCH v2 0/2] target/riscv: ISA string conversion fix and enhancement Alistair Francis
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=cover.1652181972.git.research_trasio@irq.a4lg.com \
--to=research_trasio@irq.a4lg.com \
--cc=alistair23@gmail.com \
--cc=frank.chang@sifive.com \
--cc=liweiwei@iscas.ac.cn \
--cc=qemu-devel@nongnu.org \
--cc=qemu-riscv@nongnu.org \
/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 NNTP newsgroup(s).