path: root/mm/migrate.c
diff options
authorJoonsoo Kim <>2020-08-11 18:37:17 -0700
committerLinus Torvalds <>2020-08-12 10:58:02 -0700
commitd92bbc2719bd2be237ee336113b63492a6baca3b (patch)
treef1d571ea966d80dd7bb0538822d4b884c4036274 /mm/migrate.c
parentb4b382238ed2f94f0d3860f9120b66404fa99463 (diff)
mm/hugetlb: unify migration callbacks
There is no difference between two migration callback functions, alloc_huge_page_node() and alloc_huge_page_nodemask(), except __GFP_THISNODE handling. It's redundant to have two almost similar functions in order to handle this flag. So, this patch tries to remove one by introducing a new argument, gfp_mask, to alloc_huge_page_nodemask(). After introducing gfp_mask argument, it's caller's job to provide correct gfp_mask. So, every callsites for alloc_huge_page_nodemask() are changed to provide gfp_mask. Note that it's safe to remove a node id check in alloc_huge_page_node() since there is no caller passing NUMA_NO_NODE as a node id. Signed-off-by: Joonsoo Kim <> Signed-off-by: Andrew Morton <> Reviewed-by: Mike Kravetz <> Reviewed-by: Vlastimil Babka <> Acked-by: Michal Hocko <> Cc: Christoph Hellwig <> Cc: Naoya Horiguchi <> Cc: Roman Gushchin <> Link: Signed-off-by: Linus Torvalds <>
Diffstat (limited to 'mm/migrate.c')
1 files changed, 7 insertions, 4 deletions
diff --git a/mm/migrate.c b/mm/migrate.c
index 5269bc520aee..8d084e9bc13b 100644
--- a/mm/migrate.c
+++ b/mm/migrate.c
@@ -1545,10 +1545,13 @@ struct page *new_page_nodemask(struct page *page,
unsigned int order = 0;
struct page *new_page = NULL;
- if (PageHuge(page))
- return alloc_huge_page_nodemask(
- page_hstate(compound_head(page)),
- preferred_nid, nodemask);
+ if (PageHuge(page)) {
+ struct hstate *h = page_hstate(compound_head(page));
+ gfp_mask = htlb_alloc_mask(h);
+ return alloc_huge_page_nodemask(h, preferred_nid,
+ nodemask, gfp_mask);
+ }
if (PageTransHuge(page)) {
gfp_mask |= GFP_TRANSHUGE;