Re: [ofa-general] Re: [ewg] [UPDATE][PATCH] IPoIB-UD 4K MTU patch against 2.6.24 ofed-1.3-git tree

2008-02-05 Thread Shirley Ma





Hello Tziporet,

  The problem was because of the last check  in of small UDP
performance patch. It changed the receiving path completely. And I only got
less than one day to merge/test the patch with that patch on both intel and
PPC platform. The patch was in good/stable shape before this patch. It has
passed stress test for both intel and PPC platform. I have tested the whole
night of the new patch yesterday night. It works well and passes the stress
test without any problem.

  Regarding Eli's comments, I have sent out. I am sorry for the minor
mistake because of the rushing, but I don't see any risk from my test
results. Please reconsider this patch to be in OFED-1.3.

thanks
Shirley




   
 Tziporet  
 Koren 
 tziporet  To
 @dev.mell [EMAIL PROTECTED] 
 anox.co.i  cc
 lEli Cohen [EMAIL PROTECTED], 
 Sent by:  ewg@lists.openfabrics.org, OpenFabrics  
 general-b General [EMAIL PROTECTED] 
 [EMAIL PROTECTED] 
Subject
 sts.openf [ofa-general] Re: [ewg] [UPDATE][PATCH] 
 abrics.or IPoIB-UD 4K MTU patch against 2.6.24
 g ofed-1.3-git tree   
   
   
 02/05/08  
 08:19 AM  
   
   




Shirley Ma wrote:
 I found one one line was out side for loop when merging this patch
 with current git-tree. This caused UD_POST_RCV_COUNT = 16 wrong. I have
 fixed it. This is the updated patch.

 Thanks
 Shirley




Hi Shirley,

Its seems to me that 4K MTU patch is not cooked enough for RC4.
I appreciate your hard work to push it but so many changes, possible
leaks and not enough time for review and testing means too high risk for
now

Tziporet

___
general mailing list
[EMAIL PROTECTED]
http://lists.openfabrics.org/cgi-bin/mailman/listinfo/general

To unsubscribe, please visit
http://openib.org/mailman/listinfo/openib-general
inline: graycol.gifinline: pic23340.gifinline: ecblank.gif___
ewg mailing list
ewg@lists.openfabrics.org
http://lists.openfabrics.org/cgi-bin/mailman/listinfo/ewg

Re: [ofa-general] Re: [ewg] [UPDATE][PATCH] IPoIB-UD 4K MTU patch against 2.6.24 ofed-1.3-git tree

2008-02-05 Thread Shirley Ma
Hello Tziporet,

On Tue, 2008-02-05 at 18:56 +0200, Tziporet Koren wrote:
 Shirley Ma wrote:
 
  Hello Tziporet,
 
  The problem was because of the last check in of small UDP performance 
  patch. It changed the receiving path completely. And I only got less 
  than one day to merge/test the patch with that patch on both intel and 
  PPC platform. The patch was in good/stable shape before this patch. It 
  has passed stress test for both intel and PPC platform. I have tested 
  the whole night of the new patch yesterday night. It works well and 
  passes the stress test without any problem.
 
 Which OS have you tested?

2.6.24 kernel, and I am going to test SLES10SP2 kernel. It has passed
stress test the whole night for 2K MTU test suites.

  Regarding Eli's comments, I have sent out. I am sorry for the minor 
  mistake because of the rushing, but I don't see any risk from my test 
  results. Please reconsider this patch to be in OFED-1.3.
 
 OK - we will do this - we will run one set of our regression with your 
 patch now, and also check that it pass compilation on all kernels.
 If both will be OK we will take it.
 
 I cross fingers for you :-)
 
 ziporet

Appreciate you, Vlad and Eli's help here! There is one line change
needed for backporting ++priv-stats and ++dev-stats. I didn't create
the backport patch for this.

thanks
Shirley

___
ewg mailing list
ewg@lists.openfabrics.org
http://lists.openfabrics.org/cgi-bin/mailman/listinfo/ewg


Re: [ofa-general] Re: [ewg] [UPDATE][PATCH] IPoIB-UD 4K MTU patch against 2.6.24 ofed-1.3-git tree

2008-02-05 Thread Shirley Ma




Tziporet Koren [EMAIL PROTECTED] wrote on 02/05/2008 12:07:28
PM:

 Please test on RHREL 5 too
 What are your stress tests?

Ok. The stress test is similar to netperf/netserver. But it's
bi-directional multiple streams. I have stressed the stream to 150, duplex
running overnight.

 Please send this backport patch and specify to which kernels its needed

 Tziporet

Ok. It might be out tonight.

Thanks
Shirley___
ewg mailing list
ewg@lists.openfabrics.org
http://lists.openfabrics.org/cgi-bin/mailman/listinfo/ewg