On Fri 20-01-17 13:35:40, Vlastimil Babka wrote: > On 01/19/2017 06:07 PM, Lucas Stach wrote: [...] > > @@ -7255,7 +7256,7 @@ int alloc_contig_range(unsigned long start, unsigned long end, > > .zone = page_zone(pfn_to_page(start)), > > .mode = MIGRATE_SYNC, > > .ignore_skip_hint = true, > > - .gfp_mask = GFP_KERNEL, > > + .gfp_mask = gfp_mask, > > I think you should apply memalloc_noio_flags() here (and Michal should > then convert it to the new name in his scoped gfp_nofs series). Note > that then it's technically a functional change, but it's needed. > Otherwise looks good. yes, with that added, feel free to add Acked-by: Michal Hocko <mhocko@xxxxxxxx> > > > }; > > INIT_LIST_HEAD(&cc.migratepages); > > > > -- Michal Hocko SUSE Labs
