net/mlx5: prepare Tx to support scheduling
[dpdk.git] / drivers / net / mlx5 / mlx5_flow_dv.c
index d01a7e5..8b5b683 100644 (file)
@@ -30,7 +30,6 @@
 #include <rte_vxlan.h>
 #include <rte_gtp.h>
 
-#include <mlx5_glue.h>
 #include <mlx5_devx_cmds.h>
 #include <mlx5_prm.h>
 
@@ -1484,6 +1483,13 @@ flow_dv_validate_item_meta(struct rte_eth_dev *dev __rte_unused,
                                          "isn't supported");
                if (reg != REG_A)
                        nic_mask.data = priv->sh->dv_meta_mask;
+       } else if (attr->transfer) {
+               return rte_flow_error_set(error, ENOTSUP,
+                                       RTE_FLOW_ERROR_TYPE_ITEM, item,
+                                       "extended metadata feature "
+                                       "should be enabled when "
+                                       "meta item is requested "
+                                       "with e-switch mode ");
        }
        if (!mask)
                mask = &rte_flow_item_meta_mask;
@@ -2569,6 +2575,7 @@ flow_dv_encap_decap_resource_register
        struct mlx5_flow_dv_encap_decap_resource *cache_resource;
        struct mlx5dv_dr_domain *domain;
        uint32_t idx = 0;
+       int ret;
 
        resource->flags = dev_flow->dv.group ? 0 : 1;
        if (resource->ft_type == MLX5DV_FLOW_TABLE_TYPE_FDB)
@@ -2604,13 +2611,10 @@ flow_dv_encap_decap_resource_register
                                          RTE_FLOW_ERROR_TYPE_UNSPECIFIED, NULL,
                                          "cannot allocate resource memory");
        *cache_resource = *resource;
-       cache_resource->action =
-               mlx5_glue->dv_create_flow_action_packet_reformat
-                       (sh->ctx, cache_resource->reformat_type,
-                        cache_resource->ft_type, domain, cache_resource->flags,
-                        cache_resource->size,
-                        (cache_resource->size ? cache_resource->buf : NULL));
-       if (!cache_resource->action) {
+       ret = mlx5_flow_os_create_flow_action_packet_reformat
+                                       (sh->ctx, domain, cache_resource,
+                                        &cache_resource->action);
+       if (ret) {
                rte_free(cache_resource);
                return rte_flow_error_set(error, ENOMEM,
                                          RTE_FLOW_ERROR_TYPE_UNSPECIFIED,
@@ -2652,15 +2656,14 @@ flow_dv_jump_tbl_resource_register
 {
        struct mlx5_flow_tbl_data_entry *tbl_data =
                container_of(tbl, struct mlx5_flow_tbl_data_entry, tbl);
-       int cnt;
+       int cnt, ret;
 
        MLX5_ASSERT(tbl);
        cnt = rte_atomic32_read(&tbl_data->jump.refcnt);
        if (!cnt) {
-               tbl_data->jump.action =
-                       mlx5_glue->dr_create_flow_action_dest_flow_tbl
-                       (tbl->obj);
-               if (!tbl_data->jump.action)
+               ret = mlx5_flow_os_create_flow_action_dest_flow_tbl
+                               (tbl->obj, &tbl_data->jump.action);
+               if (ret)
                        return rte_flow_error_set(error, ENOMEM,
                                        RTE_FLOW_ERROR_TYPE_UNSPECIFIED,
                                        NULL, "cannot create jump action");
@@ -2741,6 +2744,7 @@ flow_dv_port_id_action_resource_register
        struct mlx5_dev_ctx_shared *sh = priv->sh;
        struct mlx5_flow_dv_port_id_action_resource *cache_resource;
        uint32_t idx = 0;
+       int ret;
 
        /* Lookup a matching resource from cache. */
        ILIST_FOREACH(sh->ipool[MLX5_IPOOL_PORT_ID], sh->port_id_action_list,
@@ -2764,15 +2768,10 @@ flow_dv_port_id_action_resource_register
                                          RTE_FLOW_ERROR_TYPE_UNSPECIFIED, NULL,
                                          "cannot allocate resource memory");
        *cache_resource = *resource;
-       /*
-        * Depending on rdma_core version the glue routine calls
-        * either mlx5dv_dr_action_create_dest_ib_port(domain, dev_port)
-        * or mlx5dv_dr_action_create_dest_vport(domain, vport_id).
-        */
-       cache_resource->action =
-               mlx5_glue->dr_create_flow_action_dest_port
-                       (priv->sh->fdb_domain, resource->port_id);
-       if (!cache_resource->action) {
+       ret = mlx5_flow_os_create_flow_action_dest_port
+                               (priv->sh->fdb_domain, resource->port_id,
+                                &cache_resource->action);
+       if (ret) {
                rte_free(cache_resource);
                return rte_flow_error_set(error, ENOMEM,
                                          RTE_FLOW_ERROR_TYPE_UNSPECIFIED,
@@ -2817,6 +2816,7 @@ flow_dv_push_vlan_action_resource_register
        struct mlx5_flow_dv_push_vlan_action_resource *cache_resource;
        struct mlx5dv_dr_domain *domain;
        uint32_t idx = 0;
+       int ret;
 
        /* Lookup a matching resource from cache. */
        ILIST_FOREACH(sh->ipool[MLX5_IPOOL_PUSH_VLAN],
@@ -2847,10 +2847,10 @@ flow_dv_push_vlan_action_resource_register
                domain = sh->rx_domain;
        else
                domain = sh->tx_domain;
-       cache_resource->action =
-               mlx5_glue->dr_create_flow_action_push_vlan(domain,
-                                                          resource->vlan_tag);
-       if (!cache_resource->action) {
+       ret = mlx5_flow_os_create_flow_action_push_vlan
+                                       (domain, resource->vlan_tag,
+                                        &cache_resource->action);
+       if (ret) {
                rte_free(cache_resource);
                return rte_flow_error_set(error, ENOMEM,
                                          RTE_FLOW_ERROR_TYPE_UNSPECIFIED,
@@ -3991,6 +3991,7 @@ flow_dv_modify_hdr_resource_register
        struct mlx5_flow_dv_modify_hdr_resource *cache_resource;
        struct mlx5dv_dr_domain *ns;
        uint32_t actions_len;
+       int ret;
 
        resource->flags = dev_flow->dv.group ? 0 :
                          MLX5DV_DR_ACTION_FLAGS_ROOT_LEVEL;
@@ -4031,12 +4032,10 @@ flow_dv_modify_hdr_resource_register
                                          "cannot allocate resource memory");
        *cache_resource = *resource;
        rte_memcpy(cache_resource->actions, resource->actions, actions_len);
-       cache_resource->action =
-               mlx5_glue->dv_create_flow_action_modify_header
-                                       (sh->ctx, cache_resource->ft_type, ns,
-                                        cache_resource->flags, actions_len,
-                                        (uint64_t *)cache_resource->actions);
-       if (!cache_resource->action) {
+       ret = mlx5_flow_os_create_flow_action_modify_header
+                                       (sh->ctx, ns, cache_resource,
+                                        actions_len, &cache_resource->action);
+       if (ret) {
                rte_free(cache_resource);
                return rte_flow_error_set(error, ENOMEM,
                                          RTE_FLOW_ERROR_TYPE_UNSPECIFIED,
@@ -4621,6 +4620,7 @@ flow_dv_counter_alloc(struct rte_eth_dev *dev, uint32_t shared, uint32_t id,
        if (!cnt_free->action) {
                uint16_t offset;
                struct mlx5_devx_obj *dcs;
+               int ret;
 
                if (batch) {
                        offset = MLX5_CNT_ARRAY_IDX(pool, cnt_free);
@@ -4629,9 +4629,9 @@ flow_dv_counter_alloc(struct rte_eth_dev *dev, uint32_t shared, uint32_t id,
                        offset = 0;
                        dcs = cnt_ext->dcs;
                }
-               cnt_free->action = mlx5_glue->dv_create_flow_action_counter
-                                       (dcs->obj, offset);
-               if (!cnt_free->action) {
+               ret = mlx5_flow_os_create_flow_action_count(dcs->obj, offset,
+                                                           &cnt_free->action);
+               if (ret) {
                        rte_errno = errno;
                        goto err;
                }
@@ -7340,8 +7340,8 @@ flow_dv_tbl_resource_get(struct rte_eth_dev *dev,
                domain = sh->tx_domain;
        else
                domain = sh->rx_domain;
-       tbl->obj = mlx5_glue->dr_create_flow_tbl(domain, table_id);
-       if (!tbl->obj) {
+       ret = mlx5_flow_os_create_flow_tbl(domain, table_id, &tbl->obj);
+       if (ret) {
                rte_flow_error_set(error, ENOMEM,
                                   RTE_FLOW_ERROR_TYPE_UNSPECIFIED,
                                   NULL, "cannot create flow table object");
@@ -7361,7 +7361,7 @@ flow_dv_tbl_resource_get(struct rte_eth_dev *dev,
                rte_flow_error_set(error, -ret,
                                   RTE_FLOW_ERROR_TYPE_UNSPECIFIED, NULL,
                                   "cannot insert flow table data entry");
-               mlx5_glue->dr_destroy_flow_tbl(tbl->obj);
+               mlx5_flow_os_destroy_flow_tbl(tbl->obj);
                mlx5_ipool_free(sh->ipool[MLX5_IPOOL_JUMP], idx);
        }
        rte_atomic32_inc(&tbl->refcnt);
@@ -7393,7 +7393,7 @@ flow_dv_tbl_resource_release(struct rte_eth_dev *dev,
        if (rte_atomic32_dec_and_test(&tbl->refcnt)) {
                struct mlx5_hlist_entry *pos = &tbl_data->entry;
 
-               mlx5_glue->dr_destroy_flow_tbl(tbl->obj);
+               mlx5_flow_os_destroy_flow_tbl(tbl->obj);
                tbl->obj = NULL;
                /* remove the entry from the hash list and free memory. */
                mlx5_hlist_remove(sh->flow_tbls, pos);
@@ -7437,6 +7437,7 @@ flow_dv_matcher_register(struct rte_eth_dev *dev,
        };
        struct mlx5_flow_tbl_resource *tbl;
        struct mlx5_flow_tbl_data_entry *tbl_data;
+       int ret;
 
        tbl = flow_dv_tbl_resource_get(dev, key->table_id, key->direction,
                                       key->domain, error);
@@ -7479,9 +7480,9 @@ flow_dv_matcher_register(struct rte_eth_dev *dev,
        dv_attr.priority = matcher->priority;
        if (key->direction)
                dv_attr.flags |= IBV_FLOW_ATTR_FLAGS_EGRESS;
-       cache_matcher->matcher_object =
-               mlx5_glue->dv_create_flow_matcher(sh->ctx, &dv_attr, tbl->obj);
-       if (!cache_matcher->matcher_object) {
+       ret = mlx5_flow_os_create_flow_matcher(sh->ctx, &dv_attr, tbl->obj,
+                                              &cache_matcher->matcher_object);
+       if (ret) {
                rte_free(cache_matcher);
 #ifdef HAVE_MLX5DV_DR
                flow_dv_tbl_resource_release(dev, tbl);
@@ -7531,6 +7532,7 @@ flow_dv_tag_resource_register
        struct mlx5_dev_ctx_shared *sh = priv->sh;
        struct mlx5_flow_dv_tag_resource *cache_resource;
        struct mlx5_hlist_entry *entry;
+       int ret;
 
        /* Lookup a matching resource from cache. */
        entry = mlx5_hlist_lookup(sh->tag_table, (uint64_t)tag_be24);
@@ -7553,8 +7555,9 @@ flow_dv_tag_resource_register
                                          RTE_FLOW_ERROR_TYPE_UNSPECIFIED, NULL,
                                          "cannot allocate resource memory");
        cache_resource->entry.key = (uint64_t)tag_be24;
-       cache_resource->action = mlx5_glue->dv_create_flow_action_tag(tag_be24);
-       if (!cache_resource->action) {
+       ret = mlx5_flow_os_create_flow_action_tag(tag_be24,
+                                                 &cache_resource->action);
+       if (ret) {
                rte_free(cache_resource);
                return rte_flow_error_set(error, ENOMEM,
                                          RTE_FLOW_ERROR_TYPE_UNSPECIFIED,
@@ -7563,7 +7566,7 @@ flow_dv_tag_resource_register
        rte_atomic32_init(&cache_resource->refcnt);
        rte_atomic32_inc(&cache_resource->refcnt);
        if (mlx5_hlist_insert(sh->tag_table, &cache_resource->entry)) {
-               mlx5_glue->destroy_flow_action(cache_resource->action);
+               mlx5_flow_os_destroy_flow_action(cache_resource->action);
                rte_free(cache_resource);
                return rte_flow_error_set(error, EEXIST,
                                          RTE_FLOW_ERROR_TYPE_UNSPECIFIED,
@@ -7602,7 +7605,7 @@ flow_dv_tag_release(struct rte_eth_dev *dev,
                dev->data->port_id, (void *)tag,
                rte_atomic32_read(&tag->refcnt));
        if (rte_atomic32_dec_and_test(&tag->refcnt)) {
-               claim_zero(mlx5_glue->destroy_flow_action(tag->action));
+               claim_zero(mlx5_flow_os_destroy_flow_action(tag->action));
                mlx5_hlist_remove(sh->tag_table, &tag->entry);
                DRV_LOG(DEBUG, "port %u tag %p: removed",
                        dev->data->port_id, (void *)tag);
@@ -8693,11 +8696,10 @@ __flow_dv_apply(struct rte_eth_dev *dev, struct rte_flow *flow,
                        dh->rix_default_fate =  MLX5_FLOW_FATE_DEFAULT_MISS;
                        dv->actions[n++] = priv->sh->default_miss.action;
                }
-               dh->drv_flow =
-                       mlx5_glue->dv_create_flow(dv_h->matcher->matcher_object,
-                                                 (void *)&dv->value, n,
-                                                 dv->actions);
-               if (!dh->drv_flow) {
+               err = mlx5_flow_os_create_flow(dv_h->matcher->matcher_object,
+                                              (void *)&dv->value, n,
+                                              dv->actions, &dh->drv_flow);
+               if (err) {
                        rte_flow_error_set(error, errno,
                                           RTE_FLOW_ERROR_TYPE_UNSPECIFIED,
                                           NULL,
@@ -8762,7 +8764,7 @@ flow_dv_matcher_release(struct rte_eth_dev *dev,
                dev->data->port_id, (void *)matcher,
                rte_atomic32_read(&matcher->refcnt));
        if (rte_atomic32_dec_and_test(&matcher->refcnt)) {
-               claim_zero(mlx5_glue->dv_destroy_flow_matcher
+               claim_zero(mlx5_flow_os_destroy_flow_matcher
                           (matcher->matcher_object));
                LIST_REMOVE(matcher, next);
                /* table ref-- in release interface. */
@@ -8803,8 +8805,8 @@ flow_dv_encap_decap_resource_release(struct rte_eth_dev *dev,
                (void *)cache_resource,
                rte_atomic32_read(&cache_resource->refcnt));
        if (rte_atomic32_dec_and_test(&cache_resource->refcnt)) {
-               claim_zero(mlx5_glue->destroy_flow_action
-                               (cache_resource->action));
+               claim_zero(mlx5_flow_os_destroy_flow_action
+                                               (cache_resource->action));
                ILIST_REMOVE(priv->sh->ipool[MLX5_IPOOL_DECAP_ENCAP],
                             &priv->sh->encaps_decaps, idx,
                             cache_resource, next);
@@ -8845,8 +8847,8 @@ flow_dv_jump_tbl_resource_release(struct rte_eth_dev *dev,
                (void *)cache_resource,
                rte_atomic32_read(&cache_resource->refcnt));
        if (rte_atomic32_dec_and_test(&cache_resource->refcnt)) {
-               claim_zero(mlx5_glue->destroy_flow_action
-                               (cache_resource->action));
+               claim_zero(mlx5_flow_os_destroy_flow_action
+                                               (cache_resource->action));
                /* jump action memory free is inside the table release. */
                flow_dv_tbl_resource_release(dev, &tbl_data->tbl);
                DRV_LOG(DEBUG, "jump table resource %p: removed",
@@ -8906,8 +8908,8 @@ flow_dv_modify_hdr_resource_release(struct mlx5_flow_handle *handle)
                (void *)cache_resource,
                rte_atomic32_read(&cache_resource->refcnt));
        if (rte_atomic32_dec_and_test(&cache_resource->refcnt)) {
-               claim_zero(mlx5_glue->destroy_flow_action
-                               (cache_resource->action));
+               claim_zero(mlx5_flow_os_destroy_flow_action
+                                               (cache_resource->action));
                LIST_REMOVE(cache_resource, next);
                rte_free(cache_resource);
                DRV_LOG(DEBUG, "modify-header resource %p: removed",
@@ -8945,8 +8947,8 @@ flow_dv_port_id_action_resource_release(struct rte_eth_dev *dev,
                (void *)cache_resource,
                rte_atomic32_read(&cache_resource->refcnt));
        if (rte_atomic32_dec_and_test(&cache_resource->refcnt)) {
-               claim_zero(mlx5_glue->destroy_flow_action
-                               (cache_resource->action));
+               claim_zero(mlx5_flow_os_destroy_flow_action
+                                               (cache_resource->action));
                ILIST_REMOVE(priv->sh->ipool[MLX5_IPOOL_PORT_ID],
                             &priv->sh->port_id_action_list, idx,
                             cache_resource, next);
@@ -8986,8 +8988,8 @@ flow_dv_push_vlan_action_resource_release(struct rte_eth_dev *dev,
                (void *)cache_resource,
                rte_atomic32_read(&cache_resource->refcnt));
        if (rte_atomic32_dec_and_test(&cache_resource->refcnt)) {
-               claim_zero(mlx5_glue->destroy_flow_action
-                               (cache_resource->action));
+               claim_zero(mlx5_flow_os_destroy_flow_action
+                                               (cache_resource->action));
                ILIST_REMOVE(priv->sh->ipool[MLX5_IPOOL_PUSH_VLAN],
                             &priv->sh->push_vlan_action_list, idx,
                             cache_resource, next);
@@ -9061,7 +9063,7 @@ __flow_dv_remove(struct rte_eth_dev *dev, struct rte_flow *flow)
                if (!dh)
                        return;
                if (dh->drv_flow) {
-                       claim_zero(mlx5_glue->dv_destroy_flow(dh->drv_flow));
+                       claim_zero(mlx5_flow_os_destroy_flow(dh->drv_flow));
                        dh->drv_flow = NULL;
                }
                if (dh->fate_action == MLX5_FLOW_FATE_DROP ||
@@ -9242,46 +9244,46 @@ flow_dv_destroy_mtr_tbl(struct rte_eth_dev *dev,
        if (!mtd || !priv->config.dv_flow_en)
                return 0;
        if (mtd->ingress.policer_rules[RTE_MTR_DROPPED])
-               claim_zero(mlx5_glue->dv_destroy_flow
-                         (mtd->ingress.policer_rules[RTE_MTR_DROPPED]));
+               claim_zero(mlx5_flow_os_destroy_flow
+                          (mtd->ingress.policer_rules[RTE_MTR_DROPPED]));
        if (mtd->egress.policer_rules[RTE_MTR_DROPPED])
-               claim_zero(mlx5_glue->dv_destroy_flow
-                         (mtd->egress.policer_rules[RTE_MTR_DROPPED]));
+               claim_zero(mlx5_flow_os_destroy_flow
+                          (mtd->egress.policer_rules[RTE_MTR_DROPPED]));
        if (mtd->transfer.policer_rules[RTE_MTR_DROPPED])
-               claim_zero(mlx5_glue->dv_destroy_flow
-                         (mtd->transfer.policer_rules[RTE_MTR_DROPPED]));
+               claim_zero(mlx5_flow_os_destroy_flow
+                          (mtd->transfer.policer_rules[RTE_MTR_DROPPED]));
        if (mtd->egress.color_matcher)
-               claim_zero(mlx5_glue->dv_destroy_flow_matcher
-                         (mtd->egress.color_matcher));
+               claim_zero(mlx5_flow_os_destroy_flow_matcher
+                          (mtd->egress.color_matcher));
        if (mtd->egress.any_matcher)
-               claim_zero(mlx5_glue->dv_destroy_flow_matcher
-                         (mtd->egress.any_matcher));
+               claim_zero(mlx5_flow_os_destroy_flow_matcher
+                          (mtd->egress.any_matcher));
        if (mtd->egress.tbl)
                flow_dv_tbl_resource_release(dev, mtd->egress.tbl);
        if (mtd->egress.sfx_tbl)
                flow_dv_tbl_resource_release(dev, mtd->egress.sfx_tbl);
        if (mtd->ingress.color_matcher)
-               claim_zero(mlx5_glue->dv_destroy_flow_matcher
-                         (mtd->ingress.color_matcher));
+               claim_zero(mlx5_flow_os_destroy_flow_matcher
+                          (mtd->ingress.color_matcher));
        if (mtd->ingress.any_matcher)
-               claim_zero(mlx5_glue->dv_destroy_flow_matcher
-                         (mtd->ingress.any_matcher));
+               claim_zero(mlx5_flow_os_destroy_flow_matcher
+                          (mtd->ingress.any_matcher));
        if (mtd->ingress.tbl)
                flow_dv_tbl_resource_release(dev, mtd->ingress.tbl);
        if (mtd->ingress.sfx_tbl)
                flow_dv_tbl_resource_release(dev, mtd->ingress.sfx_tbl);
        if (mtd->transfer.color_matcher)
-               claim_zero(mlx5_glue->dv_destroy_flow_matcher
-                         (mtd->transfer.color_matcher));
+               claim_zero(mlx5_flow_os_destroy_flow_matcher
+                          (mtd->transfer.color_matcher));
        if (mtd->transfer.any_matcher)
-               claim_zero(mlx5_glue->dv_destroy_flow_matcher
-                         (mtd->transfer.any_matcher));
+               claim_zero(mlx5_flow_os_destroy_flow_matcher
+                          (mtd->transfer.any_matcher));
        if (mtd->transfer.tbl)
                flow_dv_tbl_resource_release(dev, mtd->transfer.tbl);
        if (mtd->transfer.sfx_tbl)
                flow_dv_tbl_resource_release(dev, mtd->transfer.sfx_tbl);
        if (mtd->drop_actn)
-               claim_zero(mlx5_glue->destroy_flow_action(mtd->drop_actn));
+               claim_zero(mlx5_flow_os_destroy_flow_action(mtd->drop_actn));
        rte_free(mtd);
        return 0;
 }
@@ -9330,6 +9332,7 @@ flow_dv_prepare_mtr_tables(struct rte_eth_dev *dev,
        struct mlx5_meter_domain_info *dtb;
        struct rte_flow_error error;
        int i = 0;
+       int ret;
 
        if (transfer)
                dtb = &mtb->transfer;
@@ -9355,10 +9358,9 @@ flow_dv_prepare_mtr_tables(struct rte_eth_dev *dev,
        /* Create matchers, Any and Color. */
        dv_attr.priority = 3;
        dv_attr.match_criteria_enable = 0;
-       dtb->any_matcher = mlx5_glue->dv_create_flow_matcher(sh->ctx,
-                                                            &dv_attr,
-                                                            dtb->tbl->obj);
-       if (!dtb->any_matcher) {
+       ret = mlx5_flow_os_create_flow_matcher(sh->ctx, &dv_attr, dtb->tbl->obj,
+                                              &dtb->any_matcher);
+       if (ret) {
                DRV_LOG(ERR, "Failed to create meter"
                             " policer default matcher.");
                goto error_exit;
@@ -9368,10 +9370,9 @@ flow_dv_prepare_mtr_tables(struct rte_eth_dev *dev,
                                1 << MLX5_MATCH_CRITERIA_ENABLE_MISC2_BIT;
        flow_dv_match_meta_reg(mask.buf, value.buf, color_reg_c_idx,
                               rte_col_2_mlx5_col(RTE_COLORS), UINT8_MAX);
-       dtb->color_matcher = mlx5_glue->dv_create_flow_matcher(sh->ctx,
-                                                              &dv_attr,
-                                                              dtb->tbl->obj);
-       if (!dtb->color_matcher) {
+       ret = mlx5_flow_os_create_flow_matcher(sh->ctx, &dv_attr, dtb->tbl->obj,
+                                              &dtb->color_matcher);
+       if (ret) {
                DRV_LOG(ERR, "Failed to create meter policer color matcher.");
                goto error_exit;
        }
@@ -9379,10 +9380,10 @@ flow_dv_prepare_mtr_tables(struct rte_eth_dev *dev,
                actions[i++] = mtb->count_actns[RTE_MTR_DROPPED];
        actions[i++] = mtb->drop_actn;
        /* Default rule: lowest priority, match any, actions: drop. */
-       dtb->policer_rules[RTE_MTR_DROPPED] =
-                       mlx5_glue->dv_create_flow(dtb->any_matcher,
-                                                (void *)&value, i, actions);
-       if (!dtb->policer_rules[RTE_MTR_DROPPED]) {
+       ret = mlx5_flow_os_create_flow(dtb->any_matcher, (void *)&value, i,
+                                      actions,
+                                      &dtb->policer_rules[RTE_MTR_DROPPED]);
+       if (ret) {
                DRV_LOG(ERR, "Failed to create meter policer drop rule.");
                goto error_exit;
        }
@@ -9431,8 +9432,8 @@ flow_dv_create_mtr_tbl(struct rte_eth_dev *dev,
                mtb->count_actns[i] = cnt->action;
        }
        /* Create drop action. */
-       mtb->drop_actn = mlx5_glue->dr_create_flow_action_drop();
-       if (!mtb->drop_actn) {
+       ret = mlx5_flow_os_create_flow_action_drop(&mtb->drop_actn);
+       if (ret) {
                DRV_LOG(ERR, "Failed to create drop action.");
                goto error_exit;
        }
@@ -9476,13 +9477,13 @@ flow_dv_destroy_domain_policer_rule(struct mlx5_meter_domain_info *dt)
 
        for (i = 0; i < RTE_MTR_DROPPED; i++) {
                if (dt->policer_rules[i]) {
-                       claim_zero(mlx5_glue->dv_destroy_flow
-                                 (dt->policer_rules[i]));
+                       claim_zero(mlx5_flow_os_destroy_flow
+                                  (dt->policer_rules[i]));
                        dt->policer_rules[i] = NULL;
                }
        }
        if (dt->jump_actn) {
-               claim_zero(mlx5_glue->destroy_flow_action(dt->jump_actn));
+               claim_zero(mlx5_flow_os_destroy_flow_action(dt->jump_actn));
                dt->jump_actn = NULL;
        }
 }
@@ -9545,13 +9546,13 @@ flow_dv_create_policer_forward_rule(struct mlx5_flow_meter *fm,
        struct mlx5_meter_domains_infos *mtb = fm->mfts;
        void *actions[METER_ACTIONS];
        int i;
+       int ret = 0;
 
        /* Create jump action. */
        if (!dtb->jump_actn)
-               dtb->jump_actn =
-                       mlx5_glue->dr_create_flow_action_dest_flow_tbl
-                                                       (dtb->sfx_tbl->obj);
-       if (!dtb->jump_actn) {
+               ret = mlx5_flow_os_create_flow_action_dest_flow_tbl
+                               (dtb->sfx_tbl->obj, &dtb->jump_actn);
+       if (ret) {
                DRV_LOG(ERR, "Failed to create policer jump action.");
                goto error;
        }
@@ -9566,11 +9567,10 @@ flow_dv_create_policer_forward_rule(struct mlx5_flow_meter *fm,
                        actions[j++] = mtb->drop_actn;
                else
                        actions[j++] = dtb->jump_actn;
-               dtb->policer_rules[i] =
-                       mlx5_glue->dv_create_flow(dtb->color_matcher,
-                                                (void *)&value,
-                                                 j, actions);
-               if (!dtb->policer_rules[i]) {
+               ret = mlx5_flow_os_create_flow(dtb->color_matcher,
+                                              (void *)&value, j, actions,
+                                              &dtb->policer_rules[i]);
+               if (ret) {
                        DRV_LOG(ERR, "Failed to create policer rule.");
                        goto error;
                }