[PATCH v3 00/14] target/riscv: Rationalize XLEN and operand length

Richard Henderson posted 14 patches 2 years, 6 months ago
Test checkpatch passed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20211016171412.3163784-1-richard.henderson@linaro.org
Maintainers: Laurent Vivier <laurent@vivier.eu>, Bin Meng <bin.meng@windriver.com>, "Alex Bennée" <alex.bennee@linaro.org>, Alistair Francis <alistair.francis@wdc.com>, Palmer Dabbelt <palmer@dabbelt.com>
There is a newer version of this series
target/riscv/cpu.h                      |  73 +++-------
target/riscv/cpu_bits.h                 |   8 +-
hw/riscv/boot.c                         |   2 +-
linux-user/elfload.c                    |   2 +-
linux-user/riscv/cpu_loop.c             |   2 +-
semihosting/arm-compat-semi.c           |   2 +-
target/riscv/cpu.c                      | 108 +++++++++------
target/riscv/cpu_helper.c               |  92 ++++++++++++-
target/riscv/csr.c                      | 104 ++++++++------
target/riscv/gdbstub.c                  |  10 +-
target/riscv/machine.c                  |  10 +-
target/riscv/monitor.c                  |   4 +-
target/riscv/translate.c                | 174 ++++++++++++++++++------
target/riscv/insn_trans/trans_rvb.c.inc | 140 ++++++++++---------
target/riscv/insn_trans/trans_rvi.c.inc |  44 +++---
target/riscv/insn_trans/trans_rvm.c.inc |  36 +++--
target/riscv/insn_trans/trans_rvv.c.inc |  29 ++--
17 files changed, 532 insertions(+), 308 deletions(-)
[PATCH v3 00/14] target/riscv: Rationalize XLEN and operand length
Posted by Richard Henderson 2 years, 6 months ago
This is a partial patch set attempting to set things in the
right direction for both the UXL and RV128 patch sets.

Notable addition for v3 is the treatment of [MS]STATUS.SD.  Because this
bit changes position depending on XLEN, it's better to split it out.
But since it's read-only and computable from [MS]STATUS.{FS,XS}, it
is even better to not store it at all.  I noticed this while reading
Frank's RVV patches which add VS to be included in SD.


r~


Changes for v3:
  * Fix CONFIG_ typo.
  * Fix ctzw typo.
  * Mark get_xlen unused (clang werror)
  * Compute MSTATUS_SD on demand.

Changes for v2:
  * Set mxl/sxl/uxl at reset.
  * Set sxl/uxl in write_mstatus.




Richard Henderson (14):
  target/riscv: Move cpu_get_tb_cpu_state out of line
  target/riscv: Create RISCVMXL enumeration
  target/riscv: Split misa.mxl and misa.ext
  target/riscv: Replace riscv_cpu_is_32bit with riscv_cpu_mxl
  target/riscv: Add MXL/SXL/UXL to TB_FLAGS
  target/riscv: Use REQUIRE_64BIT in amo_check64
  target/riscv: Properly check SEW in amo_op
  target/riscv: Replace is_32bit with get_xl/get_xlen
  target/riscv: Replace DisasContext.w with DisasContext.ol
  target/riscv: Use gen_arith_per_ol for RVM
  target/riscv: Adjust trans_rev8_32 for riscv64
  target/riscv: Use gen_unary_per_ol for RVB
  target/riscv: Use gen_shift*_per_ol for RVB, RVI
  target/riscv: Compute mstatus.sd on demand

 target/riscv/cpu.h                      |  73 +++-------
 target/riscv/cpu_bits.h                 |   8 +-
 hw/riscv/boot.c                         |   2 +-
 linux-user/elfload.c                    |   2 +-
 linux-user/riscv/cpu_loop.c             |   2 +-
 semihosting/arm-compat-semi.c           |   2 +-
 target/riscv/cpu.c                      | 108 +++++++++------
 target/riscv/cpu_helper.c               |  92 ++++++++++++-
 target/riscv/csr.c                      | 104 ++++++++------
 target/riscv/gdbstub.c                  |  10 +-
 target/riscv/machine.c                  |  10 +-
 target/riscv/monitor.c                  |   4 +-
 target/riscv/translate.c                | 174 ++++++++++++++++++------
 target/riscv/insn_trans/trans_rvb.c.inc | 140 ++++++++++---------
 target/riscv/insn_trans/trans_rvi.c.inc |  44 +++---
 target/riscv/insn_trans/trans_rvm.c.inc |  36 +++--
 target/riscv/insn_trans/trans_rvv.c.inc |  29 ++--
 17 files changed, 532 insertions(+), 308 deletions(-)

-- 
2.25.1