diff options
author | Jiri Pirko <jiri@nvidia.com> | 2023-01-27 16:50:42 +0100 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2023-01-30 08:37:46 +0000 |
commit | fb8421a94c5613fee86e192bab0892ecb1d56e4c (patch) | |
tree | b28cedef2ff2c2836ebd79b7b2a67f92f854bb6e /drivers/net/ethernet/mellanox/mlx4 | |
parent | a131315a47bbb5e89a3330cac69fe6ab4836f762 (diff) | |
download | linux-stable-fb8421a94c5613fee86e192bab0892ecb1d56e4c.tar.gz linux-stable-fb8421a94c5613fee86e192bab0892ecb1d56e4c.tar.bz2 linux-stable-fb8421a94c5613fee86e192bab0892ecb1d56e4c.zip |
devlink: remove devlink features
Devlink features were introduced to disallow devlink reload calls of
userspace before the devlink was fully initialized. The reason for this
workaround was the fact that devlink reload was originally called
without devlink instance lock held.
However, with recent changes that converted devlink reload to be
performed under devlink instance lock, this is redundant so remove
devlink features entirely.
Note that mlx5 used this to enable devlink reload conditionally only
when device didn't act as multi port slave. Move the multi port check
into mlx5_devlink_reload_down() callback alongside with the other
checks preventing the device from reload in certain states.
Signed-off-by: Jiri Pirko <jiri@nvidia.com>
Reviewed-by: Jacob Keller <jacob.e.keller@intel.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/ethernet/mellanox/mlx4')
-rw-r--r-- | drivers/net/ethernet/mellanox/mlx4/main.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/drivers/net/ethernet/mellanox/mlx4/main.c b/drivers/net/ethernet/mellanox/mlx4/main.c index 6152f77dcfd8..277738c50c56 100644 --- a/drivers/net/ethernet/mellanox/mlx4/main.c +++ b/drivers/net/ethernet/mellanox/mlx4/main.c @@ -4031,7 +4031,6 @@ static int mlx4_init_one(struct pci_dev *pdev, const struct pci_device_id *id) goto err_params_unregister; pci_save_state(pdev); - devlink_set_features(devlink, DEVLINK_F_RELOAD); devl_unlock(devlink); devlink_register(devlink); return 0; |