Re: [ewg] status of ofed ipoib changes which are not upstream

2008-04-29 Thread Eli Cohen
On Mon, 2008-04-28 at 13:43 +0300, Or Gerlitz wrote: Eli Cohen wrote: 15 kernel_patches/fixes/ipoib_0190_unsig_udqp.patch On older kernels this patch seems to improve throughput of small messages so we should make the effort to include it. I would like to verify this again and if

[ewg] [PATCH] IB/ipoib: set child MTU as the parent's

2008-04-29 Thread Eli Cohen
From 71e918e23f7f8815f3248c1089f69680ae6a203b Mon Sep 17 00:00:00 2001 From: Eli Cohen [EMAIL PROTECTED] Date: Tue, 29 Apr 2008 11:48:09 +0300 Subject: [PATCH] IB/ipoib: set child MTU as the parent's When the child joins the broadcast group reset the mtu to the real one. Signed-off-by: Eli Cohen

[ewg] Choose healthy and long lasting life.

2008-04-29 Thread ashly
You can choose any enhancement products from our site. http://www.loauvcom.com/ ___ ewg mailing list ewg@lists.openfabrics.org http://lists.openfabrics.org/cgi-bin/mailman/listinfo/ewg

Re: [ewg] status of ofed ipoib changes which are not upstream

2008-04-29 Thread Or Gerlitz
Eli Cohen wrote: I don't intend to push this to upstream kernel but I think we should keep it for ofed since for older kernels it provides better performance. Again, the patch is very sensitive by nature, it did not pass a review and numerous bugs were associated with it through the 1.3 cycle,

Re: [ewg] Re: [ofa-general] [PATCH 0/8] RDS patch set

2008-04-29 Thread Olaf Kirch
On Tuesday 29 April 2008 13:36:36 Or Gerlitz wrote: Olaf - I prefer that we wait a second for Roland to finish the review and merging of the mthca/mlx4 patches (maybe it would be splited to two patches) before merging them into 1.3.1 such that the instance in ofed would be the exact copy of

[ewg] [PATCH] IB/ehca: Allocate event queue size depending on max number of CQs and QPs

2008-04-29 Thread Stefan Roscher
Signed-off-by: Stefan Roscher stefan.roscher at de.ibm.com --- drivers/infiniband/hw/ehca/ehca_classes.h |5 drivers/infiniband/hw/ehca/ehca_cq.c | 10 drivers/infiniband/hw/ehca/ehca_main.c| 36 +++- drivers/infiniband/hw/ehca/ehca_qp.c

[ewg] Re: [PATCH] IB/ehca: Allocate event queue size depending on max number of CQs and QPs

2008-04-29 Thread Roland Dreier
Signed-off-by: Stefan Roscher stefan.roscher at de.ibm.com Kind of an inadequate changelog ;) Is this a fix or an enhancement or what? +if (atomic_read(shca-num_cqs) = ehca_max_cq) { +if (atomic_read(shca-num_qps) = ehca_max_qp) { These are racy in the sense that multiple

[ewg] Re: [ PATCH 3/3 ] RDMA/nes SFP+ cleanup

2008-04-29 Thread Roland Dreier
My bad, on the array index idiom. I can redo. Yes, please do resend without that. With regard to post patch clean-ups, I recall you telling me that is was preferred to either front-load or back-load the cleanups in a patch series. Yes, that is true. I generally cleaned-up the

[ewg] Re: [PATCH] IB/ipoib: set child MTU as the parent's

2008-04-29 Thread Roland Dreier
When the child joins the broadcast group reset the mtu to the real one. This changelog is a little too short for me to understand what this is fixing. It seems that child devices are left with a bogus MTU until they complete their multicast join, is that it? +priv-dev-mtu =

[ewg] Re: [REPOST][PATCH] IB/ehca: Allocate event queue size depending on max number of CQs and QPs

2008-04-29 Thread Roland Dreier
thanks, makes sense, applied. fast turnaround too ;) ___ ewg mailing list ewg@lists.openfabrics.org http://lists.openfabrics.org/cgi-bin/mailman/listinfo/ewg

[ewg] Re: [PATCH] IB/ipoib: set child MTU as the parent's

2008-04-29 Thread Eli Cohen
On Tue, Apr 29, 2008 at 9:18 PM, Roland Dreier [EMAIL PROTECTED] wrote: This changelog is a little too short for me to understand what this is fixing. It seems that child devices are left with a bogus MTU until they complete their multicast join, is that it? The situation is even worse since

[ewg] Re: [PATCH] IB/ipoib: set child MTU as the parent's

2008-04-29 Thread Roland Dreier
I think they do but it would require using two fields for flags at the private data. priv-flags would save flags that relate to the state of the net device, and say, priv-cap_flags, to save stuff like LRO, checksum or any other stuff related to capabilities. What do you think? We could

[ewg] Re: [PATCH] IB/ipoib: set child MTU as the parent's

2008-04-29 Thread Eli Cohen
We could do that or just copy only the flags that should be copied when creating a child device. Or we could define a clone function that will have the wisdom of which flags to copy. ___ ewg mailing list ewg@lists.openfabrics.org

[ewg] [ PATCH 2/3 v2 ] RDMA/nes SFP+ enablement

2008-04-29 Thread Glenn Streiff
From: Eric Schneider [EMAIL PROTECTED] This patch enables the iw_nes module for NetEffect RNICs to support additional PHYs including SFP+ optical transceivers (referred to as ARGUS in the code). Signed-off-by: Eric Schneider [EMAIL PROTECTED] Signed-off-by: Glenn Streiff [EMAIL PROTECTED] ---

[ewg] [ PATCH 3/3 v2 ] RDMA/nes Formatting cleanup

2008-04-29 Thread Glenn Streiff
Various cleanups: Change // to /* .. */ Place whitespace around binary operators. Trim down a few long lines. Some minor alignment formatting for better readability. Remove some silly tabs. Signed-off-by: Glenn Streiff [EMAIL PROTECTED] --- Roland, this is

[ewg] Re: [ PATCH 3/3 v2 ] RDMA/nes Formatting cleanup

2008-04-29 Thread Roland Dreier
All looks fine, I applied all three of your patches. Thanks ___ ewg mailing list ewg@lists.openfabrics.org http://lists.openfabrics.org/cgi-bin/mailman/listinfo/ewg