Nicolai Buchwitz 89771ce4f8 net: bcmgenet: unshare the skb before writing the control block ... bcmgenet stores first_cb, last_cb and bytes_sent in the skb control block and reads them back on completion to free the skb and credit BQL. ether_setup() leaves IFF_TX_SKB_SHARING set, so the stack can resubmit a shared skb that is still in the ring. The next transmit overwrites last_cb, so the earlier slots complete without crediting their bytes and BQL stalls the queue until the watchdog resets it. Unshare the skb before writing the control block. Reproduce with pktgen in clone_skb mode, which hands the driver the same skb while earlier copies are still in the ring: modprobe pktgen pg=/proc/net/pktgen echo "add_device eth0" > $pg/kpktgend_0 echo "dst_mac AA:BB:CC:DD:EE:FF" > $pg/eth0 echo "dst 192.0.2.1" > $pg/eth0 echo "clone_skb 8" > $pg/eth0 echo "count 0" > $pg/eth0 echo start > $pg/pgctrl & sleep 5 echo stop > $pg/pgctrl wait cat /sys/class/net/eth0/queues/tx-0/byte_queue_limits/inflight inflight stays nonzero on the now idle queue. clone_skb 0 leaves it at zero. Fixes: f48bed16a7 ("net: bcmgenet: Free skb after last Tx frag") Signed-off-by: Nicolai Buchwitz <nb@tipi-net.de> Reviewed-by: Florian Fainelli <florian.fainelli@broadcom.com> Link: https://patch.msgid.link/20261009-nb-genet-shared-skb-net-v1-1-c6d0a841977d@tipi-net.de Signed-off-by: Paolo Abeni <pabeni@redhat.com>		2026-10-15 12:03:14 +02:00
..
bcmgenet_wol.c	net: bcmgenet: if UMAC was suspended in SW_RESET, restore it to SW_RESET	2026-10-06 18:43:09 -07:00
bcmgenet.c	net: bcmgenet: unshare the skb before writing the control block	2026-10-15 12:03:14 +02:00
bcmgenet.h	net: bcmgenet: allocate RX buffers as page fragments	2026-09-25 17:28:17 -07:00
bcmmii.c	net: bcmgenet: use platform_device_set_of_node()	2026-07-10 16:07:09 +02:00
Makefile	treewide: Add SPDX license identifier - Makefile/Kconfig	2019-05-21 10:50:46 +02:00