From mboxrd@z Thu Jan 1 00:00:00 1970 From: Rik van Riel Subject: Re: [PATCH 3/3] mm: vmscan: Use proportional scanning during direct reclaim and full scan at DEF_PRIORITY Date: Thu, 22 May 2014 12:04:12 -0400 Message-ID: <537E1FFC.40608@redhat.com> References: <1400749779-24879-1-git-send-email-mgorman@suse.de> <1400749779-24879-4-git-send-email-mgorman@suse.de> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Cc: Johannes Weiner , Hugh Dickins , Tim Chen , Dave Chinner , Yuanhan Liu , Bob Liu , Jan Kara , Linux Kernel , Linux-MM , Linux-FSDevel To: Mel Gorman , Andrew Morton Return-path: Received: from mx1.redhat.com ([209.132.183.28]:32075 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750955AbaEVQEZ (ORCPT ); Thu, 22 May 2014 12:04:25 -0400 In-Reply-To: <1400749779-24879-4-git-send-email-mgorman@suse.de> Sender: linux-fsdevel-owner@vger.kernel.org List-ID: On 05/22/2014 05:09 AM, Mel Gorman wrote: > Commit "mm: vmscan: obey proportional scanning requirements for kswapd" > ensured that file/anon lists were scanned proportionally for reclaim from > kswapd but ignored it for direct reclaim. The intent was to minimse direct > reclaim latency but Yuanhan Liu pointer out that it substitutes one long > stall for many small stalls and distorts aging for normal workloads like > streaming readers/writers. Hugh Dickins pointed out that a side-effect of > the same commit was that when one LRU list dropped to zero that the entirety > of the other list was shrunk leading to excessive reclaim in memcgs. > This patch scans the file/anon lists proportionally for direct reclaim > to similarly age page whether reclaimed by kswapd or direct reclaim but > takes care to abort reclaim if one LRU drops to zero after reclaiming the > requested number of pages. > > Note that there are fewer allocation stalls even though the amount > of direct reclaim scanning is very approximately the same. > > Signed-off-by: Mel Gorman Acked-by: Rik van Riel -- All rights reversed