From: Josef Bacik Date: Fri, 18 Feb 2022 15:03:25 +0000 (-0500) Subject: btrfs: remove the bio argument from finish_compressed_bio_read X-Git-Url: http://git.maquefel.me/?a=commitdiff_plain;h=e14bfdb5a1f5fbb0a45f64a763c8da00637072d1;p=linux.git btrfs: remove the bio argument from finish_compressed_bio_read This bio is usually one of the compressed bio's, and we don't actually need it in this function, so remove the argument and stop passing it around. Reviewed-by: Johannes Thumshirn Signed-off-by: Josef Bacik Reviewed-by: David Sterba Signed-off-by: David Sterba --- diff --git a/fs/btrfs/compression.c b/fs/btrfs/compression.c index cba3a844f21d5..14409b3c838a1 100644 --- a/fs/btrfs/compression.c +++ b/fs/btrfs/compression.c @@ -234,7 +234,7 @@ static bool dec_and_test_compressed_bio(struct compressed_bio *cb, struct bio *b return last_io; } -static void finish_compressed_bio_read(struct compressed_bio *cb, struct bio *bio) +static void finish_compressed_bio_read(struct compressed_bio *cb) { unsigned int index; struct page *page; @@ -253,8 +253,6 @@ static void finish_compressed_bio_read(struct compressed_bio *cb, struct bio *bi struct bio_vec *bvec; struct bvec_iter_all iter_all; - ASSERT(bio); - ASSERT(!bio->bi_status); /* * We have verified the checksum already, set page checked so * the end_io handlers know about it @@ -325,7 +323,7 @@ static void end_compressed_bio_read(struct bio *bio) csum_failed: if (ret) cb->errors = 1; - finish_compressed_bio_read(cb, bio); + finish_compressed_bio_read(cb); out: bio_put(bio); } @@ -973,7 +971,7 @@ finish_cb: */ ASSERT(refcount_read(&cb->pending_sectors)); /* Now we are the only one referring @cb, can finish it safely. */ - finish_compressed_bio_read(cb, NULL); + finish_compressed_bio_read(cb); return ret; }