[PATCHv4] opensm: Reduce heap consumption by unicast routing tables (LFTs)

2009-10-13 Thread Hal Rosenstock
Heap memory consumption by the unicast and multicast routing tables can be reduced. Using valgrind --tool=massif (for heap profiling), there are couple of places that consume most of the heap memory: -38.75% (11,206,656B) 0x43267E: osm_switch_new (osm_switch.c:134) -12.89% (3,728,256B)

[PATCH] infiniband-diags/saquery.c: Modify dump_path_record

2009-10-13 Thread Hal Rosenstock
Similar to commit 27ea3c85500f7eff741c93e662a344742dbbea13 to fix gcc-4.4.1 warnings Fix dereferencing type-punned pointer will break strict-aliasing rules warning catched by gcc-4.4.1. Signed-off-by: Hal Rosenstock hal.rosenst...@gmail.com --- diff --git a/infiniband-diags/src/saquery.c

Re: [PATCH 1/2] rdma/cm: support option to allow manually setting IB path

2009-10-13 Thread Or Gerlitz
Sean Hefty wrote: Before spending any more time on this patch series, is there any disagreement to accepting this patch (as is or slightly modified) upstream? Hi Sean, This patch just sets a route to the kernel and have the kernel issue a route resolved event in return, sounds good to me, I

Re: [PATCH] opensm: Add infrastructure support for PortInfo IsMulticastPkeyTrapSuppressionSupported

2009-10-13 Thread Hal Rosenstock
On Mon, Oct 12, 2009 at 2:45 PM, Sasha Khapyorsky sas...@voltaire.com wrote: On 10:46 Fri 09 Oct     , Hal Rosenstock wrote: Per published MgtWG errata RefID 4576 Signed-off-by: Hal Rosenstock hal.rosenst...@gmail.com --- diff --git a/opensm/include/iba/ib_types.h

Re: OpenSM Failover

2009-10-13 Thread Aaron Knister
Thanks! I really appreciate that. I still have a question about the initial failover- I'm still wondering why there's a 30 second delay. Wouldn't nodeA send some type of handover message (my IB knowledge is limited) to notify a subnet manager of a lower priority to take over? As I said, the older

[PATCH] opensm - standardize on a single Windows #define - take #2

2009-10-13 Thread Stan C. Smith
Code cleanup, standardize on a single Windows #define '__WIN__'. Signed-off-by: Stan Smith stan.sm...@intel.com diff --git a/opensm/include/iba/ib_cm_types.h b/opensm/include/iba/ib_cm_types.h index c1fbfaf..061f47b 100644 --- a/opensm/include/iba/ib_cm_types.h +++

Re: [ofa-general] Re: [GIT PULL] please pull ummunotify

2009-10-13 Thread Jason Gunthorpe
On Tue, Oct 13, 2009 at 08:40:06AM +0200, Ingo Molnar wrote: * Jason Gunthorpe jguntho...@obsidianresearch.com wrote: On Mon, Oct 12, 2009 at 10:20:46PM +0200, Ingo Molnar wrote: It might be more acceptable because the flag-hint mechanism can at most cause over-flushing - while with

[Fwd: FW: OFED installation question re-AltLinux]

2009-10-13 Thread Jeff Becker
Forwarding... Please reply to Sergey (cc'd above). Thanks. -jeff P.S. The general list does not accept messages anymore - the archives are still available. Original Message Subject:FW: OFED installation question re-AltLinux Date: Tue, 13 Oct 2009 11:14:08 -0500 From:

[PATCHv2] opensm: Add infrastructure support for PortInfo IsMulticastPkeyTrapSuppressionSupported

2009-10-13 Thread Hal Rosenstock
Per published MgtWG errata RefID 4576 Signed-off-by: Hal Rosenstock hal.rosenst...@gmail.com --- Changes since v1: Don't reset reserved bits packed with subnet timeout diff --git a/opensm/include/iba/ib_types.h b/opensm/include/iba/ib_types.h index aa566f7..7a54d9e 100644 ---

Re: [PATCHv4] opensm: Reduce heap consumption by unicast routing tables (LFTs)

2009-10-13 Thread Sasha Khapyorsky
On 09:00 Tue 13 Oct , Hal Rosenstock wrote: Heap memory consumption by the unicast and multicast routing tables can be reduced. Using valgrind --tool=massif (for heap profiling), there are couple of places that consume most of the heap memory: -38.75% (11,206,656B) 0x43267E:

[PATCH] opensm/osm_state_mgr.c: in cleanup_switch() check only relevant LFT part

2009-10-13 Thread Sasha Khapyorsky
In cleanup_switch() be sure that only relevant LFT entries were updated, don't care about outdated part of LFT. Signed-off-by: Sasha Khapyorsky sas...@voltaire.com --- opensm/opensm/osm_state_mgr.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git

[PATCHv3] opensm: Add infrastructure support for PortInfo IsMulticastPkeyTrapSuppressionSupported

2009-10-13 Thread Hal Rosenstock
Per published MgtWG errata RefID 4576 Signed-off-by: Hal Rosenstock hal.rosenst...@gmail.com --- Changes since v2: Changed ib_port_info_set_client_rereg similarly to ib_port_info_set_mcast_pkey_trap_suppress Changes since v1: Don't reset reserved bits packed with subnet timeout diff --git

Re: [Fwd: FW: OFED installation question re-AltLinux]

2009-10-13 Thread Guy Coates
Jeff Becker wrote: Forwarding... Please reply to Sergey (cc'd above). Thanks. -jeff P.S. The general list does not accept messages anymore - the archives are still available. Original Message Subject: FW: OFED installation question re-AltLinux Date: Tue,

[PATCH] opensm/osm_mesh: Cosmetic changes

2009-10-13 Thread Hal Rosenstock
Signed-off-by: Hal Rosenstock hal.rosenst...@gmail.com --- diff --git a/opensm/include/opensm/osm_mesh.h b/opensm/include/opensm/osm_mesh.h index 3800372..d95b9a9 100644 --- a/opensm/include/opensm/osm_mesh.h +++ b/opensm/include/opensm/osm_mesh.h @@ -1,5 +1,5 @@ /* - * Copyright (c) 2008,2009

Re: [PATCHv3] opensm: Add infrastructure support for PortInfo IsMulticastPkeyTrapSuppressionSupported

2009-10-13 Thread Sasha Khapyorsky
On 14:10 Tue 13 Oct , Hal Rosenstock wrote: Per published MgtWG errata RefID 4576 Signed-off-by: Hal Rosenstock hal.rosenst...@gmail.com --- Changes since v2: Changed ib_port_info_set_client_rereg similarly to ib_port_info_set_mcast_pkey_trap_suppress Changes since v1: Don't

Re: [PATCHv3] opensm: Add infrastructure support for PortInfo IsMulticastPkeyTrapSuppressionSupported

2009-10-13 Thread Hal Rosenstock
On Tue, Oct 13, 2009 at 3:17 PM, Sasha Khapyorsky sas...@voltaire.com wrote: On 14:10 Tue 13 Oct     , Hal Rosenstock wrote: Per published MgtWG errata RefID 4576 Signed-off-by: Hal Rosenstock hal.rosenst...@gmail.com --- Changes since v2: Changed ib_port_info_set_client_rereg similarly to

Re: [PATCHv3] opensm: Add infrastructure support for PortInfo IsMulticastPkeyTrapSuppressionSupported

2009-10-13 Thread Hal Rosenstock
On Tue, Oct 13, 2009 at 3:20 PM, Hal Rosenstock hal.rosenst...@gmail.com wrote: On Tue, Oct 13, 2009 at 3:17 PM, Sasha Khapyorsky sas...@voltaire.com wrote: On 14:10 Tue 13 Oct     , Hal Rosenstock wrote: Per published MgtWG errata RefID 4576 Signed-off-by: Hal Rosenstock

Re: Default force_link_speed value in opensm configuration

2009-10-13 Thread Ira Weiny
On Tue, 13 Oct 2009 09:39:08 -0400 Hal Rosenstock hal.rosenst...@gmail.com wrote: On Mon, Oct 12, 2009 at 5:35 PM, Ira Weiny wei...@llnl.gov wrote: On Mon, 12 Oct 2009 11:47:37 +0200 Eli Dorfman (Voltaire) dorfman@gmail.com wrote: Ira Weiny wrote: On Tue, 06 Oct 2009 17:56:01

Re: Default force_link_speed value in opensm configuration

2009-10-13 Thread Hal Rosenstock
On Tue, Oct 13, 2009 at 3:44 PM, Ira Weiny wei...@llnl.gov wrote: On Tue, 13 Oct 2009 09:39:08 -0400 Hal Rosenstock hal.rosenst...@gmail.com wrote: On Mon, Oct 12, 2009 at 5:35 PM, Ira Weiny wei...@llnl.gov wrote: On Mon, 12 Oct 2009 11:47:37 +0200 Eli Dorfman (Voltaire)

Re: OpenSM Failover

2009-10-13 Thread Aaron Knister
I wouldn't expect an SM that goes belly up to failover instantly, but I would have thought that if you shut it down gracefully (with a regular kill signal) that it could notify a standby SM that it's going away. Either way, I appreciate the help! On Tue, Oct 13, 2009 at 12:13 PM, Yevgeny

[PATCH] link-local address fix for rdma_resolve_addr

2009-10-13 Thread David J. Wilder
Here is a patch to addr6_resolve_remote() to correctly handle link-local address. It should cover all the conditions Jason described. With this patch rping works as expected. Link-local with scope: # /usr/bin/rping -c -a fe80::202:c903:1:1925%ib0 Link-local w/out scope: # /usr/bin/rping -c

Re: [PATCH] link-local address fix for rdma_resolve_addr

2009-10-13 Thread Jason Gunthorpe
On Tue, Oct 13, 2009 at 03:09:40PM -0700, David J. Wilder wrote: Here is a patch to addr6_resolve_remote() to correctly handle link-local address. It should cover all the conditions Jason described. Looks pretty good to me, definitely on the right track. Hmm.. Actually, upon comparing to