Re: [Xen-devel] [PATCH 2/2] x86/cpuid: Alter the policy logic for leaf 0xb to be multi-invocation

[Wei Liu <wei.liu2@xxxxxxxxxx>]

On Wed, Jun 27, 2018 at 04:58:08PM +0100, Andrew Cooper wrote:
> On 27/06/18 15:58, Jan Beulich wrote:
> >>>> On 27.06.18 at 15:55, <andrew.cooper3@xxxxxxxxxx> wrote:
> >> @@ -316,6 +319,33 @@ static void __init calculate_raw_policy(void)
> >>              cpuid_count_leaf(7, i, &p->feat.raw[i]);
> >>      }
> >>  
> >> +    if ( p->basic.max_leaf >= 0xb )
> >> +    {
> >> +        union {
> >> +            struct cpuid_leaf l;
> >> +            struct cpuid_topo_leaf t;
> >> +        } u;
> >> +
> >> +        for ( i = 0; i < ARRAY_SIZE(p->topo.raw); ++i )
> >> +        {
> >> +            cpuid_count_leaf(0xb, i, &u.l);
> >> +
> >> +            if ( u.t.type == 0 )
> >> +                break;
> >> +
> >> +            p->topo.subleaf[i] = u.t;
> >> +        }
> >> +
> >> +        /*
> >> +         * The choice of CPUID_GUEST_NR_TOPO is per the manual.  It may 
> >> need
> >> +         * to grow for future harware.
> > Missing d.
> 
> Where?  I'm afraid that after repeated re-reads, I can't spot any issue.
> 

har d ware. ;p

Wei.

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel