Thanks Siyuan, I will rename the PCD name for better understanding.
> -----Original Message----- > From: Fu, Siyuan > Sent: Wednesday, September 19, 2018 8:38 AM > To: Wu, Jiaxin <jiaxin...@intel.com>; edk2-devel@lists.01.org > Cc: Ye, Ting <ting...@intel.com>; Shao, Ming <ming.s...@intel.com> > Subject: RE: [Patch 4/5] MdeModulePkg/MdeModulePkg.dec: Define one > PCD for PXE to specify MTFTP windowsize. > > Hi, Jiaxin > > I think we should also update the comments for PcdTftpWindowSize usage. > And could we change the name to PcdPxeTftpWindowSize since it only > impacts the PXE tftp download? > > BestRegards > Fu Siyuan > > > -----Original Message----- > > From: Wu, Jiaxin > > Sent: Monday, September 17, 2018 1:44 PM > > To: edk2-devel@lists.01.org > > Cc: Ye, Ting <ting...@intel.com>; Fu, Siyuan <siyuan...@intel.com>; Shao, > > Ming <ming.s...@intel.com>; Wu, Jiaxin <jiaxin...@intel.com> > > Subject: [Patch 4/5] MdeModulePkg/MdeModulePkg.dec: Define one PCD > for PXE > > to specify MTFTP windowsize. > > > > Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=886 > > > > This patch is to define one new PCD for PXE driver to specify MTFTP > > windowsize so as > > to improve the PXE download performance. The default value is set to 4. > > > > Cc: Ye Ting <ting...@intel.com> > > Cc: Fu Siyuan <siyuan...@intel.com> > > Cc: Shao Ming <ming.s...@intel.com> > > Contributed-under: TianoCore Contribution Agreement 1.1 > > Signed-off-by: Wu Jiaxin <jiaxin...@intel.com> > > --- > > MdeModulePkg/MdeModulePkg.dec | 1 + > > 1 file changed, 1 insertion(+) > > > > diff --git a/MdeModulePkg/MdeModulePkg.dec > b/MdeModulePkg/MdeModulePkg.dec > > index 74a699cbb7..bfc63e5fcb 100644 > > --- a/MdeModulePkg/MdeModulePkg.dec > > +++ b/MdeModulePkg/MdeModulePkg.dec > > @@ -1203,10 +1203,11 @@ > > ## This setting can override the default TFTP block size. A value of 0 > > computes > > # the default from MTU information. A non-zero value will be used as > > block size > > # in bytes. > > # @Prompt TFTP block size. > > > gEfiMdeModulePkgTokenSpaceGuid.PcdTftpBlockSize|0x0|UINT64|0x30001 > 026 > > + > gEfiMdeModulePkgTokenSpaceGuid.PcdTftpWindowSize|0x4|UINT64|0x30 > 00102A > > > > ## Maximum address that the DXE Core will allocate the > > EFI_SYSTEM_TABLE_POINTER > > # structure. The default value for this PCD is 0, which means that the > > DXE Core > > # will allocate the buffer from the EFI_SYSTEM_TABLE_POINTER structure > > on a 4MB > > # boundary as close to the top of memory as feasible. If this PCD is > > set to a > > -- > > 2.17.1.windows.2 _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel