8139cp: Use dev_kfree_skb_any() instead of dev_kfree_skb() in cp_clean_rings()
authorDavid Woodhouse <dwmw2@infradead.org>
Thu, 17 Sep 2015 23:19:08 +0000 (00:19 +0100)
committerDavid S. Miller <davem@davemloft.net>
Mon, 21 Sep 2015 05:23:40 +0000 (22:23 -0700)
This can be called from cp_tx_timeout() with interrupts disabled.
Spotted by Francois Romieu <romieu@fr.zoreil.com>

Signed-off-by: David Woodhouse <David.Woodhouse@intel.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/realtek/8139cp.c

index d79e33b3c1913ae41caa046860fc74fb2d4c8698..52a533445f30b7e434f82457f801963105496ce2 100644 (file)
@@ -1151,7 +1151,7 @@ static void cp_clean_rings (struct cp_private *cp)
                        desc = cp->rx_ring + i;
                        dma_unmap_single(&cp->pdev->dev,le64_to_cpu(desc->addr),
                                         cp->rx_buf_sz, PCI_DMA_FROMDEVICE);
-                       dev_kfree_skb(cp->rx_skb[i]);
+                       dev_kfree_skb_any(cp->rx_skb[i]);
                }
        }
 
@@ -1164,7 +1164,7 @@ static void cp_clean_rings (struct cp_private *cp)
                                         le32_to_cpu(desc->opts1) & 0xffff,
                                         PCI_DMA_TODEVICE);
                        if (le32_to_cpu(desc->opts1) & LastFrag)
-                               dev_kfree_skb(skb);
+                               dev_kfree_skb_any(skb);
                        cp->dev->stats.tx_dropped++;
                }
        }