From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail144.messagelabs.com (mail144.messagelabs.com [216.82.254.51]) by kanga.kvack.org (Postfix) with ESMTP id 6734A6B0085 for ; Fri, 19 Nov 2010 06:17:15 -0500 (EST) Date: Fri, 19 Nov 2010 12:16:52 +0100 From: Johannes Weiner Subject: Re: [PATCH 3/6] memcg: make throttle_vm_writeout() memcg aware Message-ID: <20101119111652.GB24635@cmpxchg.org> References: <1289294671-6865-1-git-send-email-gthelen@google.com> <1289294671-6865-4-git-send-email-gthelen@google.com> <20101112081754.GE9131@cmpxchg.org> <20101116125726.db42723c.kamezawa.hiroyu@jp.fujitsu.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20101116125726.db42723c.kamezawa.hiroyu@jp.fujitsu.com> Sender: owner-linux-mm@kvack.org To: KAMEZAWA Hiroyuki Cc: Greg Thelen , Andrew Morton , Balbir Singh , Daisuke Nishimura , Wu Fengguang , Minchan Kim , linux-mm@kvack.org, linux-kernel@vger.kernel.org List-ID: Hello, On Tue, Nov 16, 2010 at 12:57:26PM +0900, KAMEZAWA Hiroyuki wrote: > Hmm. I think this patch is troublesome. > > This patch will make memcg's pageout routine _not_ throttoled even when the whole > system vmscan's pageout is throttoled. > > So, one idea is.... > > Make this change > == > +++ b/mm/vmscan.c > @@ -1844,7 +1844,7 @@ static void shrink_zone(int priority, struct zone *zone, > if (inactive_anon_is_low(zone, sc)) > shrink_active_list(SWAP_CLUSTER_MAX, zone, sc, priority, 0); > > - throttle_vm_writeout(sc->gfp_mask); > + throttle_vm_writeout(sc->gfp_mask, sc->mem_cgroup); > } > == > as > > == > > if (!sc->mem_cgroup || throttle_vm_writeout(sc->gfp_mask, sc->mem_cgroup) == not throttled) > throttole_vm_writeout(sc->gfp_mask, NULL); > > Then, both of memcg and global dirty thresh will be checked. Good point, both limits should apply. I'd prefer to stuff it all into throttle_vm_writeout() and not encode memcg-specific behaviour into the caller, though. -- 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 policy in Canada: sign http://dissolvethecrtc.ca/ Don't email: email@kvack.org