Re: [PATCH] x86: Put the num_processors++ code in a more suitable position

[Dou Liyang]

Hi David,

At 09/07/2016 05:23 AM, David Rientjes wrote:
> On Tue, 6 Sep 2016, Dou Liyang wrote:
>
> > This is a code optimization.
>
> Not sure that it's optimization, it's just for correctness.

Yes, I see. I will improve it in next version.

Thanks,
Dou

> > If checking the topology package map of apicid and cpu is failure,
> > it will stop generating the processor info for that apicid and the
> > disabled_cpus will plus one. However, the num-processors has already
> > been added one above. That may cause the number of processors incorrect.
> >
> > Just put the num_processors++ code in the more suitable position.
> > it makes sure that the num-processors will not conflict with the
> > disabled_cpus.
> >
> > Signed-off-by: Dou Liyang <douly.fnst@xxxxxxxxxxxxxx>
>
> Acked-by: David Rientjes <rientjes@xxxxxxxxxx>