From: Lukas Wunner <lukas@wunner.de>
To: "Kumar1, Rahul" <Rahul.Kumar1@amd.com>
Cc: "Grodzovsky, Andrey" <Andrey.Grodzovsky@amd.com>,
"linux-pci@vger.kernel.org" <linux-pci@vger.kernel.org>,
"helgaas@kernel.org" <helgaas@kernel.org>,
"Antonovitch, Anatoli" <Anatoli.Antonovitch@amd.com>,
"Deucher, Alexander" <Alexander.Deucher@amd.com>
Subject: Re: Question about deadlock between AER and pceihp interrupts during resume from S3 with unplugged device
Date: Tue, 15 Feb 2022 08:02:29 +0100 [thread overview]
Message-ID: <20220215070229.GA21694@wunner.de> (raw)
In-Reply-To: <MN2PR12MB2926BBA4DCD4D7ECCA453577C0309@MN2PR12MB2926.namprd12.prod.outlook.com>
On Fri, Feb 11, 2022 at 02:42:21PM +0000, Kumar1, Rahul wrote:
> We can some changes we can see in lspci from working to non-working case. Below are changes
> Link Speed = 8GT/s -> 2.5GT/s.
> DLActive+ -> DLActive-
> BWMgmt+ -> BWMgmt+
> PresDet+ -> PresDet+
> EqualizationComplete+ -> EqualizationComplete+
>
> Also when we do reset via sysfs, we don't see this issue.
>
> I have created bug here https://bugzilla.kernel.org/show_bug.cgi?id=215590
So with the patches applied, the link doesn't come up after resume,
but if you then reset via sysfs, it does come up, is that what you're
saying?
The dmesg excerpt Andrey posted shows an AER splat after resume (even
with the patches applied):
[ 69.684921] pcieport 0000:00:01.1: AER: Root Port link has been reset
[ 69.691438] pcieport 0000:00:01.1: AER: Device recovery failed
[ 69.697327] pcieport 0000:00:01.1: AER: Multiple Uncorrected (Fatal) error received: 0000:00:01.0
[ 69.706231] pcieport 0000:00:01.1: AER: can't find device of ID0008
I suspect the Root Port refuses to train the link due to that fatal
error. Perhaps Kai-Heng Feng's patch is incomplete and it needs to
clear stale AER errors? Or maybe it re-enables AER too early?
Could you attach lspci -vv output before/after suspend to the bugzilla?
And also attach full dmesg output with the patches applied?
Thanks,
Lukas
next prev parent reply other threads:[~2022-02-15 7:02 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <0fc31d9a-f414-a412-3765-5519cbb9b7ff@amd.com>
2022-02-09 21:28 ` Question about deadlock between AER and pceihp interrupts during resume from S3 with unplugged device Andrey Grodzovsky
2022-02-10 6:23 ` Lukas Wunner
2022-02-10 14:39 ` Andrey Grodzovsky
2022-06-10 21:25 ` Andrey Grodzovsky
2022-06-14 18:07 ` Andrey Grodzovsky
2022-06-14 18:22 ` Sathyanarayanan Kuppuswamy
2022-06-14 20:35 ` Andrey Grodzovsky
2022-06-15 15:14 ` Sathyanarayanan Kuppuswamy
2022-06-15 15:49 ` Andrey Grodzovsky
2022-02-10 20:47 ` Andrey Grodzovsky
2022-02-10 21:37 ` Lukas Wunner
2022-02-10 23:12 ` Andrey Grodzovsky
2022-02-11 14:42 ` Kumar1, Rahul
2022-02-15 7:02 ` Lukas Wunner [this message]
2022-02-15 8:18 ` Kumar1, Rahul
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20220215070229.GA21694@wunner.de \
--to=lukas@wunner.de \
--cc=Alexander.Deucher@amd.com \
--cc=Anatoli.Antonovitch@amd.com \
--cc=Andrey.Grodzovsky@amd.com \
--cc=Rahul.Kumar1@amd.com \
--cc=helgaas@kernel.org \
--cc=linux-pci@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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).