diff options
| author |   Qu Wenruo <quwenruo@cn.fujitsu.com> | 2017-02-09 10:45:06 +0800 |
|---|---|---|
| committer |   David Sterba <dsterba@suse.com> | 2017-02-17 12:03:48 +0100 |
| commit | 6f6b643e44ef79cbd85e69800a06326805829aa8 (patch) | |
| tree | 7d04cfdabebaa9da49fc5f36c83024e7d882fbb7 /fs/btrfs/compression.c | |
| parent | Btrfs: add another missing end_page_writeback on submit_extent_page failure (diff) | |
| download | linux-dev-6f6b643e44ef79cbd85e69800a06326805829aa8.tar.xz linux-dev-6f6b643e44ef79cbd85e69800a06326805829aa8.zip | |
btrfs: Better csum error message for data csum mismatch
The original csum error message only outputs inode number, offset, check
sum and expected check sum.
However no root objectid is outputted, which sometimes makes debugging
quite painful under multi-subvolume case (including relocation).
Also the checksum output is decimal, which seldom makes sense for
users/developers and is hard to read in most time.
This patch will add root objectid, which will be %lld for rootid larger
than LAST_FREE_OBJECTID, and hex csum output for better readability.
Signed-off-by: Qu Wenruo <quwenruo@cn.fujitsu.com>
Reviewed-by: David Sterba <dsterba@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
Diffstat (limited to 'fs/btrfs/compression.c')
| -rw-r--r-- | fs/btrfs/compression.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/fs/btrfs/compression.c b/fs/btrfs/compression.c index 454711240649..903c32c9eb22 100644 --- a/fs/btrfs/compression.c +++ b/fs/btrfs/compression.c @@ -124,10 +124,8 @@ static int check_compressed_csum(struct inode *inode, kunmap_atomic(kaddr); if (csum != *cb_sum) { - btrfs_info(BTRFS_I(inode)->root->fs_info, - "csum failed ino %llu extent %llu csum %u wanted %u mirror %d", - btrfs_ino(BTRFS_I(inode)), disk_start, csum, *cb_sum, - cb->mirror_num); + btrfs_print_data_csum_error(inode, disk_start, csum, + *cb_sum, cb->mirror_num); ret = -EIO; goto fail; } |