* [PATCH] drivers: net: xgene: disable napi when register irq failed in xgene_enet_open()
@ 2022-11-07 4:30 Zhengchao Shao
2022-11-08 14:30 ` patchwork-bot+netdevbpf
0 siblings, 1 reply; 2+ messages in thread
From: Zhengchao Shao @ 2022-11-07 4:30 UTC (permalink / raw)
To: netdev, iyappan, keyur, quan, davem, edumazet, kuba, pabeni
Cc: weiyongjun1, yuehaibing, shaozhengchao
When failed to register irq in xgene_enet_open() for opening device,
napi isn't disabled. When open xgene device next time, it will reports
a invalid opcode issue. Fix it. Only be compiled, not be tested.
Fixes: aeb20b6b3f4e ("drivers: net: xgene: fix: ifconfig up/down crash")
Signed-off-by: Zhengchao Shao <shaozhengchao@huawei.com>
---
drivers/net/ethernet/apm/xgene/xgene_enet_main.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/drivers/net/ethernet/apm/xgene/xgene_enet_main.c b/drivers/net/ethernet/apm/xgene/xgene_enet_main.c
index d6cfea65a714..390671640388 100644
--- a/drivers/net/ethernet/apm/xgene/xgene_enet_main.c
+++ b/drivers/net/ethernet/apm/xgene/xgene_enet_main.c
@@ -1004,8 +1004,10 @@ static int xgene_enet_open(struct net_device *ndev)
xgene_enet_napi_enable(pdata);
ret = xgene_enet_register_irq(ndev);
- if (ret)
+ if (ret) {
+ xgene_enet_napi_disable(pdata);
return ret;
+ }
if (ndev->phydev) {
phy_start(ndev->phydev);
--
2.17.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] drivers: net: xgene: disable napi when register irq failed in xgene_enet_open()
2022-11-07 4:30 [PATCH] drivers: net: xgene: disable napi when register irq failed in xgene_enet_open() Zhengchao Shao
@ 2022-11-08 14:30 ` patchwork-bot+netdevbpf
0 siblings, 0 replies; 2+ messages in thread
From: patchwork-bot+netdevbpf @ 2022-11-08 14:30 UTC (permalink / raw)
To: Zhengchao Shao
Cc: netdev, iyappan, keyur, quan, davem, edumazet, kuba, pabeni,
weiyongjun1, yuehaibing
Hello:
This patch was applied to netdev/net.git (master)
by Paolo Abeni <pabeni@redhat.com>:
On Mon, 7 Nov 2022 12:30:32 +0800 you wrote:
> When failed to register irq in xgene_enet_open() for opening device,
> napi isn't disabled. When open xgene device next time, it will reports
> a invalid opcode issue. Fix it. Only be compiled, not be tested.
>
> Fixes: aeb20b6b3f4e ("drivers: net: xgene: fix: ifconfig up/down crash")
> Signed-off-by: Zhengchao Shao <shaozhengchao@huawei.com>
>
> [...]
Here is the summary with links:
- drivers: net: xgene: disable napi when register irq failed in xgene_enet_open()
https://git.kernel.org/netdev/net/c/ce9e57feeed8
You are awesome, thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2022-11-08 14:31 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2022-11-07 4:30 [PATCH] drivers: net: xgene: disable napi when register irq failed in xgene_enet_open() Zhengchao Shao
2022-11-08 14:30 ` patchwork-bot+netdevbpf
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).