Re: [PATCH net 2/2] net/mlx5: HWS, fix flow meter offset within its bulk

[Tariq Toukan <tariqt@nvidia.com>]

On 08/10/2026 18:59, Moshe Shemesh wrote:
> 
> 
> On 10/7/2026 5:38 PM, Alexandre Cassen wrote:
>>
>> The flow meter action is bound to the first ASO object of the bulk but
>> the rule offset only indexes the meter within its own object. A rule
>> past the first object runs on the wrong meter.
>>
>> Fixes: 82d3639ef7dc ("net/mlx5: fs, add support for flow meters HWS 
>> action")
>> Signed-off-by: Alexandre Cassen <acassen@corp.free.fr>
> 
> Reviewed-by: Moshe Shemesh <moshe@nvidia.com>
> 
> Thanks.

Acked-by: Tariq Toukan <tariqt@nvidia.com>


>> ---
>>   drivers/net/ethernet/mellanox/mlx5/core/steering/hws/fs_hws.c | 1 +
>>   1 file changed, 1 insertion(+)
>>
>> diff --git a/drivers/net/ethernet/mellanox/mlx5/core/steering/hws/ 
>> fs_hws.c b/drivers/net/ethernet/mellanox/mlx5/core/steering/hws/fs_hws.c
>> index 44b311b4532d..54761754e657 100644
>> --- a/drivers/net/ethernet/mellanox/mlx5/core/steering/hws/fs_hws.c
>> +++ b/drivers/net/ethernet/mellanox/mlx5/core/steering/hws/fs_hws.c
>> @@ -928,6 +928,7 @@ static int mlx5_fs_fte_get_hws_actions(struct 
>> mlx5_flow_root_namespace *ns,
>>                          goto free_actions;
>>                  }
>>                  (*ractions)[num_actions].aso_meter.offset =
>> +                       (fte_action->exe_aso.object_id - fte_action- 
>> >exe_aso.base_id) * 2 +
>>                          fte_action->exe_aso.flow_meter.meter_idx;
>>                  (*ractions)[num_actions].aso_meter.init_color =
>>                          fte_action->exe_aso.flow_meter.init_color;
>> -- 
>> 2.43.0
>>
>