netdev - Re: [PATCH] net: dsa: lan9303: consequently nested-lock physical MDIO

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date: Fri, 27 Oct 2023 13:55:16 +0200
From: Andrew Lunn <andrew@...n.ch>
To: "A. Sverdlin" <alexander.sverdlin@...mens.com>
Cc: netdev@...r.kernel.org, Florian Fainelli <f.fainelli@...il.com>,
	Vladimir Oltean <olteanv@...il.com>,
	"David S. Miller" <davem@...emloft.net>,
	Eric Dumazet <edumazet@...gle.com>,
	Jakub Kicinski <kuba@...nel.org>, Paolo Abeni <pabeni@...hat.com>,
	Juergen Beisert <jbe@...gutronix.de>,
	Jerry Ray <jerry.ray@...rochip.com>, Mans Rullgard <mans@...sr.com>,
	stable@...r.kernel.org
Subject: Re: [PATCH] net: dsa: lan9303: consequently nested-lock physical MDIO

On Fri, Oct 27, 2023 at 08:57:38AM +0200, A. Sverdlin wrote:
> From: Alexander Sverdlin <alexander.sverdlin@...mens.com>
> 
> When LAN9303 is MDIO-connected two callchains exist into
> mdio->bus->write():
> 
> 1. switch ports 1&2 ("physical" PHYs):
> 
> virtual (switch-internal) MDIO bus (lan9303_switch_ops->phy_{read|write})->
>   lan9303_mdio_phy_{read|write} -> mdiobus_{read|write}_nested
> 
> 2. LAN9303 virtual PHY:
> 
> virtual MDIO bus (lan9303_phy_{read|write}) ->
>   lan9303_virt_phy_reg_{read|write} -> regmap -> lan9303_mdio_{read|write}


> Cc: stable@...r.kernel.org
> Fixes: dc7005831523 ("net: dsa: LAN9303: add MDIO managed mode support")
> Signed-off-by: Alexander Sverdlin <alexander.sverdlin@...mens.com>

Reviewed-by: Andrew Lunn <andrew@...n.ch>

    Andrew