From mboxrd@z Thu Jan 1 00:00:00 1970 From: Alex Netes Subject: Re: [PATCH] opensm: perfmgr, only set orig_lid when we have a valid port. Otherwise leave it as 0 Date: Sun, 17 Apr 2011 18:06:14 +0300 Message-ID: <20110417150614.GA31503@calypso.voltaire.com> References: <20110415151736.809c550b.weiny2@llnl.gov> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Return-path: Content-Disposition: inline In-Reply-To: <20110415151736.809c550b.weiny2-i2BcT+NCU+M@public.gmane.org> Sender: linux-rdma-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Ira Weiny Cc: "linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" List-Id: linux-rdma@vger.kernel.org Hi Ira, On 15:17 Fri 15 Apr , Ira Weiny wrote: > > Subject: [PATCH] opensm: perfmgr, only set orig_lid when we have a valid port. Otherwise leave it as 0 > > > 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 6a1fa63..3e1575a 100644 > --- a/opensm/osm_perfmgr.c > +++ b/opensm/osm_perfmgr.c > @@ -454,7 +454,9 @@ static void collect_guids(cl_map_item_t * p_map_item, void *context) > ib_switch_info_is_enhanced_port0(&node->sw-> > switch_info)); > for (port = mon_node->esp0 ? 0 : 1; port < num_ports; port++) { > - mon_node->port[port].orig_lid = get_base_lid(node, port); > + mon_node->port[port].orig_lid = 0; > + if (osm_physp_is_valid(&node->physp_table[port])) > + mon_node->port[port].orig_lid = get_base_lid(node, port); > mon_node->port[port].valid = TRUE; Shouldn't this port marked with mon_node->port[port].valid = FALSE ? -- Alex -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html