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 73D206B00C8 for ; Tue, 19 Oct 2010 23:37:41 -0400 (EDT) Date: Wed, 20 Oct 2010 12:31:10 +0900 From: Daisuke Nishimura Subject: Re: [PATCH v3 09/11] memcg: add cgroupfs interface to memcg dirty limits Message-Id: <20101020123110.fd269ab4.nishimura@mxp.nes.nec.co.jp> In-Reply-To: <1287448784-25684-10-git-send-email-gthelen@google.com> References: <1287448784-25684-1-git-send-email-gthelen@google.com> <1287448784-25684-10-git-send-email-gthelen@google.com> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: owner-linux-mm@kvack.org To: Greg Thelen Cc: Andrew Morton , linux-kernel@vger.kernel.org, linux-mm@kvack.org, containers@lists.osdl.org, Andrea Righi , Balbir Singh , KAMEZAWA Hiroyuki , Minchan Kim , Ciju Rajan K , David Rientjes , Daisuke Nishimura List-ID: On Mon, 18 Oct 2010 17:39:42 -0700 Greg Thelen wrote: > Add cgroupfs interface to memcg dirty page limits: > Direct write-out is controlled with: > - memory.dirty_ratio > - memory.dirty_limit_in_bytes > > Background write-out is controlled with: > - memory.dirty_background_ratio > - memory.dirty_background_limit_bytes > > Other memcg cgroupfs files support 'M', 'm', 'k', 'K', 'g' > and 'G' suffixes for byte counts. This patch provides the > same functionality for memory.dirty_limit_in_bytes and > memory.dirty_background_limit_bytes. > > Signed-off-by: Andrea Righi > Signed-off-by: Balbir Singh > Signed-off-by: Greg Thelen Acked-by: Daisuke Nishimura One question: shouldn't we return -EINVAL when writing to dirty(_background)_limit_bytes a bigger value than that of global one(if any) ? Or do you intentionally set the input value without comparing it with the global value ? But, hmm..., IMHO we should check it in __mem_cgroup_dirty_param() or something not to allow dirty pages more than global limit. Thanks, Daisuke Nishimura. -- 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/ . Don't email: email@kvack.org