[PATCH v10 3/6] flow_parser: add shared parser library

Thomas Monjalon thomas at monjalon.net
Fri Feb 6 15:01:57 CET 2026


04/02/2026 15:53, Stephen Hemminger:
> On Tue, 3 Feb 2026 09:34:26 +0100
> Lukáš Šišmiš <sismis at dyna-nic.com> wrote:
> 
> > >
> > > The kernel version of checkpatch complains here. The DPDK shell script
> > > seems to be set to ignore this but.
> > >
> > > WARNING: break is not useful after a return
> > > #15008: FILE: lib/flow_parser/rte_flow_parser.c:14763:
> > > +               return cmd_flow_parsed(out);
> > > +               break;
> > >
> > > Should I create a new patch set or just let it be at this moment?  
> > Lukas
> 
> 
> I am ok with it as is.

Better to update.

There are other warnings:

WARNING:STRNCPY: Prefer strscpy, strscpy_pad, or __nonstring over strncpy - see: https://github.com/KSPP/linux/issues/90
#13052: FILE: lib/flow_parser/rte_flow_parser.c:12825:
+       strncpy(buf, str, len);

and a lot of WARNING:LONG_LINE


And on a more general note, I would have expected to ask the opinion
of rte_flow maintainers, but they are not Cc'ed in these patches.

I'm a bit skeptical about adding this outside of ethdev library
which defines the flow API.




More information about the dev mailing list