Re: [PATCH v3 0/7] iommu: Allow IOVA rcache range be configured

[John Garry]

On 03/06/2021 01:39, Lu Baolu wrote:
> > I did actually try increasing the range for a 'live' domain in the v1 
> > series, but it turned out too messy. First problem is reallocating the 
> > memory to hold the rcaches. Second problem is that we need to deal 
> > with the issue that all IOVAs in the rcache need to be a pow-of-2, 
> > which is difficult to enforce for IOVAs which weren't being cached 
> > before, but now would be.
> >
> > So now I changed to work similar to how we change the default domain 
> > type, i.e. don't operate on a 'live' domain.
>
> If these hard restrictions on users are just to walk around the messy
> code in kernel, I would rather solve those messy code to achieve a
> better user experience. :-)

Hi Baolu,

It's not that the code is messy. I'm just saying that it's difficult to 
change the rcache range of a live domain. So we take an approach similar 
to changing the domain default type - it is more straightforward to 
configure the rcache range that way.

Anyway, it would be great if you or anyone could look at the code so 
that some progress can be made here.

Thanks,
John