From: Naoya Horiguchi <n-horigu...@ah.jp.nec.com>

This patch enables thp migration for move_pages(2).

Signed-off-by: Naoya Horiguchi <n-horigu...@ah.jp.nec.com>

ChangeLog: v1 -> v5:
- fix page counting

Signed-off-by: Zi Yan <zi....@cs.rutgers.edu>
---
 mm/migrate.c | 47 +++++++++++++++++++++++++++++++++--------------
 1 file changed, 33 insertions(+), 14 deletions(-)

diff --git a/mm/migrate.c b/mm/migrate.c
index bbc856264b69..f7c1a1999c8e 100644
--- a/mm/migrate.c
+++ b/mm/migrate.c
@@ -184,8 +184,8 @@ void putback_movable_pages(struct list_head *l)
                        put_page(page);
                } else {
                        putback_lru_page(page);
-                       dec_node_page_state(page, NR_ISOLATED_ANON +
-                                       page_is_file_cache(page));
+                       mod_node_page_state(page_pgdat(page), NR_ISOLATED_ANON +
+                                       page_is_file_cache(page), 
-hpage_nr_pages(page));
                }
        }
 }
@@ -1141,8 +1141,8 @@ static ICE_noinline int unmap_and_move(new_page_t 
get_new_page,
                 * as __PageMovable
                 */
                if (likely(!__PageMovable(page)))
-                       dec_node_page_state(page, NR_ISOLATED_ANON +
-                                       page_is_file_cache(page));
+                       mod_node_page_state(page_pgdat(page), NR_ISOLATED_ANON +
+                                       page_is_file_cache(page), 
-hpage_nr_pages(page));
        }
 
        /*
@@ -1159,7 +1159,7 @@ static ICE_noinline int unmap_and_move(new_page_t 
get_new_page,
                         * it's how HWPoison flag works at the moment.
                         */
                        if (!test_set_page_hwpoison(page))
-                               num_poisoned_pages_inc();
+                               num_poisoned_pages_add(hpage_nr_pages(page));
                }
        } else {
                if (rc != -EAGAIN) {
@@ -1414,7 +1414,17 @@ static struct page *new_page_node(struct page *p, 
unsigned long private,
        if (PageHuge(p))
                return alloc_huge_page_node(page_hstate(compound_head(p)),
                                        pm->node);
-       else
+       else if (thp_migration_supported() && PageTransHuge(p)) {
+               struct page *thp;
+
+               thp = alloc_pages_node(pm->node,
+                       (GFP_TRANSHUGE | __GFP_THISNODE) & ~__GFP_RECLAIM,
+                       HPAGE_PMD_ORDER);
+               if (!thp)
+                       return NULL;
+               prep_transhuge_page(thp);
+               return thp;
+       } else
                return __alloc_pages_node(pm->node,
                                GFP_HIGHUSER_MOVABLE | __GFP_THISNODE, 0);
 }
@@ -1441,6 +1451,8 @@ static int do_move_page_to_node_array(struct mm_struct 
*mm,
        for (pp = pm; pp->node != MAX_NUMNODES; pp++) {
                struct vm_area_struct *vma;
                struct page *page;
+               struct page *head;
+               unsigned int follflags;
 
                err = -EFAULT;
                vma = find_vma(mm, pp->addr);
@@ -1448,8 +1460,10 @@ static int do_move_page_to_node_array(struct mm_struct 
*mm,
                        goto set_status;
 
                /* FOLL_DUMP to ignore special (like zero) pages */
-               page = follow_page(vma, pp->addr,
-                               FOLL_GET | FOLL_SPLIT | FOLL_DUMP);
+               follflags = FOLL_GET | FOLL_DUMP;
+               if (!thp_migration_supported())
+                       follflags |= FOLL_SPLIT;
+               page = follow_page(vma, pp->addr, follflags);
 
                err = PTR_ERR(page);
                if (IS_ERR(page))
@@ -1459,7 +1473,6 @@ static int do_move_page_to_node_array(struct mm_struct 
*mm,
                if (!page)
                        goto set_status;
 
-               pp->page = page;
                err = page_to_nid(page);
 
                if (err == pp->node)
@@ -1474,16 +1487,22 @@ static int do_move_page_to_node_array(struct mm_struct 
*mm,
                        goto put_and_set;
 
                if (PageHuge(page)) {
-                       if (PageHead(page))
+                       if (PageHead(page)) {
                                isolate_huge_page(page, &pagelist);
+                               err = 0;
+                               pp->page = page;
+                       }
                        goto put_and_set;
                }
 
-               err = isolate_lru_page(page);
+               pp->page = compound_head(page);
+               head = compound_head(page);
+               err = isolate_lru_page(head);
                if (!err) {
-                       list_add_tail(&page->lru, &pagelist);
-                       inc_node_page_state(page, NR_ISOLATED_ANON +
-                                           page_is_file_cache(page));
+                       list_add_tail(&head->lru, &pagelist);
+                       mod_node_page_state(page_pgdat(head),
+                               NR_ISOLATED_ANON + page_is_file_cache(head),
+                               hpage_nr_pages(head));
                }
 put_and_set:
                /*
-- 
2.11.0

Reply via email to