[PATCH 06/11] e1000: Call dev_kfree_skby_any instead of dev_kfree_skb.
From: Eric W. Biederman <hidden>
Date: 2014-03-11 03:21:36
Subsystem:
intel ethernet drivers, networking drivers, the rest · Maintainers:
Tony Nguyen, Przemek Kitszel, Andrew Lunn, "David S. Miller", Eric Dumazet, Jakub Kicinski, Paolo Abeni, Linus Torvalds
Replace dev_kfree_skb with dev_kfree_skb_any in functions that can be called in hard irq and other contexts. Signed-off-by: "Eric W. Biederman" <redacted> --- drivers/net/ethernet/intel/e1000/e1000_main.c | 18 +++++++++--------- 1 files changed, 9 insertions(+), 9 deletions(-)
diff --git a/drivers/net/ethernet/intel/e1000/e1000_main.c b/drivers/net/ethernet/intel/e1000/e1000_main.c
index 46e6544ed1b7..64036d3ef62b 100644
--- a/drivers/net/ethernet/intel/e1000/e1000_main.c
+++ b/drivers/net/ethernet/intel/e1000/e1000_main.c@@ -4066,7 +4066,7 @@ static bool e1000_clean_jumbo_rx_irq(struct e1000_adapter *adapter, * too */ if (rx_ring->rx_skb_top) - dev_kfree_skb(rx_ring->rx_skb_top); + dev_kfree_skb_any(rx_ring->rx_skb_top); rx_ring->rx_skb_top = NULL; goto next_desc; }
@@ -4143,7 +4143,7 @@ process_skb: /* eth type trans needs skb->data to point to something */ if (!pskb_may_pull(skb, ETH_HLEN)) { e_err(drv, "pskb_may_pull failed.\n"); - dev_kfree_skb(skb); + dev_kfree_skb_any(skb); goto next_desc; }
@@ -4394,7 +4394,7 @@ check_page: DMA_FROM_DEVICE); if (dma_mapping_error(&pdev->dev, buffer_info->dma)) { put_page(buffer_info->page); - dev_kfree_skb(skb); + dev_kfree_skb_any(skb); buffer_info->page = NULL; buffer_info->skb = NULL; buffer_info->dma = 0;
@@ -4469,21 +4469,21 @@ static void e1000_alloc_rx_buffers(struct e1000_adapter *adapter, skb = netdev_alloc_skb_ip_align(netdev, bufsz); /* Failed allocation, critical failure */ if (!skb) { - dev_kfree_skb(oldskb); + dev_kfree_skb_any(oldskb); adapter->alloc_rx_buff_failed++; break; } if (!e1000_check_64k_bound(adapter, skb->data, bufsz)) { /* give up */ - dev_kfree_skb(skb); - dev_kfree_skb(oldskb); + dev_kfree_skb_any(skb); + dev_kfree_skb_any(oldskb); adapter->alloc_rx_buff_failed++; break; /* while !buffer_info->skb */ } /* Use new allocation */ - dev_kfree_skb(oldskb); + dev_kfree_skb_any(oldskb); } buffer_info->skb = skb; buffer_info->length = adapter->rx_buffer_len;
@@ -4493,7 +4493,7 @@ map_skb: buffer_info->length, DMA_FROM_DEVICE); if (dma_mapping_error(&pdev->dev, buffer_info->dma)) { - dev_kfree_skb(skb); + dev_kfree_skb_any(skb); buffer_info->skb = NULL; buffer_info->dma = 0; adapter->alloc_rx_buff_failed++;
@@ -4511,7 +4511,7 @@ map_skb: e_err(rx_err, "dma align check failed: %u bytes at " "%p\n", adapter->rx_buffer_len, (void *)(unsigned long)buffer_info->dma); - dev_kfree_skb(skb); + dev_kfree_skb_any(skb); buffer_info->skb = NULL; dma_unmap_single(&pdev->dev, buffer_info->dma,
--
1.7.5.4