Commit 9a7a67af authored by Jens Axboe's avatar Jens Axboe Committed by Linus Torvalds

[PATCH] fix ordering on requeued request drainage

Previously, if a fs request which was being drained failed and got
requeued, blk_do_ordered() didn't allow it to be reissued, which causes
queue stall.  This patch makes blk_do_ordered() use the sequence of each
request to determine whether a request can be issued or not.  This fixes
the bug and simplifies code.
Signed-off-by: default avatarTejun Heo <htejun@gmail.com>
Acked-by: default avatarJens Axboe <axboe@suse.de>
Signed-off-by: default avatarAndrew Morton <akpm@osdl.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@osdl.org>
parent 88a2a4ac
...@@ -508,7 +508,7 @@ static inline struct request *start_ordered(request_queue_t *q, ...@@ -508,7 +508,7 @@ static inline struct request *start_ordered(request_queue_t *q,
int blk_do_ordered(request_queue_t *q, struct request **rqp) int blk_do_ordered(request_queue_t *q, struct request **rqp)
{ {
struct request *rq = *rqp, *allowed_rq; struct request *rq = *rqp;
int is_barrier = blk_fs_request(rq) && blk_barrier_rq(rq); int is_barrier = blk_fs_request(rq) && blk_barrier_rq(rq);
if (!q->ordseq) { if (!q->ordseq) {
...@@ -532,31 +532,25 @@ int blk_do_ordered(request_queue_t *q, struct request **rqp) ...@@ -532,31 +532,25 @@ int blk_do_ordered(request_queue_t *q, struct request **rqp)
} }
} }
/*
* Ordered sequence in progress
*/
/* Special requests are not subject to ordering rules. */
if (!blk_fs_request(rq) &&
rq != &q->pre_flush_rq && rq != &q->post_flush_rq)
return 1;
if (q->ordered & QUEUE_ORDERED_TAG) { if (q->ordered & QUEUE_ORDERED_TAG) {
/* Ordered by tag. Blocking the next barrier is enough. */
if (is_barrier && rq != &q->bar_rq) if (is_barrier && rq != &q->bar_rq)
*rqp = NULL; *rqp = NULL;
return 1; } else {
} /* Ordered by draining. Wait for turn. */
WARN_ON(blk_ordered_req_seq(rq) < blk_ordered_cur_seq(q));
switch (blk_ordered_cur_seq(q)) { if (blk_ordered_req_seq(rq) > blk_ordered_cur_seq(q))
case QUEUE_ORDSEQ_PREFLUSH:
allowed_rq = &q->pre_flush_rq;
break;
case QUEUE_ORDSEQ_BAR:
allowed_rq = &q->bar_rq;
break;
case QUEUE_ORDSEQ_POSTFLUSH:
allowed_rq = &q->post_flush_rq;
break;
default:
allowed_rq = NULL;
break;
}
if (rq != allowed_rq &&
(blk_fs_request(rq) || rq == &q->pre_flush_rq ||
rq == &q->post_flush_rq))
*rqp = NULL; *rqp = NULL;
}
return 1; return 1;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment