[PATCH] opensm/perfmgr: fix endian conversion of PortCounters

2012-07-24 Thread Ira Weiny
Signed-off-by: Ira Weiny --- opensm/osm_perfmgr_db.c |7 +++ 1 files changed, 3 insertions(+), 4 deletions(-) diff --git a/opensm/osm_perfmgr_db.c b/opensm/osm_perfmgr_db.c index a472287..17e839b 100644 --- a/opensm/osm_perfmgr_db.c +++ b/opensm/osm_perfmgr_db.c @@ -769,15 +769,14

Re: [PATCH 04/10] opensm: perfmgr delete "inactive" nodes from the DB

2012-07-25 Thread Ira Weiny
On Wed, 25 Jul 2012 15:57:58 +0300 Alex Netes wrote: > Hi Ira, > > On 16:53 Tue 03 Jul , Ira Weiny wrote: > > > > > > When a node drops from the fabric the default behavior was to leave the node > > data in the PerfMgr DB. Add the option (with default

Work completion error: "transport retry counter exceeded"

2012-07-25 Thread Ira Weiny
ttr, IBV_QP_STATE | IBV_QP_TIMEOUT| IBV_QP_RETRY_CNT | IBV_QP_RNR_RETRY | IBV_QP_SQ_PSN | IBV_QP_MAX_QP_RD_ATOMIC)) {

[PATCH] opensm: include release in osm_version.h

2012-07-26 Thread Ira Weiny
This ensures plugin compatibility. Signed-off-by: Ira Weiny --- include/opensm/osm_version.h.in |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/include/opensm/osm_version.h.in b/include/opensm/osm_version.h.in index d783245..7a29b77 100644 --- a/include/opensm

Re: [PATCH] opensm: include release in osm_version.h

2012-07-26 Thread Ira Weiny
Disregard this for now... I see there is more crazyness with OpenSM's gen_ver.sh script... :-( Ira On Thu, 26 Jul 2012 17:42:51 -0700 Ira Weiny wrote: > > This ensures plugin compatibility. > > Signed-off-by: Ira Weiny > --- > include/opensm/osm_version.h.

Re: Work completion error: "transport retry counter exceeded"

2012-07-27 Thread Ira Weiny
a > > Regards > > Albert > -- > 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 -- Ira Weiny Member of Technical S

[ANNOUNCE] infiniband-diags 1.6.1 release

2012-07-30 Thread Ira Weiny
dedPortInfo for ConnectIB device infiniband-diags/perfquery.c: Fix compile warning infiniband-diags/vendstat.c: Commentary change Ira Weiny (4): infiniband-diags: saquery avoid a potential seg fault in libibmad infiniband-diags: update README and configure.in for 1.6.1 release

Re: Work completion error: "transport retry counter exceeded"

2012-08-01 Thread Ira Weiny
On Fri, 27 Jul 2012 10:42:12 -0700 Ira Weiny wrote: > First, I have gotten pulled into another project so I have not been able to > debug this further. > > I __really__ appreciate all the responses and will report back when I have > found more information. I feel really stupid

[PATCH] opensm/perfmgr: update new error codes to '54' prefix

2012-08-01 Thread Ira Weiny
Signed-off-by: Ira Weiny --- opensm/osm_perfmgr.c | 26 +- 1 files changed, 13 insertions(+), 13 deletions(-) diff --git a/opensm/osm_perfmgr.c b/opensm/osm_perfmgr.c index 32357d9..ee2392c 100644 --- a/opensm/osm_perfmgr.c +++ b/opensm/osm_perfmgr.c @@ -1095,18

Re: [PATCH] infiniband-diags: Allow user to specify multiple ports

2012-09-20 Thread Ira Weiny
else > + output_aggregate_perfcounters_ext(&portid, > + cap_mask); > + } > } else > dump_perfcounters(extended, ibd_timeout, cap_mask,

Re: [PATCH] infiniband-diags/src/perfquery.c: Fix all_ports corner case

2012-09-20 Thread Ira Weiny
if (all_ports && port == ALL_PORTS) > all_ports_loop = 1; > } > > -- > 1.7.1 > > > > -- > To unsubscribe from this list: send the line "unsubscribe linux-rdma" in > the body of a message to majord...@vger.kernel.org >

[PATCH] infiniband-diags: move sa query code into it's own common file

2012-09-21 Thread Ira Weiny
Also remove bind from handle name Signed-off-by: Ira Weiny --- Makefile.am |2 +- include/ibdiag_common.h | 46 -- include/ibdiag_sa.h | 92 + src/ibdiag_common.c | 165 - src/ibdiag_sa.c

[PATCH] infiniband-diags: fix bug with loop ports where SMP queries are needed

2012-09-21 Thread Ira Weiny
Signed-off-by: Ira Weiny --- src/perfquery.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/src/perfquery.c b/src/perfquery.c index 605ece9..df8556b 100644 --- a/src/perfquery.c +++ b/src/perfquery.c @@ -659,7 +659,7 @@ static int process_opt(void *context, int ch

[PATCH] infiniband-diags: Fix SA error reporting for common MAD errors

2012-09-21 Thread Ira Weiny
Signed-off-by: Ira Weiny --- src/ibdiag_sa.c | 30 +++--- 1 files changed, 27 insertions(+), 3 deletions(-) diff --git a/src/ibdiag_sa.c b/src/ibdiag_sa.c index 2b84bfa..2a9f008 100644 --- a/src/ibdiag_sa.c +++ b/src/ibdiag_sa.c @@ -192,19 +192,43 @@ static inline

Re: opensm not reaching nodes

2012-10-11 Thread Ira Weiny
Ethernet > > Let me know and I can provide any information necessary to help debug. > > -JE > -- > 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://

Re: [PATCH] infiniband-diags: Fix SA error reporting for common MAD errors

2012-10-12 Thread Ira Weiny
On Wed, 10 Oct 2012 10:28:50 -0400 Hal Rosenstock wrote: > On 9/21/2012 6:02 PM, Ira Weiny wrote: > > > > > > Signed-off-by: Ira Weiny > > --- > > src/ibdiag_sa.c | 30 +++--- > > 1 files changed, 27 insertions(+), 3

Re: if/how to dictate IB device name per PCI BDF

2012-10-12 Thread Ira Weiny
> Jason > -- > 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 -- Ira Weiny Member of Technical Staff Lawrence Livermore Nation

Re: [PATCH] infiniband-diags/src/saquery.c: New option to saquery InformInfoRecord (IIR)

2012-11-01 Thread Ira Weiny
t;InformInfoRecord", "IIR", IB_SA_ATTR_INFORMINFORECORD, > - NULL, query_informinfo_records}, > + "[subscriber_gid]", query_inform_info_records}, > {"LinkRecord", "LR", IB_SA_ATTR_LINKRECORD, > "[[from_lid]/[fr

Re: [PATCH] infiniband-diags/ibsysstat.c: Fix non default OUI operation

2012-11-01 Thread Ira Weiny
On Thu, 01 Nov 2012 08:35:41 -0400 Hal Rosenstock wrote: > > Signed-off-by: Hal Rosenstock Thanks applied, Ira -- 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

[PATCH] opensm: update performance manager documentation

2012-11-08 Thread Ira Weiny
Signed-off-by: Ira Weiny --- doc/performance-manager-HOWTO.txt | 35 --- 1 files changed, 8 insertions(+), 27 deletions(-) diff --git a/doc/performance-manager-HOWTO.txt b/doc/performance-manager-HOWTO.txt index 11f4185..7471c6f 100644 --- a/doc/performance

[PATCH 1/2] opensm: Fix incorrect use of sizeof

2012-11-12 Thread Ira Weiny
Signed-off-by: Ira Weiny --- opensm/osm_helper.c | 53 ++- 1 files changed, 27 insertions(+), 26 deletions(-) diff --git a/opensm/osm_helper.c b/opensm/osm_helper.c index a24dd72..578f54a 100644 --- a/opensm/osm_helper.c +++ b/opensm

[PATCH 2/2] opensm: fix missing newline on DR path log messages

2012-11-12 Thread Ira Weiny
Signed-off-by: Ira Weiny --- opensm/osm_helper.c | 46 ++ 1 files changed, 30 insertions(+), 16 deletions(-) diff --git a/opensm/osm_helper.c b/opensm/osm_helper.c index 578f54a..78286a2 100644 --- a/opensm/osm_helper.c +++ b/opensm/osm_helper.c

Re: [PATCH] Remove bashisms from infiniband-diags

2012-11-12 Thread Ira Weiny
6 +++--- scripts/ibclearerrors.in| 6 +++--- scripts/ibdatacounters.in | 6 +++--- scripts/ibdatacounts.in | 8 scripts/set_nodedesc.sh | 2 +- That said, what are you trying to fix here? Is there a reason bash can't be used on your system? Thanks, Ira -- I

Re: [PATCH] Remove bashisms from infiniband-diags

2012-11-15 Thread Ira Weiny
On Mon, 12 Nov 2012 15:44:54 -0800 Garrett Cooper wrote: > On Nov 12, 2012, at 3:22 PM, Ira Weiny wrote: > > > On Sun, 11 Nov 2012 11:05:32 -0800 > > Garrett Cooper wrote: > > > >> Signed-off-by: Garrett Cooper > > > > Please note that the fol

Re: [PATCH] Remove bashisms from infiniband-diags

2012-11-26 Thread Ira Weiny
On Sun, 11 Nov 2012 11:05:32 -0800 Garrett Cooper wrote: > Signed-off-by: Garrett Cooper Thanks applied, Ira -- Ira Weiny Member of Technical Staff Lawrence Livermore National Lab 925-423-8008 wei...@llnl.gov -- To unsubscribe from this list: send the line "unsubscribe linux-rdma

[PATCH] libibumad: fix umad_register man page

2012-12-03 Thread Ira Weiny
Signed-off-by: Ira Weiny --- man/umad_register.3 |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/man/umad_register.3 b/man/umad_register.3 index 7f11b51..eb6b986 100644 --- a/man/umad_register.3 +++ b/man/umad_register.3 @@ -7,7 +7,7 @@ umad_register \- register the

Re: [PATCH] libibmad/dump.c: In mad_dump_mtu, don't set buf[0] to 0

2012-12-07 Thread Ira Weiny
; 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 -- Ira Weiny Member of Technical Staff Lawrence Livermore National Lab 925-423-8

Re: umad_send with service level higher than 0 does not work

2012-12-14 Thread Ira Weiny
hat DFSSSP does to figure out SLs exactly but > there should be no need to set this. DFSSSP requires QoS to be configured in OpenSM; "... to equally spread the load on the available SL or virtual lanes." Ira [snip] -- Ira Weiny Member of Technical Staff Lawrence Livermore National La

Re: [PATCH] infiniband-diags/ibportstate.c: Fix PortInfo sets with extended speeds

2013-01-04 Thread Ira Weiny
gt; IBERROR("smp set portinfo failed"); > > printf("\nAfter PortInfo set:\n"); > -- > To unsubscribe from this list: send the line "unsubscribe linux-rdma" in > the body of a message to majord...@vger.kernel.org > More majordomo i

[TRIVIAL][PATCH] opensm: clean up error message, function name is printed by logging code

2013-01-15 Thread Ira Weiny
Signed-off-by: Ira Weiny --- opensm/osm_req.c |3 +-- 1 files changed, 1 insertions(+), 2 deletions(-) diff --git a/opensm/osm_req.c b/opensm/osm_req.c index 1c3c172..0b48a54 100644 --- a/opensm/osm_req.c +++ b/opensm/osm_req.c @@ -104,8 +104,7 @@ static ib_net64_t req_determine_mkey(IN

[PATCH 1/2] infiniband-diags: libibnetdisc add find node by lid

2013-02-02 Thread Ira Weiny
NOTE: this change adds a glib requirement to the package. Signed-off-by: Ira Weiny --- configure.in|7 infiniband-diags.spec.in|4 +- libibnetdisc/Makefile.am|4 +- libibnetdisc/include/infiniband

[PATCH 2/2] infiniband-diags: add dump_fts tool

2013-02-02 Thread Ira Weiny
>From 2708d0965e03de1f9ae722e38c8b72a808718a0d Mon Sep 17 00:00:00 2001 From: Ira Weiny Date: Fri, 1 Feb 2013 18:13:59 -0500 Subject: [PATCH 2/2] infiniband-diags: add dump_fts tool dump_fts adds a faster version of the functionality of dump_[l|m]fts.sh. This code is based off of the ibro

Re: [PATCH 1/2] infiniband-diags: libibnetdisc add find node by lid

2013-02-03 Thread Ira Weiny
On Sun, 03 Feb 2013 10:36:19 -0500 Hal Rosenstock wrote: > On 2/2/2013 7:36 PM, Ira Weiny wrote: > > > > > > NOTE: this change adds a glib requirement to the package. > > Would you elaborate on the direction here ? I would like to remove the OpenSM complib packag

Re: [PATCH 2/2] infiniband-diags: add dump_fts tool

2013-02-03 Thread Ira Weiny
On Sun, 03 Feb 2013 10:46:41 -0500 Hal Rosenstock wrote: > On 2/2/2013 7:36 PM, Ira Weiny wrote: > >>From 2708d0965e03de1f9ae722e38c8b72a808718a0d Mon Sep 17 00:00:00 2001 > > From: Ira Weiny > > Date: Fri, 1 Feb 2013 18:13:59 -0500 > > Subject: [PATCH 2/2] infin

Re: [PATCH 1/2] infiniband-diags: libibnetdisc add find node by lid

2013-02-03 Thread Ira Weiny
On Sun, 3 Feb 2013 13:20:25 -0800 Ira Weiny wrote: > On Sun, 03 Feb 2013 10:36:19 -0500 > Hal Rosenstock wrote: > > > On 2/2/2013 7:36 PM, Ira Weiny wrote: > > > > > > [snip] > > > + > > > + /* We add the port for all "lmc"

[PATCH] infiniband-diags: ibroute: add MAD status to query failure message

2013-02-03 Thread Ira Weiny
Signed-off-by: Ira Weiny --- src/ibroute.c | 22 -- 1 files changed, 16 insertions(+), 6 deletions(-) diff --git a/src/ibroute.c b/src/ibroute.c index a0c7de8..df48382 100644 --- a/src/ibroute.c +++ b/src/ibroute.c @@ -210,15 +210,20 @@ char *dump_multicast_tables

[PATCH V2 1/3] infiniband-diags: libibnetdisc add find node by lid

2013-02-03 Thread Ira Weiny
NOTE: this change adds a glib requirement to the package. Changes since V1: Use GINT_TO_POINTER rather than allocating keys Signed-off-by: Ira Weiny --- configure.in|7 infiniband-diags.spec.in|4 +- libibnetdisc

[PATCH V2 2/3] infiniband-diags: add dump_fts tool

2013-02-03 Thread Ira Weiny
ts > /dev/null dump tables: linear forwarding table get failed real0m8.121s user0m3.032s sys 0m3.342s Changes from V1: Add status and query information to error messages. Add man page files which were missed in the first patch Signed-off-by: Ira Weiny ---

[PATCH 3/3] infiniband-diags: deprecate dump_[m|l]fts.sh scripts

2013-02-03 Thread Ira Weiny
Signed-off-by: Ira Weiny --- Makefile.am|6 +- configure.in |4 +- doc/man/dump_lfts.8.in | 177 doc/man/dump_mfts.8.in | 170 -- doc/rst/dump_lfts.8.in.rst

[PATCH V3 1/3] infiniband-diags: libibnetdisc add find node by lid

2013-02-06 Thread Ira Weiny
fabrics Signed-off-by: Ira Weiny --- configure.in|7 ++ infiniband-diags.spec.in|4 +- libibnetdisc/Makefile.am|4 +- libibnetdisc/include/infiniband/ibnetdisc.h |3 + libibnetdisc/libibnetdisc.ver

[PATCH V3 2/3] infiniband-diags: add dump_fts tool

2013-02-06 Thread Ira Weiny
l status and address information on error messages. clean up error handling when fabric scan fails. Signed-off-by: Ira Weiny --- Makefile.am |7 +- configure.in |1 + doc/man/dump_fts.8.in | 236 ++ doc/rst/dump_fts.8.i

[PATCH 3/3] infiniband-diags: deprecate dump_[m|l]fts.sh scripts

2013-02-06 Thread Ira Weiny
Signed-off-by: Ira Weiny --- Makefile.am|6 +- configure.in |4 +- doc/man/dump_lfts.8.in | 177 doc/man/dump_mfts.8.in | 170 -- doc/rst/dump_lfts.8.in.rst

Re: OFA Management maintainership

2013-02-07 Thread Ira Weiny
working > time. I still continue to work on OpenSM and will continue to contribute to > the community in the future. > > --Alex > -- > To unsubscribe from this list: send the line "unsubscribe linux-rdma" in > the body of a message to majord...@vger.kernel.org >

Re: [PATCH] libibmad: Add ib_resolve_noderecord_via API

2013-02-12 Thread Ira Weiny
; + return -1; > + > + if (ib_resolve_smlid_via(&sm_id, MAD_DEF_TIMEOUT_MS, srcport) < 0) > + return -1; > + > + return ib_node_query_via(srcport, guid, &sm_id, buf); > +} > + > + > > -- > To unsubscribe from this list: send the line "unsubscri

Re: [PATCH] libibmad: Add ib_resolve_noderecord_via API

2013-02-12 Thread Ira Weiny
On Tue, 12 Feb 2013 15:36:11 -0800 Sarat Kakarla wrote: > > > On 2/12/2013 3:28 PM, Ira Weiny wrote: > > On Tue, 12 Feb 2013 12:39:44 -0800 > > Sarat Kakarla wrote: > > > >> Add ib_resolve_noderecord_via wrapper similar to ib_resolve_portid_str_via > &

Subject: [PATCH] opensm/perfmgr: update node name when Node Description is received from node

2013-02-13 Thread Ira Weiny
Signed-off-by: Ira Weiny --- include/opensm/osm_perfmgr.h|2 ++ include/opensm/osm_perfmgr_db.h |3 +++ opensm/osm_node_desc_rcv.c |7 +++ opensm/osm_perfmgr.c|6 ++ opensm/osm_perfmgr_db.c | 13 + 5 files changed, 31

[PATCH] opensm/perfmgr: mark/report time of last counter update

2013-02-13 Thread Ira Weiny
Signed-off-by: Ira Weiny --- opensm/osm_perfmgr_db.c | 43 +++ 1 files changed, 35 insertions(+), 8 deletions(-) diff --git a/opensm/osm_perfmgr_db.c b/opensm/osm_perfmgr_db.c index 238081b..ab6c978 100644 --- a/opensm/osm_perfmgr_db.c +++ b/opensm

[PATCH] opensm/perfmgr: skip data counters when only printing errors

2013-02-13 Thread Ira Weiny
Signed-off-by: Ira Weiny --- opensm/osm_perfmgr_db.c |3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --git a/opensm/osm_perfmgr_db.c b/opensm/osm_perfmgr_db.c index ab6c978..116bb07 100644 --- a/opensm/osm_perfmgr_db.c +++ b/opensm/osm_perfmgr_db.c @@ -869,6 +869,9 @@ static

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

2013-02-13 Thread Ira Weiny
with the sysfs files. Jon I would accept a patch if you would like to redo this one but I am going to reject the patch as it stands. Thanks, Ira > > Thanks, > -Garrett > -- > To unsubscribe from this list: send the line "unsubscribe linux-rdma" in > the body of a

Re: [PATCH] libibmad: Add NodeRecord query API

2013-02-13 Thread Ira Weiny
> + sa.trid = mad_trid(); > + > + memset(buf, 0, IB_SA_NR_RECSZ); > + > + mad_encode_field(buf, IB_SA_NR_PORT_GUID_F, &guid); > + > + p = sa_rpc_call(srcport, buf, sm_id, &sa, 0); > + if (!p) { > + IBWARN("sa call node_query f

Re: [PATCH] infiniband-diags/saquery.c: Add support for ServiceID in PathRecord querying

2013-02-13 Thread Ira Weiny
ULL, "Join state (MCMemberRecord)"}, > {"proxy_join", 'X', 1, NULL, "Proxy join (MCMemberRecord)"}, > {0} > -- > To unsubscribe from this list: send the line "unsubscribe linux-rdma" in > the body of a me

Re: [PATCH][TRIVIAL] infiniband-diags/saquery.c: Display NodeRecord LID in decimal

2013-02-13 Thread Ira Weiny
.%u\n" > "\t\treserved0x%X\n" > "\t\tbase_version0x%X\n" > "\t\tclass_version...0x%X\n" > -- > To unsubscribe from this list: send the line "unsubscribe linux-rdma" in > th

Re: [PATCH] libibmad: Add NodeRecord query API

2013-02-13 Thread Ira Weiny
On Wed, 13 Feb 2013 18:16:18 -0500 Hal Rosenstock wrote: > On 2/13/2013 5:21 PM, Ira Weiny wrote: > > Due to the fact there is no corresponding patch to the diags I assume this > > is something you want to support other uses of libibmad? > > It's in out of tree apps.

Re: [PATCH] infiniband-diags/saquery.c: Add support for ServiceID in PathRecord querying

2013-02-13 Thread Ira Weiny
On Wed, 13 Feb 2013 18:14:31 -0500 Hal Rosenstock wrote: > On 2/13/2013 5:23 PM, Ira Weiny wrote: > > We should update the man page for saquery to reflect this change in the > > same patch. > > The man page is far behind the help in saquery so I didn't think i

Re: [PATCHv2] libibmad: Add NodeRecord query API

2013-02-14 Thread Ira Weiny
B_SA_NR_PORT_GUID_F, &guid); > + > + p = sa_rpc_call(srcport, buf, sm_id, &sa, 0); > + if (!p) { > + IBWARN("sa call node_query failed"); > + return -1; > + } > + > + return 0; > +} > -- > To unsubscribe fr

Re: [PATCHv2] infiniband-diags/saquery: Add support for ServiceID in PathRecord querying

2013-02-14 Thread Ira Weiny
gt; --- a/doc/rst/saquery.8.in.rst > +++ b/doc/rst/saquery.8.in.rst > @@ -135,6 +135,8 @@ OPTIONS > > **--proxy_join** Proxy join (MCMemberRecord) > > +**--service_id** ServiceID (PathRecord) > + > Supported query names (and aliases): > > :: > @@ -205,4 +

Re: [PATCH] libibmad: Add support for PortSamplesResult

2013-02-15 Thread Ira Weiny
b/src/libibmad.map > index a4d4418..a1988be 100644 > --- a/src/libibmad.map > +++ b/src/libibmad.map > @@ -35,6 +35,7 @@ IBMAD_1.3 { > mad_dump_portcapmask; > mad_dump_portinfo; > mad_dump_portsamples_control; > + ma

[PATCH] libibumad: add string functions for various defines

2013-02-18 Thread Ira Weiny
turn string representations of enums + +.SH "SYNOPSIS" +.nf +.B #include +.sp +.BI "char * umad_class_str(uint8_t mgmt_class)" +.BI "char * umad_method_str(uint8_t method)" +.BI "char * umad_mad_status_str(be16_t status, char * buf, size_t len)" + +.B "

Re: [PATCH] libibumad: add string functions for various defines

2013-02-19 Thread Ira Weiny
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 -- Ira Weiny Member of Technical Staff Lawrence Livermore National Lab 925-423-8008

Re: Subject: [PATCH] opensm/perfmgr: update node name when Node Description is received from node

2013-02-19 Thread Ira Weiny
On Tue, 19 Feb 2013 15:37:01 -0500 Hal Rosenstock wrote: > On 2/13/2013 4:50 PM, Ira Weiny wrote: > > > > > > Signed-off-by: Ira Weiny > > --- > > include/opensm/osm_perfmgr.h|2 ++ > > include/opensm/osm_perfmgr_db.h |3 +++ >

[PATCH V2] opensm/perfmgr: update node name when Node Description is received from node

2013-02-19 Thread Ira Weiny
Changes since V1: Fix use of parens nit from Hal Signed-off-by: Ira Weiny --- include/opensm/osm_perfmgr.h|2 ++ include/opensm/osm_perfmgr_db.h |3 +++ opensm/osm_node_desc_rcv.c |7 +++ opensm/osm_perfmgr.c|6 ++ opensm/osm_perfmgr_db.c

Re: [PATCH 1/7] libibumad: Provide MAD definitions with libibumad

2013-02-20 Thread Ira Weiny
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 -- Ira Weiny Member of Technical Staff Lawrence Livermore National Lab 925-423-8008 wei

Re: [PATCH]: infiniband-diags/ibqueryerrors.c: obtain destination sl before perf query

2013-02-20 Thread Ira Weiny
t; + self_lid = portid.lid; > } > > if (load_cache_file) { > @@ -996,12 +1071,17 @@ int main(int argc, char **argv) > > port = ibnd_find_port_guid(fabric, port_guid); > if (port) { > + if(path_record_q

Re: [PATCH] libibumad: add string functions for various defines

2013-02-21 Thread Ira Weiny
ot;); > > + case UMAD_SMP_ATTR_LED_INFO: > > + return ("LedInfo"); > > + case UMAD_SMP_ATTR_LINK_SPD_WIDTH_TABLE: > > + return ("LinkSpeedWidthPairsTable"); > > add: > defau

Re: [PATCH] infiniband-diags/saquery.c: switchinfo support added

2013-02-21 Thread Ira Weiny
) >{"x", 'x', 0, NULL, "get LinkRecord info"}, >{"c", 'c', 0, NULL, "get the SA's class port info"}, >{"S", 'S', 0, NULL, "get ServiceRecord info"}, > + {"W", 'W'

Re: [PATCH] infiniband-diags/saquery.c: switchinfo support added

2013-02-21 Thread Ira Weiny
On Thu, 21 Feb 2013 10:09:00 -0800 Ira Weiny wrote: [snip] > > + ib_net16_t swir_lid; > > We don't need this field or lid option. Other records simply have an > optional parameter to specify the lid. > > The usage should be: > > SwitchRecord [lid] >

Re: [PATCH] libibumad/umad.c: In resolve_ca_port, skip ethernet link layer ports

2013-02-21 Thread Ira Weiny
- > 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 -- Ira Weiny Member of Technical Staff Lawrence Livermore National Lab 925-423-8008 wei...@llnl.gov -- 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

[PATCH 0/06] opensm/perfmgr: Add support for PortCountersExtended when supported

2013-02-21 Thread Ira Weiny
This patch series adds support to the Performance Manager for the PortCountersExtended attribute when a port supports it as indicated by ClassPortInfo. Ira -- Ira Weiny Member of Technical Staff Lawrence Livermore National Lab 925-423-8008 wei...@llnl.gov -- To unsubscribe from this list

[PATCH 01/06] opensm/perfmgr: issue ClassPortInfo as first query to each port.

2013-02-21 Thread Ira Weiny
Signed-off-by: Ira Weiny --- include/opensm/osm_perfmgr.h |4 + opensm/osm_perfmgr.c | 224 +- 2 files changed, 183 insertions(+), 45 deletions(-) diff --git a/include/opensm/osm_perfmgr.h b/include/opensm/osm_perfmgr.h index 26b1ae6

[PATCH 02/06] opensm/perfmgr: clean up: break out redirect processing from pc_recv_process

2013-02-21 Thread Ira Weiny
Signed-off-by: Ira Weiny --- opensm/osm_perfmgr.c | 200 -- 1 files changed, 96 insertions(+), 104 deletions(-) diff --git a/opensm/osm_perfmgr.c b/opensm/osm_perfmgr.c index c7f..69b3d77 100644 --- a/opensm/osm_perfmgr.c +++ b/opensm

[PATCH 03/06] opensm/perfmgr: Issue PortCountersExtended query when supported

2013-02-21 Thread Ira Weiny
Signed-off-by: Ira Weiny --- include/opensm/osm_perfmgr_db.h |2 +- opensm/osm_perfmgr.c| 311 --- opensm/osm_perfmgr_db.c |2 +- 3 files changed, 258 insertions(+), 57 deletions(-) diff --git a/include/opensm/osm_perfmgr_db.h b

[PATCH 04/06] opensm/perfmgr; add support for PortCountersExtended NOIETF

2013-02-21 Thread Ira Weiny
Signed-off-by: Ira Weiny --- include/opensm/osm_perfmgr_db.h |6 +++- opensm/osm_perfmgr.c| 35 +--- opensm/osm_perfmgr_db.c | 47 +++--- 3 files changed, 58 insertions(+), 30 deletions(-) diff --git a/include

[PATCH 05/06] opensm/perfmgr: don't clear data counters in PortCounters when ExtendedPortCounters is supported

2013-02-21 Thread Ira Weiny
counters when clearing. Signed-off-by: Ira Weiny --- opensm/osm_perfmgr.c | 88 ++ 1 files changed, 74 insertions(+), 14 deletions(-) diff --git a/opensm/osm_perfmgr.c b/opensm/osm_perfmgr.c index 65886f7..eb0c4f9 100644 --- a/opensm/osm_perfmgr.c

[PATCH 06/06] opensm/perfmgr: fix access to shared sweep_state variable

2013-02-21 Thread Ira Weiny
Signed-off-by: Ira Weiny --- include/opensm/osm_perfmgr.h |1 + opensm/osm_perfmgr.c | 23 +-- 2 files changed, 22 insertions(+), 2 deletions(-) diff --git a/include/opensm/osm_perfmgr.h b/include/opensm/osm_perfmgr.h index 3fa42d5..947a3db 100644 --- a

[PATCH] opensm/libibvendor: osm_vendor_get_all_port_attr include sm_sl value in port attribute struct

2013-02-21 Thread Ira Weiny
Signed-off-by: Ira Weiny --- libvendor/osm_vendor_ibumad.c |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/libvendor/osm_vendor_ibumad.c b/libvendor/osm_vendor_ibumad.c index c33ad2e..bcaf238 100644 --- a/libvendor/osm_vendor_ibumad.c +++ b/libvendor

[PATCH V2] libibumad: add string functions for various enums

2013-02-21 Thread Ira Weiny
|sa|cm]_attr_str functions Signed-off-by: Ira Weiny --- Makefile.am | 12 +- include/infiniband/umad_sm.h| 31 ++-- include/infiniband/umad_str.h | 56 +++ include/infiniband/umad_types.h |4 +- man/umad_attribute_str.3|3 + man/umad_class_str.3

[PATCH] libibumad: update umad_[send|recv] man pages to document how rmpp is handled

2013-02-22 Thread Ira Weiny
Signed-off-by: Ira Weiny --- man/umad_recv.3 | 17 ++--- man/umad_send.3 |8 2 files changed, 22 insertions(+), 3 deletions(-) diff --git a/man/umad_recv.3 b/man/umad_recv.3 index e4d52fd..e233411 100644 --- a/man/umad_recv.3 +++ b/man/umad_recv.3 @@ -13,9 +13,18

[PATCH] libibumad: add string functions for various enums

2013-02-22 Thread Ira Weiny
|sa|cm]_attr_str functions Changes from V2: Added mgmt_class to umad_method_str call (with man page update) remove "inline" from man page Use UMAD_STATUS_x values in umad_mad_status_str decode. Signed-off-by: Ira Weiny --- Makefile.am | 12 +

Re: [PATCH] libibumad: add string functions for various enums

2013-02-22 Thread Ira Weiny
in unique values being returned.) Ira > -- Ira Weiny Member of Technical Staff Lawrence Livermore National Lab 925-423-8008 wei...@llnl.gov -- 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] libibumad: add string functions for various enums

2013-02-22 Thread Ira Weiny
On Fri, 22 Feb 2013 13:27:10 -0500 Hal Rosenstock wrote: > On 2/22/2013 12:13 PM, Ira Weiny wrote: [snip] > > +.BI "char * umad_class_str(uint8_t mgmt_class)" > > +.BI "char * umad_method_str(uint8_t mgmt_class, uint8_t method)" > > +.BI "char *

Re: [PATCH] libibumad: add string functions for various enums

2013-02-22 Thread Ira Weiny
gt; points. It is a huge PITA to use const broken libraries from code that > is const correct. You are correct. Added in V... :-( Ira > > Jason > -- > To unsubscribe from this list: send the line "unsubscribe linux-rdma" in > the body of a message to majord...@vger.k

Re: [PATCH] libibumad: add string functions for various enums

2013-02-22 Thread Ira Weiny
redirection required - bad version - success'? They are mutually exclusive now but because the spec defines separate fields it might not always be that way. That is what I meant when I said "status is not defined to be a strict enum". Ira > -- > To unsubscribe from this lis

Re: [PATCH] libibumad: add string functions for various enums

2013-02-22 Thread Ira Weiny
arises? > We could do this with separate calls for class status. Ira > - Sean > -- > 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/majord

[PATCH] libibumad: change UMAD_METHOD_RESP to UMAD_METHOD_RESP_MASK

2013-02-23 Thread Ira Weiny
Signed-off-by: Ira Weiny --- include/infiniband/umad_types.h |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/include/infiniband/umad_types.h b/include/infiniband/umad_types.h index 199edd1..48ae284 100644 --- a/include/infiniband/umad_types.h +++ b/include/infiniband

[PATCH] opensm/perfmgr: add failed port guid to error message

2013-02-23 Thread Ira Weiny
Signed-off-by: Ira Weiny --- opensm/osm_perfmgr.c |4 +++- 1 files changed, 3 insertions(+), 1 deletions(-) diff --git a/opensm/osm_perfmgr.c b/opensm/osm_perfmgr.c index 9bc1154..25033bd 100644 --- a/opensm/osm_perfmgr.c +++ b/opensm/osm_perfmgr.c @@ -831,7 +831,9 @@ void

Re: [PATCH] infiniband-diags/saquery.c: switchinfo support added

2013-02-24 Thread Ira Weiny
nks, Ira > > -----Original Message- > From: Ira Weiny > To: Ira Weiny > Cc: "Husam Kahalah" , linux-rdma@vger.kernel.org, > linux-ker...@vger.kernel.org > Date: Thu, 21 Feb 2013 10:33:44 -0800 > Subject: Re: [PATCH] infiniband-diags/saquery.c: switchinfo su

[PATCH] opensm: make osm_get_path_params public

2013-02-25 Thread Ira Weiny
Signed-off-by: Ira Weiny --- include/opensm/osm_sa.h | 21 + opensm/osm_sa_multipath_record.c | 10 -- opensm/osm_sa_path_record.c |9 - 3 files changed, 21 insertions(+), 19 deletions(-) diff --git a/include/opensm/osm_sa.h b/include

[PATCH] opensm: make osm_pr_rcv_get_end_points, osm_pr_rcv_process_pair, osm_pr_rcv_process_half public

2013-02-25 Thread Ira Weiny
Also change their name to from osm_pr_rcv_* to osm_pr_* Signed-off-by: Ira Weiny --- include/opensm/osm_sa.h | 25 + opensm/osm_sa_path_record.c | 22 +++--- 2 files changed, 36 insertions(+), 11 deletions(-) diff --git a/include/opensm/osm_sa.h

[PATCH] libibumad: add UMAD_SA_STATUS_PRI_SUGGESTED to SA status.

2013-02-25 Thread Ira Weiny
Per: A13.5.2 Subnet Administration Status Values (QoS) Signed-off-by: Ira Weiny --- include/infiniband/umad_sa.h |3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/include/infiniband/umad_sa.h b/include/infiniband/umad_sa.h index 5d6efe1..aaf2ac1 100644 --- a/include

infiniband-diags maintainership notice

2013-02-25 Thread Ira Weiny
ss to intel email. Thank you, Ira -- Ira Weiny Member of Technical Staff Lawrence Livermore National Lab 925-423-8008 wei...@llnl.gov -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majord...@vger.kernel.org More majordom

Re: [PATCH 01/06] opensm/perfmgr: issue ClassPortInfo as first query to each port.

2013-02-26 Thread Ira Weiny
On Tue, 26 Feb 2013 10:03:35 -0500 Hal Rosenstock wrote: > On 2/21/2013 4:33 PM, Ira Weiny wrote: > > > > So 2 round trips are now needed for first time ports now to determine > whether or not extended counters are supported. I don't see a better way > around this.

Re: [PATCH 02/06] opensm/perfmgr: clean up: break out redirect processing from pc_recv_process

2013-02-26 Thread Ira Weiny
On Tue, 26 Feb 2013 10:34:56 -0500 Hal Rosenstock wrote: > On 2/21/2013 4:33 PM, Ira Weiny wrote: > > > > > > Signed-off-by: Ira Weiny > > --- > > opensm/osm_perfmgr.c | 200 > > -- > > 1 f

Re: [PATCH 01/06] opensm/perfmgr: issue ClassPortInfo as first query to each port.

2013-02-26 Thread Ira Weiny
On Tue, 26 Feb 2013 14:48:26 -0500 Hal Rosenstock wrote: > On 2/26/2013 1:58 PM, Ira Weiny wrote: > > On Tue, 26 Feb 2013 10:03:35 -0500 > > Hal Rosenstock wrote: > > > >> On 2/21/2013 4:33 PM,

Re: [PATCH 01/06] opensm/perfmgr: issue ClassPortInfo as first query to each port.

2013-02-26 Thread Ira Weiny
On Tue, 26 Feb 2013 16:14:52 -0500 Hal Rosenstock wrote: > On 2/26/2013 4:07 PM, Ira Weiny wrote: > > Right, BSP0 doesn't have PM counters. So why did you ask if I had tried > > this with base SP0? > > I thought I answered that in the previous response: > "T

Re: [PATCH 02/06] opensm/perfmgr: clean up: break out redirect processing from pc_recv_process

2013-02-26 Thread Ira Weiny
On Tue, 26 Feb 2013 14:58:01 -0500 Hal Rosenstock wrote: > On 2/26/2013 2:34 PM, Ira Weiny wrote: > > On Tue, 26 Feb 2013 10:34:56 -0500 > > Hal Rosenstock wrote: > > > >> On 2/21/2013 4:33 PM, Ira Weiny wrote: > >>> > >>> > >>&g

[PATCH V2 01/06] opensm/perfmgr: issue ClassPortInfo as first query to each port.

2013-02-26 Thread Ira Weiny
Changes from V1: Fix spacing Signed-off-by: Ira Weiny --- include/opensm/osm_perfmgr.h |4 + opensm/osm_perfmgr.c | 224 +- 2 files changed, 183 insertions(+), 45 deletions(-) diff --git a/include/opensm/osm_perfmgr.h b/include

[PATCH V2 02/06] opensm/perfmgr: clean up: break out redirect processing from pc_recv_process

2013-02-26 Thread Ira Weiny
Changes from V1: Cleaned up processing when CPI redirect is invalid Signed-off-by: Ira Weiny --- opensm/osm_perfmgr.c | 199 -- 1 files changed, 95 insertions(+), 104 deletions(-) diff --git a/opensm/osm_perfmgr.c b/opensm/osm_perfmgr.c

Re: [PATCH] Add fix to _do_madrpc to set the errno after umad_recv.

2013-02-26 Thread Ira Weiny
TRID_F) != trid); > > status = umad_status(rcvbuf); > + errno = status; > if (!status) > return length; /* done */ > if (status == ENOMEM) -- Ira Weiny Member of Technical Staff Lawrence Livermore Nati

Re: [PATCHv2] infiniband-diags/ibqueryerrors.c: obtain destination sl before perf query

2013-02-26 Thread Ira Weiny
t; + } else { > + if(half_world_query) > + if(path_record_query(self_lid,0)) > + goto close_port; goto destroy_fabric since the fabric object is created at this point. > ibnd_iter_nodes(fabric, print_nod

[PATCH V3 02/06] opensm/perfmgr: clean up: break out redirect processing from pc_recv_process

2013-02-27 Thread Ira Weiny
set cpi_valid on all ports for a switch) Signed-off-by: Ira Weiny --- opensm/osm_perfmgr.c | 210 -- 1 files changed, 102 insertions(+), 108 deletions(-) diff --git a/opensm/osm_perfmgr.c b/opensm/osm_perfmgr.c index 79e5fce..d8a7c9e 100644 --- a

<    1   2   3   4   5   6   7   8   9   10   >