[openwrt/openwrt] bcm47xx: revert bgmac back to the old limited max frame size

LEDE Commits lede-commits at lists.infradead.org
Mon Jul 10 04:51:35 PDT 2023


rmilecki pushed a commit to openwrt/openwrt.git, branch openwrt-22.03:
https://git.openwrt.org/e17f9fd0e8a9d7fbe159495b142b3a890f69db3a

commit e17f9fd0e8a9d7fbe159495b142b3a890f69db3a
Author: Rafał Miłecki <rafal at milecki.pl>
AuthorDate: Tue Feb 7 18:09:40 2023 +0100

    bcm47xx: revert bgmac back to the old limited max frame size
    
    Bumping max frame size has significantly affected network performance
    and memory usage. It was done by upstream commit that first appeared in
    the 5.7 release.
    
    Allocating 512 (BGMAC_RX_RING_SLOTS) buffers, 10 k each, is clearly a
    bad idea on 32 MiB devices. This commit fixes support for Linksys E1000
    V2.1 which gives up after allocating ~346 such buffers running 5.15
    kernel.
    
    Ref: 230c9da963aa ("bcm53xx: revert bgmac back to the old limited max frame size")
    Signed-off-by: Rafał Miłecki <rafal at milecki.pl>
    (cherry picked from commit 4970dd027bced78f8a72aa2baed22c962d3b587b)
---
 ...onnect-to-PHY-even-if-it-is-BGMAC_PHY_NOR.patch | 33 ++++++++++++++++++++++
 1 file changed, 33 insertions(+)

diff --git a/target/linux/bcm47xx/patches-5.10/701-net-bgmac-connect-to-PHY-even-if-it-is-BGMAC_PHY_NOR.patch b/target/linux/bcm47xx/patches-5.10/701-net-bgmac-connect-to-PHY-even-if-it-is-BGMAC_PHY_NOR.patch
new file mode 100644
index 0000000000..2c2eb07b82
--- /dev/null
+++ b/target/linux/bcm47xx/patches-5.10/701-net-bgmac-connect-to-PHY-even-if-it-is-BGMAC_PHY_NOR.patch
@@ -0,0 +1,33 @@
+From: =?UTF-8?q?Rafa=C5=82=20Mi=C5=82ecki?= <rafal at milecki.pl>
+Date: Fri, 10 Jun 2022 13:10:47 +0200
+Subject: [PATCH] bgmac: reduce max frame size to support just MTU 1500
+MIME-Version: 1.0
+Content-Type: text/plain; charset=UTF-8
+Content-Transfer-Encoding: 8bit
+
+bgmac allocates new replacement buffer before handling each received
+frame. Allocating & DMA-preparing 9724 B each time consumes a lot of CPU
+time. Ideally bgmac should just respect currently set MTU but it isn't
+the case right now. For now just revert back to the old limited frame
+size.
+
+This change bumps NAT masquarade speed by ~95%.
+
+Ref: 8c7da63978f1 ("bgmac: configure MTU and add support for frames beyond 8192 byte size")
+Signed-off-by: Rafał Miłecki <rafal at milecki.pl>
+---
+ drivers/net/ethernet/broadcom/bgmac.h | 3 +--
+ 1 file changed, 1 insertion(+), 2 deletions(-)
+
+--- a/drivers/net/ethernet/broadcom/bgmac.h
++++ b/drivers/net/ethernet/broadcom/bgmac.h
+@@ -366,8 +366,7 @@
+ #define BGMAC_RX_FRAME_OFFSET			30		/* There are 2 unused bytes between header and real data */
+ #define BGMAC_RX_BUF_OFFSET			(NET_SKB_PAD + NET_IP_ALIGN - \
+ 						 BGMAC_RX_FRAME_OFFSET)
+-/* Jumbo frame size with FCS */
+-#define BGMAC_RX_MAX_FRAME_SIZE			9724
++#define BGMAC_RX_MAX_FRAME_SIZE			1536
+ #define BGMAC_RX_BUF_SIZE			(BGMAC_RX_FRAME_OFFSET + BGMAC_RX_MAX_FRAME_SIZE)
+ #define BGMAC_RX_ALLOC_SIZE			(SKB_DATA_ALIGN(BGMAC_RX_BUF_SIZE + BGMAC_RX_BUF_OFFSET) + \
+ 						 SKB_DATA_ALIGN(sizeof(struct skb_shared_info)))




More information about the lede-commits mailing list