Re: [PATCH] target/riscv: Adjust privilege level for HLV(X)/HSV instructions

2021-03-18 Thread Alistair Francis
On Thu, Mar 11, 2021 at 5:32 AM Georg Kotheimer wrote: > > According to the specification the "field SPVP of hstatus controls the > privilege level of the access" for the hypervisor virtual-machine load > and store instructions HLV, HLVX and HSV. > > Signed-off-by: Georg Kotheimer Thanks!

Re: [PATCH] target/riscv: Adjust privilege level for HLV(X)/HSV instructions

2021-03-16 Thread Alistair Francis
On Thu, Mar 11, 2021 at 5:32 AM Georg Kotheimer wrote: > > According to the specification the "field SPVP of hstatus controls the > privilege level of the access" for the hypervisor virtual-machine load > and store instructions HLV, HLVX and HSV. > > Signed-off-by: Georg Kotheimer Reviewed-by:

[PATCH] target/riscv: Adjust privilege level for HLV(X)/HSV instructions

2021-03-11 Thread Georg Kotheimer
According to the specification the "field SPVP of hstatus controls the privilege level of the access" for the hypervisor virtual-machine load and store instructions HLV, HLVX and HSV. Signed-off-by: Georg Kotheimer --- target/riscv/cpu_helper.c | 25 ++--- 1 file changed, 14

Re: [PATCH] target/riscv: Adjust privilege level for HLV(X)/HSV instructions

2020-10-22 Thread Richard Henderson
On 10/21/20 12:21 PM, Alistair Francis wrote: >> mmuidx, you would use cpu_lduw_mmuidx_ra. I would also split the function >> into >> two, so that one performs HLVX.HU and the other HLVX.WU, so that you don't >> have >> to pass the size as a parameter. > > I'm not clear what you mean here.

Re: [PATCH] target/riscv: Adjust privilege level for HLV(X)/HSV instructions

2020-10-21 Thread Alistair Francis
On Sun, Oct 18, 2020 at 8:58 AM Richard Henderson wrote: > > On 10/18/20 5:03 AM, Georg Kotheimer wrote: > > According to the specification the "field SPVP of hstatus controls the > > privilege level of the access" for the hypervisor virtual-machine load > > and store instructions HLV, HLVX and

Re: [PATCH] target/riscv: Adjust privilege level for HLV(X)/HSV instructions

2020-10-18 Thread Richard Henderson
On 10/18/20 5:03 AM, Georg Kotheimer wrote: > According to the specification the "field SPVP of hstatus controls the > privilege level of the access" for the hypervisor virtual-machine load > and store instructions HLV, HLVX and HSV. > > We introduce the new virtualization register field

[PATCH] target/riscv: Adjust privilege level for HLV(X)/HSV instructions

2020-10-18 Thread Georg Kotheimer
According to the specification the "field SPVP of hstatus controls the privilege level of the access" for the hypervisor virtual-machine load and store instructions HLV, HLVX and HSV. We introduce the new virtualization register field HS_HYP_LD_ST, similar to HS_TWO_STAGE, which tracks whether we