diff options
| author |   Christoph Hellwig <hch@lst.de> | 2022-07-07 07:33:31 +0200 |
|---|---|---|
| committer |   David Sterba <dsterba@suse.com> | 2022-07-25 19:56:16 +0200 |
| commit | 0b078d9db8793b1bd911e97be854e3c964235c78 (patch) | |
| tree | cab37582986dbf785fd082cc79590e1f7ab842b9 /fs/btrfs/inode.c | |
| parent | btrfs: fix repair of compressed extents (diff) | |
| download | linux-dev-0b078d9db8793b1bd911e97be854e3c964235c78.tar.xz linux-dev-0b078d9db8793b1bd911e97be854e3c964235c78.zip | |
btrfs: don't call btrfs_page_set_checked in finish_compressed_bio_read
This flag was used to communicate that the low-level compression code
already did verify the checksum to the high-level I/O completion code.
But it has been unused for a long time as the upper btrfs_bio for the
decompressed data had a NULL csum pointer basically since that pointer
existed and the code already checks for that a little later.
Note that this does not affect the other use of the checked flag, which
is only used for the COW fixup worker.
Reviewed-by: Nikolay Borisov <nborisov@suse.com>
Signed-off-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: David Sterba <dsterba@suse.com>
Diffstat (limited to 'fs/btrfs/inode.c')
| -rw-r--r-- | fs/btrfs/inode.c | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/fs/btrfs/inode.c b/fs/btrfs/inode.c index e8021d52c846..ecc5fa3343fc 100644 --- a/fs/btrfs/inode.c +++ b/fs/btrfs/inode.c @@ -3504,11 +3504,6 @@ unsigned int btrfs_verify_data_csum(struct btrfs_bio *bbio, u32 pg_off; unsigned int result = 0; - if (btrfs_page_test_checked(fs_info, page, start, end + 1 - start)) { - btrfs_page_clear_checked(fs_info, page, start, end + 1 - start); - return 0; - } - /* * This only happens for NODATASUM or compressed read. * Normally this should be covered by above check for compressed read |