common/mlx5: remove inclusion of Verbs header files
[dpdk.git] / drivers / net / mlx5 / mlx5_flow_dv.c
index eb27595..778a766 100644 (file)
@@ -8,16 +8,6 @@
 #include <string.h>
 #include <unistd.h>
 
-/* Verbs header. */
-/* ISO C doesn't support unnamed structs/unions, disabling -pedantic. */
-#ifdef PEDANTIC
-#pragma GCC diagnostic ignored "-Wpedantic"
-#endif
-#include <infiniband/verbs.h>
-#ifdef PEDANTIC
-#pragma GCC diagnostic error "-Wpedantic"
-#endif
-
 #include <rte_common.h>
 #include <rte_ether.h>
 #include <rte_ethdev_driver.h>
 #include <rte_gre.h>
 #include <rte_vxlan.h>
 #include <rte_gtp.h>
+#include <rte_eal_paging.h>
 
 #include <mlx5_glue.h>
 #include <mlx5_devx_cmds.h>
 #include <mlx5_prm.h>
+#include <mlx5_malloc.h>
 
 #include "mlx5_defs.h"
 #include "mlx5.h"
@@ -1484,6 +1476,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;
@@ -2422,6 +2421,11 @@ flow_dv_validate_action_decap(struct rte_eth_dev *dev,
 {
        const struct mlx5_priv *priv = dev->data->dev_private;
 
+       if (priv->config.hca_attr.scatter_fcs_w_decap_disable &&
+           !priv->config.decap_en)
+               return rte_flow_error_set(error, ENOTSUP,
+                                         RTE_FLOW_ERROR_TYPE_ACTION, NULL,
+                                         "decap is not enabled");
        if (action_flags & MLX5_FLOW_XCAP_ACTIONS)
                return rte_flow_error_set(error, ENOTSUP,
                                          RTE_FLOW_ERROR_TYPE_ACTION, NULL,
@@ -2569,6 +2573,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,14 +2609,11 @@ 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) {
-               rte_free(cache_resource);
+       ret = mlx5_flow_os_create_flow_action_packet_reformat
+                                       (sh->ctx, domain, cache_resource,
+                                        &cache_resource->action);
+       if (ret) {
+               mlx5_free(cache_resource);
                return rte_flow_error_set(error, ENOMEM,
                                          RTE_FLOW_ERROR_TYPE_UNSPECIFIED,
                                          NULL, "cannot create action");
@@ -2652,15 +2654,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 +2742,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,16 +2766,11 @@ 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) {
-               rte_free(cache_resource);
+       ret = mlx5_flow_os_create_flow_action_dest_port
+                               (priv->sh->fdb_domain, resource->port_id,
+                                &cache_resource->action);
+       if (ret) {
+               mlx5_free(cache_resource);
                return rte_flow_error_set(error, ENOMEM,
                                          RTE_FLOW_ERROR_TYPE_UNSPECIFIED,
                                          NULL, "cannot create action");
@@ -2817,6 +2814,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,11 +2845,11 @@ 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) {
-               rte_free(cache_resource);
+       ret = mlx5_flow_os_create_flow_action_push_vlan
+                                       (domain, resource->vlan_tag,
+                                        &cache_resource->action);
+       if (ret) {
+               mlx5_free(cache_resource);
                return rte_flow_error_set(error, ENOMEM,
                                          RTE_FLOW_ERROR_TYPE_UNSPECIFIED,
                                          NULL, "cannot create action");
@@ -3991,6 +3989,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;
@@ -4023,21 +4022,20 @@ flow_dv_modify_hdr_resource_register
                }
        }
        /* Register new modify-header resource. */
-       cache_resource = rte_calloc(__func__, 1,
-                                   sizeof(*cache_resource) + actions_len, 0);
+       cache_resource = mlx5_malloc(MLX5_MEM_ZERO,
+                                   sizeof(*cache_resource) + actions_len, 0,
+                                   SOCKET_ID_ANY);
        if (!cache_resource)
                return rte_flow_error_set(error, ENOMEM,
                                          RTE_FLOW_ERROR_TYPE_UNSPECIFIED, NULL,
                                          "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) {
-               rte_free(cache_resource);
+       ret = mlx5_flow_os_create_flow_action_modify_header
+                                       (sh->ctx, ns, cache_resource,
+                                        actions_len, &cache_resource->action);
+       if (ret) {
+               mlx5_free(cache_resource);
                return rte_flow_error_set(error, ENOMEM,
                                          RTE_FLOW_ERROR_TYPE_UNSPECIFIED,
                                          NULL, "cannot create action");
@@ -4176,7 +4174,14 @@ flow_dv_create_counter_stat_mem_mng(struct rte_eth_dev *dev, int raws_n)
                        MLX5_COUNTERS_PER_POOL +
                        sizeof(struct mlx5_counter_stats_raw)) * raws_n +
                        sizeof(struct mlx5_counter_stats_mem_mng);
-       uint8_t *mem = rte_calloc(__func__, 1, size, sysconf(_SC_PAGESIZE));
+       size_t pgsize = rte_mem_page_size();
+       if (pgsize == (size_t)-1) {
+               DRV_LOG(ERR, "Failed to get mem page size");
+               rte_errno = ENOMEM;
+               return NULL;
+       }
+       uint8_t *mem = mlx5_malloc(MLX5_MEM_ZERO, size, pgsize,
+                                 SOCKET_ID_ANY);
        int i;
 
        if (!mem) {
@@ -4189,7 +4194,7 @@ flow_dv_create_counter_stat_mem_mng(struct rte_eth_dev *dev, int raws_n)
                                                 IBV_ACCESS_LOCAL_WRITE);
        if (!mem_mng->umem) {
                rte_errno = errno;
-               rte_free(mem);
+               mlx5_free(mem);
                return NULL;
        }
        mkey_attr.addr = (uintptr_t)mem;
@@ -4208,7 +4213,7 @@ flow_dv_create_counter_stat_mem_mng(struct rte_eth_dev *dev, int raws_n)
        if (!mem_mng->dm) {
                mlx5_glue->devx_umem_dereg(mem_mng->umem);
                rte_errno = errno;
-               rte_free(mem);
+               mlx5_free(mem);
                return NULL;
        }
        mem_mng->raws = (struct mlx5_counter_stats_raw *)(mem + size);
@@ -4245,7 +4250,7 @@ flow_dv_container_resize(struct rte_eth_dev *dev,
        void *old_pools = cont->pools;
        uint32_t resize = cont->n + MLX5_CNT_CONTAINER_RESIZE;
        uint32_t mem_size = sizeof(struct mlx5_flow_counter_pool *) * resize;
-       void *pools = rte_calloc(__func__, 1, mem_size, 0);
+       void *pools = mlx5_malloc(MLX5_MEM_ZERO, mem_size, 0, SOCKET_ID_ANY);
 
        if (!pools) {
                rte_errno = ENOMEM;
@@ -4264,7 +4269,7 @@ flow_dv_container_resize(struct rte_eth_dev *dev,
                mem_mng = flow_dv_create_counter_stat_mem_mng(dev,
                          MLX5_CNT_CONTAINER_RESIZE + MLX5_MAX_PENDING_QUERIES);
                if (!mem_mng) {
-                       rte_free(pools);
+                       mlx5_free(pools);
                        return -ENOMEM;
                }
                for (i = 0; i < MLX5_MAX_PENDING_QUERIES; ++i)
@@ -4279,7 +4284,7 @@ flow_dv_container_resize(struct rte_eth_dev *dev,
        cont->pools = pools;
        rte_spinlock_unlock(&cont->resize_sl);
        if (old_pools)
-               rte_free(old_pools);
+               mlx5_free(old_pools);
        return 0;
 }
 
@@ -4368,7 +4373,7 @@ flow_dv_pool_create(struct rte_eth_dev *dev, struct mlx5_devx_obj *dcs,
        size += MLX5_COUNTERS_PER_POOL * CNT_SIZE;
        size += (batch ? 0 : MLX5_COUNTERS_PER_POOL * CNTEXT_SIZE);
        size += (!age ? 0 : MLX5_COUNTERS_PER_POOL * AGE_SIZE);
-       pool = rte_calloc(__func__, 1, size, 0);
+       pool = mlx5_malloc(MLX5_MEM_ZERO, size, 0, SOCKET_ID_ANY);
        if (!pool) {
                rte_errno = ENOMEM;
                return NULL;
@@ -4621,6 +4626,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 +4635,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;
                }
@@ -4923,6 +4929,17 @@ flow_dv_validate(struct rte_eth_dev *dev, const struct rte_flow_attr *attr,
                        .hop_limits = 0xff,
                },
        };
+       const struct rte_flow_item_ecpri nic_ecpri_mask = {
+               .hdr = {
+                       .common = {
+                               .u32 =
+                               RTE_BE32(((const struct rte_ecpri_common_hdr) {
+                                       .type = 0xFF,
+                                       }).u32),
+                       },
+                       .dummy[0] = 0xffffffff,
+               },
+       };
        struct mlx5_priv *priv = dev->data->dev_private;
        struct mlx5_dev_config *dev_conf = &priv->config;
        uint16_t queue_index = 0xFFFF;
@@ -5173,6 +5190,17 @@ flow_dv_validate(struct rte_eth_dev *dev, const struct rte_flow_attr *attr,
                                return ret;
                        last_item = MLX5_FLOW_LAYER_GTP;
                        break;
+               case RTE_FLOW_ITEM_TYPE_ECPRI:
+                       /* Capacity will be checked in the translate stage. */
+                       ret = mlx5_flow_validate_item_ecpri(items, item_flags,
+                                                           last_item,
+                                                           ether_type,
+                                                           &nic_ecpri_mask,
+                                                           error);
+                       if (ret < 0)
+                               return ret;
+                       last_item = MLX5_FLOW_LAYER_ECPRI;
+                       break;
                default:
                        return rte_flow_error_set(error, ENOTSUP,
                                                  RTE_FLOW_ERROR_TYPE_ITEM,
@@ -5737,7 +5765,15 @@ flow_dv_prepare(struct rte_eth_dev *dev,
        dev_flow = &((struct mlx5_flow *)priv->inter_flows)[priv->flow_idx++];
        dev_flow->handle = dev_handle;
        dev_flow->handle_idx = handle_idx;
-       dev_flow->dv.value.size = MLX5_ST_SZ_BYTES(fte_match_param);
+       /*
+        * In some old rdma-core releases, before continuing, a check of the
+        * length of matching parameter will be done at first. It needs to use
+        * the length without misc4 param. If the flow has misc4 support, then
+        * the length needs to be adjusted accordingly. Each param member is
+        * aligned with a 64B boundary naturally.
+        */
+       dev_flow->dv.value.size = MLX5_ST_SZ_BYTES(fte_match_param) -
+                                 MLX5_ST_SZ_BYTES(fte_match_set_misc4);
        /*
         * The matching value needs to be cleared to 0 before using. In the
         * past, it will be automatically cleared when using rte_*alloc
@@ -5882,6 +5918,7 @@ flow_dv_translate_item_eth(void *matcher, void *key,
         * Set match on ethertype only if ETH header is not followed by VLAN.
         * HW is optimized for IPv4/IPv6. In such cases, avoid setting
         * ethertype, and use ip_version field instead.
+        * eCPRI over Ether layer will use type value 0xAEFE.
         */
        if (eth_v->type == RTE_BE16(RTE_ETHER_TYPE_IPV4) &&
            eth_m->type == 0xFFFF) {
@@ -7236,6 +7273,90 @@ flow_dv_translate_item_gtp(void *matcher, void *key,
                 rte_be_to_cpu_32(gtp_v->teid & gtp_m->teid));
 }
 
+/**
+ * Add eCPRI item to matcher and to the value.
+ *
+ * @param[in] dev
+ *   The devich to configure through.
+ * @param[in, out] matcher
+ *   Flow matcher.
+ * @param[in, out] key
+ *   Flow matcher value.
+ * @param[in] item
+ *   Flow pattern to translate.
+ * @param[in] samples
+ *   Sample IDs to be used in the matching.
+ */
+static void
+flow_dv_translate_item_ecpri(struct rte_eth_dev *dev, void *matcher,
+                            void *key, const struct rte_flow_item *item)
+{
+       struct mlx5_priv *priv = dev->data->dev_private;
+       const struct rte_flow_item_ecpri *ecpri_m = item->mask;
+       const struct rte_flow_item_ecpri *ecpri_v = item->spec;
+       void *misc4_m = MLX5_ADDR_OF(fte_match_param, matcher,
+                                    misc_parameters_4);
+       void *misc4_v = MLX5_ADDR_OF(fte_match_param, key, misc_parameters_4);
+       uint32_t *samples;
+       void *dw_m;
+       void *dw_v;
+
+       if (!ecpri_v)
+               return;
+       if (!ecpri_m)
+               ecpri_m = &rte_flow_item_ecpri_mask;
+       /*
+        * Maximal four DW samples are supported in a single matching now.
+        * Two are used now for a eCPRI matching:
+        * 1. Type: one byte, mask should be 0x00ff0000 in network order
+        * 2. ID of a message: one or two bytes, mask 0xffff0000 or 0xff000000
+        *    if any.
+        */
+       if (!ecpri_m->hdr.common.u32)
+               return;
+       samples = priv->sh->fp[MLX5_FLEX_PARSER_ECPRI_0].ids;
+       /* Need to take the whole DW as the mask to fill the entry. */
+       dw_m = MLX5_ADDR_OF(fte_match_set_misc4, misc4_m,
+                           prog_sample_field_value_0);
+       dw_v = MLX5_ADDR_OF(fte_match_set_misc4, misc4_v,
+                           prog_sample_field_value_0);
+       /* Already big endian (network order) in the header. */
+       *(uint32_t *)dw_m = ecpri_m->hdr.common.u32;
+       *(uint32_t *)dw_v = ecpri_v->hdr.common.u32;
+       /* Sample#0, used for matching type, offset 0. */
+       MLX5_SET(fte_match_set_misc4, misc4_m,
+                prog_sample_field_id_0, samples[0]);
+       /* It makes no sense to set the sample ID in the mask field. */
+       MLX5_SET(fte_match_set_misc4, misc4_v,
+                prog_sample_field_id_0, samples[0]);
+       /*
+        * Checking if message body part needs to be matched.
+        * Some wildcard rules only matching type field should be supported.
+        */
+       if (ecpri_m->hdr.dummy[0]) {
+               switch (ecpri_v->hdr.common.type) {
+               case RTE_ECPRI_MSG_TYPE_IQ_DATA:
+               case RTE_ECPRI_MSG_TYPE_RTC_CTRL:
+               case RTE_ECPRI_MSG_TYPE_DLY_MSR:
+                       dw_m = MLX5_ADDR_OF(fte_match_set_misc4, misc4_m,
+                                           prog_sample_field_value_1);
+                       dw_v = MLX5_ADDR_OF(fte_match_set_misc4, misc4_v,
+                                           prog_sample_field_value_1);
+                       *(uint32_t *)dw_m = ecpri_m->hdr.dummy[0];
+                       *(uint32_t *)dw_v = ecpri_v->hdr.dummy[0];
+                       /* Sample#1, to match message body, offset 4. */
+                       MLX5_SET(fte_match_set_misc4, misc4_m,
+                                prog_sample_field_id_1, samples[1]);
+                       MLX5_SET(fte_match_set_misc4, misc4_v,
+                                prog_sample_field_id_1, samples[1]);
+                       break;
+               default:
+                       /* Others, do not match any sample ID. */
+                       break;
+               }
+       }
+}
+
 static uint32_t matcher_zero[MLX5_ST_SZ_DW(fte_match_param)] = { 0 };
 
 #define HEADER_IS_ZERO(match_criteria, headers)                                     \
@@ -7271,6 +7392,9 @@ flow_dv_matcher_enable(uint32_t *match_criteria)
        match_criteria_enable |=
                (!HEADER_IS_ZERO(match_criteria, misc_parameters_3)) <<
                MLX5_MATCH_CRITERIA_ENABLE_MISC3_BIT;
+       match_criteria_enable |=
+               (!HEADER_IS_ZERO(match_criteria, misc_parameters_4)) <<
+               MLX5_MATCH_CRITERIA_ENABLE_MISC4_BIT;
        return match_criteria_enable;
 }
 
@@ -7467,7 +7591,8 @@ flow_dv_matcher_register(struct rte_eth_dev *dev,
                }
        }
        /* Register new matcher. */
-       cache_matcher = rte_calloc(__func__, 1, sizeof(*cache_matcher), 0);
+       cache_matcher = mlx5_malloc(MLX5_MEM_ZERO, sizeof(*cache_matcher), 0,
+                                   SOCKET_ID_ANY);
        if (!cache_matcher) {
                flow_dv_tbl_resource_release(dev, tbl);
                return rte_flow_error_set(error, ENOMEM,
@@ -7483,7 +7608,7 @@ flow_dv_matcher_register(struct rte_eth_dev *dev,
        ret = mlx5_flow_os_create_flow_matcher(sh->ctx, &dv_attr, tbl->obj,
                                               &cache_matcher->matcher_object);
        if (ret) {
-               rte_free(cache_matcher);
+               mlx5_free(cache_matcher);
 #ifdef HAVE_MLX5DV_DR
                flow_dv_tbl_resource_release(dev, tbl);
 #endif
@@ -7532,6 +7657,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);
@@ -7554,9 +7680,10 @@ 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) {
-               rte_free(cache_resource);
+       ret = mlx5_flow_os_create_flow_action_tag(tag_be24,
+                                                 &cache_resource->action);
+       if (ret) {
+               mlx5_free(cache_resource);
                return rte_flow_error_set(error, ENOMEM,
                                          RTE_FLOW_ERROR_TYPE_UNSPECIFIED,
                                          NULL, "cannot create action");
@@ -7564,8 +7691,8 @@ 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);
-               rte_free(cache_resource);
+               mlx5_flow_os_destroy_flow_action(cache_resource->action);
+               mlx5_free(cache_resource);
                return rte_flow_error_set(error, EEXIST,
                                          RTE_FLOW_ERROR_TYPE_UNSPECIFIED,
                                          NULL, "cannot insert tag");
@@ -7603,7 +7730,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);
@@ -7867,7 +7994,8 @@ __flow_dv_translate(struct rte_eth_dev *dev,
        uint64_t priority = attr->priority;
        struct mlx5_flow_dv_matcher matcher = {
                .mask = {
-                       .size = sizeof(matcher.mask.buf),
+                       .size = sizeof(matcher.mask.buf) -
+                               MLX5_ST_SZ_BYTES(fte_match_set_misc4),
                },
        };
        int actions_n = 0;
@@ -8548,6 +8676,26 @@ __flow_dv_translate(struct rte_eth_dev *dev,
                                    MLX5_PRIORITY_MAP_L2 : MLX5_PRIORITY_MAP_L4;
                        last_item = MLX5_FLOW_LAYER_GTP;
                        break;
+               case RTE_FLOW_ITEM_TYPE_ECPRI:
+                       if (!mlx5_flex_parser_ecpri_exist(dev)) {
+                               /* Create it only the first time to be used. */
+                               ret = mlx5_flex_parser_ecpri_alloc(dev);
+                               if (ret)
+                                       return rte_flow_error_set
+                                               (error, -ret,
+                                               RTE_FLOW_ERROR_TYPE_ITEM,
+                                               NULL,
+                                               "cannot create eCPRI parser");
+                       }
+                       /* Adjust the length matcher and device flow value. */
+                       matcher.mask.size = MLX5_ST_SZ_BYTES(fte_match_param);
+                       dev_flow->dv.value.size =
+                                       MLX5_ST_SZ_BYTES(fte_match_param);
+                       flow_dv_translate_item_ecpri(dev, match_mask,
+                                                    match_value, items);
+                       /* No other protocol should follow eCPRI layer. */
+                       last_item = MLX5_FLOW_LAYER_ECPRI;
+                       break;
                default:
                        break;
                }
@@ -8767,7 +8915,7 @@ flow_dv_matcher_release(struct rte_eth_dev *dev,
                LIST_REMOVE(matcher, next);
                /* table ref-- in release interface. */
                flow_dv_tbl_resource_release(dev, matcher->tbl);
-               rte_free(matcher);
+               mlx5_free(matcher);
                DRV_LOG(DEBUG, "port %u matcher %p: removed",
                        dev->data->port_id, (void *)matcher);
                return 0;
@@ -8803,8 +8951,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 +8993,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,10 +9054,10 @@ 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);
+               mlx5_free(cache_resource);
                DRV_LOG(DEBUG, "modify-header resource %p: removed",
                        (void *)cache_resource);
                return 0;
@@ -8945,8 +9093,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 +9134,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);
@@ -9281,8 +9429,8 @@ flow_dv_destroy_mtr_tbl(struct rte_eth_dev *dev,
        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));
-       rte_free(mtd);
+               claim_zero(mlx5_flow_os_destroy_flow_action(mtd->drop_actn));
+       mlx5_free(mtd);
        return 0;
 }
 
@@ -9415,7 +9563,7 @@ flow_dv_create_mtr_tbl(struct rte_eth_dev *dev,
                rte_errno = ENOTSUP;
                return NULL;
        }
-       mtb = rte_calloc(__func__, 1, sizeof(*mtb), 0);
+       mtb = mlx5_malloc(MLX5_MEM_ZERO, sizeof(*mtb), 0, SOCKET_ID_ANY);
        if (!mtb) {
                DRV_LOG(ERR, "Failed to allocate memory for meter.");
                return NULL;
@@ -9430,8 +9578,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;
        }
@@ -9481,7 +9629,7 @@ flow_dv_destroy_domain_policer_rule(struct mlx5_meter_domain_info *dt)
                }
        }
        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;
        }
 }
@@ -9544,14 +9692,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;
+       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;
        }