On 30.01.19 г. 16:57 ч., David Sterba wrote: > On Fri, Jan 25, 2019 at 01:09:16PM +0800, Qu Wenruo wrote: >> Just add one extra line to show when the corruption is detected. >> Currently only read time detection is possible. >> >> Signed-off-by: Qu Wenruo <wqu@xxxxxxxx> >> Reviewed-by: Nikolay Borisov <nborisov@xxxxxxxx> >> --- >> fs/btrfs/disk-io.c | 2 ++ >> 1 file changed, 2 insertions(+) >> >> diff --git a/fs/btrfs/disk-io.c b/fs/btrfs/disk-io.c >> index 794d5bb7fe33..426e9f450f70 100644 >> --- a/fs/btrfs/disk-io.c >> +++ b/fs/btrfs/disk-io.c >> @@ -658,6 +658,8 @@ static int btree_readpage_end_io_hook(struct btrfs_io_bio *io_bio, >> >> if (!ret) >> set_extent_buffer_uptodate(eb); >> + else >> + btrfs_err(fs_info, "read time tree block corrupted detected"); > > I'm not sure the 'read time' is clear enoug, my suggestion is to use > 'post-read' (and pre-write analogicaly). What do you think? How about "error during tree block reading" or "error reading treeblock"? >
