[PATCH v5 2/2] ethdev: add indirect list METER_MARK update structures

[Gregory Etelson]

Hello Ori,

[snip]

> > +/**
> > + * @see RTE_FLOW_ACTION_TYPE_METER_MARK
> > + * @see RTE_FLOW_ACTION_TYPE_INDIRECT_LIST
> > + *
> > + * Update action mutable context.
> > + */
> > +struct rte_flow_indirect_update_action_meter_mark {
> > +	/** New meter_mark parameters to be updated. */
> > +	struct rte_flow_action_meter_mark meter_mark;
> > +	/** The profile will be updated. */
> > +	uint32_t profile_valid:1;
> > +	/** The policy will be updated. */
> > +	uint32_t policy_valid:1;
> > +	/** The color mode will be updated. */
> > +	uint32_t color_mode_valid:1;
> > +	/** The meter state will be updated. */
> > +	uint32_t state_valid:1;
> > +	/** Reserved bits for the future usage. */
> > +	uint32_t reserved:28;
> > +};
> > +
> 
> Why did you create new meter_mark structure?
> 


Fixed.

> > +/**
> > + * @see RTE_FLOW_ACTION_TYPE_METER_MARK
> > + * @see RTE_FLOW_ACTION_TYPE_INDIRECT_LIST
> > + *
> > + * Update flow mutable context.
> > + */
> > +struct rte_flow_indirect_update_flow_meter_mark {
> > +	/** Updated init color applied to packet */
> > +	enum rte_color init_color;
> > +};
> > +
> >  /* Mbuf dynamic field offset for metadata. */
> >  extern int32_t rte_flow_dynf_metadata_offs;
> >
> > --
> > 2.34.1
> 
> Best,
> Ori