test: fix hugepage file handling in EAL flags autotest
authorAnatoly Burakov <anatoly.burakov@intel.com>
Thu, 15 Nov 2018 12:18:05 +0000 (12:18 +0000)
committerThomas Monjalon <thomas@monjalon.net>
Sun, 18 Nov 2018 22:08:02 +0000 (23:08 +0100)
commit651cc78f83b53743fcf265bbb35579ab44871dfd
tree3d7f7d48988bd64194176a670f5f7283f0a70a5d
parent91dc9c13ba978fb8147240ed6fa20c41145bf0db
test: fix hugepage file handling in EAL flags autotest

Before 18.05, DPDK could not release memory back to the system
neither at runtime nor before shutting down. Over the course of
18.05 up to 18.11, code was introduced to release memory at
runtime, as well as an rte_eal_cleanup() function that is supposed
to release all EAL-allocated memory before shutting down DPDK.

When 3f9e31d71d63 ("test: clean up on exit") was introduced, the
test application started to use rte_eal_cleanup() to release all
used memory after execution. However, the EAL flags autotest
still relies on the old behavior of leaving stuff behind in the
hugetlbfs.

The fix is twofold. First, the test to check for leftover files
in hugetlbfs is no longer valid as it is, because test application
now removes all files from hugetlbfs after exit. However, if we
use the --legacy-mem option, then old behavior of leaving files
in hugetlbfs after execution is restored. So the first fix is to
add --legacy-mem to all the tests that expect files in hugetlbfs
to be leftover.

However, we also need to test if default memory mode *doesn't*
leave any files behind, so we also extend the test to check for
these scenarios as well. So, both memtest1 and memtest2 are run
in legacy and default mem modes, and are checked for any leftover
files that are or are not supposed to be there.

Fixes: 3f9e31d71d63 ("test: clean up on exit")

Signed-off-by: Anatoly Burakov <anatoly.burakov@intel.com>
test/test/test_eal_flags.c