[PATCH v1 03/12] target/riscv: Implement function kvm_arch_init_vcpu

[Jiangyifei]

> -----Original Message-----
> From: Richard Henderson [mailto:richard.henderson at linaro.org]
> Sent: Sunday, November 21, 2021 6:19 AM
> To: Jiangyifei <jiangyifei at huawei.com>; qemu-devel at nongnu.org;
> qemu-riscv at nongnu.org
> Cc: bin.meng at windriver.com; limingwang (A) <limingwang at huawei.com>;
> kvm at vger.kernel.org; libvir-list at redhat.com; anup.patel at wdc.com; wanbo (G)
> <wanbo13 at huawei.com>; Alistair Francis <alistair.francis at wdc.com>;
> kvm-riscv at lists.infradead.org; Wanghaibin (D)
> <wanghaibin.wang at huawei.com>; palmer at dabbelt.com; Fanliang (EulerOS)
> <fanliang at huawei.com>; Wubin (H) <wu.wubin at huawei.com>
> Subject: Re: [PATCH v1 03/12] target/riscv: Implement function
> kvm_arch_init_vcpu
> 
> On 11/20/21 8:46 AM, Yifei Jiang wrote:
> > +    id = kvm_riscv_reg_id(env, KVM_REG_RISCV_CONFIG,
> KVM_REG_RISCV_CONFIG_REG(isa));
> > +    ret = kvm_get_one_reg(cs, id, &isa);
> > +    if (ret) {
> > +        return ret;
> > +    }
> > +    env->misa_mxl |= isa;
> 
> This doesn't look right.
> I'm sure you meant
> 
>      env->misa_ext = isa;
> 
> 
> r~

Thanks, it will be modified in the next series.

Yifei