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

From: David Rientjes
Date: Tue Sep 06 2016 - 17:23:58 EST


On Tue, 6 Sep 2016, Dou Liyang wrote:

> This is a code optimization.
>

Not sure that it's optimization, it's just for correctness.

> 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>