| Message ID | 1363817083-2807-1-git-send-email-prarit@redhat.com (mailing list archive) |
|---|---|
| State | New, archived |
| Delegated to: | Bjorn Helgaas |
| Headers | show |
On Wed, Mar 20, 2013 at 4:04 PM, Prarit Bhargava <prarit@redhat.com> wrote: > Currently the aer_inject driver returns -ENOTTY when the target PCIE device > root port is not found or if the device or root port doesn't support AER. > > In the case where the root port isn't found, the driver should return -ENODEV, > and in the other cases it should return -EPERM. > > Signed-off-by: Prarit Bhargava <prarit@redhat.com> Applied for v3.10, thanks. Ben, chime in if you have any input about errno usage :) > --- > drivers/pci/pcie/aer/aer_inject.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/drivers/pci/pcie/aer/aer_inject.c b/drivers/pci/pcie/aer/aer_inject.c > index 4e24cb8..8d4a5e3 100644 > --- a/drivers/pci/pcie/aer/aer_inject.c > +++ b/drivers/pci/pcie/aer/aer_inject.c > @@ -334,13 +334,13 @@ static int aer_inject(struct aer_error_inj *einj) > return -ENODEV; > rpdev = pcie_find_root_port(dev); > if (!rpdev) { > - ret = -ENOTTY; > + ret = -ENODEV; > goto out_put; > } > > pos_cap_err = pci_find_ext_capability(dev, PCI_EXT_CAP_ID_ERR); > if (!pos_cap_err) { > - ret = -ENOTTY; > + ret = -EPERM; > goto out_put; > } > pci_read_config_dword(dev, pos_cap_err + PCI_ERR_UNCOR_SEVER, &sever); > @@ -350,7 +350,7 @@ static int aer_inject(struct aer_error_inj *einj) > > rp_pos_cap_err = pci_find_ext_capability(rpdev, PCI_EXT_CAP_ID_ERR); > if (!rp_pos_cap_err) { > - ret = -ENOTTY; > + ret = -EPERM; > goto out_put; > } > > -- > 1.7.9.3 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-pci" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/drivers/pci/pcie/aer/aer_inject.c b/drivers/pci/pcie/aer/aer_inject.c index 4e24cb8..8d4a5e3 100644 --- a/drivers/pci/pcie/aer/aer_inject.c +++ b/drivers/pci/pcie/aer/aer_inject.c @@ -334,13 +334,13 @@ static int aer_inject(struct aer_error_inj *einj) return -ENODEV; rpdev = pcie_find_root_port(dev); if (!rpdev) { - ret = -ENOTTY; + ret = -ENODEV; goto out_put; } pos_cap_err = pci_find_ext_capability(dev, PCI_EXT_CAP_ID_ERR); if (!pos_cap_err) { - ret = -ENOTTY; + ret = -EPERM; goto out_put; } pci_read_config_dword(dev, pos_cap_err + PCI_ERR_UNCOR_SEVER, &sever); @@ -350,7 +350,7 @@ static int aer_inject(struct aer_error_inj *einj) rp_pos_cap_err = pci_find_ext_capability(rpdev, PCI_EXT_CAP_ID_ERR); if (!rp_pos_cap_err) { - ret = -ENOTTY; + ret = -EPERM; goto out_put; }
Currently the aer_inject driver returns -ENOTTY when the target PCIE device root port is not found or if the device or root port doesn't support AER. In the case where the root port isn't found, the driver should return -ENODEV, and in the other cases it should return -EPERM. Signed-off-by: Prarit Bhargava <prarit@redhat.com> --- drivers/pci/pcie/aer/aer_inject.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-)