Re: [PATCH v3 1/1] iommu: Bind process address spaces to devices

[Jean-Philippe Brucker]

On 11/04/2019 16:28, Joerg Roedel wrote:
> On Wed, Apr 10, 2019 at 04:15:16PM +0100, Jean-Philippe Brucker wrote:
>>  drivers/iommu/iommu.c | 104 ++++++++++++++++++++++++++++++++++++++++++
>>  include/linux/iommu.h |  70 ++++++++++++++++++++++++++++
>>  2 files changed, 174 insertions(+)
> 
> Applied to the api-features branch for now, thanks Jean-Philippe!
> 
> 'for now' because I have a question below.
> 
>> diff --git a/include/linux/iommu.h b/include/linux/iommu.h
>> index ed6738c358ca..87fe1cddd817 100644
>> --- a/include/linux/iommu.h
>> +++ b/include/linux/iommu.h
>> @@ -51,10 +51,13 @@ struct device;
>>  struct iommu_domain;
>>  struct notifier_block;
>>  struct iommu_fault_event;
>> +struct iommu_sva;
> 
> The iommu_fault_event doesn't yet exist in my tree, is this patch based
> on something else I need besided Lu Baolus work? I got quite some
> conflicts while applying the patch because of this.

Ugh, sorry about this, I forgot to rebase onto master. The only required
patch is Lu Baolu's "iommu: Add APIs for multiple domains per device".
The iommu_fault_event is introduced by Jacob's fault reporting work, but
is independent from this patch.

There also is a small conflict with the SMMUv3 ATS work, which I'm
hoping to get in v5.2 as well (but will probably require more work).
I'll send next version of ATS applied on top of this patch.

I pushed a cleaner version to git://linux-arm.org/linux-jpb.git sva/api
(with the below mistake fixed as well) and can resend if necessary.

Thanks,
Jean

> 
>> + * @sva_set_ops: Set ops for a SVA handle
> 
> I also removed this line.
> 
> Regards,
> 
>         Joerg