Makes perfect sense.

> -----Original Message-----
> From: Hal Rosenstock [mailto:[EMAIL PROTECTED]
> Sent: Thursday, June 15, 2006 7:11 AM
> To: openib-general@openib.org
> Cc: Eitan Zahavi
> Subject: [PATCH] [MINOR} OpenSM/osm_port_info_rcv.c: Move assert to
> beforewhere PortInfo is assumed
> 
> OpenSM/osm_port_info_rcv.c: Move assert to before where PortInfo is
> assumed as shouldn't be processing as PortInfo unless it really is
> 
> Signed-off-by: Hal Rosenstock <[EMAIL PROTECTED]>
> 
> Index: opensm/osm_port_info_rcv.c
> ===================================================================
> --- opensm/osm_port_info_rcv.c        (revision 7961)
> +++ opensm/osm_port_info_rcv.c        (working copy)
> @@ -683,6 +683,8 @@ osm_pi_rcv_process(
>    p_context = osm_madw_get_pi_context_ptr( p_madw );
>    p_pi = (ib_port_info_t*)ib_smp_get_payload_ptr( p_smp );
> 
> +  CL_ASSERT( p_smp->attr_id == IB_MAD_ATTR_PORT_INFO );
> +
>    /* On receipt of client reregister, clear the reregister bit so
>       reregistering won't be sent again and again */
>    if ( ib_port_info_get_client_rereg( p_pi ) )
> @@ -698,8 +700,6 @@ osm_pi_rcv_process(
>    port_guid = p_context->port_guid;
>    node_guid = p_context->node_guid;
> 
> -  CL_ASSERT( p_smp->attr_id == IB_MAD_ATTR_PORT_INFO );
> -
>    osm_dump_port_info(
>      p_rcv->p_log, node_guid, port_guid, port_num, p_pi,
OSM_LOG_DEBUG);
> 
> 


_______________________________________________
openib-general mailing list
openib-general@openib.org
http://openib.org/mailman/listinfo/openib-general

To unsubscribe, please visit http://openib.org/mailman/listinfo/openib-general

Reply via email to