diff options
| author | Darrick J. Wong <darrick.wong@oracle.com> | 2019-11-01 15:46:12 -0400 |
|---|---|---|
| committer | Eric Sandeen <sandeen@sandeen.net> | 2019-11-01 15:46:12 -0400 |
| commit | d9b8ae44b6ee51329af7d81968b9feacc55ffa13 (patch) | |
| tree | 3a363b1ba744ee95152a52a0987904ee422f77a5 /spaceman | |
| parent | cac2b8b0a2bc343c3846299f544396add813e691 (diff) | |
| download | xfsprogs-dev-d9b8ae44b6ee51329af7d81968b9feacc55ffa13.tar.gz | |
xfs_spaceman: always report sick metadata, checked or not
If the kernel thinks a piece of metadata is bad, we must always report
it. This will happen with an upcoming series to mark things sick
whenever we return EFSCORRUPTED at runtime.
Signed-off-by: Darrick J. Wong <darrick.wong@oracle.com>
Reviewed-by: Eric Sandeen <sandeen@redhat.com>
Signed-off-by: Eric Sandeen <sandeen@sandeen.net>
Diffstat (limited to 'spaceman')
| -rw-r--r-- | spaceman/health.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/spaceman/health.c b/spaceman/health.c index 8fd985a291..0d3aa243ee 100644 --- a/spaceman/health.c +++ b/spaceman/health.c @@ -171,10 +171,10 @@ report_sick( for (f = maps; f->mask != 0; f++) { if (f->has_fn && !f->has_fn(&file->xfd.fsgeom)) continue; - if (!(checked & f->mask)) + bad = sick & f->mask; + if (!bad && !(checked & f->mask)) continue; reported++; - bad = sick & f->mask; if (!bad && quiet) continue; printf("%s %s: %s\n", descr, _(f->descr), |