Re: [PATCH 1/4] mm/gup: Add FOLL_INTERRUPTIBLE

From: John Hubbard
Date: Thu Jun 30 2022 - 15:02:01 EST


On 6/30/22 06:49, Peter Xu wrote:
Looks good to me, I'd tune a bit to make it less ambiguous on a few places:

/*
* FAULT_FLAG_INTERRUPTIBLE is opt-in. GUP callers must set
* FOLL_INTERRUPTIBLE to enable FAULT_FLAG_INTERRUPTIBLE.
* That's because some callers may not be prepared to
* handle early exits caused by non-fatal signals.
*/

Would that be okay to you?


Yes, looks good. With that change, please feel free to add:

Reviewed-by: John Hubbard <jhubbard@xxxxxxxxxx>



thanks,
--
John Hubbard
NVIDIA