Re: [edk2] [PATCH 06/10] OvmfPkg/Sec: Disable optimizations for TemporaryRamMigration

2019-02-20 Thread Ard Biesheuvel
On Wed, 20 Feb 2019 at 09:52, Jordan Justen wrote: > > On 2019-02-18 01:32:53, Ard Biesheuvel wrote: > > On Mon, 18 Feb 2019 at 10:08, Jordan Justen > > wrote: > > > > > > On 2019-02-17 23:53:01, Ard Biesheuvel wrote: > > > > On Mon, 18 Feb 2019 at 05:12, Jordan Justen > > > > wrote: > > > > >

Re: [edk2] [PATCH 06/10] OvmfPkg/Sec: Disable optimizations for TemporaryRamMigration

2019-02-20 Thread Jordan Justen
On 2019-02-18 01:32:53, Ard Biesheuvel wrote: > On Mon, 18 Feb 2019 at 10:08, Jordan Justen wrote: > > > > On 2019-02-17 23:53:01, Ard Biesheuvel wrote: > > > On Mon, 18 Feb 2019 at 05:12, Jordan Justen > > > wrote: > > > > > > > > > > This needs an explanation why optimization needs to be disab

Re: [edk2] [PATCH 06/10] OvmfPkg/Sec: Disable optimizations for TemporaryRamMigration

2019-02-19 Thread Jordan Justen
On 2019-02-19 14:50:13, Brian J. Johnson wrote: > On 2/18/19 3:32 AM, Ard Biesheuvel wrote: > > On Mon, 18 Feb 2019 at 10:08, Jordan Justen > > wrote: > >> > >> On 2019-02-17 23:53:01, Ard Biesheuvel wrote: > >>> On Mon, 18 Feb 2019 at 05:12, Jordan Justen > >>> wrote: > > >>> > >>> This n

Re: [edk2] [PATCH 06/10] OvmfPkg/Sec: Disable optimizations for TemporaryRamMigration

2019-02-19 Thread Brian J. Johnson
On 2/18/19 3:32 AM, Ard Biesheuvel wrote: On Mon, 18 Feb 2019 at 10:08, Jordan Justen wrote: On 2019-02-17 23:53:01, Ard Biesheuvel wrote: On Mon, 18 Feb 2019 at 05:12, Jordan Justen wrote: This needs an explanation why optimization needs to be disabled. I'm not sure this is required.

Re: [edk2] [PATCH 06/10] OvmfPkg/Sec: Disable optimizations for TemporaryRamMigration

2019-02-18 Thread Laszlo Ersek
On 02/18/19 10:32, Ard Biesheuvel wrote: > On Mon, 18 Feb 2019 at 10:08, Jordan Justen wrote: >> >> On 2019-02-17 23:53:01, Ard Biesheuvel wrote: >>> On Mon, 18 Feb 2019 at 05:12, Jordan Justen >>> wrote: >>> >>> This needs an explanation why optimization needs to be disabled. >> >> I'm not

Re: [edk2] [PATCH 06/10] OvmfPkg/Sec: Disable optimizations for TemporaryRamMigration

2019-02-18 Thread Ard Biesheuvel
On Mon, 18 Feb 2019 at 10:08, Jordan Justen wrote: > > On 2019-02-17 23:53:01, Ard Biesheuvel wrote: > > On Mon, 18 Feb 2019 at 05:12, Jordan Justen > > wrote: > > > > > > > This needs an explanation why optimization needs to be disabled. > > I'm not sure this is required. The reason I added the

Re: [edk2] [PATCH 06/10] OvmfPkg/Sec: Disable optimizations for TemporaryRamMigration

2019-02-18 Thread Jordan Justen
On 2019-02-17 23:53:01, Ard Biesheuvel wrote: > On Mon, 18 Feb 2019 at 05:12, Jordan Justen wrote: > > > > This needs an explanation why optimization needs to be disabled. I'm not sure this is required. The reason I added these patches is to hopefully prevent the compiler from removing the frame

Re: [edk2] [PATCH 06/10] OvmfPkg/Sec: Disable optimizations for TemporaryRamMigration

2019-02-17 Thread Ard Biesheuvel
On Mon, 18 Feb 2019 at 05:12, Jordan Justen wrote: > This needs an explanation why optimization needs to be disabled. > Contributed-under: TianoCore Contribution Agreement 1.1 > Signed-off-by: Jordan Justen > Cc: Laszlo Ersek > Cc: Ard Biesheuvel > Cc: Anthony Perard > Cc: Julien Grall > --

[edk2] [PATCH 06/10] OvmfPkg/Sec: Disable optimizations for TemporaryRamMigration

2019-02-17 Thread Jordan Justen
Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Jordan Justen Cc: Laszlo Ersek Cc: Ard Biesheuvel Cc: Anthony Perard Cc: Julien Grall --- OvmfPkg/Sec/SecMain.c | 12 1 file changed, 12 insertions(+) diff --git a/OvmfPkg/Sec/SecMain.c b/OvmfPkg/Sec/SecMain.