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 _tx_reclaim_skb that > can be called in hard irq and other contexts. > > dev_consume_skb is used as _tx_reclaim_skb is called after a packet > has been successfully transmitted. > > Signed-off-by: "Eric W. Biederman" <ebiederm@xxxxxxxxxxxx> > --- > drivers/net/ethernet/adi/bfin_mac.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/net/ethernet/adi/bfin_mac.c b/drivers/net/ethernet/adi/bfin_mac.c > index 83a8cdbcd936..95779b6b7394 100644 > --- a/drivers/net/ethernet/adi/bfin_mac.c > +++ b/drivers/net/ethernet/adi/bfin_mac.c > @@ -1087,7 +1087,7 @@ static inline void _tx_reclaim_skb(void) > tx_list_head->desc_a.config &= ~DMAEN; > tx_list_head->status.status_word = 0; > if (tx_list_head->skb) { > - dev_kfree_skb(tx_list_head->skb); > + dev_consume_skb_any(tx_list_head->skb); > tx_list_head->skb = NULL; > } > tx_list_head = tx_list_head->next; Acked-by: Eric Dumazet <edumazet@xxxxxxxxxx> Note this driver has a race in tx_reclaim_skb_timeout(), calling tx_reclaim_skb() without any lock (under timer interrupt, thats all) -- 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