On Fri, 18 Oct 2019 06:01:25 -0400, Zhu Yanjun wrote: > This change adds support for xmit_more based on the igb commit 6f19e12f6230 > ("igb: flush when in xmit_more mode and under descriptor pressure") and > commit 6b16f9ee89b8 ("net: move skb->xmit_more hint to softnet data") that > were made to igb to support this feature. The function netif_xmit_stopped > is called to check if transmit queue on device is currently unable to send > to determine if we must write the tail because we can add no further > buffers. > When normal packets and/or xmit_more packets fill up tx_desc, it is > necessary to trigger NIC tx reg.
Looks broken. You gotta make sure you check the kick on _every_ return path. There are 4 return statements in each function, you only touched 2. Also the labels should be lower case.