aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorYang Wei <yang.wei9@zte.com.cn>2019-02-13 10:14:54 -0500
committerDavid S. Miller <davem@davemloft.net>2019-02-14 11:56:36 -0500
commitfc67ade130382e21a43eca81c5ed098d8a82eedd (patch)
tree6cf07537035a20c3309d6baebfac2c2d3be75a8f
parent76cba8fd9c7d8dd2d18c8bee581c4176383441c6 (diff)
net: amd: replace dev_kfree_skb_irq by dev_consume_skb_irq for drop profiles
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>
-rw-r--r--drivers/net/ethernet/amd/lance.c2
-rw-r--r--drivers/net/ethernet/amd/ni65.c2
2 files changed, 2 insertions, 2 deletions
diff --git a/drivers/net/ethernet/amd/lance.c b/drivers/net/ethernet/amd/lance.c
index b56d84c7df46..f90b454b1642 100644
--- a/drivers/net/ethernet/amd/lance.c
+++ b/drivers/net/ethernet/amd/lance.c
@@ -1084,7 +1084,7 @@ static irqreturn_t lance_interrupt(int irq, void *dev_id)
1084 /* We must free the original skb if it's not a data-only copy 1084 /* We must free the original skb if it's not a data-only copy
1085 in the bounce buffer. */ 1085 in the bounce buffer. */
1086 if (lp->tx_skbuff[entry]) { 1086 if (lp->tx_skbuff[entry]) {
1087 dev_kfree_skb_irq(lp->tx_skbuff[entry]); 1087 dev_consume_skb_irq(lp->tx_skbuff[entry]);
1088 lp->tx_skbuff[entry] = NULL; 1088 lp->tx_skbuff[entry] = NULL;
1089 } 1089 }
1090 dirty_tx++; 1090 dirty_tx++;
diff --git a/drivers/net/ethernet/amd/ni65.c b/drivers/net/ethernet/amd/ni65.c
index 8931ce6bab7b..87ff5d6d1b22 100644
--- a/drivers/net/ethernet/amd/ni65.c
+++ b/drivers/net/ethernet/amd/ni65.c
@@ -1028,7 +1028,7 @@ static void ni65_xmit_intr(struct net_device *dev,int csr0)
1028 1028
1029#ifdef XMT_VIA_SKB 1029#ifdef XMT_VIA_SKB
1030 if(p->tmd_skb[p->tmdlast]) { 1030 if(p->tmd_skb[p->tmdlast]) {
1031 dev_kfree_skb_irq(p->tmd_skb[p->tmdlast]); 1031 dev_consume_skb_irq(p->tmd_skb[p->tmdlast]);
1032 p->tmd_skb[p->tmdlast] = NULL; 1032 p->tmd_skb[p->tmdlast] = NULL;
1033 } 1033 }
1034#endif 1034#endif