Revert "net: devlink: add RNLT lock assertion to devlink_compat_switch_id_get()"

This reverts commit 65f05f2f6ddc51671f0e1609cdd98542641ff468.

The assertion was intentionally removed in commit 0d580336de4f ("net:
devlink: remove redundant rtnl lock assert") and, contrary what is
described in the commit message, the comment reflects that: "Caller must
hold RTNL mutex or reference to dev...".

Signed-off-by: Vlad Buslov <vladbu@nvidia.com>
Tested-by: Leon Romanovsky <leonro@nvidia.com>
Link: https://lore.kernel.org/r/20220829121324.3980376-1-vladbu@nvidia.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>

diff --git a/net/core/devlink.c b/net/core/devlink.c
index 97a81b0a8ab4c036a9ed9371001c919655e590c8..7776dc82f88d6d7202d5028152eef77a3e1170cf 100644 (file)
--- a/net/core/devlink.c
+++ b/net/core/devlink.c
@@ -12437,8 +12437,6 @@ 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;