On 05/09/16 23:46, Linn Crosetto wrote: > To support UEFI Secure Boot and the Linux persistent store with UEFI > variables, set PcdMaxVariableSize to 0x2000 bytes as is done in OvmfPkg. > For reference, the related ovmf commits: 8cee3de7 2d441ca9 > > Contributed-under: TianoCore Contribution Agreement 1.0 > Signed-off-by: Linn Crosetto <[email protected]> > --- > ArmVirtPkg/ArmVirtQemu.dsc | 1 + > ArmVirtPkg/ArmVirtQemuKernel.dsc | 1 + > 2 files changed, 2 insertions(+) > > diff --git a/ArmVirtPkg/ArmVirtQemu.dsc b/ArmVirtPkg/ArmVirtQemu.dsc > index 1d48cd0..e72f4be 100644 > --- a/ArmVirtPkg/ArmVirtQemu.dsc > +++ b/ArmVirtPkg/ArmVirtQemu.dsc > @@ -111,6 +111,7 @@ > > gArmPlatformTokenSpaceGuid.PcdCPUCoresStackBase|0x4007c000 > gArmPlatformTokenSpaceGuid.PcdCPUCorePrimaryStackSize|0x4000 > + gEfiMdeModulePkgTokenSpaceGuid.PcdMaxVariableSize|0x2000 > > # Size of the region used by UEFI in permanent memory (Reserved 64MB) > gArmPlatformTokenSpaceGuid.PcdSystemMemoryUefiRegionSize|0x04000000 > diff --git a/ArmVirtPkg/ArmVirtQemuKernel.dsc > b/ArmVirtPkg/ArmVirtQemuKernel.dsc > index d4134f1..b1d5663 100644 > --- a/ArmVirtPkg/ArmVirtQemuKernel.dsc > +++ b/ArmVirtPkg/ArmVirtQemuKernel.dsc > @@ -100,6 +100,7 @@ > > gArmPlatformTokenSpaceGuid.PcdCPUCoresStackBase|0x4007c000 > gArmPlatformTokenSpaceGuid.PcdCPUCorePrimaryStackSize|0x4000 > + gEfiMdeModulePkgTokenSpaceGuid.PcdMaxVariableSize|0x2000 > > # Size of the region used by UEFI in permanent memory (Reserved 64MB) > gArmPlatformTokenSpaceGuid.PcdSystemMemoryUefiRegionSize|0x04000000 >
(1) in the future, please CC maintainers (from Maintainers.txt) directly. (2) If we are using OvmfPkg as an example, then please see also commit f5404a3eba1d. I don't insist on following OvmfPkg's example, but if we do (and I think it's not a bad idea at all), we should not ignore f5404a3eba1d. (3) Please search the article at <https://github.com/tianocore/tianocore.github.io/wiki/Laszlo's-unkempt-git-guide-for-edk2-contributors-and-maintainers> for the string "xfuncname". When patches are formatted without that setting, it is hard to see what section of the DSC file is being modified. Please see also the related attribute settings under <https://github.com/tianocore/tianocore.github.io/wiki/Laszlo's-unkempt-git-guide-for-edk2-contributors-and-maintainers#contrib-09>. Thanks Laszlo _______________________________________________ edk2-devel mailing list [email protected] https://lists.01.org/mailman/listinfo/edk2-devel

