On Fri, Jun 22, 2018 at 03:41:50PM +0530, poza@xxxxxxxxxxxxxx wrote:
> was thinking that current code
> pcie_do_fatal_recovery already does call
>
> if ((service == PCIE_PORT_SERVICE_AER) &&
> (dev->hdr_type == PCI_HEADER_TYPE_BRIDGE)) {
> /*
> * If the error is reported by a bridge, we think this error
> * is related to the downstream link of the bridge, so we
> * do error recovery on all subordinates of the bridge instead
> * of the bridge and clear the error status of the bridge.
> */
> pci_cleanup_aer_uncorrect_error_status(dev);
> }
>
>
> instead of calling it here in dpc driver, can we make use of that existing
> call ?
> probably we just might need to remove
> if ((service == PCIE_PORT_SERVICE_AER) condition
That's really only desirable when DPC error status is 0. It should be
harmless, though, so your update is fine with me.
