From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail137.messagelabs.com (mail137.messagelabs.com [216.82.249.19]) by kanga.kvack.org (Postfix) with ESMTP id 1ABC76B004A for ; Mon, 13 Jun 2011 10:55:01 -0400 (EDT) Received: by pvc12 with SMTP id 12so2681966pvc.14 for ; Mon, 13 Jun 2011 07:54:59 -0700 (PDT) Date: Mon, 13 Jun 2011 23:54:43 +0900 From: Minchan Kim Subject: Re: [PATCH v3 03/10] Add additional isolation mode Message-ID: <20110613145443.GB1414@barrios-desktop> References: <20110612144521.GB24323@tiehlicka.suse.cz> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20110612144521.GB24323@tiehlicka.suse.cz> Sender: owner-linux-mm@kvack.org List-ID: To: Michal Hocko Cc: Andrew Morton , linux-mm , LKML , KOSAKI Motohiro , Mel Gorman , Andrea Arcangeli , Rik van Riel , Johannes Weiner , KAMEZAWA Hiroyuki On Sun, Jun 12, 2011 at 04:45:21PM +0200, Michal Hocko wrote: > 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. Okay. > > > > > 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. Yes. Mel already pointed it out. I will merge patches in next version. And I have a idea to reduce lru_lock Mel mentiond So maybe I will include it in next version, too. But, now I have no time to revise it :( -- 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