This patch adds a function to destroy the ethertype filter.
And this patch also adds flow destroy function.

Signed-off-by: Beilei Xing <beilei.x...@intel.com>
---
 drivers/net/i40e/i40e_ethdev.c | 69 ++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 69 insertions(+)

diff --git a/drivers/net/i40e/i40e_ethdev.c b/drivers/net/i40e/i40e_ethdev.c
index ac93489..a3ed1f0 100644
--- a/drivers/net/i40e/i40e_ethdev.c
+++ b/drivers/net/i40e/i40e_ethdev.c
@@ -517,6 +517,11 @@ static struct i40e_flow *i40e_flow_create(struct 
rte_eth_dev *dev,
                                   const struct rte_flow_item *pattern,
                                   const struct rte_flow_action *actions,
                                   struct rte_flow_error *error);
+static int i40e_dev_destroy_ethertype_filter(struct i40e_pf *pf,
+                                    struct i40e_ethertype_filter *filter);
+static int i40e_flow_destroy(struct rte_eth_dev *dev,
+                            struct rte_flow *flow,
+                            struct rte_flow_error *error);
 
 struct i40e_flow {
        enum rte_filter_type filter_type;
@@ -621,6 +626,7 @@ static const struct eth_dev_ops i40e_eth_dev_ops = {
 static const struct rte_flow_ops i40e_flow_ops = {
        .validate = i40e_flow_validate,
        .create = (void *)i40e_flow_create,
+       .destroy = i40e_flow_destroy,
 };
 
 /* store statistics names and its offset in stats structure */
@@ -11665,3 +11671,66 @@ i40e_flow_create(struct rte_eth_dev *dev,
        rte_free(flow);
        return NULL;
 }
+
+static int
+i40e_dev_destroy_ethertype_filter(struct i40e_pf *pf,
+                                 struct i40e_ethertype_filter *filter)
+{
+       struct i40e_hw *hw = I40E_PF_TO_HW(pf);
+       struct i40e_ethertype_info *ethertype_info = &pf->ethertype;
+       struct i40e_ethertype_filter *node;
+       struct i40e_control_filter_stats stats;
+       uint16_t flags = 0;
+       int ret = 0;
+
+       if (!(filter->flags & RTE_ETHTYPE_FLAGS_MAC))
+               flags |= I40E_AQC_ADD_CONTROL_PACKET_FLAGS_IGNORE_MAC;
+       if (filter->flags & RTE_ETHTYPE_FLAGS_DROP)
+               flags |= I40E_AQC_ADD_CONTROL_PACKET_FLAGS_DROP;
+       flags |= I40E_AQC_ADD_CONTROL_PACKET_FLAGS_TO_QUEUE;
+
+       memset(&stats, 0, sizeof(stats));
+       ret = i40e_aq_add_rem_control_packet_filter(hw,
+                                   filter->input.mac_addr.addr_bytes,
+                                   filter->input.ether_type,
+                                   flags, pf->main_vsi->seid,
+                                   filter->queue, 0, &stats, NULL);
+       if (ret < 0)
+               return ret;
+
+       node = i40e_sw_ethertype_filter_lookup(ethertype_info, &filter->input);
+       if (node)
+               ret = i40e_sw_ethertype_filter_del(pf, node);
+       else
+               return -EINVAL;
+
+       return ret;
+}
+
+static int
+i40e_flow_destroy(struct rte_eth_dev *dev,
+                 struct rte_flow *flow,
+                 struct rte_flow_error *error)
+{
+       struct i40e_pf *pf = I40E_DEV_PRIVATE_TO_PF(dev->data->dev_private);
+       struct i40e_flow *pmd_flow = (struct i40e_flow *)flow;
+       enum rte_filter_type filter_type = pmd_flow->filter_type;
+       int ret;
+
+       switch (filter_type) {
+       case RTE_ETH_FILTER_ETHERTYPE:
+               ret = i40e_dev_destroy_ethertype_filter(pf,
+                       (struct i40e_ethertype_filter *)pmd_flow->rule);
+               break;
+       default:
+               PMD_DRV_LOG(WARNING, "Filter type (%d) not supported",
+                           filter_type);
+               ret = -EINVAL;
+               break;
+       }
+
+       if (ret)
+               error->type = RTE_FLOW_ERROR_TYPE_HANDLE;
+
+       return ret;
+}
-- 
2.5.5

Reply via email to