diff options
| author | Darrick J. Wong <darrick.wong@oracle.com> | 2018-01-17 03:52:44 +0100 |
|---|---|---|
| committer | Darrick J. Wong <darrick.wong@oracle.com> | 2018-01-18 06:00:45 +0100 |
| commit | 9a7e26956661049103493c229eda20d344d492ae (patch) | |
| tree | 07b210c6c6d158ccd1edfe5090d69fe5544a8c31 /fs/xfs/scrub/inode.c | |
| parent | xfs: set up scrub cross-referencing helpers (diff) | |
| download | linux-9a7e26956661049103493c229eda20d344d492ae.tar.xz linux-9a7e26956661049103493c229eda20d344d492ae.zip | |
xfs: fix a few erroneous process_error calls in the scrubbers
There are a few places where we make a libxfs api call on behalf of some
object other than the one we're scrubbing but inadvertently call the
regular process_error function. When this happens we mark the object
corrupt even though it was corruption in /some other/ object that
actually produced the -EFSCORRUPTED code. The correct output flag for
these situations is SCRUB_OFLAG_XFAIL, not SCRUB_OFLAG_CORRUPT, so fix
this now that we also have a helper to set these.
Signed-off-by: Darrick J. Wong <darrick.wong@oracle.com>
Reviewed-by: Dave Chinner <dchinner@redhat.com>
Diffstat (limited to 'fs/xfs/scrub/inode.c')
| -rw-r--r-- | fs/xfs/scrub/inode.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/xfs/scrub/inode.c b/fs/xfs/scrub/inode.c index 59a4fce91a2b..1c5b64667705 100644 --- a/fs/xfs/scrub/inode.c +++ b/fs/xfs/scrub/inode.c @@ -619,7 +619,7 @@ xfs_scrub_inode( if (xfs_is_reflink_inode(sc->ip)) { error = xfs_reflink_inode_has_shared_extents(sc->tp, sc->ip, &has_shared); - if (!xfs_scrub_process_error(sc, XFS_INO_TO_AGNO(mp, ino), + if (!xfs_scrub_xref_process_error(sc, XFS_INO_TO_AGNO(mp, ino), XFS_INO_TO_AGBNO(mp, ino), &error)) goto out; if (!has_shared) |