[openib-general] [PATCH] osm: setting LinkSpeedSupported as LinkSpeedEnabled only when needed

[Hal Rosenstock]

Hi Yevgeny,

On Sun, 2006-11-26 at 10:10, Yevgeny Kliteynik wrote:
> Setting LinkSpeedSupported as LinkSpeedEnabled only when needed - 
> avoiding unnecessary changes in PortInfo to prevent issuing SetPortInfo
> 
> --
> Yevgeny
> 
> Signed-off-by:  Yevgeny Kliteynik <kliteyn at dev.mellanox.co.il>
>  
> Index: opensm/osm_link_mgr.c
> ===================================================================
> --- opensm/osm_link_mgr.c	(revision 10161)
> +++ opensm/osm_link_mgr.c	(working copy)
> @@ -313,7 +313,11 @@ __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_LINK_SPEED_ENABLED_MASK );
> +    {
> +       if (ib_port_info_get_link_speed_sup(p_pi) != 
> +           ib_port_info_get_link_speed_enabled(p_pi))
> +          ib_port_info_set_link_speed_enabled( p_pi, IB_PORT_LINK_SPEED_ENABLED_MASK );
> +    }

This change looks fine.

Is a similar change needed also in osm_lid_mgr.c:
__osm_lid_mgr_set_physp_pi ?

-- Hal

>      if (memcmp( &p_pi->link_speed, &p_old_pi->link_speed,
>                  sizeof(p_pi->link_speed) ))
>        send_set = TRUE;