Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756623Ab1FIInV (ORCPT ); Thu, 9 Jun 2011 04:43:21 -0400 Received: from zene.cmpxchg.org ([85.214.230.12]:43955 "EHLO zene.cmpxchg.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752879Ab1FIInU (ORCPT ); Thu, 9 Jun 2011 04:43:20 -0400 Date: Thu, 9 Jun 2011 10:43:00 +0200 From: Johannes Weiner To: Rik van Riel Cc: Hiroyuki Kamezawa , KAMEZAWA Hiroyuki , Daisuke Nishimura , Balbir Singh , Ying Han , Michal Hocko , Andrew Morton , Minchan Kim , KOSAKI Motohiro , Mel Gorman , Greg Thelen , Michel Lespinasse , linux-mm@kvack.org, linux-kernel@vger.kernel.org Subject: Re: [patch 0/8] mm: memcg naturalization -rc2 Message-ID: <20110609084300.GD11603@cmpxchg.org> References: <1306909519-7286-1-git-send-email-hannes@cmpxchg.org> <20110602073335.GA20630@cmpxchg.org> <20110602100007.GB20725@cmpxchg.org> <4DF01EC2.8010105@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <4DF01EC2.8010105@redhat.com> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1272 Lines: 33 On Wed, Jun 08, 2011 at 09:15:46PM -0400, Rik van Riel wrote: > On 06/02/2011 08:59 AM, Hiroyuki Kamezawa wrote: > > >It seems your current series is a mixture of 2 works as > >"re-desgin of softlimit" and "removal of global LRU". > >I don't understand why you need 2 works at once. > > That seems pretty obvious. > > With the global LRU gone, the only way to reclaim > pages in a global fashion (because the zone is low > on memory), is to reclaim from all the memcgs in > the zone. That is correct. > Doing that requires that the softlimit stuff is > changed, and not only the biggest offender is > attacked. I think it's much more natural to do it that way, but it's not a requirement as such. We could just keep the extra soft limit reclaim invocation in kswapd that looks for the biggest offender and the hierarchy below it, then does a direct call to do_shrink_zone() to bypass the generic hierarchy walk. It's not very nice to have that kind of code duplication, but it's possible to leave it like that for now. -- 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/