Reviewed-by: Ruiyu Ni <ruiyu...@intel.com>
Thanks/Ray
> -----Original Message-----
> From: Dong, Eric
> Sent: Wednesday, August 30, 2017 10:22 AM
> To: edk2-devel@lists.01.org
> Cc: Kinney, Michael D <michael.d.kin...@intel.com>; Ni, Ruiyu
> <ruiyu...@intel.com>
> Subject: [Patch] UefiCpuPkg/Mplib.c: Perform complete initialization when
> enable AP.
>
> PI has description said If an AP is enabled, then the implementation must
> guarantee that a complete initialization sequence is performed on the AP, so
> the AP is in a state that is compatible with an MP operating system.
> Current implementation just set the AP to idle state when enable this AP
> which is not follow spec. This patch fix it.
>
> Cc: Michael Kinney <michael.d.kin...@intel.com>
> Cc: Ruiyu Ni <ruiyu...@intel.com>
> Contributed-under: TianoCore Contribution Agreement 1.1
> Signed-off-by: Eric Dong <eric.d...@intel.com>
> ---
> UefiCpuPkg/Library/MpInitLib/MpLib.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/UefiCpuPkg/Library/MpInitLib/MpLib.c
> b/UefiCpuPkg/Library/MpInitLib/MpLib.c
> index 8394572..924b909 100644
> --- a/UefiCpuPkg/Library/MpInitLib/MpLib.c
> +++ b/UefiCpuPkg/Library/MpInitLib/MpLib.c
> @@ -1814,7 +1814,7 @@ EnableDisableApWorker (
> if (!EnableAP) {
> SetApState (&CpuMpData->CpuData[ProcessorNumber],
> CpuStateDisabled);
> } else {
> - SetApState (&CpuMpData->CpuData[ProcessorNumber], CpuStateIdle);
> + ResetProcessorToIdleState (ProcessorNumber);
> }
>
> if (HealthFlag != NULL) {
> --
> 2.7.0.windows.1
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel
