qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH v3 0/4] target/riscv: Simplification for RVH related check an


From: Daniel Henrique Barboza
Subject: Re: [PATCH v3 0/4] target/riscv: Simplification for RVH related check and code style fix
Date: Wed, 5 Apr 2023 16:13:46 -0300
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.9.0



On 4/5/23 05:58, Weiwei Li wrote:
This patchset tries to simplify the RVH related check and fix some code style 
problems, such as problems for indentation, multi-line comments and lines with 
over 80 characters.

The port is available here:
https://github.com/plctlab/plct-qemu/tree/plct-virtfix-upstream

v2:
* add comment to specify riscv_cpu_set_virt_enabled() can only be called when 
RVH is enabled in patch 4 (suggested by Richard Henderson)
* merge patch from LIU Zhiwei(Message-ID: 
<20230325145348.1208-1-zhiwei_liu@linux.alibaba.com>) to patch 5
* use env->virt_enabled directly instead of riscv_cpu_virt_enabled() in patch 6 
(suggested by LIU Zhiwei)
* remain the orginal identation for macro name in patch 8 (suggested by LIU 
Zhiwei)

v3:
* rebase on riscv-to-apply.next (partial patches in v2 have been applied)
* Fix new found format problem in patch 2,3,4 of v3

All patches:

Reviewed-by: Daniel Henrique Barboza <dbarboza@ventanamicro.com>


Weiwei Li (4):
   target/riscv: Remove riscv_cpu_virt_enabled()
   target/riscv: Fix format for indentation
   target/riscv: Fix format for comments
   target/riscv: Fix lines with over 80 characters

  target/riscv/arch_dump.c                |   7 +-
  target/riscv/cpu.c                      |  47 ++--
  target/riscv/cpu.h                      |  31 +--
  target/riscv/cpu_bits.h                 |   2 +-
  target/riscv/cpu_helper.c               | 274 ++++++++++----------
  target/riscv/csr.c                      |  90 +++----
  target/riscv/debug.c                    |  21 +-
  target/riscv/fpu_helper.c               |  24 +-
  target/riscv/gdbstub.c                  |   3 +-
  target/riscv/insn_trans/trans_rvv.c.inc |  36 +--
  target/riscv/m128_helper.c              |  16 +-
  target/riscv/machine.c                  |  18 +-
  target/riscv/op_helper.c                |  20 +-
  target/riscv/pmp.c                      |  66 ++---
  target/riscv/pmp.h                      |   9 +-
  target/riscv/pmu.c                      |   7 +-
  target/riscv/sbi_ecall_interface.h      |   8 +-
  target/riscv/translate.c                |  26 +-
  target/riscv/vector_helper.c            | 317 ++++++++++++++----------
  19 files changed, 552 insertions(+), 470 deletions(-)




reply via email to

[Prev in Thread] Current Thread [Next in Thread]