Hi Hal, On 13:55 Thu 30 Jul , Hal Rosenstock wrote: > > Signed-off-by: Hal Rosenstock <[email protected]> > --- > diff --git a/opensm/opensm/osm_lid_mgr.c b/opensm/opensm/osm_lid_mgr.c > index 34625ba..64157e0 100644 > --- a/opensm/opensm/osm_lid_mgr.c > +++ b/opensm/opensm/osm_lid_mgr.c > @@ -1228,9 +1228,8 @@ int osm_lid_mgr_process_subnet(IN osm_lid_mgr_t * p_mgr) > min_lid_ho, max_lid_ho); > > /* the proc returns the fact it sent a set port info */ > - if (lid_mgr_set_physp_pi(p_mgr, p_port, p_port->p_physp, > - cl_hton16(min_lid_ho))) > - ret = -1; > + ret = lid_mgr_set_physp_pi(p_mgr, p_port, p_port->p_physp, > + cl_hton16(min_lid_ho));
Actually it is not correct (and I tried to explain this to you some time ago) - subsequent successive PortInfo set run will potentially overwrite a previous failure status. Sasha _______________________________________________ general mailing list [email protected] http://lists.openfabrics.org/cgi-bin/mailman/listinfo/general To unsubscribe, please visit http://openib.org/mailman/listinfo/openib-general
