kernel: use napi_build_skb in mtk_eth_soc
authorFelix Fietkau <nbd@nbd.name>
Mon, 20 May 2024 12:36:55 +0000 (14:36 +0200)
committerFelix Fietkau <nbd@nbd.name>
Thu, 6 Jun 2024 09:47:59 +0000 (11:47 +0200)
Improves performance

Signed-off-by: Felix Fietkau <nbd@nbd.name>
target/linux/generic/pending-6.6/733-01-net-ethernet-mtk_eth_soc-use-napi_build_skb.patch [new file with mode: 0644]

diff --git a/target/linux/generic/pending-6.6/733-01-net-ethernet-mtk_eth_soc-use-napi_build_skb.patch b/target/linux/generic/pending-6.6/733-01-net-ethernet-mtk_eth_soc-use-napi_build_skb.patch
new file mode 100644 (file)
index 0000000..71c0734
--- /dev/null
@@ -0,0 +1,30 @@
+From: Felix Fietkau <nbd@nbd.name>
+Date: Mon, 20 May 2024 14:29:58 +0200
+Subject: [PATCH] net: ethernet: mtk_eth_soc: use napi_build_skb()
+
+The napi_build_skb() can reuse the skb in skb cache per CPU or
+can allocate skbs in bulk, which helps improve the performance.
+
+Signed-off-by: Felix Fietkau <nbd@nbd.name>
+---
+
+--- a/drivers/net/ethernet/mediatek/mtk_eth_soc.c
++++ b/drivers/net/ethernet/mediatek/mtk_eth_soc.c
+@@ -2120,7 +2120,7 @@ static int mtk_poll_rx(struct napi_struc
+                       if (ret != XDP_PASS)
+                               goto skip_rx;
+-                      skb = build_skb(data, PAGE_SIZE);
++                      skb = napi_build_skb(data, PAGE_SIZE);
+                       if (unlikely(!skb)) {
+                               page_pool_put_full_page(ring->page_pool,
+                                                       page, true);
+@@ -2158,7 +2158,7 @@ static int mtk_poll_rx(struct napi_struc
+                       dma_unmap_single(eth->dma_dev, ((u64)trxd.rxd1 | addr64),
+                                        ring->buf_size, DMA_FROM_DEVICE);
+-                      skb = build_skb(data, ring->frag_size);
++                      skb = napi_build_skb(data, ring->frag_size);
+                       if (unlikely(!skb)) {
+                               netdev->stats.rx_dropped++;
+                               skb_free_frag(data);