On Thu, Sep 12, 2019 at 10:41 AM Krish Sadhukhan <krish.sadhukhan@xxxxxxxxxx> wrote: > > > On 9/12/19 9:55 AM, Jim Mattson wrote: > > According to the Intel SDM, volume 2, "CPUID," the index is > > significant (or partially significant) for CPUID leaves 0FH, 10H, 12H, > > 17H, 18H, and 1FH. > > > > Add the corresponding flag to these CPUID leaves in do_host_cpuid(). > > > > Signed-off-by: Jim Mattson <jmattson@xxxxxxxxxx> > > Reviewed-by: Peter Shier <pshier@xxxxxxxxxx> > > Reviewed-by: Steve Rutherford <srutherford@xxxxxxxxxx> > > Fixes: a87f2d3a6eadab ("KVM: x86: Add Intel CPUID.1F cpuid emulation support") > > --- > > arch/x86/kvm/cpuid.c | 6 ++++++ > > 1 file changed, 6 insertions(+) > > > > diff --git a/arch/x86/kvm/cpuid.c b/arch/x86/kvm/cpuid.c > > index 22c2720cd948e..e7d25f4364664 100644 > > --- a/arch/x86/kvm/cpuid.c > > +++ b/arch/x86/kvm/cpuid.c > > @@ -304,7 +304,13 @@ static void do_host_cpuid(struct kvm_cpuid_entry2 *entry, u32 function, > > case 7: > > case 0xb: > > case 0xd: > > + case 0xf: > > + case 0x10: > > + case 0x12: > > case 0x14: > > + case 0x17: > > + case 0x18: > > + case 0x1f: > > case 0x8000001d: > > entry->flags |= KVM_CPUID_FLAG_SIGNIFCANT_INDEX; > > break; > > > Reviewed-by: Krish Sadhukhan <krish.sadhukhan@xxxxxxxxxx> Ping.