diff options
| author | Maxime Chevallier <[email protected]> | 2024-09-13 12:05:14 +0200 |
|---|---|---|
| committer | Jakub Kicinski <[email protected]> | 2024-09-13 21:40:12 -0700 |
| commit | 1ad84a151af7f4b3933983677bc75074f957bdf5 (patch) | |
| tree | 7117c56776c532445f72a32f812108428114da9e | |
| parent | 06a104d55d56aae199ebac1078ca19f0f45202bc (diff) | |
net: ethtool: phy: Don't set the context dev pointer for unfiltered DUMP
The context info allows continuing DUMP requests, shall they fill the
netlink buffer.
In the case of filtered DUMP requests, a reference on the netdev is
grabbed in the .start() callback and release in .done().
Unfiltered DUMP request don't need the dev pointer to be set in the context
info, doing so will trigger an unwanted netdev_put() in .done().
Reported-by: [email protected]
Closes: https://lore.kernel.org/netdev/[email protected]/
Fixes: 17194be4c8e1 ("net: ethtool: Introduce a command to list PHYs on an interface")
Signed-off-by: Maxime Chevallier <[email protected]>
Reviewed-by: Vadim Fedorenko <[email protected]>
Link: https://patch.msgid.link/[email protected]
Signed-off-by: Jakub Kicinski <[email protected]>
| -rw-r--r-- | net/ethtool/phy.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/net/ethtool/phy.c b/net/ethtool/phy.c index 4ef7c6e32d10..ed8f690f6bac 100644 --- a/net/ethtool/phy.c +++ b/net/ethtool/phy.c @@ -251,8 +251,6 @@ static int ethnl_phy_dump_one_dev(struct sk_buff *skb, struct net_device *dev, int ret = 0; void *ehdr; - pri->base.dev = dev; - if (!dev->link_topo) return 0; |