Re: [PATCH 1/1] target/riscv: Convert env->virt to a bool env->virt_enabled

2023-04-04 Thread Alistair Francis
On Sun, Mar 26, 2023 at 12:55 AM LIU Zhiwei wrote: > > Currently we only use the env->virt to encode the virtual mode enabled > status. Let's make it a bool type. > > Signed-off-by: LIU Zhiwei Thanks! Applied to riscv-to-apply.next Alistair > --- > target/riscv/cpu.h| 2 +- > target/

Re: [PATCH 1/1] target/riscv: Convert env->virt to a bool env->virt_enabled

2023-04-04 Thread Alistair Francis
On Sun, Mar 26, 2023 at 12:55 AM LIU Zhiwei wrote: > > Currently we only use the env->virt to encode the virtual mode enabled > status. Let's make it a bool type. > > Signed-off-by: LIU Zhiwei Reviewed-by: Alistair Francis Alistair > --- > target/riscv/cpu.h| 2 +- > target/riscv/cpu

Re: [PATCH 1/1] target/riscv: Convert env->virt to a bool env->virt_enabled

2023-03-26 Thread liweiwei
On 2023/3/25 22:53, LIU Zhiwei wrote: Currently we only use the env->virt to encode the virtual mode enabled status. Let's make it a bool type. Signed-off-by: LIU Zhiwei --- I'm not quite sure the original reason to use a int for virt. However, this change is acceptable to me. Reviewed-by

[PATCH 1/1] target/riscv: Convert env->virt to a bool env->virt_enabled

2023-03-25 Thread LIU Zhiwei
Currently we only use the env->virt to encode the virtual mode enabled status. Let's make it a bool type. Signed-off-by: LIU Zhiwei --- target/riscv/cpu.h| 2 +- target/riscv/cpu_bits.h | 3 --- target/riscv/cpu_helper.c | 6 +++--- target/riscv/machine.c| 6 +++--- target/riscv/tr