The concrete function names have a prefix for flow_dv.
This emphasizes the flow engine is direct verbs.

The function flow_get_aged_flows doesn’t have this prefix.
It creates an inconsistency with the other functions.

Update the function name to include dv.

Fixes: fa2d01c87d2 ("net/mlx5: support flow aging")
Cc: sta...@dpdk.org

Signed-off-by: Asaf Penso <as...@nvidia.com>
---
 drivers/net/mlx5/mlx5_flow_dv.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/net/mlx5/mlx5_flow_dv.c b/drivers/net/mlx5/mlx5_flow_dv.c
index 736227bc0c..4b569b0bdb 100644
--- a/drivers/net/mlx5/mlx5_flow_dv.c
+++ b/drivers/net/mlx5/mlx5_flow_dv.c
@@ -17231,7 +17231,7 @@ flow_dv_counter_query(struct rte_eth_dev *dev, uint32_t 
counter, bool clear,
  * @note: only stub for now
  */
 static int
-flow_get_aged_flows(struct rte_eth_dev *dev,
+flow_dv_get_aged_flows(struct rte_eth_dev *dev,
                    void **context,
                    uint32_t nb_contexts,
                    struct rte_flow_error *error)
@@ -17854,7 +17854,7 @@ const struct mlx5_flow_driver_ops mlx5_flow_dv_drv_ops 
= {
        .counter_alloc = flow_dv_counter_allocate,
        .counter_free = flow_dv_counter_free,
        .counter_query = flow_dv_counter_query,
-       .get_aged_flows = flow_get_aged_flows,
+       .get_aged_flows = flow_dv_get_aged_flows,
        .action_validate = flow_dv_action_validate,
        .action_create = flow_dv_action_create,
        .action_destroy = flow_dv_action_destroy,
-- 
2.18.2

Reply via email to