On Mon, Mar 2, 2009 at 2:19 AM, Sasha Khapyorsky <[email protected]> wrote:
> 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.
Oh I see; it's the below:
for (; port < num_ports; port++) {
-- Hal
> 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
