Skip to content

Commit

Permalink
wlags49_h2: Call dev_kfree/consume_skb_any instead of dev_kfree_skb.
Browse files Browse the repository at this point in the history
Replace dev_kfree_skb with dev_consume_skb_any in wl_send and
wl_send_dma which can be called in hard irq and other contexts,
on the code paths where the skb was transmitted successfully.

Replace dev_kfree_skb with dev_kfree_skb_any in wl_send_dmay which can
be called in hard irq and other contexts, on the code path where a
skb is dropped.

Signed-off-by: "Eric W. Biederman" <[email protected]>
  • Loading branch information
ebiederm committed Mar 25, 2014
1 parent 979de8a commit 8d4ade2
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions drivers/staging/wlags49_h2/wl_netdev.c
Original file line number Diff line number Diff line change
Expand Up @@ -715,7 +715,7 @@ int wl_send( struct wl_private *lp )

/* Free the skb and perform queue cleanup, as the buffer was
transmitted successfully */
dev_kfree_skb( lp->txF.skb );
dev_consume_skb_any( lp->txF.skb );

lp->txF.skb = NULL;
lp->txF.port = 0;
Expand Down Expand Up @@ -1730,7 +1730,7 @@ int wl_send_dma( struct wl_private *lp, struct sk_buff *skb, int port )
WL_WDS_NETIF_STOP_QUEUE( lp );
lp->netif_queue_on = FALSE;

dev_kfree_skb( skb );
dev_kfree_skb_any( skb );
return 0;
}
}
Expand All @@ -1755,7 +1755,7 @@ int wl_send_dma( struct wl_private *lp, struct sk_buff *skb, int port )

/* Free the skb and perform queue cleanup, as the buffer was
transmitted successfully */
dev_kfree_skb( skb );
dev_consume_skb_any( skb );

return TRUE;
} // wl_send_dma
Expand Down

0 comments on commit 8d4ade2

Please sign in to comment.