From patchwork Fri Jul 12 06:43:20 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Vamsi Attunuru X-Patchwork-Id: 56382 X-Patchwork-Delegate: thomas@monjalon.net Return-Path: X-Original-To: patchwork@dpdk.org Delivered-To: patchwork@dpdk.org Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 7E8EE3977; Fri, 12 Jul 2019 08:43:30 +0200 (CEST) Received: from mx0b-0016f401.pphosted.com (mx0a-0016f401.pphosted.com [67.231.148.174]) by dpdk.org (Postfix) with ESMTP id 3B630322C for ; Fri, 12 Jul 2019 08:43:29 +0200 (CEST) Received: from pps.filterd (m0045849.ppops.net [127.0.0.1]) by mx0a-0016f401.pphosted.com (8.16.0.27/8.16.0.27) with SMTP id x6C6ePnQ015726; Thu, 11 Jul 2019 23:43:27 -0700 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=marvell.com; h=from : to : cc : subject : date : message-id : in-reply-to : references : mime-version : content-type; s=pfpt0818; bh=xPyzsWNpL7xYmqECQPg9NwScYV8pHzcXb2UM5Ade9rs=; b=OwLIsFd7eH0pj15BaC0ZnN7vOFmgJQga0MumArAuQ7hln3wdutovKNmCTjRh2S5cvz66 nU7eCZwAc+OI20E3gcqd0l9InM4IHkx9W+vO4WLN55Wj6poexF6tZ6lmCPj3974YDUmb jevxDXkR9miHM7Goi4gdbHg0ICuvWf2scVZwP7iPD8L3oA2GFjtvsU3WKZqU1rNzzWUZ ZxVzDcpsI8B/hD3Fn48jQUKtw6v7QqQXhIJuRD/eKiXM2G98hGmEZLoNaHDA8z2jNFV6 und/mqFg0J/mj0tgX3Z0uEdvzrDqHfw2vWXMR+SOOteJEr3S/90f5QZb11ugVT90BI7x Hg== Received: from sc-exch03.marvell.com ([199.233.58.183]) by mx0a-0016f401.pphosted.com with ESMTP id 2tpjxxrdkk-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-SHA384 bits=256 verify=NOT); Thu, 11 Jul 2019 23:43:27 -0700 Received: from SC-EXCH01.marvell.com (10.93.176.81) by SC-EXCH03.marvell.com (10.93.176.83) with Microsoft SMTP Server (TLS) id 15.0.1367.3; Thu, 11 Jul 2019 23:43:26 -0700 Received: from maili.marvell.com (10.93.176.43) by SC-EXCH01.marvell.com (10.93.176.81) with Microsoft SMTP Server id 15.0.1367.3 via Frontend Transport; Thu, 11 Jul 2019 23:43:26 -0700 Received: from hyd1vattunuru-dt.caveonetworks.com (unknown [10.29.52.72]) by maili.marvell.com (Postfix) with ESMTP id CAAA93F703F; Thu, 11 Jul 2019 23:43:23 -0700 (PDT) From: To: CC: , , , , Vamsi Attunuru Date: Fri, 12 Jul 2019 12:13:20 +0530 Message-ID: <20190712064320.8106-1-vattunuru@marvell.com> X-Mailer: git-send-email 2.8.4 In-Reply-To: <20190711051532.21054-1-vattunuru@marvell.com> References: <20190711051532.21054-1-vattunuru@marvell.com> MIME-Version: 1.0 X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10434:, , definitions=2019-07-12_02:, , signatures=0 Subject: [dpdk-dev] [PATCH v3 1/1] app/test: fix --socket-mem option in eal flag autotest X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" From: Vamsi Attunuru eal flag autotest fails when multiple mem size flags are passed to --socket-mem option irrespective of RTE_MAX_NUMA_NODES. Patch fixes --socket-mem option by setting enough numbers of numa node mem flags based on RTE_MAX_NUMA_NODES config. Fixes: 45f1b6e8680a ("app: add new tests on eal flags") Signed-off-by: Vamsi Attunuru Reviewed-by: Vamsi Attunuru Tested-by: Vamsi Attunuru --- V3 changes: * Pass buffer size info to populate socket mem routine which strlcpy requires. V2 changes: * Add routine to populate --socket-mem option and use it to form valid and invalid test commands. app/test/test_eal_flags.c | 121 +++++++++++++++++++++++++++------------------- 1 file changed, 71 insertions(+), 50 deletions(-) diff --git a/app/test/test_eal_flags.c b/app/test/test_eal_flags.c index 672ca0a..053d694 100644 --- a/app/test/test_eal_flags.c +++ b/app/test/test_eal_flags.c @@ -1250,12 +1250,46 @@ test_file_prefix(void) return 0; } +static void +populate_socket_mem_param(int num_sockets, const char *mem, + char *buf, size_t buf_size) +{ + char tmp[SOCKET_MEM_STRLEN]; + int i; + + snprintf(buf, SOCKET_MEM_STRLEN, "--socket-mem="); + + for (i = 0; i < num_sockets ; i++) { + snprintf(tmp, sizeof(tmp), "%s%s", buf, mem); + strlcpy(buf, tmp, buf_size); + + if (num_sockets + 1 - i > 1) { + snprintf(tmp, sizeof(tmp), "%s,", buf); + strlcpy(buf, tmp, buf_size); + } + } +} + /* * Tests for correct handling of -m and --socket-mem flags */ static int test_memory_flags(void) { + char buf[SOCKET_MEM_STRLEN]; /* to avoid copying string onto itself */ + +#ifdef RTE_EXEC_ENV_FREEBSD + int num_sockets = 1; +#else + int num_sockets = RTE_MIN(get_number_of_sockets(), + RTE_MAX_NUMA_NODES); +#endif + + if (num_sockets <= 0) { + printf("Error - cannot get number of sockets!\n"); + return -1; + } + #ifdef RTE_EXEC_ENV_FREEBSD /* BSD target doesn't support prefixes at this point */ const char * prefix = ""; @@ -1277,85 +1311,72 @@ test_memory_flags(void) "--file-prefix=" memtest, "-m", DEFAULT_MEM_SIZE}; /* valid (zero) --socket-mem flag */ + char arg2_socket_mem[SOCKET_MEM_STRLEN]; + populate_socket_mem_param(num_sockets - 1, "0", buf, sizeof(buf)); + snprintf(arg2_socket_mem, sizeof(arg2_socket_mem), "%s%s", buf, "0"); const char *argv2[] = {prgname, - "--file-prefix=" memtest, "--socket-mem=0,0,0,0"}; + "--file-prefix=" memtest, arg2_socket_mem}; /* invalid (incomplete) --socket-mem flag */ + char arg3_socket_mem[SOCKET_MEM_STRLEN]; + populate_socket_mem_param(num_sockets - 1, "2", buf, sizeof(buf)); + snprintf(arg3_socket_mem, sizeof(arg3_socket_mem), "%s%s", buf, "2,"); const char *argv3[] = {prgname, - "--file-prefix=" memtest, "--socket-mem=2,2,"}; + "--file-prefix=" memtest, arg3_socket_mem}; /* invalid (mixed with invalid data) --socket-mem flag */ + char arg4_socket_mem[SOCKET_MEM_STRLEN]; + populate_socket_mem_param(num_sockets - 1, "2", buf, sizeof(buf)); + snprintf(arg4_socket_mem, sizeof(arg4_socket_mem), "%s%s", buf, "Fred"); const char *argv4[] = {prgname, - "--file-prefix=" memtest, "--socket-mem=2,2,Fred"}; + "--file-prefix=" memtest, arg4_socket_mem}; /* invalid (with numeric value as last character) --socket-mem flag */ + char arg5_socket_mem[SOCKET_MEM_STRLEN]; + populate_socket_mem_param(num_sockets - 1, "2", buf, sizeof(buf)); + snprintf(arg5_socket_mem, sizeof(arg5_socket_mem), + "%s%s", buf, "Fred0"); const char *argv5[] = {prgname, - "--file-prefix=" memtest, "--socket-mem=2,2,Fred0"}; + "--file-prefix=" memtest, arg5_socket_mem}; /* invalid (with empty socket) --socket-mem flag */ + char arg6_socket_mem[SOCKET_MEM_STRLEN]; + populate_socket_mem_param(num_sockets - 1, "2", buf, sizeof(buf)); + snprintf(arg6_socket_mem, sizeof(arg6_socket_mem), "%s%s", buf, ",,"); const char *argv6[] = {prgname, - "--file-prefix=" memtest, "--socket-mem=2,,2"}; + "--file-prefix=" memtest, arg6_socket_mem}; /* invalid (null) --socket-mem flag */ const char *argv7[] = {prgname, "--file-prefix=" memtest, "--socket-mem="}; /* valid --socket-mem specified together with -m flag */ + char arg8_socket_mem[SOCKET_MEM_STRLEN]; + populate_socket_mem_param(num_sockets - 1, "2", buf, sizeof(buf)); + snprintf(arg8_socket_mem, sizeof(arg8_socket_mem), "%s%s", buf, "2"); const char *argv8[] = {prgname, - "--file-prefix=" memtest, "-m", DEFAULT_MEM_SIZE, "--socket-mem=2,2"}; + "--file-prefix=" memtest, "-m", + DEFAULT_MEM_SIZE, arg8_socket_mem}; /* construct an invalid socket mask with 2 megs on each socket plus * extra 2 megs on socket that doesn't exist on current system */ char invalid_socket_mem[SOCKET_MEM_STRLEN]; - char buf[SOCKET_MEM_STRLEN]; /* to avoid copying string onto itself */ - -#ifdef RTE_EXEC_ENV_FREEBSD - int i, num_sockets = 1; -#else - int i, num_sockets = RTE_MIN(get_number_of_sockets(), - RTE_MAX_NUMA_NODES); -#endif - - if (num_sockets <= 0) { - printf("Error - cannot get number of sockets!\n"); - return -1; - } - - snprintf(invalid_socket_mem, sizeof(invalid_socket_mem), "--socket-mem="); - - /* add one extra socket */ - for (i = 0; i < num_sockets + 1; i++) { - snprintf(buf, sizeof(buf), "%s%s", invalid_socket_mem, DEFAULT_MEM_SIZE); - strlcpy(invalid_socket_mem, buf, sizeof(invalid_socket_mem)); - - if (num_sockets + 1 - i > 1) { - snprintf(buf, sizeof(buf), "%s,", invalid_socket_mem); - strlcpy(invalid_socket_mem, buf, - sizeof(invalid_socket_mem)); - } - } - - /* construct a valid socket mask with 2 megs on each existing socket */ - char valid_socket_mem[SOCKET_MEM_STRLEN]; - - snprintf(valid_socket_mem, sizeof(valid_socket_mem), "--socket-mem="); - - /* add one extra socket */ - for (i = 0; i < num_sockets; i++) { - snprintf(buf, sizeof(buf), "%s%s", valid_socket_mem, DEFAULT_MEM_SIZE); - strlcpy(valid_socket_mem, buf, sizeof(valid_socket_mem)); - - if (num_sockets - i > 1) { - snprintf(buf, sizeof(buf), "%s,", valid_socket_mem); - strlcpy(valid_socket_mem, buf, - sizeof(valid_socket_mem)); - } - } + populate_socket_mem_param(num_sockets, DEFAULT_MEM_SIZE, + buf, sizeof(buf)); + snprintf(invalid_socket_mem, sizeof(invalid_socket_mem), + "%s%s", buf, DEFAULT_MEM_SIZE); /* invalid --socket-mem flag (with extra socket) */ const char *argv9[] = {prgname, "--file-prefix=" memtest, invalid_socket_mem}; + /* construct a valid socket mask with 2 megs on each existing socket */ + char valid_socket_mem[SOCKET_MEM_STRLEN]; + populate_socket_mem_param(num_sockets - 1, DEFAULT_MEM_SIZE, + buf, sizeof(buf)); + snprintf(valid_socket_mem, sizeof(valid_socket_mem), + "%s%s", buf, DEFAULT_MEM_SIZE); + /* valid --socket-mem flag */ const char *argv10[] = {prgname, "--file-prefix=" memtest, valid_socket_mem};