On Fri, Aug 31, 2018 at 03:26:32PM -0600, Keith Busch wrote: > A device still participates in error recovery even if it doesn't have > the error callbacks. This patch provides the status for user event > watchers. > > Signed-off-by: Keith Busch <keith.busch@xxxxxxxxx> > --- > drivers/pci/pcie/err.c | 12 ++++++------ > 1 file changed, 6 insertions(+), 6 deletions(-) > > diff --git a/drivers/pci/pcie/err.c b/drivers/pci/pcie/err.c > index ee9014615add..a42a17a851fb 100644 > --- a/drivers/pci/pcie/err.c > +++ b/drivers/pci/pcie/err.c > @@ -67,12 +67,12 @@ static int report_error_detected(struct pci_dev *dev, > vote = PCI_ERS_RESULT_NO_AER_DRIVER; > else > vote = PCI_ERS_RESULT_NONE; > + goto out; > } > - > + err_handler = dev->driver->err_handler; > + vote = err_handler->error_detected(dev, state); > +out: > + pci_uevent_ers(dev, vote); > *result = merge_result(*result, vote); > device_unlock(&dev->dev); > return 0; The goto out looks a little odd here. Why not keep the else and just move the uevent notification out of it?
