[dpdk-dev] [PATCH v5 5/5] net/iavf: add support for FDIR mark action

[Simei Su]

This patch enables mark action support and takes mark only case
into consideration.

Signed-off-by: Simei Su <simei...@intel.com>
Reviewed-by: Qi Zhang <qi.z.zh...@intel.com>
---
 drivers/net/iavf/iavf.h      |  1 +
 drivers/net/iavf/iavf_fdir.c | 45 +++++++++++++++++++++++++++++++++++++++++++-
 2 files changed, 45 insertions(+), 1 deletion(-)

diff --git a/drivers/net/iavf/iavf.h b/drivers/net/iavf/iavf.h
index 30208d4..5fb7881 100644
--- a/drivers/net/iavf/iavf.h
+++ b/drivers/net/iavf/iavf.h
@@ -97,6 +97,7 @@ struct iavf_fdir_conf {
        struct virtchnl_fdir_del del_fltr;
        uint64_t input_set;
        uint32_t flow_id;
+       uint32_t mark_flag;
 };
 
 struct iavf_fdir_info {
diff --git a/drivers/net/iavf/iavf_fdir.c b/drivers/net/iavf/iavf_fdir.c
index 1e50a07..406622c 100644
--- a/drivers/net/iavf/iavf_fdir.c
+++ b/drivers/net/iavf/iavf_fdir.c
@@ -18,6 +18,7 @@
 #include "iavf.h"
 #include "iavf_generic_flow.h"
 #include "virtchnl.h"
+#include "iavf_rxtx.h"
 
 #define IAVF_FDIR_MAX_QREGION_SIZE 128
 
@@ -170,6 +171,9 @@
                goto free_entry;
        }
 
+       if (filter->mark_flag == 1)
+               iavf_fdir_rx_proc_enable(ad, 1);
+
        rte_memcpy(rule, filter, sizeof(*rule));
        flow->rule = rule;
 
@@ -198,6 +202,9 @@
                return -rte_errno;
        }
 
+       if (filter->mark_flag == 1)
+               iavf_fdir_rx_proc_enable(ad, 0);
+
        flow->rule = NULL;
        rte_free(filter);
 
@@ -296,7 +303,9 @@
                        struct iavf_fdir_conf *filter)
 {
        const struct rte_flow_action_queue *act_q;
+       const struct rte_flow_action_mark *mark_spec = NULL;
        uint32_t dest_num = 0;
+       uint32_t mark_num = 0;
        int ret;
 
        int number = 0;
@@ -362,6 +371,19 @@
                        filter->add_fltr.rule_cfg.action_set.count = ++number;
                        break;
 
+               case RTE_FLOW_ACTION_TYPE_MARK:
+                       mark_num++;
+
+                       filter->mark_flag = 1;
+                       mark_spec = actions->conf;
+                       filter_action = 
&filter->add_fltr.rule_cfg.action_set.actions[number];
+
+                       filter_action->type = VIRTCHNL_ACTION_MARK;
+                       filter_action->act_conf.mark_id = mark_spec->id;
+
+                       filter->add_fltr.rule_cfg.action_set.count = ++number;
+                       break;
+
                default:
                        rte_flow_error_set(error, EINVAL,
                                        RTE_FLOW_ERROR_TYPE_ACTION, actions,
@@ -377,13 +399,34 @@
                return -rte_errno;
        }
 
-       if (dest_num == 0 || dest_num >= 2) {
+       if (dest_num >= 2) {
                rte_flow_error_set(error, EINVAL,
                        RTE_FLOW_ERROR_TYPE_ACTION, actions,
                        "Unsupported action combination");
                return -rte_errno;
        }
 
+       if (mark_num >= 2) {
+               rte_flow_error_set(error, EINVAL,
+                       RTE_FLOW_ERROR_TYPE_ACTION, actions,
+                       "Too many mark actions");
+               return -rte_errno;
+       }
+
+       if (dest_num + mark_num == 0) {
+               rte_flow_error_set(error, EINVAL,
+                       RTE_FLOW_ERROR_TYPE_ACTION, actions,
+                       "Emtpy action");
+               return -rte_errno;
+       }
+
+       /* Mark only is equal to mark + passthru. */
+       if (dest_num == 0) {
+               filter_action = 
&filter->add_fltr.rule_cfg.action_set.actions[number];
+               filter_action->type = VIRTCHNL_ACTION_PASSTHRU;
+               filter->add_fltr.rule_cfg.action_set.count = ++number;
+       }
+
        return 0;
 }
 
-- 
1.8.3.1