net: xilinx: replace dev_kfree_skb_irq by dev_consume_skb_irq for drop profiles
authorYang Wei <yang.wei9@zte.com.cn>
Thu, 14 Feb 2019 14:50:58 +0000 (22:50 +0800)
committerDavid S. Miller <davem@davemloft.net>
Thu, 14 Feb 2019 17:43:15 +0000 (12:43 -0500)
dev_consume_skb_irq() should be called when skb xmit done. It makes
drop profiles(dropwatch, perf) more friendly.

Signed-off-by: Yang Wei <yang.wei9@zte.com.cn>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/xilinx/ll_temac_main.c
drivers/net/ethernet/xilinx/xilinx_axienet_main.c
drivers/net/ethernet/xilinx/xilinx_emaclite.c

index 15bb058..44efffb 100644 (file)
@@ -630,7 +630,7 @@ static void temac_start_xmit_done(struct net_device *ndev)
                dma_unmap_single(ndev->dev.parent, cur_p->phys, cur_p->len,
                                 DMA_TO_DEVICE);
                if (cur_p->app4)
-                       dev_kfree_skb_irq((struct sk_buff *)cur_p->app4);
+                       dev_consume_skb_irq((struct sk_buff *)cur_p->app4);
                cur_p->app0 = 0;
                cur_p->app1 = 0;
                cur_p->app2 = 0;
index 0789d8a..ec7e7ec 100644 (file)
@@ -595,7 +595,7 @@ static void axienet_start_xmit_done(struct net_device *ndev)
                                (cur_p->cntrl & XAXIDMA_BD_CTRL_LENGTH_MASK),
                                DMA_TO_DEVICE);
                if (cur_p->app4)
-                       dev_kfree_skb_irq((struct sk_buff *)cur_p->app4);
+                       dev_consume_skb_irq((struct sk_buff *)cur_p->app4);
                /*cur_p->phys = 0;*/
                cur_p->app0 = 0;
                cur_p->app1 = 0;
index 639e3e9..b03a417 100644 (file)
@@ -581,7 +581,7 @@ static void xemaclite_tx_handler(struct net_device *dev)
                return;
 
        dev->stats.tx_bytes += lp->deferred_skb->len;
-       dev_kfree_skb_irq(lp->deferred_skb);
+       dev_consume_skb_irq(lp->deferred_skb);
        lp->deferred_skb = NULL;
        netif_trans_update(dev); /* prevent tx timeout */
        netif_wake_queue(dev);