diff options
| author | Josef Bacik | 2022-02-18 10:03:24 -0500 |
|---|---|---|
| committer | David Sterba | 2022-03-14 13:13:51 +0100 |
| commit | b0bbc8a3d49371a13ef9a5ff54fe2edfb4fb02d7 (patch) | |
| tree | 36acc071737aaa3f25d3e78b4429778baa456f28 /fs/btrfs | |
| parent | 1784b7d502a94b561eae58249adde5f72c26eb3c (diff) | |
btrfs: check correct bio in finish_compressed_bio_read
Commit c09abff87f90 ("btrfs: cloned bios must not be iterated by
bio_for_each_segment_all") added ASSERT()'s to make sure we weren't
calling bio_for_each_segment_all() on a RAID5/6 bio. However it was
checking the bio that the compression code passed in, not the
cb->orig_bio that we actually iterate over, so adjust this ASSERT() to
check the correct bio.
Reviewed-by: Boris Burkov <boris@bur.io>
Reviewed-by: Johannes Thumshirn <johannes.thumshirn@wdc.com>
Signed-off-by: Josef Bacik <josef@toxicpanda.com>
Reviewed-by: David Sterba <dsterba@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
Diffstat (limited to 'fs/btrfs')
| -rw-r--r-- | fs/btrfs/compression.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/btrfs/compression.c b/fs/btrfs/compression.c index 71d26b8a9396..cba3a844f21d 100644 --- a/fs/btrfs/compression.c +++ b/fs/btrfs/compression.c @@ -259,7 +259,7 @@ static void finish_compressed_bio_read(struct compressed_bio *cb, struct bio *bi * We have verified the checksum already, set page checked so * the end_io handlers know about it */ - ASSERT(!bio_flagged(bio, BIO_CLONED)); + ASSERT(!bio_flagged(cb->orig_bio, BIO_CLONED)); bio_for_each_segment_all(bvec, cb->orig_bio, iter_all) { u64 bvec_start = page_offset(bvec->bv_page) + bvec->bv_offset; |