[dpdk-dev] [PATCH v2] testpmd: call cleanup on exit
Stephen Hemminger
stephen at networkplumber.org
Wed Jan 8 16:28:15 CET 2020
On Wed, 8 Jan 2020 09:45:54 +0000
"Iremonger, Bernard" <bernard.iremonger at intel.com> wrote:
> Hi Stephen,
>
> > -----Original Message-----
> > From: dev <dev-bounces at dpdk.org> On Behalf Of Stephen Hemminger
> > Sent: Tuesday, January 7, 2020 7:00 PM
> > To: dev at dpdk.org
> > Cc: Stephen Hemminger <stephen at networkplumber.org>
> > Subject: [dpdk-dev] [PATCH v2] testpmd: call cleanup on exit
> >
> > The rte_eal_cleanup code is not exercised by testpmd which is the most
> > used DPDK test tool. Add a call at end of program.
> >
> > This helps exercise free and close paths which can be checked with tools like
> > valgrind.
> >
> > Signed-off-by: Stephen Hemminger <stephen at networkplumber.org>
> > ---
> > v2 - report errors
> >
> > app/test-pmd/testpmd.c | 7 ++++++-
> > 1 file changed, 6 insertions(+), 1 deletion(-)
> >
> > diff --git a/app/test-pmd/testpmd.c b/app/test-pmd/testpmd.c index
> > b3746822366f..2eec8afda1ec 100644
> > --- a/app/test-pmd/testpmd.c
> > +++ b/app/test-pmd/testpmd.c
> > @@ -3570,5 +3570,10 @@ main(int argc, char** argv)
> > return 1;
> > }
> >
> > - return 0;
> > + ret = rte_eal_cleanup();
> > + if (ret != 0)
> > + rte_exit(EXIT_FAILURE,
> > + "EAL cleanup failed: %s\n", strerror(-ret));
> > +
> > + return EXIT_SUCCESS;
> > }
> > --
> > 2.20.1
>
> This looks like a fix to me and is probably worth backporting, it should probably have a fixes line.
>
> Regards,
>
> Bernard.
>
Not sure if it needs to be backported. It depends on the definition
of a fix. The original definition of stable was fixes only.
Linux has moved on to fixes and backports of missing features
like this; but DPDK has mostly stuck to the original definition
of stable.
More information about the dev
mailing list