On Mon, 2014-03-24 at 23:04 -0700, Eric W. Biederman wrote: > From: "Eric W. Biederman" <ebiederm@xxxxxxxxxxxx> > > Replace dev_kfree_skb with dev_consume_skb_any in vortex_start_xmit > as it can be called in hard irq and other contexts. > > dev_consume_skb_any is used when vortext_start_xmit directly consumes > the packet instead of dmaing it to the device. > > Signed-off-by: "Eric W. Biederman" <ebiederm@xxxxxxxxxxxx> > --- > drivers/net/ethernet/3com/3c59x.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/net/ethernet/3com/3c59x.c b/drivers/net/ethernet/3com/3c59x.c > index 238ccea965c8..61477b8e8d24 100644 > --- a/drivers/net/ethernet/3com/3c59x.c > +++ b/drivers/net/ethernet/3com/3c59x.c > @@ -2086,7 +2086,7 @@ vortex_start_xmit(struct sk_buff *skb, struct net_device *dev) > /* ... and the packet rounded to a doubleword. */ > skb_tx_timestamp(skb); > iowrite32_rep(ioaddr + TX_FIFO, skb->data, (skb->len + 3) >> 2); > - dev_kfree_skb (skb); > + dev_consume_skb_any (skb); > if (ioread16(ioaddr + TxFree) > 1536) { > netif_start_queue (dev); /* AKPM: redundant? */ > } else { remove the extra space ? Acked-by: Eric Dumazet <edumazet@xxxxxxxxxx> -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html