diff options
| author | Chaitanya Kulkarni <chaitanya.kulkarni@wdc.com> | 2020-06-04 00:13:28 -0700 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2020-06-25 15:33:02 +0200 |
| commit | 48adb95b73616ed356406811159f0dd40be5b0ef (patch) | |
| tree | e539ab7b21a8fb97221a4c977e6881fd91641915 /kernel | |
| parent | fb12a1a2108a94bb04381bd9717747d23e6de0a8 (diff) | |
| download | linux-stable-48adb95b73616ed356406811159f0dd40be5b0ef.tar.gz linux-stable-48adb95b73616ed356406811159f0dd40be5b0ef.tar.bz2 linux-stable-48adb95b73616ed356406811159f0dd40be5b0ef.zip | |
blktrace: use errno instead of bi_status
[ Upstream commit 48bc3cd3e07a1486f45d9971c75d6090976c3b1b ]
In blk_add_trace_spliti() blk_add_trace_bio_remap() use
blk_status_to_errno() to pass the error instead of pasing the bi_status.
This fixes the sparse warning.
Signed-off-by: Chaitanya Kulkarni <chaitanya.kulkarni@wdc.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
Signed-off-by: Sasha Levin <sashal@kernel.org>
Diffstat (limited to 'kernel')
| -rw-r--r-- | kernel/trace/blktrace.c | 9 |
1 files changed, 6 insertions, 3 deletions
diff --git a/kernel/trace/blktrace.c b/kernel/trace/blktrace.c index 6cea8bbca03c..5c84f7421871 100644 --- a/kernel/trace/blktrace.c +++ b/kernel/trace/blktrace.c @@ -1004,8 +1004,10 @@ static void blk_add_trace_split(void *ignore, __blk_add_trace(bt, bio->bi_iter.bi_sector, bio->bi_iter.bi_size, bio_op(bio), bio->bi_opf, - BLK_TA_SPLIT, bio->bi_status, sizeof(rpdu), - &rpdu, blk_trace_bio_get_cgid(q, bio)); + BLK_TA_SPLIT, + blk_status_to_errno(bio->bi_status), + sizeof(rpdu), &rpdu, + blk_trace_bio_get_cgid(q, bio)); } rcu_read_unlock(); } @@ -1042,7 +1044,8 @@ static void blk_add_trace_bio_remap(void *ignore, r.sector_from = cpu_to_be64(from); __blk_add_trace(bt, bio->bi_iter.bi_sector, bio->bi_iter.bi_size, - bio_op(bio), bio->bi_opf, BLK_TA_REMAP, bio->bi_status, + bio_op(bio), bio->bi_opf, BLK_TA_REMAP, + blk_status_to_errno(bio->bi_status), sizeof(r), &r, blk_trace_bio_get_cgid(q, bio)); rcu_read_unlock(); } |