On Thu, 5 May 2022 19:29:54 +0200 Stanislaw Kardach <k...@semihalf.com> wrote:
> The lpm_process_event_pkt() can either process a packet using an > architecture specific (defined for X86/SSE, ARM/Neon and PPC64/Altivec) > path or a scalar one. The choice is however done using an ifdef > pre-processor macro. Because of that the scalar version was apparently > not widely excersized/compiled. > Due to some copy/paste errors, the scalar logic in > lpm_process_event_pkt() retained a "continue" statement where a BAD_PORT > should be returned after refactoring of the LPM logic in the l3fwd > example. > > Fixes: 99fc91d18082 ("examples/l3fwd: add event lpm main loop") > Cc: pbhagavat...@marvell.com > > Signed-off-by: Stanislaw Kardach <k...@semihalf.com> > Sponsored-by: Frank Zhao <frank.z...@starfivetech.com> > Sponsored-by: Sam Grove <sam.gr...@sifive.com> Would be easier to get merged if bug fixes came as separate patch submission. Also have not seen Sponsored-by before; what do you expect it to mean? Never used in DPDK or kernel git tree.