From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jens Axboe Subject: Re: Do not overload dispatch queue (Was: Re: IO scheduler based IO controller V10) Date: Sat, 3 Oct 2009 17:14:45 +0200 Message-ID: <20091003151445.GF31616__49282.2785200135$1254583001$gmane$org@kernel.dk> References: <20091002181903.GN31616@kernel.dk> <1254548931.8299.18.camel@marge.simson.net> <1254549378.8299.21.camel@marge.simson.net> <20091003112915.GA12925@redhat.com> <20091003124049.GB12925@redhat.com> <20091003132115.GB31616@kernel.dk> <20091003135623.GD12925@redhat.com> <1254578553.7499.5.camel@marge.simson.net> <20091003142840.GE31616@kernel.dk> <1254581496.8293.8.camel@marge.simson.net> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: <1254581496.8293.8.camel-YqMYhexLQo1vAv1Ojkdn7Q@public.gmane.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: containers-bounces-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org Errors-To: containers-bounces-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org To: Mike Galbraith Cc: dhaval-23VcF4HTsmIX0ybBhKVfKdBPR1lH4CV8@public.gmane.org, dm-devel-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org, agk-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org, balbir-23VcF4HTsmIX0ybBhKVfKdBPR1lH4CV8@public.gmane.org, paolo.valente-rcYM44yAMweonA0d6jMUrA@public.gmane.org, jmarchan-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org, fernando-gVGce1chcLdL9jVzuh4AOg@public.gmane.org, Ulrich Lukas , jmoyer-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org, Ingo Molnar , riel-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org, fchecconi-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org, containers-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, akpm-de/tnXTf+JLsfHDXvbKv3WD2FQJk+8+b@public.gmane.org, righi.andrea-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org, Linus Torvalds List-Id: containers.vger.kernel.org On Sat, Oct 03 2009, Mike Galbraith wrote: > On Sat, 2009-10-03 at 16:28 +0200, Jens Axboe wrote: > > On Sat, Oct 03 2009, Mike Galbraith wrote: > > > On Sat, 2009-10-03 at 09:56 -0400, Vivek Goyal wrote: > > > > > > > I have kept the overload delay period as "cfq_slice_sync" same as Mike had > > > > done. We shall have to experiment what is a good waiting perioed. Is 100ms > > > > too long if we are waiting for a request from same process which recently > > > > finished IO and we did not enable idle on it. > > > > > > > > I guess we can tweak the delay period as we move along. > > > > > > I kept the delay period very short to minimize possible damage. Without > > > the idle thing, it wasn't enough, but with, worked a treat, as does your > > > patch. > > > > Can you test the current line up of patches in for-linus? It has the > > ramp up I talked about included as well. > > Well, it hasn't hit git.kernel.org yet, it's at... > > * block-for-linus 1d22351 cfq-iosched: add a knob for desktop interactiveness It's the top three patches here, kernel.org sync sometimes takes a while... http://git.kernel.dk/?p=linux-2.6-block.git;a=shortlog;h=refs/heads/for-linus -- Jens Axboe