mbox series

[net-next,0/5] mlxsw: Show per-band ECN-marked counter on qdisc

Message ID 20211013103748.492531-1-idosch@idosch.org (mailing list archive)
Headers show
Series mlxsw: Show per-band ECN-marked counter on qdisc | expand

Message

Ido Schimmel Oct. 13, 2021, 10:37 a.m. UTC
From: Ido Schimmel <idosch@nvidia.com>

Petr says:

The RED qdisc can expose number of packets that it has marked through
the prob_marked counter (shown in iproute2 as "marked"). This counter
currently just shows number of packets marked in the SW datapath, which
in a switch deployment likely means zero.

Spectrum-3 does support per-TC counters, and in this patchset, mlxsw
supports this RED statistic properly.

Patches #1 and #2 fix typos.

Patch #3 adds a field ecn_marked_tc to the PPCNT register.

Patch #4 adds the support to publish the value of ecn_marked_tc through
the prob_marked RED qdisc counter.

Patch #5 adds selftests.

Petr Machata (5):
  mlxsw: reg: Fix a typo in a group heading
  mlxsw: reg: Rename MLXSW_REG_PPCNT_TC_CONG_TC to _CNT
  mlxsw: reg: Add ecn_marked_tc to Per-TC Congestion Counters
  mlxsw: spectrum_qdisc: Introduce per-TC ECN counters
  selftests: mlxsw: RED: Test per-TC ECN counters

 drivers/net/ethernet/mellanox/mlxsw/reg.h     | 10 +++-
 .../net/ethernet/mellanox/mlxsw/spectrum.c    | 12 +++--
 .../net/ethernet/mellanox/mlxsw/spectrum.h    |  1 +
 .../ethernet/mellanox/mlxsw/spectrum_qdisc.c  |  9 +++-
 .../drivers/net/mlxsw/sch_red_core.sh         | 51 +++++++++++++++----
 .../drivers/net/mlxsw/sch_red_ets.sh          | 11 ++++
 .../drivers/net/mlxsw/sch_red_root.sh         |  8 +++
 7 files changed, 84 insertions(+), 18 deletions(-)

Comments

patchwork-bot+netdevbpf@kernel.org Oct. 14, 2021, 1 a.m. UTC | #1
Hello:

This series was applied to netdev/net-next.git (master)
by Jakub Kicinski <kuba@kernel.org>:

On Wed, 13 Oct 2021 13:37:43 +0300 you wrote:
> From: Ido Schimmel <idosch@nvidia.com>
> 
> Petr says:
> 
> The RED qdisc can expose number of packets that it has marked through
> the prob_marked counter (shown in iproute2 as "marked"). This counter
> currently just shows number of packets marked in the SW datapath, which
> in a switch deployment likely means zero.
> 
> [...]

Here is the summary with links:
  - [net-next,1/5] mlxsw: reg: Fix a typo in a group heading
    https://git.kernel.org/netdev/net-next/c/b063e0651ced
  - [net-next,2/5] mlxsw: reg: Rename MLXSW_REG_PPCNT_TC_CONG_TC to _CNT
    https://git.kernel.org/netdev/net-next/c/fc372cc07286
  - [net-next,3/5] mlxsw: reg: Add ecn_marked_tc to Per-TC Congestion Counters
    https://git.kernel.org/netdev/net-next/c/6242b0a96302
  - [net-next,4/5] mlxsw: spectrum_qdisc: Introduce per-TC ECN counters
    https://git.kernel.org/netdev/net-next/c/15be36b8126b
  - [net-next,5/5] selftests: mlxsw: RED: Test per-TC ECN counters
    https://git.kernel.org/netdev/net-next/c/bf862732945c

You are awesome, thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html