On Tue, Feb 07, 2017 at 02:57:17PM +0800, Qu Wenruo wrote:
> 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.
Ok for the change.
> + "csum failed root %lld ino %lld off %llu csum 0x%08x expected csum
> 0x%08x",
> + "csum failed root %llu ino %llu off %llu csum 0x%08x expected csum
> 0x%08x",
> - "csum failed ino %llu extent %llu csum %u wanted %u
> mirror %d",
so the new code does not print mirror number, I think this still makes
sense in cases where we know it. Please extend the helper and callchain
that leads to the new print functions so we see the mirror as well.
btrfs_readpage_end_io_hook
__readpage_endio_check
(print the csum failed message)
--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
