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 09:37:46 +0100 |
| commit | fb8421a94c5613fee86e192bab0892ecb1d56e4c (patch) | |
| tree | b28cedef2ff2c2836ebd79b7b2a67f92f854bb6e /net/devlink/leftover.c | |
| parent | devlink: send objects notifications during devlink reload (diff) | |
| download | linux-fb8421a94c5613fee86e192bab0892ecb1d56e4c.tar.xz linux-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 '')
| -rw-r--r-- | net/devlink/leftover.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/net/devlink/leftover.c b/net/devlink/leftover.c index 4f78ef5a46af..92210587d349 100644 --- a/net/devlink/leftover.c +++ b/net/devlink/leftover.c @@ -4387,9 +4387,6 @@ static int devlink_nl_cmd_reload(struct sk_buff *skb, struct genl_info *info) u32 actions_performed; int err; - if (!(devlink->features & DEVLINK_F_RELOAD)) - return -EOPNOTSUPP; - err = devlink_resources_validate(devlink, NULL, info); if (err) { NL_SET_ERR_MSG_MOD(info->extack, "resources size validation failed"); |