Re: [PATCH] Removed unused argument vma from migrate_misplaced_folio

From: Zi Yan
Date: Tue Nov 26 2024 - 09:38:18 EST


On 25 Nov 2024, at 2:57, Donet Tom wrote:

> Commit ee86814b0562 ("mm/migrate: move NUMA hinting fault folio
> isolation + checks under PTL") removed the code that had used
> the vma argument in migrate_misplaced_folio.
>
> Since the vma argument was no longer used in migrate_misplaced_folio,
> this patch removed it.
>
> Signed-off-by: Donet Tom <donettom@xxxxxxxxxxxxx>
> ---
> include/linux/migrate.h | 6 ++----
> mm/huge_memory.c | 2 +-
> mm/memory.c | 2 +-
> mm/migrate.c | 3 +--
> 4 files changed, 5 insertions(+), 8 deletions(-)

Like Baolin suggested in another email, "mm/migrate:" prefix should be
used. Otherwise, LGTM. Thanks. Reviewed-by: Zi Yan <ziy@xxxxxxxxxx>

--
Best Regards,
Yan, Zi