aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorRick Jones <rick.jones2@hp.com>2014-09-12 13:44:06 -0400
committerJeff Kirsher <jeffrey.t.kirsher@intel.com>2014-10-02 05:04:46 -0400
commitfe1f2a972c8c8d22ac234024b331130ddddf8682 (patch)
tree274dc85dfd7e11a803b2659210d70ed19cebcef4
parentc9d49940849ff2f11f3646bc8eb1f23ed60b505d (diff)
ixgbe: Convert the normal transmit complete path to dev_consume_skb_any()
Convert the normal packet completion path to dev_consume_skb_any() so packet drop profiling via dropwatch or perf top -G -e skb_kfree_skb is not cluttered with false hits. Compile tested only. There is a dev_kfree_skb_any() in the routine ixgbe_ptp_tx_hwtstamp() in ixgbe_ptp.c that looks like a conversion candidate but I wasn't familiar enough with the code to pull the trigger. Signed-off-by: Rick Jones <rick.jones2@hp.com> Tested-by: Phil Schmitt <phillip.j.schmitt@intel.com> Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
-rw-r--r--drivers/net/ethernet/intel/ixgbe/ixgbe_main.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/ethernet/intel/ixgbe/ixgbe_main.c b/drivers/net/ethernet/intel/ixgbe/ixgbe_main.c
index 06ef5a32a893..2daf257dea7c 100644
--- a/drivers/net/ethernet/intel/ixgbe/ixgbe_main.c
+++ b/drivers/net/ethernet/intel/ixgbe/ixgbe_main.c
@@ -1094,7 +1094,7 @@ static bool ixgbe_clean_tx_irq(struct ixgbe_q_vector *q_vector,
1094 total_packets += tx_buffer->gso_segs; 1094 total_packets += tx_buffer->gso_segs;
1095 1095
1096 /* free the skb */ 1096 /* free the skb */
1097 dev_kfree_skb_any(tx_buffer->skb); 1097 dev_consume_skb_any(tx_buffer->skb);
1098 1098
1099 /* unmap skb header data */ 1099 /* unmap skb header data */
1100 dma_unmap_single(tx_ring->dev, 1100 dma_unmap_single(tx_ring->dev,