From: Sieng Piaw Liew Date: Wed, 15 Jun 2022 04:17:24 +0000 (+0800) Subject: ath79: ag71xx: reuse skbuff_head with napi skb api X-Git-Url: http://git.cdn.openwrt.org/?a=commitdiff_plain;h=265f402fbdd8444835b43954bf27a83bc9369b91;p=openwrt%2Fstaging%2Fstintel.git ath79: ag71xx: reuse skbuff_head with napi skb api napi_build_skb() reuses NAPI skbuff_head cache in order to save some cycles on freeing/allocating skbuff_heads on every new Rx or completed Tx. Use napi_consume_skb() to feed the cache with skbuff_heads of completed Tx so it's never empty. Signed-off-by: Sieng Piaw Liew [ fixed commit title ] Signed-off-by: Christian Marangi --- diff --git a/target/linux/ath79/files/drivers/net/ethernet/atheros/ag71xx/ag71xx_main.c b/target/linux/ath79/files/drivers/net/ethernet/atheros/ag71xx/ag71xx_main.c index 2fc18d5979..d6e8dd20c8 100644 --- a/target/linux/ath79/files/drivers/net/ethernet/atheros/ag71xx/ag71xx_main.c +++ b/target/linux/ath79/files/drivers/net/ethernet/atheros/ag71xx/ag71xx_main.c @@ -1278,7 +1278,7 @@ static int ag71xx_tx_packets(struct ag71xx *ag, bool flush) if (!skb) continue; - dev_kfree_skb_any(skb); + napi_consume_skb(skb, !flush); ring->buf[i].skb = NULL; bytes_compl += ring->buf[i].len; @@ -1352,7 +1352,11 @@ static int ag71xx_rx_packets(struct ag71xx *ag, int limit) dev->stats.rx_packets++; dev->stats.rx_bytes += pktlen; +#if LINUX_VERSION_CODE < KERNEL_VERSION(5,12,0) skb = build_skb(ring->buf[i].rx_buf, ag71xx_buffer_size(ag)); +#else + skb = napi_build_skb(ring->buf[i].rx_buf, ag71xx_buffer_size(ag)); +#endif if (!skb) { skb_free_frag(ring->buf[i].rx_buf); goto next;