Re: [PATCH] PCI: quirks: Add warm reset quirk for IBM Spyre accelerator

Avinash Roy <[email protected]>
Newsgroups org.ozlabs.lists.linuxppc-dev
Message-ID <[email protected]>
On 8/19/26 9:04 PM, Avinash Roy wrote:
>
>
> On 8/6/26 10:56 PM, Bjorn Helgaas wrote:
>> On Thu, Aug 06, 2026 at 10:20:02PM +0530, Shivaprasad G Bhat wrote:
>>> On 8/5/26 10:48 PM, Bjorn Helgaas wrote:
>>>> On Wed, Aug 05, 2026 at 06:21:32PM +0530, Avinash Roy wrote:
>>>>> From: Shivaprasad G Bhat<[email protected]>
>>>>>
>>>>> The IBM Spyre accelerator (PCI ID 1014:06a7) requires a dedicated warm
>>>>> reset method for reliable EEH recovery on pSeries/ppc64 platforms.
>>>>>
>>>>> FLR alone is insufficient because it does not re-initialize the Spyre
>>>>> firmware state machine. On ppc64 with EEH enabled,
>>>>> pcibios_set_pcie_reset_state() is overridden in
>>>>> arch/powerpc/kernel/eeh.c to drive the platform-level PERST# signal via
>>>>> EEH PE operations, which is the correct mechanism for full device
>>>>> recovery after an EEH event.
>>>> This makes it sound like this mechanism is needed for EEH recovery for
>>>> *all* devices, not just Spyre.  If that's the case it sounds like we
>>>> need more than just a Spyre-specific quirk.
>>> Now I re-read it, The second statement needs to be re-worded. It does sound
>>> like its true for all devices. Quirk is only for the Spyre card.
>> So I guess this is related to some Spyre erratum?  Per spec, FLR is
>> supposed to "set all Function-specific state machines to their
>> initialization values" (PCIe r7.0, sec 6.6.2).
> Yes, This is for Spyre card specificchanges. The card requires an explicit warm reset to restore it to the 
> proper initialization state.

Sorry about the typo. This is for the current version of the Spyre card, which requires an explicit warm reset to restore it to the proper initialized state.

Thanks & Regards,
Avinash
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.