On 25.01.2019 10:55, Ilya Maximets wrote: > Dynamic memory mode allowes zero socket-mem because all the > required memory could be allocated on demand. > > Fixes: 339c2244b4f1 ("eal: fix parsing zero socket memory and limits") > Cc: sta...@dpdk.org >
Reported-by: Shuai Zhu <shuaix....@intel.com> > Signed-off-by: Ilya Maximets <i.maxim...@samsung.com> > --- > 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 e3a60c7ae..81e345b87 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; > } > >