Commit 0113e34b authored by Eric Dumazet's avatar Eric Dumazet Committed by David S. Miller

net: ftgmac100: use kfree_skb() where appropriate

In order to get correct drop monitor notifications for dropped
packets, we should call kfree_skb() instead of dev_kfree_skb()
Signed-off-by: default avatarEric Dumazet <edumazet@google.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent ff1a9d35
...@@ -766,7 +766,7 @@ static void ftgmac100_free_buffers(struct ftgmac100 *priv) ...@@ -766,7 +766,7 @@ static void ftgmac100_free_buffers(struct ftgmac100 *priv)
continue; continue;
dma_unmap_single(priv->dev, map, skb_headlen(skb), DMA_TO_DEVICE); dma_unmap_single(priv->dev, map, skb_headlen(skb), DMA_TO_DEVICE);
dev_kfree_skb(skb); kfree_skb(skb);
} }
dma_free_coherent(priv->dev, sizeof(struct ftgmac100_descs), dma_free_coherent(priv->dev, sizeof(struct ftgmac100_descs),
...@@ -1148,7 +1148,7 @@ static int ftgmac100_hard_start_xmit(struct sk_buff *skb, ...@@ -1148,7 +1148,7 @@ static int ftgmac100_hard_start_xmit(struct sk_buff *skb,
netdev_dbg(netdev, "tx packet too big\n"); netdev_dbg(netdev, "tx packet too big\n");
netdev->stats.tx_dropped++; netdev->stats.tx_dropped++;
dev_kfree_skb(skb); kfree_skb(skb);
return NETDEV_TX_OK; return NETDEV_TX_OK;
} }
...@@ -1159,7 +1159,7 @@ static int ftgmac100_hard_start_xmit(struct sk_buff *skb, ...@@ -1159,7 +1159,7 @@ static int ftgmac100_hard_start_xmit(struct sk_buff *skb,
netdev_err(netdev, "map socket buffer failed\n"); netdev_err(netdev, "map socket buffer failed\n");
netdev->stats.tx_dropped++; netdev->stats.tx_dropped++;
dev_kfree_skb(skb); kfree_skb(skb);
return NETDEV_TX_OK; return NETDEV_TX_OK;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment