[openib-general] [PATCH] osm_sa_mcmember_record : MCMember Get/GetTable Trusted mode

Ofer Gigi oferg at mellanox.co.il
Wed Apr 26 02:29:54 PDT 2006


Hi Hal,
Small minor chnages:
1. Adding () for the if statement
2. Clearer messages when duplicate guids are found  

Thanks

Ofer G.

Signed-off-by:  Ofer Gigi <oferg at mellanox.co.il>

Index: osm/opensm/osm_node_info_rcv.c
===================================================================
--- osm/opensm/osm_node_info_rcv.c	(revision 6640)
+++ osm/opensm/osm_node_info_rcv.c	(working copy)
@@ -129,8 +129,8 @@ __osm_ni_rcv_set_links(
         }
         else
         {
-          if( osm_node_has_any_link( p_node, port_num ) && 
-              p_rcv->p_subn->force_immediate_heavy_sweep == FALSE )
+          if( (osm_node_has_any_link( p_node, port_num )) && 
+              (p_rcv->p_subn->force_immediate_heavy_sweep == FALSE) )
           {
             /*
               Uh oh...
@@ -205,13 +205,14 @@ __osm_ni_rcv_set_links(
                      );
 
             osm_log( p_rcv->p_log, OSM_LOG_SYS,
-                     "Errors on subnet. SM found duplicated guids or 12x " 
-                     "link with lane reversal badly configured. "
-                     "See osm log for more details\n");
+                     "Fatal: duplicated guids or 12x lane reversal.\n");
 
             if ( p_rcv->p_subn->opt.exit_on_fatal == TRUE )
+			   {
+			     osm_log( p_rcv->p_log, OSM_LOG_SYS,"Exiting ...\n");
               exit( 1 );
           }
+          }
 
           /* 
              When there are only two nodes with exact same guids (connected back to 





More information about the general mailing list