From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751908AbdJYNpQ (ORCPT ); Wed, 25 Oct 2017 09:45:16 -0400 Received: from mail.kernel.org ([198.145.29.99]:44154 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751374AbdJYNpO (ORCPT ); Wed, 25 Oct 2017 09:45:14 -0400 DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org DF946214DA Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=kernel.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=helgaas@kernel.org Date: Wed, 25 Oct 2017 08:45:11 -0500 From: Bjorn Helgaas To: Sinan Kaya Cc: linux-pci@vger.kernel.org, timur@codeaurora.org, linux-arm-msm@vger.kernel.org, Bjorn Helgaas , linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, Alex Williamson Subject: Re: [PATCH] PCI: rework error checking in the reset path Message-ID: <20171025134511.GL21840@bhelgaas-glaptop.roam.corp.google.com> References: <1508794608-15310-1-git-send-email-okaya@codeaurora.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1508794608-15310-1-git-send-email-okaya@codeaurora.org> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org [+cc Alex] On Mon, Oct 23, 2017 at 05:36:48PM -0400, Sinan Kaya wrote: > The return codes from various reset types are not consistent. The code is > assuming that all reset types will return -ENOTTY when things go wrong. > Instead of relying on negative error status, let's bail out if the > operation is successful instead. I like this (no surprise since I suggested something similar at http://lkml.kernel.org/r/20171011210057.GU25517@bhelgaas-glaptop.roam.corp.google.com), but I'd like Alex's opinion before merging it. Previously, we only tried the next reset method if one method failed with -ENOTTY. With this patch, we'll try the next reset method if one method fails for any reason, not just -ENOTTY. > Signed-off-by: Sinan Kaya > --- > drivers/pci/pci.c | 8 ++++---- > 1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/drivers/pci/pci.c b/drivers/pci/pci.c > index 6078dfc..a753e07 100644 > --- a/drivers/pci/pci.c > +++ b/drivers/pci/pci.c > @@ -4200,20 +4200,20 @@ int __pci_reset_function_locked(struct pci_dev *dev) > might_sleep(); > > rc = pci_dev_specific_reset(dev, 0); > - if (rc != -ENOTTY) > + if (!rc) > return rc; > if (pcie_has_flr(dev)) { > pcie_flr(dev); > return 0; > } > rc = pci_af_flr(dev, 0); > - if (rc != -ENOTTY) > + if (!rc) > return rc; > rc = pci_pm_reset(dev, 0); > - if (rc != -ENOTTY) > + if (!rc) > return rc; > rc = pci_dev_reset_slot_function(dev, 0); > - if (rc != -ENOTTY) > + if (!rc) > return rc; > return pci_parent_bus_reset(dev, 0); > } > -- > 1.9.1 > > > _______________________________________________ > linux-arm-kernel mailing list > linux-arm-kernel@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel