diff options
author | Ming Lei <ming.lei@canonical.com> | 2014-09-19 09:53:46 -0400 |
---|---|---|
committer | Jens Axboe <axboe@fb.com> | 2014-09-22 14:00:08 -0400 |
commit | 5e940aaa597c15e916618240ae5838864f36c91e (patch) | |
tree | 7b38546a459da2783a5f07de301eb11756ac66d3 /block/blk-timeout.c | |
parent | aedcd72f6c283dffefbb8b808ae67bdd2c6eb11a (diff) |
blk-timeout: fix blk_add_timer
Commit 8cb34819cdd5d(blk-mq: unshared timeout handler) introduces
blk-mq's own timeout handler, and removes following line:
blk_queue_rq_timed_out(q, blk_mq_rq_timed_out);
which then causes blk_add_timer() to bypass adding the timer,
since blk-mq no longer has q->rq_timed_out_fn defined.
This patch fixes the problem by bypassing the check for blk-mq,
so that both request deadlines are still set and the rolling
timer updated.
Signed-off-by: Ming Lei <ming.lei@canonical.com>
Signed-off-by: Jens Axboe <axboe@fb.com>
Diffstat (limited to 'block/blk-timeout.c')
-rw-r--r-- | block/blk-timeout.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/block/blk-timeout.c b/block/blk-timeout.c index 4d448259e622..8bae410b8a1b 100644 --- a/block/blk-timeout.c +++ b/block/blk-timeout.c | |||
@@ -186,7 +186,8 @@ void blk_add_timer(struct request *req) | |||
186 | struct request_queue *q = req->q; | 186 | struct request_queue *q = req->q; |
187 | unsigned long expiry; | 187 | unsigned long expiry; |
188 | 188 | ||
189 | if (!q->rq_timed_out_fn) | 189 | /* blk-mq has its own handler, so we don't need ->rq_timed_out_fn */ |
190 | if (!q->mq_ops && !q->rq_timed_out_fn) | ||
190 | return; | 191 | return; |
191 | 192 | ||
192 | BUG_ON(!list_empty(&req->timeout_list)); | 193 | BUG_ON(!list_empty(&req->timeout_list)); |