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
| ||
|
Date: Fri, 27 Mar 2020 01:09:56 +0100 From: Andrew Lunn <andrew@...n.ch> To: Heiner Kallweit <hkallweit1@...il.com> Cc: Florian Fainelli <f.fainelli@...il.com>, Russell King - ARM Linux <linux@...linux.org.uk>, David Miller <davem@...emloft.net>, "netdev@...r.kernel.org" <netdev@...r.kernel.org> Subject: Re: [PATCH net-next v2] net: phy: probe PHY drivers synchronously On Fri, Mar 27, 2020 at 01:00:22AM +0100, Heiner Kallweit wrote: > If we have scenarios like > > mdiobus_register() > -> loads PHY driver module(s) > -> registers PHY driver(s) > -> may schedule async probe > phydev = mdiobus_get_phy() > <phydev action involving PHY driver> > > or > > phydev = phy_device_create() > -> loads PHY driver module > -> registers PHY driver > -> may schedule async probe > <phydev action involving PHY driver> > > then we expect the PHY driver to be bound to the phydev when triggering > the action. This may not be the case in case of asynchronous probing. > Therefore ensure that PHY drivers are probed synchronously. > > Default still is sync probing, except async probing is explicitly > requested. I saw some comments that the intention is to promote > async probing for more parallelism in boot process and want to be > prepared for the case that the default is changed to async probing. > > Signed-off-by: Heiner Kallweit <hkallweit1@...il.com> Reviewed-by: Andrew Lunn <andrew@...n.ch> Andrew
Powered by blists - more mailing lists