Re: patch 'examples/l3fwd: fix scalar LPM' has been queued to stable release 21.11.2

["Stanisław Kardach" <kda@semihalf.com>]

On Thu, Jun 9, 2022 at 3:17 PM Kevin Traynor <ktraynor@redhat.com> wrote:
>
> On 09/06/2022 14:12, Kevin Traynor wrote:
> > On 09/06/2022 13:53, Stanisław Kardach wrote:
> >> On Thu, Jun 9, 2022 at 1:37 PM Kevin Traynor <ktraynor@redhat.com> wrote:
> >>>
> >>> Hi,
> >>>
> >>> FYI, your patch has been queued to stable release 21.11.2
> >>>
> >>> Note it hasn't been pushed to http://dpdk.org/browse/dpdk-stable yet.
> >>> It will be pushed if I get no objections before 06/13/22. So please
> >>> shout if anyone has objections.
> >>>
> >> Hi Kevin,
> >>
> >
> > Hi Stanslaw,
> >
> >> I'm not sure if this patch has to go to stable. Main reason being that
> >> this issue won't be encountered unless someone explicitly enables the
> >> scalar code. Until RISC-V port this never happened.
> >>
> >
> > Reading the commit, it seemed like a logic error which is why i thought
> > maybe it should be backported. I'm fine with not backporting it as
> > you've explained it's not needed. Also, it's just example/test code.
You're totally right, it's a logical error. Just by pure luck it's not
a compile error.
> >
> >> Actually same goes for:
> >>     a9fe152363e2 ("test/ring: add custom element size functional tests")
> >> that is in the stable queue too.
> >>
>
> Checked and that ^ commit is in since DPDK 20.02. Just to confirm, I
> think you meant this one?
>
>  From 3ec90e3ec0b68d06e9b9494798b79f2abaa2a49f Mon Sep 17 00:00:00 2001
> From: Stanislaw Kardach <kda@semihalf.com>
> Date: Wed, 11 May 2022 17:07:25 +0200
> Subject: [PATCH] test/ring: remove excessive inlining
>
> [ upstream commit 981a025741f80b4036f994ed30b04e757299ab19 ]
Yes, sorry I did a copy paste of a wrong hash by mistake.
>
> >
> > Ack.
> >
> > Thanks for checking the commits and replying :-)
> > Kevin.
> >
> >> Best Regards,
> >> Stanisław Kardach
> >>
> >
>