lkml.org 
[lkml]   [2022]   [Sep]   [22]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRe: [PATCH net] net: phy: Warn about incorrect mdio_bus_phy_resume() state
From


On 9/22/2022 5:31 AM, Jakub Kicinski wrote:
> On Mon, 1 Aug 2022 16:34:03 -0700 Florian Fainelli wrote:
>> Calling mdio_bus_phy_resume() with neither the PHY state machine set to
>> PHY_HALTED nor phydev->mac_managed_pm set to true is a good indication
>> that we can produce a race condition looking like this:
>>
>> CPU0 CPU1
>> bcmgenet_resume
>> -> phy_resume
>> -> phy_init_hw
>> -> phy_start
>> -> phy_resume
>> phy_start_aneg()
>> mdio_bus_phy_resume
>> -> phy_resume
>> -> phy_write(..., BMCR_RESET)
>> -> usleep() -> phy_read()
>>
>> with the phy_resume() function triggering a PHY behavior that might have
>> to be worked around with (see bf8bfc4336f7 ("net: phy: broadcom: Fix
>> brcm_fet_config_init()") for instance) that ultimately leads to an error
>> reading from the PHY.
>
> Hi Florian! There were some follow ups on this one, were all the known
> reports covered at this point or there are still platforms to tweak?

This is the only active thread that I am aware of, and Lukas and I are
in agreement as to what to do next:

https://lore.kernel.org/all/20220918191333.GA2107@wunner.de/

expect him to submit a follow on the warning condition to deal with the
smsc95xx USB driver.

Thanks!
--
Florian

\
 
 \ /
  Last update: 2022-09-22 17:30    [W:0.105 / U:0.572 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site