From 084e26e4d998c735f51da1c57a400c4cd31d54a6 Mon Sep 17 00:00:00 2001 From: Neha Bisht Date: Mon, 25 Oct 2021 18:53:53 +0530 Subject: [PATCH] qcacmn: Fix double increment of tx ifconfig stats In ifconfig athX stats, tx packets and bytes were getting double incremented in NSS offload disabled case. It was due to tx.comp_pkt stats getting updated both in per packet Tx completion path and in Tx per ppdu indication from HTT path. Fix by updating stats only in the Tx. completion path. Change-Id: Id75d914874480a4e24ccb13223818b67869eb378 --- dp/wifi3.0/monitor/1.0/dp_mon_1.0.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/dp/wifi3.0/monitor/1.0/dp_mon_1.0.c b/dp/wifi3.0/monitor/1.0/dp_mon_1.0.c index d70e738689..b6706ca804 100644 --- a/dp/wifi3.0/monitor/1.0/dp_mon_1.0.c +++ b/dp/wifi3.0/monitor/1.0/dp_mon_1.0.c @@ -1240,9 +1240,6 @@ dp_tx_stats_update(struct dp_pdev *pdev, struct dp_peer *peer, DP_STATS_INC(peer, tx.transmit_type[ppdu->ppdu_type].mpdu_tried, mpdu_tried); - DP_STATS_INC_PKT(peer, tx.comp_pkt, - num_msdu, (ppdu->success_bytes + - ppdu->retry_bytes + ppdu->failed_bytes)); DP_STATS_UPD(peer, tx.tx_rate, ppdu->tx_rate); DP_STATS_INC(peer, tx.sgi_count[ppdu->gi], num_msdu); DP_STATS_INC(peer, tx.bw[ppdu->bw], num_msdu);