Re: [PATCH v2] app/testpmd: fix indirect action list parameters parsing

2023-11-09 Thread Bruce Richardson
On Thu, Nov 09, 2023 at 11:41:37AM -0800, Stephen Hemminger wrote: > On Thu, 9 Nov 2023 20:36:37 +0200 > Gregory Etelson wrote: > > > diff --git a/app/test-pmd/cmdline_flow.c b/app/test-pmd/cmdline_flow.c > > index 0d521159e9..397f9bc3eb 100644 > > --- a/app/test-pmd/cmdline_flow.c > > +++ b/app/

Re: [PATCH v2] app/testpmd: fix indirect action list parameters parsing

2023-11-09 Thread Stephen Hemminger
On Thu, 9 Nov 2023 20:36:37 +0200 Gregory Etelson wrote: > diff --git a/app/test-pmd/cmdline_flow.c b/app/test-pmd/cmdline_flow.c > index 0d521159e9..397f9bc3eb 100644 > --- a/app/test-pmd/cmdline_flow.c > +++ b/app/test-pmd/cmdline_flow.c > @@ -11331,7 +11331,7 @@ parse_indlst_id2ptr(struct cont

[PATCH v2] app/testpmd: fix indirect action list parameters parsing

2023-11-09 Thread Gregory Etelson
Indirect actions list arguments parser was configured to place target number into 64bit value, while the code provided 32bits memory. The patch updated variable size for translation results. Fixes: 72a3dec7126f ("ethdev: add indirect flow list action") Signed-off-by: Gregory Etelson --- v2: defi