Skip to content

Clean devlink net namespace operations

Leon Romanovsky lkml netdev 2 versions Applied

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

Get series with b4
b4 am [email protected]

Versions newest first · linked by author and title

  1. v2 Latest2 patches 0 repliesApplied 2/2
  2. v12 patches 2 repliesSuperseded

Changes in v2 from the cover letter

* Patch 1: Dropped cmd argument
v1: https://lore.kernel.org/lkml/[email protected]
* Patch 1:
* Renamed function name
* Added bool parameter to the notifier function
* Patch 2:
* added Jiri's ROB and dropped word "RAW" from the comment"
v0: https://lore.kernel.org/lkml/[email protected]

Patches in v2 All 2 applied

#Subject and filesSizeRepliesApplied
1/2devlink: Break parameter notification sequence to be before/after unload/load drivernet/core/devlink.c+20 -120 repliesApplied
2/2devlink: Allocate devlink directly in requested net namespacedrivers/net/netdevsim/dev.c include/net/devlink.h net/core/devlink.c+22 -220 repliesApplied

Reviews trailers per patch

All 2 patches have a Reviewed-by or Acked-by.

PatchParav PanditJiri Pirko
1/2R-b
2/2R-b

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v2 thread

Cover letter [PATCH net-next v2 0/2] Clean devlink net namespace operations

From: Leon Romanovsky <[email protected]>

Changelog:
v2:
* Patch 1: Dropped cmd argument
v1: https://lore.kernel.org/lkml/[email protected]
* Patch 1:
* Renamed function name
* Added bool parameter to the notifier function
* Patch 2:
* added Jiri's ROB and dropped word "RAW" from the comment"
v0: https://lore.kernel.org/lkml/[email protected]

-----------------------------------------------------------------------------
Hi Dave, Jakub and Jiri

This short series continues my work on devlink core code to make devlink reload less prone to errors and harden it from API abuse.

Despite first patch being a clear fix, I would ask you to apply it to net-next anyway, because the fixed patch is anyway old and it will help us to eliminate merge conflicts that will arise for following patches or even for the second one.

Thanks

Leon Romanovsky (2):
devlink: Break parameter notification sequence to be before/after
unload/load driver
devlink: Allocate devlink directly in requested net namespace

 drivers/net/netdevsim/dev.c |  4 +--
 include/net/devlink.h       | 14 ++++++++--
 net/core/devlink.c          | 56 ++++++++++++++++++-------------------
 3 files changed, 41 insertions(+), 33 deletions(-)
Signature · 1 line
2.31.1