[dpdk-dev] [PATCH v1 61/72] net/mlx5/linux: add OS default miss flow action

[Ophir Munk]

Wrap glue call dr_create_flow_action_default_miss() with an OS API. This
commit is a follow up on (1).

(1)
commit d4d85aa6f13a ("common/mlx5: add default miss action")
commit b293fbf9672b ("net/mlx5: add OS specific flow actions operations")

Signed-off-by: Ophir Munk <ophi...@nvidia.com>
---
 drivers/net/mlx5/linux/mlx5_flow_os.h | 16 ++++++++++++++++
 drivers/net/mlx5/mlx5_flow_dv.c       |  9 +++++----
 2 files changed, 21 insertions(+), 4 deletions(-)

diff --git a/drivers/net/mlx5/linux/mlx5_flow_os.h 
b/drivers/net/mlx5/linux/mlx5_flow_os.h
index c7003a1..7706b3b 100644
--- a/drivers/net/mlx5/linux/mlx5_flow_os.h
+++ b/drivers/net/mlx5/linux/mlx5_flow_os.h
@@ -351,6 +351,22 @@ mlx5_flow_os_create_flow_action_drop(void **action)
 }
 
 /**
+ * Create flow action: default miss.
+ *
+ * @param[out] action
+ *   Pointer to a valid action on success, NULL otherwise.
+ *
+ * @return
+ *   0 on success, or -1 on failure and errno is set.
+ */
+static inline int
+mlx5_flow_os_create_flow_action_default_miss(void **action)
+{
+       *action = mlx5_glue->dr_create_flow_action_default_miss();
+       return (*action) ? 0 : -1;
+}
+
+/**
  * Destroy flow action.
  *
  * @param[in] action
diff --git a/drivers/net/mlx5/mlx5_flow_dv.c b/drivers/net/mlx5/mlx5_flow_dv.c
index 18ab409..5c772e7 100644
--- a/drivers/net/mlx5/mlx5_flow_dv.c
+++ b/drivers/net/mlx5/mlx5_flow_dv.c
@@ -2987,12 +2987,13 @@ flow_dv_default_miss_resource_register(struct 
rte_eth_dev *dev,
        struct mlx5_flow_default_miss_resource *cache_resource =
                        &sh->default_miss;
        int cnt = rte_atomic32_read(&cache_resource->refcnt);
+       int ret;
 
        if (!cnt) {
                MLX5_ASSERT(cache_resource->action);
-               cache_resource->action =
-               mlx5_glue->dr_create_flow_action_default_miss();
-               if (!cache_resource->action)
+               ret = mlx5_flow_os_create_flow_action_default_miss
+                                               (&cache_resource->action);
+               if (ret)
                        return rte_flow_error_set(error, ENOMEM,
                                        RTE_FLOW_ERROR_TYPE_UNSPECIFIED, NULL,
                                        "cannot create default miss action");
@@ -10602,7 +10603,7 @@ flow_dv_default_miss_resource_release(struct 
rte_eth_dev *dev)
                        (void *)cache_resource->action,
                        rte_atomic32_read(&cache_resource->refcnt));
        if (rte_atomic32_dec_and_test(&cache_resource->refcnt)) {
-               claim_zero(mlx5_glue->destroy_flow_action
+               claim_zero(mlx5_flow_os_destroy_flow_action
                                (cache_resource->action));
                DRV_LOG(DEBUG, "default miss resource %p: removed",
                                (void *)cache_resource->action);
-- 
2.8.4