ibmveth: Call dev_consume_skb_any instead of dev_kfree_skb.
authorEric W. Biederman <ebiederm@xmission.com>
Sun, 16 Mar 2014 00:29:34 +0000 (17:29 -0700)
committerEric W. Biederman <ebiederm@xmission.com>
Tue, 25 Mar 2014 04:19:03 +0000 (21:19 -0700)
Replace dev_kfree_skb with dev_consume_skb_any in ibmveth_start_xmit
that can be called in hard irq and other contexts.

In this code path the packet can have either been transmitted
or dropped, dev_consume_skb_any was choosen because that preserves
the existing semantics of the code, and a transmitted packet is
more likely.

Signed-off-by: "Eric W. Biederman" <ebiederm@xmission.com>
drivers/net/ethernet/ibm/ibmveth.c

index e75bdfc..c912756 100644 (file)
@@ -1044,7 +1044,7 @@ retry_bounce:
                               DMA_TO_DEVICE);
 
 out:
-       dev_kfree_skb(skb);
+       dev_consume_skb_any(skb);
        return NETDEV_TX_OK;
 
 map_failed_frags: