Message-ID: <4A5C35D7.1040502@cn.fujitsu.com>
Date: Tue, 14 Jul 2009 15:37:59 +0800
From: Gui Jianfeng <guijianfeng@cn.fujitsu.com>
User-Agent: Thunderbird 2.0.0.5 (Windows/20070716)
MIME-Version: 1.0
To: Vivek Goyal <vgoyal@redhat.com>
CC: linux-kernel@vger.kernel.org, containers@lists.linux-foundation.org,
       dm-devel@redhat.com, jens.axboe@oracle.com, nauman@google.com,
       dpshah@google.com, lizf@cn.fujitsu.com, mikew@google.com,
       fchecconi@gmail.com, paolo.valente@unimore.it, ryov@valinux.co.jp,
       fernando@oss.ntt.co.jp, s-uchida@ap.jp.nec.com, taka@valinux.co.jp,
       jmoyer@redhat.com, dhaval@linux.vnet.ibm.com, balbir@linux.vnet.ibm.com,
       righi.andrea@gmail.com, m-ikeda@ds.jp.nec.com, jbaron@redhat.com,
       agk@redhat.com, snitzer@redhat.com, akpm@linux-foundation.org,
       peterz@infradead.org
Subject: Re: [PATCH] io-controller: implement per group request allocation
 limitation
References: <1246564917-19603-1-git-send-email-vgoyal@redhat.com> <4A569FC5.7090801@cn.fujitsu.com> <20090713160352.GA3714@redhat.com>
In-Reply-To: <20090713160352.GA3714@redhat.com>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1880
Lines: 51

Vivek Goyal wrote:
> On Fri, Jul 10, 2009 at 09:56:21AM +0800, Gui Jianfeng wrote:
>> Hi Vivek,
>>
>> This patch exports a cgroup based per group request limits interface.
>> and removes the global one. Now we can use this interface to perform
>> different request allocation limitation for different groups. 
>>
> 
> Thanks Gui. Few points come to mind.
> 
> - You seem to be making this as per cgroup limit on all devices. I guess
>   that different devices in the system can have different settings of
>   q->nr_requests and hence will probably want different per group limit.
>   So we might have to make it per cgroup per device limit.

  Yes, per cgroup per device limitation seems more reasonable. I'll see what
  i can do.

> 
> - There does not seem to be any checks for making sure that children
>   cgroups don't have more request descriptors allocated than parent group.

  Do we really need to make it hierarchical? IMHO, maintaining this limitation
  for cgroups independently is enough.

> 
> - I am re-thinking that what's the advantage of configuring request
>   descriptors also through cgroups. It does bring in additional complexity
>   with it and it should justfiy the advantages. Can you think of some?

  I'll try, but at least, this feature lets us be able to do more accurate
  limitation. :)

> 
>   Until and unless we can come up with some significant advantages, I will
>   prefer to continue to use per group limit through q->nr_group_requests
>   interface instead of cgroup. Once things stablize, we can revisit it and
>   see how this interface can be improved.

  I agree.

-- 
Regards
Gui Jianfeng

--
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/