Re: [PATCH] cfq-iosched: NCQ SSDs do not need read queue merging

From: Jeff Garzik
Date: Mon Jan 11 2010 - 06:25:38 EST


On 01/10/2010 04:04 PM, Corrado Zoccolo wrote:
NCQ SSDs' performances are not affected by
distance of read requests, so there is no point in having
overhead to merge such queues.

Non-NCQ SSDs showed regression in some special cases, so
they are ruled out by this patch.

This patch intentionally doesn't affect writes, so
it changes the queued[] field, to be indexed by
READ/WRITE instead of SYNC/ASYNC, and only compute proximity
for queues with WRITE requests.

Signed-off-by: Corrado Zoccolo<czoccolo@xxxxxxxxx>

That's not really true. Overhead always increases as the total number of ATA commands issued increases.

Jeff




--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/