On 10/29/15 02:32, Jordan Justen wrote:
> InitializeMpSupport will be better able to coordinate with the APs to
> find out if the maximum number of processors have been seen, and
> therefore to stop waiting for APs earlier.
>
> Contributed-under: TianoCore Contribution Agreement 1.0
> Signed-off-by: Jordan Justen <jordan.l.jus...@intel.com>
> Cc: Jeff Fan <jeff....@intel.com>
> Cc: Laszlo Ersek <ler...@redhat.com>
> ---
> UefiCpuPkg/CpuDxe/ApStartup.c | 4 ----
> UefiCpuPkg/CpuDxe/CpuMp.c | 5 +++++
> 2 files changed, 5 insertions(+), 4 deletions(-)
>
> diff --git a/UefiCpuPkg/CpuDxe/ApStartup.c b/UefiCpuPkg/CpuDxe/ApStartup.c
> index 78fb26f..f4560d3 100644
> --- a/UefiCpuPkg/CpuDxe/ApStartup.c
> +++ b/UefiCpuPkg/CpuDxe/ApStartup.c
> @@ -451,10 +451,6 @@ StartApsStackless (
> )
> {
> SendInitSipiSipiAllExcludingSelf ((UINT32)(UINTN)(VOID*) StartupCode);
> - //
> - // Wait for APs to arrive at the ApEntryPoint routine
> - //
> - MicroSecondDelay (PcdGet32 (PcdCpuApInitTimeOutInMicroSeconds));
>
> return EFI_SUCCESS;
> }
> diff --git a/UefiCpuPkg/CpuDxe/CpuMp.c b/UefiCpuPkg/CpuDxe/CpuMp.c
> index e80835f..6a22b9d 100644
> --- a/UefiCpuPkg/CpuDxe/CpuMp.c
> +++ b/UefiCpuPkg/CpuDxe/CpuMp.c
> @@ -1747,6 +1747,11 @@ InitializeMpSupport (
> PrepareAPStartupCode ();
>
> StartApsStackless ();
> +
> + //
> + // Wait for APs to arrive at the ApEntryPoint routine
> + //
> + MicroSecondDelay (PcdGet32 (PcdCpuApInitTimeOutInMicroSeconds));
> }
>
> DEBUG ((DEBUG_INFO, "Detect CPU count: %d\n",
> mMpSystemData.NumberOfProcessors));
>
Reviewed-by: Laszlo Ersek <ler...@redhat.com>
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel
