Re: [ofa-general] [PATCH 2/2 v4] opensm: Compression of multicast group according to pkey

2009-09-30 Thread Hal Rosenstock
On Tue, Sep 29, 2009 at 9:54 AM, Slava Strebkov sla...@voltaire.com wrote: Additional data structure added: 1. Map of all partition keys opened in the fabric. 2. Map of all multicast group boxes shared same pkey. MLID assignment for multicast groups works in a usual manner, allocating free

Re: [ofa-general] This list expires... tomorrow?

2009-09-30 Thread Hal Rosenstock
On Tue, Sep 29, 2009 at 8:40 PM, Jeff Becker jeffrey.c.bec...@nasa.gov wrote: Hal Rosenstock wrote: On Tue, Sep 29, 2009 at 3:06 PM, Jeff Becker jeffrey.c.bec...@nasa.gov wrote: Hi all. I propose the following plan to shutdown the general list: 1) unsubscribe all current subscribers 2

[ofa-general] [PATCH] opensm/osm_sa_lft_record.c: In lftr_rcv_new_lftr, handle osm_switch_get_lft_block failure

2009-09-30 Thread Hal Rosenstock
Signed-off-by: Hal Rosenstock hal.rosenst...@gmail.com --- diff --git a/opensm/opensm/osm_sa_lft_record.c b/opensm/opensm/osm_sa_lft_record.c index d092129..828b277 100644 --- a/opensm/opensm/osm_sa_lft_record.c +++ b/opensm/opensm/osm_sa_lft_record.c @@ -99,8 +99,12 @@ static ib_api_status_t

[ofa-general] [PATCHv2] opensm/osm_mesh.c: Add dump_mesh routine at OSM_LOG_DEBUG level

2009-09-29 Thread Hal Rosenstock
Signed-off-by: Hal Rosenstock hal.rosenst...@gmail.com --- Changes since v1: Use snprintf rather than sprintf Also, moved output of ] diff --git a/opensm/opensm/osm_mesh.c b/opensm/opensm/osm_mesh.c index 260e2f8..53f0f58 100644 --- a/opensm/opensm/osm_mesh.c +++ b/opensm/opensm/osm_mesh.c

Re: [ofa-general] This list expires... tomorrow?

2009-09-29 Thread Hal Rosenstock
On Tue, Sep 29, 2009 at 3:06 PM, Jeff Becker jeffrey.c.bec...@nasa.gov wrote: Hi all. I propose the following plan to shutdown the general list: 1) unsubscribe all current subscribers 2) set the list to discard any incoming messages with an auto-discard message that points you to

Re: [ofa-general] This list expires... tomorrow?

2009-09-29 Thread Hal Rosenstock
On Tue, Sep 29, 2009 at 5:20 PM, Roland Dreier rdre...@cisco.com wrote: It's probably just me but I'm not ready yet. I haven't been able to post a patch to linux-rdma yet :-( What is going wrong when you try? It disappears into the ether without any response. I can see it getting a

Re: [ofa-general] Re: [PATCH] ibsim/sim_cmd.c: Only relink port if remote port is currently linked

2009-09-29 Thread Hal Rosenstock
Hi Sasha, On Tue, Sep 29, 2009 at 5:53 PM, Sasha Khapyorsky sas...@voltaire.com wrote: Hi Hal, On 15:34 Thu 24 Sep     , Hal Rosenstock wrote: When multiple switches are unlinked and then a switch is relinked, it should behave like a cable pull or power down of switch so it depends

Re: [ofa-general] Re: [PATCH] ibsim/sim_cmd.c: Only relink port if remote port is currently linked

2009-09-29 Thread Hal Rosenstock
Hi again Sasha, In my previous post, I missed answering some of your (implied) questions. On Tue, Sep 29, 2009 at 5:53 PM, Sasha Khapyorsky sas...@voltaire.com wrote: Hi Hal, On 15:34 Thu 24 Sep     , Hal Rosenstock wrote: When multiple switches are unlinked and then a switch is relinked

Re: [ofa-general] This list expires... tomorrow?

2009-09-29 Thread Hal Rosenstock
On Tue, Sep 29, 2009 at 3:06 PM, Jeff Becker jeffrey.c.bec...@nasa.gov wrote: Hi all. I propose the following plan to shutdown the general list: 1) unsubscribe all current subscribers 2) set the list to discard any incoming messages with an auto-discard message that points you to

[ofa-general] [PATCH] infiniband-diags/perfquery.c: Fix extended counter reset mask

2009-09-26 Thread Hal Rosenstock
to not have any bits on for reserved components Signed-off-by: Hal Rosenstock hal.rosenst...@gmail.com --- diff --git a/infiniband-diags/src/perfquery.c b/infiniband-diags/src/perfquery.c index d70af9e..5d4046b 100644 --- a/infiniband-diags/src/perfquery.c +++ b/infiniband-diags/src/perfquery.c

Re: [ofa-general] [PATCH] infiniband-diags/src/ibqueryerrors: Add clear errors and counters options

2009-09-25 Thread Hal Rosenstock
Ira, See one minor comment below: On Fri, Sep 25, 2009 at 2:50 AM, Ira Weiny wei...@llnl.gov wrote: Sasha, This applies after infiniband-diags/src/ibqueryerrors: move --all option and replace it with --switch, --ca, --router From: Ira Weiny wei...@llnl.gov Date: Thu, 24 Sep 2009

[ofa-general] Re: [PATCH] opensm/osm_ucast_lash: fix use after free bug

2009-09-25 Thread Hal Rosenstock
' pointers are cleared on lash_cleanup()). Signed-off-by: Sasha Khapyorsky sas...@voltaire.com Tested-by: Hal Rosenstock hal.rosenst...@gmail.com --- opensm/opensm/osm_ucast_lash.c |5 + 1 files changed, 1 insertions(+), 4 deletions(-) diff --git a/opensm/opensm/osm_ucast_lash.c b/opensm

[ofa-general] [PATCH] ibsim/sim_cmd.c: Only relink port if remote port is currently linked

2009-09-24 Thread Hal Rosenstock
-by: Hal Rosenstock hal.rosenst...@gmail.com --- diff --git a/ibsim/sim.h b/ibsim/sim.h index bf85875..52eb73b 100644 --- a/ibsim/sim.h +++ b/ibsim/sim.h @@ -210,6 +211,7 @@ struct Port { int remoteport; Node *previous_remotenode; int previous_remoteport; + int unlinked

[ofa-general] Re: osm_link_mgr.c:link_mgr_get_smsl question

2009-09-23 Thread Hal Rosenstock
On Tue, Sep 22, 2009 at 10:53 PM, Sasha Khapyorsky sas...@voltaire.comwrote: On 16:44 Tue 22 Sep , Hal Rosenstock wrote: Yeah, the port lid table will be OK but port's PortInfo won't (so base LID/LMC will be broken) for this scenario but it wouldn't affect this code in this way

[ofa-general] [PATCH] libibmad: Add support for PortXmitDiscardDetails

2009-09-23 Thread Hal Rosenstock
Also, some additional commentary changes to mad.h and fields.c Signed-off-by: Hal Rosenstock hal.rosenst...@gmail.com --- diff --git a/libibmad/include/infiniband/mad.h b/libibmad/include/infiniband/mad.h index 94b64cf..cfa9105 100644 --- a/libibmad/include/infiniband/mad.h +++ b/libibmad

[ofa-general] [PATCH] infiniband-diags/pergquery: Add support for optional PortXmitDiscardDetails counter

2009-09-23 Thread Hal Rosenstock
Signed-off-by: Hal Rosenstock hal.rosenst...@gmail.com --- diff --git a/infiniband-diags/man/perfquery.8 b/infiniband-diags/man/perfquery.8 index 2a80f30..4510e7d 100644 --- a/infiniband-diags/man/perfquery.8 +++ b/infiniband-diags/man/perfquery.8 @@ -1,4 +1,4 @@ -.TH PERFQUERY 8 March 10, 2009

[ofa-general] [PATCH] opensm/osm_link_mgr.c: In link_mgr_set_physp_pi, only call link_mgr_get_smsl when LID valid

2009-09-23 Thread Hal Rosenstock
Fix seg fault which occurs when get_osm_switch_from_port is called with NULL port (which in this case was caused by calling cl_ptr_vector_get on port LID table with LID 0) Signed-off-by: Hal Rosenstock hal.rosenst...@gmail.com --- diff --git a/opensm/opensm/osm_link_mgr.c b/opensm/opensm

[ofa-general] [PATCHv2] infiniband-diags/perfquery: Add support for optional PortXmitDiscardDetails counter

2009-09-23 Thread Hal Rosenstock
Signed-off-by: Hal Rosenstock hal.rosenst...@gmail.com --- Changes since v1: Fix typo in [PATCH] subject diff --git a/infiniband-diags/man/perfquery.8 b/infiniband-diags/man/perfquery.8 index 2a80f30..4510e7d 100644 --- a/infiniband-diags/man/perfquery.8 +++ b/infiniband-diags/man/perfquery.8

[ofa-general] Re: Possible process deadlock in RMPP flow

2009-09-23 Thread Hal Rosenstock
On Wed, Sep 23, 2009 at 12:08 PM, Sean Hefty sean.he...@intel.com wrote: ibnetdiscover D 80149b8d 0 26968 26544 (L-TLB) 8102c900bd88 0046 81037e8e 81037e8e02e8 8102c900bd78 000a 8102c5b50820 81038a929820 011837bf6105

[ofa-general] [PATCH] libibmad/dump.c: Fix typo

2009-09-22 Thread Hal Rosenstock
Signed-off-by: Hal Rosenstock hal.rosenst...@gmail.com --- diff --git a/libibmad/src/dump.c b/libibmad/src/dump.c index 1b287c0..5151882 100644 --- a/libibmad/src/dump.c +++ b/libibmad/src/dump.c @@ -523,7 +523,7 @@ void mad_dump_portcapmask(char *buf, int bufsz, void *val, int valsz

[ofa-general] [PATCH] opensm/osm_mesh.c: Add dump_mesh routine at OSM_LOG_DEBUG log level

2009-09-22 Thread Hal Rosenstock
Signed-off-by: Hal Rosenstock hal.rosenst...@gmail.com --- diff --git a/opensm/opensm/osm_mesh.c b/opensm/opensm/osm_mesh.c index 260e2f8..beb6bd7 100644 --- a/opensm/opensm/osm_mesh.c +++ b/opensm/opensm/osm_mesh.c @@ -1565,6 +1565,39 @@ err: return -1; } +static void dump_mesh(lash_t

[ofa-general] Re: osm_link_mgr.c:link_mgr_get_smsl question

2009-09-22 Thread Hal Rosenstock
On Tue, Sep 22, 2009 at 4:33 PM, Sasha Khapyorsky sas...@voltaire.comwrote: On 07:32 Thu 17 Sep , Hal Rosenstock wrote: Is that (lids in place) always the case ? I don't see immediately how it could be not. What about if the sets of PortInfo for LID fail. Set can fail

[ofa-general] [PATCH] infiniband-diags/ibportstate.c: Eliminate uninitialized variable compile warning

2009-09-22 Thread Hal Rosenstock
Signed-off-by: Hal Rosenstock hal.rosenst...@gmail.com --- diff --git a/infiniband-diags/src/ibportstate.c b/infiniband-diags/src/ibportstate.c index 6fb97a8..55e1dd5 100644 --- a/infiniband-diags/src/ibportstate.c +++ b/infiniband-diags/src/ibportstate.c @@ -208,7 +208,7 @@ int main(int argc

Re: [ofa-general] Re: [PATCH] osmtest: Add SA get PathRecord stress test

2009-09-21 Thread Hal Rosenstock
Hi Sasha, On Sun, Sep 20, 2009 at 6:20 AM, Sasha Khapyorsky sas...@voltaire.comwrote: snip... diff --git a/opensm/osmtest/osmtest.c b/opensm/osmtest/osmtest.c index 986a8d2..8357d90 100644 --- a/opensm/osmtest/osmtest.c +++ b/opensm/osmtest/osmtest.c snip... + + /* + * Do a

[ofa-general] [PATCHv2] osmtest: Add SA get PathRecord stress test

2009-09-21 Thread Hal Rosenstock
Signed-off-by: Hal Rosenstock hal.rosenst...@gmail.com --- Changes since v1: Removed unneeded mode parameter diff --git a/opensm/man/osmtest.8 b/opensm/man/osmtest.8 index fa0cd52..f0d6323 100644 --- a/opensm/man/osmtest.8 +++ b/opensm/man/osmtest.8 @@ -1,4 +1,4 @@ -.TH OSMTEST 8 August 11, 2008

[ofa-general] [PATCHv2] opensm/osm_perfmgr_db.c: Fix memory leak of db nodes

2009-09-21 Thread Hal Rosenstock
Signed-off-by: Hal Rosenstock hal.rosenst...@gmail.com --- Changes since v1: Fix use after free issue diff --git a/opensm/opensm/osm_perfmgr_db.c b/opensm/opensm/osm_perfmgr_db.c index e5dfc19..03f988d 100644 --- a/opensm/opensm/osm_perfmgr_db.c +++ b/opensm/opensm/osm_perfmgr_db.c @@ -49,6

Re: [ofa-general] Re: [PATCHv2] opensm/osm_mesh.c: Remove edges in lash matrix

2009-09-21 Thread Hal Rosenstock
Hi Sasha, On Sun, Aug 30, 2009 at 6:36 AM, Sasha Khapyorsky sas...@voltaire.comwrote: snip... @@ -878,6 +950,12 @@ static void make_geometry(lash_t *p_lash, int sw) n = s1-node-num_links; /* + * ignore chain fragments

[ofa-general] [PATCHv3] opensm/osm_mesh.c: Remove edges in lash matrix

2009-09-21 Thread Hal Rosenstock
. Signed-off-by: Robert Pearson rpear...@systemfabricworks.com Signed-off-by: Hal Rosenstock hal.rosenst...@gmail.com --- Changes since v2: In make_geometry, added comment on meaning of magic number 4 In seed_axes, made log level DEBUG and placed in osm_log_is_active clause Changes since v1: Replaced

Re: [ofa-general] Re: [PATCH] opensm: Add support for MulticastFDBTop

2009-09-21 Thread Hal Rosenstock
On Mon, Sep 21, 2009 at 8:59 AM, Sasha Khapyorsky sas...@voltaire.comwrote: On 10:31 Wed 02 Sep , Hal Rosenstock wrote: Add support for SwitchInfo:MulticastFDBTop Added by MgtWG errata #4505-4508 Also, per MgtWG RefID #4640, MulticastFDBTop value of 0xbfff means no entries

Re: [ofa-general] Re: [PATCH] osmtest: Add SA get PathRecord stress test

2009-09-21 Thread Hal Rosenstock
On Mon, Sep 21, 2009 at 10:28 AM, Sasha Khapyorsky sas...@voltaire.comwrote: On 09:20 Mon 21 Sep , Hal Rosenstock wrote: snip... + + /* + * Do a blocking query for the PathRecord. + */ + status = osmtest_get_path_rec_by_lid_pair(p_osmt, slid, dlid

[ofa-general] [PATCHv2] opensm: Add support for MulticastFDBTop

2009-09-21 Thread Hal Rosenstock
on max_block_in_use Signed-off-by: Hal Rosenstock hal.rosenst...@gmail.com --- Changes since v1: In mcast_mgr_set_mfttop, eliminated PortStateChange checking diff --git a/opensm/opensm/osm_mcast_mgr.c b/opensm/opensm/osm_mcast_mgr.c index c1d1916..0da0ef1 100644 --- a/opensm/opensm/osm_mcast_mgr.c +++ b

[ofa-general] [PATCHv3] opensm/osm_perfmgr_db.c: Fix memory leak of db nodes

2009-09-21 Thread Hal Rosenstock
Signed-off-by: Hal Rosenstock hal.rosenst...@gmail.com --- Changes since v2: Eliminated node variable Changes since v1: Fix use after free issue diff --git a/opensm/opensm/osm_perfmgr_db.c b/opensm/opensm/osm_perfmgr_db.c index e5dfc19..5321c59 100644 --- a/opensm/opensm/osm_perfmgr_db.c +++ b

Re: [ofa-general] Re: [PATCH] osmtest: Add SA get PathRecord stress test

2009-09-21 Thread Hal Rosenstock
On Mon, Sep 21, 2009 at 10:57 AM, Sasha Khapyorsky sas...@voltaire.comwrote: snip... How this ping test's timeline looks? (1) client sends one request, (2) it travels to a server, (3) server processes it and replies, (4) the response travels to client, (5) client gets it and continue from

Re: [ofa-general] Re: [PATCH] opensm: Add support for MulticastFDBTop

2009-09-21 Thread Hal Rosenstock
On Mon, Sep 21, 2009 at 11:09 AM, Sasha Khapyorsky sas...@voltaire.comwrote: On 10:20 Mon 21 Sep , Hal Rosenstock wrote: + + if (p_physp-port_info.capability_mask IB_PORT_CAP_HAS_MCAST_FDB_TOP) { BTW any reason why this capability bit if placed in PortInfo

Re: [ofa-general] Re: [PATCH] opensm: Add support for MulticastFDBTop

2009-09-21 Thread Hal Rosenstock
On Mon, Sep 21, 2009 at 11:48 AM, Sasha Khapyorsky sas...@voltaire.comwrote: On 11:22 Mon 21 Sep , Hal Rosenstock wrote: I'll try. Thanks. I removed this in the latest patch version. Ok. Let's wait with this patch up to capability mask clarification/resolution. Other than

[ofa-general] [PATCHv3] opensm: Add support for MulticastFDBTop

2009-09-21 Thread Hal Rosenstock
on max_block_in_use Signed-off-by: Hal Rosenstock hal.rosenst...@gmail.com --- Changes since v2: In mcast_mgr_set_mfttop, reverse sense of mft top test so can remove indentation of code doing update Changes since v1: In mcast_mgr_set_mfttop, eliminated PortStateChange checking diff --git a/opensm/opensm

[ofa-general] [PATCH] opensm/osm_perfmgr_db.c: Fix memory leak of db nodes

2009-09-18 Thread Hal Rosenstock
Signed-off-by: Hal Rosenstock hal.rosenst...@gmail.com --- diff --git a/opensm/opensm/osm_perfmgr_db.c b/opensm/opensm/osm_perfmgr_db.c index e5dfc19..329743a 100644 --- a/opensm/opensm/osm_perfmgr_db.c +++ b/opensm/opensm/osm_perfmgr_db.c @@ -49,6 +49,8 @@ #include opensm/osm_perfmgr.h

[ofa-general] [PATCH] ibsim/sim_cmd.c: Cosmetic change to error message

2009-09-18 Thread Hal Rosenstock
Signed-off-by: Hal Rosenstock hal.rosenst...@gmail.com --- diff --git a/ibsim/sim_cmd.c b/ibsim/sim_cmd.c index cb6e639..6d3a893 100644 --- a/ibsim/sim_cmd.c +++ b/ibsim/sim_cmd.c @@ -295,7 +295,7 @@ static int do_seterror(FILE * f, char *line) orig = strsep(s

[ofa-general] Re: osm_link_mgr.c:link_mgr_get_smsl question

2009-09-17 Thread Hal Rosenstock
On Sun, Aug 30, 2009 at 8:00 AM, Sasha Khapyorsky sas...@voltaire.comwrote: On 07:32 Sun 30 Aug , Hal Rosenstock wrote: osm_link_mgr.c:link_mgr_get_smsl has the following: /* Find osm_port of the source = p_physp */ slid = osm_physp_get_base_lid(p_physp

Re: [ofw] Re: [ofa-general] [RFC] 3/5: IB ACM: libibacm

2009-09-17 Thread Hal Rosenstock
On Thu, Sep 17, 2009 at 4:20 PM, Ira Weiny wei...@llnl.gov wrote: On Thu, 17 Sep 2009 10:35:39 -0700 Sean Hefty sean.he...@intel.com wrote: #define IB_PATH_RECORD_REVERSIBLE 0x80 struct ib_path_record { uint64_tservice_id; union ibv_gid dgid; union

Re: [ofw] Re: [ofa-general] [RFC] 3/5: IB ACM: libibacm

2009-09-17 Thread Hal Rosenstock
On Thu, Sep 17, 2009 at 5:40 PM, Ira Weiny wei...@llnl.gov wrote: On Thu, 17 Sep 2009 17:41:30 -0400 Hal Rosenstock hal.rosenst...@gmail.com wrote: On Thu, Sep 17, 2009 at 4:20 PM, Ira Weiny wei...@llnl.gov wrote: On Thu, 17 Sep 2009 10:35:39 -0700 Sean Hefty sean.he...@intel.com

Re: [ofw] Re: [ofa-general] [RFC] 3/5: IB ACM: libibacm

2009-09-17 Thread Hal Rosenstock
On Thu, Sep 17, 2009 at 5:49 PM, Sean Hefty sean.he...@intel.com wrote: I'm not sure this is a good idea. ibutils (ibis and ibmgtsim) wants ib_types.h but does not want libibumad. Well, libibumad is pretty useless without some network structure definitions. ib_types.h is more akin to what

Re: [ofw] Re: [ofa-general] [RFC] 3/5: IB ACM: libibacm

2009-09-17 Thread Hal Rosenstock
On Thu, Sep 17, 2009 at 6:11 PM, Ira Weiny wei...@llnl.gov wrote: On Thu, 17 Sep 2009 14:49:50 -0700 Sean Hefty sean.he...@intel.com wrote: I'm not sure this is a good idea. ibutils (ibis and ibmgtsim) wants ib_types.h but does not want libibumad. Well, libibumad is pretty useless

Re: [ofw] Re: [ofa-general] [RFC] 3/5: IB ACM: libibacm

2009-09-17 Thread Hal Rosenstock
On Thu, Sep 17, 2009 at 7:16 PM, Hal Rosenstock hal.rosenst...@gmail.comwrote: On Thu, Sep 17, 2009 at 6:11 PM, Ira Weiny wei...@llnl.gov wrote: On Thu, 17 Sep 2009 14:49:50 -0700 Sean Hefty sean.he...@intel.com wrote: I'm not sure this is a good idea. ibutils (ibis and ibmgtsim) wants

Re: [ofw] Re: [ofa-general] [RFC] 3/5: IB ACM: libibacm

2009-09-17 Thread Hal Rosenstock
On Thu, Sep 17, 2009 at 7:12 PM, Hal Rosenstock hal.rosenst...@gmail.comwrote: On Thu, Sep 17, 2009 at 5:49 PM, Sean Hefty sean.he...@intel.com wrote: I'm not sure this is a good idea. ibutils (ibis and ibmgtsim) wants ib_types.h but does not want libibumad. Well, libibumad is pretty

Re: [ofa-general] [PATCH] opensm: use mgrp pointer in port mcm_info

2009-09-16 Thread Hal Rosenstock
On Tue, Sep 15, 2009 at 7:26 AM, Hal Rosenstock hal.rosenst...@gmail.comwrote: On Tue, Sep 15, 2009 at 6:08 AM, Sasha Khapyorsky sas...@voltaire.comwrote: On 08:45 Mon 14 Sep , Hal Rosenstock wrote: Does this mean consolidate_ipv6_snm_req does not work now ? No, it doesn't

Re: [ofa-general] [PATCH] opensm: use mgrp pointer in port mcm_info

2009-09-15 Thread Hal Rosenstock
On Tue, Sep 15, 2009 at 6:08 AM, Sasha Khapyorsky sas...@voltaire.comwrote: On 08:45 Mon 14 Sep , Hal Rosenstock wrote: Does this mean consolidate_ipv6_snm_req does not work now ? No, it doesn't. As you may remember 'consolidate_ipv6_snm_req' workaround does nothing with MGIDs to MLID

[ofa-general] [PATCH] perftest: Remove unneeded executable permissions

2009-09-15 Thread Hal Rosenstock
Signed-off-by: Hal Rosenstock hal.rosenst...@gmail.com --- diff --git a/COPYING b/COPYING old mode 100755 new mode 100644 diff --git a/Makefile b/Makefile old mode 100755 new mode 100644 diff --git a/README b/README old mode 100755 new mode 100644 diff --git a/clock_test.c b/clock_test.c old mode

[ofa-general] [PATCH] perftest: Make rdma_lat, rdma_bw, and clock_test executable names rdma neutral

2009-09-15 Thread Hal Rosenstock
Since rdma_lat and rdma_bw use RDMA CM, they can be used with both IB and iWARP so make their executable names neutral (by removing ib_) IB only tests only require linking with libibverbs Also, spec file change for executable name changes Signed-off-by: Hal Rosenstock hal.rosenst...@gmail.com

[ofa-general] [PATCH] opensm/opensm.8.in: Indicate default rule for Default partition

2009-09-14 Thread Hal Rosenstock
Also, similar change to doc/partition-config.txt Signed-off-by: Hal Rosenstock hal.rosenst...@gmail.com --- diff --git a/opensm/doc/partition-config.txt b/opensm/doc/partition-config.txt index f855268..ae3d6f6 100644 --- a/opensm/doc/partition-config.txt +++ b/opensm/doc/partition-config.txt

Re: [ofa-general] [PATCH] opensm: use mgrp pointer in port mcm_info

2009-09-14 Thread Hal Rosenstock
On Sun, Sep 6, 2009 at 11:49 AM, Sasha Khapyorsky sas...@voltaire.comwrote: Port needs to access multicast groups where it is joined to. Now it is implemented by keeping list of list of mcm_info elements where MLID of each multicast group is stored. Obviously this assumes single MGID to MLID

Re: [ofa-general] [PATCH] infiniband-diags/scripts: Add ibcheckroutes to scripts

2009-09-14 Thread Hal Rosenstock
On Sun, Sep 13, 2009 at 3:55 AM, Doron Shoham dor...@voltaire.com wrote: Hal Rosenstock wrote: On Thu, Sep 10, 2009 at 7:56 AM, Doron Shoham dor...@voltaire.com mailto:dor...@voltaire.com wrote: ibcheckroutes validates route between all hosts in the fabric. This script

Re: [ofa-general] Re: [PATCH v2] infiniband-diags/scripts: Add 'ibcheckspeed' and 'ibcheckportspeed' to scripts

2009-09-14 Thread Hal Rosenstock
Switch SFS7000D) 0x0005ad092106 8 (Cisco Switch SFS7000D) Ira On Thu, 10 Sep 2009 09:23:35 -0400 Hal Rosenstock hal.rosenst...@gmail.com wrote: On Thu, Sep 10, 2009 at 9:02 AM, Keshetti Mahesh keshetti.mah...@gmail.comwrote: Added 'ibcheckspeed' and 'ibcheckportspeed

Re: [ofa-general] [PATCH] infiniband-diags/scripts: Add ibcheckroutes to scripts

2009-09-14 Thread Hal Rosenstock
On Thu, Sep 10, 2009 at 7:56 AM, Doron Shoham dor...@voltaire.com wrote: ibcheckroutes validates route between all hosts in the fabric. This script finds all leaf switches (switches that are connected to HCAs) and runs ibtracert between them. When using various routing algorithms (e.g.

Re: [ofa-general] [PATCH] infiniband-diags/scripts: Add ibcheckroutes to scripts

2009-09-14 Thread Hal Rosenstock
On Mon, Sep 14, 2009 at 11:05 AM, Eli Dorfman (Voltaire) dorfman@gmail.com wrote: Hal Rosenstock wrote: On Mon, Sep 14, 2009 at 10:32 AM, Eli Dorfman (Voltaire) dorfman@gmail.com mailto:dorfman@gmail.com wrote: Hal Rosenstock wrote: On Sun, Sep

[ofa-general] [PATCHv2] opensm/opensm.8.in: Indicate default rule for Default partition

2009-09-14 Thread Hal Rosenstock
Also, similar change to doc/partition-config.txt Signed-off-by: Hal Rosenstock hal.rosenst...@gmail.com --- Changes since v1: Fixed Default rule for non SM ports based on comment from Eli diff --git a/opensm/doc/partition-config.txt b/opensm/doc/partition-config.txt index f855268..cb3bcf7

Re: [ofa-general] [PATCH] infiniband-diags/scripts: Add ibcheckroutes to scripts

2009-09-14 Thread Hal Rosenstock
On Mon, Sep 14, 2009 at 10:32 AM, Eli Dorfman (Voltaire) dorfman@gmail.com wrote: Hal Rosenstock wrote: On Sun, Sep 13, 2009 at 3:55 AM, Doron Shoham dor...@voltaire.com mailto:dor...@voltaire.com wrote: Hal Rosenstock wrote: On Thu, Sep 10, 2009 at 7:56

Re: [ofa-general] Re: [PATCH v2] infiniband-diags/scripts: Add 'ibcheckspeed' and 'ibcheckportspeed' to scripts

2009-09-14 Thread Hal Rosenstock
) 0x0005ad092106 21 (Cisco Switch SFS7000D) 0x0005ad092106 20 (Cisco Switch SFS7000D) 0x0005ad092106 9 (Cisco Switch SFS7000D) 0x0005ad092106 8 (Cisco Switch SFS7000D) Ira On Thu, 10 Sep 2009 09:23:35 -0400 Hal Rosenstock hal.rosenst...@gmail.com wrote

[ofa-general] [PATCH] opensm/opensm.8.in: Cosmetic formatting change

2009-09-11 Thread Hal Rosenstock
Signed-off-by: Hal Rosenstock hal.rosenst...@gmail.com --- diff --git a/opensm/man/opensm.8.in b/opensm/man/opensm.8.in index 5ad7631..fcdc168 100644 --- a/opensm/man/opensm.8.in +++ b/opensm/man/opensm.8.in @@ -257,7 +257,6 @@ This option provides the means to define a set of ports equalization

Re: [ofa-general] Re: [PATCH v2] infiniband-diags/scripts: Add 'ibcheckspeed' and 'ibcheckportspeed' to scripts

2009-09-11 Thread Hal Rosenstock
On Fri, Sep 11, 2009 at 2:13 AM, Barry Mavin barry.ma...@recital.comwrote: When I start the subnet manager on redhat 5.3 with: # service opensm restart I get the following messages in the log file. Sep 11 11:41:46 252576 [4EF83940] 0x01 - __osm_mcmr_rcv_join_mgrp: ERR 1B12:

Re: [ofa-general] question about partitioning IB networks

2009-09-11 Thread Hal Rosenstock
On Fri, Sep 11, 2009 at 11:18 AM, Sasha Khapyorsky sas...@voltaire.comwrote: On 09:17 Thu 03 Sep , Hal Rosenstock wrote: Also it says the default partition will be created “unconditionally even when partition configuration file does not exist or cannot be accessed.” Will it also

Re: [ofa-general] [PATCH] infiniband-diags/scripts: Add ibcheckroutes to scripts

2009-09-10 Thread Hal Rosenstock
On Thu, Sep 10, 2009 at 7:56 AM, Doron Shoham dor...@voltaire.com wrote: ibcheckroutes validates route between all hosts in the fabric. This script finds all leaf switches (switches that are connected to HCAs) CAs or HCAs ? What about switch port 0s ? and runs ibtracert between them.

[ofa-general] Re: [PATCH v2] infiniband-diags/scripts: Add 'ibcheckspeed' and 'ibcheckportspeed' to scripts

2009-09-10 Thread Hal Rosenstock
On Thu, Sep 10, 2009 at 9:02 AM, Keshetti Mahesh keshetti.mah...@gmail.comwrote: Added 'ibcheckspeed' and 'ibcheckportspeed': Similar to 'ibcheckwidth/ibcheckportwidth' in functionality and implementation. Reports error/warning messages if the LinkSpeedActive is configured as 2.5 Gbps when

[ofa-general] [PATCHv2] opensm/osm_inform.c: For traps 64-67, use GID from DataDetails in log message

2009-09-09 Thread Hal Rosenstock
Issuer GID is uninteresting for SM generated notices Signed-off-by: Hal Rosenstock hal.rosenst...@gmail.com --- Changes since v1: Unified OSM_LOG call for traps 64-67 Also, added log level check diff --git a/opensm/opensm/osm_inform.c b/opensm/opensm/osm_inform.c index 990f1e0..6e1a2b5 100644

[ofa-general] [PATCH] infiniband-diags/ibportstate: Support changing of link width

2009-09-08 Thread Hal Rosenstock
Also, update man page Signed-off-by: Hal Rosenstock hal.rosenst...@gmail.com --- diff --git a/infiniband-diags/man/ibportstate.8 b/infiniband-diags/man/ibportstate.8 index 9b5e618..b64c18d 100644 --- a/infiniband-diags/man/ibportstate.8 +++ b/infiniband-diags/man/ibportstate.8 @@ -1,4 +1,4

[ofa-general] [PATCH] opensm doc: Indicated limited (rather than partial) partition membership

2009-09-03 Thread Hal Rosenstock
Signed-off-by: Hal Rosenstock hal.rosenst...@gmail.com --- diff --git a/opensm/doc/partition-config.txt b/opensm/doc/partition-config.txt index ead3f76..f855268 100644 --- a/opensm/doc/partition-config.txt +++ b/opensm/doc/partition-config.txt @@ -10,7 +10,7 @@ when partition configuration file

Re: [ofa-general] question about partitioning IB networks

2009-09-03 Thread Hal Rosenstock
On Mon, Aug 31, 2009 at 3:29 PM, Meyer, Donald J donald.j.me...@intel.comwrote: I am trying to partition my IB network but I don’t seem to be able to understand the opensm man page. First it says “The default partition has P_Key value 0x7fff. OpenSM´s port will have full membership in

Re: [ofa-general] question about partitioning IB networks

2009-09-03 Thread Hal Rosenstock
On Thu, Sep 3, 2009 at 9:43 AM, Yevgeny Kliteynik klit...@dev.mellanox.co.il wrote: Hal Rosenstock wrote: On Mon, Aug 31, 2009 at 3:29 PM, Meyer, Donald J donald.j.me...@intel.com mailto:donald.j.me...@intel.com wrote: ... ... Just as a sanity check, my interpretation so far

Re: [ofa-general] question about partitioning IB networks

2009-09-03 Thread Hal Rosenstock
* -- *From:* Hal Rosenstock [mailto:hal.rosenst...@gmail.com] *Sent:* Thursday, September 03, 2009 6:46 AM *To:* klit...@dev.mellanox.co.il *Cc:* Meyer, Donald J; general@lists.openfabrics.org *Subject:* Re: [ofa-general] question about partitioning IB networks On Thu

[ofa-general] [PATCH] libibmad/dump.c: In mad_dump_portcapmask, decode new capabilities

2009-09-02 Thread Hal Rosenstock
Per published MgtWG errata RefID 4484 - vendor specific MADs table support RefID 4626 - reverse path PKey support in PathRecord responses RefID 4635 - multicast FDB top support RefID 4644 - hierarchy support Signed-off-by: Hal Rosenstock hal.rosenst...@gmail.com --- diff --git a/libibmad/src

[ofa-general] [PATCH] opensm: Add support for MulticastFDBTop

2009-09-02 Thread Hal Rosenstock
on max_block_in_use when switch port 0 indicates IsMulticastFDBTop is supported. Signed-off-by: Hal Rosenstock hal.rosenst...@gmail.com --- diff --git a/opensm/opensm/osm_mcast_mgr.c b/opensm/opensm/osm_mcast_mgr.c index d7c5ce1..3671e08 100644 --- a/opensm/opensm/osm_mcast_mgr.c +++ b/opensm/opensm/osm_mcast_mgr.c

[ofa-general] [PATCH] libibmad/mad.h: Add a couple of SM class attribute IDs

2009-09-02 Thread Hal Rosenstock
VendorSpecificMadsTable added by MgtWG errata RefID 4482 Signed-off-by: Hal Rosenstock hal.rosenst...@gmail.com --- diff --git a/libibmad/include/infiniband/mad.h b/libibmad/include/infiniband/mad.h index 5f3b52b..94b64cf 100644 --- a/libibmad/include/infiniband/mad.h +++ b/libibmad/include

[ofa-general] [PATCH] opensm: Add infrastructure support for more newly allocated PortInfo CapabilityMask bits

2009-09-01 Thread Hal Rosenstock
Per published MgtWG errata: RefID 4484 - vendor specific MADs RefID 4575 - multicast PKey trap suppression RefID 4641 - hierarchy info Signed-off-by: Hal Rosenstock hal.rosenst...@gmail.com --- diff --git a/opensm/include/iba/ib_types.h b/opensm/include/iba/ib_types.h index c9d81cb..25ed35f

[ofa-general] [PATCH] opensm/osm_base.h: Add new SA ClassPortInfo:CapabilityMask2 bit allocations

2009-09-01 Thread Hal Rosenstock
Per published MgtWG errata: RefID 4626 - reverse path PKey support in PathRecord responses RefID 4635 - multicast FDB top support RefID 4644 - hierarchy support Signed-off-by: Hal Rosenstock hal.rosenst...@gmail.com --- diff --git a/opensm/include/opensm/osm_base.h b/opensm/include/opensm

[ofa-general] [PATCHv2] opensm: Parallelize (Stripe) MFT sets across switches

2009-08-31 Thread Hal Rosenstock
switches can be responding concurrently. Speedup is dependent on number of MFT blocks in use (number of MLIDs) which is dependent on the number of multicast groups. Signed-off-by: Hal Rosenstock hal.rosenst...@gmail.com --- Changes since v1: Fixed loop which stripes MFT block across switches Changed

Re: [ofa-general] Re: [PATCH] infiniband-diags/ibroute: Add support for MulticastFDBTop

2009-08-31 Thread Hal Rosenstock
On 8/31/09, Sasha Khapyorsky sas...@voltaire.com wrote: On 12:35 Sun 30 Aug , Hal Rosenstock wrote: Doesn't the loop: for (block = startblock; block = lastblock; block++) terminates without any blocks read ? So it shows no entries. Sorry, I still don't understand. Let's suppose

[ofa-general] [PATCH] osmtest: Add SA get PathRecord stress test

2009-08-31 Thread Hal Rosenstock
Signed-off-by: Hal Rosenstock hal.rosenst...@gmail.com --- diff --git a/opensm/man/osmtest.8 b/opensm/man/osmtest.8 index fa0cd52..f0d6323 100644 --- a/opensm/man/osmtest.8 +++ b/opensm/man/osmtest.8 @@ -1,4 +1,4 @@ -.TH OSMTEST 8 August 11, 2008 OpenIB OpenIB Management +.TH OSMTEST 8 August 31

[ofa-general] Re: osm_link_mgr.c:link_mgr_get_smsl question

2009-08-30 Thread Hal Rosenstock
Hi Sasha, On 8/29/09, Sasha Khapyorsky sas...@voltaire.com wrote: Hi Hal, On 14:38 Fri 07 Aug , Hal Rosenstock wrote: osm_link_mgr.c:link_mgr_get_smsl has the following: /* Find osm_port of the source = p_physp */ slid = osm_physp_get_base_lid(p_physp

Re: [ofa-general] Re: [PATCH] opensm/ib_types.h: Add CounterSelect2 field to PortCounters attribute

2009-08-30 Thread Hal Rosenstock
On 8/30/09, Sasha Khapyorsky sas...@voltaire.com wrote: On 11:54 Wed 26 Aug , Hal Rosenstock wrote: Per MgtWG RefID #4527 Also, cosmetic commentary change Signed-off-by: Hal Rosenstock hal.rosenst...@gmail.com Applied. Thanks. Next time could you add more descriptive change

Re: [ofa-general] Re: [PATCH] opensm: Add infrastructure support for MulticastFDBTop

2009-08-30 Thread Hal Rosenstock
On 8/30/09, Sasha Khapyorsky sas...@voltaire.com wrote: On 10:04 Wed 26 Aug , Hal Rosenstock wrote: @@ -5899,6 +5899,8 @@ typedef struct _ib_switch_info { ib_net16_t lids_per_port; ib_net16_t enforce_cap; uint8_t flags; + uint8_t resvd; + ib_net16_t

[ofa-general] [PATCHv2] opensm: Add infrastructure support for MulticastFDBTop

2009-08-30 Thread Hal Rosenstock
Add support for SwitchInfo:MulticastFDBTop Added by MgtWG errata #4505-4508 Add OpenSM infrastructure support to ib_types.h and osm_helper.c Signed-off-by: Hal Rosenstock hal.rosenst...@gmail.com --- Changes since v1: Removed erroneous pad byte left remaining in ib_switch_info_record_t diff

[ofa-general] [PATCHv2] infiniband-diags/ibroute: Add support for MulticastFDBTop

2009-08-30 Thread Hal Rosenstock
) Signed-off-by: Hal Rosenstock hal.rosenst...@gmail.com --- Changes since v1: Fixed top range check diff --git a/infiniband-diags/src/ibroute.c b/infiniband-diags/src/ibroute.c index 106c934..1112b87 100644 --- a/infiniband-diags/src/ibroute.c +++ b/infiniband-diags/src/ibroute.c @@ -1,5 +1,6

Re: [ofa-general] Re: [PATCH] infiniband-diags/ibroute: Add support for MulticastFDBTop

2009-08-30 Thread Hal Rosenstock
On 8/30/09, Sasha Khapyorsky sas...@voltaire.com wrote: On 08:42 Sun 30 Aug , Hal Rosenstock wrote: This is handled by the block loop inside of dump_multicast_tables. Where? I don't see this. Should not it to show nothing (no entries) when top = 0xbfff and dump_all is not set

[ofa-general] [PATCHv3] infiniband-diags/ibroute: Add support for MulticastFDBTop

2009-08-30 Thread Hal Rosenstock
) Signed-off-by: Hal Rosenstock hal.rosenst...@gmail.com --- Changes since v2: Removed redundant clause in top range check Changes since v1: Fixed top range check diff --git a/infiniband-diags/src/ibroute.c b/infiniband-diags/src/ibroute.c index 106c934..00df1ec 100644 --- a/infiniband-diags/src

[ofa-general] [PATCH] opensm: Reduce heap consumption by unicast routing tables (LFTs)

2009-08-29 Thread Hal Rosenstock
. Signed-off-by: Hal Rosenstock hal.rosenst...@gmail.com --- diff --git a/opensm/config/osmvsel.m4 b/opensm/config/osmvsel.m4 index c24930b..1c7c8a2 100644 --- a/opensm/config/osmvsel.m4 +++ b/opensm/config/osmvsel.m4 @@ -232,6 +232,25 @@ fi # --- END OPENIB_OSM_PERF_MGR_SEL --- ]) dnl

Re: [ofa-general] [PATCH] opensm/osm_ucast_mgr.c: simplify fwd tables setup flow

2009-08-29 Thread Hal Rosenstock
On 8/29/09, Sasha Khapyorsky sas...@voltaire.com wrote: On 12:03 Fri 28 Aug , Hal Rosenstock wrote: lash_core: ERR 4D02: Lane requirements (9) exceed available lanes (8) with starting lane (0) ucast_mgr_route: lash: cannot build fwd tables. osm_ucast_mgr_process: minhop tables

[ofa-general] [PATCH] opensm/osm_helper.c: Add SM priority changed into trap 144 description

2009-08-28 Thread Hal Rosenstock
Per MgtWG RefID #4503 Signed-off-by: Hal Rosenstock hal.rosenst...@gmail.com --- diff --git a/opensm/opensm/osm_helper.c b/opensm/opensm/osm_helper.c index 3692474..1b83a9e 100644 --- a/opensm/opensm/osm_helper.c +++ b/opensm/opensm/osm_helper.c @@ -531,7 +531,7 @@ const char *ib_get_trap_str

Re: [ofa-general] [PATCH] opensm/osm_ucast_mgr.c: simplify fwd tables setup flow

2009-08-28 Thread Hal Rosenstock
On 8/28/09, Sasha Khapyorsky sas...@voltaire.com wrote: Simplify (and unify) forwarding tables setup decision flow. Seems to work for all engines but I got a failure for a test case where lash fell back to min hop: lash_core: ERR 4D02: Lane requirements (9) exceed available lanes (8) with

Re: [ofa-general] [PATCH] opensm/osm_ucast_mgr: better lft setup

2009-08-28 Thread Hal Rosenstock
be repeated again and again. This patch cleans this ineffectiveness. Also rename it to set_lft_block() since block number is treated as parameters and *not* next block is processed and merges some code. Signed-off-by: Sasha Khapyorsky sas...@voltaire.com Acked-by: Hal Rosenstock hal.rosenst

[ofa-general] Re: [ewg] [PATCH] IB/ehca: Construct MAD redirect replies from request MAD

2009-08-27 Thread Hal Rosenstock
On 8/27/09, Joachim Fenkes fen...@de.ibm.com wrote: Hal Rosenstock hal.rosenst...@gmail.com wrote on 26.08.2009 17:15:03: Thanks for doing this. It looks sane to me. The only issue I recall that appears to be remaining is a better setting of ClassPortInfo:RespTimeValue rather than

[ofa-general] [PATCH] opensm: Add infrastructure support for MulticastFDBTop

2009-08-26 Thread Hal Rosenstock
Add support for SwitchInfo:MulticastFDBTop Added by MgtWG errata #4505-4508 Add OpenSM infrastructure support to ib_types.h and osm_helper.c Signed-off-by: Hal Rosenstock hal.rosenst...@gmail.com --- diff --git a/opensm/include/iba/ib_types.h b/opensm/include/iba/ib_types.h index fe3f051

[ofa-general] [PATCH] infiniband-diags/ibroute: Add support for MulticastFDBTop

2009-08-26 Thread Hal Rosenstock
#4640) Signed-off-by: Hal Rosenstock hal.rosenst...@gmail.com --- diff --git a/infiniband-diags/src/ibroute.c b/infiniband-diags/src/ibroute.c index 106c934..f3ebe56 100644 --- a/infiniband-diags/src/ibroute.c +++ b/infiniband-diags/src/ibroute.c @@ -1,5 +1,6 @@ /* * Copyright (c) 2004-2008 Voltaire

[ofa-general] [PATCH] libibmad: Add support for MulticastFDBTop

2009-08-26 Thread Hal Rosenstock
Add support for SwitchInfo:MulticastFDBTop and PortInfo:CapabilityMask.IsMulticastFDBTopSupported Added by MgtWG errata #4505-4508 Signed-off-by: Hal Rosenstock hal.rosenst...@gmail.com --- diff --git a/libibmad/include/infiniband/mad.h b/libibmad/include/infiniband/mad.h index 3093fbd

Re: [ofa-general] Combined DR path with empty DR path, what is the expected behavior?

2009-08-26 Thread Hal Rosenstock
On 8/25/09, Ira Weiny wei...@llnl.gov wrote: On Tue, 25 Aug 2009 19:15:19 -0400 Hal Rosenstock hal.rosenst...@gmail.com wrote: On 8/24/09, Ira Weiny wei...@llnl.gov wrote: If I send a combined DR path with a start lid but an empty (0 length) DR path. Hop Count 0 ? Yes

[ofa-general] Re: [ewg] [PATCH] IB/ehca: Construct MAD redirect replies from request MAD

2009-08-26 Thread Hal Rosenstock
On 8/26/09, Joachim Fenkes fen...@de.ibm.com wrote: The old code used a lot of hardcoded values, which might not be valid in all environments (especially routed fabrics or partitioned subnets). Copy as much information as possible from the incoming request to prevent that. Signed-off-by:

Re: [ofa-general] ofed 1.3.2 opensmd failover

2009-08-26 Thread Hal Rosenstock
be run manually or made to run automatically but I'm not familiar with the details. -- Hal Thanks a lot. Regards, PN 2009/8/26 Hal Rosenstock hal.rosenst...@gmail.com On 8/25/09, kovlen...@interia.pl kovlen...@interia.pl wrote: Hi all, Quick question - is there a need to run

Re: [ofa-general] Problems with OpenSM from ofed 1.4.1 and MESH topology.

2009-08-26 Thread Hal Rosenstock
Hi Rafael, On 8/25/09, Rafael David Tinoco rafael.tin...@sun.com wrote: Hello Hal, Bellow... Hal Rosenstock wrote: On 8/24/09, Rafael David Tinoco rafael.tin...@sun.com wrote: Hello, I'm installing an HPC cluster using 2 Sun Blades 6048 with QNEMs (2 asics each, 8 qnems

[ofa-general] [PATCH] opensm/ib_types.h: Add CounterSelect2 field to PortCounters attribute

2009-08-26 Thread Hal Rosenstock
Per MgtWG RefID #4527 Also, cosmetic commentary change Signed-off-by: Hal Rosenstock hal.rosenst...@gmail.com --- diff --git a/opensm/include/iba/ib_types.h b/opensm/include/iba/ib_types.h index fe3f051..42ec794 100644 --- a/opensm/include/iba/ib_types.h +++ b/opensm/include/iba/ib_types.h

[ofa-general] [PATCH] infiniband-diags/perfquery.c: Indicate whether PortXmitWait counter is supported

2009-08-26 Thread Hal Rosenstock
Indicate extended v. (normal) port counters in output Also, some cosmetic formatting changes and commentary typo fixed Signed-off-by: Hal Rosenstock hal.rosenst...@gmail.com --- diff --git a/infiniband-diags/src/perfquery.c b/infiniband-diags/src/perfquery.c index 39ae2f6..0fd083e 100644

Re: [ofa-general] ofed 1.3.2 opensmd failover

2009-08-25 Thread Hal Rosenstock
On 8/25/09, kovlen...@interia.pl kovlen...@interia.pl wrote: Hi all, Quick question - is there a need to run anything except opensmd deamons to provide failover capability on ib network in ofed 1.3? In terms of SM failover, modulo bugs fixed relative to this feature since OFED 1.3 (there

  1   2   3   4   5   6   7   8   9   10   >