diff options
| author |   Alex Williamson <alex.williamson@redhat.com> | 2015-04-07 11:14:44 -0600 |
|---|---|---|
| committer | Alex Williamson <alex.williamson@redhat.com> | 2015-04-07 11:14:44 -0600 |
| commit | 561d72ddbbe480e2cae6c1a63c688986586ef2b1 (patch) | |
| tree | 08dce63e805fd681d1d96e6aebac630d4bd85cee /drivers/vfio | |
| parent | vfio-pci: Allow PCI IDs to be specified as module options (diff) | |
| download | linux-dev-561d72ddbbe480e2cae6c1a63c688986586ef2b1.tar.xz linux-dev-561d72ddbbe480e2cae6c1a63c688986586ef2b1.zip | |
vfio-pci: Remove warning if try-reset fails
As indicated in the comment, this is not entirely uncommon and
causes user concern for no reason.
Signed-off-by: Alex Williamson <alex.williamson@redhat.com>
Diffstat (limited to 'drivers/vfio')
| -rw-r--r-- | drivers/vfio/pci/vfio_pci.c | 10 |
1 files changed, 2 insertions, 8 deletions
diff --git a/drivers/vfio/pci/vfio_pci.c b/drivers/vfio/pci/vfio_pci.c index 43517ce930f2..d0f1e7026d5d 100644 --- a/drivers/vfio/pci/vfio_pci.c +++ b/drivers/vfio/pci/vfio_pci.c @@ -217,14 +217,8 @@ static void vfio_pci_disable(struct vfio_pci_device *vdev) * Try to reset the device. The success of this is dependent on * being able to lock the device, which is not always possible. */ - if (vdev->reset_works) { - int ret = pci_try_reset_function(pdev); - if (ret) - pr_warn("%s: Failed to reset device %s (%d)\n", - __func__, dev_name(&pdev->dev), ret); - else - vdev->needs_reset = false; - } + if (vdev->reset_works && !pci_try_reset_function(pdev)) + vdev->needs_reset = false; pci_restore_state(pdev); out: |