Reviewed-by: Jiaxin Wu <jiaxin...@intel.com> > -----Original Message----- > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Ni, Ray > Sent: Wednesday, May 10, 2023 4:42 PM > To: Li, Zhihao <zhihao...@intel.com>; devel@edk2.groups.io > Cc: Dong, Eric <eric.d...@intel.com> > Subject: Re: [edk2-devel] [PATCH v1 1/1] UefiCpuPkg/CpuService.c:check > cpu sync mode in SmmCpuRendezvous() > > Reviewed-by: Ray Ni <ray...@intel.com> > > > -----Original Message----- > > From: Li, Zhihao <zhihao...@intel.com> > > Sent: Wednesday, May 10, 2023 4:38 PM > > To: devel@edk2.groups.io > > Cc: Dong, Eric <eric.d...@intel.com>; Ni, Ray <ray...@intel.com> > > Subject: [PATCH v1 1/1] UefiCpuPkg/CpuService.c:check cpu sync mode in > > SmmCpuRendezvous() > > > > REF: https://bugzilla.tianocore.org/show_bug.cgi?id=4431 > > > > In Ap relaxed mode, some SMI handlers should call SmmWaitForApArrival() > to let > > all ap arrive in SmmCpuRendezvous(). But in traditional mode, these SMI > > handlers don't need to call SmmWaitForApArrival() again. So it need to be > check > > cpu sync mode before calling SmmWaitForApArrival(). > > > > Cc: Eric Dong <eric.d...@intel.com> > > Cc: Ray Ni <ray...@intel.com> > > > > Signed-off-by: Zhihao Li <zhihao...@intel.com> > > --- > > UefiCpuPkg/PiSmmCpuDxeSmm/CpuService.c | 19 +++++++++++++------ > > 1 file changed, 13 insertions(+), 6 deletions(-) > > > > diff --git a/UefiCpuPkg/PiSmmCpuDxeSmm/CpuService.c > > b/UefiCpuPkg/PiSmmCpuDxeSmm/CpuService.c > > index 2ebf4543c3ed..391b64e9f222 100644 > > --- a/UefiCpuPkg/PiSmmCpuDxeSmm/CpuService.c > > +++ b/UefiCpuPkg/PiSmmCpuDxeSmm/CpuService.c > > @@ -1,7 +1,7 @@ > > /** @file > > > > Implementation of SMM CPU Services Protocol. > > > > > > > > -Copyright (c) 2011 - 2022, Intel Corporation. All rights reserved.<BR> > > > > +Copyright (c) 2011 - 2023, Intel Corporation. All rights reserved.<BR> > > > > SPDX-License-Identifier: BSD-2-Clause-Patent > > > > > > > > **/ > > > > @@ -421,11 +421,18 @@ SmmCpuRendezvous ( > > goto ON_EXIT; > > > > } > > > > > > > > - // > > > > - // There are some APs outside SMM, Wait for all avaiable APs to arrive. > > > > - // > > > > - SmmWaitForApArrival (); > > > > - Status = mSmmMpSyncData->AllApArrivedWithException ? EFI_SUCCESS : > > EFI_TIMEOUT; > > > > + if ((mSmmMpSyncData->EffectiveSyncMode != > SmmCpuSyncModeTradition) > > && !SmmCpuFeaturesNeedConfigureMtrrs ()) { > > > > + // > > > > + // There are some APs outside SMM, Wait for all avaiable APs to arrive. > > > > + // > > > > + SmmWaitForApArrival (); > > > > + Status = mSmmMpSyncData->AllApArrivedWithException ? > EFI_SUCCESS : > > EFI_TIMEOUT; > > > > + } else { > > > > + // > > > > + // BSP has already waitted for APs to arrive SMM if SmmCpuSyncMode > > selected or need config MTRR. > > > > + // > > > > + Status = EFI_TIMEOUT; > > > > + } > > > > > > > > ON_EXIT: > > > > if (!mSmmMpSyncData->AllApArrivedWithException) { > > > > -- > > 2.26.2.windows.1 > > > > >
-=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#105507): https://edk2.groups.io/g/devel/message/105507 Mute This Topic: https://groups.io/mt/98801800/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-