Re: [PATCH v5 07/17] dma-mapping: Implement link/unlink ranges API

Jason Gunthorpe
2025-01-16T20:18:55Z

On Wed, Jan 15, 2025 at 10:33:40AM +0200, Leon Romanovsky wrote:
> > > + do {
> > > + phys_addr_t phys;
> > > + size_t len;
> > > +
> > > + phys = iommu_iova_to_phys(domain, addr);
> > > + if (WARN_ON(!phys))
> > > + continue;
> >
> > Infinite WARN_ON loop, nice.
>
> No problem, will change it to WARN_ON_ONCE.

I think the other point is that the addr doesn't increase, so this
loop will lock up.

Possibly just do return? I suppose something is hopelessly corrupted
if we ever hit this..

Jason