[v4,net-next,0/8] net: dsa: mv88e6xxx: Add "eth-mac" and "rmon" counter group support

Message

Tobias Waldekranz Dec. 14, 2023, 1:50 p.m. UTC

The majority of the changes (2/8) are about refactoring the existing
ethtool statistics support to make it possible to read individual
counters, rather than the whole set.

4/8 tries to collect all information about a stat in a single place
using a mapper macro, which is then used to generate the original list
of stats, along with a matching enum. checkpatch is less than amused
with this construct, but prior art exists (__BPF_FUNC_MAPPER in
include/uapi/linux/bpf.h, for example).

To support the histogram counters from the "rmon" group, we have to
change mv88e6xxx's configuration of them. Instead of counting rx and
tx, we restrict them to rx-only. 6/8 has the details.

With that in place, adding the actual counter groups is pretty
straight forward (5,7/8).

Tie it all together with a selftest (8/8).

v3 -> v4:
- Return size_t from mv88e6xxx_stats_get_stats
- Spelling errors in commit message of 6/8
- Improve selftest:
  - Report progress per-bucket
  - Test both ports in the pair
  - Increase MTU, if required

v2 -> v3:
- Added 6/8
- Added 8/8

v1 -> v2:
- Added 1/6
- Added 3/6
- Changed prototype of stats operation to reflect the fact that the
  number of read stats are returned, no errors
- Moved comma into MV88E6XXX_HW_STAT_MAPPER definition
- Avoid the construction of mapping table iteration which relied on
  struct layouts outside of mv88e6xxx's control

Tobias Waldekranz (8):
  net: dsa: mv88e6xxx: Push locking into stats snapshotting
  net: dsa: mv88e6xxx: Create API to read a single stat counter
  net: dsa: mv88e6xxx: Fix mv88e6352_serdes_get_stats error path
  net: dsa: mv88e6xxx: Give each hw stat an ID
  net: dsa: mv88e6xxx: Add "eth-mac" counter group support
  net: dsa: mv88e6xxx: Limit histogram counters to ingress traffic
  net: dsa: mv88e6xxx: Add "rmon" counter group support
  selftests: forwarding: ethtool_rmon: Add histogram counter test

 drivers/net/dsa/mv88e6xxx/chip.c              | 392 ++++++++++++------
 drivers/net/dsa/mv88e6xxx/chip.h              |  31 +-
 drivers/net/dsa/mv88e6xxx/global1.c           |   7 +-
 drivers/net/dsa/mv88e6xxx/serdes.c            |  10 +-
 drivers/net/dsa/mv88e6xxx/serdes.h            |   8 +-
 .../testing/selftests/net/forwarding/Makefile |   1 +
 .../selftests/net/forwarding/ethtool_rmon.sh  | 143 +++++++
 tools/testing/selftests/net/forwarding/lib.sh |   9 +
 8 files changed, 435 insertions(+), 166 deletions(-)
 create mode 100755 tools/testing/selftests/net/forwarding/ethtool_rmon.sh

Comments

[e-mail] Dec. 15, 2023, 11:10 a.m. UTC | #1

Hello:

This series was applied to netdev/net-next.git (main)
by David S. Miller <[e-mail]>:

On Thu, 14 Dec 2023 14:50:21 +0100 you wrote:
> The majority of the changes (2/8) are about refactoring the existing
> ethtool statistics support to make it possible to read individual
> counters, rather than the whole set.
> 
> 4/8 tries to collect all information about a stat in a single place
> using a mapper macro, which is then used to generate the original list
> of stats, along with a matching enum. checkpatch is less than amused
> with this construct, but prior art exists (__BPF_FUNC_MAPPER in
> include/uapi/linux/bpf.h, for example).
> 
> [...]

Here is the summary with links:
  - [v4,net-next,1/8] net: dsa: mv88e6xxx: Push locking into stats snapshotting
    https://git.kernel.org/netdev/net-next/c/d624afaf4c79
  - [v4,net-next,2/8] net: dsa: mv88e6xxx: Create API to read a single stat counter
    https://git.kernel.org/netdev/net-next/c/3def80e52db3
  - [v4,net-next,3/8] net: dsa: mv88e6xxx: Fix mv88e6352_serdes_get_stats error path
    https://git.kernel.org/netdev/net-next/c/fc82a08ae795
  - [v4,net-next,4/8] net: dsa: mv88e6xxx: Give each hw stat an ID
    https://git.kernel.org/netdev/net-next/c/5780acbd2499
  - [v4,net-next,5/8] net: dsa: mv88e6xxx: Add "eth-mac" counter group support
    https://git.kernel.org/netdev/net-next/c/0e047cec7796
  - [v4,net-next,6/8] net: dsa: mv88e6xxx: Limit histogram counters to ingress traffic
    https://git.kernel.org/netdev/net-next/c/ceea48efa358
  - [v4,net-next,7/8] net: dsa: mv88e6xxx: Add "rmon" counter group support
    https://git.kernel.org/netdev/net-next/c/394518e3c119
  - [v4,net-next,8/8] selftests: forwarding: ethtool_rmon: Add histogram counter test
    https://git.kernel.org/netdev/net-next/c/00e7f29d9b89

You are awesome, thank you!