From: "Darrick J. Wong" <darrick.wong@oracle.com>
To: sandeen@sandeen.net, darrick.wong@oracle.com
Cc: linux-xfs@vger.kernel.org
Subject: [PATCH 2/3] xfs_scrub: perform media scans of entire devices
Date: Thu, 05 Sep 2019 20:40:45 -0700 [thread overview]
Message-ID: <156774124577.2646704.15341474642686239741.stgit@magnolia> (raw)
In-Reply-To: <156774123336.2646704.1827381294403838403.stgit@magnolia>
From: Darrick J. Wong <darrick.wong@oracle.com>
Add a new feature to xfs_scrub where specifying multiple -x will cause
it to perform a media scan of the entire disk, not just the file data
areas.
Signed-off-by: Darrick J. Wong <darrick.wong@oracle.com>
---
man/man8/xfs_scrub.8 | 3 +++
scrub/phase6.c | 60 ++++++++++++++++++++++++++++++++++++++++++++++----
scrub/phase7.c | 5 ++++
scrub/xfs_scrub.c | 4 ++-
scrub/xfs_scrub.h | 1 +
5 files changed, 66 insertions(+), 7 deletions(-)
diff --git a/man/man8/xfs_scrub.8 b/man/man8/xfs_scrub.8
index 18948a4e..872a088c 100644
--- a/man/man8/xfs_scrub.8
+++ b/man/man8/xfs_scrub.8
@@ -97,6 +97,9 @@ Prints the version number and exits.
.TP
.B \-x
Read all file data extents to look for disk errors.
+If this option is given more than once, scrub all disk contents.
+If this option is given more than twice, report errors even if they have not
+yet caused data loss.
.B xfs_scrub
will issue O_DIRECT reads to the block device directly.
If the block device is a SCSI disk, it will instead issue READ VERIFY commands
diff --git a/scrub/phase6.c b/scrub/phase6.c
index c50fb8fb..7bfb856a 100644
--- a/scrub/phase6.c
+++ b/scrub/phase6.c
@@ -167,7 +167,9 @@ report_data_loss(
int ret;
/* Only report errors for real extents. */
- if (bmap->bm_flags & (BMV_OF_PREALLOC | BMV_OF_DELALLOC))
+ if (scrub_data < 3 && (bmap->bm_flags & BMV_OF_PREALLOC))
+ return true;
+ if (bmap->bm_flags & BMV_OF_DELALLOC)
return true;
if (fsx->fsx_xflags & FS_XFLAG_REALTIME)
@@ -355,7 +357,7 @@ ioerr_fsmap_report(
uint64_t err_off;
/* Don't care about unwritten extents. */
- if (map->fmr_flags & FMR_OF_PREALLOC)
+ if (scrub_data < 3 && (map->fmr_flags & FMR_OF_PREALLOC))
return true;
if (err_physical > map->fmr_physical)
@@ -602,6 +604,49 @@ clean_pool(
return ret;
}
+/* Schedule an entire disk for read verification. */
+static int
+verify_entire_disk(
+ struct read_verify_pool *rvp,
+ struct disk *disk,
+ struct media_verify_state *vs)
+{
+ return read_verify_schedule_io(rvp, 0, disk->d_size, vs);
+}
+
+/* Scan every part of every disk. */
+static bool
+verify_all_disks(
+ struct scrub_ctx *ctx,
+ struct media_verify_state *vs)
+{
+ int ret;
+
+ ret = verify_entire_disk(vs->rvp_data, ctx->datadev, vs);
+ if (ret) {
+ str_liberror(ctx, ret, _("scheduling datadev verify"));
+ return false;
+ }
+
+ if (ctx->logdev) {
+ ret = verify_entire_disk(vs->rvp_log, ctx->logdev, vs);
+ if (ret) {
+ str_liberror(ctx, ret, _("scheduling logdev verify"));
+ return false;
+ }
+ }
+
+ if (ctx->rtdev) {
+ ret = verify_entire_disk(vs->rvp_realtime, ctx->rtdev, vs);
+ if (ret) {
+ str_liberror(ctx, ret, _("scheduling rtdev verify"));
+ return false;
+ }
+ }
+
+ return true;
+}
+
/*
* Read verify all the file data blocks in a filesystem. Since XFS doesn't
* do data checksums, we trust that the underlying storage will pass back
@@ -657,7 +702,11 @@ xfs_scan_blocks(
goto out_logpool;
}
}
- moveon = xfs_scan_all_spacemaps(ctx, xfs_check_rmap, &vs);
+
+ if (scrub_data > 1)
+ moveon = verify_all_disks(ctx, &vs);
+ else
+ moveon = xfs_scan_all_spacemaps(ctx, xfs_check_rmap, &vs);
if (!moveon)
goto out_rtpool;
@@ -729,8 +778,9 @@ xfs_estimate_verify_work(
if (!moveon)
return moveon;
- *items = cvt_off_fsb_to_b(&ctx->mnt,
- (d_blocks - d_bfree) + (r_blocks - r_bfree));
+ *items = cvt_off_fsb_to_b(&ctx->mnt, d_blocks + r_blocks);
+ if (scrub_data == 1)
+ *items -= cvt_off_fsb_to_b(&ctx->mnt, d_bfree + r_bfree);
*nr_threads = disk_heads(ctx->datadev);
*rshift = 20;
return moveon;
diff --git a/scrub/phase7.c b/scrub/phase7.c
index bc959f5b..570ceb3f 100644
--- a/scrub/phase7.c
+++ b/scrub/phase7.c
@@ -255,6 +255,11 @@ _("%.*f%s inodes counted; %.*f%s inodes checked.\n"),
double b1, b2;
char *b1u, *b2u;
+ if (scrub_data > 1) {
+ used_data = cvt_off_fsb_to_b(&ctx->mnt, d_blocks);
+ used_rt = cvt_off_fsb_to_b(&ctx->mnt, r_blocks);
+ }
+
b1 = auto_space_units(used_data + used_rt, &b1u);
b2 = auto_space_units(ctx->bytes_checked, &b2u);
fprintf(stdout,
diff --git a/scrub/xfs_scrub.c b/scrub/xfs_scrub.c
index 2d554340..46876522 100644
--- a/scrub/xfs_scrub.c
+++ b/scrub/xfs_scrub.c
@@ -139,7 +139,7 @@ unsigned int force_nr_threads;
bool verbose;
/* Should we scrub the data blocks? */
-static bool scrub_data;
+int scrub_data;
/* Size of a memory page. */
long page_size;
@@ -666,7 +666,7 @@ main(
fflush(stdout);
return SCRUB_RET_SUCCESS;
case 'x':
- scrub_data = true;
+ scrub_data++;
break;
case '?':
/* fall through */
diff --git a/scrub/xfs_scrub.h b/scrub/xfs_scrub.h
index 54876acb..6558bad7 100644
--- a/scrub/xfs_scrub.h
+++ b/scrub/xfs_scrub.h
@@ -21,6 +21,7 @@ extern bool want_fstrim;
extern bool stderr_isatty;
extern bool stdout_isatty;
extern bool is_service;
+extern int scrub_data;
enum scrub_mode {
SCRUB_MODE_DRY_RUN,
next prev parent reply other threads:[~2019-09-06 3:43 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-09-06 3:40 [PATCH 0/3] xfs_scrub: media scan entire disks Darrick J. Wong
2019-09-06 3:40 ` [PATCH 1/3] xfs_scrub: implement background mode for phase 6 Darrick J. Wong
2019-09-12 23:42 ` Allison Collins
2019-09-06 3:40 ` Darrick J. Wong [this message]
2019-09-12 23:42 ` [PATCH 2/3] xfs_scrub: perform media scans of entire devices Allison Collins
2019-09-06 3:40 ` [PATCH 3/3] xfs_scrub: relabel verified data block counts in output Darrick J. Wong
2019-09-12 23:42 ` Allison Collins
-- strict thread matches above, loose matches on Subject: below --
2019-10-22 18:49 [PATCH 0/3] xfs_scrub: media scan entire disks Darrick J. Wong
2019-10-22 18:50 ` [PATCH 2/3] xfs_scrub: perform media scans of entire devices Darrick J. Wong
2019-11-06 20:15 ` Eric Sandeen
2019-11-06 20:22 ` Darrick J. Wong
2019-09-25 21:37 [PATCH 0/3] xfs_scrub: media scan entire disks Darrick J. Wong
2019-09-25 21:37 ` [PATCH 2/3] xfs_scrub: perform media scans of entire devices Darrick J. Wong
2019-08-26 21:32 [PATCH 0/3] xfs_scrub: media scan entire disks Darrick J. Wong
2019-08-26 21:33 ` [PATCH 2/3] xfs_scrub: perform media scans of entire devices Darrick J. Wong
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=156774124577.2646704.15341474642686239741.stgit@magnolia \
--to=darrick.wong@oracle.com \
--cc=linux-xfs@vger.kernel.org \
--cc=sandeen@sandeen.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).