Skip to content

mlxsw: Misc devlink changes

Petr Machata netdev 1 version Applied

Status from this message: This series was applied to netdev/net-next.git (master)

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest6 patches 2 repliesApplied 6/6

Patches in v1 All 6 applied

#Subject and filesSizeRepliesApplied
1/6mlxsw: spectrum: Remove pointless call to devlink_param_driverinit_value_set()drivers/net/ethernet/mellanox/mlxsw/spectrum.c+2 -121 replyApplied
2/6mlxsw: spectrum_acl_tcam: Add missing mutex_destroy()drivers/net/ethernet/mellanox/mlxsw/spectrum_acl_tcam.c+6 -20 repliesApplied
3/6mlxsw: spectrum_acl_tcam: Make fini symmetric to initdrivers/net/ethernet/mellanox/mlxsw/spectrum_acl_tcam.c+1 -10 repliesApplied
4/6mlxsw: spectrum_acl_tcam: Reorder functions to avoid forward declarationsdrivers/net/ethernet/mellanox/mlxsw/spectrum_acl_tcam.c+65 -650 repliesApplied
5/6mlxsw: spectrum_acl_tcam: Move devlink param to TCAM codedrivers/net/ethernet/mellanox/mlxsw/core.c drivers/net/ethernet/mellanox/mlxsw/core.h drivers/net/ethernet/mellanox/mlxsw/spectrum.c +4 more+90 -1300 repliesApplied
6/6mlxsw: core: Register devlink instance before sub-objectsdrivers/net/ethernet/mellanox/mlxsw/core.c+6 -60 repliesApplied

Reviews trailers per patch

All 6 patches have a Reviewed-by or Acked-by of their own; the series has one for all of them.

PatchIdo SchimmelJiri PirkoJacob Keller
SeriesR-b
1/6R-b R-b
2/6R-b
3/6R-b
4/6R-b
5/6R-b
6/6R-b

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes. Scroll the table sideways for all 3 reviewers.

Read the v1 thread

Cover letter [PATCH net-next 0/6] mlxsw: Misc devlink changes

This patchset adjusts mlxsw to recent devlink changes in net-next.

Patch #1 removes a devl_param_driverinit_value_set() call that was unnecessary, but now additionally triggers a WARN_ON.

Patches #2-#4 are non-functional preparations for the following patches.

Patch #5 fixes a use-after-free that is triggered while changing network namespaces.

Patch #6 makes mlxsw consistent with netdevsim by having mlxsw register its devlink instance before its sub-objects. It helps us avoid a warning described in the commit message.

Danielle Ratson (1):
mlxsw: spectrum: Remove pointless call to
devlink_param_driverinit_value_set()

Ido Schimmel (5):
mlxsw: spectrum_acl_tcam: Add missing mutex_destroy()
mlxsw: spectrum_acl_tcam: Make fini symmetric to init
mlxsw: spectrum_acl_tcam: Reorder functions to avoid forward
declarations
mlxsw: spectrum_acl_tcam: Move devlink param to TCAM code
mlxsw: core: Register devlink instance before sub-objects

 drivers/net/ethernet/mellanox/mlxsw/core.c    |  31 +--
 drivers/net/ethernet/mellanox/mlxsw/core.h    |   2 -
 .../net/ethernet/mellanox/mlxsw/spectrum.c    |  62 -----
 .../net/ethernet/mellanox/mlxsw/spectrum.h    |   3 +-
 .../ethernet/mellanox/mlxsw/spectrum_acl.c    |  21 +-
 .../mellanox/mlxsw/spectrum_acl_tcam.c        | 244 +++++++++++-------
 .../mellanox/mlxsw/spectrum_acl_tcam.h        |   5 -
 7 files changed, 161 insertions(+), 207 deletions(-)
Signature · 1 line
2.39.0