mbox series

[net-next,0/4] net: devlink: allow parallel commands on multiple devlinks

Message ID 20220729071038.983101-1-jiri@resnulli.us (mailing list archive)
Headers show
Series net: devlink: allow parallel commands on multiple devlinks | expand

Message

Jiri Pirko July 29, 2022, 7:10 a.m. UTC
From: Jiri Pirko <jiri@nvidia.com>

Aim of this patchset is to remove devlink_mutex and eventually to enable
parallel ops on devlink netlink interface.

Jiri Pirko (4):
  net: devlink: introduce "unregistering" mark and use it during
    devlinks iteration
  net: devlink: convert reload command to take implicit devlink->lock
  net: devlink: remove devlink_mutex
  net: devlink: enable parallel ops on netlink interface

 drivers/net/ethernet/mellanox/mlx4/main.c     |   4 -
 .../net/ethernet/mellanox/mlx5/core/devlink.c |   4 -
 drivers/net/ethernet/mellanox/mlxsw/core.c    |   4 -
 drivers/net/netdevsim/dev.c                   |   6 -
 net/core/devlink.c                            | 110 ++++--------------
 5 files changed, 21 insertions(+), 107 deletions(-)

Comments

Jakub Kicinski July 30, 2022, 3:04 a.m. UTC | #1
On Fri, 29 Jul 2022 09:10:34 +0200 Jiri Pirko wrote:
> From: Jiri Pirko <jiri@nvidia.com>
> 
> Aim of this patchset is to remove devlink_mutex and eventually to enable
> parallel ops on devlink netlink interface.

Let's do this! Worse comes to worst it's a relatively small revert.

Reviewed-by: Jakub Kicinski <kuba@kernel.org>

Thanks to everyone involved!
Jiri Pirko July 30, 2022, 9:26 a.m. UTC | #2
Sat, Jul 30, 2022 at 05:04:34AM CEST, kuba@kernel.org wrote:
>On Fri, 29 Jul 2022 09:10:34 +0200 Jiri Pirko wrote:
>> From: Jiri Pirko <jiri@nvidia.com>
>> 
>> Aim of this patchset is to remove devlink_mutex and eventually to enable
>> parallel ops on devlink netlink interface.
>
>Let's do this! Worse comes to worst it's a relatively small revert.
>
>Reviewed-by: Jakub Kicinski <kuba@kernel.org>
>
>Thanks to everyone involved!

Thanks!
Ido Schimmel July 30, 2022, 3:53 p.m. UTC | #3
On Fri, Jul 29, 2022 at 09:10:34AM +0200, Jiri Pirko wrote:
> From: Jiri Pirko <jiri@nvidia.com>
> 
> Aim of this patchset is to remove devlink_mutex and eventually to enable
> parallel ops on devlink netlink interface.

Will report regression results tomorrow morning.
Ido Schimmel July 31, 2022, 6:02 a.m. UTC | #4
On Fri, Jul 29, 2022 at 09:10:34AM +0200, Jiri Pirko wrote:
> From: Jiri Pirko <jiri@nvidia.com>
> 
> Aim of this patchset is to remove devlink_mutex and eventually to enable
> parallel ops on devlink netlink interface.

Tested-by: Ido Schimmel <idosch@nvidia.com>
patchwork-bot+netdevbpf@kernel.org Aug. 1, 2022, 11:40 a.m. UTC | #5
Hello:

This series was applied to netdev/net-next.git (master)
by David S. Miller <davem@davemloft.net>:

On Fri, 29 Jul 2022 09:10:34 +0200 you wrote:
> From: Jiri Pirko <jiri@nvidia.com>
> 
> Aim of this patchset is to remove devlink_mutex and eventually to enable
> parallel ops on devlink netlink interface.
> 
> Jiri Pirko (4):
>   net: devlink: introduce "unregistering" mark and use it during
>     devlinks iteration
>   net: devlink: convert reload command to take implicit devlink->lock
>   net: devlink: remove devlink_mutex
>   net: devlink: enable parallel ops on netlink interface
> 
> [...]

Here is the summary with links:
  - [net-next,1/4] net: devlink: introduce "unregistering" mark and use it during devlinks iteration
    https://git.kernel.org/netdev/net-next/c/c2368b19807a
  - [net-next,2/4] net: devlink: convert reload command to take implicit devlink->lock
    https://git.kernel.org/netdev/net-next/c/644a66c60f02
  - [net-next,3/4] net: devlink: remove devlink_mutex
    https://git.kernel.org/netdev/net-next/c/d3efc2a6a6d8
  - [net-next,4/4] net: devlink: enable parallel ops on netlink interface
    https://git.kernel.org/netdev/net-next/c/09b278462f16

You are awesome, thank you!