Hi, Sorry for the late reply.
> -----Original Message----- > From: Thomas Monjalon <[email protected]> > Sent: Friday, October 14, 2022 9:11 PM > To: Jiawei(Jonny) Wang <[email protected]> > Cc: Raslan Darawsheh <[email protected]>; Asaf Penso <[email protected]>; > Matan Azrad <[email protected]>; Slava Ovsiienko > <[email protected]>; [email protected] > Subject: Re: [PATCH] doc: update sample action description for mlx5 > > 14/10/2022 12:14, Jiawei Wang: > > This patch adds mlx5 description about E-Switch mirroring flow > > (RTE_FLOW_ACTION_TYPE_SAMPLE with sample ratio=1) with encap action, > > then supports the uplink port only in the sample actions list or in > > the one flow. > > > > Signed-off-by: Jiawei Wang <[email protected]> > [...] > > + - For E-Switch mirroring flow, supports ``ENCAP`` actions only to the > > + UpLink Port and in either of sample actions list or in the flow. > > Sorry I don't understand the second part of the sentence: > "and in either of sample actions list or in the flow." > > Please could you explain so we can find a simpler wording? > RTE sample action support a list of actions that be defined as below structure: struct rte_flow_action_sample { uint32_t ratio; /**< packets sampled equals to '1/ratio'. */ const struct rte_flow_action *actions; /**< sub-action list specific for the sampling hit cases. */ }; So the second part meaning that 'encap' with 'port' combination can be in the 'sub-action' list of sample Or flow actions while user calls rte_flow_create(,, actions[]); >

