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
