From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wm0-f69.google.com (mail-wm0-f69.google.com [74.125.82.69]) by kanga.kvack.org (Postfix) with ESMTP id 741566B0033 for ; Mon, 6 Feb 2017 17:24:23 -0500 (EST) Received: by mail-wm0-f69.google.com with SMTP id x4so22025788wme.3 for ; Mon, 06 Feb 2017 14:24:23 -0800 (PST) Received: from outbound-smtp04.blacknight.com (outbound-smtp04.blacknight.com. [81.17.249.35]) by mx.google.com with ESMTPS id v203si9705337wmb.51.2017.02.06.14.24.22 for (version=TLS1 cipher=AES128-SHA bits=128/128); Mon, 06 Feb 2017 14:24:22 -0800 (PST) Received: from mail.blacknight.com (pemlinmail01.blacknight.ie [81.17.254.10]) by outbound-smtp04.blacknight.com (Postfix) with ESMTPS id 1CA1F9892E for ; Mon, 6 Feb 2017 22:24:22 +0000 (UTC) Date: Mon, 6 Feb 2017 22:24:21 +0000 From: Mel Gorman Subject: Re: [PATCH] vmscan: fix zone balance check in prepare_kswapd_sleep Message-ID: <20170206222421.wgrd3gih6hugvraf@techsingularity.net> References: <719282122.1183240.1486298780546.ref@mail.yahoo.com> <719282122.1183240.1486298780546@mail.yahoo.com> <20170206161715.sfz6lm3vmahlnxx6@techsingularity.net> <3a10b870-2dd6-74c2-75c8-92823e2ba4e1@suse.cz> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-15 Content-Disposition: inline In-Reply-To: <3a10b870-2dd6-74c2-75c8-92823e2ba4e1@suse.cz> Sender: owner-linux-mm@kvack.org List-ID: To: Vlastimil Babka Cc: Shantanu Goel , "linux-mm@kvack.org" On Mon, Feb 06, 2017 at 06:43:08PM +0100, Vlastimil Babka wrote: > On 02/06/2017 05:17 PM, Mel Gorman wrote: > > > > > > > Thanks, > > > Shantanu > > > > > From 46f2e4b02ac263bf50d69cdab3bcbd7bcdea7415 Mon Sep 17 00:00:00 2001 > > > From: Shantanu Goel > > > Date: Sat, 4 Feb 2017 19:07:53 -0500 > > > Subject: [PATCH] vmscan: fix zone balance check in prepare_kswapd_sleep > > > > > > The check in prepare_kswapd_sleep needs to match the one in balance_pgdat > > > since the latter will return as soon as any one of the zones in the > > > classzone is above the watermark. This is specially important for > > > higher order allocations since balance_pgdat will typically reset > > > the order to zero relying on compaction to create the higher order > > > pages. Without this patch, prepare_kswapd_sleep fails to wake up > > > kcompactd since the zone balance check fails. > > > > > > Signed-off-by: Shantanu Goel > > > > I don't recall specifically why I made that change but I've no objections > > to the patch so; > > > > Acked-by: Mel Gorman > > > > However, note that there is a slight risk that kswapd will sleep for a > > short interval early due to a very small zone such as ZONE_DMA. If this > > is a general problem then it'll manifest as less kswapd reclaim and more > > direct reclaim. If it turns out this is an issue then a revert will not > > be the right fix. Instead, all the checks for zone_balance will need to > > account for the only balanced zone being a tiny percentage of memory in > > the node. > > Hopefully the lowmem reserves should take care of this in that case? They > easily make a low zone inaccessible even when fully free. Unless the small > zone is high one though, such as Normal zone on system with only 4GB memory, > so most of it is in DMA32. More than likely, it'll be ok. It'll simply be something to check if direct reclaim goes through the roof and it bisects to this patch for some reason. -- 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