From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id 904ADA04F2; Mon, 8 Jun 2020 18:57:05 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id E30BC374C; Mon, 8 Jun 2020 18:57:04 +0200 (CEST) Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by dpdk.org (Postfix) with ESMTP id 056BA2BF7; Mon, 8 Jun 2020 07:50:05 +0200 (CEST) Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 508CE1FB; Sun, 7 Jun 2020 22:50:05 -0700 (PDT) Received: from net-arm-c2400.shanghai.arm.com (net-arm-c2400.shanghai.arm.com [10.169.40.212]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPA id 304DD3F52E; Sun, 7 Jun 2020 22:50:01 -0700 (PDT) From: Feifei Wang To: Honnappa Nagarahalli , Konstantin Ananyev , Ruifeng Wang , Olivier Matz , Ola Liljedahl , Dharmik Thakkar Cc: dev@dpdk.org, nd@arm.com, Feifei Wang , stable@dpdk.org Date: Mon, 8 Jun 2020 13:49:38 +0800 Message-Id: <20200608054938.46599-1-feifei.wang2@arm.com> X-Mailer: git-send-email 2.17.1 X-Mailman-Approved-At: Mon, 08 Jun 2020 18:57:03 +0200 Subject: [dpdk-dev] [PATCH] test/ring_perf: fix error statistics in bulk enq/dequeue 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" In size 32 bulk ring enq/dequeue performance test, the "Total count" statistics is incorrect. For example, running the test on lcore 25 and lcore 26, the output is as follows: The test command: $sudo ./arm64-armv8a-linuxapp-gcc/app/test -l 25-26 RTE>>ring_perf_autotest Bulk enq/dequeue count on size 32 Core [25] count = 288268 Core [26] count = 288281 Total count (size: 32): 1066323 Fixed it by reset the counter at the beginning of each loop. The revised output is as follows: Bulk enq/dequeue count on size 32 Core [25] count = 285643 Core [26] count = 285688 Total count (size: 32): 571331 Fixes: 759cf9b5632c ("test/ring: enhance mp/mc coverage") Cc: stable@dpdk.org Signed-off-by: Feifei Wang Reviewed-by: Ruifeng Wang Reviewed-by: Phil Yang --- app/test/test_ring_perf.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/app/test/test_ring_perf.c b/app/test/test_ring_perf.c index ee21faf71..ac9bf5608 100644 --- a/app/test/test_ring_perf.c +++ b/app/test/test_ring_perf.c @@ -380,7 +380,7 @@ load_loop_fn_16B(void *p) static int run_on_all_cores(struct rte_ring *r, const int esize) { - uint64_t total = 0; + uint64_t total; struct thread_params param; lcore_function_t *lcore_f; unsigned int i, c; @@ -392,6 +392,7 @@ run_on_all_cores(struct rte_ring *r, const int esize) memset(¶m, 0, sizeof(struct thread_params)); for (i = 0; i < RTE_DIM(bulk_sizes); i++) { + total = 0; printf("\nBulk enq/dequeue count on size %u\n", bulk_sizes[i]); param.size = bulk_sizes[i]; param.r = r; -- 2.17.1