On Tue, Jun 20, 2023 at 9:55 AM Jason Gunthorpe <jgg@xxxxxxxxxx> wrote: > > On Tue, Jun 20, 2023 at 07:57:57AM -0700, Alexander Duyck wrote: > > > I think this may have gone off down a rathole as my original question > > wasn't anything about adding extra reserved regions. It was about > > exposing what the IOVA is already reserving so it could be user > > visible. > > Your question points out that dma-iommu.c uses a different set of > reserved regions than everything else, and its set is closer to > functionally correct. > > IMHO the resolution to what you are talking about is not to add more > debugging to dma-iommu but to make the set of reserved regions > consistently correct for everyone, which will make them viewable in > sysfs. Okay, that makes sense to me, and I agree. If we had a consistent set of reserved regions then it would make it easier to understand. If nothing else my request would be to expose the iova reserved regions and then most likely the other ones could be deprecated since they seem to all be consolidated in the IOVA anyway. Thanks, - Alex
