| Message ID | 1532764330-14522-1-git-send-email-stefan.wahren@i2se.com |
|---|---|
| State | Accepted, archived |
| Delegated to: | David Miller |
| Headers | show |
| Series | net: lan78xx: fix rx handling before first packet is send | expand |
From: Stefan Wahren <stefan.wahren@i2se.com> Date: Sat, 28 Jul 2018 09:52:10 +0200 > As long the bh tasklet isn't scheduled once, no packet from the rx path > will be handled. Since the tx path also schedule the same tasklet > this situation only persits until the first packet transmission. > So fix this issue by scheduling the tasklet after link reset. > > Link: https://github.com/raspberrypi/linux/issues/2617 > Fixes: 55d7de9de6c3 ("Microchip's LAN7800 family USB 2/3 to 10/100/1000 Ethernet") > Suggested-by: Floris Bos <bos@je-eigen-domein.nl> > Signed-off-by: Stefan Wahren <stefan.wahren@i2se.com> Applied and queued up for -stable, thanks.
diff --git a/drivers/net/usb/lan78xx.c b/drivers/net/usb/lan78xx.c index ed10d49..aeca484 100644 --- a/drivers/net/usb/lan78xx.c +++ b/drivers/net/usb/lan78xx.c @@ -1242,6 +1242,8 @@ static int lan78xx_link_reset(struct lan78xx_net *dev) mod_timer(&dev->stat_monitor, jiffies + STAT_UPDATE_TIMER); } + + tasklet_schedule(&dev->bh); } return ret;
As long the bh tasklet isn't scheduled once, no packet from the rx path will be handled. Since the tx path also schedule the same tasklet this situation only persits until the first packet transmission. So fix this issue by scheduling the tasklet after link reset. Link: https://github.com/raspberrypi/linux/issues/2617 Fixes: 55d7de9de6c3 ("Microchip's LAN7800 family USB 2/3 to 10/100/1000 Ethernet") Suggested-by: Floris Bos <bos@je-eigen-domein.nl> Signed-off-by: Stefan Wahren <stefan.wahren@i2se.com> --- drivers/net/usb/lan78xx.c | 2 ++ 1 file changed, 2 insertions(+)