2021-09-22 04:50:43

by John Hubbard

[permalink] [raw]
Subject: Re: [PATCH v2 1/2] mm/debug: sync up MR_CONTIG_RANGE and MR_LONGTERM_PIN

On 9/20/21 23:45, Weizhao Ouyang wrote:
> Sync up MR_CONTIG_RANGE and MR_LONGTERM_PIN to migrate_reason_names.
>
> Fixes: 310253514bbf ("mm/migrate: rename migration reason MR_CMA to MR_CONTIG_RANGE")
> Fixes: d1e153fea2a8 ("mm/gup: migrate pinned pages out of movable zone")
> Cc: [email protected]
> Signed-off-by: Weizhao Ouyang <[email protected]>
> Reviewed-by: "Huang, Ying" <[email protected]>
> ---
> mm/debug.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/mm/debug.c b/mm/debug.c
> index e73fe0a8ec3d..e61037cded98 100644
> --- a/mm/debug.c
> +++ b/mm/debug.c
> @@ -24,7 +24,8 @@ const char *migrate_reason_names[MR_TYPES] = {
> "syscall_or_cpuset",
> "mempolicy_mbind",
> "numa_misplaced",
> - "cma",
> + "contig_range",
> + "longterm_pin",
> };
>
> const struct trace_print_flags pageflag_names[] = {
>

Looks good.

Reviewed-by: John Hubbard <[email protected]>


thanks,
--
John Hubbard
NVIDIA