Allow parallel devlink execution
Get series with b4
b4 am [email protected]Versions newest first
- v1 Latest6 patches 6 repliesIn review
Patches in v1 No applies recorded yet
| # | Subject and files | Size | Replies | Applied |
|---|---|---|---|---|
| 1/6 | devlink: Clean registration of devlink portnet/ | +8 | 0 replies | |
| 2/6 | devlink: Be explicit with devlink port protectionnet/ | +22 | 0 replies | |
| 3/6 | devlink: Add devlink nested locking primitivenet/ | +88 | 0 replies | |
| 4/6 | devlink: Require devlink lock during device reloadnet/ | +5 | 0 replies | |
| 5/6 | devlink: Use xarray locking mechanism instead big devlink locknet/ | +119 | 0 replies | |
| 6/6 | devlink: Open devlink to parallel operationsnet/ | +1 | 0 replies |
Reviews trailers per patch
None of the 6 patches has a Reviewed-by or Acked-by.
| Patch | Ido Schimmel |
|---|---|
| 1/6 | T-b |
| 2/6 | T-b |
| 3/6 | T-b |
| 4/6 | T-b |
| 5/6 | T-b |
| 6/6 | T-b |
Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.
Cover letter [PATCH net-next 0/6] Allow parallel devlink execution
From: Leon Romanovsky <[email protected]>
Hi,
This is final piece of devlink locking puzzle, where I remove global mutex lock (devlink_mutex), so we can run devlink commands in parallel.
The series starts with addition of port_list_lock, which is needed to prevent locking dependency between netdevsim sysfs and devlink. It follows by the patch that adds context aware locking primitives. Such primitives allow us to make sure that devlink instance is locked and stays locked even during reload operation. The last patches opens devlink to parallel commands.
It tested on mlx5, mlx4, mlxsw, netdevsim + Ido's syzkaller.
BTW, in the future patches, we will clean devlink from over-engineered APIs and implementations. It will open a venue to revise all the places that hold devlink lock, if they really need to do it.
Thanks
[1] https://lore.kernel.org/netdev/[email protected]/
[2] https://lore.kernel.org/netdev/9716f9a13e217a0a163b745b6e92e02d40973d2c.1635701665.git.leonro@nvidia.com/
Leon Romanovsky (6):
devlink: Clean registration of devlink port
devlink: Be explicit with devlink port protection
devlink: Add devlink nested locking primitive
devlink: Require devlink lock during device reload
devlink: Use xarray locking mechanism instead big devlink lock
devlink: Open devlink to parallel operations
net/core/devlink.c | 432 ++++++++++++++++++++++++--------------------- 1 file changed, 232 insertions(+), 200 deletions(-)
Signature · 1 line
2.33.1