On Fri, 2014-09-05 at 12:53 +0200, Joerg Roedel wrote: > From: Joerg Roedel <[email protected]> > > Cc: Alex Williamson <[email protected]> > Signed-off-by: Joerg Roedel <[email protected]>
Nice, we can do some further simplifications if I no longer need to create a domain and add properties to it in order to test capabilities, but that can all be done later. Acked-by: Alex Williamson <[email protected]> > --- > drivers/vfio/vfio_iommu_type1.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/vfio/vfio_iommu_type1.c b/drivers/vfio/vfio_iommu_type1.c > index 0734fbe..562f686 100644 > --- a/drivers/vfio/vfio_iommu_type1.c > +++ b/drivers/vfio/vfio_iommu_type1.c > @@ -713,14 +713,14 @@ static int vfio_iommu_type1_attach_group(void > *iommu_data, > list_add(&group->next, &domain->group_list); > > if (!allow_unsafe_interrupts && > - !iommu_domain_has_cap(domain->domain, IOMMU_CAP_INTR_REMAP)) { > + !iommu_capable(bus, IOMMU_CAP_INTR_REMAP)) { > pr_warn("%s: No interrupt remapping support. Use the module > param \"allow_unsafe_interrupts\" to enable VFIO IOMMU support on this > platform\n", > __func__); > ret = -EPERM; > goto out_detach; > } > > - if (iommu_domain_has_cap(domain->domain, IOMMU_CAP_CACHE_COHERENCY)) > + if (iommu_capable(bus, IOMMU_CAP_CACHE_COHERENCY)) > domain->prot |= IOMMU_CACHE; > > /* -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
