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
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>