summaryrefslogtreecommitdiff
path: root/scripts/basic
diff options
context:
space:
mode:
authorFlorian Fainelli <f.fainelli@gmail.com>2026-10-26 09:47:33 -0700
committerJakub Kicinski <kuba@kernel.org>2026-10-27 16:05:33 -0700
commitdf66431f39426b685f6693f0731cd5881c51679c (patch)
tree87d6a6c0c3511e3f786d7b10bbb47770830f83a0 /scripts/basic
parent9663edfa71e8807890203578dab6db60c6b548c4 (diff)
net: dsa: mv88e6xxx: fix 88E6171 statistics type
The 88E6171 uses the 6095 statistics operations, which advertise and name BANK0 and PORT counters. Its stats_type bitmap instead selects BANK0 and BANK1. Since statistics collection started using that bitmap, it has written a different set of counters from the one advertised to ethtool. For this chip, get_sset_count() accounts for 30 BANK0 counters, three PORT counters and five ATU/VTU counters. Collection instead writes 30 BANK0 counters, 26 BANK1 counters and five ATU/VTU counters: 61 entries rather than the advertised 38. This mislabels the tail of the returned statistics and writes past the advertised statistics buffer. Select BANK0 and PORT counters to match the chip's statistics operations and restore the filter used before the bitmap was introduced. Tested on an EA4500, where the two-bridge forwarding and multicast snooping tests now pass their hardware counter checks. Fixes: 5595e3613ea7 ("dsa: mv88e6xxx: Move available stats into info structure") Reviewed-by: Andrew Lunn <andrew@lunn.ch> Signed-off-by: Florian Fainelli <f.fainelli@gmail.com> Link: https://patch.msgid.link/20261026164733.737299-3-f.fainelli@gmail.com Signed-off-by: Jakub Kicinski <kuba@kernel.org>
Diffstat (limited to 'scripts/basic')
0 files changed, 0 insertions, 0 deletions