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 7E91AA00C2; Fri, 24 Apr 2020 13:08:08 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id C410F1BC24; Fri, 24 Apr 2020 13:08:07 +0200 (CEST) Received: from mail.chinasoftinc.com (unknown [114.113.233.8]) by dpdk.org (Postfix) with ESMTP id 00A7D1B5E1 for ; Fri, 24 Apr 2020 13:08:02 +0200 (CEST) Received: from localhost.localdomain (114.119.4.74) by INCCAS002.ito.icss (10.168.0.60) with Microsoft SMTP Server id 14.3.487.0; Fri, 24 Apr 2020 19:07:55 +0800 From: "Wei Hu (Xavier)" To: Date: Fri, 24 Apr 2020 19:07:50 +0800 Message-ID: <20200424110750.42456-1-huwei013@chinasoftinc.com> X-Mailer: git-send-email 2.23.0 MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Content-Type: text/plain X-Originating-IP: [114.119.4.74] Subject: [dpdk-dev] [PATCH] app/testpmd: fix Rx/Tx stats after clear stats command 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: Chengwen Feng Currently, when running start/clear stats&xstats/stop command many times based on testpmd application, there are incorrect RX/TX-packets stats as below: ---------------------- Forward statistics for port 0 -------------- RX-packets: 18446744073709544808 RX-dropped: 0 ...ignore TX-packets: 18446744073709536616 TX-dropped: 0 ...ignore -------------------------------------------------------------------- The root cause as below: 1. The struct rte_port of testpmd.h has a member variable "struct rte_eth_stats stats" to store the last port statistics. 2. When runnig start command, it execute cmd_start_parsed -> start_packet_forwarding -> fwd_stats_reset, which call rte_eth_stats_get API function to save current port statistics. 3. When running stop command, it execute fwd_stats_display, which call rte_eth_stats_get to get current port statistics, and then minus last port statistics. 4. If we run clear stats or xstats after start command, then run stop, it may display above incorrect stats because the current Rx/Tx-packets is lower than the last saved RX/TX-packets(uint64_t overflow). This patch fixes it by clearing last port statistics when executing "clear stats/xstats" command. Fixes: af75078fece3 ("first public release") Cc: stable@dpdk.org Signed-off-by: Chengwen Feng Signed-off-by: Wei Hu (Xavier) --- app/test-pmd/config.c | 11 +++++++++++ 1 file changed, 11 insertions(+) diff --git a/app/test-pmd/config.c b/app/test-pmd/config.c index 72f25d152..0d2375607 100644 --- a/app/test-pmd/config.c +++ b/app/test-pmd/config.c @@ -234,10 +234,16 @@ nic_stats_display(portid_t port_id) void nic_stats_clear(portid_t port_id) { + struct rte_port *port; + if (port_id_is_invalid(port_id, ENABLED_WARN)) { print_valid_ports(); return; } + + port = &ports[port_id]; + /* clear last port statistics because eth stats reset */ + memset(&port->stats, 0, sizeof(port->stats)); rte_eth_stats_reset(port_id); printf("\n NIC statistics for port %d cleared\n", port_id); } @@ -308,12 +314,17 @@ nic_xstats_display(portid_t port_id) void nic_xstats_clear(portid_t port_id) { + struct rte_port *port; int ret; if (port_id_is_invalid(port_id, ENABLED_WARN)) { print_valid_ports(); return; } + + port = &ports[port_id]; + /* clear last port statistics because eth xstats(include stats) reset */ + memset(&port->stats, 0, sizeof(port->stats)); ret = rte_eth_xstats_reset(port_id); if (ret != 0) { printf("%s: Error: failed to reset xstats (port %u): %s", -- 2.23.0