Re: [PATCH v2 2/3] i386/cpuid: Remove subleaf constraint on CPUID leaf 1F

[Yang, Weijiang]

On 6/13/2023 9:19 PM, Xiaoyao Li wrote:

> No such constraint that subleaf index needs to be less than 64.
>
> Signed-off-by: Xiaoyao Li <xiaoyao.li@intel.com>
> ---
>   target/i386/kvm/kvm.c | 4 ----
>   1 file changed, 4 deletions(-)
>
> diff --git a/target/i386/kvm/kvm.c b/target/i386/kvm/kvm.c
> index afa97799d89a..d7e235ce35a6 100644
> --- a/target/i386/kvm/kvm.c
> +++ b/target/i386/kvm/kvm.c
> @@ -1968,10 +1968,6 @@ int kvm_arch_init_vcpu(CPUState *cs)
>                       break;
>                   }
>   
> -                if (i == 0x1f && j == 64) {
> -                    break;
> -                }
> -
>                   c->function = i;
>                   c->flags = KVM_CPUID_FLAG_SIGNIFCANT_INDEX;
>                   c->index = j;
Reviewed-by:Yang Weijiang <weijiang.yang@intel.com>