Re: [PATCH 0/4] OvmfPkg: measure temp stack usage, restore temp RAM to 64KB

[Jordan Justen <jordan.l.justen@intel.com>]

On 2017-11-10 07:49:04, Laszlo Ersek wrote:
> The first three patches enable the PEI_CORE to report OVMF's temp
> SEC/PEI stack and heap usage.
> 
>   - This depends on the new fixed PCD "PcdInitValueInTempStack",
>     recently added for
>     <https://bugzilla.tianocore.org/show_bug.cgi?id=740>
>     ("INIT_CAR_VALUE should be defined in a central location").
> 
>   - Ard recently implemented the same in ArmPlatformPkg, for
>     <https://bugzilla.tianocore.org/show_bug.cgi?id=748> ("measure temp
>     SEC/PEI stack usage").
> 
> The last (fourth) patch adapts OVMF to the larger MtrrLib stack demand
> that originates from commit 2bbd7e2fbd4b ("UefiCpuPkg/MtrrLib: Update
> algorithm to calculate optimal settings", 2017-09-27). OVMF's temp RAM
> size is restored to the historical / original 64KB.
> 
> This work is tracked in
> <https://bugzilla.tianocore.org/show_bug.cgi?id=747> ("measure temp
> SEC/PEI stack usage; grow temp SEC/PEI RAM"), with links to related
> mailing list discussions.
> 
> Repo:   https://github.com/lersek/edk2.git
> Branch: temp_ram_tweaks
> 
> Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>
> Cc: Jordan Justen <jordan.l.justen@intel.com>
> Cc: Ruiyu Ni <ruiyu.ni@intel.com>
> 
> Thanks
> Laszlo
> 
> Laszlo Ersek (4):
>   OvmfPkg/Sec/Ia32: free up EAX for other uses while setting up the
>     stack
>   OvmfPkg/Sec/Ia32: seed the temporary RAM with PcdInitValueInTempStack
>   OvmfPkg/Sec/X64: seed the temporary RAM with PcdInitValueInTempStack

I'd like to try a different option for these 3. Can you hold off a bit
before pushing this series?

-Jordan

>   OvmfPkg: restore temporary SEC/PEI RAM size to 64KB
> 
>  OvmfPkg/OvmfPkgIa32.fdf        |  2 +-
>  OvmfPkg/OvmfPkgIa32X64.fdf     |  2 +-
>  OvmfPkg/OvmfPkgX64.fdf         |  2 +-
>  OvmfPkg/Sec/SecMain.inf        |  1 +
>  OvmfPkg/Sec/Ia32/SecEntry.nasm | 19 ++++++++++++++++---
>  OvmfPkg/Sec/X64/SecEntry.nasm  | 15 +++++++++++++++
>  6 files changed, 35 insertions(+), 6 deletions(-)
> 
> -- 
> 2.14.1.3.gb7cf6e02401b
>