[ofa-general] Re: [PATCH] opensm/osm_perfmgr.c: In perfmgr_send_pc_mad, only set CounterSelect when Set method is used

[Sasha Khapyorsky]

On 10:04 Tue 14 Jul     , Hal Rosenstock wrote:
> 
> Signed-off-by: Hal Rosenstock <hal.rosenstock at gmail.com>
> ---
> diff --git a/opensm/opensm/osm_perfmgr.c b/opensm/opensm/osm_perfmgr.c
> index ecfdbda..0437d47 100644
> --- a/opensm/opensm/osm_perfmgr.c
> +++ b/opensm/opensm/osm_perfmgr.c
> @@ -376,7 +376,8 @@ static ib_api_status_t perfmgr_send_pc_mad(osm_perfmgr_t * perfmgr,
>  	port_counter = (ib_port_counters_t *) & pm_mad->data;
>  	memset(port_counter, 0, sizeof(*port_counter));
>  	port_counter->port_select = port;
> -	port_counter->counter_select = 0xFFFF;
> +	if (mad_method == IB_MAD_METHOD_SET)
> +		port_counter->counter_select = 0xFFFF;

Could you explain why?

Sasha

>  
>  	p_madw->mad_addr.dest_lid = dest_lid;
>  	p_madw->mad_addr.addr_type.gsi.remote_qp = dest_qp;
>