Re: [PATCH 4/4] PCI/DPC: Print AER status in DPC event handling

Keith Busch
2018-01-16T02:44:28Z

On Wed, Jan 10, 2018 at 10:52:23AM -0500, Sinan Kaya wrote:
> Since the AER status is set when we observe DPC event and nobody is clearing these
> we won't observe another DPC event until somebody clears these. We can say that
> we are resetting the endpoints as part of the DPC but we are not touching the
> switch downstream port or the root port registers.
>
> Somebody still needs to clear these in addition to printing whatever information
> is available in the AER registers.
>
> Do you agree?

We should clear the downstream port's AER uncorrectable status register if
the trigger reason is of that type, but DPC definitely does not require
we clear it in order to observe a new event.