diff options
| author | Darrick J. Wong <darrick.wong@oracle.com> | 2019-04-16 08:22:00 -0700 |
|---|---|---|
| committer | Darrick J. Wong <darrick.wong@oracle.com> | 2019-04-16 10:01:57 -0700 |
| commit | 160b5a784525e8a44902873f1938e1f4d77fe112 (patch) | |
| tree | 5f7df442f9d137e108140020c8910deb755c19fc /fs/xfs/scrub/scrub.h | |
| parent | f8c2a2257ca12fb3a04830bbacc85ce860f04ba9 (diff) | |
| download | lwn-160b5a784525e8a44902873f1938e1f4d77fe112.tar.gz lwn-160b5a784525e8a44902873f1938e1f4d77fe112.zip | |
xfs: hoist the already_fixed variable to the scrub context
Now that we no longer memset the scrub context, we can move the
already_fixed variable into the scrub context's state flags instead of
passing around pointers to separate stack variables.
Signed-off-by: Darrick J. Wong <darrick.wong@oracle.com>
Reviewed-by: Dave Chinner <dchinner@redhat.com>
Diffstat (limited to 'fs/xfs/scrub/scrub.h')
| -rw-r--r-- | fs/xfs/scrub/scrub.h | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/fs/xfs/scrub/scrub.h b/fs/xfs/scrub/scrub.h index 60359e7de930..9e8d3d7377f2 100644 --- a/fs/xfs/scrub/scrub.h +++ b/fs/xfs/scrub/scrub.h @@ -63,16 +63,17 @@ struct xfs_scrub { void *buf; uint ilock_flags; - /* See the XCHK state flags below. */ + /* See the XCHK/XREP state flags below. */ unsigned int flags; /* State tracking for single-AG operations. */ struct xchk_ag sa; }; -/* XCHK state flags */ +/* XCHK state flags grow up from zero, XREP state flags grown down from 2^31 */ #define XCHK_TRY_HARDER (1 << 0) /* can't get resources, try again */ #define XCHK_HAS_QUOTAOFFLOCK (1 << 1) /* we hold the quotaoff lock */ +#define XREP_ALREADY_FIXED (1 << 31) /* checking our repair work */ /* Metadata scrubbers */ int xchk_tester(struct xfs_scrub *sc); |