From 8d085a0ab340b787d4063976cd223b7fd08a13c5 Mon Sep 17 00:00:00 2001 From: Marvin Liu Date: Wed, 29 Jul 2015 11:54:29 +0800 Subject: [PATCH] app/test: fix combined options --no-huge and -m '--no-huge' option now can workable with -m option. Unit test for eal flag should change pass criterion. Fixes: a7de7e6beb69 ("eal: allow combining -m and --no-huge") Signed-off-by: Marvin Liu --- app/test/test_eal_flags.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/app/test/test_eal_flags.c b/app/test/test_eal_flags.c index 0352f874e8..e6f7035310 100644 --- a/app/test/test_eal_flags.c +++ b/app/test/test_eal_flags.c @@ -748,8 +748,8 @@ test_no_hpet_flag(void) } /* - * Test that the app runs with --no-huge and doesn't run when either - * -m or --socket-mem are specified with --no-huge. + * Test that the app runs with --no-huge and doesn't run when --socket-mem are + * specified with --no-huge. */ static int test_no_huge_flag(void) @@ -778,8 +778,8 @@ test_no_huge_flag(void) printf("Error - process did not run ok with --no-huge flag\n"); return -1; } - if (launch_proc(argv2) == 0) { - printf("Error - process run ok with --no-huge and -m flags\n"); + if (launch_proc(argv2) != 0) { + printf("Error - process did not run ok with --no-huge and -m flags\n"); return -1; } #ifdef RTE_EXEC_ENV_BSDAPP -- 2.20.1