Message ID | 20240517173926.965351-35-seanjc@google.com (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
Series | KVM: x86: CPUID overhaul, fixes, and caching | expand |
On Fri, 2024-05-17 at 10:39 -0700, Sean Christopherson wrote: > Unconditionally advertise "support" for the HYPERVISOR feature in CPUID, > as the flag simply communicates to the guest that's it's running under a > hypervisor. > > Signed-off-by: Sean Christopherson <seanjc@google.com> > --- > arch/x86/kvm/cpuid.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/arch/x86/kvm/cpuid.c b/arch/x86/kvm/cpuid.c > index d1f427284ccc..de898d571faa 100644 > --- a/arch/x86/kvm/cpuid.c > +++ b/arch/x86/kvm/cpuid.c > @@ -681,7 +681,8 @@ void kvm_set_cpu_caps(void) > F(PCID) | 0 /* Reserved, DCA */ | F(XMM4_1) | > F(XMM4_2) | EMUL_F(X2APIC) | F(MOVBE) | F(POPCNT) | > EMUL_F(TSC_DEADLINE_TIMER) | F(AES) | F(XSAVE) | > - 0 /* OSXSAVE */ | F(AVX) | F(F16C) | F(RDRAND) > + 0 /* OSXSAVE */ | F(AVX) | F(F16C) | F(RDRAND) | > + EMUL_F(HYPERVISOR) > ); > > kvm_cpu_cap_init(CPUID_1_EDX, This makes sense. Reviewed-by: Maxim Levitsky <mlevitsk@redhat.com> Best regards, Maxim Levitsky
diff --git a/arch/x86/kvm/cpuid.c b/arch/x86/kvm/cpuid.c index d1f427284ccc..de898d571faa 100644 --- a/arch/x86/kvm/cpuid.c +++ b/arch/x86/kvm/cpuid.c @@ -681,7 +681,8 @@ void kvm_set_cpu_caps(void) F(PCID) | 0 /* Reserved, DCA */ | F(XMM4_1) | F(XMM4_2) | EMUL_F(X2APIC) | F(MOVBE) | F(POPCNT) | EMUL_F(TSC_DEADLINE_TIMER) | F(AES) | F(XSAVE) | - 0 /* OSXSAVE */ | F(AVX) | F(F16C) | F(RDRAND) + 0 /* OSXSAVE */ | F(AVX) | F(F16C) | F(RDRAND) | + EMUL_F(HYPERVISOR) ); kvm_cpu_cap_init(CPUID_1_EDX,
Unconditionally advertise "support" for the HYPERVISOR feature in CPUID, as the flag simply communicates to the guest that's it's running under a hypervisor. Signed-off-by: Sean Christopherson <seanjc@google.com> --- arch/x86/kvm/cpuid.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)