[PATCH] drivers/infiniband: Add a check for strict_strtoul()

2011-04-01 Thread Liu Yuan
From: Liu Yuan tailai...@taobao.com It should check if strict_strtoul() succeeds before using 'wqm_quanta_value'. Signed-off-by: Liu Yuan tailai...@taobao.com --- drivers/infiniband/hw/nes/nes.c |4 +++- 1 files changed, 3 insertions(+), 1 deletions(-) diff --git

HA mode bonding for IB over SDP

2011-04-01 Thread Bhavin
Hi, I am a student pursuing masters. I have chosen to work on infiniband for my final year project. As part of my project I am trying to implement infiniband over SDP. I have also setup two systems for this purpose having Mallanox IB cards. I found that OFED doesn’t have support for HA mode

Re: [PATCH] opensm/perfmgr: fix overflow processing

2011-04-01 Thread Alex Netes
On 18:11 Thu 24 Mar , Ira Weiny wrote: From: Ira Weiny wei...@llnl.gov Date: Wed, 9 Mar 2011 16:28:23 -0800 Subject: [PATCH] opensm/perfmgr: fix overflow processing The stored error counters need to be cleared when the clear is issued because of overflow. Signed-off-by: Ira

Re: [PATCH] opensm: Allow comma in plugin names parsing

2011-04-01 Thread Alex Netes
On 10:46 Tue 29 Mar , Eli Dorfman wrote: Subject: [PATCH] Allow comma in plugin names parsing Signed-off-by: Eli Dorfman el...@mellanox.com --- Apllied, thanks. -- To unsubscribe from this list: send the line unsubscribe linux-rdma in the body of a message to majord...@vger.kernel.org

Re: [PATCH 0/4] opensm: [RFC] Add PortCountersExtended support to the PerfMgr

2011-04-01 Thread Alex Netes
Hi Ira, On 18:20 Thu 24 Mar , Ira Weiny wrote: I have briefly tested the following 4 patches to implement PortCountersExtended collection in the performance manager of opensm. They work well for my limited testing. Can you please provide more info regarding the tests you've run.

Re: [PATCH 0/4] opensm: [RFC] Add PortCountersExtended support to the PerfMgr

2011-04-01 Thread Alex Netes
Hi Ira, On 18:20 Thu 24 Mar , Ira Weiny wrote: I have briefly tested the following 4 patches to implement PortCountersExtended collection in the performance manager of opensm. They work well for my limited testing. Can you provide more info regarding the tested you've done?

Re: [PATCH] librdmacm/doc: document the limitations on the user provided private data length

2011-04-01 Thread Yann Droneaud
Le lundi 28 mars 2011 à 10:49 +0200, Or Gerlitz a écrit : Document the limitations on the user provided private data length over Infiniband networks. These limitations are calculated by subtracting the rdma-cm header size (see IBA Annex A11 RDMA CM IP Service) from IB's private data len for

RE: [PATCH] librdmacm/doc: document the limitations on the user provided private data length

2011-04-01 Thread Hefty, Sean
For general REP messages, one have to look to table 103 : REP Message Contents in section 12.6.8 REP - Reply to Request For Communication. Am I correct ? I believe so. The rdma_accept size should be 196 for RDMA_PS_TCP.

[PATCH 1/ 11] opensm/osm_pkey_mgr.c: Fix commentary typo

2011-04-01 Thread Hal Rosenstock
Signed-off-by: Hal Rosenstock h...@mellanox.com --- diff --git a/opensm/osm_pkey_mgr.c b/opensm/osm_pkey_mgr.c index 612130c..a81fa78 100644 --- a/opensm/osm_pkey_mgr.c +++ b/opensm/osm_pkey_mgr.c @@ -36,7 +36,7 @@ /* * Abstract: - * Implementation of the P_Key Manager (Partititon Manager). +

[PATCH 2/11] opensm/osm_pkey_mgr.c: Move clearing of empty_block after any early returns

2011-04-01 Thread Hal Rosenstock
Signed-off-by: Hal Rosenstock h...@mellanox.com --- diff --git a/opensm/osm_pkey_mgr.c b/opensm/osm_pkey_mgr.c index 612130c..ce50f27 100644 --- a/opensm/osm_pkey_mgr.c +++ b/opensm/osm_pkey_mgr.c @@ -263,12 +263,12 @@ static int pkey_mgr_update_port(osm_log_t * p_log, osm_sm_t * sm,

[PATCH 3/11] opensm/include/opensm/osm_mcm_port.h: Cosmetic formatting changes

2011-04-01 Thread Hal Rosenstock
Signed-off-by: Hal Rosenstock hal.rosenst...@gmail.com --- diff --git a/include/opensm/osm_mcm_port.h b/include/opensm/osm_mcm_port.h index ad25b4f..ca126cd 100644 --- a/include/opensm/osm_mcm_port.h +++ b/include/opensm/osm_mcm_port.h @@ -62,14 +62,14 @@ struct osm_mgrp; /s* OpenSM: MCM

[PATCH 5/11] opensm/osm_sa_mcmember_record.c: Changed status in a couple of SA responses

2011-04-01 Thread Hal Rosenstock
For SubnAdmSet and SubnAdmDelete, return insufficient components rather than invalid request when component mask doesn't have at least the required components for a join Signed-off-by: Hal Rosenstock h...@mellanox.com --- diff --git a/opensm/osm_sa_mcmember_record.c

[PATCH 6/11] opensm/osm_sa_mcmember_record.c: Fix handling of invalid PKey

2011-04-01 Thread Hal Rosenstock
In mcmr_rcv_join_mgrp, when pkey is supplied, this needs to be checked as to whether it is invalid or not as subsequent PKey checks are too lenient Signed-off-by: Hal Rosenstock h...@mellanox.com --- diff --git a/opensm/osm_sa_mcmember_record.c b/opensm/osm_sa_mcmember_record.c index

[PATCH 7/11] opensm/osm_pkey.c: Cosmetic formatting change

2011-04-01 Thread Hal Rosenstock
Signed-off-by: Hal Rosenstock h...@mellanox.com --- diff --git a/opensm/osm_pkey.c b/opensm/osm_pkey.c index 4c5a852..1032317 100644 --- a/opensm/osm_pkey.c +++ b/opensm/osm_pkey.c @@ -380,7 +380,6 @@ Exit: boolean_t osm_physp_has_pkey(IN osm_log_t * p_log, IN ib_net16_t pkey,

[PATCH 8/11] opensm/osm_prtn.c: Cosmetic formatting change

2011-04-01 Thread Hal Rosenstock
Signed-off-by: Hal Rosenstock h...@mellanox.com --- diff --git a/opensm/osm_prtn.c b/opensm/osm_prtn.c index 6a23c76..3fd4fc0 100644 --- a/opensm/osm_prtn.c +++ b/opensm/osm_prtn.c @@ -253,7 +253,6 @@ osm_prtn_t *osm_prtn_make_new(osm_log_t * p_log, osm_subn_t * p_subn, osm_prtn_t *p =

[PATCH 9/11] osmtest/osmt_multicast.c: Fixed a couple of typos in OSM_LOG messages

2011-04-01 Thread Hal Rosenstock
Signed-off-by: Hal Rosenstock h...@mellanox.com --- diff --git a/osmtest/osmt_multicast.c b/osmtest/osmt_multicast.c index b6dfdfc..4f0deb5 100644 --- a/osmtest/osmt_multicast.c +++ b/osmtest/osmt_multicast.c @@ -742,7 +742,7 @@ ib_api_status_t osmt_run_mcast_flow(IN osmtest_t * const p_osmt)

[PATCH 10/11] osmtest/osmt_multicast.c: Fixed some error codes in OSM_LOG messages

2011-04-01 Thread Hal Rosenstock
Signed-off-by: Hal Rosenstock h...@mellanox.com --- diff --git a/osmtest/osmt_multicast.c b/osmtest/osmt_multicast.c index b6dfdfc..cfeb61b 100644 --- a/osmtest/osmt_multicast.c +++ b/osmtest/osmt_multicast.c @@ -677,7 +677,7 @@ ib_api_status_t osmt_run_mcast_flow(IN osmtest_t * const p_osmt)

[PATCH 11/11] osmtest/osmt_multicast.c: Fix some typos

2011-04-01 Thread Hal Rosenstock
Signed-off-by: Hal Rosenstock h...@mellanox.com --- diff --git a/osmtest/osmt_multicast.c b/osmtest/osmt_multicast.c index b6dfdfc..6d0ed73 100644 --- a/osmtest/osmt_multicast.c +++ b/osmtest/osmt_multicast.c @@ -729,7 +729,7 @@ ib_api_status_t osmt_run_mcast_flow(IN osmtest_t * const p_osmt)

[PATCH] infiniband-diags.spec.in: Remove duplication of ibdiscover.pl

2011-04-01 Thread Hal Rosenstock
Signed-off-by: Hal Rosenstock h...@mellanox.com --- diff --git a/infiniband-diags.spec.in b/infiniband-diags.spec.in index c50ce09..692ebd7 100644 --- a/infiniband-diags.spec.in +++ b/infiniband-diags.spec.in @@ -37,7 +37,6 @@ rm -rf $RPM_BUILD_ROOT %files %defattr(-,root,root)

[PATCH 1/2] libibmad/smp.c: Fix to smp_query_via for attribute data

2011-04-01 Thread Hal Rosenstock
smp_query_via() does not allow setting the sent mad data. Change the data to be both in and out arguments. Signed-off-by: Oren Kladnitsky or...@dev.mellanox.co.il Signed-off-by: Hal Rosenstock h...@mellanox.com --- diff --git a/src/smp.c b/src/smp.c index 3ff58ce..2809ffe 100644 --- a/src/smp.c

[PATCH 2/2] libibmad: Added smp_query/set _status_via functions

2011-04-01 Thread Hal Rosenstock
The smp_query/set functions do not return the MAD status. The new extended API allows getting the SMP MAD status, and to differentiate between timeout on MAD response and response with bad status. Signed-off-by: Oren Kladnitsky or...@dev.mellanox.co.il Signed-off-by: Hal Rosenstock

[PATCH 1/3] ibacm: Reduce overhead on multicast groups not used

2011-04-01 Thread Hefty, Sean
ACM may join several multicast groups when using the multicast group protocol. No data is ever sent on those groups. They exist simply to see if a port *could* send data across those groups, for example, to validate the mtu or rate between two endpoints. Since we don't send data on them,

[PATCH 2/3] ibacm: Join only the base multicast group if using SA path resolution

2011-04-01 Thread Hefty, Sean
If the ACM service is configured to use the SA to resolve path record data, then only join the base multicast group. We only need to join other groups if we're creating path records based on multicast data, using the acm route protocol. Signed-off-by: Sean Hefty sean.he...@intel.com ---

[PATCH 3/3] ibacm: Allocate address handles dynamically when needed

2011-04-01 Thread Hefty, Sean
ibacm allocates an address handle for every remote destination that it tracks. However, under normal operation, the handle is used infrequently - typically only once by the target service to send a response and not at all on the initiator service. Avoid the overhead of having 1 address handle