From: Hugh Dickins do_try_to_free_pages returns 0 or 1, and try_to_free_mem_cgroup_pages returns 0 or 1 accordingly: let's accomplish that in the obvious way. Signed-off-by: Hugh Dickins Acked-by: Mel Gorman Cc: Christoph Lameter Cc: David Rientjes Cc: Lee Schermerhorn Cc: KAMEZAWA Hiroyuki Cc: Nick Piggin Cc: Balbir Singh Cc: Pavel Emelyanov Signed-off-by: Andrew Morton --- mm/vmscan.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff -puN mm/vmscan.c~mm-have-zonelist-contains-structs-with-both-a-zone-pointer-and-zone_idx-just-return-do_try_to_free_pages mm/vmscan.c --- a/mm/vmscan.c~mm-have-zonelist-contains-structs-with-both-a-zone-pointer-and-zone_idx-just-return-do_try_to_free_pages +++ a/mm/vmscan.c @@ -1432,9 +1432,7 @@ unsigned long try_to_free_mem_cgroup_pag sc.gfp_mask = (gfp_mask & GFP_RECLAIM_MASK) | (GFP_HIGHUSER_MOVABLE & ~GFP_RECLAIM_MASK); zonelist = NODE_DATA(numa_node_id())->node_zonelists; - if (do_try_to_free_pages(zonelist, sc.gfp_mask, &sc)) - return 1; - return 0; + return do_try_to_free_pages(zonelist, sc.gfp_mask, &sc); } #endif _