* [PATCH net-next] net: dlink: fix multicast stats being counted incorrectly
@ 2025-08-21 11:42 Yeounsu Moon
2025-08-22 0:23 ` Andrew Lunn
0 siblings, 1 reply; 3+ messages in thread
From: Yeounsu Moon @ 2025-08-21 11:42 UTC (permalink / raw)
To: Andrew Lunn, David S. Miller, Eric Dumazet, Jakub Kicinski,
Paolo Abeni
Cc: Yeounsu Moon, netdev, linux-kernel
`McstFramesRcvdOk` counts the number of received multicast packets, and
it reports the value correctly.
However, reading `McstFramesRcvdOk` clears the register to zero. As a
result, the driver was reporting only the packets since the last read,
instead of the accumulated total.
Fix this by updating the multicast statistics accumulatively instaed of
instantaneously.
Tested-on: D-Link DGE-550T Rev-A3
Signed-off-by: Yeounsu Moon <yyyynoom@gmail.com>
---
drivers/net/ethernet/dlink/dl2k.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/net/ethernet/dlink/dl2k.c b/drivers/net/ethernet/dlink/dl2k.c
index cc60ee454bf9..6bbf6e5584e5 100644
--- a/drivers/net/ethernet/dlink/dl2k.c
+++ b/drivers/net/ethernet/dlink/dl2k.c
@@ -1099,7 +1099,7 @@ get_stats (struct net_device *dev)
dev->stats.rx_bytes += dr32(OctetRcvOk);
dev->stats.tx_bytes += dr32(OctetXmtOk);
- dev->stats.multicast = dr32(McstFramesRcvdOk);
+ dev->stats.multicast += dr32(McstFramesRcvdOk);
dev->stats.collisions += dr32(SingleColFrames)
+ dr32(MultiColFrames);
--
2.50.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH net-next] net: dlink: fix multicast stats being counted incorrectly
2025-08-21 11:42 [PATCH net-next] net: dlink: fix multicast stats being counted incorrectly Yeounsu Moon
@ 2025-08-22 0:23 ` Andrew Lunn
2025-08-22 3:45 ` Yeounsu Moon
0 siblings, 1 reply; 3+ messages in thread
From: Andrew Lunn @ 2025-08-22 0:23 UTC (permalink / raw)
To: Yeounsu Moon
Cc: Andrew Lunn, David S. Miller, Eric Dumazet, Jakub Kicinski,
Paolo Abeni, netdev, linux-kernel
On Thu, Aug 21, 2025 at 08:42:53PM +0900, Yeounsu Moon wrote:
> `McstFramesRcvdOk` counts the number of received multicast packets, and
> it reports the value correctly.
>
> However, reading `McstFramesRcvdOk` clears the register to zero. As a
> result, the driver was reporting only the packets since the last read,
> instead of the accumulated total.
>
> Fix this by updating the multicast statistics accumulatively instaed of
> instantaneously.
This looks like a fix. Please could you add a Fixes: tag and submit to
net.
https://www.kernel.org/doc/html/latest/process/maintainer-netdev.html
Andrew
---
pw-bot: cr
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH net-next] net: dlink: fix multicast stats being counted incorrectly
2025-08-22 0:23 ` Andrew Lunn
@ 2025-08-22 3:45 ` Yeounsu Moon
0 siblings, 0 replies; 3+ messages in thread
From: Yeounsu Moon @ 2025-08-22 3:45 UTC (permalink / raw)
To: Andrew Lunn
Cc: Andrew Lunn, David S. Miller, Eric Dumazet, Jakub Kicinski,
Paolo Abeni, netdev, linux-kernel
Thank you for the review!
I sometimes get confused about whether a patch should go to net or net-next.
I will send v2 with the changes you suggested.
Yeounsu Moon
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2025-08-22 3:45 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-08-21 11:42 [PATCH net-next] net: dlink: fix multicast stats being counted incorrectly Yeounsu Moon
2025-08-22 0:23 ` Andrew Lunn
2025-08-22 3:45 ` Yeounsu Moon
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).