[PATCH] xfs_repair: fix sibling pointer tests in verify_dir2_path()
Christoph Hellwig
hch at infradead.org
Thu Feb 20 09:41:22 CST 2014
On Wed, Feb 19, 2014 at 03:10:13PM -0600, Eric Sandeen wrote:
> RH QE reported that if we create a 1G filesystem with default
> options, mount it, and create inodes until full, then run
> repair, repair reports corruption in verify_dir2_path() with:
>
> > bad back pointer in block 8390324 for directory inode 131
Can you please wire this up for xfstests?
>
> The commit 88b32f0 xfs: add CRCs to dir2/da node blocks
> had a small error which regressed this; although we switch
> to the "newnode," to check sibling pointers, we re-populate
> the node hdr with the old "node" data. This causes the
> backpointer test to be testing the wrong node's values.
>
> Fixing this bug fixes the testcase.
>
> Signed-off-by: Eric Sandeen <sandeen at redhat.com>
Looks good,
Reviewed-by: Christoph Hellwig <hch at lst.de>
More information about the xfs
mailing list