cfq-iosched: make nonrot check logic consistent
authorShaohua Li <shaohua.li@intel.com>
Fri, 4 Dec 2009 12:12:06 +0000 (13:12 +0100)
committerJens Axboe <jens.axboe@oracle.com>
Fri, 4 Dec 2009 12:12:06 +0000 (13:12 +0100)
cfq_arm_slice_timer() has logic to disable idle window for SSD device. The same
thing should be done at cfq_select_queue() too, otherwise we will still see
idle window. This makes the nonrot check logic consistent in cfq.
Tests in a intel SSD with low_latency knob close, below patch can triple disk
thoughput for muti-thread sequential read.

Signed-off-by: Shaohua Li <shaohua.li@intel.com>
Signed-off-by: Jens Axboe <jens.axboe@oracle.com>
block/cfq-iosched.c

index 43ec3340cb582bff0f1e7dfd1d1fe906a2696544..b00ca4c86e257b7d4520086bd18c9dd530efd8b1 100644 (file)
@@ -1796,7 +1796,8 @@ static bool cfq_should_idle(struct cfq_data *cfqd, struct cfq_queue *cfqq)
                return false;
 
        /* We do for queues that were marked with idle window flag. */
-       if (cfq_cfqq_idle_window(cfqq))
+       if (cfq_cfqq_idle_window(cfqq) &&
+          !(blk_queue_nonrot(cfqd->queue) && cfqd->hw_tag))
                return true;
 
        /*