On Mon, Mar 06, 2023 at 09:44:00PM +1100, Alexey Kardashevskiy wrote:
> > +static const struct iommu_domain_ops spapr_tce_blocking_domain_ops = {
> > + .attach_dev = spapr_tce_blocking_iommu_attach_dev,
> > + .detach_dev = spapr_tce_blocking_iommu_detach_dev,
>
>
> This .detach_dev() is gone now, the upstream has it removed. I am not quite
> sure now what should be calling release_ownership(). Probably
> spapr_tce_iommu_ops is going to need default_domain_ops, which attach_dev()
> will call release_ownership().
set_platform_dma_ops is what you want here
Jason
