Re: [PATCH v7 03/10] iommu/sva: Add iommu_sva_domain support

[Baolu Lu]

On 2022/5/24 17:44, Tian, Kevin wrote:
> > From: Baolu Lu <baolu.lu@xxxxxxxxxxxxxxx>
> > Sent: Monday, May 23, 2022 3:13 PM
> > > @@ -254,6 +259,7 @@ struct iommu_ops {
> > >    	int (*def_domain_type)(struct device *dev);
> > >
> > >    	const struct iommu_domain_ops *default_domain_ops;
> > > +	const struct iommu_domain_ops *sva_domain_ops;
> >
> > Per Joerg's comment in anther thread,
> >
> > https://lore.kernel.org/linux-iommu/YodVJ7ervpIdWfg+@xxxxxxxxxx/
> >
> > adding a sva_domain_ops here is not the right way to go.
> >
> > If no objection, I will make the sva domain go through the
> > generic domain_alloc/free() callbacks in the next version.
>
> suppose it's just back to what v1-v6 did which all registered the ops
> in domain_alloc() callback?

Yes.

Best regards,
baolu