net/dpaa2: add Tx/Rx burst mode info
[dpdk.git] / drivers / net / dpaa2 / dpaa2_flow.c
index ad27f78..8430213 100644 (file)
@@ -30,6 +30,8 @@
 int mc_l4_port_identification;
 
 static char *dpaa2_flow_control_log;
+static int dpaa2_flow_miss_flow_id =
+       DPNI_FS_MISS_DROP;
 
 #define FIXED_ENTRY_SIZE 54
 
@@ -491,6 +493,42 @@ static int dpaa2_flow_extract_add(
        return 0;
 }
 
+static int dpaa2_flow_extract_add_raw(struct dpaa2_key_extract *key_extract,
+                                     int size)
+{
+       struct dpkg_profile_cfg *dpkg = &key_extract->dpkg;
+       struct dpaa2_key_info *key_info = &key_extract->key_info;
+       int last_extract_size, index;
+
+       if (dpkg->num_extracts != 0 && dpkg->extracts[0].type !=
+           DPKG_EXTRACT_FROM_DATA) {
+               DPAA2_PMD_WARN("RAW extract cannot be combined with others");
+               return -1;
+       }
+
+       last_extract_size = (size % DPAA2_FLOW_MAX_KEY_SIZE);
+       dpkg->num_extracts = (size / DPAA2_FLOW_MAX_KEY_SIZE);
+       if (last_extract_size)
+               dpkg->num_extracts++;
+       else
+               last_extract_size = DPAA2_FLOW_MAX_KEY_SIZE;
+
+       for (index = 0; index < dpkg->num_extracts; index++) {
+               dpkg->extracts[index].type = DPKG_EXTRACT_FROM_DATA;
+               if (index == dpkg->num_extracts - 1)
+                       dpkg->extracts[index].extract.from_data.size =
+                               last_extract_size;
+               else
+                       dpkg->extracts[index].extract.from_data.size =
+                               DPAA2_FLOW_MAX_KEY_SIZE;
+               dpkg->extracts[index].extract.from_data.offset =
+                       DPAA2_FLOW_MAX_KEY_SIZE * index;
+       }
+
+       key_info->key_total_size = size;
+       return 0;
+}
+
 /* Protocol discrimination.
  * Discriminate IPv4/IPv6/vLan by Eth type.
  * Discriminate UDP/TCP/ICMP by next proto of IP.
@@ -672,6 +710,18 @@ dpaa2_flow_rule_data_set(
        return 0;
 }
 
+static inline int
+dpaa2_flow_rule_data_set_raw(struct dpni_rule_cfg *rule,
+                            const void *key, const void *mask, int size)
+{
+       int offset = 0;
+
+       memcpy((void *)(size_t)(rule->key_iova + offset), key, size);
+       memcpy((void *)(size_t)(rule->mask_iova + offset), mask, size);
+
+       return 0;
+}
+
 static inline int
 _dpaa2_flow_rule_move_ipaddr_tail(
        struct dpaa2_key_extract *key_extract,
@@ -2812,6 +2862,83 @@ dpaa2_configure_flow_gre(struct rte_flow *flow,
        return 0;
 }
 
+static int
+dpaa2_configure_flow_raw(struct rte_flow *flow,
+                        struct rte_eth_dev *dev,
+                        const struct rte_flow_attr *attr,
+                        const struct rte_flow_item *pattern,
+                        const struct rte_flow_action actions[] __rte_unused,
+                        struct rte_flow_error *error __rte_unused,
+                        int *device_configured)
+{
+       struct dpaa2_dev_priv *priv = dev->data->dev_private;
+       const struct rte_flow_item_raw *spec = pattern->spec;
+       const struct rte_flow_item_raw *mask = pattern->mask;
+       int prev_key_size =
+               priv->extract.qos_key_extract.key_info.key_total_size;
+       int local_cfg = 0, ret;
+       uint32_t group;
+
+       /* Need both spec and mask */
+       if (!spec || !mask) {
+               DPAA2_PMD_ERR("spec or mask not present.");
+               return -EINVAL;
+       }
+       /* Only supports non-relative with offset 0 */
+       if (spec->relative || spec->offset != 0 ||
+           spec->search || spec->limit) {
+               DPAA2_PMD_ERR("relative and non zero offset not supported.");
+               return -EINVAL;
+       }
+       /* Spec len and mask len should be same */
+       if (spec->length != mask->length) {
+               DPAA2_PMD_ERR("Spec len and mask len mismatch.");
+               return -EINVAL;
+       }
+
+       /* Get traffic class index and flow id to be configured */
+       group = attr->group;
+       flow->tc_id = group;
+       flow->tc_index = attr->priority;
+
+       if (prev_key_size < spec->length) {
+               ret = dpaa2_flow_extract_add_raw(&priv->extract.qos_key_extract,
+                                                spec->length);
+               if (ret) {
+                       DPAA2_PMD_ERR("QoS Extract RAW add failed.");
+                       return -1;
+               }
+               local_cfg |= DPAA2_QOS_TABLE_RECONFIGURE;
+
+               ret = dpaa2_flow_extract_add_raw(
+                                       &priv->extract.tc_key_extract[group],
+                                       spec->length);
+               if (ret) {
+                       DPAA2_PMD_ERR("FS Extract RAW add failed.");
+                       return -1;
+               }
+               local_cfg |= DPAA2_FS_TABLE_RECONFIGURE;
+       }
+
+       ret = dpaa2_flow_rule_data_set_raw(&flow->qos_rule, spec->pattern,
+                                          mask->pattern, spec->length);
+       if (ret) {
+               DPAA2_PMD_ERR("QoS RAW rule data set failed");
+               return -1;
+       }
+
+       ret = dpaa2_flow_rule_data_set_raw(&flow->fs_rule, spec->pattern,
+                                          mask->pattern, spec->length);
+       if (ret) {
+               DPAA2_PMD_ERR("FS RAW rule data set failed");
+               return -1;
+       }
+
+       (*device_configured) |= local_cfg;
+
+       return 0;
+}
+
 /* The existing QoS/FS entry with IP address(es)
  * needs update after
  * new extract(s) are inserted before IP
@@ -2872,11 +2999,13 @@ dpaa2_flow_entry_update(
 
                dpaa2_flow_qos_entry_log("Before update", curr, qos_index);
 
-               ret = dpni_remove_qos_entry(dpni, CMD_PRI_LOW,
-                               priv->token, &curr->qos_rule);
-               if (ret) {
-                       DPAA2_PMD_ERR("Qos entry remove failed.");
-                       return -1;
+               if (priv->num_rx_tc > 1) {
+                       ret = dpni_remove_qos_entry(dpni, CMD_PRI_LOW,
+                                       priv->token, &curr->qos_rule);
+                       if (ret) {
+                               DPAA2_PMD_ERR("Qos entry remove failed.");
+                               return -1;
+                       }
                }
 
                extend = -1;
@@ -2977,13 +3106,15 @@ dpaa2_flow_entry_update(
 
                dpaa2_flow_qos_entry_log("Start update", curr, qos_index);
 
-               ret = dpni_add_qos_entry(dpni, CMD_PRI_LOW,
-                               priv->token, &curr->qos_rule,
-                               curr->tc_id, qos_index,
-                               0, 0);
-               if (ret) {
-                       DPAA2_PMD_ERR("Qos entry update failed.");
-                       return -1;
+               if (priv->num_rx_tc > 1) {
+                       ret = dpni_add_qos_entry(dpni, CMD_PRI_LOW,
+                                       priv->token, &curr->qos_rule,
+                                       curr->tc_id, qos_index,
+                                       0, 0);
+                       if (ret) {
+                               DPAA2_PMD_ERR("Qos entry update failed.");
+                               return -1;
+                       }
                }
 
                if (curr->action != RTE_FLOW_ACTION_TYPE_QUEUE) {
@@ -3124,6 +3255,67 @@ dpaa2_flow_verify_attr(
        return 0;
 }
 
+static inline int
+dpaa2_flow_verify_action(
+       struct dpaa2_dev_priv *priv,
+       const struct rte_flow_attr *attr,
+       const struct rte_flow_action actions[])
+{
+       int end_of_list = 0, i, j = 0;
+       const struct rte_flow_action_queue *dest_queue;
+       const struct rte_flow_action_rss *rss_conf;
+       struct dpaa2_queue *rxq;
+
+       while (!end_of_list) {
+               switch (actions[j].type) {
+               case RTE_FLOW_ACTION_TYPE_QUEUE:
+                       dest_queue = (const struct rte_flow_action_queue *)
+                                       (actions[j].conf);
+                       rxq = priv->rx_vq[dest_queue->index];
+                       if (attr->group != rxq->tc_index) {
+                               DPAA2_PMD_ERR(
+                                       "RXQ[%d] does not belong to the group %d",
+                                       dest_queue->index, attr->group);
+
+                               return -1;
+                       }
+                       break;
+               case RTE_FLOW_ACTION_TYPE_RSS:
+                       rss_conf = (const struct rte_flow_action_rss *)
+                                       (actions[j].conf);
+                       if (rss_conf->queue_num > priv->dist_queues) {
+                               DPAA2_PMD_ERR(
+                                       "RSS number exceeds the distrbution size");
+                               return -ENOTSUP;
+                       }
+                       for (i = 0; i < (int)rss_conf->queue_num; i++) {
+                               if (rss_conf->queue[i] >= priv->nb_rx_queues) {
+                                       DPAA2_PMD_ERR(
+                                               "RSS queue index exceeds the number of RXQs");
+                                       return -ENOTSUP;
+                               }
+                               rxq = priv->rx_vq[rss_conf->queue[i]];
+                               if (rxq->tc_index != attr->group) {
+                                       DPAA2_PMD_ERR(
+                                               "Queue/Group combination are not supported\n");
+                                       return -ENOTSUP;
+                               }
+                       }
+
+                       break;
+               case RTE_FLOW_ACTION_TYPE_END:
+                       end_of_list = 1;
+                       break;
+               default:
+                       DPAA2_PMD_ERR("Invalid action type");
+                       return -ENOTSUP;
+               }
+               j++;
+       }
+
+       return 0;
+}
+
 static int
 dpaa2_generic_flow_set(struct rte_flow *flow,
                       struct rte_eth_dev *dev,
@@ -3136,7 +3328,7 @@ dpaa2_generic_flow_set(struct rte_flow *flow,
        const struct rte_flow_action_rss *rss_conf;
        int is_keycfg_configured = 0, end_of_list = 0;
        int ret = 0, i = 0, j = 0;
-       struct dpni_rx_tc_dist_cfg tc_cfg;
+       struct dpni_rx_dist_cfg tc_cfg;
        struct dpni_qos_tbl_cfg qos_cfg;
        struct dpni_fs_action_cfg action;
        struct dpaa2_dev_priv *priv = dev->data->dev_private;
@@ -3150,6 +3342,10 @@ dpaa2_generic_flow_set(struct rte_flow *flow,
        if (ret)
                return ret;
 
+       ret = dpaa2_flow_verify_action(priv, attr, actions);
+       if (ret)
+               return ret;
+
        /* Parse pattern list to get the matching parameters */
        while (!end_of_list) {
                switch (pattern[i].type) {
@@ -3226,6 +3422,16 @@ dpaa2_generic_flow_set(struct rte_flow *flow,
                                return ret;
                        }
                        break;
+               case RTE_FLOW_ITEM_TYPE_RAW:
+                       ret = dpaa2_configure_flow_raw(flow,
+                                                      dev, attr, &pattern[i],
+                                                      actions, error,
+                                                      &is_keycfg_configured);
+                       if (ret) {
+                               DPAA2_PMD_ERR("RAW flow configuration failed!");
+                               return ret;
+                       }
+                       break;
                case RTE_FLOW_ITEM_TYPE_END:
                        end_of_list = 1;
                        break; /*End of List*/
@@ -3248,31 +3454,8 @@ dpaa2_generic_flow_set(struct rte_flow *flow,
                        flow->action = RTE_FLOW_ACTION_TYPE_QUEUE;
                        memset(&action, 0, sizeof(struct dpni_fs_action_cfg));
                        action.flow_id = rxq->flow_id;
-                       if (is_keycfg_configured & DPAA2_QOS_TABLE_RECONFIGURE) {
-                               dpaa2_flow_qos_table_extracts_log(priv);
-                               if (dpkg_prepare_key_cfg(
-                                       &priv->extract.qos_key_extract.dpkg,
-                                       (uint8_t *)(size_t)priv->extract.qos_extract_param)
-                                       < 0) {
-                                       DPAA2_PMD_ERR(
-                                       "Unable to prepare extract parameters");
-                                       return -1;
-                               }
 
-                               memset(&qos_cfg, 0, sizeof(struct dpni_qos_tbl_cfg));
-                               qos_cfg.discard_on_miss = true;
-                               qos_cfg.keep_entries = true;
-                               qos_cfg.key_cfg_iova =
-                                       (size_t)priv->extract.qos_extract_param;
-                               ret = dpni_set_qos_table(dpni, CMD_PRI_LOW,
-                                               priv->token, &qos_cfg);
-                               if (ret < 0) {
-                                       DPAA2_PMD_ERR(
-                                       "Distribution cannot be configured.(%d)"
-                                       , ret);
-                                       return -1;
-                               }
-                       }
+                       /* Configure FS table first*/
                        if (is_keycfg_configured & DPAA2_FS_TABLE_RECONFIGURE) {
                                dpaa2_flow_fs_table_extracts_log(priv, flow->tc_id);
                                if (dpkg_prepare_key_cfg(
@@ -3284,34 +3467,66 @@ dpaa2_generic_flow_set(struct rte_flow *flow,
                                        return -1;
                                }
 
-                               memset(&tc_cfg, 0, sizeof(struct dpni_rx_tc_dist_cfg));
+                               memset(&tc_cfg, 0,
+                                       sizeof(struct dpni_rx_dist_cfg));
                                tc_cfg.dist_size = priv->nb_rx_queues / priv->num_rx_tc;
-                               tc_cfg.dist_mode = DPNI_DIST_MODE_FS;
                                tc_cfg.key_cfg_iova =
                                        (uint64_t)priv->extract.tc_extract_param[flow->tc_id];
-                               tc_cfg.fs_cfg.miss_action = DPNI_FS_MISS_DROP;
-                               tc_cfg.fs_cfg.keep_entries = true;
-                               ret = dpni_set_rx_tc_dist(dpni, CMD_PRI_LOW,
-                                                        priv->token,
-                                                        flow->tc_id, &tc_cfg);
+                               tc_cfg.tc = flow->tc_id;
+                               tc_cfg.enable = false;
+                               ret = dpni_set_rx_hash_dist(dpni, CMD_PRI_LOW,
+                                               priv->token, &tc_cfg);
+                               if (ret < 0) {
+                                       DPAA2_PMD_ERR(
+                                               "TC hash cannot be disabled.(%d)",
+                                               ret);
+                                       return -1;
+                               }
+                               tc_cfg.enable = true;
+                               tc_cfg.fs_miss_flow_id =
+                                       dpaa2_flow_miss_flow_id;
+                               ret = dpni_set_rx_fs_dist(dpni, CMD_PRI_LOW,
+                                                        priv->token, &tc_cfg);
                                if (ret < 0) {
                                        DPAA2_PMD_ERR(
-                                       "Distribution cannot be configured.(%d)"
-                                       , ret);
+                                               "TC distribution cannot be configured.(%d)",
+                                               ret);
                                        return -1;
                                }
                        }
-                       /* Configure QoS table first */
 
-                       qos_index = flow->tc_id * priv->fs_entries +
-                               flow->tc_index;
+                       /* Configure QoS table then.*/
+                       if (is_keycfg_configured & DPAA2_QOS_TABLE_RECONFIGURE) {
+                               dpaa2_flow_qos_table_extracts_log(priv);
+                               if (dpkg_prepare_key_cfg(
+                                       &priv->extract.qos_key_extract.dpkg,
+                                       (uint8_t *)(size_t)priv->extract.qos_extract_param) < 0) {
+                                       DPAA2_PMD_ERR(
+                                               "Unable to prepare extract parameters");
+                                       return -1;
+                               }
 
-                       if (qos_index >= priv->qos_entries) {
-                               DPAA2_PMD_ERR("QoS table with %d entries full",
-                                       priv->qos_entries);
-                               return -1;
+                               memset(&qos_cfg, 0, sizeof(struct dpni_qos_tbl_cfg));
+                               qos_cfg.discard_on_miss = false;
+                               qos_cfg.default_tc = 0;
+                               qos_cfg.keep_entries = true;
+                               qos_cfg.key_cfg_iova =
+                                       (size_t)priv->extract.qos_extract_param;
+                               /* QoS table is effecitive for multiple TCs.*/
+                               if (priv->num_rx_tc > 1) {
+                                       ret = dpni_set_qos_table(dpni, CMD_PRI_LOW,
+                                               priv->token, &qos_cfg);
+                                       if (ret < 0) {
+                                               DPAA2_PMD_ERR(
+                                               "RSS QoS table can not be configured(%d)\n",
+                                                       ret);
+                                               return -1;
+                                       }
+                               }
                        }
-                       flow->qos_rule.key_size = FIXED_ENTRY_SIZE;
+
+                       flow->qos_real_key_size = priv->extract
+                               .qos_key_extract.key_info.key_total_size;
                        if (flow->ipaddr_rule.ipaddr_type == FLOW_IPV4_ADDR) {
                                if (flow->ipaddr_rule.qos_ipdst_offset >=
                                        flow->ipaddr_rule.qos_ipsrc_offset) {
@@ -3337,21 +3552,30 @@ dpaa2_generic_flow_set(struct rte_flow *flow,
                                }
                        }
 
-                       flow->qos_rule.key_size = FIXED_ENTRY_SIZE;
+                       /* QoS entry added is only effective for multiple TCs.*/
+                       if (priv->num_rx_tc > 1) {
+                               qos_index = flow->tc_id * priv->fs_entries +
+                                       flow->tc_index;
+                               if (qos_index >= priv->qos_entries) {
+                                       DPAA2_PMD_ERR("QoS table with %d entries full",
+                                               priv->qos_entries);
+                                       return -1;
+                               }
+                               flow->qos_rule.key_size = FIXED_ENTRY_SIZE;
 
-                       dpaa2_flow_qos_entry_log("Start add", flow, qos_index);
+                               dpaa2_flow_qos_entry_log("Start add", flow, qos_index);
 
-                       ret = dpni_add_qos_entry(dpni, CMD_PRI_LOW,
+                               ret = dpni_add_qos_entry(dpni, CMD_PRI_LOW,
                                                priv->token, &flow->qos_rule,
                                                flow->tc_id, qos_index,
                                                0, 0);
-                       if (ret < 0) {
-                               DPAA2_PMD_ERR(
-                               "Error in addnig entry to QoS table(%d)", ret);
-                               return ret;
+                               if (ret < 0) {
+                                       DPAA2_PMD_ERR(
+                                               "Error in addnig entry to QoS table(%d)", ret);
+                                       return ret;
+                               }
                        }
 
-                       /* Then Configure FS table */
                        if (flow->tc_index >= priv->fs_entries) {
                                DPAA2_PMD_ERR("FS table with %d entries full",
                                        priv->fs_entries);
@@ -3405,25 +3629,6 @@ dpaa2_generic_flow_set(struct rte_flow *flow,
                        break;
                case RTE_FLOW_ACTION_TYPE_RSS:
                        rss_conf = (const struct rte_flow_action_rss *)(actions[j].conf);
-                       if (rss_conf->queue_num > priv->dist_queues) {
-                               DPAA2_PMD_ERR(
-                                       "RSS number exceeds the distrbution size");
-                               return -ENOTSUP;
-                       }
-
-                       for (i = 0; i < (int)rss_conf->queue_num; i++) {
-                               if (rss_conf->queue[i] >= priv->nb_rx_queues) {
-                                       DPAA2_PMD_ERR(
-                                               "RSS RXQ number exceeds the total number");
-                                       return -ENOTSUP;
-                               }
-                               rxq = priv->rx_vq[rss_conf->queue[i]];
-                               if (rxq->tc_index != attr->group) {
-                                       DPAA2_PMD_ERR(
-                                               "RSS RXQ distributed is not in current group");
-                                       return -ENOTSUP;
-                               }
-                       }
 
                        flow->action = RTE_FLOW_ACTION_TYPE_RSS;
                        ret = dpaa2_distset_to_dpkg_profile_cfg(rss_conf->types,
@@ -3450,18 +3655,17 @@ dpaa2_generic_flow_set(struct rte_flow *flow,
                                return -1;
                        }
 
-                       memset(&tc_cfg, 0, sizeof(struct dpni_rx_tc_dist_cfg));
+                       memset(&tc_cfg, 0, sizeof(struct dpni_rx_dist_cfg));
                        tc_cfg.dist_size = rss_conf->queue_num;
-                       tc_cfg.dist_mode = DPNI_DIST_MODE_HASH;
                        tc_cfg.key_cfg_iova = (size_t)param;
-                       tc_cfg.fs_cfg.miss_action = DPNI_FS_MISS_DROP;
-
-                       ret = dpni_set_rx_tc_dist(dpni, CMD_PRI_LOW,
-                                                priv->token, flow->tc_id,
-                                                &tc_cfg);
+                       tc_cfg.enable = true;
+                       tc_cfg.tc = flow->tc_id;
+                       ret = dpni_set_rx_hash_dist(dpni, CMD_PRI_LOW,
+                                                priv->token, &tc_cfg);
                        if (ret < 0) {
                                DPAA2_PMD_ERR(
-                               "Distribution cannot be configured: %d\n", ret);
+                                       "RSS TC table cannot be configured: %d\n",
+                                       ret);
                                rte_free((void *)param);
                                return -1;
                        }
@@ -3485,7 +3689,7 @@ dpaa2_generic_flow_set(struct rte_flow *flow,
                                                         priv->token, &qos_cfg);
                                if (ret < 0) {
                                        DPAA2_PMD_ERR(
-                                       "Distribution can't be configured %d\n",
+                                       "RSS QoS dist can't be configured-%d\n",
                                        ret);
                                        return -1;
                                }
@@ -3702,6 +3906,20 @@ struct rte_flow *dpaa2_flow_create(struct rte_eth_dev *dev,
        dpaa2_flow_control_log =
                getenv("DPAA2_FLOW_CONTROL_LOG");
 
+       if (getenv("DPAA2_FLOW_CONTROL_MISS_FLOW")) {
+               struct dpaa2_dev_priv *priv = dev->data->dev_private;
+
+               dpaa2_flow_miss_flow_id =
+                       atoi(getenv("DPAA2_FLOW_CONTROL_MISS_FLOW"));
+               if (dpaa2_flow_miss_flow_id >= priv->dist_queues) {
+                       DPAA2_PMD_ERR(
+                               "The missed flow ID %d exceeds the max flow ID %d",
+                               dpaa2_flow_miss_flow_id,
+                               priv->dist_queues - 1);
+                       return NULL;
+               }
+       }
+
        flow = rte_zmalloc(NULL, sizeof(struct rte_flow), RTE_CACHE_LINE_SIZE);
        if (!flow) {
                DPAA2_PMD_ERR("Failure to allocate memory for flow");
@@ -3795,13 +4013,15 @@ int dpaa2_flow_destroy(struct rte_eth_dev *dev,
 
        switch (flow->action) {
        case RTE_FLOW_ACTION_TYPE_QUEUE:
-               /* Remove entry from QoS table first */
-               ret = dpni_remove_qos_entry(dpni, CMD_PRI_LOW, priv->token,
-                                          &flow->qos_rule);
-               if (ret < 0) {
-                       DPAA2_PMD_ERR(
-                               "Error in adding entry to QoS table(%d)", ret);
-                       goto error;
+               if (priv->num_rx_tc > 1) {
+                       /* Remove entry from QoS table first */
+                       ret = dpni_remove_qos_entry(dpni, CMD_PRI_LOW, priv->token,
+                                       &flow->qos_rule);
+                       if (ret < 0) {
+                               DPAA2_PMD_ERR(
+                                       "Error in removing entry from QoS table(%d)", ret);
+                               goto error;
+                       }
                }
 
                /* Then remove entry from FS table */
@@ -3809,17 +4029,19 @@ int dpaa2_flow_destroy(struct rte_eth_dev *dev,
                                           flow->tc_id, &flow->fs_rule);
                if (ret < 0) {
                        DPAA2_PMD_ERR(
-                               "Error in entry addition in FS table(%d)", ret);
+                               "Error in removing entry from FS table(%d)", ret);
                        goto error;
                }
                break;
        case RTE_FLOW_ACTION_TYPE_RSS:
-               ret = dpni_remove_qos_entry(dpni, CMD_PRI_LOW, priv->token,
-                                          &flow->qos_rule);
-               if (ret < 0) {
-                       DPAA2_PMD_ERR(
-                       "Error in entry addition in QoS table(%d)", ret);
-                       goto error;
+               if (priv->num_rx_tc > 1) {
+                       ret = dpni_remove_qos_entry(dpni, CMD_PRI_LOW, priv->token,
+                                       &flow->qos_rule);
+                       if (ret < 0) {
+                               DPAA2_PMD_ERR(
+                                       "Error in entry addition in QoS table(%d)", ret);
+                               goto error;
+                       }
                }
                break;
        default: