net/mlx5: HWS, use list_move() instead of del/add

Wherever applicable, use list_move function instead of list_del + list_add.

Signed-off-by: Yevgeny Kliteynik <kliteyn@nvidia.com>
Reviewed-by: Vlad Dogaru <vdogaru@nvidia.com>
Reviewed-by: Mark Bloch <mbloch@nvidia.com>
Signed-off-by: Tariq Toukan <tariqt@nvidia.com>
Link: https://patch.msgid.link/1741780194-137519-3-git-send-email-tariqt@nvidia.com
Reviewed-by: Dawid Osuchowski <dawid.osuchowski@linux.intel.com>
Reviewed-by: Michal Kubiak <michal.kubiak@intel.com>
Signed-off-by: Paolo Abeni <pabeni@redhat.com>

diff --git a/drivers/net/ethernet/mellanox/mlx5/core/steering/hws/definer.c b/drivers/net/ethernet/mellanox/mlx5/core/steering/hws/definer.c
index 10ece7df1cfaff904b9b9424ee1c32cecd12cfd8..c4851d5584b7546b1a1650a5b1e9e62945568f38 100644 (file)
--- a/drivers/net/ethernet/mellanox/mlx5/core/steering/hws/definer.c
+++ b/drivers/net/ethernet/mellanox/mlx5/core/steering/hws/definer.c
@@ -1985,8 +1985,7 @@ int mlx5hws_definer_get_obj(struct mlx5hws_context *ctx,
                        continue;
 
                /* Reuse definer and set LRU (move to be first in the list) */
-               list_del_init(&cached_definer->list_node);
-               list_add(&cached_definer->list_node, &cache->list_head);
+               list_move(&cached_definer->list_node, &cache->list_head);
                cached_definer->refcount++;
                return cached_definer->definer.obj_id;
        }

diff --git a/drivers/net/ethernet/mellanox/mlx5/core/steering/hws/pat_arg.c b/drivers/net/ethernet/mellanox/mlx5/core/steering/hws/pat_arg.c
index d9dc4f2d0dc680e381c43ad49723302462033733..f51ed24526b91917e2e0a32079b894e8da1f672e 100644 (file)
--- a/drivers/net/ethernet/mellanox/mlx5/core/steering/hws/pat_arg.c
+++ b/drivers/net/ethernet/mellanox/mlx5/core/steering/hws/pat_arg.c
@@ -153,8 +153,7 @@ mlx5hws_pat_get_existing_cached_pattern(struct mlx5hws_pattern_cache *cache,
        cached_pattern = mlx5hws_pat_find_cached_pattern(cache, num_of_actions, actions);
        if (cached_pattern) {
                /* LRU: move it to be first in the list */
-               list_del_init(&cached_pattern->ptrn_list_node);
-               list_add(&cached_pattern->ptrn_list_node, &cache->ptrn_list);
+               list_move(&cached_pattern->ptrn_list_node, &cache->ptrn_list);
                cached_pattern->refcount++;
        }