aboutsummaryrefslogtreecommitdiff
path: root/block/elevator.c
diff options
context:
space:
mode:
authorGravatar Davide Sapienza <sapienza.dav@gmail.com> 2018-05-31 16:45:08 +0200
committerGravatar Jens Axboe <axboe@kernel.dk> 2018-05-31 08:54:41 -0600
commitf6c3ca0e58446e8de6c6f1a2a73f037fb900c372 (patch)
tree4bb57aa40714ad8cdeff7eebe0ecd26a08d13662 /block/elevator.c
parentblock, bfq: increase weight-raising duration for interactive apps (diff)
downloadlinux-f6c3ca0e58446e8de6c6f1a2a73f037fb900c372.tar.gz
linux-f6c3ca0e58446e8de6c6f1a2a73f037fb900c372.tar.bz2
linux-f6c3ca0e58446e8de6c6f1a2a73f037fb900c372.zip
block, bfq: prevent soft_rt_next_start from being stuck at infinity
BFQ can deem a bfq_queue as soft real-time only if the queue - periodically becomes completely idle, i.e., empty and with no still-outstanding I/O request; - after becoming idle, gets new I/O only after a special reference time soft_rt_next_start. In this respect, after commit "block, bfq: consider also past I/O in soft real-time detection", the value of soft_rt_next_start can never decrease. This causes a problem with the following special updating case for soft_rt_next_start: to prevent queues that are not completely idle to be wrongly detected as soft real-time (when they become non-empty again), soft_rt_next_start is temporarily set to infinity for empty queues with still outstanding I/O requests. But, if such an update is actually performed, then, because of the above commit, soft_rt_next_start will be stuck at infinity forever, and the queue will have no more chance to be considered soft real-time. On slow systems, this problem does cause actual soft real-time applications to be occasionally not detected as such. This commit addresses this issue by eliminating the pushing of soft_rt_next_start to infinity, and by changing the way non-empty queues are prevented from being wrongly detected as soft real-time. Simply, a queue that becomes non-empty again can now be detected as soft real-time only if it has no outstanding I/O request. Signed-off-by: Davide Sapienza <sapienza.dav@gmail.com> Signed-off-by: Paolo Valente <paolo.valente@linaro.org> Signed-off-by: Jens Axboe <axboe@kernel.dk>
Diffstat (limited to 'block/elevator.c')
0 files changed, 0 insertions, 0 deletions