Yea, that comment does not make sense.

I am searching the node records for the node which has the port defined in the
member record.  The interface ID is simply the port guid, is it not?

        uint64_t gid_interface_id = cl_ntoh64( 
p_mcmr->port_gid.unicast.interface_id );

I guess the comments should read.

   /* go through the node records searching for a port guid which matches
    * this port gid interface id.
    * This gives us a node description to print.
    */

Does that make more sense?

Ira

On Tue, 10 Jun 2008 07:46:26 -0700
Hal Rosenstock <[EMAIL PROTECTED]> wrote:

> Ira,
> 
> In saquery.c:print_multicast_member_record, there is the following:
> 
>         /* go through and find the node description for this node GID */
>         for (i = 0; i < result.result_cnt; i++) {
>                 node_record = osmv_get_query_node_rec(result.p_result_madw, 
> i);
>                 if (cl_ntoh64(node_record->node_info.port_guid) == 
> gid_interface_id)
>                         break;
>         }
> 
> I have a question about the intention here:
> 
> The comment mentions the node description for this node GID. Is that
> what is meant or is the intention to get the node description
> corresponding to the port GUID for each MCMember record ?
> 
> -- Hal
> 
> 
_______________________________________________
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