Reviewed-by: Jiaxin Wu <[email protected]>
> -----Original Message----- > From: Ni, Ruiyu > Sent: Wednesday, July 26, 2017 5:27 PM > To: [email protected] > Cc: Wu, Jiaxin <[email protected]> > Subject: [PATCH] Nt32Pkg: Always use 0x2000 for maxim variable size > > Currently the PcdMaxVariableSize is controlled by the below condition > check: > > !if $(SECURE_BOOT_ENABLE) == TRUE || $(TLS_ENABLE) == TRUE > gEfiMdeModulePkgTokenSpaceGuid.PcdMaxVariableSize|0x2000 > !endif > > But for other case, PcdMaxVariableSize is also required with the value > of 0x2000 (e,g. NetworkPkg/IScsiDxe), so remove the condition of > PcdMaxVariableSize for NT32 platform. > > Contributed-under: TianoCore Contribution Agreement 1.0 > Signed-off-by: Ruiyu Ni <[email protected]> > Cc: Jiaxin Wu <[email protected]> > --- > Nt32Pkg/Nt32Pkg.dsc | 2 -- > 1 file changed, 2 deletions(-) > > diff --git a/Nt32Pkg/Nt32Pkg.dsc b/Nt32Pkg/Nt32Pkg.dsc > index fa3446be06..30d67309cd 100644 > --- a/Nt32Pkg/Nt32Pkg.dsc > +++ b/Nt32Pkg/Nt32Pkg.dsc > @@ -272,9 +272,7 @@ [PcdsFixedAtBuild] > gEfiNt32PkgTokenSpaceGuid.PcdWinNtFirmwareBlockSize|0x10000 > gEfiMdePkgTokenSpaceGuid.PcdReportStatusCodePropertyMask|0x0f > > gEfiMdeModulePkgTokenSpaceGuid.PcdResetOnMemoryTypeInformationC > hange|FALSE > -!if $(SECURE_BOOT_ENABLE) == TRUE || $(TLS_ENABLE) == TRUE > gEfiMdeModulePkgTokenSpaceGuid.PcdMaxVariableSize|0x2000 > -!endif > > !if $(ALLOW_HTTP_CONNECTIONS) == TRUE > gEfiNetworkPkgTokenSpaceGuid.PcdAllowHttpConnections|TRUE > -- > 2.12.2.windows.2 _______________________________________________ edk2-devel mailing list [email protected] https://lists.01.org/mailman/listinfo/edk2-devel

