netdev - Re: Regression with commit e532a096be0e ("net: usb: asix: ax88772: add phylib support")

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]

Message-ID: <20210820040635.GA3573@pengutronix.de>
Date:   Fri, 20 Aug 2021 06:06:35 +0200
From:   Oleksij Rempel <o.rempel@...gutronix.de>
To:     Jarkko Nikula <jarkko.nikula@...ux.intel.com>
Cc:     Jakub Kicinski <kuba@...nel.org>,
        Oleksij Rempel <linux@...pel-privat.de>,
        netdev@...r.kernel.org, linux-usb@...r.kernel.org,
        "David S. Miller" <davem@...emloft.net>
Subject: Re: Regression with commit e532a096be0e ("net: usb: asix: ax88772:
 add phylib support")

Hi Jarkko,

On Tue, Aug 17, 2021 at 02:55:20PM +0300, Jarkko Nikula wrote:
> On 8/17/21 12:09 PM, Oleksij Rempel wrote:
> > OK thx, I'll need to your help to debug it:
> > - please send me complete log, or at least parts related to the asix
> >    (dmesg | grep -i Asix)
> 
> I added following dyndebug options if that helps:
> asix.dyndbg=+p ax88179_178a.dyndbg=+p
> 
> Around ~8 s or so after bootup I run following:
> ifconfig eth0 down; sleep 5; ifconfig eth0 up; sleep 5; ifconfig eth0 down
> 
> Attachments have grepped dmesg from v5.13 and linux-next next-20210816.

thank you! It helps.

> > - do the interface is not able to go up at all? For example, it works on
> >    hot plug, but is not working on reboot.
> > - Can you please test it with other link partners.
> > 
> I'm now testing this remotely but can test these later this week or next
> week at the office.

Can you please test attached patch.
And may be add some prints to the ax88772_hw_reset() and ax88772a_hw_reset().
It would be interesting to know, which variant is actually affected.

Regards,
Oleksij
-- 
Pengutronix e.K.                           |                             |
Steuerwalder Str. 21                       | http://www.pengutronix.de/  |
31137 Hildesheim, Germany                  | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

View attachment "0001-net-usb-asix-ax88772-move-embedded-PHY-detection-as-.patch" of type "text/x-diff" (4336 bytes)