diff options
author | Mike Christie <michaelc@cs.wisc.edu> | 2013-09-18 10:33:55 -0400 |
---|---|---|
committer | Jens Axboe <axboe@kernel.dk> | 2013-09-18 10:33:55 -0400 |
commit | 7652113c2f508b1c8176640dcd034730fe79bc48 (patch) | |
tree | 826a32d8341f9747f997db6715f6cb4a802d618a | |
parent | adbe6991efd36104ac9eaf751993d35eaa7f493a (diff) |
If the queue is dying then we only call the rq->end_io callout.
This leaves bios setup on the request, because the caller assumes when
the blk_execute_rq_nowait/blk_execute_rq call has completed that
the rq->bios have been cleaned up.
This patch has blk_execute_rq_nowait use __blk_end_request_all
to free bios and also call rq->end_io.
Signed-off-by: Mike Christie <michaelc@cs.wisc.edu>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
-rw-r--r-- | block/blk-exec.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/block/blk-exec.c b/block/blk-exec.c index e70621396129..ae4f27d7944e 100644 --- a/block/blk-exec.c +++ b/block/blk-exec.c | |||
@@ -68,9 +68,9 @@ void blk_execute_rq_nowait(struct request_queue *q, struct gendisk *bd_disk, | |||
68 | spin_lock_irq(q->queue_lock); | 68 | spin_lock_irq(q->queue_lock); |
69 | 69 | ||
70 | if (unlikely(blk_queue_dying(q))) { | 70 | if (unlikely(blk_queue_dying(q))) { |
71 | rq->cmd_flags |= REQ_QUIET; | ||
71 | rq->errors = -ENXIO; | 72 | rq->errors = -ENXIO; |
72 | if (rq->end_io) | 73 | __blk_end_request_all(rq, rq->errors); |
73 | rq->end_io(rq, rq->errors); | ||
74 | spin_unlock_irq(q->queue_lock); | 74 | spin_unlock_irq(q->queue_lock); |
75 | return; | 75 | return; |
76 | } | 76 | } |