diff options
| author | Bart Van Assche <bart.vanassche@wdc.com> | 2018-06-22 16:18:09 -0400 |
|---|---|---|
| committer | Jens Axboe <axboe@kernel.dk> | 2018-06-23 12:25:45 -0400 |
| commit | f5e350f021e04ea41d2e5d58487c33b05ba3d25b (patch) | |
| tree | 6cf89499e307b48964e4539842f502575617afe7 | |
| parent | 3ee7e8697d5860b173132606d80a9cd35e7113ee (diff) | |
blk-mq: Fix timeout handling in case the timeout handler returns BLK_EH_DONE
Make sure that RQF_TIMED_OUT is cleared when a request is reused
after a block driver timeout handler has returned BLK_EH_DONE.
Fixes: da6612673988 ("blk-mq: don't time out requests again that are in the timeout handler")
Signed-off-by: Bart Van Assche <bart.vanassche@wdc.com>
Cc: Christoph Hellwig <hch@lst.de>
Cc: Jianchao Wang <jianchao.w.wang@oracle.com>
Cc: Andrew Randrianasulu <randrianasulu@gmail.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
| -rw-r--r-- | block/blk-mq.c | 1 | ||||
| -rw-r--r-- | block/blk-timeout.c | 1 |
2 files changed, 1 insertions, 1 deletions
diff --git a/block/blk-mq.c b/block/blk-mq.c index 8e57b84e50e9..b6888ff556cf 100644 --- a/block/blk-mq.c +++ b/block/blk-mq.c | |||
| @@ -781,7 +781,6 @@ static void blk_mq_rq_timed_out(struct request *req, bool reserved) | |||
| 781 | WARN_ON_ONCE(ret != BLK_EH_RESET_TIMER); | 781 | WARN_ON_ONCE(ret != BLK_EH_RESET_TIMER); |
| 782 | } | 782 | } |
| 783 | 783 | ||
| 784 | req->rq_flags &= ~RQF_TIMED_OUT; | ||
| 785 | blk_add_timer(req); | 784 | blk_add_timer(req); |
| 786 | } | 785 | } |
| 787 | 786 | ||
diff --git a/block/blk-timeout.c b/block/blk-timeout.c index 4b8a48d48ba1..f2cfd56e1606 100644 --- a/block/blk-timeout.c +++ b/block/blk-timeout.c | |||
| @@ -210,6 +210,7 @@ void blk_add_timer(struct request *req) | |||
| 210 | if (!req->timeout) | 210 | if (!req->timeout) |
| 211 | req->timeout = q->rq_timeout; | 211 | req->timeout = q->rq_timeout; |
| 212 | 212 | ||
| 213 | req->rq_flags &= ~RQF_TIMED_OUT; | ||
| 213 | blk_rq_set_deadline(req, jiffies + req->timeout); | 214 | blk_rq_set_deadline(req, jiffies + req->timeout); |
| 214 | 215 | ||
| 215 | /* | 216 | /* |
