On 11/3/2020 2:20 PM, Andrey Vesnovaty wrote:
-----Original Message-----
From: Ferruh Yigit <ferruh.yi...@intel.com>
Sent: Monday, November 2, 2020 8:55 PM
To: Ivan Malov <ivan.ma...@oktetlabs.ru>; dev@dpdk.org; Andrey Vesnovaty
<andr...@nvidia.com>
Cc: Xueming(Steven) Li <xuemi...@nvidia.com>; Ori Kam <or...@nvidia.com>;
NBU-Contact-Thomas Monjalon <tho...@monjalon.net>; Andrew Rybchenko
<andrew.rybche...@oktetlabs.ru>
Subject: Re: [PATCH v3 1/2] ethdev: introduce transfer attribute to shared
action
conf
On 11/2/2020 11:43 AM, Ivan Malov wrote:
In a flow rule, attribute "transfer" means operation level
at which both traffic is matched and actions are conducted.
Add the very same attribute to shared action configuration.
If a driver needs to prepare HW resources in two different
ways, depending on the operation level, in order to set up
an action, then this new attribute will indicate the level.
Also, when handling a flow rule insertion, the driver will
be able to turn down a shared action if its level is unfit.
Signed-off-by: Ivan Malov <ivan.ma...@oktetlabs.ru>
Acked-by: Ori Kam <or...@nvidia.com>
---
lib/librte_ethdev/rte_flow.h | 8 ++++++++
1 file changed, 8 insertions(+)
diff --git a/lib/librte_ethdev/rte_flow.h b/lib/librte_ethdev/rte_flow.h
index a8eac4deb..8b970ba0b 100644
--- a/lib/librte_ethdev/rte_flow.h
+++ b/lib/librte_ethdev/rte_flow.h
@@ -3487,6 +3487,14 @@ struct rte_flow_shared_action_conf {
/**< Action valid for rules applied to ingress traffic. */
uint32_t egress:1;
/**< Action valid for rules applied to egress traffic. */
+
+ /**
+ * When set to 1, indicates that the action is valid for
+ * transfer traffic; otherwise, for non-transfer traffic.
+ *
+ * See struct rte_flow_attr.
+ */
+ uint32_t transfer:1;
Is this require any documentation update?
Also cc'ed Andrey, as he is author of the shared action feature, @Andrey can
you
please review this update?
Acked-by: Andrey Vesnovaty <andr...@nvidia.com>
Series applied to dpdk-next-net/main, thanks.
