net: atheros: replace dev_kfree_skb_irq by dev_consume_skb_irq for drop profiles
authorYang Wei <yang.wei9@zte.com.cn>
Tue, 12 Feb 2019 15:51:45 +0000 (23:51 +0800)
committerDavid S. Miller <davem@davemloft.net>
Thu, 14 Feb 2019 04:50:42 +0000 (20:50 -0800)
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/atheros/atl1e/atl1e_main.c
drivers/net/ethernet/atheros/atlx/atl1.c

index 3164aad29bcf879aa0841a912d6bbeaa757caad6..9dfe6a9431e6ee485f57b4ed2addc0baa9d7baa7 100644 (file)
@@ -1259,7 +1259,7 @@ static bool atl1e_clean_tx_irq(struct atl1e_adapter *adapter)
                }
 
                if (tx_buffer->skb) {
-                       dev_kfree_skb_irq(tx_buffer->skb);
+                       dev_consume_skb_irq(tx_buffer->skb);
                        tx_buffer->skb = NULL;
                }
 
index 63edc5706c096338d5dd16422f18d9652ea33e0c..9e07b469066a4bbc41119e0801d716a25dce9a55 100644 (file)
@@ -2088,7 +2088,7 @@ static int atl1_intr_tx(struct atl1_adapter *adapter)
                }
 
                if (buffer_info->skb) {
-                       dev_kfree_skb_irq(buffer_info->skb);
+                       dev_consume_skb_irq(buffer_info->skb);
                        buffer_info->skb = NULL;
                }