[edk2] [PATCH v2 0/4] UefiCpuPkg, OvmfPkg: add reset vector docs, tweak temp stack / RAM

Laszlo Ersek posted 4 patches 6 years, 5 months ago
Failed in applying to current master (apply log)
OvmfPkg/OvmfPkgIa32.fdf                             |  2 +-
OvmfPkg/OvmfPkgIa32X64.fdf                          |  2 +-
OvmfPkg/OvmfPkgX64.fdf                              |  2 +-
OvmfPkg/Sec/Ia32/SecEntry.nasm                      | 18 +++++++++++++++++
OvmfPkg/Sec/SecMain.inf                             |  1 +
OvmfPkg/Sec/X64/SecEntry.nasm                       | 21 ++++++++++++++++++++
UefiCpuPkg/ResetVector/Vtf0/Ia16/Real16ToFlat32.asm |  6 ++++++
UefiCpuPkg/ResetVector/Vtf0/Main.asm                |  5 +++++
8 files changed, 54 insertions(+), 3 deletions(-)
[edk2] [PATCH v2 0/4] UefiCpuPkg, OvmfPkg: add reset vector docs, tweak temp stack / RAM
Posted by Laszlo Ersek 6 years, 5 months ago
Repo:   https://github.com/lersek/edk2.git
Branch: temp_ram_tweaks_v2

This is a series for

  https://bugzilla.tianocore.org/show_bug.cgi?id=747

specifically an update on the v1 series at

  https://lists.01.org/pipermail/edk2-devel/2017-November/017239.html

Each patch has its notes section on the changes relative to v1.

I re-ran the unit tests described in some of the patches; in addition to
the functional tests and my usual regression tests.

Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>
Cc: Eric Dong <eric.dong@intel.com>
Cc: Jordan Justen <jordan.l.justen@intel.com>

Thanks
Laszlo

Laszlo Ersek (4):
  UefiCpuPkg/ResetVector/Vtf0: document segment register setup
  OvmfPkg/Sec/Ia32: seed the temporary RAM with PcdInitValueInTempStack
  OvmfPkg/Sec/X64: seed the temporary RAM with PcdInitValueInTempStack
  OvmfPkg: restore temporary SEC/PEI RAM size to 64KB

 OvmfPkg/OvmfPkgIa32.fdf                             |  2 +-
 OvmfPkg/OvmfPkgIa32X64.fdf                          |  2 +-
 OvmfPkg/OvmfPkgX64.fdf                              |  2 +-
 OvmfPkg/Sec/Ia32/SecEntry.nasm                      | 18 +++++++++++++++++
 OvmfPkg/Sec/SecMain.inf                             |  1 +
 OvmfPkg/Sec/X64/SecEntry.nasm                       | 21 ++++++++++++++++++++
 UefiCpuPkg/ResetVector/Vtf0/Ia16/Real16ToFlat32.asm |  6 ++++++
 UefiCpuPkg/ResetVector/Vtf0/Main.asm                |  5 +++++
 8 files changed, 54 insertions(+), 3 deletions(-)

-- 
2.14.1.3.gb7cf6e02401b

_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel
Re: [edk2] [PATCH v2 0/4] UefiCpuPkg, OvmfPkg: add reset vector docs, tweak temp stack / RAM
Posted by Jordan Justen 6 years, 5 months ago
Series Reviewed-by: Jordan Justen <jordan.l.justen@intel.com>

On 2017-11-15 13:56:59, Laszlo Ersek wrote:
> Repo:   https://github.com/lersek/edk2.git
> Branch: temp_ram_tweaks_v2
> 
> This is a series for
> 
>   https://bugzilla.tianocore.org/show_bug.cgi?id=747
> 
> specifically an update on the v1 series at
> 
>   https://lists.01.org/pipermail/edk2-devel/2017-November/017239.html
> 
> Each patch has its notes section on the changes relative to v1.
> 
> I re-ran the unit tests described in some of the patches; in addition to
> the functional tests and my usual regression tests.
> 
> Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>
> Cc: Eric Dong <eric.dong@intel.com>
> Cc: Jordan Justen <jordan.l.justen@intel.com>
> 
> Thanks
> Laszlo
> 
> Laszlo Ersek (4):
>   UefiCpuPkg/ResetVector/Vtf0: document segment register setup
>   OvmfPkg/Sec/Ia32: seed the temporary RAM with PcdInitValueInTempStack
>   OvmfPkg/Sec/X64: seed the temporary RAM with PcdInitValueInTempStack
>   OvmfPkg: restore temporary SEC/PEI RAM size to 64KB
> 
>  OvmfPkg/OvmfPkgIa32.fdf                             |  2 +-
>  OvmfPkg/OvmfPkgIa32X64.fdf                          |  2 +-
>  OvmfPkg/OvmfPkgX64.fdf                              |  2 +-
>  OvmfPkg/Sec/Ia32/SecEntry.nasm                      | 18 +++++++++++++++++
>  OvmfPkg/Sec/SecMain.inf                             |  1 +
>  OvmfPkg/Sec/X64/SecEntry.nasm                       | 21 ++++++++++++++++++++
>  UefiCpuPkg/ResetVector/Vtf0/Ia16/Real16ToFlat32.asm |  6 ++++++
>  UefiCpuPkg/ResetVector/Vtf0/Main.asm                |  5 +++++
>  8 files changed, 54 insertions(+), 3 deletions(-)
> 
> -- 
> 2.14.1.3.gb7cf6e02401b
> 
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel
Re: [edk2] [PATCH v2 0/4] UefiCpuPkg, OvmfPkg: add reset vector docs, tweak temp stack / RAM
Posted by Laszlo Ersek 6 years, 5 months ago
On 11/15/17 22:56, Laszlo Ersek wrote:
> Repo:   https://github.com/lersek/edk2.git
> Branch: temp_ram_tweaks_v2
> 
> This is a series for
> 
>   https://bugzilla.tianocore.org/show_bug.cgi?id=747
> 
> specifically an update on the v1 series at
> 
>   https://lists.01.org/pipermail/edk2-devel/2017-November/017239.html
> 
> Each patch has its notes section on the changes relative to v1.
> 
> I re-ran the unit tests described in some of the patches; in addition to
> the functional tests and my usual regression tests.
> 
> Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>
> Cc: Eric Dong <eric.dong@intel.com>
> Cc: Jordan Justen <jordan.l.justen@intel.com>
> 
> Thanks
> Laszlo
> 
> Laszlo Ersek (4):
>   UefiCpuPkg/ResetVector/Vtf0: document segment register setup
>   OvmfPkg/Sec/Ia32: seed the temporary RAM with PcdInitValueInTempStack
>   OvmfPkg/Sec/X64: seed the temporary RAM with PcdInitValueInTempStack
>   OvmfPkg: restore temporary SEC/PEI RAM size to 64KB
> 
>  OvmfPkg/OvmfPkgIa32.fdf                             |  2 +-
>  OvmfPkg/OvmfPkgIa32X64.fdf                          |  2 +-
>  OvmfPkg/OvmfPkgX64.fdf                              |  2 +-
>  OvmfPkg/Sec/Ia32/SecEntry.nasm                      | 18 +++++++++++++++++
>  OvmfPkg/Sec/SecMain.inf                             |  1 +
>  OvmfPkg/Sec/X64/SecEntry.nasm                       | 21 ++++++++++++++++++++
>  UefiCpuPkg/ResetVector/Vtf0/Ia16/Real16ToFlat32.asm |  6 ++++++
>  UefiCpuPkg/ResetVector/Vtf0/Main.asm                |  5 +++++
>  8 files changed, 54 insertions(+), 3 deletions(-)
> 

Pushed as commit range 1195b8578764..d41fd8e839a3.

(I didn't get a maintainer review from Eric for patch #1, but it adds
only comments (no regressions possible) and Ard and Jordan reviewed that
too. My queue is overflowing. :/)

Thanks!
Laszlo
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel
Re: [edk2] [PATCH v2 0/4] UefiCpuPkg, OvmfPkg: add reset vector docs, tweak temp stack / RAM
Posted by Dong, Eric 6 years, 5 months ago
Hi Laszlo,

Reviewed-by: Eric Dong <eric.dong@intel.com>

I'm ok with this change, Sorry for later response.

Thanks,
Eric
> -----Original Message-----
> From: Laszlo Ersek [mailto:lersek@redhat.com]
> Sent: Saturday, November 18, 2017 1:15 AM
> To: edk2-devel-01 <edk2-devel@lists.01.org>
> Cc: Justen, Jordan L <jordan.l.justen@intel.com>; Dong, Eric
> <eric.dong@intel.com>; Ard Biesheuvel <ard.biesheuvel@linaro.org>
> Subject: Re: [edk2] [PATCH v2 0/4] UefiCpuPkg, OvmfPkg: add reset vector
> docs, tweak temp stack / RAM
> 
> On 11/15/17 22:56, Laszlo Ersek wrote:
> > Repo:   https://github.com/lersek/edk2.git
> > Branch: temp_ram_tweaks_v2
> >
> > This is a series for
> >
> >   https://bugzilla.tianocore.org/show_bug.cgi?id=747
> >
> > specifically an update on the v1 series at
> >
> >   https://lists.01.org/pipermail/edk2-devel/2017-November/017239.html
> >
> > Each patch has its notes section on the changes relative to v1.
> >
> > I re-ran the unit tests described in some of the patches; in addition
> > to the functional tests and my usual regression tests.
> >
> > Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>
> > Cc: Eric Dong <eric.dong@intel.com>
> > Cc: Jordan Justen <jordan.l.justen@intel.com>
> >
> > Thanks
> > Laszlo
> >
> > Laszlo Ersek (4):
> >   UefiCpuPkg/ResetVector/Vtf0: document segment register setup
> >   OvmfPkg/Sec/Ia32: seed the temporary RAM with
> PcdInitValueInTempStack
> >   OvmfPkg/Sec/X64: seed the temporary RAM with
> PcdInitValueInTempStack
> >   OvmfPkg: restore temporary SEC/PEI RAM size to 64KB
> >
> >  OvmfPkg/OvmfPkgIa32.fdf                             |  2 +-
> >  OvmfPkg/OvmfPkgIa32X64.fdf                          |  2 +-
> >  OvmfPkg/OvmfPkgX64.fdf                              |  2 +-
> >  OvmfPkg/Sec/Ia32/SecEntry.nasm                      | 18 +++++++++++++++++
> >  OvmfPkg/Sec/SecMain.inf                             |  1 +
> >  OvmfPkg/Sec/X64/SecEntry.nasm                       | 21
> ++++++++++++++++++++
> >  UefiCpuPkg/ResetVector/Vtf0/Ia16/Real16ToFlat32.asm |  6 ++++++
> >  UefiCpuPkg/ResetVector/Vtf0/Main.asm                |  5 +++++
> >  8 files changed, 54 insertions(+), 3 deletions(-)
> >
> 
> Pushed as commit range 1195b8578764..d41fd8e839a3.
> 
> (I didn't get a maintainer review from Eric for patch #1, but it adds only
> comments (no regressions possible) and Ard and Jordan reviewed that too.
> My queue is overflowing. :/)
> 
> Thanks!
> Laszlo
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel