[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20130709141833.GA2237@redhat.com>
Date: Tue, 9 Jul 2013 10:18:33 -0400
From: Vivek Goyal <vgoyal@...hat.com>
To: Tejun Heo <tj@...nel.org>
Cc: Konstantin Khlebnikov <khlebnikov@...nvz.org>, linux-mm@...ck.org,
linux-kernel@...r.kernel.org, Michal Hocko <mhocko@...e.cz>,
cgroups@...r.kernel.org, Andrew Morton <akpm@...ux-foundation.org>,
Sha Zhengju <handai.szj@...il.com>, devel@...nvz.org,
Jens Axboe <axboe@...nel.dk>
Subject: Re: [PATCH RFC] fsio: filesystem io accounting cgroup
On Tue, Jul 09, 2013 at 06:45:58AM -0700, Tejun Heo wrote:
> On Tue, Jul 09, 2013 at 05:43:10PM +0400, Konstantin Khlebnikov wrote:
> > My concept it cgroup which would control io operation on vfs layer
> > for all filesystems. It will account and manage IO operations. I've
> > found really lightweight technique for accounting and throttling
> > which don't introduce new locks or priority inversions (which is
> > major problem in all existing throttlers, including cpu cgroup rate
> > limiter) So, I've tried to keep code smaller, cleaner and saner as
> > possible while you guys are trying to push me into the block layer
> > =)
>
> You're trying to implement QoS in the place where you don't have
> control of the queue itself.
> You aren't even managing the right type
> of resource for disks which is time slice rather than iops or
> bandwidth
For implementing throttling one as such does not have to do time
slice management on the queue. For providing constructs like IOPS
or bandwidth throttling, one just need to put one throttling knob
in the cgroup pipe irrespective of time slice management on the
backing device/network.
Also time slice management is one way of managing the backend resource.
CFQ did that and it works only for slow devices. For faster devices
we anyway need some kind of token mechanism instead of keeping track
of time.
So I don't think trying to manage time slice is the requirement here.
> and by the time you implemented proper hierarchy support and
> proportional control, yours isn't gonna be that simple either.
I suspect he is not plannnig to do any proportional control at that
layer. Just throttling mechanism.
Thanks
Vivek
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists