Re: [edk2-devel] [RFC] Remove PI spec TemporaryRamSupport PPI from PEI Core

2023-01-12 Thread Ni, Ray
Nate, GCC generates assembly code that uses `RBP` to store `Private` local variable for below C code in PeiCore/Dispatcher.c ```c if (StackOffsetPositive) { SecCoreData = (CONST EFI_SEC_PEI_HAND_OFF *)((UINTN)(VOID *)SecCoreData + StackOffset); Private = (PEI_CORE_INSTA

Re: [edk2-devel] [RFC] Remove PI spec TemporaryRamSupport PPI from PEI Core

2019-04-24 Thread Nate DeSimone
Hi Jordan, The TemporaryRamSupport PPI is mandatory for FSP. The reason why is the stack migration algorithm for FSP is quite different compared to the standard method implemented in PEI core. During FspMemoryInit(), PEI core runs on top of the bootloader stack. PEI core is not the root of the

Re: [edk2-devel] [RFC] Remove PI spec TemporaryRamSupport PPI from PEI Core

2019-04-24 Thread Laszlo Ersek
On 04/24/19 09:51, Jordan Justen wrote: > Short summary: Despite being part of the Platorm Initialization > Specificication, this proposal would remove support for the > TemporaryRamSupport PPI from PEI Core. Arguably, this would mean the > PEI Core does not support the PI spec, but we do not curre