]> git.ipfire.org Git - thirdparty/kernel/stable-queue.git/commitdiff
drop 3.18 broken patch
authorGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 23 Aug 2018 20:51:47 +0000 (22:51 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 23 Aug 2018 20:51:47 +0000 (22:51 +0200)
queue-3.18/octeon_mgmt-fix-mix-registers-configuration-on-mtu-setup.patch [deleted file]
queue-3.18/series

diff --git a/queue-3.18/octeon_mgmt-fix-mix-registers-configuration-on-mtu-setup.patch b/queue-3.18/octeon_mgmt-fix-mix-registers-configuration-on-mtu-setup.patch
deleted file mode 100644 (file)
index 372dae6..0000000
+++ /dev/null
@@ -1,67 +0,0 @@
-From foo@baz Wed Aug 22 10:36:41 CEST 2018
-From: Alexander Sverdlin <alexander.sverdlin@nsn.com>
-Date: Fri, 13 Jul 2018 17:31:50 +0200
-Subject: octeon_mgmt: Fix MIX registers configuration on MTU setup
-
-From: Alexander Sverdlin <alexander.sverdlin@nsn.com>
-
-[ Upstream commit 4aac0b43474d18f6160302a3caa147d77fa3baa1 ]
-
-octeon_mgmt driver doesn't drop RX frames that are 1-4 bytes bigger than
-MTU set for the corresponding interface. The problem is in the
-AGL_GMX_RX0/1_FRM_MAX register setting, which should not account for VLAN
-tagging.
-
-According to Octeon HW manual:
-"For tagged frames, MAX increases by four bytes for each VLAN found up to a
-maximum of two VLANs, or MAX + 8 bytes."
-
-OCTEON_FRAME_HEADER_LEN "define" is fine for ring buffer management, but
-should not be used for AGL_GMX_RX0/1_FRM_MAX.
-
-The problem could be easily reproduced using "ping" command. If affected
-system has default MTU 1500, other host (having MTU >= 1504) can
-successfully "ping" the affected system with payload size 1473-1476,
-resulting in IP packets of size 1501-1504 accepted by the mgmt driver.
-Fixed system still accepts IP packets of 1500 bytes even with VLAN tagging,
-because the limits are lifted in HW as expected, for every VLAN tag.
-
-Signed-off-by: Alexander Sverdlin <alexander.sverdlin@nokia.com>
-Signed-off-by: David S. Miller <davem@davemloft.net>
-Signed-off-by: Sasha Levin <alexander.levin@microsoft.com>
-Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
----
- drivers/net/ethernet/octeon/octeon_mgmt.c |   14 +++++++++++---
- 1 file changed, 11 insertions(+), 3 deletions(-)
-
---- a/drivers/net/ethernet/octeon/octeon_mgmt.c
-+++ b/drivers/net/ethernet/octeon/octeon_mgmt.c
-@@ -644,7 +644,7 @@ static int octeon_mgmt_set_mac_address(s
- static int octeon_mgmt_change_mtu(struct net_device *netdev, int new_mtu)
- {
-       struct octeon_mgmt *p = netdev_priv(netdev);
--      int size_without_fcs = new_mtu + OCTEON_MGMT_RX_HEADROOM;
-+      int max_packet = new_mtu + ETH_HLEN + ETH_FCS_LEN;
-       /* Limit the MTU to make sure the ethernet packets are between
-        * 64 bytes and 16383 bytes.
-@@ -658,9 +658,17 @@ static int octeon_mgmt_change_mtu(struct
-       netdev->mtu = new_mtu;
--      cvmx_write_csr(p->agl + AGL_GMX_RX_FRM_MAX, size_without_fcs);
-+      /* HW lifts the limit if the frame is VLAN tagged
-+       * (+4 bytes per each tag, up to two tags)
-+       */
-+      cvmx_write_csr(p->agl + AGL_GMX_RX_FRM_MAX, max_packet);
-+      /* Set the hardware to truncate packets larger than the MTU. The jabber
-+       * register must be set to a multiple of 8 bytes, so round up. JABBER is
-+       * an unconditional limit, so we need to account for two possible VLAN
-+       * tags.
-+       */
-       cvmx_write_csr(p->agl + AGL_GMX_RX_JABBER,
--                     (size_without_fcs + 7) & 0xfff8);
-+                     (max_packet + 7 + VLAN_HLEN * 2) & 0xfff8);
-       return 0;
- }
index 78d9775e2f308ebff6927d67fdc82d7260342f1c..daa0786214753d02b47afad562e4c6e4d68b47f0 100644 (file)
@@ -44,7 +44,6 @@ drm-armada-fix-colorkey-mode-property.patch
 arm-dts-am3517.dtsi-disable-reference-to-omap3-otg-controller.patch
 ixgbe-be-more-careful-when-modifying-mac-filters.patch
 qlogic-check-kstrtoul-for-errors.patch
-octeon_mgmt-fix-mix-registers-configuration-on-mtu-setup.patch
 net-usb-rtl8150-demote-allmulti-message-to-dev_dbg.patch
 net-qca_spi-avoid-packet-drop-during-initial-sync.patch
 net-qca_spi-make-sure-the-qca7000-reset-is-triggered.patch