ax88796c: fix fetching error stats from percpu containers
authorAlexander Lobakin <alobakin@pm.me>
Sat, 23 Oct 2021 12:19:16 +0000 (12:19 +0000)
committerJakub Kicinski <kuba@kernel.org>
Tue, 26 Oct 2021 00:33:16 +0000 (17:33 -0700)
rx_dropped, tx_dropped, rx_frame_errors and rx_crc_errors are being
wrongly fetched from the target container rather than source percpu
ones.
No idea if that goes from the vendor driver or was brainoed during
the refactoring, but fix it either way.

Fixes: a97c69ba4f30e ("net: ax88796c: ASIX AX88796C SPI Ethernet Adapter Driver")
Signed-off-by: Alexander Lobakin <alobakin@pm.me>
Acked-by: Ɓukasz Stelmach <l.stelmach@samsung.com>
Link: https://lore.kernel.org/r/20211023121148.113466-1-alobakin@pm.me
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
drivers/net/ethernet/asix/ax88796c_main.c

index cfc597f..91fa049 100644 (file)
@@ -672,10 +672,10 @@ static void ax88796c_get_stats64(struct net_device *ndev,
                stats->tx_packets += tx_packets;
                stats->tx_bytes   += tx_bytes;
 
-               rx_dropped      += stats->rx_dropped;
-               tx_dropped      += stats->tx_dropped;
-               rx_frame_errors += stats->rx_frame_errors;
-               rx_crc_errors   += stats->rx_crc_errors;
+               rx_dropped      += s->rx_dropped;
+               tx_dropped      += s->tx_dropped;
+               rx_frame_errors += s->rx_frame_errors;
+               rx_crc_errors   += s->rx_crc_errors;
        }
 
        stats->rx_dropped = rx_dropped;