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/broadcom | |
| parent | a131315a47bbb5e89a3330cac69fe6ab4836f762 (diff) | |
| download | lwn-fb8421a94c5613fee86e192bab0892ecb1d56e4c.tar.gz lwn-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/broadcom')
| -rw-r--r-- | drivers/net/ethernet/broadcom/bnxt/bnxt_devlink.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/drivers/net/ethernet/broadcom/bnxt/bnxt_devlink.c b/drivers/net/ethernet/broadcom/bnxt/bnxt_devlink.c index 26913dc816d3..8b3e7697390f 100644 --- a/drivers/net/ethernet/broadcom/bnxt/bnxt_devlink.c +++ b/drivers/net/ethernet/broadcom/bnxt/bnxt_devlink.c @@ -1303,7 +1303,6 @@ int bnxt_dl_register(struct bnxt *bp) if (rc) goto err_dl_port_unreg; - devlink_set_features(dl, DEVLINK_F_RELOAD); out: devlink_register(dl); return 0; |