On 07/25/18 09:50, Eric Dong wrote: > This patch series include changes: > 1. StartAllAPs should not use disabled APs, this is required by UEFI spec. > 2. Refine the code to remove the redundant definitions. > > V2 changes: > Use CpuStateReady to distinguish the AP state from CpuStateIdle. > > V3 Changes: > Only change 3/3 patch. Only not use disabled APs when WakeUpAP called > by StartAllAps function. In other cases, also include disabled APs. > > Eric Dong (3): > UefiCpuPkg/MpInitLib: Remove redundant CpuStateFinished State. > UefiCpuPkg/MpInitLib: Remove StartCount and volatile definition. > UefiCpuPkg/MpInitLib: Not use disabled AP when call StartAllAPs. > > UefiCpuPkg/Library/MpInitLib/MpLib.c | 33 +++++++++++++++------------------ > UefiCpuPkg/Library/MpInitLib/MpLib.h | 4 +--- > 2 files changed, 16 insertions(+), 21 deletions(-) >
I requested commit message updates for all three patches. With those implemented, please add: Reviewed-by: Laszlo Ersek <[email protected]> (No need to repost.) Please give me some more time to regression-test this series as well. Thanks! Laszlo _______________________________________________ edk2-devel mailing list [email protected] https://lists.01.org/mailman/listinfo/edk2-devel
