Revision: 8bd2028f9ac391144c67eaf6eb08c3f727c32498 Author: Kuo, Ted <ted....@intel.com> Date: 12/16/2022 8:46:27 PM Message: MdeModulePkg: Supporting S3 in 64bit PEI
https://bugzilla.tianocore.org/show_bug.cgi?id=4195 Transfer from DXE to OS waking vector by calling SwitchStack() when both are in the same execution mode. Reviewed-by: Ray Ni <ray...@intel.com> Reviewed-by: Zhiguang Liu <zhiguang....@intel.com> Cc: Chasel Chiu <chasel.c...@intel.com> Cc: Nate DeSimone <nathaniel.l.desim...@intel.com> Cc: Star Zeng <star.z...@intel.com> Cc: Ashraf Ali S <ashraf.al...@intel.com> Cc: Chinni B Duggapu <chinni.b.dugg...@intel.com> Signed-off-by: Ted Kuo <ted....@intel.com> ---- Modified: MdeModulePkg/Universal/Acpi/BootScriptExecutorDxe/ScriptExecute.c Modified: MdeModulePkg/Universal/Acpi/BootScriptExecutorDxe/X64/SetIdtEntry.c Modified: MdeModulePkg/Universal/Acpi/S3SaveStateDxe/AcpiS3ContextSave.c Revision: 6acf72901a2e811a2838cafd496239a34779066a Author: Kuo, Ted <ted....@intel.com> Date: 12/16/2022 8:46:26 PM Message: UefiCpuPkg: Supporting S3 in 64bit PEI https://bugzilla.tianocore.org/show_bug.cgi?id=4195 1.Updated the GDT table in VTF0 to align with the one in S3Resume2Pei. By doing so can simplify the changes to enable S3 in 64bit PEI. 2.Use SwitchStack() between PEI and SMM in S3 resume path when both are in the same execution mode. 3.Transfer from PEI to OS waking vector by calling SwitchStack() when both are in the same execution mode. 4.Removed the debug assertion in S3Resume.c to support 64bit PEI. Reviewed-by: Ray Ni <ray...@intel.com> Reviewed-by: Zhiguang Liu <zhiguang....@intel.com> Cc: Chasel Chiu <chasel.c...@intel.com> Cc: Nate DeSimone <nathaniel.l.desim...@intel.com> Cc: Star Zeng <star.z...@intel.com> Cc: Ashraf Ali S <ashraf.al...@intel.com> Cc: Chinni B Duggapu <chinni.b.dugg...@intel.com> Signed-off-by: Ted Kuo <ted....@intel.com> ---- Modified: UefiCpuPkg/PiSmmCpuDxeSmm/CpuS3.c Modified: UefiCpuPkg/PiSmmCpuDxeSmm/PiSmmCpuDxeSmm.inf Modified: UefiCpuPkg/ResetVector/Vtf0/Ia16/Real16ToFlat32.asm Modified: UefiCpuPkg/Universal/Acpi/S3Resume2Pei/S3Resume.c > -----Original Message----- > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Laszlo Ersek > Sent: Tuesday, May 23, 2023 1:46 PM > To: Gerd Hoffmann <kra...@redhat.com>; Ni, Ray <ray...@intel.com> > Cc: Ard Biesheuvel <a...@kernel.org>; edk2-devel-groups-io > <devel@edk2.groups.io>; Yao, Jiewen <jiewen....@intel.com>; Taylor Beebe > <t...@taylorbeebe.com>; Oliver Smith-Denny <o...@smith-denny.com> > Subject: Re: [edk2-devel] managing memory attributes in PEI > > On 5/23/23 06:49, Gerd Hoffmann wrote: > > On Mon, May 22, 2023 at 11:20:16PM +0000, Ni, Ray wrote: > >> Gerd, > >> The S3 path has been 64bit ready. > >> Can you check if OVMF pei64 image can do s3? > >> > >> At least internally in some real platform we tested s3 flow with 64bit PEI. > > > > Tested on OVMF, passed too. > > > > Which are the commits implementing this? I'd like the reference them in > > the commit message for S3Verification() removal. > > commit 6acf72901a2e / bug 4195, AFAIK; there could be other related > commits since I last looked, of course > > > > > -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#105148): https://edk2.groups.io/g/devel/message/105148 Mute This Topic: https://groups.io/mt/99062463/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/leave/9847357/21656/1706620634/xyzzy [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
