Re: [PATCH rc 1/2] iommufd: Do not access the area pointer after unlocking

[Jason Gunthorpe <jgg@nvidia.com>]

On Tue, Jun 20, 2023 at 05:30:39AM +0000, Tian, Kevin wrote:

> this looks incorrect. If the unmapped range covers more than 1000 areas
> and each area is attached by an access, this logic implies that an error will
> be returned after the first 1000 areas are unmapped.

Yes, that makes sense
 
> IMHO here we want to record the last notified area and compare it in
> retry, e.g.:
> 
> 	struct iopt_area *last_notify_area = NULL;

We can't use the area pointer, but we can keep track of the IOVA and
reset tries as it progresses.

Thanks,
Jason