summaryrefslogtreecommitdiff
path: root/block
diff options
context:
space:
mode:
authorDivyesh Shah <dpshah@google.com>2010-03-25 17:45:57 +0300
committerJens Axboe <jens.axboe@oracle.com>2010-03-25 17:46:38 +0300
commit39c01b219fd30c74869b6fc8749f7900f04e9ef6 (patch)
tree7ce59210c28e2b53daf2957174d3ae164701b181 /block
parentb1ffe737f5b743115ee46ffb59e338e580c54902 (diff)
downloadlinux-39c01b219fd30c74869b6fc8749f7900f04e9ef6.tar.xz
cfq-iosched: Do not merge queues of BE and IDLE classes
Even if they are found to be co-operating. The prio_trees do not have any IDLE cfqqs on them. cfq_close_cooperator() is called from cfq_select_queue() and cfq_completed_request(). The latter ensures that the close cooperator code does not get invoked if the current cfqq is of class IDLE but the former doesn't seem to have any such checks. So an IDLE cfqq may get merged with a BE cfqq from the same group which should be avoided. Signed-off-by: Divyesh Shah<dpshah@google.com> Acked-by: Vivek Goyal <vgoyal@redhat.com> Signed-off-by: Jens Axboe <jens.axboe@oracle.com>
Diffstat (limited to 'block')
-rw-r--r--block/cfq-iosched.c2
1 files changed, 2 insertions, 0 deletions
diff --git a/block/cfq-iosched.c b/block/cfq-iosched.c
index 2f91c5351949..2c7a0f4f3cd7 100644
--- a/block/cfq-iosched.c
+++ b/block/cfq-iosched.c
@@ -1723,6 +1723,8 @@ static struct cfq_queue *cfq_close_cooperator(struct cfq_data *cfqd,
{
struct cfq_queue *cfqq;
+ if (cfq_class_idle(cur_cfqq))
+ return NULL;
if (!cfq_cfqq_sync(cur_cfqq))
return NULL;
if (CFQQ_SEEKY(cur_cfqq))