[dpdk-dev] [PATCH 12/14] test/eal: make the test pass again

David Marchand david.marchand at redhat.com
Tue Jun 4 15:50:48 CEST 2019


On Tue, Jun 4, 2019 at 3:29 PM Aaron Conole <aconole at redhat.com> wrote:

> Hi David and Michael,
>
> David Marchand <david.marchand at redhat.com> writes:
>
> > From: Michael Santana <msantana at redhat.com>
> >
> > The eal_flags_autotest test currently fails due to a memory leak in the
> > timer library[1][2]. This failure occurs when the test calls one of its
> > subtests test_file_prefix().
> >
> > Fixing the memory leak is not trivial, so this patch is a workaround that
> > makes the eal_flags_autotest test pass. This is accomplished by moving
> the
> > test_file_prefix test to its own test unit. This is a temporary measure
> > until the leak is fixed.
> >
> > [1] http://patchwork.dpdk.org/patch/53268/
> > [2] http://patchwork.dpdk.org/patch/53334/
> >
> > Signed-off-by: Michael Santana <msantana at redhat.com>
> > ---
>
> I'm wondering if it's better to just fix the leak outright.
>
> Then again, it might be useful to have the file-prefix test as an
> enumerable test anyway.
>

We have a lot of tests that embed subtests that can be entirely skipped at
the first subtest failure.
Their execution time can also be quite long making it harder to adjust the
timeout.
So splitting the existing tests into their subtests is something I have
been considering.


-- 
David Marchand


More information about the dev mailing list