From mboxrd@z Thu Jan 1 00:00:00 1970 From: Johannes Weiner Subject: Re: [PATCH 06/16] mm: page_alloc: Calculate classzone_idx once from the zonelist ref Date: Fri, 18 Apr 2014 14:03:09 -0400 Message-ID: <20140418180309.GC29210@cmpxchg.org> References: <1397832643-14275-1-git-send-email-mgorman@suse.de> <1397832643-14275-7-git-send-email-mgorman@suse.de> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Linux-MM , Linux-FSDevel To: Mel Gorman Return-path: Content-Disposition: inline In-Reply-To: <1397832643-14275-7-git-send-email-mgorman@suse.de> Sender: owner-linux-mm@kvack.org List-Id: linux-fsdevel.vger.kernel.org On Fri, Apr 18, 2014 at 03:50:33PM +0100, Mel Gorman wrote: > @@ -2463,7 +2462,7 @@ static inline struct page * > __alloc_pages_slowpath(gfp_t gfp_mask, unsigned int order, > struct zonelist *zonelist, enum zone_type high_zoneidx, > nodemask_t *nodemask, struct zone *preferred_zone, > - int migratetype) > + int classzone_idx, int migratetype) > { > const gfp_t wait = gfp_mask & __GFP_WAIT; > struct page *page = NULL; There is another potential update of preferred_zone in this function after which the classzone_idx should probably be refreshed: /* * Find the true preferred zone if the allocation is unconstrained by * cpusets. */ if (!(alloc_flags & ALLOC_CPUSET) && !nodemask) first_zones_zonelist(zonelist, high_zoneidx, NULL, &preferred_zone); -- 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