diff options
author | Yang Wei <yang.wei9@zte.com.cn> | 2019-02-13 10:14:54 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2019-02-14 11:56:36 -0500 |
commit | fc67ade130382e21a43eca81c5ed098d8a82eedd (patch) | |
tree | 6cf07537035a20c3309d6baebfac2c2d3be75a8f | |
parent | 76cba8fd9c7d8dd2d18c8bee581c4176383441c6 (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.c | 2 | ||||
-rw-r--r-- | drivers/net/ethernet/amd/ni65.c | 2 |
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 |