diff options
author | Jens Axboe <jens.axboe@oracle.com> | 2007-07-10 13:43:25 +0200 |
---|---|---|
committer | Jens Axboe <jens.axboe@oracle.com> | 2007-07-10 13:43:25 +0200 |
commit | 15c31be4d5bd2402c6f5a288d56a24edc9252b71 (patch) | |
tree | 0fca6e97186080d83ff3f36bb359bcb4ef06a9e2 /include/linux/ioprio.h | |
parent | 72d3a38ee083a96c09032e608a4c7e047ce26760 (diff) | |
download | lwn-15c31be4d5bd2402c6f5a288d56a24edc9252b71.tar.gz lwn-15c31be4d5bd2402c6f5a288d56a24edc9252b71.zip |
cfq-iosched: fix async queue behaviour
With the cfq_queue hash removal, we inadvertently got rid of the
async queue sharing. This was not intentional, in fact CFQ purposely
shares the async queue per priority level to get good merging for
async writes.
So put some logic in cfq_get_queue() to track the shared queues.
Signed-off-by: Jens Axboe <jens.axboe@oracle.com>
Diffstat (limited to 'include/linux/ioprio.h')
-rw-r--r-- | include/linux/ioprio.h | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/include/linux/ioprio.h b/include/linux/ioprio.h index 8e2042b9d471..2eaa142cd061 100644 --- a/include/linux/ioprio.h +++ b/include/linux/ioprio.h @@ -47,8 +47,10 @@ enum { #define IOPRIO_NORM (4) static inline int task_ioprio(struct task_struct *task) { - WARN_ON(!ioprio_valid(task->ioprio)); - return IOPRIO_PRIO_DATA(task->ioprio); + if (ioprio_valid(task->ioprio)) + return IOPRIO_PRIO_DATA(task->ioprio); + + return IOPRIO_NORM; } static inline int task_nice_ioprio(struct task_struct *task) |