From 10d66cee5bf51bbd27d58b565b3340282e18ef07 Mon Sep 17 00:00:00 2001 From: Ilya Maximets Date: Fri, 25 Jan 2019 10:55:58 +0300 Subject: [PATCH] test: check zero socket memory as valid Dynamic memory mode allowes zero socket-mem because all the required memory could be allocated on demand. Fixes: 0a703f0f36c1 ("eal/linux: fix parsing zero socket memory and limits") Cc: stable@dpdk.org Reported-by: Shuai Zhu Signed-off-by: Ilya Maximets Reviewed-by: David Marchand Acked-by: Anatoly Burakov --- test/test/test_eal_flags.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/test/test/test_eal_flags.c b/test/test/test_eal_flags.c index e3a60c7ae1..81e345b879 100644 --- a/test/test/test_eal_flags.c +++ b/test/test/test_eal_flags.c @@ -1158,7 +1158,7 @@ test_memory_flags(void) const char *argv1[] = {prgname, "-c", "10", "-n", "2", "--file-prefix=" memtest, "-m", DEFAULT_MEM_SIZE}; - /* invalid (zero) --socket-mem flag */ + /* valid (zero) --socket-mem flag */ const char *argv2[] = {prgname, "-c", "10", "-n", "2", "--file-prefix=" memtest, "--socket-mem=0,0,0,0"}; @@ -1256,8 +1256,8 @@ test_memory_flags(void) printf("Error - process failed with valid -m flag!\n"); return -1; } - if (launch_proc(argv2) == 0) { - printf("Error - process run ok with invalid (zero) --socket-mem!\n"); + if (launch_proc(argv2) != 0) { + printf("Error - process failed with valid (zero) --socket-mem!\n"); return -1; } -- 2.20.1