|
Jiri Slaby |
ef7db2 |
From: Yang Yingliang <yangyingliang@huawei.com>
|
|
Jiri Slaby |
ef7db2 |
Date: Wed, 7 Dec 2022 23:00:06 +0800
|
|
Jiri Slaby |
ef7db2 |
Subject: [PATCH] wifi: libertas: if_usb: don't call kfree_skb() under
|
|
Jiri Slaby |
ef7db2 |
spin_lock_irqsave()
|
|
Jiri Slaby |
ef7db2 |
References: bsc#1012628
|
|
Jiri Slaby |
ef7db2 |
Patch-mainline: 6.2.3
|
|
Jiri Slaby |
ef7db2 |
Git-commit: 3968e81ba644f10a7d45bae2539560db9edac501
|
|
Jiri Slaby |
ef7db2 |
|
|
Jiri Slaby |
ef7db2 |
[ Upstream commit 3968e81ba644f10a7d45bae2539560db9edac501 ]
|
|
Jiri Slaby |
ef7db2 |
|
|
Jiri Slaby |
ef7db2 |
It is not allowed to call kfree_skb() from hardware interrupt
|
|
Jiri Slaby |
ef7db2 |
context or with interrupts being disabled. So replace kfree_skb()
|
|
Jiri Slaby |
ef7db2 |
with dev_kfree_skb_irq() under spin_lock_irqsave(). Compile
|
|
Jiri Slaby |
ef7db2 |
tested only.
|
|
Jiri Slaby |
ef7db2 |
|
|
Jiri Slaby |
ef7db2 |
Fixes: a3128feef6d5 ("libertas: use irqsave() in USB's complete callback")
|
|
Jiri Slaby |
ef7db2 |
Signed-off-by: Yang Yingliang <yangyingliang@huawei.com>
|
|
Jiri Slaby |
ef7db2 |
Signed-off-by: Kalle Valo <kvalo@kernel.org>
|
|
Jiri Slaby |
ef7db2 |
Link: https://lore.kernel.org/r/20221207150008.111743-3-yangyingliang@huawei.com
|
|
Jiri Slaby |
ef7db2 |
Signed-off-by: Sasha Levin <sashal@kernel.org>
|
|
Jiri Slaby |
ef7db2 |
Signed-off-by: Jiri Slaby <jslaby@suse.cz>
|
|
Jiri Slaby |
ef7db2 |
---
|
|
Jiri Slaby |
ef7db2 |
drivers/net/wireless/marvell/libertas/if_usb.c | 2 +-
|
|
Jiri Slaby |
ef7db2 |
1 file changed, 1 insertion(+), 1 deletion(-)
|
|
Jiri Slaby |
ef7db2 |
|
|
Jiri Slaby |
ef7db2 |
diff --git a/drivers/net/wireless/marvell/libertas/if_usb.c b/drivers/net/wireless/marvell/libertas/if_usb.c
|
|
Jiri Slaby |
ef7db2 |
index 32fdc415..2240b4db 100644
|
|
Jiri Slaby |
ef7db2 |
--- a/drivers/net/wireless/marvell/libertas/if_usb.c
|
|
Jiri Slaby |
ef7db2 |
+++ b/drivers/net/wireless/marvell/libertas/if_usb.c
|
|
Jiri Slaby |
ef7db2 |
@@ -637,7 +637,7 @@ static inline void process_cmdrequest(int recvlength, uint8_t *recvbuff,
|
|
Jiri Slaby |
ef7db2 |
priv->resp_len[i] = (recvlength - MESSAGE_HEADER_LEN);
|
|
Jiri Slaby |
ef7db2 |
memcpy(priv->resp_buf[i], recvbuff + MESSAGE_HEADER_LEN,
|
|
Jiri Slaby |
ef7db2 |
priv->resp_len[i]);
|
|
Jiri Slaby |
ef7db2 |
- kfree_skb(skb);
|
|
Jiri Slaby |
ef7db2 |
+ dev_kfree_skb_irq(skb);
|
|
Jiri Slaby |
ef7db2 |
lbs_notify_command_response(priv, i);
|
|
Jiri Slaby |
ef7db2 |
|
|
Jiri Slaby |
ef7db2 |
spin_unlock_irqrestore(&priv->driver_lock, flags);
|
|
Jiri Slaby |
ef7db2 |
--
|
|
Jiri Slaby |
ef7db2 |
2.35.3
|
|
Jiri Slaby |
ef7db2 |
|