From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752223Ab1HHLHM (ORCPT ); Mon, 8 Aug 2011 07:07:12 -0400 Received: from relay.parallels.com ([195.214.232.42]:55639 "EHLO relay.parallels.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751811Ab1HHLHI (ORCPT ); Mon, 8 Aug 2011 07:07:08 -0400 Subject: [PATCH 2/2] vmscan: activate executable pages after first usage To: , Andrew Morton , From: Konstantin Khlebnikov CC: Wu Fengguang , KAMEZAWA Hiroyuki , Johannes Weiner Date: Mon, 8 Aug 2011 15:07:00 +0400 Message-ID: <20110808110659.31053.92935.stgit@localhost6> In-Reply-To: <20110808110658.31053.55013.stgit@localhost6> References: <20110808110658.31053.55013.stgit@localhost6> User-Agent: StGit/0.15 MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Logic added in commit v2.6.30-5507-g8cab475 (vmscan: make mapped executable pages the first class citizen) was noticeably weakened in commit v2.6.33-5448-g6457474 (vmscan: detect mapped file pages used only once) Currently these pages can become "first class citizens" only after second usage. After this patch page_check_references() will activate they after first usage, and executable code gets yet better chance to stay in memory. TODO: run some cool tests like in v2.6.30-5507-g8cab475 =) Signed-off-by: Konstantin Khlebnikov --- mm/vmscan.c | 6 ++++++ 1 files changed, 6 insertions(+), 0 deletions(-) diff --git a/mm/vmscan.c b/mm/vmscan.c index 3cd766d..29b3612 100644 --- a/mm/vmscan.c +++ b/mm/vmscan.c @@ -727,6 +727,12 @@ static enum page_references page_check_references(struct page *page, if (referenced_page || referenced_ptes > 1) return PAGEREF_ACTIVATE; + /* + * Activate file-backed executable pages after first usage. + */ + if (vm_flags & VM_EXEC) + return PAGEREF_ACTIVATE; + return PAGEREF_KEEP; }