From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mel Gorman Subject: Re: [PATCH 08/16] mm: page_alloc: Only check the alloc flags and gfp_mask for dirty once Date: Sat, 19 Apr 2014 12:19:34 +0100 Message-ID: <20140419111934.GC4225@suse.de> References: <1397832643-14275-1-git-send-email-mgorman@suse.de> <1397832643-14275-9-git-send-email-mgorman@suse.de> <20140418180836.GE29210@cmpxchg.org> Mime-Version: 1.0 Content-Type: text/plain; charset=iso-8859-15 Cc: Linux-MM , Linux-FSDevel To: Johannes Weiner Return-path: Content-Disposition: inline In-Reply-To: <20140418180836.GE29210@cmpxchg.org> Sender: owner-linux-mm@kvack.org List-Id: linux-fsdevel.vger.kernel.org On Fri, Apr 18, 2014 at 02:08:36PM -0400, Johannes Weiner wrote: > On Fri, Apr 18, 2014 at 03:50:35PM +0100, Mel Gorman wrote: > > Currently it's calculated once per zone in the zonelist. > > > > Signed-off-by: Mel Gorman > > I would have assumed the compiler can detect such a loop invariant... > Alas, > Surprisingly it didn't in my case but the benefit of the patch is marginal at best. I can drop it if it makes the code more obscure to peoples eyes. -- Mel Gorman SUSE Labs -- 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