[v3] Add the implied rules for G and B extensions

[PATCH v3 0/2] Add the implied rules for G and B extensions

frank.chang@sifive.com posted 2 patches 1 month ago

Diff against v2
Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20260424042946.3875690-1-frank.chang@sifive.com

Maintainers: Palmer Dabbelt <palmer@dabbelt.com>, Alistair Francis <alistair.francis@wdc.com>, Weiwei Li <liwei1518@gmail.com>, Daniel Henrique Barboza <daniel.barboza@oss.qualcomm.com>, Liu Zhiwei <zhiwei_liu@linux.alibaba.com>, Chao Liu <chao.liu.zevorn@gmail.com>

target/riscv/cpu.c         | 26 ++++++++++++++++++++++++-
target/riscv/tcg/tcg-cpu.c | 39 +++++++-------------------------------
2 files changed, 32 insertions(+), 33 deletions(-)

Expand all Fold all

[PATCH v3 0/2] Add the implied rules for G and B extensions

Posted by frank.chang@sifive.com 1 month ago

From: Frank Chang <frank.chang@sifive.com>

Add the implied rules for G and B extensions to enable the implied
extensions.

Changelog:
  v3: Keep riscv_cpu_validate_g() and riscv_cpu_validate_b() but remove
      the auto-enables as the implied extensions can be enabled by the
      implied rules.

  v2: Remove riscv_cpu_validate_g() and riscv_cpu_validate_b() as the
      implied extensions can be enabled by the implied rules.

Frank Chang (1):
  target/riscv: Add standard B extension implied rule

Jim Shu (1):
  target/riscv: Add the implied rule for G extension

 target/riscv/cpu.c         | 26 ++++++++++++++++++++++++-
 target/riscv/tcg/tcg-cpu.c | 39 +++++++-------------------------------
 2 files changed, 32 insertions(+), 33 deletions(-)

--
2.43.0

Re: [PATCH v3 0/2] Add the implied rules for G and B extensions

Posted by Peter Maydell 1 month ago

On Fri, 24 Apr 2026 at 05:30, <frank.chang@sifive.com> wrote:
>
> From: Frank Chang <frank.chang@sifive.com>
>
> Add the implied rules for G and B extensions to enable the implied
> extensions.
>
> Changelog:
>   v3: Keep riscv_cpu_validate_g() and riscv_cpu_validate_b() but remove
>       the auto-enables as the implied extensions can be enabled by the
>       implied rules.
>
>   v2: Remove riscv_cpu_validate_g() and riscv_cpu_validate_b() as the
>       implied extensions can be enabled by the implied rules.
>
> Frank Chang (1):
>   target/riscv: Add standard B extension implied rule
>
> Jim Shu (1):
>   target/riscv: Add the implied rule for G extension

Reminder to riscv folks -- please put "target/riscv" or
similar in the subject of your cover letter, so that people
who aren't interested in riscv patches can skip over it
rather than wondering which of the many possible areas
of QEMU "G and B extensions" might be talking about...

thanks
-- PMM

Re: [PATCH v3 0/2] Add the implied rules for G and B extensions

Posted by Frank Chang 1 month ago

On Fri, Apr 24, 2026 at 4:18 PM Peter Maydell <peter.maydell@linaro.org>
wrote:

> On Fri, 24 Apr 2026 at 05:30, <frank.chang@sifive.com> wrote:
> >
> > From: Frank Chang <frank.chang@sifive.com>
> >
> > Add the implied rules for G and B extensions to enable the implied
> > extensions.
> >
> > Changelog:
> >   v3: Keep riscv_cpu_validate_g() and riscv_cpu_validate_b() but remove
> >       the auto-enables as the implied extensions can be enabled by the
> >       implied rules.
> >
> >   v2: Remove riscv_cpu_validate_g() and riscv_cpu_validate_b() as the
> >       implied extensions can be enabled by the implied rules.
> >
> > Frank Chang (1):
> >   target/riscv: Add standard B extension implied rule
> >
> > Jim Shu (1):
> >   target/riscv: Add the implied rule for G extension
>
> Reminder to riscv folks -- please put "target/riscv" or
> similar in the subject of your cover letter, so that people
> who aren't interested in riscv patches can skip over it
> rather than wondering which of the many possible areas
> of QEMU "G and B extensions" might be talking about...
>
> thanks
> -- PMM
>

Agh, sorry for missing that.
I read the mail after sending out another patchset:
https://lore.kernel.org/qemu-devel/20260424102800.24022-1-frank.chang@sifive.com/

I will fix it in the later patchsets.
Sorry for the disturbance

Regard,
Frank Chang