mbox series

[net-next,00/10] mlxsw: Add support for egress and policy-based sampling

Message ID 20210316150303.2868588-1-idosch@idosch.org (mailing list archive)
Headers show
Series mlxsw: Add support for egress and policy-based sampling | expand

Message

Ido Schimmel March 16, 2021, 3:02 p.m. UTC
From: Ido Schimmel <idosch@nvidia.com>

So far mlxsw only supported ingress sampling using matchall classifier.
This series adds support for egress sampling and policy-based sampling
using flower classifier on Spectrum-2 and newer ASICs. As such, it is
now possible to issue these commands:

 # tc filter add dev swp1 egress pref 1 proto all matchall action sample rate 100 group 1

 # tc filter add dev swp2 ingress pref 1 proto ip flower dst_ip 198.51.100.1 action sample rate 100 group 2

When performing egress sampling (using either matchall or flower) the
ASIC is able to report the end-to-end latency which is passed to the
psample module.

Series overview:

Patches #1-#3 are preparations without any functional changes

Patch #4 generalizes the idea of sampling triggers and creates a hash
table to track active sampling triggers in preparation for egress and
policy-based triggers. The motivation is explained in the changelog

Patch #5 flips mlxsw to start using this hash table instead of storing
ingress sampling triggers as an attribute of the sampled port

Patch #6 finally adds support for egress sampling using matchall
classifier

Patches #7-#8 add support for policy-based sampling using flower
classifier

Patches #9 extends the mlxsw sampling selftest to cover the new triggers

Patch #10 makes sure that egress sampling configuration only fails on
Spectrum-1

Ido Schimmel (10):
  mlxsw: spectrum_matchall: Propagate extack further
  mlxsw: spectrum_matchall: Push sampling checks to per-ASIC operations
  mlxsw: spectrum_matchall: Pass matchall entry to sampling operations
  mlxsw: spectrum: Track sampling triggers in a hash table
  mlxsw: spectrum: Start using sampling triggers hash table
  mlxsw: spectrum_matchall: Add support for egress sampling
  mlxsw: core_acl_flex_actions: Add mirror sampler action
  mlxsw: spectrum_acl: Offload FLOW_ACTION_SAMPLE
  selftests: mlxsw: Add tc sample tests for new triggers
  selftests: mlxsw: Test egress sampling limitation on Spectrum-1 only

 .../mellanox/mlxsw/core_acl_flex_actions.c    | 131 ++++++++++++++
 .../mellanox/mlxsw/core_acl_flex_actions.h    |  11 ++
 .../net/ethernet/mellanox/mlxsw/spectrum.c    | 148 ++++++++++++++++
 .../net/ethernet/mellanox/mlxsw/spectrum.h    |  52 +++++-
 .../ethernet/mellanox/mlxsw/spectrum_acl.c    |  25 +++
 .../mlxsw/spectrum_acl_flex_actions.c         |  83 +++++++++
 .../ethernet/mellanox/mlxsw/spectrum_flow.c   |   2 +-
 .../ethernet/mellanox/mlxsw/spectrum_flower.c |  18 ++
 .../mellanox/mlxsw/spectrum_matchall.c        | 167 +++++++++++-------
 .../ethernet/mellanox/mlxsw/spectrum_trap.c   | 111 +++++++++++-
 .../drivers/net/mlxsw/tc_restrictions.sh      |   4 +-
 .../selftests/drivers/net/mlxsw/tc_sample.sh  | 135 ++++++++++++++
 12 files changed, 808 insertions(+), 79 deletions(-)

Comments

Jakub Kicinski March 16, 2021, 10:21 p.m. UTC | #1
On Tue, 16 Mar 2021 17:02:53 +0200 Ido Schimmel wrote:
> From: Ido Schimmel <idosch@nvidia.com>
> 
> So far mlxsw only supported ingress sampling using matchall classifier.
> This series adds support for egress sampling and policy-based sampling
> using flower classifier on Spectrum-2 and newer ASICs. As such, it is
> now possible to issue these commands:
> 
>  # tc filter add dev swp1 egress pref 1 proto all matchall action sample rate 100 group 1
> 
>  # tc filter add dev swp2 ingress pref 1 proto ip flower dst_ip 198.51.100.1 action sample rate 100 group 2
> 
> When performing egress sampling (using either matchall or flower) the
> ASIC is able to report the end-to-end latency which is passed to the
> psample module.

Acked-by: Jakub Kicinski <kuba@kernel.org>
patchwork-bot+netdevbpf@kernel.org March 16, 2021, 10:40 p.m. UTC | #2
Hello:

This series was applied to netdev/net-next.git (refs/heads/master):

On Tue, 16 Mar 2021 17:02:53 +0200 you wrote:
> From: Ido Schimmel <idosch@nvidia.com>
> 
> So far mlxsw only supported ingress sampling using matchall classifier.
> This series adds support for egress sampling and policy-based sampling
> using flower classifier on Spectrum-2 and newer ASICs. As such, it is
> now possible to issue these commands:
> 
> [...]

Here is the summary with links:
  - [net-next,01/10] mlxsw: spectrum_matchall: Propagate extack further
    https://git.kernel.org/netdev/net-next/c/6561df560833
  - [net-next,02/10] mlxsw: spectrum_matchall: Push sampling checks to per-ASIC operations
    https://git.kernel.org/netdev/net-next/c/559313b2cbb7
  - [net-next,03/10] mlxsw: spectrum_matchall: Pass matchall entry to sampling operations
    https://git.kernel.org/netdev/net-next/c/e09a59555a30
  - [net-next,04/10] mlxsw: spectrum: Track sampling triggers in a hash table
    https://git.kernel.org/netdev/net-next/c/1b9fc42e46df
  - [net-next,05/10] mlxsw: spectrum: Start using sampling triggers hash table
    https://git.kernel.org/netdev/net-next/c/90f53c53ec4a
  - [net-next,06/10] mlxsw: spectrum_matchall: Add support for egress sampling
    https://git.kernel.org/netdev/net-next/c/54d0e963f683
  - [net-next,07/10] mlxsw: core_acl_flex_actions: Add mirror sampler action
    https://git.kernel.org/netdev/net-next/c/ca19ea63f739
  - [net-next,08/10] mlxsw: spectrum_acl: Offload FLOW_ACTION_SAMPLE
    https://git.kernel.org/netdev/net-next/c/45aad0b7043d
  - [net-next,09/10] selftests: mlxsw: Add tc sample tests for new triggers
    https://git.kernel.org/netdev/net-next/c/f0b692c4ee2f
  - [net-next,10/10] selftests: mlxsw: Test egress sampling limitation on Spectrum-1 only
    https://git.kernel.org/netdev/net-next/c/0f967d9e5a20

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