mirrors/linux
2
0
Fork 0
mirror of https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git synced 2026-07-17 07:57:43 -04:00
Code Issues Packages Projects Releases Wiki Activity

net: ethtool: make sure __ethtool_get_link_ksettings() is ops-locked

Browse Source 
All drivers which may call *_get_link_ksettings() on ops-locked
devices from paths already holding the ops lock are ready now.
Make __ethtool_get_link_ksettings() take the ops lock, and assert
that it's held in netif_get_link_ksettings().

Reviewed-by: Nicolai Buchwitz <nb@tipi-net.de>
Acked-by: Stanislav Fomichev <sdf@fomichev.me>
Link: https://patch.msgid.link/20260603012840.2254293-12-kuba@kernel.org
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
This commit is contained in:
Jakub Kicinski
2026-06-02 18:28:40 -07:00
parent af1737dab0
commit 9f275c2e90
1 changed files with 7 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
net/ethtool/ioctl.c
View File

@@ -439,7 +439,7 @@ struct ethtool_link_usettings {
int netif_get_link_ksettings(struct net_device *dev,
struct ethtool_link_ksettings *link_ksettings)
{
/* once callers fixed - assert ops locked */
netdev_assert_locked_ops_compat(dev);
if (!dev->ethtool_ops->get_link_ksettings)
return -EOPNOTSUPP;
@@ -456,10 +456,14 @@ EXPORT_SYMBOL(netif_get_link_ksettings);
int __ethtool_get_link_ksettings(struct net_device *dev,
struct ethtool_link_ksettings *link_ksettings)
{
int ret;
ASSERT_RTNL();
/* once callers fixed - take the ops lock around this call */
return netif_get_link_ksettings(dev, link_ksettings);
netdev_lock_ops(dev);
ret = netif_get_link_ksettings(dev, link_ksettings);
netdev_unlock_ops(dev);
return ret;
}
EXPORT_SYMBOL(__ethtool_get_link_ksettings);