[lkml]   [2010]   [Jun]   [21]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    Patch in this message
    Subject[PATCH 1/2] cfq: always return false from should_idle if slice_idle is set to zero

    In testing a competing fsync-ing process and a sequential reader on
    mid-grade storage, I found that cfq was incapable of achieving the I/O
    rates of deadline, no matter how it was tuned. Investigation, and insight
    from Vivek (mostly the latter), led to identifying that we were still
    idling for the last queue in the service tree.

    Modifying cfq_should_idle to not idle when slice_idle is set to zero got
    us much closer to the performance of deadline for this workload. I have
    one follow-on patch that gets us on-par with deadline, but I think this
    patch stands alone.

    Comments, as always, are appreciated.


    Signed-off-by: Jeff Moyer <>
    block/cfq-iosched.c | 4 ++++
    1 files changed, 4 insertions(+), 0 deletions(-)

    diff --git a/block/cfq-iosched.c b/block/cfq-iosched.c
    index 5ff4f48..572a050 100644
    --- a/block/cfq-iosched.c
    +++ b/block/cfq-iosched.c
    @@ -1842,6 +1842,10 @@ static bool cfq_should_idle(struct cfq_data *cfqd, struct cfq_queue *cfqq)
    if (prio == IDLE_WORKLOAD)
    return false;

    + /* Don't idle if slice idling is disabled by the user */
    + if (cfqd->cfq_slice_idle == 0)
    + return false;
    /* We do for queues that were marked with idle window flag. */
    if (cfq_cfqq_idle_window(cfqq) &&
    !(blk_queue_nonrot(cfqd->queue) && cfqd->hw_tag))

     \ /
      Last update: 2010-06-21 21:53    [W:0.164 / U:0.076 seconds]
    ©2003-2017 Jasper Spaans. hosted at Digital OceanAdvertise on this site