diff options
Diffstat (limited to 'block/blk-timeout.c')
-rw-r--r-- | block/blk-timeout.c | 73 |
1 files changed, 46 insertions, 27 deletions
diff --git a/block/blk-timeout.c b/block/blk-timeout.c index 65f103563969..22846cf3595a 100644 --- a/block/blk-timeout.c +++ b/block/blk-timeout.c | |||
@@ -7,6 +7,7 @@ | |||
7 | #include <linux/fault-inject.h> | 7 | #include <linux/fault-inject.h> |
8 | 8 | ||
9 | #include "blk.h" | 9 | #include "blk.h" |
10 | #include "blk-mq.h" | ||
10 | 11 | ||
11 | #ifdef CONFIG_FAIL_IO_TIMEOUT | 12 | #ifdef CONFIG_FAIL_IO_TIMEOUT |
12 | 13 | ||
@@ -88,11 +89,18 @@ static void blk_rq_timed_out(struct request *req) | |||
88 | ret = q->rq_timed_out_fn(req); | 89 | ret = q->rq_timed_out_fn(req); |
89 | switch (ret) { | 90 | switch (ret) { |
90 | case BLK_EH_HANDLED: | 91 | case BLK_EH_HANDLED: |
91 | __blk_complete_request(req); | 92 | /* Can we use req->errors here? */ |
93 | if (q->mq_ops) | ||
94 | blk_mq_complete_request(req, req->errors); | ||
95 | else | ||
96 | __blk_complete_request(req); | ||
92 | break; | 97 | break; |
93 | case BLK_EH_RESET_TIMER: | 98 | case BLK_EH_RESET_TIMER: |
94 | blk_clear_rq_complete(req); | 99 | blk_clear_rq_complete(req); |
95 | blk_add_timer(req); | 100 | if (q->mq_ops) |
101 | blk_mq_add_timer(req); | ||
102 | else | ||
103 | blk_add_timer(req); | ||
96 | break; | 104 | break; |
97 | case BLK_EH_NOT_HANDLED: | 105 | case BLK_EH_NOT_HANDLED: |
98 | /* | 106 | /* |
@@ -108,6 +116,23 @@ static void blk_rq_timed_out(struct request *req) | |||
108 | } | 116 | } |
109 | } | 117 | } |
110 | 118 | ||
119 | void blk_rq_check_expired(struct request *rq, unsigned long *next_timeout, | ||
120 | unsigned int *next_set) | ||
121 | { | ||
122 | if (time_after_eq(jiffies, rq->deadline)) { | ||
123 | list_del_init(&rq->timeout_list); | ||
124 | |||
125 | /* | ||
126 | * Check if we raced with end io completion | ||
127 | */ | ||
128 | if (!blk_mark_rq_complete(rq)) | ||
129 | blk_rq_timed_out(rq); | ||
130 | } else if (!*next_set || time_after(*next_timeout, rq->deadline)) { | ||
131 | *next_timeout = rq->deadline; | ||
132 | *next_set = 1; | ||
133 | } | ||
134 | } | ||
135 | |||
111 | void blk_rq_timed_out_timer(unsigned long data) | 136 | void blk_rq_timed_out_timer(unsigned long data) |
112 | { | 137 | { |
113 | struct request_queue *q = (struct request_queue *) data; | 138 | struct request_queue *q = (struct request_queue *) data; |
@@ -117,21 +142,8 @@ void blk_rq_timed_out_timer(unsigned long data) | |||
117 | 142 | ||
118 | spin_lock_irqsave(q->queue_lock, flags); | 143 | spin_lock_irqsave(q->queue_lock, flags); |
119 | 144 | ||
120 | list_for_each_entry_safe(rq, tmp, &q->timeout_list, timeout_list) { | 145 | list_for_each_entry_safe(rq, tmp, &q->timeout_list, timeout_list) |
121 | if (time_after_eq(jiffies, rq->deadline)) { | 146 | blk_rq_check_expired(rq, &next, &next_set); |
122 | list_del_init(&rq->timeout_list); | ||
123 | |||
124 | /* | ||
125 | * Check if we raced with end io completion | ||
126 | */ | ||
127 | if (blk_mark_rq_complete(rq)) | ||
128 | continue; | ||
129 | blk_rq_timed_out(rq); | ||
130 | } else if (!next_set || time_after(next, rq->deadline)) { | ||
131 | next = rq->deadline; | ||
132 | next_set = 1; | ||
133 | } | ||
134 | } | ||
135 | 147 | ||
136 | if (next_set) | 148 | if (next_set) |
137 | mod_timer(&q->timeout, round_jiffies_up(next)); | 149 | mod_timer(&q->timeout, round_jiffies_up(next)); |
@@ -157,15 +169,7 @@ void blk_abort_request(struct request *req) | |||
157 | } | 169 | } |
158 | EXPORT_SYMBOL_GPL(blk_abort_request); | 170 | EXPORT_SYMBOL_GPL(blk_abort_request); |
159 | 171 | ||
160 | /** | 172 | void __blk_add_timer(struct request *req, struct list_head *timeout_list) |
161 | * blk_add_timer - Start timeout timer for a single request | ||
162 | * @req: request that is about to start running. | ||
163 | * | ||
164 | * Notes: | ||
165 | * Each request has its own timer, and as it is added to the queue, we | ||
166 | * set up the timer. When the request completes, we cancel the timer. | ||
167 | */ | ||
168 | void blk_add_timer(struct request *req) | ||
169 | { | 173 | { |
170 | struct request_queue *q = req->q; | 174 | struct request_queue *q = req->q; |
171 | unsigned long expiry; | 175 | unsigned long expiry; |
@@ -184,7 +188,8 @@ void blk_add_timer(struct request *req) | |||
184 | req->timeout = q->rq_timeout; | 188 | req->timeout = q->rq_timeout; |
185 | 189 | ||
186 | req->deadline = jiffies + req->timeout; | 190 | req->deadline = jiffies + req->timeout; |
187 | list_add_tail(&req->timeout_list, &q->timeout_list); | 191 | if (timeout_list) |
192 | list_add_tail(&req->timeout_list, timeout_list); | ||
188 | 193 | ||
189 | /* | 194 | /* |
190 | * If the timer isn't already pending or this timeout is earlier | 195 | * If the timer isn't already pending or this timeout is earlier |
@@ -196,5 +201,19 @@ void blk_add_timer(struct request *req) | |||
196 | if (!timer_pending(&q->timeout) || | 201 | if (!timer_pending(&q->timeout) || |
197 | time_before(expiry, q->timeout.expires)) | 202 | time_before(expiry, q->timeout.expires)) |
198 | mod_timer(&q->timeout, expiry); | 203 | mod_timer(&q->timeout, expiry); |
204 | |||
205 | } | ||
206 | |||
207 | /** | ||
208 | * blk_add_timer - Start timeout timer for a single request | ||
209 | * @req: request that is about to start running. | ||
210 | * | ||
211 | * Notes: | ||
212 | * Each request has its own timer, and as it is added to the queue, we | ||
213 | * set up the timer. When the request completes, we cancel the timer. | ||
214 | */ | ||
215 | void blk_add_timer(struct request *req) | ||
216 | { | ||
217 | __blk_add_timer(req, &req->q->timeout_list); | ||
199 | } | 218 | } |
200 | 219 | ||