Re: [PATCH v10 04/12] iommu: Add attach/detach_dev_pasid iommu interface

[Baolu Lu]

On 2022/7/29 20:22, Jason Gunthorpe wrote:
> The RID issue is that we can't reliably tell the source apart in a
> group - so all the RIDs in a group have to be considered as the same
> RID, and mapped to the same PASID table.
>
> But that is the only restriction of a group we have left, because the
> 'iommu doesn't isolate all traffic' restriction is defined not to
> exist if PASID is supported.

Get you. Thank you for the guide.

> > So yes, from this angle leaving one table per group is a simpler
> > thing to do, especially when it's unclear whether there is real
> > demand to enable PASID for multi-device group. 😊
> Except it is confusing, complicated and unnecessary. Treating PASID of
> multi-device groups the same as everything else is logically simple.

Yes. Considering that current PASID use cases occur only on singleton
groups, to make things simple, let's start our PASID attachment support
simply from singleton groups.

Best regards,
baolu