mbox series

[net-next,0/6] mlxsw: Two sampling fixes

Message ID 20210329100948.355486-1-idosch@idosch.org (mailing list archive)
Headers show
Series mlxsw: Two sampling fixes | expand

Message

Ido Schimmel March 29, 2021, 10:09 a.m. UTC
From: Ido Schimmel <idosch@nvidia.com>

This patchset fixes two bugs in recent sampling submissions.

The first fix, in patch #3, prevents matchall rules with sample action
to be added in front of flower rules on egress. Patches #1-#2 are
preparations meant at avoiding similar bugs in the future. Patch #4 is a
selftest.

The second fix, in patch #5, prevents sampling from being enabled on a
port if already enabled. Patch #6 is a selftest.

Ido Schimmel (6):
  mlxsw: spectrum_matchall: Perform protocol check earlier
  mlxsw: spectrum_matchall: Convert if statements to a switch statement
  mlxsw: spectrum_matchall: Perform priority checks earlier
  selftests: mlxsw: Test matchall failure with protocol match
  mlxsw: spectrum: Veto sampling if already enabled on port
  selftests: mlxsw: Test vetoing of double sampling

 .../net/ethernet/mellanox/mlxsw/spectrum.c    |  5 ++
 .../mellanox/mlxsw/spectrum_matchall.c        | 46 ++++++++++---------
 .../drivers/net/mlxsw/tc_restrictions.sh      | 17 +++++++
 .../selftests/drivers/net/mlxsw/tc_sample.sh  | 30 ++++++++++++
 4 files changed, 76 insertions(+), 22 deletions(-)

Comments

patchwork-bot+netdevbpf@kernel.org March 29, 2021, 8:50 p.m. UTC | #1
Hello:

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

On Mon, 29 Mar 2021 13:09:42 +0300 you wrote:
> From: Ido Schimmel <idosch@nvidia.com>
> 
> This patchset fixes two bugs in recent sampling submissions.
> 
> The first fix, in patch #3, prevents matchall rules with sample action
> to be added in front of flower rules on egress. Patches #1-#2 are
> preparations meant at avoiding similar bugs in the future. Patch #4 is a
> selftest.
> 
> [...]

Here is the summary with links:
  - [net-next,1/6] mlxsw: spectrum_matchall: Perform protocol check earlier
    https://git.kernel.org/netdev/net-next/c/4947e7309a31
  - [net-next,2/6] mlxsw: spectrum_matchall: Convert if statements to a switch statement
    https://git.kernel.org/netdev/net-next/c/50401f292434
  - [net-next,3/6] mlxsw: spectrum_matchall: Perform priority checks earlier
    https://git.kernel.org/netdev/net-next/c/b24303048a6b
  - [net-next,4/6] selftests: mlxsw: Test matchall failure with protocol match
    https://git.kernel.org/netdev/net-next/c/c3572a0b731f
  - [net-next,5/6] mlxsw: spectrum: Veto sampling if already enabled on port
    https://git.kernel.org/netdev/net-next/c/17b96a5cbe3d
  - [net-next,6/6] selftests: mlxsw: Test vetoing of double sampling
    https://git.kernel.org/netdev/net-next/c/7ede22e65832

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