git: sys/vfs/hammer: Add error messages for crc test failure
Tomohiro Kusumi
tkusumi at crater.dragonflybsd.org
Sat Oct 10 05:46:00 PDT 2015
commit babd2e48fa1809dd684239b1113915cd37c044f8
Author: Tomohiro Kusumi <kusumi.tomohiro at gmail.com>
Date: Sat Sep 26 02:04:37 2015 +0900
sys/vfs/hammer: Add error messages for crc test failure
Add crc error messages based on the existing error message
in hammer_btree_extract(). The one in hammer_mirror.c is
a part of #if0'd function.
Reported-by: profmakx
Suggested-by: dillon
23:27 (tkusumi) too many error = EIO; but possibly bad crc ?
23:28 (tkusumi) if bad crc is the case turning on hammer_debug_critical leads to Debugger("CRC FAILED: B-TREE NODE");
23:40 (dillon) it's likely a CRC failure, though I think it should print those out on the console too
23:54 (tkusumi) some bad crc are missing "CRC DATA @ %016llx/%d FAILED\n"
23:59 (dillon) those could be added for clarity. Almost universally EIO from hammer means a bad CRC and not an actual disk read error.
23:59 (dillon) USB-connected hard drives seem to get these. I've never gotten a bad CRC from a SATA-connected HDD via the AHCI driver.
Summary of changes:
sys/vfs/hammer/hammer_mirror.c | 4 +++-
sys/vfs/hammer/hammer_object.c | 4 +++-
sys/vfs/hammer/hammer_ondisk.c | 3 +++
3 files changed, 9 insertions(+), 2 deletions(-)
http://gitweb.dragonflybsd.org/dragonfly.git/commitdiff/babd2e48fa1809dd684239b1113915cd37c044f8
--
DragonFly BSD source repository
More information about the Commits
mailing list