RE: [PATCH 3/3] app/testpmd: fix meter mark handle update

2023-07-18 Thread Alexander Kozyrev
> On Tuesday, July 11, 2023 12:15 PM > Stephen Hemminger wrote: > On Tue, 11 Jul 2023 18:20:44 +0300 > Alexander Kozyrev wrote: > > > + case RTE_FLOW_ACTION_TYPE_METER_MARK: > > + rte_memcpy(&mtr_update.meter_mark, action->conf, > > + sizeof(struct rte_flow_action_m

Re: [PATCH 3/3] app/testpmd: fix meter mark handle update

2023-07-11 Thread Stephen Hemminger
On Tue, 11 Jul 2023 18:20:44 +0300 Alexander Kozyrev wrote: > + case RTE_FLOW_ACTION_TYPE_METER_MARK: > + rte_memcpy(&mtr_update.meter_mark, action->conf, > + sizeof(struct rte_flow_action_meter_mark)); Why use rte_memcpy? memcpy of fixed size is faster. Also,

[PATCH 3/3] app/testpmd: fix meter mark handle update

2023-07-11 Thread Alexander Kozyrev
The indirect action handle update for the METER_MERK action was implemented only for the async RTE Flow API. Allow updating the METER_MARK parameters via the old sync method. Fixes: 9c4a0c1859a3 ("ethdev: add meter color mark flow action") Signed-off-by: Alexander Kozyrev --- app/test-pmd/confi