From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from psmtp.com (na3sys010amx124.postini.com [74.125.245.124]) by kanga.kvack.org (Postfix) with SMTP id 7A9796B004D for ; Thu, 15 Mar 2012 02:03:55 -0400 (EDT) Received: by bkwq16 with SMTP id q16so2568920bkw.14 for ; Wed, 14 Mar 2012 23:03:53 -0700 (PDT) Message-ID: <4F618645.8020507@openvz.org> Date: Thu, 15 Mar 2012 10:03:49 +0400 From: Konstantin Khlebnikov MIME-Version: 1.0 Subject: Re: [PATCH 3/7 v2] mm: rework __isolate_lru_page() file/anon filter References: <20120229091547.29236.28230.stgit@zurg> <20120303091327.17599.80336.stgit@zurg> <20120308143034.f3521b1e.kamezawa.hiroyu@jp.fujitsu.com> <4F59AE3C.5040200@openvz.org> <4F5AFAF0.6060608@openvz.org> <4F5B22DE.4020402@openvz.org> In-Reply-To: Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Sender: owner-linux-mm@kvack.org List-ID: To: Hugh Dickins Cc: KAMEZAWA Hiroyuki , Andrew Morton , Johannes Weiner , "linux-mm@kvack.org" , "linux-kernel@vger.kernel.org" Hugh Dickins wrote: > On Sat, 10 Mar 2012, Konstantin Khlebnikov wrote: >> Konstantin Khlebnikov wrote: >>> >>> No, for non-lumpy isolation we don't need this check at all, >>> because all pages already picked from right lru list. >>> >>> I'll send separate patch for this (on top v5 patchset), after meditation =) >> >> Heh, looks like we don't need these checks at all: >> without RECLAIM_MODE_LUMPYRECLAIM we isolate only pages from right lru, >> with RECLAIM_MODE_LUMPYRECLAIM we isolate pages from all evictable lru. >> Thus we should check only PageUnevictable() on lumpy reclaim. > > Yes, those were great simplfying insights: I'm puzzling over why you > didn't follow through on them in your otherwise nice 4.5/7, which > still involves lru bits in the isolate mode? Actually filter is required for single case: lumpy isolation for shrink_active_list(). Maybe I'm wrong, or this is bug, but I don't see any reasons why this can not happen: sc->reclaim_mode manipulations are very tricky. > > Hugh > > -- > 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 -- 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