lkml.org 
[lkml]   [2023]   [Jan]   [17]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRe: [PATCH 3/4] iommu: Introduce IOMMU call-back for processing struct KVM assigned to VFIO
From
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

\
 
 \ /
  Last update: 2023-03-26 23:43    [W:1.193 / U:0.044 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site