Sunil Mushran
2010-Aug-12 23:24 UTC
[Ocfs2-devel] [PATCH 1/2] ocfs2: Fix metaecc error messages
Like tools, the checksum validate function now prints the values in hex. Signed-off-by: Sunil Mushran <sunil.mushran at oracle.com> --- fs/ocfs2/blockcheck.c | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/fs/ocfs2/blockcheck.c b/fs/ocfs2/blockcheck.c index ec6d123..c7ee03c 100644 --- a/fs/ocfs2/blockcheck.c +++ b/fs/ocfs2/blockcheck.c @@ -439,7 +439,7 @@ int ocfs2_block_check_validate(void *data, size_t blocksize, ocfs2_blockcheck_inc_failure(stats); mlog(ML_ERROR, - "CRC32 failed: stored: %u, computed %u. Applying ECC.\n", + "CRC32 failed: stored: 0x%x, computed 0x%x. Applying ECC.\n", (unsigned int)check.bc_crc32e, (unsigned int)crc); /* Ok, try ECC fixups */ @@ -453,7 +453,7 @@ int ocfs2_block_check_validate(void *data, size_t blocksize, goto out; } - mlog(ML_ERROR, "Fixed CRC32 failed: stored: %u, computed %u\n", + mlog(ML_ERROR, "Fixed CRC32 failed: stored: 0x%x, computed 0x%x\n", (unsigned int)check.bc_crc32e, (unsigned int)crc); rc = -EIO; -- 1.7.0.4
Sunil Mushran
2010-Aug-12 23:24 UTC
[Ocfs2-devel] [PATCH 2/2] ocfs2: Fix incorrect checksum validation error
For local mounts, ocfs2_read_locked_inode() calls ocfs2_read_blocks_sync() to read the inode off the disk. The latter first checks to see if that block is cached in the journal, and, if so, returns that block. That is ok. But ocfs2_read_locked_inode() goes wrong when it tries to validate the checksum of such blocks. Blocks that are cached in the journal may not have had their checksum computed as yet. We should not validate the checksums of such blocks. Fixes ossbz#1282 oss.oracle.com/bugzilla/show_bug.cgi?id=1282 Signed-off-by: Sunil Mushran <sunil.mushran at oracle.com> --- fs/ocfs2/inode.c | 6 +++++- 1 files changed, 5 insertions(+), 1 deletions(-) diff --git a/fs/ocfs2/inode.c b/fs/ocfs2/inode.c index abb0a95..c9fcce3 100644 --- a/fs/ocfs2/inode.c +++ b/fs/ocfs2/inode.c @@ -488,7 +488,11 @@ static int ocfs2_read_locked_inode(struct inode *inode, OCFS2_BH_IGNORE_CACHE); } else { status = ocfs2_read_blocks_sync(osb, args->fi_blkno, 1, &bh); - if (!status) + /* + * If buffer is in jbd, then its checksum may not have been + * computed as yet. + */ + if (!status && (!buffer_jbd(bh))) status = ocfs2_validate_inode_block(osb->sb, bh); } if (status < 0) { -- 1.7.0.4
Mark Fasheh
2010-Aug-13 23:28 UTC
[Ocfs2-devel] [PATCH 1/2] ocfs2: Fix metaecc error messages
On Thu, Aug 12, 2010 at 04:24:25PM -0700, Sunil Mushran wrote:> Like tools, the checksum validate function now prints the values in hex. > > Signed-off-by: Sunil Mushran <sunil.mushran at oracle.com>Acked-by: Mark Fasheh <mfasheh at suse.com> And thanks, this was rather annoying when I was last comparing checksums between different parts of the code / project. --Mark -- Mark Fasheh
Mark Fasheh
2010-Aug-13 23:31 UTC
[Ocfs2-devel] [PATCH 2/2] ocfs2: Fix incorrect checksum validation error
On Thu, Aug 12, 2010 at 04:24:26PM -0700, Sunil Mushran wrote:> For local mounts, ocfs2_read_locked_inode() calls ocfs2_read_blocks_sync() to > read the inode off the disk. The latter first checks to see if that block is > cached in the journal, and, if so, returns that block. That is ok. > > But ocfs2_read_locked_inode() goes wrong when it tries to validate the checksum > of such blocks. Blocks that are cached in the journal may not have had their > checksum computed as yet. We should not validate the checksums of such blocks. > > Fixes ossbz#1282 > oss.oracle.com/bugzilla/show_bug.cgi?id=1282 > > Signed-off-by: Sunil Mushran <sunil.mushran at oracle.com>Good catch. Acked-by: Mark Fasheh <mfasheh at suse.com> --Mark -- Mark Fasheh
Seemingly Similar Threads
- [PATCH] ocfs2: Add statistics for the checksum and ecc operations.
- [PATCH] ocfs2: Add statistics for the checksum and ecc operations.
- Diagnosing some OCFS2 error messages
- add error check for ocfs2_read_locked_inode() call
- [PATCH 1/1] OCFS2: add error check for ocfs2_read_locked_inode() call