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

From: Tian, Kevin
Date: Tue May 24 2022 - 05:44:19 EST


> 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?