Re: [edk2] [Patch 1/3] MdeModulePkg: Update the default size of MNP TX buffer pool.

2016-02-02 Thread Laszlo Ersek
s code in SNP Transmit function. --> patch is good as-is. Reviewed-by: Laszlo Ersek <ler...@redhat.com> Thanks! Laszlo > > Best Regards > Siyuan > > -Original Message----- > From: Laszlo Ersek [mailto:ler...@redhat.com] > Sent: Tuesday, February 2, 2016 12:13 AM

Re: [edk2] [Patch 1/3] MdeModulePkg: Update the default size of MNP TX buffer pool.

2016-02-02 Thread Fu, Siyuan
L <harry.l.hsi...@intel.com> Subject: Re: [edk2] [Patch 1/3] MdeModulePkg: Update the default size of MNP TX buffer pool. Hello Siyuan, On 02/02/16 02:03, Fu, Siyuan wrote: > Hi, Laszlo > > Yes it's to make it same as the MAX_XMIT_BUFFERS in PxeGetStatus(), > and I agree

Re: [edk2] [Patch 1/3] MdeModulePkg: Update the default size of MNP TX buffer pool.

2016-02-02 Thread Subramanian, Sriram (EG Servers Platform SW)
iyuan...@intel.com] Sent: Tuesday, February 02, 2016 1:56 PM To: Laszlo Ersek; Subramanian, Sriram (EG Servers Platform SW) Cc: edk2-de...@ml01.01.org; Ye, Ting; Wu, Jiaxin; El-Haj-Mahmoud, Samer; Li, Ruth; Hsiung, Harry L Subject: RE: [edk2] [Patch 1/3] MdeModulePkg: Update the default size of MNP

Re: [edk2] [Patch 1/3] MdeModulePkg: Update the default size of MNP TX buffer pool.

2016-02-02 Thread Fu, Siyuan
-Original Message- > From: Fu, Siyuan [mailto:siyuan...@intel.com] > Sent: Tuesday, February 02, 2016 1:56 PM > To: Laszlo Ersek; Subramanian, Sriram (EG Servers Platform SW) > Cc: edk2-de...@ml01.01.org; Ye, Ting; Wu, Jiaxin; El-Haj-Mahmoud, > Samer; Li, Ruth; Hsiung, Harry

Re: [edk2] [Patch 1/3] MdeModulePkg: Update the default size of MNP TX buffer pool.

2016-02-02 Thread Subramanian, Sriram (EG Servers Platform SW)
-Mahmoud, Samer; Li, Ruth; Hsiung, Harry L Subject: RE: [edk2] [Patch 1/3] MdeModulePkg: Update the default size of MNP TX buffer pool. Hi, Laszlo and Sriram I have sent out the v2 patch, only the commit message is updated, please help to review. Thanks Siyuan -Original Message- From

Re: [edk2] [Patch 1/3] MdeModulePkg: Update the default size of MNP TX buffer pool.

2016-02-01 Thread Laszlo Ersek
On 02/01/16 03:51, Fu Siyuan wrote: > This patch update the default MNP TX buffer increasement to 32, so the default > TX pool length is same as the maximum recycled buffer numbers in one UNDI > GetStatus command. Is that MAX_XMIT_BUFFERS in PxeGetStatus()? > Contributed-under: TianoCore

[edk2] [Patch 1/3] MdeModulePkg: Update the default size of MNP TX buffer pool.

2016-01-31 Thread Fu Siyuan
This patch update the default MNP TX buffer increasement to 32, so the default TX pool length is same as the maximum recycled buffer numbers in one UNDI GetStatus command. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Fu Siyuan CC: Ye Ting