Check lower 24 bits of ProcessorNumber instead of
the value of ProcessorNumber in the API
MpInitLibGetProcessorInfo() of MpInitLibUp instance.
Lower 24 bits of ProcessorNumber contains the actual
processor number.
The BIT24 of input ProcessorNumber might be set to
indicate if the EXTENDED_PROCESSOR
On 1/4/24 08:32, duntan wrote:
> Check lower 24 bits of ProcessorNumber instead of
> the value of ProcessorNumber in the API
> MpInitLibGetProcessorInfo() of MpInitLibUp instance.
> Lower 24 bits of ProcessorNumber contains the actual
> processor number.
> The BIT24 of input ProcessorNumber might b
> > - if (ProcessorNumber != 0) {
> > + //
> > + // Lower 24 bits contains the actual processor number.
> > + //
> > + if ((ProcessorNumber & (CPU_V2_EXTENDED_TOPOLOGY - 1)) != 0) {
I suggest we explicitly use BIT24 instead of CPU_V2_EXTENDED_TOPOLOGY.
Using BIT24 clearly tells that processor
On 1/5/24 13:55, Ni, Ray wrote:
>>> - if (ProcessorNumber != 0) {
>>> + //
>>> + // Lower 24 bits contains the actual processor number.
>>> + //
>>> + if ((ProcessorNumber & (CPU_V2_EXTENDED_TOPOLOGY - 1)) != 0) {
> I suggest we explicitly use BIT24 instead of CPU_V2_EXTENDED_TOPOLOGY.
> Using
] [PATCH 2/2] UefiCpuPkg: Check lower 24 bits of
ProcessorNumber
On 1/5/24 13:55, Ni, Ray wrote:
>>> - if (ProcessorNumber != 0) {
>>> + //
>>> + // Lower 24 bits contains the actual processor number.
>>> + //
>>> + if ((ProcessorNumber & (CPU_