[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 0/4] target/riscv: rename ext_i* to ext_zi*
From: |
Alistair Francis |
Subject: |
Re: [PATCH 0/4] target/riscv: rename ext_i* to ext_zi* |
Date: |
Mon, 16 Oct 2023 15:00:56 +1000 |
On Fri, Oct 13, 2023 at 2:47 AM Daniel Henrique Barboza
<dbarboza@ventanamicro.com> wrote:
>
> Hi,
>
> This series renames the following internal extensions flags:
>
> ext_ifencei, ext_icsr, ext_icbom, ext_icboz
>
> To add a leading 'z':
>
> ext_zifencei, ext_zicsr, ext_zicbom, ext_zicboz
>
> This was proposed by Andrew Jones. The motivation, in his own words:
>
> "I'd prefer we name the booleans with their 'z' to improve grepping.
> When one wants to search for uses of zifencei, I imagine they're more
> likely to do 'grep -i zifencei' than 'grep -i ifencei', particularly
> because the 's' is maintained for S extensions. "
>
>
> Aside from that, these are the only 4 Z-extension flags that don't use a
> leading 'z' in their name, so there's also the benefit of making
> everything equal.
>
>
>
> Daniel Henrique Barboza (4):
> target/riscv: rename ext_ifencei to ext_zifencei
> target/riscv: rename ext_icsr to ext_zicsr
> target/riscv: rename ext_icbom to ext_zicbom
> target/riscv: rename ext_icboz to ext_zicboz
Thanks!
Applied to riscv-to-apply.next
Alistair
>
> hw/riscv/boot.c | 2 +-
> hw/riscv/virt.c | 4 +-
> target/riscv/cpu.c | 56 ++++++++++-----------
> target/riscv/cpu_cfg.h | 8 +--
> target/riscv/csr.c | 2 +-
> target/riscv/gdbstub.c | 2 +-
> target/riscv/insn_trans/trans_rvi.c.inc | 2 +-
> target/riscv/insn_trans/trans_rvzicbo.c.inc | 16 +++---
> target/riscv/kvm/kvm-cpu.c | 12 ++---
> target/riscv/tcg/tcg-cpu.c | 20 ++++----
> 10 files changed, 62 insertions(+), 62 deletions(-)
>
> --
> 2.41.0
>
>
- [PATCH 0/4] target/riscv: rename ext_i* to ext_zi*, Daniel Henrique Barboza, 2023/10/12
- [PATCH 1/4] target/riscv: rename ext_ifencei to ext_zifencei, Daniel Henrique Barboza, 2023/10/12
- [PATCH 2/4] target/riscv: rename ext_icsr to ext_zicsr, Daniel Henrique Barboza, 2023/10/12
- [PATCH 3/4] target/riscv: rename ext_icbom to ext_zicbom, Daniel Henrique Barboza, 2023/10/12
- [PATCH 4/4] target/riscv: rename ext_icboz to ext_zicboz, Daniel Henrique Barboza, 2023/10/12
- Re: [PATCH 0/4] target/riscv: rename ext_i* to ext_zi*, Andrew Jones, 2023/10/13
- Re: [PATCH 0/4] target/riscv: rename ext_i* to ext_zi*,
Alistair Francis <=