[openib-general] [PATCH] OpenSM: When not forcing link speed, use LinkSpeedSupported as LinkSpeedEnabled

[Eitan Zahavi]

Hi Hal,

What the patch does is to force the LinkSpeedEnabled to 
IB_PORT_LINK_SPEED_ENABLED_MASK.
The text implies that this would make the LinkSpeedEnabled  be equal to 
LinkSpeedSupported.
What I do not get is how come a static mask can represent a port 
specific attribute like LinkSpeedSupported.

Eitan

Hal Rosenstock wrote:
> OpenSM: When not forcing link speed, use LinkSpeedSupported as
> LinkSpeedEnabled.
>
> Signed-off-by: Hal Rosenstock <halr at voltaire.com> 
> Index: opensm/osm_lid_mgr.c
> ===================================================================
> -- opensm/osm_lid_mgr.c	(revision 10056)
> +++ opensm/osm_lid_mgr.c	(working copy)
> @@ -1155,7 +1155,7 @@ __osm_lid_mgr_set_physp_pi(
>      if ( p_mgr->p_subn->opt.force_link_speed )
>        ib_port_info_set_link_speed_enabled( p_pi, IB_LINK_SPEED_ACTIVE_2_5 );
>      else
> -      ib_port_info_set_link_speed_enabled( p_pi, ib_port_info_get_link_speed_enabled(p_old_pi) );
> +      ib_port_info_set_link_speed_enabled( p_pi, IB_PORT_LINK_SPEED_ENABLED_MASK );
>      if (memcmp( &p_pi->link_speed, &p_old_pi->link_speed,
>                  sizeof(p_pi->link_speed) ))
>        send_set = TRUE;
> Index: opensm/osm_link_mgr.c
> ===================================================================
> -- opensm/osm_link_mgr.c	(revision 10056)
> +++ opensm/osm_link_mgr.c	(working copy)
> @@ -313,7 +313,7 @@ __osm_link_mgr_set_physp_pi(
>      if ( p_mgr->p_subn->opt.force_link_speed )
>        ib_port_info_set_link_speed_enabled( p_pi, IB_LINK_SPEED_ACTIVE_2_5 );
>      else
> -      ib_port_info_set_link_speed_enabled( p_pi, ib_port_info_get_link_speed_enabled(p_old_pi) );
> +      ib_port_info_set_link_speed_enabled( p_pi, IB_PORT_LINK_SPEED_ENABLED_MASK );
>      if (memcmp( &p_pi->link_speed, &p_old_pi->link_speed,
>                  sizeof(p_pi->link_speed) ))
>        send_set = TRUE;
>
>
>
> _______________________________________________
> openib-general mailing list
> openib-general at openib.org
> http://openib.org/mailman/listinfo/openib-general
>
> To unsubscribe, please visit http://openib.org/mailman/listinfo/openib-general
>