ipq40xx: essedma: Disable TCP segmentation offload for IPv6
authorSven Eckelmann <sven@narfation.org>
Tue, 9 Jun 2020 13:23:04 +0000 (15:23 +0200)
committerChristian Lamparter <chunkeey@gmail.com>
Sat, 13 Jun 2020 15:41:57 +0000 (17:41 +0200)
It was noticed that the the whole MAC can hang when transferring data from
one ar40xx port (WAN ports) to the CPU and from the CPU back to another
ar40xx port (LAN ports). The CPU was doing only NATing in that process.

Usually, the problem first starts with a simple data corruption:

  $ wget https://cdimage.debian.org/debian-cd/current/amd64/iso-cd/debian-10.4.0-amd64-netinst.iso -O /dev/null
  ...
  Connecting to saimei.ftp.acc.umu.se (saimei.ftp.acc.umu.se)|2001:6b0:19::138|:443... connected.
  ...
  Read  error at byte 48807936/352321536 (Decryption has failed.). Retrying.

But after a short while, the whole MAC will stop to react. No traffic can
be transported anymore from the CPU port from/to the AR40xx PHY/switch and
the MAC has to be resetted.

The whole problem can be avoided by disabling the TSO for IPv6 for this
ethernet MAC driver.

Signed-off-by: Sven Eckelmann <sven@narfation.org>
Acked-by: John Crispin <john@phrozen.org>
(backported from commit 678569505623e50bbbbc344c7e820fb315b79ede,
with updated commit message)
Signed-off-by: Christian Lamparter <chunkeey@gmail.com>
target/linux/ipq40xx/patches-4.14/713-essedma-Disable-TCP-segmentation-offload-for-IPv6.patch [new file with mode: 0644]

diff --git a/target/linux/ipq40xx/patches-4.14/713-essedma-Disable-TCP-segmentation-offload-for-IPv6.patch b/target/linux/ipq40xx/patches-4.14/713-essedma-Disable-TCP-segmentation-offload-for-IPv6.patch
new file mode 100644 (file)
index 0000000..bb36af7
--- /dev/null
@@ -0,0 +1,46 @@
+From: Sven Eckelmann <sven@narfation.org>
+Date: Tue, 9 Jun 2020 14:08:44 +0200
+Subject: essedma: Disable TCP segmentation offload for IPv6
+
+It was noticed that the the whole MAC can hang when transferring data from
+one ar40xx port (WAN ports) to the CPU and from the CPU back to another
+ar40xx port (LAN ports). The CPU was doing only NATing in that process.
+
+Usually, the problem first starts with a simple data corruption:
+
+  $ wget https://cdimage.debian.org/debian-cd/current/amd64/iso-cd/debian-10.4.0-amd64-netinst.iso -O /dev/null
+  ...
+  Connecting to saimei.ftp.acc.umu.se (saimei.ftp.acc.umu.se)|2001:6b0:19::138|:443... connected.
+  ...
+  Read  error at byte 48807936/352321536 (Decryption has failed.). Retrying.
+
+But after a short while, the whole MAC will stop to react. No traffic can
+be transported anymore from the CPU port from/to the AR40xx PHY/switch and
+the MAC has to be resetted.
+
+Signed-off-by: Sven Eckelmann <sven@narfation.org>
+
+--- a/drivers/net/ethernet/qualcomm/essedma/edma_axi.c
++++ b/drivers/net/ethernet/qualcomm/essedma/edma_axi.c
+@@ -972,17 +972,14 @@ static int edma_axi_probe(struct platfor
+               edma_netdev[i]->features = NETIF_F_HW_CSUM | NETIF_F_RXCSUM
+                                     | NETIF_F_HW_VLAN_CTAG_TX
+                                     | NETIF_F_HW_VLAN_CTAG_RX | NETIF_F_SG |
+-                                    NETIF_F_TSO | NETIF_F_TSO6 | NETIF_F_GRO;
++                                    NETIF_F_TSO | NETIF_F_GRO;
+               edma_netdev[i]->hw_features = NETIF_F_HW_CSUM | NETIF_F_RXCSUM |
+                               NETIF_F_HW_VLAN_CTAG_RX
+-                              | NETIF_F_SG | NETIF_F_TSO | NETIF_F_TSO6 |
+-                              NETIF_F_GRO;
++                              | NETIF_F_SG | NETIF_F_TSO | NETIF_F_GRO;
+               edma_netdev[i]->vlan_features = NETIF_F_HW_CSUM | NETIF_F_SG |
+-                                         NETIF_F_TSO | NETIF_F_TSO6 |
+-                                         NETIF_F_GRO;
++                                         NETIF_F_TSO | NETIF_F_GRO;
+               edma_netdev[i]->wanted_features = NETIF_F_HW_CSUM | NETIF_F_SG |
+-                                           NETIF_F_TSO | NETIF_F_TSO6 |
+-                                           NETIF_F_GRO;
++                                           NETIF_F_TSO | NETIF_F_GRO;
+ #ifdef CONFIG_RFS_ACCEL
+               edma_netdev[i]->features |=  NETIF_F_RXHASH | NETIF_F_NTUPLE;