From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753252AbdARRsp (ORCPT ); Wed, 18 Jan 2017 12:48:45 -0500 Received: from outbound-smtp08.blacknight.com ([46.22.139.13]:50043 "EHLO outbound-smtp08.blacknight.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753023AbdARRsm (ORCPT ); Wed, 18 Jan 2017 12:48:42 -0500 Date: Wed, 18 Jan 2017 17:48:39 +0000 From: Mel Gorman To: Michal Hocko Cc: Trevor Cordes , linux-kernel@vger.kernel.org, Joonsoo Kim , Minchan Kim , Rik van Riel , Srikar Dronamraju Subject: Re: mm, vmscan: commit makes PAE kernel crash nightly (bisected) Message-ID: <20170118174839.o2vjdfwux7otc456@techsingularity.net> References: <20170111103243.GA27795@pog.tecnopolis.ca> <20170111121146.i53jyerxont7b74i@techsingularity.net> <20170111121428.4sswbkzyqz7vf6so@techsingularity.net> <20170111165232.0b7abe18@pog.tecnopolis.ca> <20170112093613.GF2264@dhcp22.suse.cz> <20170115002752.4559f668@pog.tecnopolis.ca> <20170116110934.7zopy3ecg2lfadkd@techsingularity.net> <20170117135228.GN19699@dhcp22.suse.cz> <20170117142114.r7abr3x2bbik47sd@techsingularity.net> <20170117145450.GQ19699@dhcp22.suse.cz> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-15 Content-Disposition: inline In-Reply-To: <20170117145450.GQ19699@dhcp22.suse.cz> 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 03:54:51PM +0100, Michal Hocko wrote: > On Tue 17-01-17 14:21:14, Mel Gorman wrote: > > On Tue, Jan 17, 2017 at 02:52:28PM +0100, Michal Hocko wrote: > > > On Mon 16-01-17 11:09:34, Mel Gorman wrote: > > > [...] > > > > diff --git a/mm/vmscan.c b/mm/vmscan.c > > > > index 532a2a750952..46aac487b89a 100644 > > > > --- a/mm/vmscan.c > > > > +++ b/mm/vmscan.c > > > > @@ -2684,6 +2684,7 @@ static void shrink_zones(struct zonelist *zonelist, struct scan_control *sc) > > > > continue; > > > > > > > > if (sc->priority != DEF_PRIORITY && > > > > + !buffer_heads_over_limit && > > > > !pgdat_reclaimable(zone->zone_pgdat)) > > > > continue; /* Let kswapd poll it */ > > > > > > I think we should rather remove pgdat_reclaimable here. This sounds like > > > a wrong layer to decide whether we want to reclaim and how much. > > > > > > > I had considered that but it'd also be important to add the other 32-bit > > patches you have posted to see the impact. Because of the ratio of LRU pages > > to slab pages, it may not have an impact but it'd need to be eliminated. > > OK, Trevor you can pull from > git://git.kernel.org/pub/scm/linux/kernel/git/mhocko/mm.git tree > fixes/highmem-node-fixes branch. This contains the current mmotm tree + > the latest highmem fixes. I also do not expect this would help much in > your case but as Mel've said we should rule that out at least. > After considering slab shrinking of lower nodes, it occurs to me that your fixes also impacts slab shrinking. For lowmem-constrained allocations, we accounted for scans on the lower zones but shrunk slabs proportional to the total LRU size. If the lower zones had few LRU pages and were mostly slab pages then the proportional calculation would be way off. This may have a bigger impact on Trevor Cordes' situation that I had imagined at the start of today. -- Mel Gorman SUSE Labs