From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753542AbcBJTrW (ORCPT ); Wed, 10 Feb 2016 14:47:22 -0500 Received: from ud10.udmedia.de ([194.117.254.50]:44625 "EHLO mail.ud10.udmedia.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753428AbcBJTrT (ORCPT ); Wed, 10 Feb 2016 14:47:19 -0500 Date: Wed, 10 Feb 2016 20:47:15 +0100 From: Markus Trippelsdorf To: Andreas Herrmann Cc: Christoph Hellwig , Jens Axboe , linux-kernel@vger.kernel.org, Johannes Thumshirn , Jan Kara , linux-block@vger.kernel.org, linux-scsi@vger.kernel.org, Hannes Reinecke Subject: Re: [RFC PATCH v2] blk-mq: Introduce per sw queue time-slice Message-ID: <20160210194715.GA324@x4> References: <20151119120235.GA7966@suselix.suse.de> <20160201224340.GA16639@suselix.suse.de> <20160209171239.GE12437@suselix.suse.de> <20160209174156.GA323@x4> <20160210193414.GC14676@suselix.suse.de> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20160210193414.GC14676@suselix.suse.de> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 2016.02.10 at 20:34 +0100, Andreas Herrmann wrote: > On Tue, Feb 09, 2016 at 06:41:56PM +0100, Markus Trippelsdorf wrote: > > > Recently Johannes sent a patch to enable scsi-mq per driver, see > > > http://marc.info/?l=linux-scsi&m=145347009631192&w=2 > > > > > > Probably that is a good solution (at least in the short term) to allow > > > users to switch to blk-mq for some host adapters (with fast storage > > > attached) but to stick to legacy stuff on other host adapters with > > > rotary devices. > > > > I don't think that Johannes' patch is a good solution. > > Why? Because it's not per device? Yes. Like Christoph said in his reply to the patch: »The host is simply the wrong place to decide these things.« > > The best solution for the user would be if blk-mq could be toggled > > per drive (or even automatically enabled if queue/rotational == 0). > > Yes, I aggree, but ... > > > Is there a fundamental reason why this is not feasible? > > ... it's not possible (*) with the current implementation. > > Tag handling/command allocation differs. Respective functions are set > per host. > > (*) Or maybe it's possible but just hard to achieve and I didn't look > long enough into relevant code to get an idea how to do it. > > > Your solution is better than nothing, but it requires that the user > > finds out the drive <=> host mapping by hand and then runs something > > like: > > echo "250" > /sys/devices/pci0000:00/0000:00:11.0/ata2/host1/target1:0:0/1:0:0:0/block/sdb/mq/0/time_slice_us > > during boot for spinning rust drives... > > Or it could automatically be set in case of rotational device. > (Once we know for sure that it doesn't cause performance degradation.) Yes, this sound like a good idea. But, if I understand things correctly, your patch is only an interim solution until proper I/O scheduler support gets implemented for blk-mq, no? -- Markus