Messages in this thread | | | Subject | Re: [PATCH v4 03/11] mm/hugetlb: unify migration callbacks | From | Vlastimil Babka <> | Date | Tue, 7 Jul 2020 13:05:31 +0200 |
| |
On 7/7/20 9:44 AM, js1304@gmail.com wrote: > From: Joonsoo Kim <iamjoonsoo.kim@lge.com> > > 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. > > Reviewed-by: Mike Kravetz <mike.kravetz@oracle.com> > Signed-off-by: Joonsoo Kim <iamjoonsoo.kim@lge.com>
Yeah, this version looks very good :) Reviewed-by: Vlastimil Babka <vbabka@suse.cz>
Thanks!
| |