clean up comment. prepare cgroup return 0 or -ENOMEN, others return -EAGAIN. avoid conflict meanings. Signed-off-by: Figo.zhang <figo1802@xxxxxxxxx> --- mm/migrate.c | 3 +-- 1 files changed, 1 insertions(+), 2 deletions(-) diff --git a/mm/migrate.c b/mm/migrate.c index 34132f8..d65b351 100644 --- a/mm/migrate.c +++ b/mm/migrate.c @@ -647,7 +647,6 @@ static int unmap_and_move(new_page_t get_new_page, unsigned long private, if (unlikely(split_huge_page(page))) goto move_newpage; - /* prepare cgroup just returns 0 or -ENOMEM */ rc = -EAGAIN; if (!trylock_page(page)) { @@ -687,7 +686,7 @@ static int unmap_and_move(new_page_t get_new_page, unsigned long private, goto unlock; } - /* charge against new page */ + /* charge against new page, return 0 or -ENOMEM */ charge = mem_cgroup_prepare_migration(page, newpage, &mem, GFP_KERNEL); if (charge == -ENOMEM) { rc = -ENOMEM; -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@xxxxxxxxxx For more info on Linux MM, see: http://www.linux-mm.org/ . Fight unfair telecom internet charges in Canada: sign http://stopthemeter.ca/ Don't email: <a href=mailto:"dont@xxxxxxxxx"> email@xxxxxxxxx </a>