Re: [edk2-devel] [Patch V4 0/5] Eliminate the second INIT-SIPI-SIPI sequence

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

Gard, any comments?

> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Ni, Ray
> Sent: Wednesday, July 5, 2023 10:49 AM
> To: devel@edk2.groups.io; Xie, Yuanhao <yuanhao.xie@intel.com>
> Subject: Re: [edk2-devel] [Patch V4 0/5] Eliminate the second INIT-SIPI-SIPI
> sequence
> 
> Reviewed-by: Ray Ni <Ray.ni@intel.com>
> 
> > -----Original Message-----
> > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Yuanhao
> Xie
> > Sent: Wednesday, June 28, 2023 4:47 PM
> > To: devel@edk2.groups.io
> > Subject: [edk2-devel] [Patch V4 0/5] Eliminate the second INIT-SIPI-SIPI
> > sequence
> >
> > To speed up MP initialization, this set of patches replaces the
> > time-consuming init-sipi-sipi process in the DXE phase. Instead,
> > a start-up signal is used to wake up the APs and switch context
> >  from the PEI phase to the DXE phase. This optimization is
> >  effective when both PEI and DXE operate in the same bit mode.
> > The current HOB characterized by mCpuInitMpLibHobGuid has
> > two purposes:
> >   Acting as a global variable for the PEI phase.
> >   Transferring information from the PEI phase to the DXE phase.
> > This series of patches creates a new HOB specifically designed
> > to transfer only the minimal necessary information
> > (MpHandoff structure) from the PEI phase to the DXE phase.
> >
> > YuanhaoXie (5):
> >   UefiCpuPkg: Refactor the logic for placing APs in HltLoop.
> >   UefiCpuPkg: Refactor the logic for placing APs in Mwait/Runloop.
> >   UefiCpuPkg: Create MpHandOff.
> >   UefiCpuPkg: ApWakeupFunction directly use CpuMpData.
> >   UefiCpuPkg: Eliminate the second INIT-SIPI-SIPI sequence.
> >
> >  UefiCpuPkg/Library/MpInitLib/AmdSev.c          |  17 +++++++++++++++++
> >  UefiCpuPkg/Library/MpInitLib/DxeMpInitLib.inf  |   1 +
> >  UefiCpuPkg/Library/MpInitLib/Ia32/MpFuncs.nasm |   4 ++--
> >  UefiCpuPkg/Library/MpInitLib/MpHandOff.h       |  47
> > +++++++++++++++++++++++++++++++++++++++++++++++
> >  UefiCpuPkg/Library/MpInitLib/MpLib.c           | 345
> >
> +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
> >
> +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
> >
> +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
> >
> +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
> > ++++++++++++++++++++-----------------------------------------------------------------
> >  UefiCpuPkg/Library/MpInitLib/MpLib.h           |  28
> > ++++++++++++++++++++++++++++
> >  UefiCpuPkg/Library/MpInitLib/PeiMpInitLib.inf  |   2 +-
> >  UefiCpuPkg/Library/MpInitLib/PeiMpLib.c        |  32
> > +++++++++++++++++++++++++++++++-
> >  UefiCpuPkg/Library/MpInitLib/X64/MpFuncs.nasm  |   3 +--
> >  9 files changed, 408 insertions(+), 71 deletions(-)
> >  create mode 100644 UefiCpuPkg/Library/MpInitLib/MpHandOff.h
> >
> > --
> > 2.36.1.windows.1
> >
> >
> >
> >
> >
> 
> 
> 
> 
>