On Thu, Aug 14, 2025 at 09:25:43PM +0530, Anup Patel wrote: > The hedeleg may be updated by ONE_REG interface before the VCPU > is run at least once hence set the initial value of hedeleg in > kvm_arch_vcpu_create() instead of kvm_riscv_vcpu_setup_config(). > > Signed-off-by: Anup Patel <apatel@xxxxxxxxxxxxxxxx> > --- > arch/riscv/kvm/vcpu.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/arch/riscv/kvm/vcpu.c b/arch/riscv/kvm/vcpu.c > index f001e56403f9..86025f68c374 100644 > --- a/arch/riscv/kvm/vcpu.c > +++ b/arch/riscv/kvm/vcpu.c > @@ -133,6 +133,8 @@ int kvm_arch_vcpu_create(struct kvm_vcpu *vcpu) > > /* Mark this VCPU never ran */ > vcpu->arch.ran_atleast_once = false; > + > + vcpu->arch.cfg.hedeleg = KVM_HEDELEG_DEFAULT; > vcpu->arch.mmu_page_cache.gfp_zero = __GFP_ZERO; > bitmap_zero(vcpu->arch.isa, RISCV_ISA_EXT_MAX); > > @@ -570,7 +572,6 @@ static void kvm_riscv_vcpu_setup_config(struct kvm_vcpu *vcpu) > cfg->hstateen0 |= SMSTATEEN0_SSTATEEN0; > } > > - cfg->hedeleg = KVM_HEDELEG_DEFAULT; > if (vcpu->guest_debug) > cfg->hedeleg &= ~BIT(EXC_BREAKPOINT); > } > -- > 2.43.0 > Reviewed-by: Andrew Jones <ajones@xxxxxxxxxxxxxxxx>
