Sorry. Please ignore this 1 patch. It is an old one that
I mistakenly re-sent.

-Jordan

On Mon, 2013-07-15 at 10:37 -0700, Jordan Justen wrote:
> r14252 causes OVMF to crash if SECURE_BOOT_ENABLE is set,
> because PcdMaxVariableSize is set to a larger value than
> required. In other platforms, 0x2000 seems to be sufficient.
> 
> Reported-by: Gary Ching-Pang Lin <[email protected]>
> Signed-off-by: Jordan Justen <[email protected]>
> Cc: Lee G Rosenbaum <[email protected]>
> Cc: Laszlo Ersek <[email protected]>
> ---
>  OvmfPkg/OvmfPkgIa32.dsc    |    2 +-
>  OvmfPkg/OvmfPkgIa32X64.dsc |    2 +-
>  OvmfPkg/OvmfPkgX64.dsc     |    2 +-
>  3 files changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/OvmfPkg/OvmfPkgIa32.dsc b/OvmfPkg/OvmfPkgIa32.dsc
> index 0e6d187..cbd99c3 100644
> --- a/OvmfPkg/OvmfPkgIa32.dsc
> +++ b/OvmfPkg/OvmfPkgIa32.dsc
> @@ -272,7 +272,7 @@
>    gEfiMdeModulePkgTokenSpaceGuid.PcdPeiCoreMaxFvSupported|6
>    gEfiMdeModulePkgTokenSpaceGuid.PcdPeiCoreMaxPeimPerFv|32
>  !if $(SECURE_BOOT_ENABLE) == TRUE
> -  gEfiMdeModulePkgTokenSpaceGuid.PcdMaxVariableSize|0x10000
> +  gEfiMdeModulePkgTokenSpaceGuid.PcdMaxVariableSize|0x2000
>  !else
>    gEfiMdeModulePkgTokenSpaceGuid.PcdMaxVariableSize|0x400
>  !endif
> diff --git a/OvmfPkg/OvmfPkgIa32X64.dsc b/OvmfPkg/OvmfPkgIa32X64.dsc
> index 48da532..75552bb 100644
> --- a/OvmfPkg/OvmfPkgIa32X64.dsc
> +++ b/OvmfPkg/OvmfPkgIa32X64.dsc
> @@ -277,7 +277,7 @@
>    gEfiMdeModulePkgTokenSpaceGuid.PcdPeiCoreMaxFvSupported|6
>    gEfiMdeModulePkgTokenSpaceGuid.PcdPeiCoreMaxPeimPerFv|32
>  !if $(SECURE_BOOT_ENABLE) == TRUE
> -  gEfiMdeModulePkgTokenSpaceGuid.PcdMaxVariableSize|0x10000
> +  gEfiMdeModulePkgTokenSpaceGuid.PcdMaxVariableSize|0x2000
>  !else
>    gEfiMdeModulePkgTokenSpaceGuid.PcdMaxVariableSize|0x400
>  !endif
> diff --git a/OvmfPkg/OvmfPkgX64.dsc b/OvmfPkg/OvmfPkgX64.dsc
> index 7f138ce..75d167c 100644
> --- a/OvmfPkg/OvmfPkgX64.dsc
> +++ b/OvmfPkg/OvmfPkgX64.dsc
> @@ -277,7 +277,7 @@
>    gEfiMdeModulePkgTokenSpaceGuid.PcdPeiCoreMaxFvSupported|6
>    gEfiMdeModulePkgTokenSpaceGuid.PcdPeiCoreMaxPeimPerFv|32
>  !if $(SECURE_BOOT_ENABLE) == TRUE
> -  gEfiMdeModulePkgTokenSpaceGuid.PcdMaxVariableSize|0x10000
> +  gEfiMdeModulePkgTokenSpaceGuid.PcdMaxVariableSize|0x2000
>  !else
>    gEfiMdeModulePkgTokenSpaceGuid.PcdMaxVariableSize|0x400
>  !endif



------------------------------------------------------------------------------
See everything from the browser to the database with AppDynamics
Get end-to-end visibility with application monitoring from AppDynamics
Isolate bottlenecks and diagnose root cause in seconds.
Start your free trial of AppDynamics Pro today!
http://pubads.g.doubleclick.net/gampad/clk?id=48808831&iu=/4140/ostg.clktrk
_______________________________________________
edk2-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/edk2-devel

Reply via email to