[1/1] net/mlx5: fix typo in flow engine type
Checks
Commit Message
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: stable@dpdk.org
Signed-off-by: Asaf Penso <asafp@nvidia.com>
---
drivers/net/mlx5/mlx5_flow_dv.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
Comments
From: Asaf Penso
> 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: stable@dpdk.org
>
> Signed-off-by: Asaf Penso <asafp@nvidia.com>
Acked-by: Matan Azrad <matan@nvidia.com>
Hi,
> -----Original Message-----
> From: Asaf Penso <asafp@nvidia.com>
> Sent: Wednesday, July 28, 2021 12:40 PM
> To: dev@dpdk.org
> Cc: NBU-Contact-Thomas Monjalon <thomas@monjalon.net>; Slava
> Ovsiienko <viacheslavo@nvidia.com>; Matan Azrad <matan@nvidia.com>;
> Raslan Darawsheh <rasland@nvidia.com>; stable@dpdk.org
> Subject: [PATCH 1/1] net/mlx5: fix typo in flow engine type
>
> 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: stable@dpdk.org
>
> Signed-off-by: Asaf Penso <asafp@nvidia.com>
> ---
Patch applied to next-net-mlx,
Kindest regards,
Raslan Darawsheh
@@ -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,