mbox series

[net-next,0/3] net/mlx5: HW Steering cleanups

Message ID 1741780194-137519-1-git-send-email-tariqt@nvidia.com (mailing list archive)
Headers show
Series net/mlx5: HW Steering cleanups | expand

Message

Tariq Toukan March 12, 2025, 11:49 a.m. UTC 
This short series by Yevgeny contains several small HW Steering cleanups:

- Patch 1: removing unused FW commands
- Patch 2: using list_move() instead of list_del/add
- Patch 3: printing the unsupported combination of match fields

Regards,
Tariq

Yevgeny Kliteynik (3):
  net/mlx5: HWS, remove unused code for alias flow tables
  net/mlx5: HWS, use list_move() instead of del/add
  net/mlx5: HWS, log the unsupported mask in definer

 drivers/net/ethernet/mellanox/mlx5/core/steering/hws/cmd.c  | 6 ------
 drivers/net/ethernet/mellanox/mlx5/core/steering/hws/cmd.h  | 3 ---
 .../net/ethernet/mellanox/mlx5/core/steering/hws/definer.c  | 6 +++---
 .../net/ethernet/mellanox/mlx5/core/steering/hws/pat_arg.c  | 3 +--
 4 files changed, 4 insertions(+), 14 deletions(-)


base-commit: 0ea09cbf8350b70ad44d67a1dcb379008a356034

Comments

Dawid Osuchowski March 12, 2025, 12:32 p.m. UTC | #1 
On 2025-03-12 12:49 PM, Tariq Toukan wrote:
> This short series by Yevgeny contains several small HW Steering cleanups:
> 
> - Patch 1: removing unused FW commands
> - Patch 2: using list_move() instead of list_del/add
> - Patch 3: printing the unsupported combination of match fields
> 
> Regards,
> Tariq
> 
> Yevgeny Kliteynik (3):
>    net/mlx5: HWS, remove unused code for alias flow tables
>    net/mlx5: HWS, use list_move() instead of del/add
>    net/mlx5: HWS, log the unsupported mask in definer
> 
>   drivers/net/ethernet/mellanox/mlx5/core/steering/hws/cmd.c  | 6 ------
>   drivers/net/ethernet/mellanox/mlx5/core/steering/hws/cmd.h  | 3 ---
>   .../net/ethernet/mellanox/mlx5/core/steering/hws/definer.c  | 6 +++---
>   .../net/ethernet/mellanox/mlx5/core/steering/hws/pat_arg.c  | 3 +--
>   4 files changed, 4 insertions(+), 14 deletions(-)
> 
> 
> base-commit: 0ea09cbf8350b70ad44d67a1dcb379008a356034

Hi, thanks for the submission!

Reviewed-by: Dawid Osuchowski <dawid.osuchowski@linux.intel.com>
for the whole series
Michal Kubiak March 12, 2025, 12:40 p.m. UTC | #2 
On Wed, Mar 12, 2025 at 01:49:51PM +0200, Tariq Toukan wrote:
> This short series by Yevgeny contains several small HW Steering cleanups:
> 
> - Patch 1: removing unused FW commands
> - Patch 2: using list_move() instead of list_del/add
> - Patch 3: printing the unsupported combination of match fields
> 
> Regards,
> Tariq
> 
> Yevgeny Kliteynik (3):
>   net/mlx5: HWS, remove unused code for alias flow tables
>   net/mlx5: HWS, use list_move() instead of del/add
>   net/mlx5: HWS, log the unsupported mask in definer
> 
>  drivers/net/ethernet/mellanox/mlx5/core/steering/hws/cmd.c  | 6 ------
>  drivers/net/ethernet/mellanox/mlx5/core/steering/hws/cmd.h  | 3 ---
>  .../net/ethernet/mellanox/mlx5/core/steering/hws/definer.c  | 6 +++---
>  .../net/ethernet/mellanox/mlx5/core/steering/hws/pat_arg.c  | 3 +--
>  4 files changed, 4 insertions(+), 14 deletions(-)
> 
> 
> base-commit: 0ea09cbf8350b70ad44d67a1dcb379008a356034
> -- 
> 2.31.1
> 
> 

The series looks OK.

Thanks,
Michal

For the series:
Reviewed-by: Michal Kubiak <michal.kubiak@intel.com>