On Wed, 31 Oct 2018 at 03:25, Jim Mattson <jmattson@xxxxxxxxxx> wrote:
>
> If userspace has provided a different value for this MSR (e.g with the
> turbo bits set), the userspace-provided value should survive a vCPU
> reset. For backwards compatibility, MSR_PLATFORM_INFO is initialized
> in kvm_arch_vcpu_setup.
>
> Signed-off-by: Jim Mattson <jmattson@xxxxxxxxxx>
> Reviewed-by: Drew Schmitt <dasch@xxxxxxxxxx>
> Cc: Abhiroop Dabral <adabral@xxxxxxxxxxxxxxxxxxxx>
> ---
> arch/x86/kvm/x86.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/x86/kvm/x86.c b/arch/x86/kvm/x86.c
> index 66d66d77caee..6ab3149bf9dc 100644
> --- a/arch/x86/kvm/x86.c
> +++ b/arch/x86/kvm/x86.c
> @@ -8617,6 +8617,7 @@ struct kvm_vcpu *kvm_arch_vcpu_create(struct kvm *kvm,
>
> int kvm_arch_vcpu_setup(struct kvm_vcpu *vcpu)
> {
> + vcpu->arch.msr_platform_info = MSR_PLATFORM_INFO_CPUID_FAULT;
> kvm_vcpu_mtrr_init(vcpu);
> vcpu_load(vcpu);
> kvm_vcpu_reset(vcpu, false);
> @@ -8719,7 +8720,6 @@ void kvm_vcpu_reset(struct kvm_vcpu *vcpu, bool init_event)
> kvm_pmu_reset(vcpu);
> vcpu->arch.smbase = 0x30000;
>
> - vcpu->arch.msr_platform_info = MSR_PLATFORM_INFO_CPUID_FAULT;
I think you miss the init_event check?
Regards,
Wanpeng Li
> vcpu->arch.msr_misc_features_enables = 0;
>
> vcpu->arch.xcr0 = XFEATURE_MASK_FP;
> --
> 2.19.1.568.g152ad8e336-goog
>
