On Tue, Jun 4, 2019 at 10:52 AM Krish Sadhukhan <krish.sadhukhan@xxxxxxxxxx> wrote: > > > On 5/31/19 11:41 AM, Aaron Lewis wrote: > > According to the SDM, bit 0 and bits 63:10 of the IA32_VMX_VMCS_ENUM > > MSR are reserved and are read as 0. > > > > Signed-off-by: Aaron Lewis <aaronlewis@xxxxxxxxxx> > > Reviewed-by: Jim Mattson <jmattson@xxxxxxxxxx> > > Reviewed-by: Krish Sadhukhan <krish.sadhukhan@xxxxxxxxxx> > > --- > > arch/x86/kvm/vmx/nested.c | 2 ++ > > 1 file changed, 2 insertions(+) > > > > diff --git a/arch/x86/kvm/vmx/nested.c b/arch/x86/kvm/vmx/nested.c > > index 6401eb7ef19c..3438279e76bb 100644 > > --- a/arch/x86/kvm/vmx/nested.c > > +++ b/arch/x86/kvm/vmx/nested.c > > @@ -1219,6 +1219,8 @@ int vmx_set_vmx_msr(struct kvm_vcpu *vcpu, u32 msr_index, u64 data) > > case MSR_IA32_VMX_EPT_VPID_CAP: > > return vmx_restore_vmx_ept_vpid_cap(vmx, data); > > case MSR_IA32_VMX_VMCS_ENUM: > > + if (data & (GENMASK_ULL(63, 10) | BIT_ULL(0))) > > + return -EINVAL; > > vmx->nested.msrs.vmcs_enum = data; > > return 0; > > default: > > > Reviewed-by: Krish Sadhukhan <krish.sadhukhan@xxxxxxxxxx> > ping
