* [PATCH] mt76: mt7921e: fix reset failure in mt7921_pci_probe
@ 2022-01-05 22:20 sean.wang
2022-01-07 0:53 ` Sean Wang
0 siblings, 1 reply; 2+ messages in thread
From: sean.wang @ 2022-01-05 22:20 UTC (permalink / raw)
To: nbd, lorenzo.bianconi
Cc: sean.wang, Soul.Huang, YN.Chen, Leon.Yen, Eric-SY.Chang,
Mark-YW.Chen, Deren.Wu, km.lin, jenhao.yang, robin.chiu,
Eddie.Chen, ch.yeh, posh.sun, ted.huang, Eric.Liang,
Stella.Chang, Tom.Chou, steve.lee, jsiuda, frankgor, jemele,
abhishekpandit, shawnku, linux-wireless, linux-mediatek,
Deren Wu
From: Deren Wu <deren.wu@mediatek.com>
Ensure that the device returns to the driver's own state to switch ASPM
into the active state before we start resetting the device to fix
the following kernel panic.
[ 17.164952] dump_stack_lvl+0x48/0x5e
[ 17.164955] bad_page.cold+0x63/0x94
[ 17.164957] free_pcppages_bulk+0x1f2/0x360
[ 17.164961] free_unref_page+0xb4/0x120
[ 17.164963] mt76_dma_rx_cleanup+0x94/0x110 [mt76]
[ 17.164969] mt7921_wpdma_reset+0xbc/0x1c0 [mt7921e]
[ 17.164972] mt7921_register_device+0x32b/0x5d0 [mt7921_common]
[ 17.164976] mt7921_pci_probe+0x1d5/0x210 [mt7921e]
[ 17.164977] ? __pm_runtime_resume+0x58/0x80
[ 17.164979] local_pci_probe+0x45/0x80
[ 17.164981] ? pci_match_device+0xac/0x130
[ 17.164983] pci_device_probe+0xd2/0x1c0
[ 17.164984] really_probe+0x1f5/0x3f0
[ 17.164987] __driver_probe_device+0xfe/0x180
[ 17.164988] driver_probe_device+0x1e/0x90
[ 17.164990] __driver_attach+0xc0/0x1c0
[ 17.164991] ? __device_attach_driver+0xe0/0xe0
[ 17.164992] ? __device_attach_driver+0xe0/0xe0
[ 17.164994] bus_for_each_dev+0x78/0xc0
[ 17.164995] new_id_store+0xfc/0x1d0
[ 17.164996] kernfs_fop_write_iter+0x11c/0x1b0
[ 17.164998] new_sync_write+0x11f/0x1b0
[ 17.165001] vfs_write+0x209/0x2a0
[ 17.165003] ksys_write+0x5f/0xe0
[ 17.165004] do_syscall_64+0x3b/0xc0
[ 17.165006]
Fixes: bf3747ae2e25 ("mt76: mt7921: enable aspm by default")
Reported-by: Kai-Chuan Hsieh <kaichuan.hsieh@canonical.com>
Co-developed-by: Sean Wang <sean.wang@mediatek.com>
Signed-off-by: Sean Wang <sean.wang@mediatek.com>
Signed-off-by: Deren Wu <deren.wu@mediatek.com>
---
.../net/wireless/mediatek/mt76/mt7921/dma.c | 4 ++++
.../net/wireless/mediatek/mt76/mt7921/mt7921.h | 1 +
.../wireless/mediatek/mt76/mt7921/pci_mcu.c | 18 ++++++++++++++----
3 files changed, 19 insertions(+), 4 deletions(-)
diff --git a/drivers/net/wireless/mediatek/mt76/mt7921/dma.c b/drivers/net/wireless/mediatek/mt76/mt7921/dma.c
index cdff1fd52d93..0c4b0e0e634a 100644
--- a/drivers/net/wireless/mediatek/mt76/mt7921/dma.c
+++ b/drivers/net/wireless/mediatek/mt76/mt7921/dma.c
@@ -364,6 +364,10 @@ int mt7921_dma_init(struct mt7921_dev *dev)
if (ret)
return ret;
+ ret = __mt7921e_mcu_drv_pmctrl(dev);
+ if (ret)
+ return ret;
+
ret = mt7921_wfsys_reset(dev);
if (ret)
return ret;
diff --git a/drivers/net/wireless/mediatek/mt76/mt7921/mt7921.h b/drivers/net/wireless/mediatek/mt76/mt7921/mt7921.h
index 8b674e042568..63e3c7ef5e89 100644
--- a/drivers/net/wireless/mediatek/mt76/mt7921/mt7921.h
+++ b/drivers/net/wireless/mediatek/mt76/mt7921/mt7921.h
@@ -443,6 +443,7 @@ int mt7921e_mcu_init(struct mt7921_dev *dev);
int mt7921s_wfsys_reset(struct mt7921_dev *dev);
int mt7921s_mac_reset(struct mt7921_dev *dev);
int mt7921s_init_reset(struct mt7921_dev *dev);
+int __mt7921e_mcu_drv_pmctrl(struct mt7921_dev *dev);
int mt7921e_mcu_drv_pmctrl(struct mt7921_dev *dev);
int mt7921e_mcu_fw_pmctrl(struct mt7921_dev *dev);
diff --git a/drivers/net/wireless/mediatek/mt76/mt7921/pci_mcu.c b/drivers/net/wireless/mediatek/mt76/mt7921/pci_mcu.c
index f9e350b67fdc..36669e5aeef3 100644
--- a/drivers/net/wireless/mediatek/mt76/mt7921/pci_mcu.c
+++ b/drivers/net/wireless/mediatek/mt76/mt7921/pci_mcu.c
@@ -59,10 +59,8 @@ int mt7921e_mcu_init(struct mt7921_dev *dev)
return err;
}
-int mt7921e_mcu_drv_pmctrl(struct mt7921_dev *dev)
+int __mt7921e_mcu_drv_pmctrl(struct mt7921_dev *dev)
{
- struct mt76_phy *mphy = &dev->mt76.phy;
- struct mt76_connac_pm *pm = &dev->pm;
int i, err = 0;
for (i = 0; i < MT7921_DRV_OWN_RETRY_COUNT; i++) {
@@ -75,9 +73,21 @@ int mt7921e_mcu_drv_pmctrl(struct mt7921_dev *dev)
if (i == MT7921_DRV_OWN_RETRY_COUNT) {
dev_err(dev->mt76.dev, "driver own failed\n");
err = -EIO;
- goto out;
}
+ return err;
+}
+
+int mt7921e_mcu_drv_pmctrl(struct mt7921_dev *dev)
+{
+ struct mt76_phy *mphy = &dev->mt76.phy;
+ struct mt76_connac_pm *pm = &dev->pm;
+ int err;
+
+ err = __mt7921e_mcu_drv_pmctrl(dev);
+ if (err < 0)
+ goto out;
+
mt7921_wpdma_reinit_cond(dev);
clear_bit(MT76_STATE_PM, &mphy->state);
--
2.25.1
_______________________________________________
Linux-mediatek mailing list
Linux-mediatek@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-mediatek
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] mt76: mt7921e: fix reset failure in mt7921_pci_probe
2022-01-05 22:20 [PATCH] mt76: mt7921e: fix reset failure in mt7921_pci_probe sean.wang
@ 2022-01-07 0:53 ` Sean Wang
0 siblings, 0 replies; 2+ messages in thread
From: Sean Wang @ 2022-01-07 0:53 UTC (permalink / raw)
To: nbd
Cc: lorenzo.bianconi, Sean Wang (王志亘),
Soul.Huang, YN.Chen, Leon.Yen, Eric-SY.Chang, Mark-YW.Chen,
Deren.Wu, km.lin, jenhao.yang, robin.chiu, Eddie.Chen, ch.yeh,
posh.sun, ted.huang, Eric.Liang, Stella.Chang, Tom.Chou,
steve.lee, jsiuda, frankgor, jemele, abhishekpandit, shawnku,
linux-wireless, moderated list:ARM/Mediatek SoC support
@nbd please drop the patch, I will post a new one to fix the aspm issue
On Thu, Jan 6, 2022 at 1:32 PM <sean.wang@mediatek.com> wrote:
>
> From: Deren Wu <deren.wu@mediatek.com>
>
> Ensure that the device returns to the driver's own state to switch ASPM
> into the active state before we start resetting the device to fix
> the following kernel panic.
>
> [ 17.164952] dump_stack_lvl+0x48/0x5e
> [ 17.164955] bad_page.cold+0x63/0x94
> [ 17.164957] free_pcppages_bulk+0x1f2/0x360
> [ 17.164961] free_unref_page+0xb4/0x120
> [ 17.164963] mt76_dma_rx_cleanup+0x94/0x110 [mt76]
> [ 17.164969] mt7921_wpdma_reset+0xbc/0x1c0 [mt7921e]
> [ 17.164972] mt7921_register_device+0x32b/0x5d0 [mt7921_common]
> [ 17.164976] mt7921_pci_probe+0x1d5/0x210 [mt7921e]
> [ 17.164977] ? __pm_runtime_resume+0x58/0x80
> [ 17.164979] local_pci_probe+0x45/0x80
> [ 17.164981] ? pci_match_device+0xac/0x130
> [ 17.164983] pci_device_probe+0xd2/0x1c0
> [ 17.164984] really_probe+0x1f5/0x3f0
> [ 17.164987] __driver_probe_device+0xfe/0x180
> [ 17.164988] driver_probe_device+0x1e/0x90
> [ 17.164990] __driver_attach+0xc0/0x1c0
> [ 17.164991] ? __device_attach_driver+0xe0/0xe0
> [ 17.164992] ? __device_attach_driver+0xe0/0xe0
> [ 17.164994] bus_for_each_dev+0x78/0xc0
> [ 17.164995] new_id_store+0xfc/0x1d0
> [ 17.164996] kernfs_fop_write_iter+0x11c/0x1b0
> [ 17.164998] new_sync_write+0x11f/0x1b0
> [ 17.165001] vfs_write+0x209/0x2a0
> [ 17.165003] ksys_write+0x5f/0xe0
> [ 17.165004] do_syscall_64+0x3b/0xc0
> [ 17.165006]
>
> Fixes: bf3747ae2e25 ("mt76: mt7921: enable aspm by default")
> Reported-by: Kai-Chuan Hsieh <kaichuan.hsieh@canonical.com>
> Co-developed-by: Sean Wang <sean.wang@mediatek.com>
> Signed-off-by: Sean Wang <sean.wang@mediatek.com>
> Signed-off-by: Deren Wu <deren.wu@mediatek.com>
> ---
> .../net/wireless/mediatek/mt76/mt7921/dma.c | 4 ++++
> .../net/wireless/mediatek/mt76/mt7921/mt7921.h | 1 +
> .../wireless/mediatek/mt76/mt7921/pci_mcu.c | 18 ++++++++++++++----
> 3 files changed, 19 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/net/wireless/mediatek/mt76/mt7921/dma.c b/drivers/net/wireless/mediatek/mt76/mt7921/dma.c
> index cdff1fd52d93..0c4b0e0e634a 100644
> --- a/drivers/net/wireless/mediatek/mt76/mt7921/dma.c
> +++ b/drivers/net/wireless/mediatek/mt76/mt7921/dma.c
> @@ -364,6 +364,10 @@ int mt7921_dma_init(struct mt7921_dev *dev)
> if (ret)
> return ret;
>
> + ret = __mt7921e_mcu_drv_pmctrl(dev);
> + if (ret)
> + return ret;
> +
> ret = mt7921_wfsys_reset(dev);
> if (ret)
> return ret;
> diff --git a/drivers/net/wireless/mediatek/mt76/mt7921/mt7921.h b/drivers/net/wireless/mediatek/mt76/mt7921/mt7921.h
> index 8b674e042568..63e3c7ef5e89 100644
> --- a/drivers/net/wireless/mediatek/mt76/mt7921/mt7921.h
> +++ b/drivers/net/wireless/mediatek/mt76/mt7921/mt7921.h
> @@ -443,6 +443,7 @@ int mt7921e_mcu_init(struct mt7921_dev *dev);
> int mt7921s_wfsys_reset(struct mt7921_dev *dev);
> int mt7921s_mac_reset(struct mt7921_dev *dev);
> int mt7921s_init_reset(struct mt7921_dev *dev);
> +int __mt7921e_mcu_drv_pmctrl(struct mt7921_dev *dev);
> int mt7921e_mcu_drv_pmctrl(struct mt7921_dev *dev);
> int mt7921e_mcu_fw_pmctrl(struct mt7921_dev *dev);
>
> diff --git a/drivers/net/wireless/mediatek/mt76/mt7921/pci_mcu.c b/drivers/net/wireless/mediatek/mt76/mt7921/pci_mcu.c
> index f9e350b67fdc..36669e5aeef3 100644
> --- a/drivers/net/wireless/mediatek/mt76/mt7921/pci_mcu.c
> +++ b/drivers/net/wireless/mediatek/mt76/mt7921/pci_mcu.c
> @@ -59,10 +59,8 @@ int mt7921e_mcu_init(struct mt7921_dev *dev)
> return err;
> }
>
> -int mt7921e_mcu_drv_pmctrl(struct mt7921_dev *dev)
> +int __mt7921e_mcu_drv_pmctrl(struct mt7921_dev *dev)
> {
> - struct mt76_phy *mphy = &dev->mt76.phy;
> - struct mt76_connac_pm *pm = &dev->pm;
> int i, err = 0;
>
> for (i = 0; i < MT7921_DRV_OWN_RETRY_COUNT; i++) {
> @@ -75,9 +73,21 @@ int mt7921e_mcu_drv_pmctrl(struct mt7921_dev *dev)
> if (i == MT7921_DRV_OWN_RETRY_COUNT) {
> dev_err(dev->mt76.dev, "driver own failed\n");
> err = -EIO;
> - goto out;
> }
>
> + return err;
> +}
> +
> +int mt7921e_mcu_drv_pmctrl(struct mt7921_dev *dev)
> +{
> + struct mt76_phy *mphy = &dev->mt76.phy;
> + struct mt76_connac_pm *pm = &dev->pm;
> + int err;
> +
> + err = __mt7921e_mcu_drv_pmctrl(dev);
> + if (err < 0)
> + goto out;
> +
> mt7921_wpdma_reinit_cond(dev);
> clear_bit(MT76_STATE_PM, &mphy->state);
>
> --
> 2.25.1
>
_______________________________________________
Linux-mediatek mailing list
Linux-mediatek@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-mediatek
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2022-01-07 0:54 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-01-05 22:20 [PATCH] mt76: mt7921e: fix reset failure in mt7921_pci_probe sean.wang
2022-01-07 0:53 ` Sean Wang
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).