bus/pci: fix VFIO device reset
authorJerin Jacob <jerin.jacob@caviumnetworks.com>
Sat, 28 Oct 2017 06:22:55 +0000 (11:52 +0530)
committerThomas Monjalon <thomas@monjalon.net>
Tue, 31 Oct 2017 18:23:36 +0000 (19:23 +0100)
If the device is not capable of resetting, then Linux kernel updates
the errno as EINVAL.
http://elixir.free-electrons.com/linux/v4.9/source/drivers/vfio/pci/vfio_pci.c#L887

Honor the EINVAL errno value to avoid pci vfio setup failure.

Fixes: f25f8f367644 ("bus/pci: check VFIO reset ioctl error")

Signed-off-by: Jerin Jacob <jerin.jacob@caviumnetworks.com>
Reviewed-by: Jonas Pfefferle <jpf@zurich.ibm.com>
drivers/bus/pci/linux/pci_vfio.c

index 13d52a8..3ca35b6 100644 (file)
@@ -338,8 +338,11 @@ pci_vfio_setup_device(struct rte_pci_device *dev, int vfio_dev_fd)
                return -1;
        }
 
-       /* Reset the device */
-       if (ioctl(vfio_dev_fd, VFIO_DEVICE_RESET)) {
+       /*
+        * Reset the device. If the device is not capable of resetting,
+        * then it updates errno as EINVAL.
+        */
+       if (ioctl(vfio_dev_fd, VFIO_DEVICE_RESET) && errno != EINVAL) {
                RTE_LOG(ERR, EAL, "Unable to reset device! Error: %d (%s)\n",
                                errno, strerror(errno));
                return -1;