Commit f7d7b7a7 authored by Jens Axboe's avatar Jens Axboe

block: as/cfq ssd idle check update

We really need to know about the hardware tagging support as well,
since if the SSD does not do tagging then we still want to idle.
Otherwise have the same dependent sync IO vs flooding async IO
problem as on rotational media.
Signed-off-by: default avatarJens Axboe <jens.axboe@oracle.com>
parent 8bff7c6b
...@@ -745,11 +745,13 @@ static int as_can_break_anticipation(struct as_data *ad, struct request *rq) ...@@ -745,11 +745,13 @@ static int as_can_break_anticipation(struct as_data *ad, struct request *rq)
*/ */
static int as_can_anticipate(struct as_data *ad, struct request *rq) static int as_can_anticipate(struct as_data *ad, struct request *rq)
{ {
#if 0 /* disable for now, we need to check tag level as well */
/* /*
* SSD device without seek penalty, disable idling * SSD device without seek penalty, disable idling
*/ */
if (blk_queue_nonrot(ad->q)) if (blk_queue_nonrot(ad->q)) axman
return 0; return 0;
#endif
if (!ad->io_context) if (!ad->io_context)
/* /*
......
...@@ -879,9 +879,11 @@ static void cfq_arm_slice_timer(struct cfq_data *cfqd) ...@@ -879,9 +879,11 @@ static void cfq_arm_slice_timer(struct cfq_data *cfqd)
unsigned long sl; unsigned long sl;
/* /*
* SSD device without seek penalty, disable idling * SSD device without seek penalty, disable idling. But only do so
* for devices that support queuing, otherwise we still have a problem
* with sync vs async workloads.
*/ */
if (blk_queue_nonrot(cfqd->queue)) if (blk_queue_nonrot(cfqd->queue) && cfqd->hw_tag)
return; return;
WARN_ON(!RB_EMPTY_ROOT(&cfqq->sort_list)); WARN_ON(!RB_EMPTY_ROOT(&cfqq->sort_list));
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment