* [REGRESSION] AX88772 card booted without cable can't receive
@ 2022-06-22 14:16 Anton Lundin
2022-06-23 6:36 ` Oleksij Rempel
0 siblings, 1 reply; 5+ messages in thread
From: Anton Lundin @ 2022-06-22 14:16 UTC (permalink / raw)
To: Oleksij Rempel, David S. Miller; +Cc: linux-kernel, linux-usb, netdev
Hi.
I've found a issue with a Dlink usb ether adapter, that can't receive
anything until it self transmits if it's plugged in while booting, and
doesn't have link.
Later when a cable is attached, link is detected but nothing is received
either by daemons listening to ip address on that interface, or seen
with tcpdump.
The dongle is a:
D-Link Corp. DUB-E100 Fast Ethernet Adapter(rev.C1) [ASIX AX88772]
And it's detected at boot as:
libphy: Asix MDIO Bus: probed
Asix Electronics AX88772C usb-003:004:10: attached PHY driver (mii_bus:phy_addr=usb-003:004:10, irq=POLL)
asix 3-10.4:1.0 eth1: register 'asix' at usb-0000:00:14.0-10.4, ASIX AX88772 USB 2.0 Ethernet, <masked-mac>
usbcore: registered new interface driver asix
While in this state, the hardware starts sending pause frames to the
network when it has recived a couple of frames, and they look like:
0000 01 80 c2 00 00 01 00 00 00 00 00 00 88 08 00 01
0010 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
0020 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
0030 00 00 00 00 00 00 00 00 00 00 00 00
0000 01 80 c2 00 00 01 00 00 00 00 00 00 88 08 00 01
0010 ff ff 00 00 00 00 00 00 00 00 00 00 00 00 00 00
0020 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
0030 00 00 00 00 00 00 00 00 00 00 00 00
And these two frames are repeated every couple of seconds.
The card wakes up when something triggers a transmit on that card, and
then starts receiving traffic as normal.
I've bisected this issue down to:
"net: usb: asix: ax88772: add phylib support" (e532a096be0e)
Reverting that makes the interface work as normal, even if the machine
boots without a cable plugged in.
Another issue found with exactly the same patch is that if it's loaded
as a module, then unloaded and loaded again, it fails to initialize the
card with:
sysfs: cannot create duplicate filename '/devices/virtual/mdio_bus/usb-003:004'
CPU: 0 PID: 3733 Comm: modprobe Tainted: G O 5.15.10-core_64_preempt #3
Hardware name: <masked-hardware-name>
Call Trace:
<TASK>
? dump_stack_lvl+0x34/0x44
? sysfs_warn_dup.cold+0x17/0x24
? sysfs_create_dir_ns+0xbc/0xd0
? kobject_add_internal+0xa6/0x260
? kobject_add+0x7e/0xb0
? preempt_count_add+0x68/0xa0
? device_add+0x10f/0x8d0
? dev_set_name+0x53/0x70
? __mdiobus_register+0xc2/0x350
? __devm_mdiobus_register+0x64/0xb0
? ax88772_bind+0x22a/0x340 [asix]
? usbnet_probe+0x346/0x870
? usb_match_dynamic_id+0x8f/0xa0
? usb_probe_interface+0x9b/0x150
? really_probe.part.0+0x237/0x280
? __driver_probe_device+0x8c/0xd0
? driver_probe_device+0x1e/0xe0
? __driver_attach+0xa8/0x170
? __device_attach_driver+0xe0/0xe0
? bus_for_each_dev+0x77/0xc0
? bus_add_driver+0x10b/0x1c0
? driver_register+0x8b/0xe0
? usb_register_driver+0x84/0x120
? 0xffffffffc06e4000
? do_one_initcall+0x41/0x1f0
? kmem_cache_alloc_trace+0x3f/0x1b0
? do_init_module+0x5c/0x260
? __do_sys_finit_module+0xa0/0xe0
? do_syscall_64+0x35/0x80
? entry_SYSCALL_64_after_hwframe+0x44/0xae
</TASK>
kobject_add_internal failed for usb-003:004 with -EEXIST, don't try to register things with the same name in the same directory.
libphy: mii_bus usb-003:004 failed to register
asix: probe of 3-10.4:1.0 failed with error -22
usbcore: registered new interface driver asix
Both these issues with "net: usb: asix: ax88772: add phylib support"
(e532a096be0e) can be reproduced all the way from when it was introduced
to linus current tree.
I'm sorry to say that I don't know enough about either libphy or asix to
figure out what cause the issues can be.
//Anton
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [REGRESSION] AX88772 card booted without cable can't receive
2022-06-22 14:16 [REGRESSION] AX88772 card booted without cable can't receive Anton Lundin
@ 2022-06-23 6:36 ` Oleksij Rempel
2022-06-24 8:17 ` Oleksij Rempel
0 siblings, 1 reply; 5+ messages in thread
From: Oleksij Rempel @ 2022-06-23 6:36 UTC (permalink / raw)
To: Anton Lundin; +Cc: David S. Miller, linux-kernel, linux-usb, netdev
Hi Anton,
Thank you for your report! I'll take a look on it ASAP.
Regards,
Oleksij
On Wed, Jun 22, 2022 at 04:16:38PM +0200, Anton Lundin wrote:
> Hi.
>
> I've found a issue with a Dlink usb ether adapter, that can't receive
> anything until it self transmits if it's plugged in while booting, and
> doesn't have link.
>
> Later when a cable is attached, link is detected but nothing is received
> either by daemons listening to ip address on that interface, or seen
> with tcpdump.
>
> The dongle is a:
> D-Link Corp. DUB-E100 Fast Ethernet Adapter(rev.C1) [ASIX AX88772]
>
> And it's detected at boot as:
> libphy: Asix MDIO Bus: probed
> Asix Electronics AX88772C usb-003:004:10: attached PHY driver (mii_bus:phy_addr=usb-003:004:10, irq=POLL)
> asix 3-10.4:1.0 eth1: register 'asix' at usb-0000:00:14.0-10.4, ASIX AX88772 USB 2.0 Ethernet, <masked-mac>
> usbcore: registered new interface driver asix
>
>
> While in this state, the hardware starts sending pause frames to the
> network when it has recived a couple of frames, and they look like:
> 0000 01 80 c2 00 00 01 00 00 00 00 00 00 88 08 00 01
> 0010 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
> 0020 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
> 0030 00 00 00 00 00 00 00 00 00 00 00 00
>
> 0000 01 80 c2 00 00 01 00 00 00 00 00 00 88 08 00 01
> 0010 ff ff 00 00 00 00 00 00 00 00 00 00 00 00 00 00
> 0020 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
> 0030 00 00 00 00 00 00 00 00 00 00 00 00
>
> And these two frames are repeated every couple of seconds.
>
> The card wakes up when something triggers a transmit on that card, and
> then starts receiving traffic as normal.
>
> I've bisected this issue down to:
> "net: usb: asix: ax88772: add phylib support" (e532a096be0e)
>
>
> Reverting that makes the interface work as normal, even if the machine
> boots without a cable plugged in.
>
> Another issue found with exactly the same patch is that if it's loaded
> as a module, then unloaded and loaded again, it fails to initialize the
> card with:
>
> sysfs: cannot create duplicate filename '/devices/virtual/mdio_bus/usb-003:004'
> CPU: 0 PID: 3733 Comm: modprobe Tainted: G O 5.15.10-core_64_preempt #3
> Hardware name: <masked-hardware-name>
> Call Trace:
> <TASK>
> ? dump_stack_lvl+0x34/0x44
> ? sysfs_warn_dup.cold+0x17/0x24
> ? sysfs_create_dir_ns+0xbc/0xd0
> ? kobject_add_internal+0xa6/0x260
> ? kobject_add+0x7e/0xb0
> ? preempt_count_add+0x68/0xa0
> ? device_add+0x10f/0x8d0
> ? dev_set_name+0x53/0x70
> ? __mdiobus_register+0xc2/0x350
> ? __devm_mdiobus_register+0x64/0xb0
> ? ax88772_bind+0x22a/0x340 [asix]
> ? usbnet_probe+0x346/0x870
> ? usb_match_dynamic_id+0x8f/0xa0
> ? usb_probe_interface+0x9b/0x150
> ? really_probe.part.0+0x237/0x280
> ? __driver_probe_device+0x8c/0xd0
> ? driver_probe_device+0x1e/0xe0
> ? __driver_attach+0xa8/0x170
> ? __device_attach_driver+0xe0/0xe0
> ? bus_for_each_dev+0x77/0xc0
> ? bus_add_driver+0x10b/0x1c0
> ? driver_register+0x8b/0xe0
> ? usb_register_driver+0x84/0x120
> ? 0xffffffffc06e4000
> ? do_one_initcall+0x41/0x1f0
> ? kmem_cache_alloc_trace+0x3f/0x1b0
> ? do_init_module+0x5c/0x260
> ? __do_sys_finit_module+0xa0/0xe0
> ? do_syscall_64+0x35/0x80
> ? entry_SYSCALL_64_after_hwframe+0x44/0xae
> </TASK>
> kobject_add_internal failed for usb-003:004 with -EEXIST, don't try to register things with the same name in the same directory.
> libphy: mii_bus usb-003:004 failed to register
> asix: probe of 3-10.4:1.0 failed with error -22
> usbcore: registered new interface driver asix
>
>
> Both these issues with "net: usb: asix: ax88772: add phylib support"
> (e532a096be0e) can be reproduced all the way from when it was introduced
> to linus current tree.
>
>
> I'm sorry to say that I don't know enough about either libphy or asix to
> figure out what cause the issues can be.
>
>
>
> //Anton
>
--
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 |
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [REGRESSION] AX88772 card booted without cable can't receive
2022-06-23 6:36 ` Oleksij Rempel
@ 2022-06-24 8:17 ` Oleksij Rempel
2022-06-27 12:32 ` Anton Lundin
0 siblings, 1 reply; 5+ messages in thread
From: Oleksij Rempel @ 2022-06-24 8:17 UTC (permalink / raw)
To: Anton Lundin
Cc: David S. Miller, linux-kernel, linux-usb, netdev, Lukas Wunner
Hi Anton,
On Thu, Jun 23, 2022 at 08:36:49AM +0200, Oleksij Rempel wrote:
> Hi Anton,
>
> Thank you for your report! I'll take a look on it ASAP.
>
> Regards,
> Oleksij
>
> On Wed, Jun 22, 2022 at 04:16:38PM +0200, Anton Lundin wrote:
> > Hi.
> >
> > I've found a issue with a Dlink usb ether adapter, that can't receive
> > anything until it self transmits if it's plugged in while booting, and
> > doesn't have link.
> >
> > Later when a cable is attached, link is detected but nothing is received
> > either by daemons listening to ip address on that interface, or seen
> > with tcpdump.
> >
> > The dongle is a:
> > D-Link Corp. DUB-E100 Fast Ethernet Adapter(rev.C1) [ASIX AX88772]
> >
> > And it's detected at boot as:
> > libphy: Asix MDIO Bus: probed
> > Asix Electronics AX88772C usb-003:004:10: attached PHY driver (mii_bus:phy_addr=usb-003:004:10, irq=POLL)
> > asix 3-10.4:1.0 eth1: register 'asix' at usb-0000:00:14.0-10.4, ASIX AX88772 USB 2.0 Ethernet, <masked-mac>
> > usbcore: registered new interface driver asix
> >
> >
> > While in this state, the hardware starts sending pause frames to the
> > network when it has recived a couple of frames, and they look like:
> > 0000 01 80 c2 00 00 01 00 00 00 00 00 00 88 08 00 01
> > 0010 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
> > 0020 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
> > 0030 00 00 00 00 00 00 00 00 00 00 00 00
> >
> > 0000 01 80 c2 00 00 01 00 00 00 00 00 00 88 08 00 01
> > 0010 ff ff 00 00 00 00 00 00 00 00 00 00 00 00 00 00
> > 0020 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
> > 0030 00 00 00 00 00 00 00 00 00 00 00 00
> >
> > And these two frames are repeated every couple of seconds.
> >
> > The card wakes up when something triggers a transmit on that card, and
> > then starts receiving traffic as normal.
> >
> > I've bisected this issue down to:
> > "net: usb: asix: ax88772: add phylib support" (e532a096be0e)
> >
> >
> > Reverting that makes the interface work as normal, even if the machine
> > boots without a cable plugged in.
> >
> > Another issue found with exactly the same patch is that if it's loaded
> > as a module, then unloaded and loaded again, it fails to initialize the
> > card with:
> >
> > sysfs: cannot create duplicate filename '/devices/virtual/mdio_bus/usb-003:004'
> > CPU: 0 PID: 3733 Comm: modprobe Tainted: G O 5.15.10-core_64_preempt #3
> > Hardware name: <masked-hardware-name>
> > Call Trace:
> > <TASK>
> > ? dump_stack_lvl+0x34/0x44
> > ? sysfs_warn_dup.cold+0x17/0x24
> > ? sysfs_create_dir_ns+0xbc/0xd0
> > ? kobject_add_internal+0xa6/0x260
> > ? kobject_add+0x7e/0xb0
> > ? preempt_count_add+0x68/0xa0
> > ? device_add+0x10f/0x8d0
> > ? dev_set_name+0x53/0x70
> > ? __mdiobus_register+0xc2/0x350
> > ? __devm_mdiobus_register+0x64/0xb0
> > ? ax88772_bind+0x22a/0x340 [asix]
> > ? usbnet_probe+0x346/0x870
> > ? usb_match_dynamic_id+0x8f/0xa0
> > ? usb_probe_interface+0x9b/0x150
> > ? really_probe.part.0+0x237/0x280
> > ? __driver_probe_device+0x8c/0xd0
> > ? driver_probe_device+0x1e/0xe0
> > ? __driver_attach+0xa8/0x170
> > ? __device_attach_driver+0xe0/0xe0
> > ? bus_for_each_dev+0x77/0xc0
> > ? bus_add_driver+0x10b/0x1c0
> > ? driver_register+0x8b/0xe0
> > ? usb_register_driver+0x84/0x120
> > ? 0xffffffffc06e4000
> > ? do_one_initcall+0x41/0x1f0
> > ? kmem_cache_alloc_trace+0x3f/0x1b0
> > ? do_init_module+0x5c/0x260
> > ? __do_sys_finit_module+0xa0/0xe0
> > ? do_syscall_64+0x35/0x80
> > ? entry_SYSCALL_64_after_hwframe+0x44/0xae
> > </TASK>
> > kobject_add_internal failed for usb-003:004 with -EEXIST, don't try to register things with the same name in the same directory.
> > libphy: mii_bus usb-003:004 failed to register
> > asix: probe of 3-10.4:1.0 failed with error -22
> > usbcore: registered new interface driver asix
> >
> >
> > Both these issues with "net: usb: asix: ax88772: add phylib support"
> > (e532a096be0e) can be reproduced all the way from when it was introduced
> > to linus current tree.
> >
> >
> > I'm sorry to say that I don't know enough about either libphy or asix to
> > figure out what cause the issues can be.
It looks like we have here 3 different bugs:
- no rx before tx
addresses by this patch:
https://lore.kernel.org/all/20220624075139.3139300-1-o.rempel@pengutronix.de/
- pause frames flood without advertising pause frames support
addresses by this patch:
https://lore.kernel.org/all/20220624075139.3139300-2-o.rempel@pengutronix.de/
Can you please test this patches.
- not everything is properly cleaned after module unload.
The last one i as bit more complicated. Lukas Wunner spend last months to solve
this issues:
https://lore.kernel.org/all/d1c87ebe9fc502bffcd1576e238d685ad08321e4.1655987888.git.lukas@wunner.de/
Not sure if this work can be easily backported to v5.15.
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 |
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [REGRESSION] AX88772 card booted without cable can't receive
2022-06-24 8:17 ` Oleksij Rempel
@ 2022-06-27 12:32 ` Anton Lundin
2022-06-27 13:02 ` Oleksij Rempel
0 siblings, 1 reply; 5+ messages in thread
From: Anton Lundin @ 2022-06-27 12:32 UTC (permalink / raw)
To: Oleksij Rempel
Cc: David S. Miller, linux-kernel, linux-usb, netdev, Lukas Wunner
On 24 June, 2022 - Oleksij Rempel wrote:
> Hi Anton,
>
> On Thu, Jun 23, 2022 at 08:36:49AM +0200, Oleksij Rempel wrote:
> > Hi Anton,
> >
> > Thank you for your report! I'll take a look on it ASAP.
> >
> > Regards,
> > Oleksij
> >
> > On Wed, Jun 22, 2022 at 04:16:38PM +0200, Anton Lundin wrote:
> > > Hi.
> > >
> > > I've found a issue with a Dlink usb ether adapter, that can't receive
> > > anything until it self transmits if it's plugged in while booting, and
> > > doesn't have link.
> > >
> > > Later when a cable is attached, link is detected but nothing is received
> > > either by daemons listening to ip address on that interface, or seen
> > > with tcpdump.
> > >
> > > The dongle is a:
> > > D-Link Corp. DUB-E100 Fast Ethernet Adapter(rev.C1) [ASIX AX88772]
> > >
> > > And it's detected at boot as:
> > > libphy: Asix MDIO Bus: probed
> > > Asix Electronics AX88772C usb-003:004:10: attached PHY driver (mii_bus:phy_addr=usb-003:004:10, irq=POLL)
> > > asix 3-10.4:1.0 eth1: register 'asix' at usb-0000:00:14.0-10.4, ASIX AX88772 USB 2.0 Ethernet, <masked-mac>
> > > usbcore: registered new interface driver asix
> > >
> > >
> > > While in this state, the hardware starts sending pause frames to the
> > > network when it has recived a couple of frames, and they look like:
> > > 0000 01 80 c2 00 00 01 00 00 00 00 00 00 88 08 00 01
> > > 0010 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
> > > 0020 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
> > > 0030 00 00 00 00 00 00 00 00 00 00 00 00
> > >
> > > 0000 01 80 c2 00 00 01 00 00 00 00 00 00 88 08 00 01
> > > 0010 ff ff 00 00 00 00 00 00 00 00 00 00 00 00 00 00
> > > 0020 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
> > > 0030 00 00 00 00 00 00 00 00 00 00 00 00
> > >
> > > And these two frames are repeated every couple of seconds.
> > >
> > > The card wakes up when something triggers a transmit on that card, and
> > > then starts receiving traffic as normal.
> > >
> > > I've bisected this issue down to:
> > > "net: usb: asix: ax88772: add phylib support" (e532a096be0e)
> > >
> > >
> > > Reverting that makes the interface work as normal, even if the machine
> > > boots without a cable plugged in.
> > >
> > > Another issue found with exactly the same patch is that if it's loaded
> > > as a module, then unloaded and loaded again, it fails to initialize the
> > > card with:
> > >
> > > sysfs: cannot create duplicate filename '/devices/virtual/mdio_bus/usb-003:004'
> > > CPU: 0 PID: 3733 Comm: modprobe Tainted: G O 5.15.10-core_64_preempt #3
> > > Hardware name: <masked-hardware-name>
> > > Call Trace:
> > > <TASK>
> > > ? dump_stack_lvl+0x34/0x44
> > > ? sysfs_warn_dup.cold+0x17/0x24
> > > ? sysfs_create_dir_ns+0xbc/0xd0
> > > ? kobject_add_internal+0xa6/0x260
> > > ? kobject_add+0x7e/0xb0
> > > ? preempt_count_add+0x68/0xa0
> > > ? device_add+0x10f/0x8d0
> > > ? dev_set_name+0x53/0x70
> > > ? __mdiobus_register+0xc2/0x350
> > > ? __devm_mdiobus_register+0x64/0xb0
> > > ? ax88772_bind+0x22a/0x340 [asix]
> > > ? usbnet_probe+0x346/0x870
> > > ? usb_match_dynamic_id+0x8f/0xa0
> > > ? usb_probe_interface+0x9b/0x150
> > > ? really_probe.part.0+0x237/0x280
> > > ? __driver_probe_device+0x8c/0xd0
> > > ? driver_probe_device+0x1e/0xe0
> > > ? __driver_attach+0xa8/0x170
> > > ? __device_attach_driver+0xe0/0xe0
> > > ? bus_for_each_dev+0x77/0xc0
> > > ? bus_add_driver+0x10b/0x1c0
> > > ? driver_register+0x8b/0xe0
> > > ? usb_register_driver+0x84/0x120
> > > ? 0xffffffffc06e4000
> > > ? do_one_initcall+0x41/0x1f0
> > > ? kmem_cache_alloc_trace+0x3f/0x1b0
> > > ? do_init_module+0x5c/0x260
> > > ? __do_sys_finit_module+0xa0/0xe0
> > > ? do_syscall_64+0x35/0x80
> > > ? entry_SYSCALL_64_after_hwframe+0x44/0xae
> > > </TASK>
> > > kobject_add_internal failed for usb-003:004 with -EEXIST, don't try to register things with the same name in the same directory.
> > > libphy: mii_bus usb-003:004 failed to register
> > > asix: probe of 3-10.4:1.0 failed with error -22
> > > usbcore: registered new interface driver asix
> > >
> > >
> > > Both these issues with "net: usb: asix: ax88772: add phylib support"
> > > (e532a096be0e) can be reproduced all the way from when it was introduced
> > > to linus current tree.
> > >
> > >
> > > I'm sorry to say that I don't know enough about either libphy or asix to
> > > figure out what cause the issues can be.
>
> It looks like we have here 3 different bugs:
> - no rx before tx
> addresses by this patch:
> https://lore.kernel.org/all/20220624075139.3139300-1-o.rempel@pengutronix.de/
> - pause frames flood without advertising pause frames support
> addresses by this patch:
> https://lore.kernel.org/all/20220624075139.3139300-2-o.rempel@pengutronix.de/
>
> Can you please test this patches.
I've tested these and after these patches I think these two issues are
fixed.
If you'd like you can add:
Tested-by: Anton Lundin <glance@acc.umu.se>
> - not everything is properly cleaned after module unload.
>
> The last one i as bit more complicated. Lukas Wunner spend last months to solve
> this issues:
> https://lore.kernel.org/all/d1c87ebe9fc502bffcd1576e238d685ad08321e4.1655987888.git.lukas@wunner.de/
>
Ok, no worries. If it's already known and fixed in later revisions, I'm
fine with that.
//Anton
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [REGRESSION] AX88772 card booted without cable can't receive
2022-06-27 12:32 ` Anton Lundin
@ 2022-06-27 13:02 ` Oleksij Rempel
0 siblings, 0 replies; 5+ messages in thread
From: Oleksij Rempel @ 2022-06-27 13:02 UTC (permalink / raw)
To: Anton Lundin
Cc: David S. Miller, linux-kernel, linux-usb, netdev, Lukas Wunner
On Mon, Jun 27, 2022 at 02:32:29PM +0200, Anton Lundin wrote:
> On 24 June, 2022 - Oleksij Rempel wrote:
>
> > Hi Anton,
> >
> > On Thu, Jun 23, 2022 at 08:36:49AM +0200, Oleksij Rempel wrote:
> > > Hi Anton,
> > >
> > > Thank you for your report! I'll take a look on it ASAP.
> > >
> > > Regards,
> > > Oleksij
> > >
> > > On Wed, Jun 22, 2022 at 04:16:38PM +0200, Anton Lundin wrote:
> > > > Hi.
> > > >
> > > > I've found a issue with a Dlink usb ether adapter, that can't receive
> > > > anything until it self transmits if it's plugged in while booting, and
> > > > doesn't have link.
> > > >
> > > > Later when a cable is attached, link is detected but nothing is received
> > > > either by daemons listening to ip address on that interface, or seen
> > > > with tcpdump.
> > > >
> > > > The dongle is a:
> > > > D-Link Corp. DUB-E100 Fast Ethernet Adapter(rev.C1) [ASIX AX88772]
> > > >
> > > > And it's detected at boot as:
> > > > libphy: Asix MDIO Bus: probed
> > > > Asix Electronics AX88772C usb-003:004:10: attached PHY driver (mii_bus:phy_addr=usb-003:004:10, irq=POLL)
> > > > asix 3-10.4:1.0 eth1: register 'asix' at usb-0000:00:14.0-10.4, ASIX AX88772 USB 2.0 Ethernet, <masked-mac>
> > > > usbcore: registered new interface driver asix
> > > >
> > > >
> > > > While in this state, the hardware starts sending pause frames to the
> > > > network when it has recived a couple of frames, and they look like:
> > > > 0000 01 80 c2 00 00 01 00 00 00 00 00 00 88 08 00 01
> > > > 0010 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
> > > > 0020 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
> > > > 0030 00 00 00 00 00 00 00 00 00 00 00 00
> > > >
> > > > 0000 01 80 c2 00 00 01 00 00 00 00 00 00 88 08 00 01
> > > > 0010 ff ff 00 00 00 00 00 00 00 00 00 00 00 00 00 00
> > > > 0020 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
> > > > 0030 00 00 00 00 00 00 00 00 00 00 00 00
> > > >
> > > > And these two frames are repeated every couple of seconds.
> > > >
> > > > The card wakes up when something triggers a transmit on that card, and
> > > > then starts receiving traffic as normal.
> > > >
> > > > I've bisected this issue down to:
> > > > "net: usb: asix: ax88772: add phylib support" (e532a096be0e)
> > > >
> > > >
> > > > Reverting that makes the interface work as normal, even if the machine
> > > > boots without a cable plugged in.
> > > >
> > > > Another issue found with exactly the same patch is that if it's loaded
> > > > as a module, then unloaded and loaded again, it fails to initialize the
> > > > card with:
> > > >
> > > > sysfs: cannot create duplicate filename '/devices/virtual/mdio_bus/usb-003:004'
> > > > CPU: 0 PID: 3733 Comm: modprobe Tainted: G O 5.15.10-core_64_preempt #3
> > > > Hardware name: <masked-hardware-name>
> > > > Call Trace:
> > > > <TASK>
> > > > ? dump_stack_lvl+0x34/0x44
> > > > ? sysfs_warn_dup.cold+0x17/0x24
> > > > ? sysfs_create_dir_ns+0xbc/0xd0
> > > > ? kobject_add_internal+0xa6/0x260
> > > > ? kobject_add+0x7e/0xb0
> > > > ? preempt_count_add+0x68/0xa0
> > > > ? device_add+0x10f/0x8d0
> > > > ? dev_set_name+0x53/0x70
> > > > ? __mdiobus_register+0xc2/0x350
> > > > ? __devm_mdiobus_register+0x64/0xb0
> > > > ? ax88772_bind+0x22a/0x340 [asix]
> > > > ? usbnet_probe+0x346/0x870
> > > > ? usb_match_dynamic_id+0x8f/0xa0
> > > > ? usb_probe_interface+0x9b/0x150
> > > > ? really_probe.part.0+0x237/0x280
> > > > ? __driver_probe_device+0x8c/0xd0
> > > > ? driver_probe_device+0x1e/0xe0
> > > > ? __driver_attach+0xa8/0x170
> > > > ? __device_attach_driver+0xe0/0xe0
> > > > ? bus_for_each_dev+0x77/0xc0
> > > > ? bus_add_driver+0x10b/0x1c0
> > > > ? driver_register+0x8b/0xe0
> > > > ? usb_register_driver+0x84/0x120
> > > > ? 0xffffffffc06e4000
> > > > ? do_one_initcall+0x41/0x1f0
> > > > ? kmem_cache_alloc_trace+0x3f/0x1b0
> > > > ? do_init_module+0x5c/0x260
> > > > ? __do_sys_finit_module+0xa0/0xe0
> > > > ? do_syscall_64+0x35/0x80
> > > > ? entry_SYSCALL_64_after_hwframe+0x44/0xae
> > > > </TASK>
> > > > kobject_add_internal failed for usb-003:004 with -EEXIST, don't try to register things with the same name in the same directory.
> > > > libphy: mii_bus usb-003:004 failed to register
> > > > asix: probe of 3-10.4:1.0 failed with error -22
> > > > usbcore: registered new interface driver asix
> > > >
> > > >
> > > > Both these issues with "net: usb: asix: ax88772: add phylib support"
> > > > (e532a096be0e) can be reproduced all the way from when it was introduced
> > > > to linus current tree.
> > > >
> > > >
> > > > I'm sorry to say that I don't know enough about either libphy or asix to
> > > > figure out what cause the issues can be.
> >
> > It looks like we have here 3 different bugs:
> > - no rx before tx
> > addresses by this patch:
> > https://lore.kernel.org/all/20220624075139.3139300-1-o.rempel@pengutronix.de/
> > - pause frames flood without advertising pause frames support
> > addresses by this patch:
> > https://lore.kernel.org/all/20220624075139.3139300-2-o.rempel@pengutronix.de/
> >
> > Can you please test this patches.
>
> I've tested these and after these patches I think these two issues are
> fixed.
>
> If you'd like you can add:
> Tested-by: Anton Lundin <glance@acc.umu.se>
Can you please add your Tested-by directly as response to the patches.
This will reduce some work for maintainers.
> > - not everything is properly cleaned after module unload.
> >
> > The last one i as bit more complicated. Lukas Wunner spend last months to solve
> > this issues:
> > https://lore.kernel.org/all/d1c87ebe9fc502bffcd1576e238d685ad08321e4.1655987888.git.lukas@wunner.de/
> >
>
> Ok, no worries. If it's already known and fixed in later revisions, I'm
> fine with that.
OK,
Thank you for testing.
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 |
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2022-06-27 13:03 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-06-22 14:16 [REGRESSION] AX88772 card booted without cable can't receive Anton Lundin
2022-06-23 6:36 ` Oleksij Rempel
2022-06-24 8:17 ` Oleksij Rempel
2022-06-27 12:32 ` Anton Lundin
2022-06-27 13:02 ` Oleksij Rempel
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).