netdev - Re: [Regression] net/phy/micrel.c v4.9.94

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:   Thu, 19 Apr 2018 09:34:42 +0800
From:   Chris Ruehl <chris.ruehl@...ys.com.hk>
To:     Andrew Lunn <andrew@...n.ch>
Cc:     f.fainelli@...il.com, netdev@...r.kernel.org
Subject: Re: [Regression] net/phy/micrel.c v4.9.94



On Thursday, April 19, 2018 09:21 AM, Chris Ruehl wrote:
> On Wednesday, April 18, 2018 09:02 PM, Andrew Lunn wrote:
>> On Wed, Apr 18, 2018 at 02:56:01PM +0200, Andrew Lunn wrote:
>>> On Wed, Apr 18, 2018 at 09:34:16AM +0800, Chris Ruehl wrote:
>>>> Hello,
>>>>
>>>> I like to get your heads up at a regression introduced in 4.9.94
>>>> commitment lead to a kernel ops and make the network unusable on my MX6DL
>>>> customized board.
>>>>
>>>> Race condition resume is called on startup and the phy not yet initialized.
>>>
>>> Hi Chris
>>>
>>> Please could you try
>>>
>>> bfe72442578b ("net: phy: micrel: fix crash when statistic requested for 
>>> KSZ9031 phy")
>>
>> I don't think it is a complete fix. I suspect "Micrel KSZ8795",
>> "Micrel KSZ886X Switch", "Micrel KSZ8061", and "Micrel KS8737" will
>> still have problems.
>>
>> Those four probably need a:
>>
>>          .probe          = kszphy_probe,
>>
>>     Andrew
>>
> 
> Indeed I have the
> [    7.385851] Micrel KSZ9031 Gigabit PHY 2188000.ethernet-1:05: attached PHY 
> driver [Micrel KSZ9031 Gigabit PHY] (mii_bus:phy_addr=2188000.ethernet-1:05, 
> irq=-1)
> 
> first I rollback to a non crashing stable kernel.
> 
> As "bfe72442578b" gonna fix it I check with the next update and verify its works 
> for me.
> 
> Thanks
> Chris

Andrew,

Change my mind. Find the patch you mentioned and will apply and test.

Chris