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-02 Thread Fu, Siyuan
, Siyuan 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 TX buffer pool. On 02/02/16 11:40, Subramanian, Sriram (EG Servers Platform SW) wrote: > Hi Las

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

2016-02-02 Thread Laszlo Ersek
trees make it easier to understand. > Therefore I propose to drop patch > > MdeModulePkg: Update the default size of MNP TX buffer pool > > from the series. > > Sriram, do you agree with this analysis? > > > ** In summary: > > - "[Patch 1/3] MdeModul

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)
lto: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 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
PM To: Fu, Siyuan ; Subramanian Sriram 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 TX buffer pool. Hello Siyuan, On 02/02/16 02:03, Fu, Siyuan wrote: > Hi

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

2016-02-02 Thread Laszlo Ersek
one return status code in SNP Transmit function. --> patch is good as-is. Reviewed-by: Laszlo Ersek 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-01 Thread Fu, Siyuan
SNP if the transmitted buffer queue if full. Best Regards Siyuan -Original Message- From: Laszlo Ersek [mailto:ler...@redhat.com] Sent: Tuesday, February 2, 2016 12:13 AM To: Fu, Siyuan ; edk2-de...@ml01.01.org Cc: Ye, Ting ; Wu, Jiaxin Subject: Re: [edk2] [Patch 1/3] MdeModulePkg: Update t

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 Contribu

[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 CC: Wu Jiaxin --- MdeModulePk