Re: [PATCH v3 0/2] UefiCpuPkg/MpInitLib: Fix ASSERT in AP procedure

["Ni, Ray" <ray.ni@intel.com>]

Send to correct mail address: devel@edk2.groups.io

> -----Original Message-----
> From: Ni, Ray
> Sent: Sunday, April 26, 2020 11:46 AM
> To: Dong, Eric <eric.dong@intel.com>; \x16devel@edk2.groups.io
> Cc: Laszlo Ersek <lersek@redhat.com>; Kumar, Chandana C <chandana.c.kumar@intel.com>
> Subject: RE: [PATCH v3 0/2] UefiCpuPkg/MpInitLib: Fix ASSERT in AP procedure
> 
> Reviewed-by: Ray Ni <ray.ni@intel.com>
> 
> > -----Original Message-----
> > From: Dong, Eric <eric.dong@intel.com>
> > Sent: Friday, April 24, 2020 4:47 PM
> > To: \x16devel@edk2.groups.io
> > Cc: Ni, Ray <ray.ni@intel.com>; Laszlo Ersek <lersek@redhat.com>; Kumar,
> > Chandana C <chandana.c.kumar@intel.com>
> > Subject: [PATCH v3 0/2] UefiCpuPkg/MpInitLib: Fix ASSERT in AP procedure
> >
> > REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2683
> >
> >
> >
> > This patch serial used to fix an ASSERT issue. Because AP can't find
> >
> > the CpuMpData through IDT, it raised the ASSERT.
> >
> >
> >
> > V3:
> >
> > 1. Remove invalid save Volatile Registers process. Refine restore
> >
> >    Volatile Registers process.
> >
> >
> >
> > V2:
> >
> > 1. Enhance code comments.
> >
> > 2. Enhance code to remove CpuMpData->ApLoopMode == ApInHltLoop
> > check.
> >
> >
> >
> > Cc: Ray Ni <ray.ni@intel.com>
> >
> > Cc: Laszlo Ersek <lersek@redhat.com>
> >
> > Cc: Chandana Kumar <chandana.c.kumar@intel.com>
> >
> >
> >
> > Eric Dong (2):
> >   UefiCpuPkg/MpInitLib: Restore IDT context for APs.
> >   UefiCpuPkg/MpInitLib: Avoid ApInitReconfig in PEI.
> >
> >  UefiCpuPkg/Library/MpInitLib/MpLib.c | 47 ++++++++++++++++++++--------
> >  1 file changed, 34 insertions(+), 13 deletions(-)
> >
> > --
> > 2.23.0.windows.1