No functional change is intended.
Signed-off-by: Yi Liu <yi.l.liu@xxxxxxxxx>
---
drivers/vfio/pci/vfio_pci_core.c | 55 ++++++++++++++++----------------
1 file changed, 28 insertions(+), 27 deletions(-)
diff --git a/drivers/vfio/pci/vfio_pci_core.c b/drivers/vfio/pci/vfio_pci_core.c
index b68fcba67a4b..b6b5624c8b15 100644
--- a/drivers/vfio/pci/vfio_pci_core.c
+++ b/drivers/vfio/pci/vfio_pci_core.c
@@ -177,10 +177,10 @@ static void vfio_pci_probe_mmaps(struct vfio_pci_core_device *vdev)
}
}
-struct vfio_pci_group_info;
+struct vfio_pci_file_info;
static void vfio_pci_dev_set_try_reset(struct vfio_device_set *dev_set);
static int vfio_pci_dev_set_hot_reset(struct vfio_device_set *dev_set,
- struct vfio_pci_group_info *groups,
+ struct vfio_pci_file_info *info,
struct iommufd_ctx *iommufd_ctx);
/*
@@ -800,7 +800,7 @@ static int vfio_pci_fill_devs(struct pci_dev *pdev, void *data)
return 0;
}
-struct vfio_pci_group_info {
+struct vfio_pci_file_info {
int count;
struct file **files;
};
@@ -1257,14 +1257,14 @@ static int vfio_pci_ioctl_get_pci_hot_reset_info(
}
static int
-vfio_pci_ioctl_pci_hot_reset_groups(struct vfio_pci_core_device *vdev,
- struct vfio_pci_hot_reset *hdr,
- bool slot,
- struct vfio_pci_hot_reset __user *arg)
+vfio_pci_ioctl_pci_hot_reset_files(struct vfio_pci_core_device *vdev,
+ struct vfio_pci_hot_reset *hdr,
+ bool slot,
+ struct vfio_pci_hot_reset __user *arg)
{
- int32_t *group_fds;
+ int32_t *fds;
struct file **files;
- struct vfio_pci_group_info info;
+ struct vfio_pci_file_info info;
int file_idx, count = 0, ret = 0;
/*
@@ -1281,17 +1281,17 @@ vfio_pci_ioctl_pci_hot_reset_groups(struct vfio_pci_core_device *vdev,
if (hdr->count > count)
return -EINVAL;
- group_fds = kcalloc(hdr->count, sizeof(*group_fds), GFP_KERNEL);
+ fds = kcalloc(hdr->count, sizeof(*fds), GFP_KERNEL);
files = kcalloc(hdr->count, sizeof(*files), GFP_KERNEL);
- if (!group_fds || !files) {
- kfree(group_fds);
+ if (!fds || !files) {
+ kfree(fds);
kfree(files);
return -ENOMEM;
}
- if (copy_from_user(group_fds, arg->group_fds,
- hdr->count * sizeof(*group_fds))) {
- kfree(group_fds);
+ if (copy_from_user(fds, arg->group_fds,
+ hdr->count * sizeof(*fds))) {
+ kfree(fds);
kfree(files);
return -EFAULT;
}
@@ -1301,7 +1301,7 @@ vfio_pci_ioctl_pci_hot_reset_groups(struct vfio_pci_core_device *vdev,
* the reset
*/
for (file_idx = 0; file_idx < hdr->count; file_idx++) {
- struct file *file = fget(group_fds[file_idx]);
+ struct file *file = fget(fds[file_idx]);
if (!file) {
ret = -EBADF;
@@ -1318,9 +1318,9 @@ vfio_pci_ioctl_pci_hot_reset_groups(struct vfio_pci_core_device *vdev,
files[file_idx] = file;
}
- kfree(group_fds);
+ kfree(fds);
- /* release reference to groups on error */
+ /* release reference to fds on error */
if (ret)
goto hot_reset_release;
@@ -1358,7 +1358,7 @@ static int vfio_pci_ioctl_pci_hot_reset(struct vfio_pci_core_device *vdev,
return -ENODEV;
if (hdr.count)
- return vfio_pci_ioctl_pci_hot_reset_groups(vdev, &hdr, slot, arg);
+ return vfio_pci_ioctl_pci_hot_reset_files(vdev, &hdr, slot, arg);
iommufd = vfio_iommufd_physical_ictx(&vdev->vdev);
@@ -2329,16 +2329,16 @@ const struct pci_error_handlers vfio_pci_core_err_handlers = {
};
EXPORT_SYMBOL_GPL(vfio_pci_core_err_handlers);
-static bool vfio_dev_in_groups(struct vfio_pci_core_device *vdev,
- struct vfio_pci_group_info *groups)
+static bool vfio_dev_in_files(struct vfio_pci_core_device *vdev,
+ struct vfio_pci_file_info *info)
{
unsigned int i;
- if (!groups)
+ if (!info)
return false;
- for (i = 0; i < groups->count; i++)
- if (vfio_file_has_dev(groups->files[i], &vdev->vdev))
+ for (i = 0; i < info->count; i++)
+ if (vfio_file_has_dev(info->files[i], &vdev->vdev))
return true;
return false;
}
@@ -2429,7 +2429,7 @@ static bool vfio_dev_in_iommufd_ctx(struct vfio_pci_core_device *vdev,
* get each memory_lock.
*/
static int vfio_pci_dev_set_hot_reset(struct vfio_device_set *dev_set,
- struct vfio_pci_group_info *groups,
+ struct vfio_pci_file_info *info,
struct iommufd_ctx *iommufd_ctx)
{
struct vfio_pci_core_device *cur_mem;
@@ -2469,7 +2469,8 @@ static int vfio_pci_dev_set_hot_reset(struct vfio_device_set *dev_set,
* cannot race being opened by another user simultaneously.
*
* Otherwise all opened devices in the dev_set must be
- * contained by the set of groups provided by the user.
+ * contained by the set of groups/devices provided by
+ * the user.
*
* If user provides a zero-length array, then all the
* opened devices must be bound to a same iommufd_ctx.
@@ -2478,7 +2479,7 @@ static int vfio_pci_dev_set_hot_reset(struct vfio_device_set *dev_set,
* the calling device is in a singleton dev_set.
*/
if (cur_vma->vdev.open_count &&
- !vfio_dev_in_groups(cur_vma, groups) &&
+ !vfio_dev_in_files(cur_vma, info) &&
!vfio_dev_in_iommufd_ctx(cur_vma, iommufd_ctx) &&
(dev_set->device_count > 1)) {
ret = -EINVAL;
--
2.34.1
