kevin/linux
1
Fork 0
Code

xfs: exchange-range for repairs is no longer dynamic

Browse Source 
The atomic file exchange-range functionality is now a permanent
filesystem feature instead of a dynamic log-incompat feature.  It cannot
be turned on at runtime, so we no longer need the XCHK_FSGATES flags and
whatnot that supported it.  Remove the flag and the enable function, and
move the xfs_has_exchange_range checks to the start of the repair
functions.

Signed-off-by: Darrick J. Wong <djwong@kernel.org>
Reviewed-by: Christoph Hellwig <hch@lst.de>
This commit is contained in:
Darrick J. Wong 2024-04-22 09:48:29 -07:00
parent b44bfc0695
commit 6d335233fe
10 changed files with 25 additions and 45 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
fs/xfs/scrub/attr_repair.c
View File

@ -1630,6 +1630,9 @@ xrep_xattr(
/* The rmapbt is required to reap the old attr fork. */
if (!xfs_has_rmapbt(sc->mp))
return -EOPNOTSUPP;
/* We require atomic file exchange range to rebuild anything. */
if (!xfs_has_exchange_range(sc->mp))
return -EOPNOTSUPP;
error = xrep_xattr_setup_scan(sc, &rx);
if (error)

3
fs/xfs/scrub/dir_repair.c
View File

@ -1993,6 +1993,9 @@ xrep_directory(
/* The rmapbt is required to reap the old data fork. */
if (!xfs_has_rmapbt(sc->mp))
return -EOPNOTSUPP;
/* We require atomic file exchange range to rebuild anything. */
if (!xfs_has_exchange_range(sc->mp))
return -EOPNOTSUPP;
error = xrep_dir_setup_scan(rd);
if (error)

10
fs/xfs/scrub/parent_repair.c
View File

@ -1571,10 +1571,14 @@ xrep_parent(
/*
* When the parent pointers feature is enabled, repairs are committed
* by atomically committing a new xattr structure and reaping the old
* attr fork. Reaping requires rmap to be enabled.
* attr fork. Reaping requires rmap and exchange-range to be enabled.
*/
if (xfs_has_parent(sc->mp) && !xfs_has_rmapbt(sc->mp))
return -EOPNOTSUPP;
if (xfs_has_parent(sc->mp)) {
if (!xfs_has_rmapbt(sc->mp))
return -EOPNOTSUPP;
if (!xfs_has_exchange_range(sc->mp))
return -EOPNOTSUPP;
}
error = xrep_parent_setup_scan(rp);
if (error)

10
fs/xfs/scrub/rtsummary_repair.c
View File

@ -62,12 +62,7 @@ xrep_setup_rtsummary(
return -EOPNOTSUPP;
rts->resblks += blocks;
/*
* Grab support for atomic file content exchanges before we allocate
* any transactions or grab ILOCKs.
*/
return xrep_tempexch_enable(sc);
return 0;
}
static int
@ -111,6 +106,9 @@ xrep_rtsummary(
/* We require the rmapbt to rebuild anything. */
if (!xfs_has_rmapbt(mp))
return -EOPNOTSUPP;
/* We require atomic file exchange range to rebuild anything. */
if (!xfs_has_exchange_range(mp))
return -EOPNOTSUPP;
/* Walk away if we disagree on the size of the rt bitmap. */
if (rts->rbmblocks != mp->m_sb.sb_rbmblocks)

8
fs/xfs/scrub/scrub.c
View File

@ -154,15 +154,14 @@ xchk_probe(
/* Scrub setup and teardown */
#define FSGATES_MASK (XCHK_FSGATES_ALL | XREP_FSGATES_ALL)
static inline void
xchk_fsgates_disable(
struct xfs_scrub *sc)
{
if (!(sc->flags & FSGATES_MASK))
if (!(sc->flags & XCHK_FSGATES_ALL))
return;
trace_xchk_fsgates_disable(sc, sc->flags & FSGATES_MASK);
trace_xchk_fsgates_disable(sc, sc->flags & XCHK_FSGATES_ALL);
if (sc->flags & XCHK_FSGATES_DRAIN)
xfs_drain_wait_disable();
@ -176,9 +175,8 @@ xchk_fsgates_disable(
if (sc->flags & XCHK_FSGATES_RMAP)
xfs_rmap_hook_disable();
sc->flags &= ~FSGATES_MASK;
sc->flags &= ~XCHK_FSGATES_ALL;
}
#undef FSGATES_MASK
/* Free the resources associated with a scrub subtype. */
void

7
fs/xfs/scrub/scrub.h
View File

@ -188,7 +188,6 @@ struct xfs_scrub {
#define XCHK_FSGATES_QUOTA (1U << 4) /* quota live update enabled */
#define XCHK_FSGATES_DIRENTS (1U << 5) /* directory live update enabled */
#define XCHK_FSGATES_RMAP (1U << 6) /* rmapbt live update enabled */
#define XREP_FSGATES_EXCHANGE_RANGE (1U << 29) /* uses file content exchange */
#define XREP_RESET_PERAG_RESV (1U << 30) /* must reset AG space reservation */
#define XREP_ALREADY_FIXED (1U << 31) /* checking our repair work */
@ -203,12 +202,6 @@ struct xfs_scrub {
XCHK_FSGATES_DIRENTS | \
XCHK_FSGATES_RMAP)
/*
* The sole XREP_FSGATES* flag reflects a log intent item that is protected
* by a log-incompat feature flag. No code patching in use here.
*/
#define XREP_FSGATES_ALL (XREP_FSGATES_EXCHANGE_RANGE)
struct xfs_scrub_subord {
struct xfs_scrub sc;
struct xfs_scrub *parent_sc;

3
fs/xfs/scrub/symlink_repair.c
View File

@ -490,6 +490,9 @@ xrep_symlink(
/* The rmapbt is required to reap the old data fork. */
if (!xfs_has_rmapbt(sc->mp))
return -EOPNOTSUPP;
/* We require atomic file exchange range to rebuild anything. */
if (!xfs_has_exchange_range(sc->mp))
return -EOPNOTSUPP;
ASSERT(sc->ilock_flags & XFS_ILOCK_EXCL);

1
fs/xfs/scrub/tempexch.h
View File

@ -11,7 +11,6 @@ struct xrep_tempexch {
struct xfs_exchmaps_req req;
};
int xrep_tempexch_enable(struct xfs_scrub *sc);
int xrep_tempexch_trans_reserve(struct xfs_scrub *sc, int whichfork,
struct xrep_tempexch *ti);
int xrep_tempexch_trans_alloc(struct xfs_scrub *sc, int whichfork,

24
fs/xfs/scrub/tempfile.c
View File

@ -486,23 +486,6 @@ xrep_tempfile_roll_trans(
return 0;
}
/* Enable file content exchanges. */
int
xrep_tempexch_enable(
struct xfs_scrub *sc)
{
if (sc->flags & XREP_FSGATES_EXCHANGE_RANGE)
return 0;
if (!xfs_has_exchange_range(sc->mp))
return -EOPNOTSUPP;
trace_xchk_fsgates_enable(sc, XREP_FSGATES_EXCHANGE_RANGE);
sc->flags |= XREP_FSGATES_EXCHANGE_RANGE;
return 0;
}
/*
* Fill out the mapping exchange request in preparation for atomically
* committing the contents of a metadata file that we've rebuilt in the temp
@ -745,6 +728,7 @@ xrep_tempexch_trans_alloc(
int error;
ASSERT(sc->tp == NULL);
ASSERT(xfs_has_exchange_range(sc->mp));
error = xrep_tempexch_prep_request(sc, whichfork, tx);
if (error)
@ -757,10 +741,6 @@ xrep_tempexch_trans_alloc(
if (xfs_has_lazysbcount(sc->mp))
flags |= XFS_TRANS_RES_FDBLKS;
error = xrep_tempexch_enable(sc);
if (error)
return error;
error = xfs_trans_alloc(sc->mp, &M_RES(sc->mp)->tr_itruncate,
tx->req.resblks, 0, flags, &sc->tp);
if (error)
@ -785,7 +765,7 @@ xrep_tempexch_contents(
{
int error;
ASSERT(sc->flags & XREP_FSGATES_EXCHANGE_RANGE);
ASSERT(xfs_has_exchange_range(sc->mp));
xfs_exchange_mappings(sc->tp, &tx->req);
error = xfs_defer_finish(&sc->tp);

1
fs/xfs/scrub/trace.h
View File

@ -122,7 +122,6 @@ TRACE_DEFINE_ENUM(XFS_SCRUB_TYPE_BARRIER);
{ XCHK_FSGATES_QUOTA, "fsgates_quota" }, \
{ XCHK_FSGATES_DIRENTS, "fsgates_dirents" }, \
{ XCHK_FSGATES_RMAP, "fsgates_rmap" }, \
{ XREP_FSGATES_EXCHANGE_RANGE, "fsgates_exchrange" }, \
{ XREP_RESET_PERAG_RESV, "reset_perag_resv" }, \
{ XREP_ALREADY_FIXED, "already_fixed" }