* [PATCH net 1/1] atlantic: Fix driver resume flow.
@ 2021-08-27 11:52 Sudarsana Reddy Kalluru
2021-08-28 0:23 ` Jakub Kicinski
2021-08-29 9:50 ` patchwork-bot+netdevbpf
0 siblings, 2 replies; 3+ messages in thread
From: Sudarsana Reddy Kalluru @ 2021-08-27 11:52 UTC (permalink / raw)
To: davem; +Cc: netdev, irusskikh
Driver crashes when restoring from the Hibernate. In the resume flow,
driver need to clean up the older nic/vec objects and re-initialize them.
Fixes: 8aaa112a57c1d ("net: atlantic: refactoring pm logic")
Signed-off-by: Sudarsana Reddy Kalluru <skalluru@marvell.com>
Signed-off-by: Igor Russkikh <irusskikh@marvell.com>
---
drivers/net/ethernet/aquantia/atlantic/aq_pci_func.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/drivers/net/ethernet/aquantia/atlantic/aq_pci_func.c b/drivers/net/ethernet/aquantia/atlantic/aq_pci_func.c
index 59253846e885..f26d03735619 100644
--- a/drivers/net/ethernet/aquantia/atlantic/aq_pci_func.c
+++ b/drivers/net/ethernet/aquantia/atlantic/aq_pci_func.c
@@ -417,6 +417,9 @@ static int atl_resume_common(struct device *dev, bool deep)
pci_restore_state(pdev);
if (deep) {
+ /* Reinitialize Nic/Vecs objects */
+ aq_nic_deinit(nic, !nic->aq_hw->aq_nic_cfg->wol);
+
ret = aq_nic_init(nic);
if (ret)
goto err_exit;
--
2.27.0
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH net 1/1] atlantic: Fix driver resume flow.
2021-08-27 11:52 [PATCH net 1/1] atlantic: Fix driver resume flow Sudarsana Reddy Kalluru
@ 2021-08-28 0:23 ` Jakub Kicinski
2021-08-29 9:50 ` patchwork-bot+netdevbpf
1 sibling, 0 replies; 3+ messages in thread
From: Jakub Kicinski @ 2021-08-28 0:23 UTC (permalink / raw)
To: Sudarsana Reddy Kalluru; +Cc: davem, netdev, irusskikh
On Fri, 27 Aug 2021 04:52:25 -0700 Sudarsana Reddy Kalluru wrote:
> Driver crashes when restoring from the Hibernate. In the resume flow,
> driver need to clean up the older nic/vec objects and re-initialize them.
>
> Fixes: 8aaa112a57c1d ("net: atlantic: refactoring pm logic")
> Signed-off-by: Sudarsana Reddy Kalluru <skalluru@marvell.com>
> Signed-off-by: Igor Russkikh <irusskikh@marvell.com>
> ---
> drivers/net/ethernet/aquantia/atlantic/aq_pci_func.c | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/drivers/net/ethernet/aquantia/atlantic/aq_pci_func.c b/drivers/net/ethernet/aquantia/atlantic/aq_pci_func.c
> index 59253846e885..f26d03735619 100644
> --- a/drivers/net/ethernet/aquantia/atlantic/aq_pci_func.c
> +++ b/drivers/net/ethernet/aquantia/atlantic/aq_pci_func.c
> @@ -417,6 +417,9 @@ static int atl_resume_common(struct device *dev, bool deep)
> pci_restore_state(pdev);
>
> if (deep) {
> + /* Reinitialize Nic/Vecs objects */
> + aq_nic_deinit(nic, !nic->aq_hw->aq_nic_cfg->wol);
Why does the deinit() happen on resume, not suspend?
> ret = aq_nic_init(nic);
> if (ret)
> goto err_exit;
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH net 1/1] atlantic: Fix driver resume flow.
2021-08-27 11:52 [PATCH net 1/1] atlantic: Fix driver resume flow Sudarsana Reddy Kalluru
2021-08-28 0:23 ` Jakub Kicinski
@ 2021-08-29 9:50 ` patchwork-bot+netdevbpf
1 sibling, 0 replies; 3+ messages in thread
From: patchwork-bot+netdevbpf @ 2021-08-29 9:50 UTC (permalink / raw)
To: Sudarsana Reddy Kalluru; +Cc: davem, netdev, irusskikh
Hello:
This patch was applied to netdev/net.git (refs/heads/master):
On Fri, 27 Aug 2021 04:52:25 -0700 you wrote:
> Driver crashes when restoring from the Hibernate. In the resume flow,
> driver need to clean up the older nic/vec objects and re-initialize them.
>
> Fixes: 8aaa112a57c1d ("net: atlantic: refactoring pm logic")
> Signed-off-by: Sudarsana Reddy Kalluru <skalluru@marvell.com>
> Signed-off-by: Igor Russkikh <irusskikh@marvell.com>
>
> [...]
Here is the summary with links:
- [net,1/1] atlantic: Fix driver resume flow.
https://git.kernel.org/netdev/net/c/57f780f1c433
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] 3+ messages in thread
end of thread, other threads:[~2021-08-29 9:50 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-08-27 11:52 [PATCH net 1/1] atlantic: Fix driver resume flow Sudarsana Reddy Kalluru
2021-08-28 0:23 ` Jakub Kicinski
2021-08-29 9:50 ` patchwork-bot+netdevbpf
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.