I suggest to directly change [FixedPcd] to [Pcd] section. All Pcds can support FixedAtBuild and PatchableInModule.
With this change, Reviewed-by: Liming Gao <gaolim...@byosoft.com.cn> Thanks Liming > -----邮件原件----- > 发件人: bounce+27952+71734+4905953+8761...@groups.io > <bounce+27952+71734+4905953+8761...@groups.io> 代表 Sughosh Ganu > 发送时间: 2021年2月17日 19:27 > 收件人: devel@edk2.groups.io > 抄送: Sami Mujawar <sami.muja...@arm.com>; Ilias Apalodimas > <ilias.apalodi...@linaro.org>; Ard Biesheuvel <ardb+tianoc...@kernel.org> > 主题: [edk2-devel] [PATCH v4 13/14] MdeModulePkg/VariableStandaloneMm: > Set PcdFlashNvStorageVariableBase to Pcd > > From: Ilias Apalodimas <ilias.apalodi...@linaro.org> > > Instead of running StMM in SPM, OP-TEE creates a new secure partition, > which emulates SPM and isolates StMM from the rest of the Trusted > Applications (TAs). We can then compile StMM as an FD image and run it > in OP-TEE. With the addition of a new RPMB driver, we can leverage OP-TEE > and store variables to an RPMB device. > > Since EDK2 upper layers expect byte addressable code, for the RPMB to > work, we need to allocate memory and sync it with the hardware on > read/writes. Since DynamicPCDs are not supported in that context we > can only use PatchablePCDs. So let's switch them to Pcd instead of > FixedPcd and accomodate the new driver. > > Signed-off-by: Ilias Apalodimas <ilias.apalodi...@linaro.org> > Reviewed-by: Sami Mujawar <sami.muja...@arm.com> > --- > > Changes since V3: None > > MdeModulePkg/Universal/Variable/RuntimeDxe/VariableStandaloneMm.inf > | 6 ++++-- > 1 file changed, 4 insertions(+), 2 deletions(-) > > diff --git > a/MdeModulePkg/Universal/Variable/RuntimeDxe/VariableStandaloneMm.in > f > b/MdeModulePkg/Universal/Variable/RuntimeDxe/VariableStandaloneMm.in > f > index fada0bf3c5..2a25fbdada 100644 > --- > a/MdeModulePkg/Universal/Variable/RuntimeDxe/VariableStandaloneMm.in > f > +++ > b/MdeModulePkg/Universal/Variable/RuntimeDxe/VariableStandaloneMm.in > f > @@ -119,10 +119,12 @@ > ## SOMETIMES_PRODUCES ## Variable:L"VarErrorFlag" > gEdkiiVarErrorFlagGuid > > -[FixedPcd] > - gEfiMdeModulePkgTokenSpaceGuid.PcdFlashNvStorageVariableSize > ## CONSUMES > +[Pcd] > gEfiMdeModulePkgTokenSpaceGuid.PcdFlashNvStorageVariableBase > ## SOMETIMES_CONSUMES > gEfiMdeModulePkgTokenSpaceGuid.PcdFlashNvStorageVariableBase64 > ## CONSUMES > + gEfiMdeModulePkgTokenSpaceGuid.PcdFlashNvStorageVariableSize > ## CONSUMES > + > +[FixedPcd] > gEfiMdeModulePkgTokenSpaceGuid.PcdMaxVariableSize > ## CONSUMES > gEfiMdeModulePkgTokenSpaceGuid.PcdMaxAuthVariableSize > ## CONSUMES > gEfiMdeModulePkgTokenSpaceGuid.PcdMaxVolatileVariableSize > ## CONSUMES > -- > 2.17.1 > > > > > -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#71762): https://edk2.groups.io/g/devel/message/71762 Mute This Topic: https://groups.io/mt/80721762/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-