diff options
| author | Alex Williamson <alex.williamson@redhat.com> | 2015-04-07 13:14:44 -0400 |
|---|---|---|
| committer | Alex Williamson <alex.williamson@redhat.com> | 2015-04-07 13:14:44 -0400 |
| commit | 561d72ddbbe480e2cae6c1a63c688986586ef2b1 (patch) | |
| tree | 08dce63e805fd681d1d96e6aebac630d4bd85cee /drivers/vfio | |
| parent | 80c7e8cc2aaa36944acdfbce46f373101b9f21ff (diff) | |
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) | |||
| 217 | * Try to reset the device. The success of this is dependent on | 217 | * Try to reset the device. The success of this is dependent on |
| 218 | * being able to lock the device, which is not always possible. | 218 | * being able to lock the device, which is not always possible. |
| 219 | */ | 219 | */ |
| 220 | if (vdev->reset_works) { | 220 | if (vdev->reset_works && !pci_try_reset_function(pdev)) |
| 221 | int ret = pci_try_reset_function(pdev); | 221 | vdev->needs_reset = false; |
| 222 | if (ret) | ||
| 223 | pr_warn("%s: Failed to reset device %s (%d)\n", | ||
| 224 | __func__, dev_name(&pdev->dev), ret); | ||
| 225 | else | ||
| 226 | vdev->needs_reset = false; | ||
| 227 | } | ||
| 228 | 222 | ||
| 229 | pci_restore_state(pdev); | 223 | pci_restore_state(pdev); |
| 230 | out: | 224 | out: |
