From: Vivek Goyal <vgoyal@redhat.com> To: Gui Jianfeng <guijianfeng@cn.fujitsu.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 04/19] io-controller: Modify cfq to make use of flat elevator fair queuing Date: Mon, 15 Jun 2009 09:02:25 -0400 [thread overview] Message-ID: <20090615130225.GB5717@redhat.com> (raw) In-Reply-To: <4A360CD2.8060707@cn.fujitsu.com> On Mon, Jun 15, 2009 at 04:56:50PM +0800, Gui Jianfeng wrote: > > -static inline int > > -cfq_prio_to_slice(struct cfq_data *cfqd, struct cfq_queue *cfqq) > > +static inline int cfq_class_rt(struct cfq_queue *cfqq) > > { > > - return cfq_prio_slice(cfqd, cfq_cfqq_sync(cfqq), cfqq->ioprio); > > + return elv_ioq_class_rt(cfqq->ioq); > > } > > Hi Vivek, > > cfq_class_rt isn't needed now, clean it up. > Thanks Gui. Queued up for next posting. Vivek > Signed-off-by: Gui Jianfeng <guijianfeng@cn.fujitsu.com> > --- > block/cfq-iosched.c | 5 ----- > 1 files changed, 0 insertions(+), 5 deletions(-) > > diff --git a/block/cfq-iosched.c b/block/cfq-iosched.c > index 705e7ba..23b24b9 100644 > --- a/block/cfq-iosched.c > +++ b/block/cfq-iosched.c > @@ -234,11 +234,6 @@ static inline int cfq_class_idle(struct cfq_queue *cfqq) > return elv_ioq_class_idle(cfqq->ioq); > } > > -static inline int cfq_class_rt(struct cfq_queue *cfqq) > -{ > - return elv_ioq_class_rt(cfqq->ioq); > -} > - > static inline int cfq_cfqq_sync(struct cfq_queue *cfqq) > { > return elv_ioq_sync(cfqq->ioq); > -- > 1.5.4.rc3 >
WARNING: multiple messages have this Message-ID (diff)
From: Vivek Goyal <vgoyal@redhat.com> To: Gui Jianfeng <guijianfeng@cn.fujitsu.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, m-ikeda@ds.jp.nec.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, righi.andrea@gmail.com, containers@lists.linux-foundation.org Subject: Re: [PATCH 04/19] io-controller: Modify cfq to make use of flat elevator fair queuing Date: Mon, 15 Jun 2009 09:02:25 -0400 [thread overview] Message-ID: <20090615130225.GB5717@redhat.com> (raw) In-Reply-To: <4A360CD2.8060707@cn.fujitsu.com> On Mon, Jun 15, 2009 at 04:56:50PM +0800, Gui Jianfeng wrote: > > -static inline int > > -cfq_prio_to_slice(struct cfq_data *cfqd, struct cfq_queue *cfqq) > > +static inline int cfq_class_rt(struct cfq_queue *cfqq) > > { > > - return cfq_prio_slice(cfqd, cfq_cfqq_sync(cfqq), cfqq->ioprio); > > + return elv_ioq_class_rt(cfqq->ioq); > > } > > Hi Vivek, > > cfq_class_rt isn't needed now, clean it up. > Thanks Gui. Queued up for next posting. Vivek > Signed-off-by: Gui Jianfeng <guijianfeng@cn.fujitsu.com> > --- > block/cfq-iosched.c | 5 ----- > 1 files changed, 0 insertions(+), 5 deletions(-) > > diff --git a/block/cfq-iosched.c b/block/cfq-iosched.c > index 705e7ba..23b24b9 100644 > --- a/block/cfq-iosched.c > +++ b/block/cfq-iosched.c > @@ -234,11 +234,6 @@ static inline int cfq_class_idle(struct cfq_queue *cfqq) > return elv_ioq_class_idle(cfqq->ioq); > } > > -static inline int cfq_class_rt(struct cfq_queue *cfqq) > -{ > - return elv_ioq_class_rt(cfqq->ioq); > -} > - > static inline int cfq_cfqq_sync(struct cfq_queue *cfqq) > { > return elv_ioq_sync(cfqq->ioq); > -- > 1.5.4.rc3 >
next prev parent reply other threads:[~2009-06-15 13:05 UTC|newest] Thread overview: 130+ messages / expand[flat|nested] mbox.gz Atom feed top 2009-06-09 2:08 [RFC] IO scheduler based IO controller V4 Vivek Goyal 2009-06-09 2:08 ` Vivek Goyal 2009-06-09 2:08 ` [PATCH 01/19] io-controller: Documentation Vivek Goyal 2009-06-09 2:08 ` Vivek Goyal 2009-06-09 2:08 ` [PATCH 02/19] io-controller: Common flat fair queuing code in elevaotor layer Vivek Goyal 2009-06-09 2:08 ` Vivek Goyal 2009-06-09 8:45 ` Gui Jianfeng 2009-06-09 8:45 ` Gui Jianfeng 2009-06-16 3:54 ` Gui Jianfeng 2009-06-16 3:54 ` Gui Jianfeng 2009-06-16 13:02 ` Vivek Goyal 2009-06-16 13:02 ` Vivek Goyal [not found] ` <4A37175C.80906-BthXqXjhjHXQFUHtdCDX3A@public.gmane.org> 2009-06-16 13:02 ` Vivek Goyal [not found] ` <1244513342-11758-3-git-send-email-vgoyal-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> 2009-06-09 8:45 ` Gui Jianfeng 2009-06-16 3:54 ` Gui Jianfeng 2009-06-16 19:44 ` Divyesh Shah 2009-06-16 19:44 ` Divyesh Shah 2009-06-16 19:44 ` Divyesh Shah 2009-06-17 12:59 ` Vivek Goyal 2009-06-17 12:59 ` Vivek Goyal [not found] ` <af41c7c40906161244q61fd47c5u1a10243c8839938-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> 2009-06-17 12:59 ` Vivek Goyal 2009-06-09 2:08 ` [PATCH 03/19] io-controller: Charge for time slice based on average disk rate Vivek Goyal 2009-06-09 2:08 ` Vivek Goyal 2009-06-09 2:08 ` [PATCH 04/19] io-controller: Modify cfq to make use of flat elevator fair queuing Vivek Goyal 2009-06-09 2:08 ` Vivek Goyal 2009-06-11 6:12 ` Gui Jianfeng 2009-06-11 14:21 ` Vivek Goyal 2009-06-11 14:21 ` Vivek Goyal [not found] ` <4A30A05C.4060703-BthXqXjhjHXQFUHtdCDX3A@public.gmane.org> 2009-06-11 14:21 ` Vivek Goyal [not found] ` <1244513342-11758-5-git-send-email-vgoyal-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> 2009-06-11 6:12 ` Gui Jianfeng 2009-06-15 8:56 ` Gui Jianfeng 2009-06-17 2:22 ` Gui Jianfeng 2009-06-15 8:56 ` Gui Jianfeng 2009-06-15 8:56 ` Gui Jianfeng 2009-06-15 13:02 ` Vivek Goyal [this message] 2009-06-15 13:02 ` Vivek Goyal [not found] ` <4A360CD2.8060707-BthXqXjhjHXQFUHtdCDX3A@public.gmane.org> 2009-06-15 13:02 ` Vivek Goyal 2009-06-17 2:22 ` Gui Jianfeng [not found] ` <4A385374.7060708-BthXqXjhjHXQFUHtdCDX3A@public.gmane.org> 2009-06-17 13:00 ` Vivek Goyal 2009-06-17 13:00 ` Vivek Goyal 2009-06-17 13:00 ` Vivek Goyal 2009-06-09 2:08 ` [PATCH 05/19] io-controller: Common hierarchical fair queuing code in elevaotor layer Vivek Goyal 2009-06-09 2:08 ` Vivek Goyal 2009-06-09 2:08 ` [PATCH 06/19] io-controller: cfq changes to use " Vivek Goyal 2009-06-09 2:08 ` Vivek Goyal 2009-06-09 2:08 ` [PATCH 07/19] io-controller: Export disk time used and nr sectors dipatched through cgroups Vivek Goyal 2009-06-09 2:08 ` Vivek Goyal 2009-06-09 2:08 ` [PATCH 08/19] io-controller: idle for sometime on sync queue before expiring it Vivek Goyal 2009-06-09 2:08 ` Vivek Goyal [not found] ` <1244513342-11758-1-git-send-email-vgoyal-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> 2009-06-09 2:08 ` [PATCH 01/19] io-controller: Documentation Vivek Goyal 2009-06-09 2:08 ` [PATCH 02/19] io-controller: Common flat fair queuing code in elevaotor layer Vivek Goyal 2009-06-09 2:08 ` [PATCH 03/19] io-controller: Charge for time slice based on average disk rate Vivek Goyal 2009-06-09 2:08 ` [PATCH 04/19] io-controller: Modify cfq to make use of flat elevator fair queuing Vivek Goyal 2009-06-09 2:08 ` [PATCH 05/19] io-controller: Common hierarchical fair queuing code in elevaotor layer Vivek Goyal 2009-06-09 2:08 ` [PATCH 06/19] io-controller: cfq changes to use " Vivek Goyal 2009-06-09 2:08 ` [PATCH 07/19] io-controller: Export disk time used and nr sectors dipatched through cgroups Vivek Goyal 2009-06-09 2:08 ` [PATCH 08/19] io-controller: idle for sometime on sync queue before expiring it Vivek Goyal 2009-06-09 2:08 ` [PATCH 09/19] io-controller: Separate out queue and data Vivek Goyal 2009-06-09 2:08 ` [PATCH 10/19] io-conroller: Prepare elevator layer for single queue schedulers Vivek Goyal 2009-06-09 2:08 ` [PATCH 11/19] io-controller: noop changes for hierarchical fair queuing Vivek Goyal 2009-06-09 2:08 ` [PATCH 12/19] io-controller: deadline " Vivek Goyal 2009-06-09 2:08 ` [PATCH 13/19] io-controller: anticipatory " Vivek Goyal 2009-06-09 2:08 ` [PATCH 14/19] blkio_cgroup patches from Ryo to track async bios Vivek Goyal 2009-06-09 2:08 ` [PATCH 15/19] io-controller: map async requests to appropriate cgroup Vivek Goyal 2009-06-09 2:08 ` [PATCH 16/19] io-controller: Per cgroup request descriptor support Vivek Goyal 2009-06-09 2:09 ` [PATCH 17/19] io-controller: Support per cgroup per device weights and io class Vivek Goyal 2009-06-09 2:09 ` [PATCH 18/19] io-controller: Debug hierarchical IO scheduling Vivek Goyal 2009-06-09 2:09 ` [PATCH 19/19] io-controller: experimental debug patch for async queue wait before expiry Vivek Goyal 2009-06-09 4:22 ` [RFC] IO scheduler based IO controller V4 Gui Jianfeng 2009-06-09 2:08 ` [PATCH 09/19] io-controller: Separate out queue and data Vivek Goyal 2009-06-09 2:08 ` Vivek Goyal 2009-06-09 2:08 ` [PATCH 10/19] io-conroller: Prepare elevator layer for single queue schedulers Vivek Goyal 2009-06-09 2:08 ` Vivek Goyal 2009-06-11 8:10 ` Gui Jianfeng [not found] ` <4A30BC0F.3000401-BthXqXjhjHXQFUHtdCDX3A@public.gmane.org> 2009-06-11 14:41 ` Vivek Goyal 2009-06-11 14:41 ` Vivek Goyal 2009-06-11 14:41 ` Vivek Goyal [not found] ` <20090611144136.GC27892-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> 2009-06-12 0:37 ` Gui Jianfeng 2009-06-12 0:37 ` Gui Jianfeng 2009-06-15 13:00 ` Vivek Goyal 2009-06-15 13:00 ` Vivek Goyal [not found] ` <4A31A345.50705-BthXqXjhjHXQFUHtdCDX3A@public.gmane.org> 2009-06-15 13:00 ` Vivek Goyal [not found] ` <1244513342-11758-11-git-send-email-vgoyal-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> 2009-06-11 8:10 ` Gui Jianfeng 2009-06-09 2:08 ` [PATCH 11/19] io-controller: noop changes for hierarchical fair queuing Vivek Goyal 2009-06-09 2:08 ` Vivek Goyal 2009-06-09 2:08 ` [PATCH 12/19] io-controller: deadline " Vivek Goyal 2009-06-09 2:08 ` Vivek Goyal 2009-06-09 2:08 ` [PATCH 13/19] io-controller: anticipatory " Vivek Goyal 2009-06-09 2:08 ` Vivek Goyal 2009-06-09 2:08 ` [PATCH 14/19] blkio_cgroup patches from Ryo to track async bios Vivek Goyal 2009-06-09 2:08 ` Vivek Goyal 2009-06-09 2:08 ` [PATCH 15/19] io-controller: map async requests to appropriate cgroup Vivek Goyal 2009-06-09 2:08 ` Vivek Goyal [not found] ` <1244513342-11758-16-git-send-email-vgoyal-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> 2009-06-17 9:17 ` Gui Jianfeng 2009-06-17 9:17 ` Gui Jianfeng 2009-06-17 9:17 ` Gui Jianfeng 2009-06-17 13:00 ` Vivek Goyal 2009-06-17 13:00 ` Vivek Goyal [not found] ` <4A38B4B4.7060101-BthXqXjhjHXQFUHtdCDX3A@public.gmane.org> 2009-06-17 13:00 ` Vivek Goyal 2009-06-09 2:08 ` [PATCH 16/19] io-controller: Per cgroup request descriptor support Vivek Goyal 2009-06-09 2:08 ` Vivek Goyal 2009-06-09 2:09 ` [PATCH 17/19] io-controller: Support per cgroup per device weights and io class Vivek Goyal 2009-06-09 2:09 ` Vivek Goyal [not found] ` <1244513342-11758-18-git-send-email-vgoyal-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> 2009-06-10 7:44 ` Gui Jianfeng 2009-06-10 7:44 ` Gui Jianfeng 2009-06-10 13:22 ` Vivek Goyal 2009-06-10 13:22 ` Vivek Goyal [not found] ` <4A2F646A.8070403-BthXqXjhjHXQFUHtdCDX3A@public.gmane.org> 2009-06-10 13:22 ` Vivek Goyal 2009-06-09 2:09 ` [PATCH 18/19] io-controller: Debug hierarchical IO scheduling Vivek Goyal 2009-06-09 2:09 ` Vivek Goyal 2009-06-19 1:40 ` Gui Jianfeng 2009-06-19 14:05 ` Vivek Goyal 2009-06-19 14:05 ` Vivek Goyal [not found] ` <4A3AEC96.7050500-BthXqXjhjHXQFUHtdCDX3A@public.gmane.org> 2009-06-19 14:05 ` Vivek Goyal 2009-06-19 6:26 ` Gui Jianfeng 2009-06-19 6:26 ` Gui Jianfeng [not found] ` <4A3B2F7E.10405-BthXqXjhjHXQFUHtdCDX3A@public.gmane.org> 2009-06-19 14:08 ` Vivek Goyal 2009-06-19 14:08 ` Vivek Goyal 2009-06-19 14:08 ` Vivek Goyal [not found] ` <1244513342-11758-19-git-send-email-vgoyal-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> 2009-06-19 1:40 ` Gui Jianfeng 2009-06-19 6:26 ` Gui Jianfeng 2009-06-09 2:09 ` [PATCH 19/19] io-controller: experimental debug patch for async queue wait before expiry Vivek Goyal 2009-06-09 2:09 ` Vivek Goyal 2009-06-09 4:22 ` [RFC] IO scheduler based IO controller V4 Gui Jianfeng 2009-06-09 4:22 ` Gui Jianfeng [not found] ` <4A2DE381.6090205-BthXqXjhjHXQFUHtdCDX3A@public.gmane.org> 2009-06-09 13:52 ` Vivek Goyal 2009-06-09 13:52 ` Vivek Goyal 2009-06-09 13:52 ` Vivek Goyal [not found] ` <20090609135256.GA13476-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> 2009-06-10 1:05 ` Gui Jianfeng 2009-06-10 1:05 ` Gui Jianfeng
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20090615130225.GB5717@redhat.com \ --to=vgoyal@redhat.com \ --cc=agk@redhat.com \ --cc=akpm@linux-foundation.org \ --cc=balbir@linux.vnet.ibm.com \ --cc=containers@lists.linux-foundation.org \ --cc=dhaval@linux.vnet.ibm.com \ --cc=dm-devel@redhat.com \ --cc=dpshah@google.com \ --cc=fchecconi@gmail.com \ --cc=fernando@oss.ntt.co.jp \ --cc=guijianfeng@cn.fujitsu.com \ --cc=jbaron@redhat.com \ --cc=jens.axboe@oracle.com \ --cc=jmoyer@redhat.com \ --cc=linux-kernel@vger.kernel.org \ --cc=lizf@cn.fujitsu.com \ --cc=m-ikeda@ds.jp.nec.com \ --cc=mikew@google.com \ --cc=nauman@google.com \ --cc=paolo.valente@unimore.it \ --cc=peterz@infradead.org \ --cc=righi.andrea@gmail.com \ --cc=ryov@valinux.co.jp \ --cc=s-uchida@ap.jp.nec.com \ --cc=snitzer@redhat.com \ --cc=taka@valinux.co.jp \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.