From mboxrd@z Thu Jan 1 00:00:00 1970 From: Phil Yang Subject: [PATCH] test/eal_flags: Optimize memory flags test Date: Tue, 6 Feb 2018 10:21:38 +0800 Message-ID: <1517883698-12921-1-git-send-email-phil.yang@arm.com> Cc: nd@arm.com, phil.yang@arm.com, anatoly.burakov@intel.com, herbert.guan@arm.com To: dev@dpdk.org Return-path: Received: from foss.arm.com (foss.arm.com [217.140.101.70]) by dpdk.org (Postfix) with ESMTP id EBF711B3ED for ; Tue, 6 Feb 2018 03:21:53 +0100 (CET) List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" Since RTE_MAX_NUMA_NODES is configurable, the existing socket number could greater than RTE_MAX_NUMA_NODES. Optimize test case to cover this situation.(i.e RTE_MAX_NUMA_NODES=1) Fixes: 45f1b6e8680a ('app: add new tests on eal flags') Signed-off-by: Phil Yang --- test/test/test_eal_flags.c | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/test/test/test_eal_flags.c b/test/test/test_eal_flags.c index 37c42ef..7df0b8f 100644 --- a/test/test/test_eal_flags.c +++ b/test/test/test_eal_flags.c @@ -33,7 +33,7 @@ #define memtest "memtest" #define memtest1 "memtest1" #define memtest2 "memtest2" -#define SOCKET_MEM_STRLEN (RTE_MAX_NUMA_NODES * 10) +#define SOCKET_MEM_STRLEN (RTE_MAX_NUMA_NODES * 20) #define launch_proc(ARGV) process_dup(ARGV, \ sizeof(ARGV)/(sizeof(ARGV[0])), __func__) @@ -1138,10 +1138,11 @@ test_memory_flags(void) #ifdef RTE_EXEC_ENV_BSDAPP int i, num_sockets = 1; #else - int i, num_sockets = get_number_of_sockets(); + int i, num_sockets = RTE_MIN(get_number_of_sockets(), + RTE_MAX_NUMA_NODES); #endif - if (num_sockets <= 0 || num_sockets > RTE_MAX_NUMA_NODES) { + if (num_sockets <= 0) { printf("Error - cannot get number of sockets!\n"); return -1; } -- 2.7.4