net: devlink: add RNLT lock assertion to devlink_compat_switch_id_get()
authorJiri Pirko <jiri@nvidia.com>
Thu, 25 Aug 2022 11:29:23 +0000 (13:29 +0200)
committerJakub Kicinski <kuba@kernel.org>
Sat, 27 Aug 2022 00:14:15 +0000 (17:14 -0700)
Similar to devlink_compat_phys_port_name_get(), make sure that
devlink_compat_switch_id_get() is called with RTNL lock held. Comment
already says so, so put this in code as well.

Signed-off-by: Jiri Pirko <jiri@nvidia.com>
Link: https://lore.kernel.org/r/20220825112923.1359194-1-jiri@resnulli.us
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
net/core/devlink.c

index 0f7078d..2afbeb6 100644 (file)
@@ -12505,6 +12505,8 @@ int devlink_compat_switch_id_get(struct net_device *dev,
         * devlink_port instance cannot disappear in the middle. No need to take
         * any devlink lock as only permanent values are accessed.
         */
+       ASSERT_RTNL();
+
        devlink_port = netdev_to_devlink_port(dev);
        if (!devlink_port || !devlink_port->switch_port)
                return -EOPNOTSUPP;