From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wi0-f169.google.com (mail-wi0-f169.google.com [209.85.212.169]) by kanga.kvack.org (Postfix) with ESMTP id 1BD1F6B0038 for ; Wed, 7 Jan 2015 04:11:50 -0500 (EST) Received: by mail-wi0-f169.google.com with SMTP id r20so7623305wiv.0 for ; Wed, 07 Jan 2015 01:11:49 -0800 (PST) Received: from mx2.suse.de (cantor2.suse.de. [195.135.220.15]) by mx.google.com with ESMTPS id t3si3427995wiw.92.2015.01.07.01.11.48 for (version=TLSv1 cipher=ECDHE-RSA-RC4-SHA bits=128/128); Wed, 07 Jan 2015 01:11:48 -0800 (PST) Message-ID: <54ACF851.3040002@suse.cz> Date: Wed, 07 Jan 2015 10:11:45 +0100 From: Vlastimil Babka MIME-Version: 1.0 Subject: Re: [PATCH V4 3/4] mm: reduce try_to_compact_pages parameters References: <1420478263-25207-1-git-send-email-vbabka@suse.cz> <1420478263-25207-4-git-send-email-vbabka@suse.cz> <20150106145710.GD20860@dhcp22.suse.cz> In-Reply-To: <20150106145710.GD20860@dhcp22.suse.cz> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Sender: owner-linux-mm@kvack.org List-ID: To: Michal Hocko Cc: Andrew Morton , linux-mm@kvack.org, linux-kernel@vger.kernel.org, Mel Gorman , Zhang Yanfei , Minchan Kim , David Rientjes , Rik van Riel , "Aneesh Kumar K.V" , "Kirill A. Shutemov" , Johannes Weiner , Joonsoo Kim On 01/06/2015 03:57 PM, Michal Hocko wrote: > Hmm, wait a minute > > On Mon 05-01-15 18:17:42, Vlastimil Babka wrote: > [...] >> -unsigned long try_to_compact_pages(struct zonelist *zonelist, >> - int order, gfp_t gfp_mask, nodemask_t *nodemask, >> - enum migrate_mode mode, int *contended, >> - int alloc_flags, int classzone_idx) >> +unsigned long try_to_compact_pages(gfp_t gfp_mask, unsigned int order, >> + int alloc_flags, const struct alloc_context *ac, >> + enum migrate_mode mode, int *contended) >> { >> - enum zone_type high_zoneidx = gfp_zone(gfp_mask); >> int may_enter_fs = gfp_mask & __GFP_FS; >> int may_perform_io = gfp_mask & __GFP_IO; >> struct zoneref *z; > > gfp_mask might change since the high_zoneidx was set up in the call > chain. I guess this shouldn't change to the gfp_zone output but it is > worth double checking. Yeah I checked that. gfp_zone() operates just on GFP_ZONEMASK part of the flags, and we don't change that. >> @@ -1365,8 +1363,8 @@ unsigned long try_to_compact_pages(struct zonelist *zonelist, >> return COMPACT_SKIPPED; >> >> /* Compact each zone in the list */ >> - for_each_zone_zonelist_nodemask(zone, z, zonelist, high_zoneidx, >> - nodemask) { >> + for_each_zone_zonelist_nodemask(zone, z, ac->zonelist, ac->high_zoneidx, >> + ac->nodemask) { >> int status; >> int zone_contended; >> > -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: email@kvack.org