| From 43bbc4924980f86b7cca628929e1674f8645e2f1 Mon Sep 17 00:00:00 2001 |
| From: Eric Dumazet <edumazet@google.com> |
| Date: Mon, 13 Jan 2020 09:27:11 -0800 |
| Subject: [PATCH] net: usb: lan78xx: limit size of local TSO packets |
| |
| commit f8d7408a4d7f60f8b2df0f81decdc882dd9c20dc upstream. |
| |
| lan78xx_tx_bh() makes sure to not exceed MAX_SINGLE_PACKET_SIZE |
| bytes in the aggregated packets it builds, but does |
| nothing to prevent large GSO packets being submitted. |
| |
| Pierre-Francois reported various hangs when/if TSO is enabled. |
| |
| For localy generated packets, we can use netif_set_gso_max_size() |
| to limit the size of TSO packets. |
| |
| Note that forwarded packets could still hit the issue, |
| so a complete fix might require implementing .ndo_features_check |
| for this driver, forcing a software segmentation if the size |
| of the TSO packet exceeds MAX_SINGLE_PACKET_SIZE. |
| |
| Fixes: 55d7de9de6c3 ("Microchip's LAN7800 family USB 2/3 to 10/100/1000 Ethernet device driver") |
| Signed-off-by: Eric Dumazet <edumazet@google.com> |
| Reported-by: RENARD Pierre-Francois <pfrenard@gmail.com> |
| Tested-by: RENARD Pierre-Francois <pfrenard@gmail.com> |
| Cc: Stefan Wahren <stefan.wahren@i2se.com> |
| Cc: Woojung Huh <woojung.huh@microchip.com> |
| Cc: Microchip Linux Driver Support <UNGLinuxDriver@microchip.com> |
| Signed-off-by: David S. Miller <davem@davemloft.net> |
| Signed-off-by: Paul Gortmaker <paul.gortmaker@windriver.com> |
| |
| diff --git a/drivers/net/usb/lan78xx.c b/drivers/net/usb/lan78xx.c |
| index 2999cae8654e..7dacffd98586 100644 |
| --- a/drivers/net/usb/lan78xx.c |
| +++ b/drivers/net/usb/lan78xx.c |
| @@ -3757,6 +3757,7 @@ static int lan78xx_probe(struct usb_interface *intf, |
| |
| /* MTU range: 68 - 9000 */ |
| netdev->max_mtu = MAX_SINGLE_PACKET_SIZE; |
| + netif_set_gso_max_size(netdev, MAX_SINGLE_PACKET_SIZE - MAX_HEADER); |
| |
| dev->ep_blkin = (intf->cur_altsetting)->endpoint + 0; |
| dev->ep_blkout = (intf->cur_altsetting)->endpoint + 1; |
| -- |
| 2.7.4 |
| |