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 BIT24 clearly tells that processor number only occupies the lower 24
> bits.
Yes, I've noticed this discrepancy too; I agree BIT24 is clearer here!
>
>
>>> return EFI_NOT_FOUND;
>>> }
>>>
>>
>> Reviewed-by: Laszlo Ersek <ler...@redhat.com>
>
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#113287): https://edk2.groups.io/g/devel/message/113287
Mute This Topic: https://groups.io/mt/103518743/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe:
https://edk2.groups.io/g/devel/leave/9847357/21656/1706620634/xyzzy
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-
