Hi Thomas, > -----Original Message----- > From: Thomas Monjalon <[email protected]> > Sent: Tuesday, March 9, 2021 9:39 PM > To: Jiawei(Jonny) Wang <[email protected]> > Cc: Slava Ovsiienko <[email protected]>; Matan Azrad > <[email protected]>; Ori Kam <[email protected]>; [email protected]; Raslan > Darawsheh <[email protected]> > Subject: Re: [PATCH] doc: update sample actions support in mlx5 guide > > 09/03/2021 14:09, Jiawei Wang: > > + - Supports ``MARK``, ``COUNT``, ``QUEUE``, ``RSS`` as sample actions for > NIC Rx sampling/mirroring flow. > > "sample actions for NIC Rx sampling" is redundant. > I suggest "Supports [...] actions for NIC Rx flow sampling/mirroring." >
These actions are support in the sample action list, user also can add modify, decap , encap in the NIC RX flow; How about this: "For NIC RX flow with sample action, Supports [...] actions in the sample action list" ? > > + - Supports ``RAW ENCAP``, ``Port ID`` as sample actions for E-Switch > mirroring (sample ratio = 1) flow. > > I suggest "Supports [...] actions for E-Switch flow mirroring." > > If lines are long, do not hesitate to start a new line after a punctuation. > Ok, I will split into two lines. Thanks.
