> > - 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.
> > 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 (#113281): https://edk2.groups.io/g/devel/message/113281 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] -=-=-=-=-=-=-=-=-=-=-=-