[PATCH 0/3] batch of mlx4 patches for 3.4

2012-03-06 Thread Or Gerlitz
Hi Roland, Here's a small batch of patches for 3.4 to the mlx4 driver, its made of a fix from Eli, report on thermal error event from Jack and a cleanup from me. Or. Eli Cohen (1): IB/mlx4: Fix mlx4_ib_poll_cq possible missed completion Jack Morgenstein (1): net/mlx4: report thermal

[PATCH 2/3] net/mlx4: remove redundant maintainance of extended port capabilities

2012-03-06 Thread Or Gerlitz
While doing the work for commit a6f7feae6d IB/mlx4: pass SMP vendor-specific attribute MADs to firmware we realized that the firmware would respond on all sorts of vendor specific mads. Hence, commit 97285b7817 mlx4_core: Add extended port capabilities support adds redundant code into the driver,

[PATCH 1/3] net/mlx4: report thermal error event

2012-03-06 Thread Or Gerlitz
From: Jack Morgenstein ja...@dev.mellanox.co.il Print an error message when an async event of thermal error is being reported by the HW. Signed-off-by: Jack Morgenstein ja...@dev.mellanox.co.il Signed-off-by: Dotan Barak dot...@mellanox.com --- drivers/net/ethernet/mellanox/mlx4/eq.c | 32

[PATCH 3/3] IB/mlx4: Fix mlx4_ib_poll_cq possible missed completion

2012-03-06 Thread Or Gerlitz
From: Eli Cohen e...@mellanox.com If an erroneous CQE was polled in the first iteration (e.g. npolled == 0), we would not update the consumer index hence the hardware could get a wrong notion of how many CQEs software polled. Fix this by unconditionally updating the doorbell record. We could put

Re: mlx4 verbs question

2012-03-06 Thread Or Gerlitz
On 3/6/2012 5:44 AM, Bob Pearson wrote: From the RTS state issuing a modify qp with qp_attr_mask only set to state to the SQD state AFAIK SQD isn't supported by connectx Or. -- To unsubscribe from this list: send the line unsubscribe linux-rdma in the body of a message to

rds cq event handler issue

2012-03-06 Thread Steve Wise
Hey Venkat, I think I see a bug in the RDS RDMA module where RDS is not adhering to the RDMA locking context. From the kernel tree Documentation/infiniband/core_locking.txt: --- The context in which completion event and asynchronous event callbacks run is not defined. Depending on the

Re: rds cq event handler issue

2012-03-06 Thread Venkat Venkatsubra
On 3/6/2012 12:05 PM, Steve Wise wrote: Hey Venkat, I think I see a bug in the RDS RDMA module where RDS is not adhering to the RDMA locking context. From the kernel tree Documentation/infiniband/core_locking.txt: --- The context in which completion event and asynchronous event

Re: rds cq event handler issue

2012-03-06 Thread Steve Wise
On 03/06/2012 12:58 PM, Venkat Venkatsubra wrote: On 3/6/2012 12:05 PM, Steve Wise wrote: Hey Venkat, I think I see a bug in the RDS RDMA module where RDS is not adhering to the RDMA locking context. From the kernel tree Documentation/infiniband/core_locking.txt: --- The context in which

[PATCH] libibmad: Add MKey support to SMP requests via smp_mkey_get/set()

2012-03-06 Thread Jim Foraker
Signed-off-by: Jim Foraker forak...@llnl.gov --- include/infiniband/mad.h |2 ++ src/libibmad.map |2 ++ src/mad_internal.h |1 + src/smp.c| 12 4 files changed, 17 insertions(+), 0 deletions(-) diff --git a/include/infiniband/mad.h

[PATCH 1/3] infiniband-diags: Make ibportstate support displaying/changing MKey, lease, and protect bits

2012-03-06 Thread Jim Foraker
Displaying/changing are both supported for the MKey itself, plus lease and protect bits Signed-off-by: Jim Foraker forak...@llnl.gov --- src/ibportstate.c | 64 +++- 1 files changed, 57 insertions(+), 7 deletions(-) diff --git

[PATCH 2/3] infiniband-diags: Allow specification of an mkey on the command line

2012-03-06 Thread Jim Foraker
Signed-off-by: Jim Foraker forak...@llnl.gov --- include/ibdiag_common.h |1 + libibnetdisc/include/infiniband/ibnetdisc.h |3 ++- libibnetdisc/src/ibnetdisc.c|1 + man/ibaddr.8|2 ++ src/ibaddr.c

[PATCH 3/3] infiniband-diags: Add mkey support to config file

2012-03-06 Thread Jim Foraker
Support for m_key= parameter, plus config file now installs user-read-only. Signed-off-by: Jim Foraker forak...@llnl.gov --- Makefile.am |2 +- etc/ibdiag.conf |2 ++ src/ibdiag_common.c |2 ++ 3 files changed, 5 insertions(+), 1 deletions(-) diff --git a/Makefile.am

[PATCH fixed] libibmad: Add MKey support to SMP requests via smp_mkey_get/set()

2012-03-06 Thread Jim Foraker
Signed-off-by: Jim Foraker forak...@llnl.gov --- include/infiniband/mad.h |2 ++ src/libibmad.map |2 ++ src/mad_internal.h |1 + src/smp.c| 12 4 files changed, 17 insertions(+), 0 deletions(-) diff --git a/include/infiniband/mad.h

[PATCH 1/3 fixed] infiniband-diags: Make ibportstate support displaying/changing MKey, lease, and protect bits

2012-03-06 Thread Jim Foraker
Displaying/changing are both supported for the MKey itself, plus lease and protect bits Signed-off-by: Jim Foraker forak...@llnl.gov --- src/ibportstate.c | 64 +++- 1 files changed, 57 insertions(+), 7 deletions(-) diff --git

[PATCH 2/3 fixed] infiniband-diags: Allow specification of an mkey on the command line

2012-03-06 Thread Jim Foraker
Signed-off-by: Jim Foraker forak...@llnl.gov --- include/ibdiag_common.h |1 + libibnetdisc/include/infiniband/ibnetdisc.h |3 ++- libibnetdisc/src/ibnetdisc.c|1 + man/ibaddr.8|2 ++ src/ibaddr.c

RE: mlx4 verbs question

2012-03-06 Thread Bob Pearson
Oops. Take that back. It didn't work in fact that was the one that returned the '1'. -Original Message- From: Or Gerlitz [mailto:ogerl...@mellanox.com] Sent: Tuesday, March 06, 2012 11:16 AM To: Bob Pearson Cc: linux-rdma@vger.kernel.org; Tziporet Koren Subject: Re: mlx4 verbs

RE: mlx4 verbs question

2012-03-06 Thread Bob Pearson
-Original Message- From: Or Gerlitz [mailto:ogerl...@mellanox.com] Sent: Tuesday, March 06, 2012 11:16 AM To: Bob Pearson Cc: linux-rdma@vger.kernel.org; Tziporet Koren Subject: Re: mlx4 verbs question On 3/6/2012 5:44 AM, Bob Pearson wrote: From the RTS state issuing a modify