diff options
author | Wei Yongjun <weiyongjun1@huawei.com> | 2016-11-01 14:48:40 +0000 |
---|---|---|
committer | Kalle Valo <kvalo@codeaurora.org> | 2016-11-17 08:48:33 +0200 |
commit | e49656147359411fb7529a211e852e587e5970dd (patch) | |
tree | cf64cd1fcca84c1b452c522f0f15c50fe3394488 | |
parent | 7fc1503c906f0fac62d3506a6e993e49fb996248 (diff) | |
download | op-kernel-dev-e49656147359411fb7529a211e852e587e5970dd.zip op-kernel-dev-e49656147359411fb7529a211e852e587e5970dd.tar.gz |
rtlwifi: Use dev_kfree_skb_irq instead of kfree_skb
It is not allowed to call kfree_skb() from hardware interrupt
context or with interrupts being disabled, spin_lock_irqsave()
make sure always in irq disable context. So the kfree_skb()
should be replaced with dev_kfree_skb_irq().
This is detected by Coccinelle semantic patch.
Signed-off-by: Wei Yongjun <weiyongjun1@huawei.com>
Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
-rw-r--r-- | drivers/net/wireless/realtek/rtlwifi/core.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/wireless/realtek/rtlwifi/core.c b/drivers/net/wireless/realtek/rtlwifi/core.c index f95760c..a11e8f5 100644 --- a/drivers/net/wireless/realtek/rtlwifi/core.c +++ b/drivers/net/wireless/realtek/rtlwifi/core.c @@ -1832,7 +1832,7 @@ bool rtl_cmd_send_packet(struct ieee80211_hw *hw, struct sk_buff *skb) spin_lock_irqsave(&rtlpriv->locks.irq_th_lock, flags); pskb = __skb_dequeue(&ring->queue); - kfree_skb(pskb); + dev_kfree_skb_irq(pskb); /*this is wrong, fill_tx_cmddesc needs update*/ pdesc = &ring->desc[0]; |