From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail190.messagelabs.com (mail190.messagelabs.com [216.82.249.51]) by kanga.kvack.org (Postfix) with ESMTP id 3139C6B0023 for ; Mon, 16 May 2011 20:19:01 -0400 (EDT) Date: Mon, 16 May 2011 17:18:20 -0700 From: Andrew Morton Subject: Re: [PATCH] memcg: fix typo in the soft_limit stats. Message-Id: <20110516171820.124a8fbc.akpm@linux-foundation.org> In-Reply-To: References: <1305583230-2111-1-git-send-email-yinghan@google.com> <20110516231512.GW16531@cmpxchg.org> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: owner-linux-mm@kvack.org List-ID: To: Ying Han Cc: Johannes Weiner , KOSAKI Motohiro , Minchan Kim , Daisuke Nishimura , Balbir Singh , Tejun Heo , Pavel Emelyanov , KAMEZAWA Hiroyuki , Li Zefan , Mel Gorman , Christoph Lameter , Rik van Riel , Hugh Dickins , Michal Hocko , Dave Hansen , Zhu Yanhai , linux-mm@kvack.org On Mon, 16 May 2011 17:05:02 -0700 Ying Han wrote: > On Mon, May 16, 2011 at 4:15 PM, Johannes Weiner wrote: > > > On Mon, May 16, 2011 at 03:00:30PM -0700, Ying Han wrote: > > > This fixes the typo in the memory.stat including the following two > > > stats: > > > > > > $ cat /dev/cgroup/memory/A/memory.stat > > > total_soft_steal 0 > > > total_soft_scan 0 > > > > > > And change it to: > > > > > > $ cat /dev/cgroup/memory/A/memory.stat > > > total_soft_kswapd_steal 0 > > > total_soft_kswapd_scan 0 > > > > > > Signed-off-by: Ying Han > > > > I am currently proposing and working on a scheme that makes the soft > > limit not only a factor for global memory pressure, but for > > hierarchical reclaim in general, to prefer child memcgs during reclaim > > that are in excess of their soft limit. > > > > Because this means prioritizing memcgs over one another, rather than > > having explicit soft limit reclaim runs, there is no natural counter > > for pages reclaimed due to the soft limit anymore. > > > > Thus, for the patch that introduces this counter: > > > > Nacked-by: Johannes Weiner > > > > This patch is fixing a typo of the stats being integrated into mmotm. Does > it make sense to fix the > existing stats first while we are discussing other approaches? > It would be quite bad to add new userspace-visible stats and to then take them away again. But given that memcg-add-stats-to-monitor-soft_limit-reclaim.patch is queued for 2.6.39-rc1, we could proceed with that plan and then make sure that Johannes's changes are merged either prior to 2.6.40 or they are never merged at all. Or we could just leave out the stats until we're sure. Not having them for a while is not as bad as adding them and then removing them. -- 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