On 20:24 Sun 01 Mar     , Hal Rosenstock wrote:
> > + ?? ?? ?? if (ni->node_type == IB_NODE_TYPE_SWITCH) {
> > + ?? ?? ?? ?? ?? ?? ?? port = 0;
> > + ?? ?? ?? ?? ?? ?? ?? num_ports = osm_node_get_num_physp(node);
> > + ?? ?? ?? } else {
> > + ?? ?? ?? ?? ?? ?? ?? port = ib_node_info_get_local_port_num(ni);
> > + ?? ?? ?? ?? ?? ?? ?? num_ports = port + 1;
> 
> The else clause determination of num_ports doesn't look right to me.
> Shouldn't it be
> num_ports = ni->num_ports ?

No, it's non-switch case where we are querying a single port.

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

Reply via email to