Re: [PATCH 4/4] mm/gup: adapt get_user_page_vma_remote() to never return NULL

From: Jason Gunthorpe
Date: Mon Oct 09 2023 - 18:24:10 EST


On Sun, Oct 01, 2023 at 05:00:05PM +0100, Lorenzo Stoakes wrote:
> get_user_pages_remote() will never return 0 except in the case of
> FOLL_NOWAIT being specified, which we explicitly disallow.
>
> This simplifies error handling for the caller and avoids the awkwardness of
> dealing with both errors and failing to pin. Failing to pin here is an
> error.
>
> Suggested-by: Arnd Bergmann <arnd@xxxxxxxx>
> Signed-off-by: Lorenzo Stoakes <lstoakes@xxxxxxxxx>
> ---
> arch/arm64/kernel/mte.c | 4 ++--
> include/linux/mm.h | 16 +++++++++++++---
> kernel/events/uprobes.c | 4 ++--
> mm/memory.c | 3 +--
> 4 files changed, 18 insertions(+), 9 deletions(-)

Good riddance to IS_ERR_OR_NULL

Reviewed-by: Jason Gunthorpe <jgg@xxxxxxxxxx>

Jason