[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <20090203133242.29904063@mjolnir.drzeus.cx>
Date:	Tue, 3 Feb 2009 13:32:42 +0100
From:	Pierre Ossman <drzeus-list@...eus.cx>
To:	Jens Axboe <jens.axboe@...cle.com>
Cc:	Hugh Dickins <hugh@...itas.com>,
	James Bottomley <James.Bottomley@...senPartnership.com>,
	Grant Grundler <grundler@...gle.com>,
	Greg Freemyer <greg.freemyer@...il.com>,
	Tejun Heo <htejun@...il.com>, Michael Tokarev <mjt@....msk.ru>,
	Kay Sievers <kay.sievers@...y.org>,
	Bartlomiej Zolnierkiewicz <bzolnier@...il.com>,
	linux-kernel@...r.kernel.org, linux-ide@...r.kernel.org,
	Alan Cox <alan@...rguk.ukuu.org.uk>
Subject: Re: [PATCH] block: export SSD/non-rotational queue flag through
 sysfs
On Fri, 16 Jan 2009 07:48:18 +0100
Jens Axboe <jens.axboe@...cle.com> wrote:
> 
> For non-queuing devices, I think our current check is a bit too drastic.
> We probably want to change that to
> 
> int dont_plug(q)
> {
>         return blk_queue_nonrot(q) && blk_queue_tagged(q);
> }
> 
Please do. The MMC block driver sets nonrot as of late, but it is
extremely dependent on merging. The command overhead is just silly, so
to get anywhere near the speed numbers indicated in marketing you need
large requests.
It seems that too many confuse "non-rotational" with "low latency",
which isn't always the same. :)
Rgds
-- 
     -- Pierre Ossman
  Linux kernel, MMC maintainer        http://www.kernel.org
  rdesktop, core developer          http://www.rdesktop.org
  WARNING: This correspondence is being monitored by the
  Swedish government. Make sure your server uses encryption
  for SMTP traffic and consider using PGP for end-to-end
  encryption.
Download attachment "signature.asc" of type "application/pgp-signature" (198 bytes)
Powered by blists - more mailing lists
 
