Re: [PATCH] target/riscv KVM_RISCV_SET_TIMER macro is not configured correctly

2023-07-09 Thread Alistair Francis
On Fri, Jul 7, 2023 at 10:26 PM yang.zhang wrote: > > From: "yang.zhang" > > Should set/get riscv all reg timer,i.e, time/compare/frequency/state. > > Signed-off-by:Yang Zhang > Resolves: https://gitlab.com/qemu-project/qemu/-/issues/1688 Thanks! Applied to riscv-to-apply.next Alistair > ---

Re: [PATCH] target/riscv KVM_RISCV_SET_TIMER macro is not configured correctly

2023-07-09 Thread Alistair Francis
On Fri, Jul 7, 2023 at 10:26 PM yang.zhang wrote: > > From: "yang.zhang" > > Should set/get riscv all reg timer,i.e, time/compare/frequency/state. > > Signed-off-by:Yang Zhang > Resolves: https://gitlab.com/qemu-project/qemu/-/issues/1688 Reviewed-by: Alistair Francis Alistair > --- > targe

[PATCH] target/riscv KVM_RISCV_SET_TIMER macro is not configured correctly

2023-07-07 Thread yang.zhang
From: "yang.zhang" Should set/get riscv all reg timer,i.e, time/compare/frequency/state. Signed-off-by:Yang Zhang Resolves: https://gitlab.com/qemu-project/qemu/-/issues/1688 --- target/riscv/kvm.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/target/riscv/kvm.c b/target/

Re: [PATCH] target/riscv KVM_RISCV_SET_TIMER macro is not configured correctly

2023-07-07 Thread Daniel Henrique Barboza
On 7/7/23 00:23, yang.zhang wrote: From: "yang.zhang" Should set/get riscv all reg timer,i.e, time/compare/frequency/state. Nice catch. The reason why this went under the radar for 18 months is because kvm.c is using an external 'time' variable. Signed-off-by:Yang Zhang Resolves: http