From patchwork Tue Aug 4 02:04:09 2009 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "IKEDA, Munehiro" X-Patchwork-Id: 39044 Received: from hormel.redhat.com (hormel1.redhat.com [209.132.177.33]) by demeter.kernel.org (8.14.2/8.14.2) with ESMTP id n7425Pd6016244 for ; Tue, 4 Aug 2009 02:05:25 GMT Received: from listman.util.phx.redhat.com (listman.util.phx.redhat.com [10.8.4.110]) by hormel.redhat.com (Postfix) with ESMTP id 37F2B619B73; Mon, 3 Aug 2009 22:05:25 -0400 (EDT) Received: from int-mx2.corp.redhat.com ([172.16.27.26]) by listman.util.phx.redhat.com (8.13.1/8.13.1) with ESMTP id n7425N5l031879 for ; Mon, 3 Aug 2009 22:05:23 -0400 Received: from ns3.rdu.redhat.com (ns3.rdu.redhat.com [10.11.255.199]) by int-mx2.corp.redhat.com (8.13.1/8.13.1) with ESMTP id n7425MiW012344; Mon, 3 Aug 2009 22:05:22 -0400 Received: from ronnie.usersys.redhat.com (dhcp-100-2-140.bos.redhat.com [10.16.2.140]) by ns3.rdu.redhat.com (8.13.8/8.13.8) with ESMTP id n7425KLV021216; Mon, 3 Aug 2009 22:05:20 -0400 Message-ID: <4A779719.1070900@ds.jp.nec.com> Date: Mon, 03 Aug 2009 22:04:09 -0400 From: Munehiro Ikeda User-Agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.1b3pre) Gecko/20090513 Fedora/3.0-2.3.beta2.fc11 Thunderbird/3.0b2 MIME-Version: 1.0 To: Gui Jianfeng References: <1246564917-19603-1-git-send-email-vgoyal@redhat.com> <4A569FC5.7090801@cn.fujitsu.com> In-Reply-To: <4A569FC5.7090801@cn.fujitsu.com> X-Scanned-By: MIMEDefang 2.58 on 172.16.27.26 X-loop: dm-devel@redhat.com Cc: dhaval@linux.vnet.ibm.com, snitzer@redhat.com, peterz@infradead.org, dm-devel@redhat.com, dpshah@google.com, jens.axboe@oracle.com, agk@redhat.com, balbir@linux.vnet.ibm.com, paolo.valente@unimore.it, fernando@oss.ntt.co.jp, mikew@google.com, jmoyer@redhat.com, nauman@google.com, Vivek Goyal , righi.andrea@gmail.com, lizf@cn.fujitsu.com, fchecconi@gmail.com, akpm@linux-foundation.org, jbaron@redhat.com, linux-kernel@vger.kernel.org, s-uchida@ap.jp.nec.com, containers@lists.linux-foundation.org Subject: [dm-devel] Re: [PATCH] io-controller: implement per group request allocation limitation X-BeenThere: dm-devel@redhat.com X-Mailman-Version: 2.1.5 Precedence: junk Reply-To: device-mapper development List-Id: device-mapper development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: dm-devel-bounces@redhat.com Errors-To: dm-devel-bounces@redhat.com Gui Jianfeng wrote, on 07/09/2009 09:56 PM: > 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. > > Signed-off-by: Gui Jianfeng (snip) Hi Jianfeng, If this helps, again. A patch posted from Gui Jianfeng on 2009/07/09 adds per-cgroup nr_requests control by io.nr_group_requests. The patch missed to update iog->nr_congestion_{on|off} and this patch adds the missing-link. Signed-off-by: Munehiro "Muuhh" Ikeda --- block/elevator-fq.c | 11 +++++++++++ 1 files changed, 11 insertions(+), 0 deletions(-) diff --git a/block/elevator-fq.c b/block/elevator-fq.c index 673e490..316bd8d 100644 --- a/block/elevator-fq.c +++ b/block/elevator-fq.c @@ -1607,6 +1607,10 @@ static int io_cgroup_nr_requests_write(struct cgroup *cgroup, u64 val) { struct io_cgroup *iocg; + struct io_group *iog; + struct elv_fq_data *efqd; + struct request_queue *q; + struct hlist_node *n; if (val < BLKDEV_MIN_RQ) val = BLKDEV_MIN_RQ; @@ -1618,6 +1622,13 @@ static int io_cgroup_nr_requests_write(struct cgroup *cgroup, spin_lock_irq(&iocg->lock); iocg->nr_group_requests = (unsigned long)val; + hlist_for_each_entry(iog, n, &iocg->group_data, group_node) { + rcu_read_lock(); + efqd = rcu_dereference(iog->key); + q = efqd->queue; + rcu_read_unlock(); + elv_io_group_congestion_threshold(q, iog); + } spin_unlock_irq(&iocg->lock); cgroup_unlock();