On Tue, Jun 4, 2019 at 3:29 PM Aaron Conole <acon...@redhat.com> wrote:
> Hi David and Michael, > > David Marchand <david.march...@redhat.com> writes: > > > From: Michael Santana <msant...@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 <msant...@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
