Re: [PATCH v3.10.y 0/4] IB/IPoIB fixes for SRIOV / cloud environment

2013-08-06 Thread Greg KH
On Tue, Aug 06, 2013 at 11:38:38AM +0300, Or Gerlitz wrote: Hi Greg, This series contains few fixes to the IB core, mlx4 IB driver and IPoIB. The patches enable working properly with IPoIB devices set over SRIOV Virtual Functions probed to VMs in cloud environment when the cloud

Re: [PATCH v3.10.y 0/4] IB/IPoIB fixes for SRIOV / cloud environment

2013-08-06 Thread Or Gerlitz
On 06/08/2013 12:38, Greg KH wrote: On Tue, Aug 06, 2013 at 11:38:38AM +0300, Or Gerlitz wrote: Hi Greg, This series contains few fixes to the IB core, mlx4 IB driver and IPoIB. The patches enable working properly with IPoIB devices set over SRIOV Virtual Functions probed to VMs in cloud

[PATCH ibacm 4/8] acm.c: Fix endian of transaction ID

2013-08-06 Thread Hal Rosenstock
It should be in network rather than host endian. Signed-off-by: Hal Rosenstock h...@mellanox.com --- src/acm.c |6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/src/acm.c b/src/acm.c index 7de5873..d68746b 100644 --- a/src/acm.c +++ b/src/acm.c @@ -839,7 +839,7 @@

[PATCH ibacm 2/8] acm.c: In acm_port_up, handle noncontiguous pkeys

2013-08-06 Thread Hal Rosenstock
There can be holes in pkey table although this is not usually the case but since IBA spec allows for this, it should be handled. Signed-off-by: Hal Rosenstock h...@mellanox.com --- src/acm.c | 14 +++--- 1 files changed, 7 insertions(+), 7 deletions(-) diff --git a/src/acm.c

[PATCH ibacm 1/8] libacm.c: Remove unneeded structures

2013-08-06 Thread Hal Rosenstock
acm_port and acm_device are not used/needed Signed-off-by: Hal Rosenstock h...@mellanox.com --- src/libacm.c | 15 --- 1 files changed, 0 insertions(+), 15 deletions(-) diff --git a/src/libacm.c b/src/libacm.c index 1bf7030..4e42233 100644 --- a/src/libacm.c +++ b/src/libacm.c @@

[PATCH ibacm 5/8] acm.c: Make sure MGID has full member pkey bit on

2013-08-06 Thread Hal Rosenstock
even when limited member of partition Signed-off-by: Hal Rosenstock h...@mellanox.com --- src/acm.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/src/acm.c b/src/acm.c index d68746b..146d6e8 100644 --- a/src/acm.c +++ b/src/acm.c @@ -1493,7 +1493,7 @@ static void

[PATCH ibacm 3/8] acm.c: Invalid pkey has low order 15 bits 0

2013-08-06 Thread Hal Rosenstock
so add in handling for pkey 0x8000 as well as 0 for invalid Signed-off-by: Hal Rosenstock h...@mellanox.com --- src/acm.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/src/acm.c b/src/acm.c index a15f094..7de5873 100644 --- a/src/acm.c +++ b/src/acm.c @@ -3211,7

[PATCH ibacm 0/8} PKey and QKey related ACM fixes

2013-08-06 Thread Hal Rosenstock
Miscellaneous fixes for P_Key and Q_Key handling in ACM. Hal Rosenstock (8): libacm.c: Remove unneeded structures acm.c: In acm_port_up, handle noncontiguous pkeys acm.c: Invalid pkey has low order 15 bits 0 acm.c: Fix endian of transaction ID acm.c: Make sure MGID has full member pkey

[PATCH ibacm 7/8] acm.c: Fix endian of qkey in MCMemberRecord in acm_init_join

2013-08-06 Thread Hal Rosenstock
Signed-off-by: Hal Rosenstock h...@mellanox.com --- src/acm.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/src/acm.c b/src/acm.c index b35c986..83c6b70 100644 --- a/src/acm.c +++ b/src/acm.c @@ -1496,7 +1496,7 @@ static void acm_init_join(struct ib_sa_mad *mad, union

[PATCH ibacm 6/8] acm.c: Send multicast joins to SA on default partition

2013-08-06 Thread Hal Rosenstock
rather than on EP partition Prefer full default partition in case both full and limited pkeys for default partition are in pkey table. Signed-off-by: Hal Rosenstock h...@mellanox.com --- src/acm.c | 15 +-- 1 files changed, 13 insertions(+), 2 deletions(-) diff --git a/src/acm.c

[PATCH ibacm 8/8] acm.c: In acm_join_group, no need to set qkey in umad addr to send to SA

2013-08-06 Thread Hal Rosenstock
Signed-off-by: Hal Rosenstock h...@mellanox.com --- src/acm.c |1 - 1 files changed, 0 insertions(+), 1 deletions(-) diff --git a/src/acm.c b/src/acm.c index 83c6b70..30129f7 100644 --- a/src/acm.c +++ b/src/acm.c @@ -1524,7 +1524,6 @@ static void acm_join_group(struct acm_ep *ep, union

Re: [PATCH V3 for-next 0/4] Add receive Flow Steering support

2013-08-06 Thread Or Gerlitz
On 05/08/2013 23:54, Shawn Bohrer wrote: The problem with running and testing this code is that many of these features are posted in incomplete (from the user point of view) chunks. As an example I don't really see how anyone is currently using raw QPs on upstream code since this flow steering

Re: [PATCH V1 FIXES for-3.11 0/4] Pkey fixes for IB core and IPoIB

2013-08-06 Thread Or Gerlitz
On Tue, Aug 6, 2013 at 6:13 AM, Roland Dreier rol...@kernel.org wrote: On Mon, Aug 5, 2013 at 2:02 PM, Or Gerlitz or.gerl...@gmail.com wrote: OK, good. So we're post 3.11-rc4 now, which means the merge window can be three weeks away. Are you planning to pick 3.12 materials such as the iser

Re: [PATCH V9 libmlx4 0/2] Add extension and XRC QP support

2013-08-06 Thread Or Gerlitz
On Thu, Aug 1, 2013 at 6:05 PM, Yishai Hadas yish...@mellanox.com wrote: Extend XRC support to user space. Because XRC requires new verbs and extensions to existing verbs, we first implement the infra-structure changes needed to support verbs extensions matching to libibverbs scheme, later XRC

[PATCH 8/8] RDMA/cxgb4: Issue RI.FINI before closing when entering TERM

2013-08-06 Thread Vipul Pandya
From: Steve Wise sw...@opengridcomputing.com Signed-off-by: Steve Wise sw...@opengridcomputing.com Signed-off-by: Vipul Pandya vi...@chelsio.com --- drivers/infiniband/hw/cxgb4/qp.c | 7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/drivers/infiniband/hw/cxgb4/qp.c

[PATCH 1/8] RDMA/cxgb4: Use correct bit shift macros for vlan filter tuples

2013-08-06 Thread Vipul Pandya
From: Steve Wise sw...@opengridcomputing.com Signed-off-by: Steve Wise sw...@opengridcomputing.com --- drivers/infiniband/hw/cxgb4/cm.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/infiniband/hw/cxgb4/cm.c b/drivers/infiniband/hw/cxgb4/cm.c index 9423441..31dee27

[PATCH 0/8] Critical bug fixes for RDMA/cxgb4

2013-08-06 Thread Vipul Pandya
This patch series fixes critical bugs for RDMA/cxgb4. It fixes bugs in QP flush path, handles newer firmware changes, deals with wrap around issue of the queue, sets arp error handler etc. The patch-series is based on Roland's infiniband tree for-next branch. This series is on top of previously

[PATCH 6/8] RDMA/cxgb4: Always do GTS write if cidx_inc == CIDXINC_MASK.

2013-08-06 Thread Vipul Pandya
From: Steve Wise sw...@opengridcomputing.com When polling, we do a GTS update if the accumulated cidx_inc == the CQ depth / 16. However, if the CQ is large enough, Cq depth / 16 exceeds the size of the field in the GTS word. So we also need to update if cidx_inc hits CIDXINC_MASK to avoid

[PATCH 5/8] RDMA/cxgb4: Set arp error handler for PASS_ACCEPT_RPL messages.

2013-08-06 Thread Vipul Pandya
From: Steve Wise sw...@opengridcomputing.com accept_cr() failed to set the arp error handler on a reused skb. This results in a kernel crash if the arp does indeed time out. Signed-off-by: Steve Wise sw...@opengridcomputing.com Signed-off-by: Vipul Pandya vi...@chelsio.com ---

[PATCH 2/8] RDMA/cxgb4: Handle newer firmware changes

2013-08-06 Thread Vipul Pandya
From: Steve Wise sw...@opengridcomputing.com Move QP to TERMINATE instead to allow the peer to get the TERM message. This bug wasn't detectable until newer FW that moves connections out of RDMA mode as soon as an error is detected. QP can exit RTS before the last AE arrives. This was introduced

[PATCH 3/8] RDMA/cxgb4: Fix QP flush logic

2013-08-06 Thread Vipul Pandya
From: Steve Wise sw...@opengridcomputing.com This patch makes following fixes in QP flush logic: - correctly flushes unsignaled WRs followed by a signaled WR - supports for flushing a CQ bound to multiple QPs - resets cidx_flush if a active queue starts getting HW CQEs again - marks WQ in error

Re: [PATCH V1 FIXES for-3.11 0/4] Pkey fixes for IB core and IPoIB

2013-08-06 Thread Roland Dreier
On Tue, Aug 6, 2013 at 6:14 AM, Or Gerlitz or.gerl...@gmail.com wrote: Any time estimate for when you start accepting 3.12 patches into your for-next branch? No. - R. -- To unsubscribe from this list: send the line unsubscribe linux-rdma in the body of a message to majord...@vger.kernel.org

Re: [PATCH V9 libmlx4 0/2] Add extension and XRC QP support

2013-08-06 Thread Roland Dreier
On Tue, Aug 6, 2013 at 8:03 AM, Or Gerlitz or.gerl...@gmail.com wrote: Could you finally-long-lastish post some comments on how you see this? any bugs or flames you find here, is this going to be accepted? when? Have you read the patches? Any comments? - R. -- To unsubscribe from this list:

Re: [PATCH 0/8] Critical bug fixes for RDMA/cxgb4

2013-08-06 Thread Roland Dreier
On Tue, Aug 6, 2013 at 8:34 AM, Vipul Pandya vi...@chelsio.com wrote: This series is on top of previously submitted series Add IPv6 support for iWARP which can be found at following location: http://comments.gmane.org/gmane.linux.network/272124 We request Roland to merge above series first

Re: [PATCH 0/8] Critical bug fixes for RDMA/cxgb4

2013-08-06 Thread Or Gerlitz
On Tue, Aug 6, 2013 at 6:34 PM, Vipul Pandya vi...@chelsio.com wrote: [...] The patch-series is based on Roland's infiniband tree for-next branch. This series is on top of previously submitted series Add IPv6 support for iWARP We request Roland to merge above series first and then apply

Re: [PATCH V1 FIXES for-3.11 0/4] Pkey fixes for IB core and IPoIB

2013-08-06 Thread Or Gerlitz
On Tue, Aug 6, 2013 at 8:40 PM, Roland Dreier rol...@kernel.org wrote: On Tue, Aug 6, 2013 at 6:14 AM, Or Gerlitz or.gerl...@gmail.com wrote: Any time estimate for when you start accepting 3.12 patches into your for-next branch? No. So we are five days from RC5 and you don't have even

Re: [PATCH V9 libmlx4 0/2] Add extension and XRC QP support

2013-08-06 Thread Or Gerlitz
On Tue, Aug 6, 2013 at 8:40 PM, Roland Dreier rol...@kernel.org wrote: On Tue, Aug 6, 2013 at 8:03 AM, Or Gerlitz or.gerl...@gmail.com wrote: Could you finally-long-lastish post some comments on how you see this? any bugs or flames you find here, is this going to be accepted? when? Have you