On 11/25/2021 12:29 PM, Ori Kam wrote:
Hi

-----Original Message-----
From: Ferruh Yigit <ferruh.yi...@intel.com>
Sent: Thursday, November 25, 2021 2:19 PM
Subject: Re: [PATCH v3] ethdev: deprecate header fields and metadata flow 
actions

On 11/24/2021 3:37 PM, Viacheslav Ovsiienko wrote:
@@ -3331,6 +3430,9 @@ struct rte_flow_action_set_tag {
   };

   /**
+ * @deprecated
+ * @see RTE_FLOW_ACTION_TYPE_MODIFY_FIELD
+ *
    * @warning
    * @b EXPERIMENTAL: this structure may change without prior notice
    *
@@ -3355,6 +3457,9 @@ struct rte_flow_action_set_meta {
   };

   /**
+ * @deprecated
+ * @see RTE_FLOW_ACTION_TYPE_MODIFY_FIELD
+ *
    * RTE_FLOW_ACTION_TYPE_SET_IPV4_DSCP
    * RTE_FLOW_ACTION_TYPE_SET_IPV6_DSCP
    *

Hi Viacheslav, Ori,

Just recognized that some of the "struct rte_flow_*" added as experimental
(experimental note in the struct comment), some without experimental note.

Is it because some forgotten the add the experimental note?

What is the rule to add new "struct rte_flow_*"?


In the beginning struct were never experimental,
and in the last few releases we started to put experimental also on structs
which I think is much better since at the end this is a new API and just like 
any other
new API it is not stable.


Got it, thanks.

So to record for archives, new "struct rte_flow_*" structs should be
experimental by default.

Does it applies to enums too?

Reply via email to