From: Darrick J. Wong <darrick.wong@xxxxxxxxxx>
The xfs_verify_error_info structure is a strict subset of
media_verify_info so just pass that around.
Signed-off-by: Darrick J. Wong <darrick.wong@xxxxxxxxxx>
---
scrub/phase6.c | 20 +++++---------------
1 file changed, 5 insertions(+), 15 deletions(-)
diff --git a/scrub/phase6.c b/scrub/phase6.c
index 2b4f6ff1..f173f248 100644
--- a/scrub/phase6.c
+++ b/scrub/phase6.c
@@ -111,11 +111,6 @@ xfs_decode_special_owner(
/* Routines to translate bad physical extents into file paths and offsets. */
-struct xfs_verify_error_info {
- struct bitmap *d_bad; /* bytes */
- struct bitmap *r_bad; /* bytes */
-};
-
/* Report if this extent overlaps a bad region. */
static bool
xfs_report_verify_inode_bmap(
@@ -127,7 +122,7 @@ xfs_report_verify_inode_bmap(
struct xfs_bmap *bmap,
void *arg)
{
- struct xfs_verify_error_info *vei = arg;
+ struct media_verify_state *vei = arg;
struct bitmap *bmp;
/* Only report errors for real extents. */
@@ -277,23 +272,18 @@ xfs_report_verify_dirent(
static bool
xfs_report_verify_errors(
struct scrub_ctx *ctx,
- struct bitmap *d_bad,
- struct bitmap *r_bad)
+ struct media_verify_state *ve)
{
- struct xfs_verify_error_info vei;
bool moveon;
- vei.d_bad = d_bad;
- vei.r_bad = r_bad;
-
/* Scan the directory tree to get file paths. */
moveon = scan_fs_tree(ctx, xfs_report_verify_dir,
- xfs_report_verify_dirent, &vei);
+ xfs_report_verify_dirent, ve);
if (!moveon)
return false;
/* Scan for unlinked files. */
- return xfs_scan_all_inodes(ctx, xfs_report_verify_inode, &vei);
+ return xfs_scan_all_inodes(ctx, xfs_report_verify_inode, ve);
}
/* Report an IO error resulting from read-verify based off getfsmap. */
@@ -518,7 +508,7 @@ _("Could not create data device media verifier."));
/* Scan the whole dir tree to see what matches the bad extents. */
if (!bitmap_empty(ve.d_bad) || !bitmap_empty(ve.r_bad))
- moveon = xfs_report_verify_errors(ctx, ve.d_bad, ve.r_bad);
+ moveon = xfs_report_verify_errors(ctx, &ve);
bitmap_free(&ve.r_bad);
bitmap_free(&ve.d_bad);
