From: Yang Yingliang <yangyingliang@huawei.com>
Date: Wed, 7 Dec 2022 15:04:53 +0000 (+0800)
Subject: wifi: wl3501_cs: don't call kfree_skb() under spin_lock_irqsave()
X-Git-Tag: v6.6.17~5509^2~227^2~54
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=44bacbdf9066c590423259dbd6d520baac99c1a8;p=platform%2Fkernel%2Flinux-rpi.git

wifi: wl3501_cs: don't call kfree_skb() under spin_lock_irqsave()

It is not allowed to call kfree_skb() from hardware interrupt
context or with interrupts being disabled. So replace kfree_skb()
with dev_kfree_skb_irq() under spin_lock_irqsave(). Compile
tested only.

Fixes: 1da177e4c3f4 ("Linux-2.6.12-rc2")
Signed-off-by: Yang Yingliang <yangyingliang@huawei.com>
Signed-off-by: Kalle Valo <kvalo@kernel.org>
Link: https://lore.kernel.org/r/20221207150453.114742-1-yangyingliang@huawei.com
---

diff --git a/drivers/net/wireless/wl3501_cs.c b/drivers/net/wireless/wl3501_cs.c
index 1b532e0..7fb2f95 100644
--- a/drivers/net/wireless/wl3501_cs.c
+++ b/drivers/net/wireless/wl3501_cs.c
@@ -1328,7 +1328,7 @@ static netdev_tx_t wl3501_hard_start_xmit(struct sk_buff *skb,
 	} else {
 		++dev->stats.tx_packets;
 		dev->stats.tx_bytes += skb->len;
-		kfree_skb(skb);
+		dev_kfree_skb_irq(skb);
 
 		if (this->tx_buffer_cnt < 2)
 			netif_stop_queue(dev);