Re: [PATCH v2 2/2] mm/huge_memory: Remove useless folio pointers passing

[Zi Yan]

On 25 Apr 2025, at 6:38, Gavin Guo wrote:

> Since the previous commit "mm/huge_memory: Adjust try_to_migrate_one() and
> split_huge_pmd_locked()" has simplified the logic by leveraging the
> folio verification in page_vma_mapped_walk(), this patch removes the
> unnecessary folio pointers passing.
>
> Suggested-by: David Hildenbrand <david@xxxxxxxxxx>
> Link: https://lore.kernel.org/all/98d1d195-7821-4627-b518-83103ade56c0@xxxxxxxxxx/
> Link: https://lore.kernel.org/all/91599a3c-e69e-4d79-bac5-5013c96203d7@xxxxxxxxxx/
> Signed-off-by: Gavin Guo <gavinguo@xxxxxxxxxx>
> Acked-by: David Hildenbrand <david@xxxxxxxxxx>
> ---
>  include/linux/huge_mm.h | 15 +++++++--------
>  mm/huge_memory.c        | 16 ++++++++--------
>  mm/memory.c             |  4 ++--
>  mm/mprotect.c           |  2 +-
>  mm/rmap.c               |  4 ++--
>  5 files changed, 20 insertions(+), 21 deletions(-)
>

LGTM. Thanks. Reviewed-by: Zi Yan <ziy@xxxxxxxxxx>

--
Best Regards,
Yan, Zi