vfio/pci: Fix handling of pci use accessor return codes
authorHeiner Kallweit <hkallweit1@gmail.com>
Sun, 24 Jan 2021 15:35:41 +0000 (16:35 +0100)
committerAlex Williamson <alex.williamson@redhat.com>
Mon, 1 Feb 2021 20:40:52 +0000 (13:40 -0700)
The pci user accessors return negative errno's on error.

Signed-off-by: Heiner Kallweit <hkallweit1@gmail.com>
Reviewed-by: Cornelia Huck <cohuck@redhat.com>
[aw: drop Fixes tag, pcibios_err_to_errno() behaves correctly for -errno]
Signed-off-by: Alex Williamson <alex.williamson@redhat.com>
drivers/vfio/pci/vfio_pci_igd.c

index 53d97f4..e66dfb0 100644 (file)
@@ -127,7 +127,7 @@ static size_t vfio_pci_igd_cfg_rw(struct vfio_pci_device *vdev,
 
                ret = pci_user_read_config_byte(pdev, pos, &val);
                if (ret)
-                       return pcibios_err_to_errno(ret);
+                       return ret;
 
                if (copy_to_user(buf + count - size, &val, 1))
                        return -EFAULT;
@@ -141,7 +141,7 @@ static size_t vfio_pci_igd_cfg_rw(struct vfio_pci_device *vdev,
 
                ret = pci_user_read_config_word(pdev, pos, &val);
                if (ret)
-                       return pcibios_err_to_errno(ret);
+                       return ret;
 
                val = cpu_to_le16(val);
                if (copy_to_user(buf + count - size, &val, 2))
@@ -156,7 +156,7 @@ static size_t vfio_pci_igd_cfg_rw(struct vfio_pci_device *vdev,
 
                ret = pci_user_read_config_dword(pdev, pos, &val);
                if (ret)
-                       return pcibios_err_to_errno(ret);
+                       return ret;
 
                val = cpu_to_le32(val);
                if (copy_to_user(buf + count - size, &val, 4))
@@ -171,7 +171,7 @@ static size_t vfio_pci_igd_cfg_rw(struct vfio_pci_device *vdev,
 
                ret = pci_user_read_config_word(pdev, pos, &val);
                if (ret)
-                       return pcibios_err_to_errno(ret);
+                       return ret;
 
                val = cpu_to_le16(val);
                if (copy_to_user(buf + count - size, &val, 2))
@@ -186,7 +186,7 @@ static size_t vfio_pci_igd_cfg_rw(struct vfio_pci_device *vdev,
 
                ret = pci_user_read_config_byte(pdev, pos, &val);
                if (ret)
-                       return pcibios_err_to_errno(ret);
+                       return ret;
 
                if (copy_to_user(buf + count - size, &val, 1))
                        return -EFAULT;