Re: [edk2] [PATCH] EmulatorPkg/Sec: Don't install TemporaryRamSupport PPI

2019-03-05 Thread Ni, Ray
> -Original Message- > From: edk2-devel On Behalf Of Jordan > Justen > Sent: Wednesday, March 6, 2019 5:48 AM > To: Ni, Ray ; edk2-devel@lists.01.org > Cc: Kinney, Michael D ; Laszlo Ersek > > Subject: Re: [edk2] [PATCH] EmulatorPkg/Sec: Don't install >

Re: [edk2] [PATCH] EmulatorPkg/Sec: Don't install TemporaryRamSupport PPI

2019-03-05 Thread Jordan Justen
On 2019-03-04 18:37:03, Ni, Ray wrote: > > > -Original Message- > > From: edk2-devel On Behalf Of Jordan > > Justen > > Sent: Sunday, March 3, 2019 5:45 AM > > To: Ni, Ray ; edk2-devel@lists.01.org > > Subject: Re: [edk2] [PATCH] EmulatorPkg/Sec: Do

Re: [edk2] [PATCH] EmulatorPkg/Sec: Don't install TemporaryRamSupport PPI

2019-03-04 Thread Ni, Ray
> -Original Message- > From: edk2-devel On Behalf Of Jordan > Justen > Sent: Sunday, March 3, 2019 5:45 AM > To: Ni, Ray ; edk2-devel@lists.01.org > Subject: Re: [edk2] [PATCH] EmulatorPkg/Sec: Don't install > TemporaryRamSupport PPI > > On 2019-03-02 04

Re: [edk2] [PATCH] EmulatorPkg/Sec: Don't install TemporaryRamSupport PPI

2019-03-02 Thread Jordan Justen
On 2019-03-02 04:11:11, Ray Ni wrote: > TemporaryRamSupport PPI is called by PeiCore to migrate temporary > memory from permanent memory. But the implementation assumes > ebp/rbp contains the original esp/rsp value when migrating which > is not always true the compiler optimization is turned on. >

[edk2] [PATCH] EmulatorPkg/Sec: Don't install TemporaryRamSupport PPI

2019-03-02 Thread Ray Ni
TemporaryRamSupport PPI is called by PeiCore to migrate temporary memory from permanent memory. But the implementation assumes ebp/rbp contains the original esp/rsp value when migrating which is not always true the compiler optimization is turned on. A real boot failure is seen using GCC5. In fact,