diff options
author | Darrick J. Wong <darrick.wong@oracle.com> | 2017-03-28 14:51:10 -0700 |
---|---|---|
committer | Darrick J. Wong <darrick.wong@oracle.com> | 2017-03-28 14:51:10 -0700 |
commit | 005c5db8fd7b2c9fc0b8055fa824ecaa7b15d1bf (patch) | |
tree | 79ba07b0a639b449e5ea8579be794c4569d3e162 /fs/xfs/libxfs/xfs_alloc_btree.c | |
parent | c02ed2e75ef4c74e41e421acb4ef1494671585e8 (diff) |
xfs: rework the inline directory verifiers
The inline directory verifiers should be called on the inode fork data,
which means after iformat_local on the read side, and prior to
ifork_flush on the write side. This makes the fork verifier more
consistent with the way buffer verifiers work -- i.e. they will operate
on the memory buffer that the code will be reading and writing directly.
Furthermore, revise the verifier function to return -EFSCORRUPTED so
that we don't flood the logs with corruption messages and assert
notices. This has been a particular problem with xfs/348, which
triggers the XFS_WANT_CORRUPTED_RETURN assertions, which halts the
kernel when CONFIG_XFS_DEBUG=y. Disk corruption isn't supposed to do
that, at least not in a verifier.
Reviewed-by: Brian Foster <bfoster@redhat.com>
Reviewed-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Darrick J. Wong <darrick.wong@oracle.com>
---
v2: get the inode d_ops the proper way
v3: describe the bug that this patch fixes; no code changes
Diffstat (limited to 'fs/xfs/libxfs/xfs_alloc_btree.c')
0 files changed, 0 insertions, 0 deletions