On Tue, May 30, 2023 at 04:46:17PM -0300, Daniel Henrique Barboza wrote: > At this moment we're retrieving env->misa_ext during > kvm_arch_init_cpu(), leaving env->misa_ext_mask behind. > > We want to set env->misa_ext_mask, and we want to set it as early as > possible. The reason is that we're going to use it in the validation > process of the KVM MISA properties we're going to add next. Setting it > during arch_init_cpu() is too late for user validation. > > Move the code to a new helper that is going to be called during init() > time, via kvm_riscv_init_user_properties(), like we're already doing for > the machine ID properties. Set both misa_ext and misa_ext_mask to the > same value retrieved by the 'isa' config reg. > > Signed-off-by: Daniel Henrique Barboza <dbarb...@ventanamicro.com> > --- > target/riscv/kvm.c | 34 +++++++++++++++++++++++----------- > 1 file changed, 23 insertions(+), 11 deletions(-) >
Reviewed-by: Andrew Jones <ajo...@ventanamicro.com>