[PATCH] net/mana: fix incorrectly reported counters in stats

[PATCH] net/mana: fix incorrectly reported counters in stats

[longli]

From: Long Li <longli@microsoft.com>

For per port counters and we should report summed values from all queues.

Cc: stable@dpdk.org
Signed-off-by: Long Li <longli@microsoft.com>
---
 drivers/net/mana/mana.c | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/drivers/net/mana/mana.c b/drivers/net/mana/mana.c
index 43221e743e..8a782c0d63 100644
--- a/drivers/net/mana/mana.c
+++ b/drivers/net/mana/mana.c
@@ -616,9 +616,9 @@ mana_dev_stats_get(struct rte_eth_dev *dev, struct rte_eth_stats *stats)
 		if (!txq)
 			continue;
 
-		stats->opackets = txq->stats.packets;
-		stats->obytes = txq->stats.bytes;
-		stats->oerrors = txq->stats.errors;
+		stats->opackets += txq->stats.packets;
+		stats->obytes += txq->stats.bytes;
+		stats->oerrors += txq->stats.errors;
 
 		if (i < RTE_ETHDEV_QUEUE_STAT_CNTRS) {
 			stats->q_opackets[i] = txq->stats.packets;
@@ -633,9 +633,9 @@ mana_dev_stats_get(struct rte_eth_dev *dev, struct rte_eth_stats *stats)
 		if (!rxq)
 			continue;
 
-		stats->ipackets = rxq->stats.packets;
-		stats->ibytes = rxq->stats.bytes;
-		stats->ierrors = rxq->stats.errors;
+		stats->ipackets += rxq->stats.packets;
+		stats->ibytes += rxq->stats.bytes;
+		stats->ierrors += rxq->stats.errors;
 
 		/* There is no good way to get stats->imissed, not setting it */
 
-- 
2.32.0

Re: [PATCH] net/mana: fix incorrectly reported counters in stats

[Ferruh Yigit]

On 2/22/2023 11:27 PM, longli@linuxonhyperv.com wrote:
> From: Long Li <longli@microsoft.com>
> 
> For per port counters and we should report summed values from all queues.
> 
> Cc: stable@dpdk.org

    Fixes: e350b56889bb ("net/mana: report queue statistics")
    Cc: stable@dpdk.org

> Signed-off-by: Long Li <longli@microsoft.com>

Acked-by: Ferruh Yigit <ferruh.yigit@amd.com>


Applied to dpdk-next-net/main, thanks.



It seems this issue commented on v4 [1], fixed in v5 [1], but issue
crept in back in next versions resulting it to be merged.

[1]
https://inbox.dpdk.org/dev/9fc85998-0802-f1f4-fa97-86114a511e76@xilinx.com/

[2]
https://inbox.dpdk.org/dev/1661560509-11009-18-git-send-email-longli@linuxonhyperv.com/