netdev - Re: [PATCH stable 4.14] net: phy: bcm7xxx: Fixed indirect MMD operations

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] [day] [month] [year] [list]

Date:   Wed, 13 Oct 2021 11:11:44 +0200
From:   Greg KH <greg@...ah.com>
To:     Florian Fainelli <f.fainelli@...il.com>
Cc:     linux-kernel@...r.kernel.org, stable@...r.kernel.org,
        "David S . Miller" <davem@...emloft.net>,
        Andrew Lunn <andrew@...n.ch>,
        "open list:ETHERNET PHY LIBRARY" <netdev@...r.kernel.org>
Subject: Re: [PATCH stable 4.14] net: phy: bcm7xxx: Fixed indirect MMD
 operations

On Mon, Oct 11, 2021 at 11:15:16AM -0700, Florian Fainelli wrote:
> commit d88fd1b546ff19c8040cfaea76bf16aed1c5a0bb upstream
> 
> When EEE support was added to the 28nm EPHY it was assumed that it would
> be able to support the standard clause 45 over clause 22 register access
> method. It turns out that the PHY does not support that, which is the
> very reason for using the indirect shadow mode 2 bank 3 access method.
> 
> Implement {read,write}_mmd to allow the standard PHY library routines
> pertaining to EEE querying and configuration to work correctly on these
> PHYs. This forces us to implement a __phy_set_clr_bits() function that
> does not grab the MDIO bus lock since the PHY driver's {read,write}_mmd
> functions are always called with that lock held.
> 
> Fixes: 83ee102a6998 ("net: phy: bcm7xxx: add support for 28nm EPHY")
> [florian: adjust locking since phy_{read,write}_mmd are called with no
> PHYLIB locks held]
> Signed-off-by: Florian Fainelli <f.fainelli@...il.com>
> Signed-off-by: David S. Miller <davem@...emloft.net>
> ---
>  drivers/net/phy/bcm7xxx.c | 94 +++++++++++++++++++++++++++++++++++++++
>  1 file changed, 94 insertions(+)

All 3 now queued up, thanks!

greg k-h