[dpdk-dev] [PATCH 12/14] test/eal: make the test pass again
David Marchand
david.marchand at redhat.com
Tue Jun 4 10:59:50 CEST 2019
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>
---
app/test/autotest_data.py | 6 ++++++
app/test/meson.build | 1 +
app/test/test_eal_flags.c | 7 +------
3 files changed, 8 insertions(+), 6 deletions(-)
diff --git a/app/test/autotest_data.py b/app/test/autotest_data.py
index 6cf7eca..15e672f 100644
--- a/app/test/autotest_data.py
+++ b/app/test/autotest_data.py
@@ -93,6 +93,12 @@
"Report": None,
},
{
+ "Name": "EAL flags file prefix autotest",
+ "Command": "eal_flags_prefix_autotest",
+ "Func": default_autotest,
+ "Report": None,
+ },
+ {
"Name": "Hash autotest",
"Command": "hash_autotest",
"Func": default_autotest,
diff --git a/app/test/meson.build b/app/test/meson.build
index 7ad3684..212cd1b 100644
--- a/app/test/meson.build
+++ b/app/test/meson.build
@@ -156,6 +156,7 @@ fast_parallel_test_names = [
'cycles_autotest',
'debug_autotest',
'eal_flags_autotest',
+ 'eal_flags_prefix_autotest',
'eal_fs_autotest',
'errno_autotest',
'event_ring_autotest',
diff --git a/app/test/test_eal_flags.c b/app/test/test_eal_flags.c
index cfa8a61..1e227aa 100644
--- a/app/test/test_eal_flags.c
+++ b/app/test/test_eal_flags.c
@@ -1397,12 +1397,6 @@ enum hugepage_action {
return ret;
}
- ret = test_file_prefix();
- if (ret < 0) {
- printf("Error in test_file_prefix()\n");
- return ret;
- }
-
ret = test_misc_flags();
if (ret < 0) {
printf("Error in test_misc_flags()");
@@ -1413,3 +1407,4 @@ enum hugepage_action {
}
REGISTER_TEST_COMMAND(eal_flags_autotest, test_eal_flags);
+REGISTER_TEST_COMMAND(eal_flags_prefix_autotest, test_file_prefix);
--
1.8.3.1
More information about the dev
mailing list