From: Marc Zyngier <maz@kernel.org>
To: Remi Pommarel <repk@triplefau.lt>
Cc: Lorenzo Pieralisi <lorenzo.pieralisi@arm.com>,
Andrew Murray <andrew.murray@arm.com>,
Ellie Reeves <ellierevves@gmail.com>, <linux-pci@vger.kernel.org>,
<linux-kernel@vger.kernel.org>,
Bjorn Helgaas <helgaas@kernel.org>,
Thomas Petazzoni <thomas.petazzoni@bootlin.com>,
<linux-arm-kernel@lists.infradead.org>
Subject: Re: [PATCH v3] PCI: aardvark: Use LTSSM state to build link training flag
Date: Mon, 14 Oct 2019 15:18:28 +0100 [thread overview]
Message-ID: <971be151d24312cc533989a64bd454b4@www.loen.fr> (raw)
In-Reply-To: <20191014140026.GB1426@voidbox>
On 2019-10-14 15:00, Remi Pommarel wrote:
> On Mon, Oct 14, 2019 at 02:45:34PM +0100, Marc Zyngier wrote:
>> Hi Remi,
>>
>> On 2019-10-14 14:06, Remi Pommarel wrote:
>> > Hi Lorenzo, Marc,
[...]
>> > Sure, I think this could be considered a fix for the following
>> commit :
>> > Fixes: 8a3ebd8de328 ("PCI: aardvark: Implement emulated root PCI
>> > bridge config space")
>> >
>> > Moreover, Marc, I am also a bit supprised that you did not have to
>> use
>> > [1] to even be able to boot.
>>
>> No, I don't have that one, and yet the system boots fine (although
>> PCI
>> doesn't get much use on this box). I guess I'm lucky...
>>
>> > Also if you want to be completely immune to this kind of SError
>> (that
>> > could theoretically happen if the link goes down for other reasons
>> than
>> > being retrained) you would have to use mainline ATF along with
>> [2]. But
>> > the chances to hit that are low (could only happen in case of link
>> > errors).
>>
>> Now you've got me worried. Can you point me to that ATF patch? I'm
>> quite
>> curious as to how you recover from an SError on a v8.0 CPU given
>> that it
>> has no syndrome information and may as well signal "CPU on fire!"...
>>
>
> The patch is at [1]. Please note that this is done quite similarly
> for
> rcar.
>
> [1]
> https://review.trustedfirmware.org/c/TF-A/trusted-firmware-a/+/1541
That patch, without any other information, looks quite flaky. Unless
there
is a strong guarantee that ESR_EL3.ISS==2 only when the PCIe controller
goes wrong, it looks like this only papers over the issue...
That's pretty much independent from the patch at hand in this thread,
but
I certainly wouldn't trust this ATF patch without some more information
about how the fault is reported to the CPU.
M.
--
Jazz is not dead. It just smells funny...
next prev parent reply other threads:[~2019-10-14 14:18 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-05-22 21:33 [PATCH v3] PCI: aardvark: Use LTSSM state to build link training flag Remi Pommarel
2019-08-06 18:50 ` Remi Pommarel
2019-09-25 12:32 ` Thomas Petazzoni
2019-09-30 15:40 ` Andrew Murray
2019-09-30 16:52 ` Remi Pommarel
2019-10-01 8:05 ` Andrew Murray
2019-10-13 10:34 ` Marc Zyngier
2019-10-14 10:01 ` Lorenzo Pieralisi
2019-10-14 13:06 ` Remi Pommarel
2019-10-14 13:45 ` Marc Zyngier
2019-10-14 14:00 ` Remi Pommarel
2019-10-14 14:18 ` Marc Zyngier [this message]
2019-10-14 16:50 ` Lorenzo Pieralisi
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=971be151d24312cc533989a64bd454b4@www.loen.fr \
--to=maz@kernel.org \
--cc=andrew.murray@arm.com \
--cc=ellierevves@gmail.com \
--cc=helgaas@kernel.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pci@vger.kernel.org \
--cc=lorenzo.pieralisi@arm.com \
--cc=repk@triplefau.lt \
--cc=thomas.petazzoni@bootlin.com \
/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).