net/ice: support traffic to original DCF port
Checks
Commit Message
Add support for sending traffic to the original DCF port
with 'port_representor' action by using DCF port id as 'port_id'.
For example:
testpmd> flow create 0 ingress pattern any
/ end actions port_representor port_id 0 / end
Signed-off-by: Zhichao Zeng <zhichaox.zeng@intel.com>
---
doc/guides/nics/features/ice.ini | 1 +
drivers/net/ice/ice_switch_filter.c | 25 +++++++++++++++++++++++++
2 files changed, 26 insertions(+)
Comments
> -----Original Message-----
> From: Zeng, ZhichaoX <zhichaox.zeng@intel.com>
> Sent: Saturday, September 17, 2022 9:49 AM
> To: dev@dpdk.org
> Cc: Zhou, YidingX <yidingx.zhou@intel.com>; Zeng, ZhichaoX
> <zhichaox.zeng@intel.com>; Yang, Qiming <qiming.yang@intel.com>; Zhang,
> Qi Z <qi.z.zhang@intel.com>
> Subject: [PATCH] net/ice: support traffic to original DCF port
>
> Add support for sending traffic to the original DCF port with
> 'port_representor' action by using DCF port id as 'port_id'.
>
> For example:
> testpmd> flow create 0 ingress pattern any
> / end actions port_representor port_id 0 / end
>
> Signed-off-by: Zhichao Zeng <zhichaox.zeng@intel.com>
Acked-by: Qi Zhang <qi.z.zhang@intel.com>
Applied to dpdk-next-net-intel.
Thanks
Qi
> -----Original Message-----
> From: Zhichao Zeng <zhichaox.zeng@intel.com>
> Sent: Saturday, September 17, 2022 9:49 AM
> To: dev@dpdk.org
> Cc: Zhou, YidingX <yidingx.zhou@intel.com>; Zeng, ZhichaoX
> <zhichaox.zeng@intel.com>; Yang, Qiming <qiming.yang@intel.com>; Zhang,
> Qi Z <qi.z.zhang@intel.com>
> Subject: [PATCH] net/ice: support traffic to original DCF port
>
> Add support for sending traffic to the original DCF port with
> 'port_representor' action by using DCF port id as 'port_id'.
>
> For example:
> testpmd> flow create 0 ingress pattern any
> / end actions port_representor port_id 0 / end
>
> Signed-off-by: Zhichao Zeng <zhichaox.zeng@intel.com>
> ---
> doc/guides/nics/features/ice.ini | 1 +
> drivers/net/ice/ice_switch_filter.c | 25 +++++++++++++++++++++++++
> 2 files changed, 26 insertions(+)
>
Tested and passed, using 'port_representor' action covering dcf to self dcf, dcf to other dcf, dcf to iavf, dcf to iavf of other dcf.
Tested-by: Ke Xu <ke1.xu@intel.com>
@@ -82,3 +82,4 @@ passthru = Y
queue = Y
rss = Y
represented_port = Y
+port_representor = Y
@@ -1628,6 +1628,30 @@ ice_switch_parse_dcf_action(struct ice_dcf_adapter *ad,
RTE_FLOW_ACTION_TYPE_END; action++) {
action_type = action->type;
switch (action_type) {
+ case RTE_FLOW_ACTION_TYPE_PORT_REPRESENTOR:
+ rule_info->sw_act.fltr_act = ICE_FWD_TO_VSI;
+ act_ethdev = action->conf;
+
+ if (!rte_eth_dev_is_valid_port(act_ethdev->port_id))
+ goto invalid_port_id;
+
+ /* For traffic to original DCF port */
+ rule_port_id = ad->parent.pf.dev_data->port_id;
+
+ if (rule_port_id != act_ethdev->port_id)
+ goto invalid_port_id;
+
+ rule_info->sw_act.vsi_handle = 0;
+
+ break;
+
+invalid_port_id:
+ rte_flow_error_set(error,
+ EINVAL, RTE_FLOW_ERROR_TYPE_ACTION,
+ actions,
+ "Invalid port_id");
+ return -rte_errno;
+
case RTE_FLOW_ACTION_TYPE_REPRESENTED_PORT:
rule_info->sw_act.fltr_act = ICE_FWD_TO_VSI;
act_ethdev = action->conf;
@@ -1799,6 +1823,7 @@ ice_switch_check_action(const struct rte_flow_action *actions,
case RTE_FLOW_ACTION_TYPE_QUEUE:
case RTE_FLOW_ACTION_TYPE_DROP:
case RTE_FLOW_ACTION_TYPE_REPRESENTED_PORT:
+ case RTE_FLOW_ACTION_TYPE_PORT_REPRESENTOR:
actions_num++;
break;
case RTE_FLOW_ACTION_TYPE_VOID: