Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752155Ab1FLOp0 (ORCPT ); Sun, 12 Jun 2011 10:45:26 -0400 Received: from cantor2.suse.de ([195.135.220.15]:55179 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751159Ab1FLOpY (ORCPT ); Sun, 12 Jun 2011 10:45:24 -0400 Date: Sun, 12 Jun 2011 16:45:21 +0200 From: Michal Hocko To: Minchan Kim Cc: Andrew Morton , linux-mm , LKML , KOSAKI Motohiro , Mel Gorman , Andrea Arcangeli , Rik van Riel , Johannes Weiner , KAMEZAWA Hiroyuki Subject: Re: [PATCH v3 03/10] Add additional isolation mode Message-ID: <20110612144521.GB24323@tiehlicka.suse.cz> References: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3226 Lines: 91 On Tue 07-06-11 23:38:16, Minchan Kim wrote: > There are some places to isolate lru page and I believe > users of isolate_lru_page will be growing. > The purpose of them is each different so part of isolated pages > should put back to LRU, again. > > The problem is when we put back the page into LRU, > we lose LRU ordering and the page is inserted at head of LRU list. > It makes unnecessary LRU churning so that vm can evict working set pages > rather than idle pages. I guess that, although this is true, it doesn't fit in with this patch very much because this patch doesn't fix this problem. It is a preparation for for further work. I would expect this description with the core patch that actlually handles this issue. > > This patch adds new modes when we isolate page in LRU so we don't isolate pages > if we can't handle it. It could reduce LRU churning. > > This patch doesn't change old behavior. It's just used by next patches. It doesn't because there is not user of those flags but maybe it would be better to have those to see why it actually can reduce LRU isolations. > > Cc: KOSAKI Motohiro > Cc: Mel Gorman > Cc: Andrea Arcangeli > Cc: KAMEZAWA Hiroyuki > Acked-by: Rik van Riel > Acked-by: Johannes Weiner > Signed-off-by: Minchan Kim > --- > include/linux/swap.h | 2 ++ > mm/vmscan.c | 6 ++++++ > 2 files changed, 8 insertions(+), 0 deletions(-) > > diff --git a/include/linux/swap.h b/include/linux/swap.h > index 48d50e6..731f5dd 100644 > --- a/include/linux/swap.h > +++ b/include/linux/swap.h > @@ -248,6 +248,8 @@ enum ISOLATE_MODE { > ISOLATE_NONE, > ISOLATE_INACTIVE = 1, /* Isolate inactive pages */ > ISOLATE_ACTIVE = 2, /* Isolate active pages */ > + ISOLATE_CLEAN = 8, /* Isolate clean file */ > + ISOLATE_UNMAPPED = 16, /* Isolate unmapped file */ > }; > > /* linux/mm/vmscan.c */ > diff --git a/mm/vmscan.c b/mm/vmscan.c > index 4cbe114..26aa627 100644 > --- a/mm/vmscan.c > +++ b/mm/vmscan.c > @@ -990,6 +990,12 @@ int __isolate_lru_page(struct page *page, enum ISOLATE_MODE mode, int file) > > ret = -EBUSY; > > + if (mode & ISOLATE_CLEAN && (PageDirty(page) || PageWriteback(page))) > + return ret; > + > + if (mode & ISOLATE_UNMAPPED && page_mapped(page)) > + return ret; > + > if (likely(get_page_unless_zero(page))) { > /* > * Be careful not to clear PageLRU until after we're > -- > 1.7.0.4 > > -- > 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 -- Michal Hocko SUSE Labs SUSE LINUX s.r.o. Lihovarska 1060/12 190 00 Praha 9 Czech Republic -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/