diff options
author | Jason Gunthorpe <jgg@nvidia.com> | 2022-04-18 08:49:58 +0800 |
---|---|---|
committer | Joerg Roedel <jroedel@suse.de> | 2022-04-28 15:32:20 +0200 |
commit | 93219ea94388b5195d1eeadb0bda6ed1471fba56 (patch) | |
tree | 3a3271641e44dba41a85a34378f5ed8ae58901e0 /include/linux/bpf.h | |
parent | 31076af0cb001f698138c8158c798ec9f4a8bbae (diff) |
vfio: Delete the unbound_list
commit 60720a0fc646 ("vfio: Add device tracking during unbind") added the
unbound list to plug a problem with KVM where KVM_DEV_VFIO_GROUP_DEL
relied on vfio_group_get_external_user() succeeding to return the
vfio_group from a group file descriptor. The unbound list allowed
vfio_group_get_external_user() to continue to succeed in edge cases.
However commit 5d6dee80a1e9 ("vfio: New external user group/file match")
deleted the call to vfio_group_get_external_user() during
KVM_DEV_VFIO_GROUP_DEL. Instead vfio_external_group_match_file() is used
to directly match the file descriptor to the group pointer.
This in turn avoids the call down to vfio_dev_viable() during
KVM_DEV_VFIO_GROUP_DEL and also avoids the trouble the first commit was
trying to fix.
There are no other users of vfio_dev_viable() that care about the time
after vfio_unregister_group_dev() returns, so simply delete the
unbound_list entirely.
Reviewed-by: Chaitanya Kulkarni <kch@nvidia.com>
Signed-off-by: Jason Gunthorpe <jgg@nvidia.com>
Signed-off-by: Lu Baolu <baolu.lu@linux.intel.com>
Acked-by: Alex Williamson <alex.williamson@redhat.com>
Link: https://lore.kernel.org/r/20220418005000.897664-10-baolu.lu@linux.intel.com
Signed-off-by: Joerg Roedel <jroedel@suse.de>
Diffstat (limited to 'include/linux/bpf.h')
0 files changed, 0 insertions, 0 deletions