diff options
author | Christoph Hellwig <hch@lst.de> | 2017-06-03 03:38:06 -0400 |
---|---|---|
committer | Jens Axboe <axboe@fb.com> | 2017-06-09 11:27:32 -0400 |
commit | 4e4cbee93d56137ebff722be022cae5f70ef84fb (patch) | |
tree | 4fa7345155599fc6bdd653fca8c5224ddf90a5be /drivers/lightnvm | |
parent | fc17b6534eb8395f0b3133eb31d87deec32c642b (diff) |
block: switch bios to blk_status_t
Replace bi_error with a new bi_status to allow for a clear conversion.
Note that device mapper overloaded bi_error with a private value, which
we'll have to keep arround at least for now and thus propagate to a
proper blk_status_t value.
Signed-off-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Jens Axboe <axboe@fb.com>
Diffstat (limited to 'drivers/lightnvm')
-rw-r--r-- | drivers/lightnvm/pblk-core.c | 4 | ||||
-rw-r--r-- | drivers/lightnvm/pblk-read.c | 4 | ||||
-rw-r--r-- | drivers/lightnvm/pblk-write.c | 2 | ||||
-rw-r--r-- | drivers/lightnvm/rrpc.c | 8 |
4 files changed, 9 insertions, 9 deletions
diff --git a/drivers/lightnvm/pblk-core.c b/drivers/lightnvm/pblk-core.c index 5e44768ccffa..4e0de995cd90 100644 --- a/drivers/lightnvm/pblk-core.c +++ b/drivers/lightnvm/pblk-core.c | |||
@@ -296,8 +296,8 @@ void pblk_flush_writer(struct pblk *pblk) | |||
296 | pr_err("pblk: tear down bio failed\n"); | 296 | pr_err("pblk: tear down bio failed\n"); |
297 | } | 297 | } |
298 | 298 | ||
299 | if (bio->bi_error) | 299 | if (bio->bi_status) |
300 | pr_err("pblk: flush sync write failed (%u)\n", bio->bi_error); | 300 | pr_err("pblk: flush sync write failed (%u)\n", bio->bi_status); |
301 | 301 | ||
302 | bio_put(bio); | 302 | bio_put(bio); |
303 | } | 303 | } |
diff --git a/drivers/lightnvm/pblk-read.c b/drivers/lightnvm/pblk-read.c index 4a12f14d78c6..762c0b73cb67 100644 --- a/drivers/lightnvm/pblk-read.c +++ b/drivers/lightnvm/pblk-read.c | |||
@@ -114,7 +114,7 @@ static void pblk_end_io_read(struct nvm_rq *rqd) | |||
114 | pblk_log_read_err(pblk, rqd); | 114 | pblk_log_read_err(pblk, rqd); |
115 | #ifdef CONFIG_NVM_DEBUG | 115 | #ifdef CONFIG_NVM_DEBUG |
116 | else | 116 | else |
117 | WARN_ONCE(bio->bi_error, "pblk: corrupted read error\n"); | 117 | WARN_ONCE(bio->bi_status, "pblk: corrupted read error\n"); |
118 | #endif | 118 | #endif |
119 | 119 | ||
120 | if (rqd->nr_ppas > 1) | 120 | if (rqd->nr_ppas > 1) |
@@ -123,7 +123,7 @@ static void pblk_end_io_read(struct nvm_rq *rqd) | |||
123 | bio_put(bio); | 123 | bio_put(bio); |
124 | if (r_ctx->orig_bio) { | 124 | if (r_ctx->orig_bio) { |
125 | #ifdef CONFIG_NVM_DEBUG | 125 | #ifdef CONFIG_NVM_DEBUG |
126 | WARN_ONCE(r_ctx->orig_bio->bi_error, | 126 | WARN_ONCE(r_ctx->orig_bio->bi_status, |
127 | "pblk: corrupted read bio\n"); | 127 | "pblk: corrupted read bio\n"); |
128 | #endif | 128 | #endif |
129 | bio_endio(r_ctx->orig_bio); | 129 | bio_endio(r_ctx->orig_bio); |
diff --git a/drivers/lightnvm/pblk-write.c b/drivers/lightnvm/pblk-write.c index aef6fd7c4a0c..79b90d8dbcb3 100644 --- a/drivers/lightnvm/pblk-write.c +++ b/drivers/lightnvm/pblk-write.c | |||
@@ -186,7 +186,7 @@ static void pblk_end_io_write(struct nvm_rq *rqd) | |||
186 | } | 186 | } |
187 | #ifdef CONFIG_NVM_DEBUG | 187 | #ifdef CONFIG_NVM_DEBUG |
188 | else | 188 | else |
189 | WARN_ONCE(rqd->bio->bi_error, "pblk: corrupted write error\n"); | 189 | WARN_ONCE(rqd->bio->bi_status, "pblk: corrupted write error\n"); |
190 | #endif | 190 | #endif |
191 | 191 | ||
192 | pblk_complete_write(pblk, rqd, c_ctx); | 192 | pblk_complete_write(pblk, rqd, c_ctx); |
diff --git a/drivers/lightnvm/rrpc.c b/drivers/lightnvm/rrpc.c index cf0e28a0ff61..8d3b53bb3307 100644 --- a/drivers/lightnvm/rrpc.c +++ b/drivers/lightnvm/rrpc.c | |||
@@ -279,8 +279,8 @@ static void rrpc_end_sync_bio(struct bio *bio) | |||
279 | { | 279 | { |
280 | struct completion *waiting = bio->bi_private; | 280 | struct completion *waiting = bio->bi_private; |
281 | 281 | ||
282 | if (bio->bi_error) | 282 | if (bio->bi_status) |
283 | pr_err("nvm: gc request failed (%u).\n", bio->bi_error); | 283 | pr_err("nvm: gc request failed (%u).\n", bio->bi_status); |
284 | 284 | ||
285 | complete(waiting); | 285 | complete(waiting); |
286 | } | 286 | } |
@@ -359,7 +359,7 @@ try: | |||
359 | goto finished; | 359 | goto finished; |
360 | } | 360 | } |
361 | wait_for_completion_io(&wait); | 361 | wait_for_completion_io(&wait); |
362 | if (bio->bi_error) { | 362 | if (bio->bi_status) { |
363 | rrpc_inflight_laddr_release(rrpc, rqd); | 363 | rrpc_inflight_laddr_release(rrpc, rqd); |
364 | goto finished; | 364 | goto finished; |
365 | } | 365 | } |
@@ -385,7 +385,7 @@ try: | |||
385 | wait_for_completion_io(&wait); | 385 | wait_for_completion_io(&wait); |
386 | 386 | ||
387 | rrpc_inflight_laddr_release(rrpc, rqd); | 387 | rrpc_inflight_laddr_release(rrpc, rqd); |
388 | if (bio->bi_error) | 388 | if (bio->bi_status) |
389 | goto finished; | 389 | goto finished; |
390 | 390 | ||
391 | bio_reset(bio); | 391 | bio_reset(bio); |