NVMe: Fix bug in error handling
When an I/O completed with an error, we would call bio_endio twice (once with -EIO and once with 0). Found by inspection. Signed-off-by: Matthew Wilcox <matthew.r.wilcox@intel.com>
This commit is contained in:
parent
22605f9681
commit
09a58f5364
@ -310,9 +310,9 @@ static void bio_completion(struct nvme_queue *nvmeq, void *ctx,
|
|||||||
dma_unmap_sg(nvmeq->q_dmadev, nbio->sg, nbio->nents,
|
dma_unmap_sg(nvmeq->q_dmadev, nbio->sg, nbio->nents,
|
||||||
bio_data_dir(bio) ? DMA_TO_DEVICE : DMA_FROM_DEVICE);
|
bio_data_dir(bio) ? DMA_TO_DEVICE : DMA_FROM_DEVICE);
|
||||||
free_nbio(nvmeq, nbio);
|
free_nbio(nvmeq, nbio);
|
||||||
if (status)
|
if (status) {
|
||||||
bio_endio(bio, -EIO);
|
bio_endio(bio, -EIO);
|
||||||
if (bio->bi_vcnt > bio->bi_idx) {
|
} else if (bio->bi_vcnt > bio->bi_idx) {
|
||||||
bio_list_add(&nvmeq->sq_cong, bio);
|
bio_list_add(&nvmeq->sq_cong, bio);
|
||||||
wake_up_process(nvme_thread);
|
wake_up_process(nvme_thread);
|
||||||
} else {
|
} else {
|
||||||
|
Loading…
Reference in New Issue
Block a user