Chen Wandun <chenwandun@xxxxxxxxxx> writes: > migrate_hugetlbs returns -ENOMEM when no enough hugetlb, > however there may be free non-hugetlb folios available, > so continue to migrate for non-hugetlb folios. > > Nowdays the only negtive return value of migrate_hugetlbs > is -ENOMEM, keep compatible for coming negtive return value > instead of ignore return value. > > Signed-off-by: Chen Wandun <chenwandun@xxxxxxxxxx> LGTM, Thanks! Reviewed-by: "Huang, Ying" <ying.huang@xxxxxxxxx> > > --- > v1 --> v2: modify changelog and comment. thanks Huang Ying. > --- > mm/migrate.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/mm/migrate.c b/mm/migrate.c > index b971edbf32fc..68927aa06f9b 100644 > --- a/mm/migrate.c > +++ b/mm/migrate.c > @@ -1939,7 +1939,8 @@ int migrate_pages(struct list_head *from, new_page_t get_new_page, > > rc_gather = migrate_hugetlbs(from, get_new_page, put_new_page, private, > mode, reason, &stats, &ret_folios); > - if (rc_gather < 0) > + /* There may be free non-hugetlb folios available, continue to migrate. */ > + if (rc_gather < 0 && rc_gather != -ENOMEM) > goto out; > again: > nr_pages = 0;
