Commit 8c608a32 authored by Roland Dreier's avatar Roland Dreier

[IPoIB] no need to set skb->dev right before freeing skb

For cut-and-paste reasons, the IPoIB driver was setting skb->dev right
before calling dev_kfree_skb_any().  Get rid of this.
Signed-off-by: default avatarRoland Dreier <rolandd@cisco.com>
parent 2f76e829
...@@ -120,12 +120,8 @@ static void ipoib_mcast_free(struct ipoib_mcast *mcast) ...@@ -120,12 +120,8 @@ static void ipoib_mcast_free(struct ipoib_mcast *mcast)
if (mcast->ah) if (mcast->ah)
ipoib_put_ah(mcast->ah); ipoib_put_ah(mcast->ah);
while (!skb_queue_empty(&mcast->pkt_queue)) { while (!skb_queue_empty(&mcast->pkt_queue))
struct sk_buff *skb = skb_dequeue(&mcast->pkt_queue); dev_kfree_skb_any(skb_dequeue(&mcast->pkt_queue));
skb->dev = dev;
dev_kfree_skb_any(skb);
}
kfree(mcast); kfree(mcast);
} }
...@@ -317,13 +313,8 @@ ipoib_mcast_sendonly_join_complete(int status, ...@@ -317,13 +313,8 @@ ipoib_mcast_sendonly_join_complete(int status,
IPOIB_GID_ARG(mcast->mcmember.mgid), status); IPOIB_GID_ARG(mcast->mcmember.mgid), status);
/* Flush out any queued packets */ /* Flush out any queued packets */
while (!skb_queue_empty(&mcast->pkt_queue)) { while (!skb_queue_empty(&mcast->pkt_queue))
struct sk_buff *skb = skb_dequeue(&mcast->pkt_queue); dev_kfree_skb_any(skb_dequeue(&mcast->pkt_queue));
skb->dev = dev;
dev_kfree_skb_any(skb);
}
/* Clear the busy flag so we try again */ /* Clear the busy flag so we try again */
clear_bit(IPOIB_MCAST_FLAG_BUSY, &mcast->flags); clear_bit(IPOIB_MCAST_FLAG_BUSY, &mcast->flags);
......
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