Re: [PATCH v3] iommu: revisit iommu_insert_resv_region() implementation

2019-08-30 Thread Joerg Roedel
On Wed, Aug 21, 2019 at 02:09:40PM +0200, Eric Auger wrote: > drivers/iommu/iommu.c | 96 +-- > 1 file changed, 47 insertions(+), 49 deletions(-) Applied, thanks.

Re: [PATCH v3] iommu: revisit iommu_insert_resv_region() implementation

2019-08-25 Thread Christoph Hellwig
Looks good, Reviewed-by: Christoph Hellwig

[PATCH v3] iommu: revisit iommu_insert_resv_region() implementation

2019-08-21 Thread Eric Auger
Current implementation is recursive and in case of allocation failure the existing @regions list is altered. A non recursive version looks better for maintainability and simplifies the error handling. We use a separate stack for overlapping segment merging. The elements are sorted by start address