From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932374Ab0FPXqv (ORCPT ); Wed, 16 Jun 2010 19:46:51 -0400 Received: from mx1.redhat.com ([209.132.183.28]:1025 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755556Ab0FPXqu (ORCPT ); Wed, 16 Jun 2010 19:46:50 -0400 Message-ID: <4C196219.6000901@redhat.com> Date: Wed, 16 Jun 2010 19:45:29 -0400 From: Rik van Riel User-Agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.1.9) Gecko/20100430 Fedora/3.0.4-2.fc12 Lightning/1.0b2pre Thunderbird/3.0.4 MIME-Version: 1.0 To: Andrew Morton CC: Mel Gorman , linux-kernel@vger.kernel.org, linux-fsdevel@vger.kernel.org, linux-mm@kvack.org, Dave Chinner , Chris Mason , Nick Piggin , Johannes Weiner , Christoph Hellwig , KAMEZAWA Hiroyuki Subject: Re: [PATCH 05/12] vmscan: kill prev_priority completely References: <1276514273-27693-1-git-send-email-mel@csn.ul.ie> <1276514273-27693-6-git-send-email-mel@csn.ul.ie> <20100616163709.1e0f6b56.akpm@linux-foundation.org> In-Reply-To: <20100616163709.1e0f6b56.akpm@linux-foundation.org> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 06/16/2010 07:37 PM, Andrew Morton wrote: > This would have been badder in earlier days when we were using the > scanning priority to decide when to start unmapping pte-mapped pages - > page reclaim would have been recirculating large blobs of mapped pages > around the LRU until the priority had built to the level where we > started to unmap them. > > However that priority-based decision got removed and right now I don't > recall what it got replaced with. Aren't we now unmapping pages way > too early and suffering an increased major&minor fault rate? Worried. We keep a different set of statistics to decide whether to reclaim only page cache pages, or both page cache and anonymous pages. The function get_scan_ratio parses those statistics. > So. What's up with that? I don't even remember _why_ we disable > the swap token once the scanning priority gets severe and the code > comments there are risible. And why do we wait until priority==0 > rather than priority==1? The reason is that we never page out the pages belonging to the process owning the swap token (with the exception of that process evicting its own pages). If that process has a really large RSS in the current zone, and we are having problems freeing pages, it may be beneficial to also evict pages from that process. Now that the LRU lists are split out into file backed and swap backed, it may be a lot easier to find pages to evict. That may mean we could notice we're getting into trouble at much higher priority levels and disable the swap token at a higher priority level. I do not believe prev_priority will be very useful here, since we'd like to start out with small scans whenever possible. > - Busted prev_priority means that lumpy reclaim will act oddly. > Every time someone goes into do some recalim, they'll start out not > doing lumpy reclaim. Then, after a while, they'll get a clue and > will start doing the lumpy thing. Then they return from reclaim and > the next recalim caller will again forget that he should have done > lumpy reclaim. How common are lumpy reclaims, anyway? Isn't it more likely that in-between every two higher-order reclaims, a number of order zero reclaims will be happening? In that case, the prev_priority logic may have introduced the kind of behavioural bug you describe above... > And one has to wonder: if we're making these incorrect decisions based > upon a bogus view of the current scanning difficulty, why are these > various priority-based thresholding heuristics even in there? Are they > doing anything useful? The prev_priority code was useful when we had filesystem and swap backed pages mixed on the same LRU list. I am not convinced it still has any use. -- All rights reversed From mboxrd@z Thu Jan 1 00:00:00 1970 From: Rik van Riel Subject: Re: [PATCH 05/12] vmscan: kill prev_priority completely Date: Wed, 16 Jun 2010 19:45:29 -0400 Message-ID: <4C196219.6000901@redhat.com> References: <1276514273-27693-1-git-send-email-mel@csn.ul.ie> <1276514273-27693-6-git-send-email-mel@csn.ul.ie> <20100616163709.1e0f6b56.akpm@linux-foundation.org> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Cc: Mel Gorman , linux-kernel@vger.kernel.org, linux-fsdevel@vger.kernel.org, linux-mm@kvack.org, Dave Chinner , Chris Mason , Nick Piggin , Johannes Weiner , Christoph Hellwig , KAMEZAWA Hiroyuki To: Andrew Morton Return-path: In-Reply-To: <20100616163709.1e0f6b56.akpm@linux-foundation.org> Sender: owner-linux-mm@kvack.org List-Id: linux-fsdevel.vger.kernel.org On 06/16/2010 07:37 PM, Andrew Morton wrote: > This would have been badder in earlier days when we were using the > scanning priority to decide when to start unmapping pte-mapped pages - > page reclaim would have been recirculating large blobs of mapped pages > around the LRU until the priority had built to the level where we > started to unmap them. > > However that priority-based decision got removed and right now I don't > recall what it got replaced with. Aren't we now unmapping pages way > too early and suffering an increased major&minor fault rate? Worried. We keep a different set of statistics to decide whether to reclaim only page cache pages, or both page cache and anonymous pages. The function get_scan_ratio parses those statistics. > So. What's up with that? I don't even remember _why_ we disable > the swap token once the scanning priority gets severe and the code > comments there are risible. And why do we wait until priority==0 > rather than priority==1? The reason is that we never page out the pages belonging to the process owning the swap token (with the exception of that process evicting its own pages). If that process has a really large RSS in the current zone, and we are having problems freeing pages, it may be beneficial to also evict pages from that process. Now that the LRU lists are split out into file backed and swap backed, it may be a lot easier to find pages to evict. That may mean we could notice we're getting into trouble at much higher priority levels and disable the swap token at a higher priority level. I do not believe prev_priority will be very useful here, since we'd like to start out with small scans whenever possible. > - Busted prev_priority means that lumpy reclaim will act oddly. > Every time someone goes into do some recalim, they'll start out not > doing lumpy reclaim. Then, after a while, they'll get a clue and > will start doing the lumpy thing. Then they return from reclaim and > the next recalim caller will again forget that he should have done > lumpy reclaim. How common are lumpy reclaims, anyway? Isn't it more likely that in-between every two higher-order reclaims, a number of order zero reclaims will be happening? In that case, the prev_priority logic may have introduced the kind of behavioural bug you describe above... > And one has to wonder: if we're making these incorrect decisions based > upon a bogus view of the current scanning difficulty, why are these > various priority-based thresholding heuristics even in there? Are they > doing anything useful? The prev_priority code was useful when we had filesystem and swap backed pages mixed on the same LRU list. I am not convinced it still has any use. -- All rights reversed -- 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