From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751302AbdAQLNl (ORCPT ); Tue, 17 Jan 2017 06:13:41 -0500 Received: from mx2.suse.de ([195.135.220.15]:49806 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750958AbdAQLNk (ORCPT ); Tue, 17 Jan 2017 06:13:40 -0500 Date: Tue, 17 Jan 2017 11:13:16 +0000 From: Mel Gorman To: Michal Hocko Cc: Andrew Morton , Johannes Weiner , Minchan Kim , Hillf Danton , linux-mm@kvack.org, LKML Subject: Re: [PATCH 0/3] follow up nodereclaim for 32b fix Message-ID: <20170117111316.6eakdx7ow6yodtf2@suse.de> References: <20170117103702.28542-1-mhocko@kernel.org> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-15 Content-Disposition: inline In-Reply-To: <20170117103702.28542-1-mhocko@kernel.org> User-Agent: Mutt/1.6.2 (2016-07-01) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Jan 17, 2017 at 11:36:59AM +0100, Michal Hocko wrote: > Hi, > I have previously posted this as an RFC [1] but there didn't seem to be > any objections other than some requests to reorganize the changes in > a slightly different way so I am reposting the series and asking for > inclusion. > > This is a follow up on top of [2]. The patch 1 cleans up the code a bit. > I haven't seen any real issues or bug reports but conceptualy ignoring > the maximum eligible zone in get_scan_count is wrong by definition. This > is what patch 2 does. Patch 3 removes inactive_reclaimable_pages > which was a kind of hack around for the problem which should have been > addressed at get_scan_count. > > There is one more place which needs a special handling which is not > a part of this series. too_many_isolated can get confused as well. I > already have some preliminary work but it still needs some testing so I > will post it separatelly. > > Michal Hocko (3): > mm, vmscan: cleanup lru size claculations > mm, vmscan: consider eligible zones in get_scan_count > Revert "mm: bail out in shrink_inactive_list()" > Acked-by: Mel Gorman -- Mel Gorman SUSE Labs