Hi Leif, 2021年11月27日(土) 3:19 Leif Lindholm <l...@nuviainc.com>: > > On Fri, Nov 05, 2021 at 18:24:01 +0900, Masami Hiramatsu wrote: > > Expand NvStorage Variable size and FTW spare/working size > > for the DeveloperBox platform. > > > > Since the size of the NvStorage VariableSize is not enough > > large, FWTS uefirttime test, which updates the NV > > variables in runtime, failes. This expands the size to fix > > this issue. > > Does this change erase all existing variables?
Ah, indeed. It may need to erase all variables. > > If so, I think it is worth introducing this as a non-default build > option, in order to not wreck existing installations on a firmware > update. > > I think it would also be worth considering whether to update > PcdLowestSupportedFirmwareVersion. PcdFirmwareRevision > should definitely be updated. I'm not sure about this point. You meant we should have 2 different revisions like a branch? - Branch A(current version): keep the variable area size the same. - Branch B(new version): expand the variable area. And a build option will change the branch by updating the PcdFirmwareRevision? Also PcdLowestSupportedFirmwareVersion you meant is in the capsule file? Thank you, > > / > Leif > > > Signed-off-by: Masami Hiramatsu <masami.hirama...@linaro.org> > > Reported-by: Kazuhiko Sakamoto <sakamoto.kazuh...@socionext.com> > > --- > > .../Socionext/DeveloperBox/DeveloperBox.dsc.inc | 10 +++++----- > > 1 file changed, 5 insertions(+), 5 deletions(-) > > > > diff --git a/Platform/Socionext/DeveloperBox/DeveloperBox.dsc.inc > > b/Platform/Socionext/DeveloperBox/DeveloperBox.dsc.inc > > index 0a364bc457..3baf97ecc0 100644 > > --- a/Platform/Socionext/DeveloperBox/DeveloperBox.dsc.inc > > +++ b/Platform/Socionext/DeveloperBox/DeveloperBox.dsc.inc > > @@ -280,11 +280,11 @@ > > gFip006DxeTokenSpaceGuid.PcdFip006DxeMemBaseAddress|0x08000000 > > > > gEfiMdeModulePkgTokenSpaceGuid.PcdFlashNvStorageVariableBase|0x08400000 > > - gEfiMdeModulePkgTokenSpaceGuid.PcdFlashNvStorageVariableSize|0x00010000 > > - gEfiMdeModulePkgTokenSpaceGuid.PcdFlashNvStorageFtwWorkingBase|0x08410000 > > - gEfiMdeModulePkgTokenSpaceGuid.PcdFlashNvStorageFtwWorkingSize|0x00010000 > > - gEfiMdeModulePkgTokenSpaceGuid.PcdFlashNvStorageFtwSpareBase|0x08420000 > > - gEfiMdeModulePkgTokenSpaceGuid.PcdFlashNvStorageFtwSpareSize|0x00010000 > > + gEfiMdeModulePkgTokenSpaceGuid.PcdFlashNvStorageVariableSize|0x00080000 > > + gEfiMdeModulePkgTokenSpaceGuid.PcdFlashNvStorageFtwWorkingBase|0x08480000 > > + gEfiMdeModulePkgTokenSpaceGuid.PcdFlashNvStorageFtwWorkingSize|0x00080000 > > + gEfiMdeModulePkgTokenSpaceGuid.PcdFlashNvStorageFtwSpareBase|0x08500000 > > + gEfiMdeModulePkgTokenSpaceGuid.PcdFlashNvStorageFtwSpareSize|0x00080000 > > > > gEfiMdeModulePkgTokenSpaceGuid.PcdAcpiDefaultOemId|"SNI " > > > > gEfiMdeModulePkgTokenSpaceGuid.PcdAcpiDefaultOemTableId|0x52434155514e5953 > > # SYNQUACR > > -- Masami Hiramatsu -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#84114): https://edk2.groups.io/g/devel/message/84114 Mute This Topic: https://groups.io/mt/86836385/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
