[openib-general] opensm and faulty hardware

[Hal Rosenstock]

On Tue, 2005-09-27 at 16:00, Viswanath Krishnamurthy wrote:
> Hal,
> 
> I added a hack now to get around the problem. There needs to be a
> proper fix later..

Can you try this instead ? Thanks.

-- Hal

Index: include/opensm/osm_port.h
===================================================================
--- include/opensm/osm_port.h	(revision 3567)
+++ include/opensm/osm_port.h	(working copy)
@@ -346,7 +346,7 @@ osm_physp_is_healthy(
 *	Returns TRUE if the Physical Port has been maked as healthy
 *	FALSE otherwise.
 *  All physical ports are initialized as "healthy" but may be marked 
-*  otherwise if a  received trap claims otherwise.
+*  otherwise if a received trap claims otherwise.
 *
 * NOTES
 *
@@ -456,6 +456,42 @@ osm_physp_set_port_info(
 *	Port, Physical Port
 *********/
 
+/****f* OpenSM: Physical Port/osm_physp_validate_base_lid
+* NAME
+*  osm_physp_validate_base_lid
+*
+* DESCRIPTION
+*  Validates the base LID in the Physical Port object.
+*
+* SYNOPSIS
+*/
+static inline boolean_t
+osm_physp_validate_base_lid(
+	IN osm_physp_t* const p_physp )
+{
+	CL_ASSERT( osm_physp_is_valid( p_physp ) );
+	if ( cl_ntoh16( p_physp->port_info.base_lid ) > IB_LID_UCAST_END_HO )
+	{
+		p_physp->port_info.base_lid = 0;
+		return FALSE;
+	}
+	return TRUE;
+}
+/*
+* PARAMETERS
+*	p_physp
+*		[in] Pointer to an osm_physp_t object.
+*
+* RETURN VALUES
+*	Returns TRUE if the base LID in the Physical port object is valid.
+*	FALSE otherwise.
+*
+* NOTES
+*
+* SEE ALSO
+*	Port, Physical Port
+*********/
+
 /****f* OpenSM: Physical Port/osm_physp_set_pkey_tbl
 * NAME
 *  osm_physp_set_pkey_tbl
Index: opensm/osm_port_info_rcv.c
===================================================================
--- opensm/osm_port_info_rcv.c	(revision 3579)
+++ opensm/osm_port_info_rcv.c	(working copy)
@@ -346,8 +346,12 @@ __osm_pi_rcv_process_switch_port(
 
   if (port_num == 0)
   {
-    /* This is a management port 0 */
-		__osm_pi_rcv_process_endport(p_rcv, p_physp, p_pi);
+ 	/* This is switch management port 0 */
+	if ( !osm_physp_validate_base_lid( p_physp ) )
+		osm_log( p_rcv->p_log, OSM_LOG_ERROR,
+			 "__osm_pi_rcv_process_switch_port: ERR 0F04: "
+			 "Invalid base LID corrected.\n" );
+	__osm_pi_rcv_process_endport(p_rcv, p_physp, p_pi);
   }
 
   OSM_LOG_EXIT( p_rcv->p_log );
@@ -367,6 +371,10 @@ __osm_pi_rcv_process_ca_port(
   UNUSED_PARAM( p_node );
 
   osm_physp_set_port_info( p_physp, p_pi );
+  if ( !osm_physp_validate_base_lid( p_physp ) )
+    osm_log( p_rcv->p_log, OSM_LOG_ERROR,
+	     "__osm_pi_rcv_process_ca_port: ERR 0F08: "
+	     "Invalid base LID corrected.\n" );
 
   __osm_pi_rcv_process_endport(p_rcv, p_physp, p_pi);
 
@@ -390,6 +398,10 @@ __osm_pi_rcv_process_router_port(
     Update the PortInfo attribute.
   */
   osm_physp_set_port_info( p_physp, p_pi );
+  if ( !osm_physp_validate_base_lid( p_physp ) )
+    osm_log( p_rcv->p_log, OSM_LOG_ERROR,
+	     "__osm_pi_rcv_process_router_port: ERR 0F09: "
+	     "Invalid base LID corrected.\n" );
 
   OSM_LOG_EXIT( p_rcv->p_log );
 }