Re: [PATCH v2 1/2] mm/gup.c: Don't pass gup_flags to check_and_migrate_movable_pages()

From: David Hildenbrand
Date: Fri Aug 05 2022 - 04:10:31 EST


On 05.08.22 08:29, Alistair Popple wrote:
> gup_flags is passed to check_and_migrate_movable_pages() so that it can
> call either put_page() or unpin_user_page() to drop the page reference.
> However check_and_migrate_movable_pages() is only called for
> FOLL_LONGTERM, which implies FOLL_PIN so there is no need to pass
> gup_flags.
>
> Signed-off-by: Alistair Popple <apopple@xxxxxxxxxx>
> ---
> mm/gup.c | 21 ++++++++-------------
> 1 file changed, 8 insertions(+), 13 deletions(-)

Reviewed-by: David Hildenbrand <david@xxxxxxxxxx>


--
Thanks,

David / dhildenb