Messages in this thread | | | Date | Thu, 23 Jun 2022 09:47:02 +0800 | Subject | Re: [PATCH v2 2/2] vfio: Use device_iommu_capable() | From | Baolu Lu <> |
| |
On 2022/6/22 20:04, Robin Murphy wrote: > Use the new interface to check the capabilities for our device > specifically. > > Signed-off-by: Robin Murphy <robin.murphy@arm.com> > --- > drivers/vfio/vfio.c | 2 +- > drivers/vfio/vfio_iommu_type1.c | 2 +- > 2 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/vfio/vfio.c b/drivers/vfio/vfio.c > index 73bab04880d0..765d68192c88 100644 > --- a/drivers/vfio/vfio.c > +++ b/drivers/vfio/vfio.c > @@ -621,7 +621,7 @@ int vfio_register_group_dev(struct vfio_device *device) > * VFIO always sets IOMMU_CACHE because we offer no way for userspace to > * restore cache coherency. > */ > - if (!iommu_capable(device->dev->bus, IOMMU_CAP_CACHE_COHERENCY)) > + if (!device_iommu_capable(device->dev, IOMMU_CAP_CACHE_COHERENCY)) > return -EINVAL; > > return __vfio_register_dev(device, > diff --git a/drivers/vfio/vfio_iommu_type1.c b/drivers/vfio/vfio_iommu_type1.c > index e38b8bfde677..2107e95eb743 100644 > --- a/drivers/vfio/vfio_iommu_type1.c > +++ b/drivers/vfio/vfio_iommu_type1.c > @@ -2247,7 +2247,7 @@ static int vfio_iommu_type1_attach_group(void *iommu_data, > list_add(&group->next, &domain->group_list); > > msi_remap = irq_domain_check_msi_remap() || > - iommu_capable(iommu_api_dev->dev->bus, IOMMU_CAP_INTR_REMAP); > + device_iommu_capable(iommu_api_dev->dev, IOMMU_CAP_INTR_REMAP); > > if (!allow_unsafe_interrupts && !msi_remap) { > pr_warn("%s: No interrupt remapping support. Use the module param \"allow_unsafe_interrupts\" to enable VFIO IOMMU support on this platform\n",
Reviewed-by: Lu Baolu <baolu.lu@linux.intel.com>
Best regards, baolu
| |