From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from d23relay03.au.ibm.com (d23relay03.au.ibm.com [202.81.18.234]) by e23smtp05.au.ibm.com (8.13.1/8.13.1) with ESMTP id m454LU3x027892 for ; Mon, 5 May 2008 14:21:30 +1000 Received: from d23av03.au.ibm.com (d23av03.au.ibm.com [9.190.234.97]) by d23relay03.au.ibm.com (8.13.8/8.13.8/NCO v8.7) with ESMTP id m454Ln6F4153494 for ; Mon, 5 May 2008 14:21:49 +1000 Received: from d23av03.au.ibm.com (loopback [127.0.0.1]) by d23av03.au.ibm.com (8.12.11.20060308/8.13.3) with ESMTP id m454Lvr1009094 for ; Mon, 5 May 2008 14:21:58 +1000 Message-ID: <481E8B3F.3050508@linux.vnet.ibm.com> Date: Mon, 05 May 2008 09:51:19 +0530 From: Balbir Singh Reply-To: balbir@linux.vnet.ibm.com MIME-Version: 1.0 Subject: Re: [-mm][PATCH 0/4] Add rlimit controller to cgroups (v3) References: <20080503213726.3140.68845.sendpatchset@localhost.localdomain> <23630056.1209914669637.kamezawa.hiroyu@jp.fujitsu.com> In-Reply-To: <23630056.1209914669637.kamezawa.hiroyu@jp.fujitsu.com> Content-Type: text/plain; charset=ISO-2022-JP Content-Transfer-Encoding: 7bit Sender: owner-linux-mm@kvack.org Return-Path: To: kamezawa.hiroyu@jp.fujitsu.com Cc: linux-mm@kvack.org, Sudhir Kumar , YAMAMOTO Takashi , Paul Menage , lizf@cn.fujitsu.com, linux-kernel@vger.kernel.org, David Rientjes , Pavel Emelianov , Andrew Morton List-ID: kamezawa.hiroyu@jp.fujitsu.com wrote: >> >> This is the third version of the address space control patches. These >> patches are against 2.6.25-mm1 and have been tested using KVM in SMP mode, >> both with and without the config enabled. >> >> The first patch adds the user interface. The second patch fixes the >> cgroup mm_owner_changed callback to pass the task struct, so that >> accounting can be adjusted on owner changes. The thrid patch adds accounting >> and control. The fourth patch updates documentation. >> >> An earlier post of the patchset can be found at >> http://lwn.net/Articles/275143/ >> >> This patch is built on top of the mm owner patches and utilizes that feature >> to virtually group tasks by mm_struct. >> >> Reviews, Comments? >> > > I can't read the whole patch deeply now but this new concept "rlimit-controlle > r" seems make sense to me. > > At quick glance, I have some thoughts. > > 1. kerner/rlimit_cgroup.c is better for future expansion. I have no problem with that name, I can rename the files. > 2. why > "+This controller framework is designed to be extensible to control any > "+resource limit (memory related) with little effort." > memory only ? Ok, all you want to do is related to memory, but someone > may want to limit RLIMIT_CPU by group or RLIMIT_CORE by group or.... > (I have no plan but they seems useful.;) I currently mentioned memory, since we have the infrastructure to group using mm->owner infrastructure. For other purposes, we'll need to enhance the controller quite a bit. That is why I put memory related in brackets. > So, could you add design hint of rlimit contoller to the documentation ? > OK, I'll update the documentation > 3. Rleated to 2. Showing what kind of "rlimit" params are supported by > cgroup will be good. > Do you mean in init/Kconfig or documentation?. I should probably rename limit_in_bytes and usage_in_bytes to add an as_ prefix, so that the UI clearly shows what is supported as well. > I don't think you have to implement all things at once. Staring from > "only RLIMIT_AS is supported now" is good. Someone will expand it if > he needs. But showing basic view of "gerenal purpose rlimit contoller" in _doc > ument_ or _comments_ or _codes_ is a good thing to do. > I can add to the documentation > If you don't want to provide RLIMIT feature other than address space, > it's better to avoid using the name of RLIMIT. It's confusing. > I used RLIMIT since I want to extend it later to control memory locked pages :) I open to other names as well. > Thanks, > -Kame > > > > > > -- > 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 -- Warm Regards, Balbir Singh Linux Technology Center IBM, ISTL -- 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