[openib-general] OpenSM/osm_sa_path_record.c: In __osm_pr_rcv_get_lid_pair_path, remove double calculation of reversible path

Hal Rosenstock halr at voltaire.com
Mon Aug 21 06:22:03 PDT 2006


OpenSM/osm_sa_path_record.c: In __osm_pr_rcv_get_lid_pair_path, remove
double calculation of reversible path

Pointed out by Sasha Khapyorsky <sashak at voltaire.com>
Signed-off-by: Hal Rosenstock <halr at voltaire.com>

Index: opensm/osm_sa_path_record.c
===================================================================
--- opensm/osm_sa_path_record.c	(revision 9021)
+++ opensm/osm_sa_path_record.c	(working copy)
@@ -728,7 +728,7 @@ __osm_pr_rcv_get_lid_pair_path(
   rev_path_status = __osm_pr_rcv_get_path_parms( p_rcv, p_pr, p_dest_port,
                                                  p_src_port, src_lid_ho,
                                                  comp_mask, &rev_path_parms );
-  path_parms.reversible = (rev_path_status == IB_SUCCESS);
+  path_parms.reversible = ( rev_path_status == IB_SUCCESS );
 
   /* did we get a Reversible Path compmask ? */
   /* 
@@ -738,11 +738,6 @@ __osm_pr_rcv_get_lid_pair_path(
   */
   if( comp_mask & IB_PR_COMPMASK_REVERSIBLE )
   {
-    /* now try the reversible path */
-    rev_path_status = __osm_pr_rcv_get_path_parms( p_rcv, p_pr, p_dest_port,
-                                                   p_src_port, src_lid_ho,
-                                                   comp_mask, &rev_path_parms );
-    path_parms.reversible = (rev_path_status == IB_SUCCESS);
     if( (! path_parms.reversible && ( p_pr->num_path & 0x80 ) ) )
     {
       osm_log( p_rcv->p_log, OSM_LOG_DEBUG,







More information about the general mailing list