On 1/8/24 06:08, duntan wrote: > Please ignore the V2 PATCH set. No other change except adding BaseMemoryLib > headfile and lib instance in .inf to pass build since ZeroMem() is used.
Reviewed-by: Laszlo Ersek <ler...@redhat.com> > > Comparing to the V1 patch set: > In "set EXTENDED_PROCESSOR_INFORMATION to 0", set > EXTENDED_PROCESSOR_INFORMATION to 0 in API MpInitLibGetProcessorInfo() of > MpInitLibUp. This commit use ZeroMem() to set all fileds in output > EFI_PROCESSOR_INFORMATION to 0 before StatusFlag field is reassigned. > > In "Check lower 24 bits of ProcessorNumber", use BIT24 instead of > CPU_V2_EXTENDED_TOPOLOGY to clearly tell that processor number only occupies > the lower 24 bits. > > Dun Tan (2): > UefiCpuPkg: set EXTENDED_PROCESSOR_INFORMATION to 0 > UefiCpuPkg: Check lower 24 bits of ProcessorNumber > > UefiCpuPkg/Include/Library/MpInitLib.h | 2 ++ > UefiCpuPkg/Library/MpInitLib/MpLib.c | 2 ++ > UefiCpuPkg/Library/MpInitLibUp/MpInitLibUp.c | 20 ++++++++++++-------- > UefiCpuPkg/Library/MpInitLibUp/MpInitLibUp.inf | 1 + > 4 files changed, 17 insertions(+), 8 deletions(-) > -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#113388): https://edk2.groups.io/g/devel/message/113388 Mute This Topic: https://groups.io/mt/103592277/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/leave/9847357/21656/1706620634/xyzzy [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-