From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail143.messagelabs.com (mail143.messagelabs.com [216.82.254.35]) by kanga.kvack.org (Postfix) with ESMTP id 2FA1D6B016A for ; Mon, 8 Aug 2011 19:36:40 -0400 (EDT) Received: by qyk27 with SMTP id 27so2149832qyk.14 for ; Mon, 08 Aug 2011 16:36:37 -0700 (PDT) MIME-Version: 1.0 In-Reply-To: <20110808110658.31053.55013.stgit@localhost6> References: <20110808110658.31053.55013.stgit@localhost6> Date: Tue, 9 Aug 2011 08:36:37 +0900 Message-ID: Subject: Re: [PATCH 1/2] vmscan: promote shared file mapped pages From: Minchan Kim Content-Type: text/plain; charset=UTF-8 Sender: owner-linux-mm@kvack.org List-ID: To: Konstantin Khlebnikov Cc: linux-mm@kvack.org, Andrew Morton , linux-kernel@vger.kernel.org, Wu Fengguang , KAMEZAWA Hiroyuki , Johannes Weiner , Rik van Riel , KOSAKI Motohiro On Mon, Aug 8, 2011 at 8:06 PM, Konstantin Khlebnikov wrote: > Commit v2.6.33-5448-g6457474 (vmscan: detect mapped file pages used only once) > greatly decreases lifetime of single-used mapped file pages. > Unfortunately it also decreases life time of all shared mapped file pages. > Because after commit v2.6.28-6130-gbf3f3bc (mm: don't mark_page_accessed in fault path) > page-fault handler does not mark page active or even referenced. > > Thus page_check_references() activates file page only if it was used twice while > it stays in inactive list, meanwhile it activates anon pages after first access. > Inactive list can be small enough, this way reclaimer can accidentally > throw away any widely used page if it wasn't used twice in short period. > > After this patch page_check_references() also activate file mapped page at first > inactive list scan if this page is already used multiple times via several ptes. > > Signed-off-by: Konstantin Khlebnikov Looks good to me. But the issue is that we prefer shared mapped file pages aggressively by your patch. Shared page already have bigger chance to promote than single page during same time window as many processes can touch the page. The your concern is when file LRU is too small or scanning aggressively, shared mapping page could lose the chance to activate but it is applied single page, too. And still, shared mapping pages have a bigger chance to activate compared to single page. So, our algorithm already reflect shared mapping preference a bit. Fundamental problem is our eviction algorithm consider only recency, not frequency. It's very long time problem and it's not easy to fix it practically. Anyway, it's a not subject it's right or not but it's policy thing and I support yours. Acked-by: Minchan Kim -- Kind regards, Minchan Kim -- 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/ . Fight unfair telecom internet charges in Canada: sign http://stopthemeter.ca/ Don't email: email@kvack.org