[PATCH V3 0/5] Add LE hash collision bug fix for active and passive offloaded connections

2012-12-03 Thread Vipul Pandya
This patch series fixes the LE hash collision issue in cxgb4 and RDMA/cxgb4 drivers in kernel.org. If the hash functionality is enabled in T4 then tuple information of active and passive offloaded connections are stored in DDR3 memory. LE (Lookup Engine) implements the interface to search this

[PATCH V3 1/5] cxgb4: Add T4 filter support

2012-12-03 Thread Vipul Pandya
The T4 architecture is capable of filtering ingress packets at line rate using the rule in TCAM. If packet hits a rule in the TCAM then it can be either dropped or passed to the receive queues based on a rule settings. This patch adds framework for managing filters and to use T4's filter

[PATCH V3 2/5] cxgb4: Add LE hash collision bug fix path in LLD driver

2012-12-03 Thread Vipul Pandya
It supports establishing passive open connection through firmware filter work request. Passive open connection will go through this path as now instead of listening server we create a server filter which will redirect the incoming SYN packet to the offload queue. It divides filter region into

[PATCH V3 5/5] RDMA/cxgb4: Fix bug for active and passive LE hash collision path

2012-12-03 Thread Vipul Pandya
Retries active opens for INUSE errors. Logs any active ofld_connect_wr error replies. Sends ofld_connect_wr on same ctrlq. It needs to go on the same control txq as regular CPL active/passive messages. Retries on active open replies with EADDRINUSE. Uses active open fw wr only if active

[PATCH V3 3/5] RDMA/cxgb4: Fix LE hash collision bug for active open connection

2012-12-03 Thread Vipul Pandya
It enables establishing active open connection using fw_ofld_connection work request when cpl_act_open_rpl says TCAM full error which may be because of LE hash collision. Current support is only for IPv4 active open connections. Sets ntuple bits in active open requests. For T4 firmware greater

[PATCH V3 4/5] RDMA/cxgb4: Fix LE hash collision bug for passive open connection

2012-12-03 Thread Vipul Pandya
It establishes passive open connection through firmware work request. Passive open connection will go through this path as now instead of listening server we create a server filter which will redirect the incoming SYN packet to the offload queue. After this driver tries to establish the connection

Re: Dapltest test error DAT_CONN_QUAL_IN_USE

2012-12-03 Thread Vipul Pandya
Hi Arlin, There was already a bug logged in openfabrics bugzilla regarding this. Following is a link for the same. http://bugs.openfabrics.org/bugzilla/show_bug.cgi?id=2400 I have assigned this bug to your name. Thanks, Vipul On 01-12-2012 01:46, Davis, Arlin R wrote:

[PATCH] opensm/osm_req.c: In req_determine_mkey, add more info when ERR 1107 occurs

2012-12-03 Thread Hal Rosenstock
Also dump DR path of SM class query Signed-off-by: Hal Rosenstock h...@mellanox.com --- opensm/osm_req.c |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/opensm/osm_req.c b/opensm/osm_req.c index 1c3c172..1f5db3f 100644 --- a/opensm/osm_req.c +++ b/opensm/osm_req.c @@

[PATCH 1/2] opensm/osm_req.c: In req_determine_mkey, fix DR algorithm

2012-12-03 Thread Hal Rosenstock
so that it works regardless of SM port type Now works on switch port 0 as well as CA and router ports Signed-off-by: Hal Rosenstock h...@mellanox.com --- opensm/osm_req.c | 12 1 files changed, 8 insertions(+), 4 deletions(-) diff --git a/opensm/osm_req.c b/opensm/osm_req.c

[PATCH 2/2] opensm/osm_req.c: In req_determine_mkey, use osm_physp APIs

2012-12-03 Thread Hal Rosenstock
rather tha direct accesses to physp struct Signed-off-by: Hal Rosenstock h...@mellanox.com --- opensm/osm_req.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/opensm/osm_req.c b/opensm/osm_req.c index b48be48..caf94fb 100644 --- a/opensm/osm_req.c +++

[PATCH] opensm/osm_subnet.c: Improve error messages in subn_validate_neighbor

2012-12-03 Thread Hal Rosenstock
Signed-off-by: Hal Rosenstock h...@mellanox.com --- opensm/osm_subnet.c | 22 -- 1 files changed, 16 insertions(+), 6 deletions(-) diff --git a/opensm/osm_subnet.c b/opensm/osm_subnet.c index a3fc0b5..c716839 100644 --- a/opensm/osm_subnet.c +++ b/opensm/osm_subnet.c @@

[PATCH] opensm/osm_ucast_ftree.c: Remove duplicate free in fabric_create_leaf_switch_array

2012-12-03 Thread Hal Rosenstock
Signed-off-by: Dan Ben-Yosef da...@mellanox.com Signed-off-by: Hal Rosenstock h...@mellanox.com --- opensm/osm_ucast_ftree.c |1 - 1 files changed, 0 insertions(+), 1 deletions(-) diff --git a/opensm/osm_ucast_ftree.c b/opensm/osm_ucast_ftree.c index 532a29f..133fd4f 100644 ---

[PATCH] opensm/osm_ucast_ftree.c: Eliminate unneeded NULL pointer checks prior to calls to free

2012-12-03 Thread Hal Rosenstock
Signed-off-by: Hal Rosenstock h...@mellanox.com --- opensm/osm_ucast_ftree.c | 20 ++-- 1 files changed, 6 insertions(+), 14 deletions(-) diff --git a/opensm/osm_ucast_ftree.c b/opensm/osm_ucast_ftree.c index 133fd4f..d58fff1 100644 --- a/opensm/osm_ucast_ftree.c +++

[PATCH] opensm/osm_torus.c: Fix crash in torus_update_osm_vlarb

2012-12-03 Thread Hal Rosenstock
Signed-off-by: Alex Netes ale...@mellanox.com Signed-off-by: Hal Rosenstock h...@mellanox.com --- opensm/osm_torus.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/opensm/osm_torus.c b/opensm/osm_torus.c index c06f8d4..075f84a 100644 --- a/opensm/osm_torus.c +++

Re: [PATCH V3 1/5] cxgb4: Add T4 filter support

2012-12-03 Thread David Miller
From: Vipul Pandya vi...@chelsio.com Date: Mon, 3 Dec 2012 16:52:57 +0530 + /* + * If the new or old filter have loopback rewriteing rules then we'll + * need to free any existing Layer Two Table (L2T) entries of the old + * filter rule. The firmware will handle freeing

RE: [PATCH librdmacm] Fixed build problem due to missing macro

2012-12-03 Thread Hefty, Sean
Thanks - applied. I'm using a locally modified version of verbs, so I don't end up seeing this error. -- To unsubscribe from this list: send the line unsubscribe linux-rdma in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html

Re: [PATCH librdmacm] Fixed build problem due to missing macro

2012-12-03 Thread Or Gerlitz
On Mon, Dec 3, 2012 at 9:35 PM, Hefty, Sean sean.he...@intel.com wrote: I'm using a locally modified version of verbs, so I don't end up seeing this error. could this be related to different results while running netperf .w fork in our setup vs. yours? Or. -- To unsubscribe from this list:

RE: [PATCH librdmacm] Fixed build problem due to missing macro

2012-12-03 Thread Hefty, Sean
I'm using a locally modified version of verbs, so I don't end up seeing this error. could this be related to different results while running netperf .w fork in our setup vs. yours? I was referring to the build issue, not the fork problem. According to my notes, I ran netperf with fork

[PATCH] infiniband-diags: Remove Red Hat-ism.

2012-12-03 Thread Jon Stanley
Future release of Fedora are going to remove /etc/sysconfig/network which we source to get $HOSTNAME. Bash (and sh) set $HOSTNAME in the shell by default, so we should be safe to use that here. This adds the benefit of working across multiple distributions if that is required in the future.

Re: [PATCH] opensm/osm_torus.c: Fix crash in torus_update_osm_vlarb

2012-12-03 Thread Hal Rosenstock
On 12/3/2012 4:09 PM, Jim Schutt wrote: On 12/03/2012 08:26 AM, Hal Rosenstock wrote: Signed-off-by: Alex Netesale...@mellanox.com Signed-off-by: Hal Rosenstockh...@mellanox.com --- opensm/osm_torus.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git

opensm/osm_torus.c: Fix crash in torus_update_osm_vlarb

2012-12-03 Thread Hal Rosenstock
Signed-off-by: Alex Netes ale...@mellanox.com Signed-off-by: Hal Rosenstock h...@mellanox.com --- Change since v1: Fixed NULL pointer check on sw-port[port_num] Pointed out by: Jim Schutt jasc...@sandia.gov opensm/osm_torus.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff

[PATCHv2} opensm/osm_torus.c: Fix crash in torus_update_osm_vlarb

2012-12-03 Thread Hal Rosenstock
Signed-off-by: Alex Netes ale...@mellanox.com Signed-off-by: Hal Rosenstock h...@mellanox.com --- Change since v1: Fixed NULL pointer check on sw-port[port_num] Pointed out by: Jim Schutt jasc...@sandia.gov opensm/osm_torus.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff

[PATCH] opensm/osm_port_info_rcv.c: Don't modify subnet minimal values when PortState DOWN

2012-12-03 Thread Hal Rosenstock
as they are vendor dependent fields in PortInfo and can't be relied on due to: C14-24.2.1: If PortInfo:Portstate=Down, then o a SubnGet(PortInfo) shall produce valid data for PortInfo:PortState and PortInfo:PortPhysicalState; whether any other component has valid data is vendor-dependent.

Re: [PATCH] opensm/osm_torus.c: Fix crash in torus_update_osm_vlarb

2012-12-03 Thread Jim Schutt
On 12/03/2012 08:26 AM, Hal Rosenstock wrote: Signed-off-by: Alex Netesale...@mellanox.com Signed-off-by: Hal Rosenstockh...@mellanox.com --- opensm/osm_torus.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/opensm/osm_torus.c b/opensm/osm_torus.c index

Re: [PATCHv2} opensm/osm_torus.c: Fix crash in torus_update_osm_vlarb

2012-12-03 Thread Jim Schutt
On 12/03/2012 02:25 PM, Hal Rosenstock wrote: Signed-off-by: Alex Netesale...@mellanox.com Signed-off-by: Hal Rosenstockh...@mellanox.com Acked-by: Jim Schutt jasc...@sandia.gov --- Change since v1: Fixed NULL pointer check on sw-port[port_num] Pointed out by: Jim Schuttjasc...@sandia.gov

Re: [PATCH] infiniband-diags: Remove Red Hat-ism.

2012-12-03 Thread Jason Gunthorpe
On Mon, Dec 03, 2012 at 03:28:18PM -0500, Jon Stanley wrote: Future release of Fedora are going to remove /etc/sysconfig/network which we source to get $HOSTNAME. Bash (and sh) set $HOSTNAME in the shell by default, so we should be safe to use that here. This adds the benefit of working across