Messages in this thread | | | Date | Tue, 17 Jan 2023 12:31:07 +0700 | Subject | Re: [PATCH 3/4] iommu: Introduce IOMMU call-back for processing struct KVM assigned to VFIO | From | "Suthikulpanit, Suravee" <> |
| |
Hi Jason,
On 1/13/2023 10:35 PM, Jason Gunthorpe wrote: > On Tue, Jan 10, 2023 at 08:31:36AM -0600, Suravee Suthikulpanit wrote: >> Currently, VFIO provide an kvm_vfio_file_set_kvm() interface for assigning >> a KVM structure to a VFIO group. The information in struct KVM is also >> useful for IOMMU drivers when setting up VFIO domain. >> >> Introduce struct iommu_domain_ops.set_kvm call-back function to allow >> IOMMU drivers to provide call-back to process the struct KVM >> assigned. > > Also NAK > > Connecting the iommu driver to KVM has to be properly architected > though iommufd. >
My understanding is the kvm_vfio_file_set_kvm() from the following call-path:
* kvm_vfio_group_add() * kvm_vfio_group_del() * kvm_vfio_destroy()
to attach/detach KVM to/from a particular VFIO domain. This is an existing interface from kvm_vfio_set_group()
Here is the call-path:
kvm_vfio_file_set_kvm() vfio_file_set_kvm() iommu_set_kvm() <-- New interface amd_iommu_set_kvm()
Could you please elaborate what you have in mind for a properly architected interface via iommufd?
>> @@ -1652,6 +1652,7 @@ void vfio_file_set_kvm(struct file *file, struct kvm *kvm) >> >> mutex_lock(&group->group_lock); >> group->kvm = kvm; >> + iommu_set_kvm(group->iommu_group, kvm); >> mutex_unlock(&group->group_lock); >> } > > This also has obvious lifetime bugs
Could you please also elaborate on this part? For detaching case, KVM is NULL, and the same information is passed to the IOMMU driver to handle the detaching case. Am I missing anything?
Thanks, Suravee
> > Jason
| |