Thread (38 messages) 38 messages, 6 authors, 27d ago

Re: [PATCH net-next v2 02/12] net: ethtool: relax ethnl_req_get_phydev() locking assertion

From: Eric Dumazet <edumazet@google.com>
Date: 2026-06-09 15:34:00

On Thu, Jun 4, 2026 at 5:29 PM Jakub Kicinski [off-list ref] wrote:
phydev <> netdev linking and lifecycle depends on rtnl_lock.
We want to switch to instance locks for most ethtool ops.
Let's add an assert that ops locked devices don't use phydev
today. If one does we can either opt the phy ops out of
being purely ops locked, or do deeper surgery to make phy
locking ops-compatible. I don't think there's any fundamental
challenge to make that work.

Reviewed-by: Nicolai Buchwitz <redacted>
Reviewed-by: Maxime Chevallier <maxime.chevallier@bootlin.com>
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
---
Reviewed-by: Eric Dumazet <edumazet@google.com>
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help