qemu-riscv
[Top][All Lists]
Advanced

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

Re: [PATCH v8 14/20] target/riscv/kvm.c: add multi-letter extension KVM


From: Andrew Jones
Subject: Re: [PATCH v8 14/20] target/riscv/kvm.c: add multi-letter extension KVM properties
Date: Thu, 6 Jul 2023 11:37:48 +0200

On Thu, Jul 06, 2023 at 10:45:04AM +0200, Andrew Jones wrote:
...
>  1. Audit KVM to ensure using EINVAL for get-one-reg's "register not
>     present" is currently safe, and keep our eye on KVM to make sure
>     it remains safe until get-reg-list lands.

I just did the audit and it looks fine. There's only one way these types
of get-one-reg invocations can return EINVAL other than for "register
not present" and that's for the encoded reg size not being
sizeof(unsigned long). As long as we construct the register ID with the
correct size then that can't happen. We currently construct it with
kvm_riscv_reg_id() which does the right thing for these invocations.

Thanks,
drew



reply via email to

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