On Thu, Jun 08, 2023 at 05:52:30PM +0800, Zhenzhong Duan wrote: > During address space unmap, corresponding IOVA tree entries are > also removed. But DMAMap is set beyond notifier's scope by 1, so > in theory there is possibility to remove a continuous entry above > the notifier's scope but falling in adjacent notifier's scope.
This function is only called in "loop over all notifiers" case (or replay() that just got removed, but even so there'll be only 1 notifier normally iiuc at least for vt-d), hopefully it means no bug exist (no Fixes needed, no backport needed either), but still worth fixing it up. > > There is no issue currently as no use cases allocate notifiers > continuously, but let's be robust. > > Signed-off-by: Zhenzhong Duan <zhenzhong.d...@intel.com> Reviewed-by: Peter Xu <pet...@redhat.com> -- Peter Xu