From: Darrick J. Wong <djwong@xxxxxxxxxx>
"Summary" metadata, at least in the scrub context, are metadata whose
values depend on some kind of computation and therefore can only be
checked after we've looked at all the other metadata. Currently, the
superblock summary counters are the only thing that are like this, but
since they run in a totally separate xfs_scrub phase (7 vs. 2), make
them their own group and remove the group+flag mix.
Signed-off-by: Darrick J. Wong <djwong@xxxxxxxxxx>
---
io/scrub.c | 3 +++
libfrog/scrub.c | 3 +--
libfrog/scrub.h | 8 +-------
scrub/phase4.c | 2 +-
scrub/phase7.c | 4 ++--
scrub/scrub.c | 16 ++++++++++++----
scrub/scrub.h | 3 ++-
7 files changed, 22 insertions(+), 17 deletions(-)
diff --git a/io/scrub.c b/io/scrub.c
index 0cad69253dc..4cfa4bc3c4c 100644
--- a/io/scrub.c
+++ b/io/scrub.c
@@ -67,6 +67,7 @@ scrub_ioctl(
break;
case XFROG_SCRUB_GROUP_NONE:
case XFROG_SCRUB_GROUP_FS:
+ case XFROG_SCRUB_GROUP_SUMMARY:
/* no control parameters */
break;
}
@@ -168,6 +169,7 @@ parse_args(
break;
case XFROG_SCRUB_GROUP_FS:
case XFROG_SCRUB_GROUP_NONE:
+ case XFROG_SCRUB_GROUP_SUMMARY:
if (optind != argc) {
fprintf(stderr,
_("No parameters allowed.\n"));
@@ -259,6 +261,7 @@ repair_ioctl(
break;
case XFROG_SCRUB_GROUP_NONE:
case XFROG_SCRUB_GROUP_FS:
+ case XFROG_SCRUB_GROUP_SUMMARY:
/* no control parameters */
break;
}
diff --git a/libfrog/scrub.c b/libfrog/scrub.c
index 90fc2b1a40c..5a5f522a425 100644
--- a/libfrog/scrub.c
+++ b/libfrog/scrub.c
@@ -132,8 +132,7 @@ const struct xfrog_scrub_descr xfrog_scrubbers[XFS_SCRUB_TYPE_NR] = {
[XFS_SCRUB_TYPE_FSCOUNTERS] = {
.name = "fscounters",
.descr = "filesystem summary counters",
- .group = XFROG_SCRUB_GROUP_FS,
- .flags = XFROG_SCRUB_DESCR_SUMMARY,
+ .group = XFROG_SCRUB_GROUP_SUMMARY,
},
};
diff --git a/libfrog/scrub.h b/libfrog/scrub.h
index 43a882321f9..68f1a968103 100644
--- a/libfrog/scrub.h
+++ b/libfrog/scrub.h
@@ -13,6 +13,7 @@ enum xfrog_scrub_group {
XFROG_SCRUB_GROUP_PERAG, /* per-AG metadata */
XFROG_SCRUB_GROUP_FS, /* per-FS metadata */
XFROG_SCRUB_GROUP_INODE, /* per-inode metadata */
+ XFROG_SCRUB_GROUP_SUMMARY, /* summary metadata */
};
/* Catalog of scrub types and names, indexed by XFS_SCRUB_TYPE_* */
@@ -20,15 +21,8 @@ struct xfrog_scrub_descr {
const char *name;
const char *descr;
enum xfrog_scrub_group group;
- unsigned int flags;
};
-/*
- * The type of metadata checked by this scrubber is a summary of other types
- * of metadata. This scrubber should be run after all the others.
- */
-#define XFROG_SCRUB_DESCR_SUMMARY (1 << 0)
-
extern const struct xfrog_scrub_descr xfrog_scrubbers[XFS_SCRUB_TYPE_NR];
int xfrog_scrub_metadata(struct xfs_fd *xfd, struct xfs_scrub_metadata *meta);
diff --git a/scrub/phase4.c b/scrub/phase4.c
index ecd56056ca2..3c985cdc13a 100644
--- a/scrub/phase4.c
+++ b/scrub/phase4.c
@@ -139,7 +139,7 @@ phase4_func(
* counters, so counter repairs have to be put on the list now so that
* they get fixed before we stop retrying unfixed metadata repairs.
*/
- ret = scrub_fs_summary(ctx, &ctx->action_lists[0]);
+ ret = scrub_fs_counters(ctx, &ctx->action_lists[0]);
if (ret)
return ret;
diff --git a/scrub/phase7.c b/scrub/phase7.c
index 8d8034c36af..2b83e0a471d 100644
--- a/scrub/phase7.c
+++ b/scrub/phase7.c
@@ -116,9 +116,9 @@ phase7_func(
int ip;
int error;
- /* Check and fix the fs summary counters. */
+ /* Check and fix the summary metadata. */
action_list_init(&alist);
- error = scrub_fs_summary(ctx, &alist);
+ error = scrub_summary_metadata(ctx, &alist);
if (error)
return error;
error = action_list_process(ctx, -1, &alist,
diff --git a/scrub/scrub.c b/scrub/scrub.c
index 61a111db080..b7e120e91d6 100644
--- a/scrub/scrub.c
+++ b/scrub/scrub.c
@@ -46,6 +46,7 @@ format_scrub_descr(
_(sc->descr));
break;
case XFROG_SCRUB_GROUP_FS:
+ case XFROG_SCRUB_GROUP_SUMMARY:
return snprintf(buf, buflen, _("%s"), _(sc->descr));
break;
case XFROG_SCRUB_GROUP_NONE:
@@ -356,8 +357,6 @@ scrub_group(
if (sc->group != group)
continue;
- if (sc->flags & XFROG_SCRUB_DESCR_SUMMARY)
- continue;
ret = scrub_meta_type(ctx, type, agno, alist);
if (ret)
@@ -410,9 +409,18 @@ scrub_fs_metadata(
return scrub_group(ctx, XFROG_SCRUB_GROUP_FS, 0, alist);
}
-/* Scrub FS summary metadata. */
+/* Scrub all FS summary metadata. */
int
-scrub_fs_summary(
+scrub_summary_metadata(
+ struct scrub_ctx *ctx,
+ struct action_list *alist)
+{
+ return scrub_group(ctx, XFROG_SCRUB_GROUP_SUMMARY, 0, alist);
+}
+
+/* Scrub /only/ the superblock summary counters. */
+int
+scrub_fs_counters(
struct scrub_ctx *ctx,
struct action_list *alist)
{
diff --git a/scrub/scrub.h b/scrub/scrub.h
index 023069ee066..56836cf2ba3 100644
--- a/scrub/scrub.h
+++ b/scrub/scrub.h
@@ -23,7 +23,8 @@ int scrub_ag_headers(struct scrub_ctx *ctx, xfs_agnumber_t agno,
int scrub_ag_metadata(struct scrub_ctx *ctx, xfs_agnumber_t agno,
struct action_list *alist);
int scrub_fs_metadata(struct scrub_ctx *ctx, struct action_list *alist);
-int scrub_fs_summary(struct scrub_ctx *ctx, struct action_list *alist);
+int scrub_summary_metadata(struct scrub_ctx *ctx, struct action_list *alist);
+int scrub_fs_counters(struct scrub_ctx *ctx, struct action_list *alist);
bool can_scrub_fs_metadata(struct scrub_ctx *ctx);
bool can_scrub_inode(struct scrub_ctx *ctx);
