From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jens Axboe Subject: Re: elevator priorities vs. full request queues Date: Tue, 13 Jul 2004 18:59:07 +0200 Sender: linux-fsdevel-owner@vger.kernel.org Message-ID: <20040713165906.GB29655@suse.de> References: <20040622074852.GW12881@suse.de> <20040622052644.D1325@almesberger.net> <20040622101434.GB12881@suse.de> <20040622160859.I1325@almesberger.net> <20040623101430.GI1120@suse.de> <20040712205227.A12285@almesberger.net> <20040713053749.GA14759@suse.de> <20040713092904.A1795@almesberger.net> <20040713123541.GA28956@suse.de> <20040713133632.B1795@almesberger.net> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: linux-fsdevel@vger.kernel.org Return-path: Received: from ns.virtualhost.dk ([195.184.98.160]:48840 "EHLO virtualhost.dk") by vger.kernel.org with ESMTP id S265510AbUGMQ7K (ORCPT ); Tue, 13 Jul 2004 12:59:10 -0400 To: Werner Almesberger Content-Disposition: inline In-Reply-To: <20040713133632.B1795@almesberger.net> List-Id: linux-fsdevel.vger.kernel.org On Tue, Jul 13 2004, Werner Almesberger wrote: > Jens Axboe wrote: > > It's just that if default io prio was something other than 0, then it > > would make sense to set it there. > > Oh, I see. Yes, of course. It actually does this, implicitly, > since the default value of zero just happens to be what bio_init > sets already ;-) Only if you define 0 to be the default value :-) > > But I suppose you just want io submission to set it, in which case it > > doesn't matter what bio_init() sets it to. > > Except for those bios that get submitted without ever touching > submit_bh. You mean things like bio_map_user/bio_copy_user and (at the other end, doubt these would count), bios submitted privately to the drive itself (ala ide-scsi)? The latter can be disregarded imho, nothing to worry about. The former allocate requests on its own, so you must pass in the priority there. So instead of assigning bio priority and using that for request allocation, you are just a step or two further down in the call chain and specify the priority there directly. In short, I don't see a problem there. -- Jens Axboe