Re: [PATCH v3 3/3] target/riscv: Legalize MPP value in write_mstatus

2023-04-10 Thread Alistair Francis
On Fri, Apr 7, 2023 at 11:49 AM Weiwei Li wrote: > > mstatus.MPP field is a WARL field since priv version 1.11, so we > remain it unchanged if an invalid value is written into it. And > after this, RVH shouldn't be passed to riscv_cpu_set_mode(). > > Signed-off-by: Weiwei Li > Signed-off-by: Junq

Re: [PATCH v3 3/3] target/riscv: Legalize MPP value in write_mstatus

2023-04-07 Thread Richard Henderson
On 4/6/23 18:47, Weiwei Li wrote: mstatus.MPP field is a WARL field since priv version 1.11, so we remain it unchanged if an invalid value is written into it. And after this, RVH shouldn't be passed to riscv_cpu_set_mode(). Signed-off-by: Weiwei Li Signed-off-by: Junqiang Wang --- target/riscv

[PATCH v3 3/3] target/riscv: Legalize MPP value in write_mstatus

2023-04-06 Thread Weiwei Li
mstatus.MPP field is a WARL field since priv version 1.11, so we remain it unchanged if an invalid value is written into it. And after this, RVH shouldn't be passed to riscv_cpu_set_mode(). Signed-off-by: Weiwei Li Signed-off-by: Junqiang Wang --- target/riscv/cpu_helper.c | 8 ++-- target