Reviewed-by: Ray Ni <[email protected]> > -----Original Message----- > From: Wu, Jiaxin <[email protected]> > Sent: Monday, February 13, 2023 4:44 PM > To: [email protected] > Cc: Dong, Eric <[email protected]>; Ni, Ray <[email protected]>; Zeng, Star > <[email protected]>; Laszlo Ersek <[email protected]>; Gerd Hoffmann > <[email protected]>; Kumar, Rahul R <[email protected]> > Subject: [PATCH v6 1/6] UefiCpuPkg/PiSmmCpuDxeSmm: Fix invalid > InitializeMpSyncData call > > REF: https://bugzilla.tianocore.org/show_bug.cgi?id=4338 > > No need call InitializeMpSyncData during normal boot SMI init, > because mSmmMpSyncData is NULL at that time. mSmmMpSyncData is > allocated in InitializeMpServiceData, which is invoked after > normal boot SMI init (SmmRelocateBases). > > Cc: Eric Dong <[email protected]> > Cc: Ray Ni <[email protected]> > Cc: Zeng Star <[email protected]> > Cc: Laszlo Ersek <[email protected]> > Cc: Gerd Hoffmann <[email protected]> > Cc: Rahul Kumar <[email protected]> > Signed-off-by: Jiaxin Wu <[email protected]> > --- > UefiCpuPkg/PiSmmCpuDxeSmm/PiSmmCpuDxeSmm.c | 6 ++---- > 1 file changed, 2 insertions(+), 4 deletions(-) > > diff --git a/UefiCpuPkg/PiSmmCpuDxeSmm/PiSmmCpuDxeSmm.c > b/UefiCpuPkg/PiSmmCpuDxeSmm/PiSmmCpuDxeSmm.c > index 655175a2c6..2ac655d032 100644 > --- a/UefiCpuPkg/PiSmmCpuDxeSmm/PiSmmCpuDxeSmm.c > +++ b/UefiCpuPkg/PiSmmCpuDxeSmm/PiSmmCpuDxeSmm.c > @@ -1,9 +1,9 @@ > /** @file > Agent Module to load other modules to deploy SMM Entry Vector for X86 > CPU. > > -Copyright (c) 2009 - 2019, Intel Corporation. All rights reserved.<BR> > +Copyright (c) 2009 - 2023, Intel Corporation. All rights reserved.<BR> > Copyright (c) 2017, AMD Incorporated. All rights reserved.<BR> > > SPDX-License-Identifier: BSD-2-Clause-Patent > > **/ > @@ -369,13 +369,11 @@ SmmInitHandler ( > if (!mSmmS3Flag) { > // > // Check XD and BTS features on each processor on normal boot > // > CheckFeatureSupported (); > - } > - > - if (mIsBsp) { > + } else if (mIsBsp) { > // > // BSP rebase is already done above. > // Initialize private data during S3 resume > // > InitializeMpSyncData (); > -- > 2.16.2.windows.1
-=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#100068): https://edk2.groups.io/g/devel/message/100068 Mute This Topic: https://groups.io/mt/96931997/21656 Group Owner: [email protected] Unsubscribe: https://edk2.groups.io/g/devel/leave/9847357/21656/1706620634/xyzzy [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
