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 <iamjoonsoo.kim@lge.com> Signed-off-by: Andrew Morton <akpm@linux-foundation.org> Reviewed-by: Mike Kravetz <mike.kravetz@oracle.com> Reviewed-by: Vlastimil Babka <vbabka@suse.cz> Acked-by: Michal Hocko <mhocko@suse.com> Cc: Christoph Hellwig <hch@infradead.org> Cc: Naoya Horiguchi <n-horiguchi@ah.jp.nec.com> Cc: Roman Gushchin <guro@fb.com> Link: http://lkml.kernel.org/r/1594622517-20681-4-git-send-email-iamjoonsoo.kim@lge.com Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Este cometimento está contido em:

cometido por
Linus Torvalds

ascendente
b4b382238e
cometimento
d92bbc2719
11
mm/migrate.c
11
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;
|
||||
|
Criar uma nova questão referindo esta
Bloquear um utilizador