+Oza On 12/19/2017 4:06 PM, Keith Busch wrote: > A DPC enabled device will suppress sending ERR_FATAL and ERR_NONFATAL, > which prevents the AER handler from reporting the details of the > error. This patch will have the DPC driver get the AER status registers > from the downstream port that detected the uncorrectable error, and > print out additional information. > > Signed-off-by: Keith Busch <keith.busch@xxxxxxxxx> > --- Oza is doing some restructuring to unify DPC and AER error handling path per feedback from Bjorn. It is almost done. He is testing it. Can this patch wait until you review his version? I'm thinking this could be something that can be added to his series instead. Coming back to this patch. The interrupt number for DPC and AER could be the same or different. According to the spec, AER errors are always reported regardless of DPC driver presence (see the famous flow chart). If the interrupt IDs are the same for AER and DPC, your patch would introduce double printing for AER errors. Sinan -- Sinan Kaya Qualcomm Datacenter Technologies, Inc. as an affiliate of Qualcomm Technologies, Inc. Qualcomm Technologies, Inc. is a member of the Code Aurora Forum, a Linux Foundation Collaborative Project.
