xref: /openbmc/linux/fs/xfs/scrub/fscounters.c (revision 75efa57d0bf5fcf650a183f0ce0dc011ba8c4bc8)
1*75efa57dSDarrick J. Wong // SPDX-License-Identifier: GPL-2.0+
2*75efa57dSDarrick J. Wong /*
3*75efa57dSDarrick J. Wong  * Copyright (C) 2019 Oracle.  All Rights Reserved.
4*75efa57dSDarrick J. Wong  * Author: Darrick J. Wong <darrick.wong@oracle.com>
5*75efa57dSDarrick J. Wong  */
6*75efa57dSDarrick J. Wong #include "xfs.h"
7*75efa57dSDarrick J. Wong #include "xfs_fs.h"
8*75efa57dSDarrick J. Wong #include "xfs_shared.h"
9*75efa57dSDarrick J. Wong #include "xfs_format.h"
10*75efa57dSDarrick J. Wong #include "xfs_trans_resv.h"
11*75efa57dSDarrick J. Wong #include "xfs_mount.h"
12*75efa57dSDarrick J. Wong #include "xfs_defer.h"
13*75efa57dSDarrick J. Wong #include "xfs_btree.h"
14*75efa57dSDarrick J. Wong #include "xfs_bit.h"
15*75efa57dSDarrick J. Wong #include "xfs_log_format.h"
16*75efa57dSDarrick J. Wong #include "xfs_trans.h"
17*75efa57dSDarrick J. Wong #include "xfs_sb.h"
18*75efa57dSDarrick J. Wong #include "xfs_inode.h"
19*75efa57dSDarrick J. Wong #include "xfs_alloc.h"
20*75efa57dSDarrick J. Wong #include "xfs_ialloc.h"
21*75efa57dSDarrick J. Wong #include "xfs_rmap.h"
22*75efa57dSDarrick J. Wong #include "xfs_error.h"
23*75efa57dSDarrick J. Wong #include "xfs_errortag.h"
24*75efa57dSDarrick J. Wong #include "xfs_icache.h"
25*75efa57dSDarrick J. Wong #include "xfs_health.h"
26*75efa57dSDarrick J. Wong #include "xfs_bmap.h"
27*75efa57dSDarrick J. Wong #include "scrub/xfs_scrub.h"
28*75efa57dSDarrick J. Wong #include "scrub/scrub.h"
29*75efa57dSDarrick J. Wong #include "scrub/common.h"
30*75efa57dSDarrick J. Wong #include "scrub/trace.h"
31*75efa57dSDarrick J. Wong 
32*75efa57dSDarrick J. Wong /*
33*75efa57dSDarrick J. Wong  * FS Summary Counters
34*75efa57dSDarrick J. Wong  * ===================
35*75efa57dSDarrick J. Wong  *
36*75efa57dSDarrick J. Wong  * The basics of filesystem summary counter checking are that we iterate the
37*75efa57dSDarrick J. Wong  * AGs counting the number of free blocks, free space btree blocks, per-AG
38*75efa57dSDarrick J. Wong  * reservations, inodes, delayed allocation reservations, and free inodes.
39*75efa57dSDarrick J. Wong  * Then we compare what we computed against the in-core counters.
40*75efa57dSDarrick J. Wong  *
41*75efa57dSDarrick J. Wong  * However, the reality is that summary counters are a tricky beast to check.
42*75efa57dSDarrick J. Wong  * While we /could/ freeze the filesystem and scramble around the AGs counting
43*75efa57dSDarrick J. Wong  * the free blocks, in practice we prefer not do that for a scan because
44*75efa57dSDarrick J. Wong  * freezing is costly.  To get around this, we added a per-cpu counter of the
45*75efa57dSDarrick J. Wong  * delalloc reservations so that we can rotor around the AGs relatively
46*75efa57dSDarrick J. Wong  * quickly, and we allow the counts to be slightly off because we're not taking
47*75efa57dSDarrick J. Wong  * any locks while we do this.
48*75efa57dSDarrick J. Wong  *
49*75efa57dSDarrick J. Wong  * So the first thing we do is warm up the buffer cache in the setup routine by
50*75efa57dSDarrick J. Wong  * walking all the AGs to make sure the incore per-AG structure has been
51*75efa57dSDarrick J. Wong  * initialized.  The expected value calculation then iterates the incore per-AG
52*75efa57dSDarrick J. Wong  * structures as quickly as it can.  We snapshot the percpu counters before and
53*75efa57dSDarrick J. Wong  * after this operation and use the difference in counter values to guess at
54*75efa57dSDarrick J. Wong  * our tolerance for mismatch between expected and actual counter values.
55*75efa57dSDarrick J. Wong  */
56*75efa57dSDarrick J. Wong 
57*75efa57dSDarrick J. Wong /*
58*75efa57dSDarrick J. Wong  * Since the expected value computation is lockless but only browses incore
59*75efa57dSDarrick J. Wong  * values, the percpu counters should be fairly close to each other.  However,
60*75efa57dSDarrick J. Wong  * we'll allow ourselves to be off by at least this (arbitrary) amount.
61*75efa57dSDarrick J. Wong  */
62*75efa57dSDarrick J. Wong #define XCHK_FSCOUNT_MIN_VARIANCE	(512)
63*75efa57dSDarrick J. Wong 
64*75efa57dSDarrick J. Wong /*
65*75efa57dSDarrick J. Wong  * Make sure the per-AG structure has been initialized from the on-disk header
66*75efa57dSDarrick J. Wong  * contents and trust that the incore counters match the ondisk counters.  (The
67*75efa57dSDarrick J. Wong  * AGF and AGI scrubbers check them, and a normal xfs_scrub run checks the
68*75efa57dSDarrick J. Wong  * summary counters after checking all AG headers).  Do this from the setup
69*75efa57dSDarrick J. Wong  * function so that the inner AG aggregation loop runs as quickly as possible.
70*75efa57dSDarrick J. Wong  *
71*75efa57dSDarrick J. Wong  * This function runs during the setup phase /before/ we start checking any
72*75efa57dSDarrick J. Wong  * metadata.
73*75efa57dSDarrick J. Wong  */
74*75efa57dSDarrick J. Wong STATIC int
75*75efa57dSDarrick J. Wong xchk_fscount_warmup(
76*75efa57dSDarrick J. Wong 	struct xfs_scrub	*sc)
77*75efa57dSDarrick J. Wong {
78*75efa57dSDarrick J. Wong 	struct xfs_mount	*mp = sc->mp;
79*75efa57dSDarrick J. Wong 	struct xfs_buf		*agi_bp = NULL;
80*75efa57dSDarrick J. Wong 	struct xfs_buf		*agf_bp = NULL;
81*75efa57dSDarrick J. Wong 	struct xfs_perag	*pag = NULL;
82*75efa57dSDarrick J. Wong 	xfs_agnumber_t		agno;
83*75efa57dSDarrick J. Wong 	int			error = 0;
84*75efa57dSDarrick J. Wong 
85*75efa57dSDarrick J. Wong 	for (agno = 0; agno < mp->m_sb.sb_agcount; agno++) {
86*75efa57dSDarrick J. Wong 		pag = xfs_perag_get(mp, agno);
87*75efa57dSDarrick J. Wong 
88*75efa57dSDarrick J. Wong 		if (pag->pagi_init && pag->pagf_init)
89*75efa57dSDarrick J. Wong 			goto next_loop_perag;
90*75efa57dSDarrick J. Wong 
91*75efa57dSDarrick J. Wong 		/* Lock both AG headers. */
92*75efa57dSDarrick J. Wong 		error = xfs_ialloc_read_agi(mp, sc->tp, agno, &agi_bp);
93*75efa57dSDarrick J. Wong 		if (error)
94*75efa57dSDarrick J. Wong 			break;
95*75efa57dSDarrick J. Wong 		error = xfs_alloc_read_agf(mp, sc->tp, agno, 0, &agf_bp);
96*75efa57dSDarrick J. Wong 		if (error)
97*75efa57dSDarrick J. Wong 			break;
98*75efa57dSDarrick J. Wong 		error = -ENOMEM;
99*75efa57dSDarrick J. Wong 		if (!agf_bp || !agi_bp)
100*75efa57dSDarrick J. Wong 			break;
101*75efa57dSDarrick J. Wong 
102*75efa57dSDarrick J. Wong 		/*
103*75efa57dSDarrick J. Wong 		 * These are supposed to be initialized by the header read
104*75efa57dSDarrick J. Wong 		 * function.
105*75efa57dSDarrick J. Wong 		 */
106*75efa57dSDarrick J. Wong 		error = -EFSCORRUPTED;
107*75efa57dSDarrick J. Wong 		if (!pag->pagi_init || !pag->pagf_init)
108*75efa57dSDarrick J. Wong 			break;
109*75efa57dSDarrick J. Wong 
110*75efa57dSDarrick J. Wong 		xfs_buf_relse(agf_bp);
111*75efa57dSDarrick J. Wong 		agf_bp = NULL;
112*75efa57dSDarrick J. Wong 		xfs_buf_relse(agi_bp);
113*75efa57dSDarrick J. Wong 		agi_bp = NULL;
114*75efa57dSDarrick J. Wong next_loop_perag:
115*75efa57dSDarrick J. Wong 		xfs_perag_put(pag);
116*75efa57dSDarrick J. Wong 		pag = NULL;
117*75efa57dSDarrick J. Wong 		error = 0;
118*75efa57dSDarrick J. Wong 
119*75efa57dSDarrick J. Wong 		if (fatal_signal_pending(current))
120*75efa57dSDarrick J. Wong 			break;
121*75efa57dSDarrick J. Wong 	}
122*75efa57dSDarrick J. Wong 
123*75efa57dSDarrick J. Wong 	if (agf_bp)
124*75efa57dSDarrick J. Wong 		xfs_buf_relse(agf_bp);
125*75efa57dSDarrick J. Wong 	if (agi_bp)
126*75efa57dSDarrick J. Wong 		xfs_buf_relse(agi_bp);
127*75efa57dSDarrick J. Wong 	if (pag)
128*75efa57dSDarrick J. Wong 		xfs_perag_put(pag);
129*75efa57dSDarrick J. Wong 	return error;
130*75efa57dSDarrick J. Wong }
131*75efa57dSDarrick J. Wong 
132*75efa57dSDarrick J. Wong int
133*75efa57dSDarrick J. Wong xchk_setup_fscounters(
134*75efa57dSDarrick J. Wong 	struct xfs_scrub	*sc,
135*75efa57dSDarrick J. Wong 	struct xfs_inode	*ip)
136*75efa57dSDarrick J. Wong {
137*75efa57dSDarrick J. Wong 	struct xchk_fscounters	*fsc;
138*75efa57dSDarrick J. Wong 	int			error;
139*75efa57dSDarrick J. Wong 
140*75efa57dSDarrick J. Wong 	sc->buf = kmem_zalloc(sizeof(struct xchk_fscounters), KM_SLEEP);
141*75efa57dSDarrick J. Wong 	if (!sc->buf)
142*75efa57dSDarrick J. Wong 		return -ENOMEM;
143*75efa57dSDarrick J. Wong 	fsc = sc->buf;
144*75efa57dSDarrick J. Wong 
145*75efa57dSDarrick J. Wong 	xfs_icount_range(sc->mp, &fsc->icount_min, &fsc->icount_max);
146*75efa57dSDarrick J. Wong 
147*75efa57dSDarrick J. Wong 	/* We must get the incore counters set up before we can proceed. */
148*75efa57dSDarrick J. Wong 	error = xchk_fscount_warmup(sc);
149*75efa57dSDarrick J. Wong 	if (error)
150*75efa57dSDarrick J. Wong 		return error;
151*75efa57dSDarrick J. Wong 
152*75efa57dSDarrick J. Wong 	/*
153*75efa57dSDarrick J. Wong 	 * Pause background reclaim while we're scrubbing to reduce the
154*75efa57dSDarrick J. Wong 	 * likelihood of background perturbations to the counters throwing off
155*75efa57dSDarrick J. Wong 	 * our calculations.
156*75efa57dSDarrick J. Wong 	 */
157*75efa57dSDarrick J. Wong 	xchk_stop_reaping(sc);
158*75efa57dSDarrick J. Wong 
159*75efa57dSDarrick J. Wong 	return xchk_trans_alloc(sc, 0);
160*75efa57dSDarrick J. Wong }
161*75efa57dSDarrick J. Wong 
162*75efa57dSDarrick J. Wong /*
163*75efa57dSDarrick J. Wong  * Calculate what the global in-core counters ought to be from the incore
164*75efa57dSDarrick J. Wong  * per-AG structure.  Callers can compare this to the actual in-core counters
165*75efa57dSDarrick J. Wong  * to estimate by how much both in-core and on-disk counters need to be
166*75efa57dSDarrick J. Wong  * adjusted.
167*75efa57dSDarrick J. Wong  */
168*75efa57dSDarrick J. Wong STATIC int
169*75efa57dSDarrick J. Wong xchk_fscount_aggregate_agcounts(
170*75efa57dSDarrick J. Wong 	struct xfs_scrub	*sc,
171*75efa57dSDarrick J. Wong 	struct xchk_fscounters	*fsc)
172*75efa57dSDarrick J. Wong {
173*75efa57dSDarrick J. Wong 	struct xfs_mount	*mp = sc->mp;
174*75efa57dSDarrick J. Wong 	struct xfs_perag	*pag;
175*75efa57dSDarrick J. Wong 	uint64_t		delayed;
176*75efa57dSDarrick J. Wong 	xfs_agnumber_t		agno;
177*75efa57dSDarrick J. Wong 	int			tries = 8;
178*75efa57dSDarrick J. Wong 
179*75efa57dSDarrick J. Wong retry:
180*75efa57dSDarrick J. Wong 	fsc->icount = 0;
181*75efa57dSDarrick J. Wong 	fsc->ifree = 0;
182*75efa57dSDarrick J. Wong 	fsc->fdblocks = 0;
183*75efa57dSDarrick J. Wong 
184*75efa57dSDarrick J. Wong 	for (agno = 0; agno < mp->m_sb.sb_agcount; agno++) {
185*75efa57dSDarrick J. Wong 		pag = xfs_perag_get(mp, agno);
186*75efa57dSDarrick J. Wong 
187*75efa57dSDarrick J. Wong 		/* This somehow got unset since the warmup? */
188*75efa57dSDarrick J. Wong 		if (!pag->pagi_init || !pag->pagf_init) {
189*75efa57dSDarrick J. Wong 			xfs_perag_put(pag);
190*75efa57dSDarrick J. Wong 			return -EFSCORRUPTED;
191*75efa57dSDarrick J. Wong 		}
192*75efa57dSDarrick J. Wong 
193*75efa57dSDarrick J. Wong 		/* Count all the inodes */
194*75efa57dSDarrick J. Wong 		fsc->icount += pag->pagi_count;
195*75efa57dSDarrick J. Wong 		fsc->ifree += pag->pagi_freecount;
196*75efa57dSDarrick J. Wong 
197*75efa57dSDarrick J. Wong 		/* Add up the free/freelist/bnobt/cntbt blocks */
198*75efa57dSDarrick J. Wong 		fsc->fdblocks += pag->pagf_freeblks;
199*75efa57dSDarrick J. Wong 		fsc->fdblocks += pag->pagf_flcount;
200*75efa57dSDarrick J. Wong 		fsc->fdblocks += pag->pagf_btreeblks;
201*75efa57dSDarrick J. Wong 
202*75efa57dSDarrick J. Wong 		/*
203*75efa57dSDarrick J. Wong 		 * Per-AG reservations are taken out of the incore counters,
204*75efa57dSDarrick J. Wong 		 * so they must be left out of the free blocks computation.
205*75efa57dSDarrick J. Wong 		 */
206*75efa57dSDarrick J. Wong 		fsc->fdblocks -= pag->pag_meta_resv.ar_reserved;
207*75efa57dSDarrick J. Wong 		fsc->fdblocks -= pag->pag_rmapbt_resv.ar_orig_reserved;
208*75efa57dSDarrick J. Wong 
209*75efa57dSDarrick J. Wong 		xfs_perag_put(pag);
210*75efa57dSDarrick J. Wong 
211*75efa57dSDarrick J. Wong 		if (fatal_signal_pending(current))
212*75efa57dSDarrick J. Wong 			break;
213*75efa57dSDarrick J. Wong 	}
214*75efa57dSDarrick J. Wong 
215*75efa57dSDarrick J. Wong 	/*
216*75efa57dSDarrick J. Wong 	 * The global incore space reservation is taken from the incore
217*75efa57dSDarrick J. Wong 	 * counters, so leave that out of the computation.
218*75efa57dSDarrick J. Wong 	 */
219*75efa57dSDarrick J. Wong 	fsc->fdblocks -= mp->m_resblks_avail;
220*75efa57dSDarrick J. Wong 
221*75efa57dSDarrick J. Wong 	/*
222*75efa57dSDarrick J. Wong 	 * Delayed allocation reservations are taken out of the incore counters
223*75efa57dSDarrick J. Wong 	 * but not recorded on disk, so leave them and their indlen blocks out
224*75efa57dSDarrick J. Wong 	 * of the computation.
225*75efa57dSDarrick J. Wong 	 */
226*75efa57dSDarrick J. Wong 	delayed = percpu_counter_sum(&mp->m_delalloc_blks);
227*75efa57dSDarrick J. Wong 	fsc->fdblocks -= delayed;
228*75efa57dSDarrick J. Wong 
229*75efa57dSDarrick J. Wong 	trace_xchk_fscounters_calc(mp, fsc->icount, fsc->ifree, fsc->fdblocks,
230*75efa57dSDarrick J. Wong 			delayed);
231*75efa57dSDarrick J. Wong 
232*75efa57dSDarrick J. Wong 
233*75efa57dSDarrick J. Wong 	/* Bail out if the values we compute are totally nonsense. */
234*75efa57dSDarrick J. Wong 	if (fsc->icount < fsc->icount_min || fsc->icount > fsc->icount_max ||
235*75efa57dSDarrick J. Wong 	    fsc->fdblocks > mp->m_sb.sb_dblocks ||
236*75efa57dSDarrick J. Wong 	    fsc->ifree > fsc->icount_max)
237*75efa57dSDarrick J. Wong 		return -EFSCORRUPTED;
238*75efa57dSDarrick J. Wong 
239*75efa57dSDarrick J. Wong 	/*
240*75efa57dSDarrick J. Wong 	 * If ifree > icount then we probably had some perturbation in the
241*75efa57dSDarrick J. Wong 	 * counters while we were calculating things.  We'll try a few times
242*75efa57dSDarrick J. Wong 	 * to maintain ifree <= icount before giving up.
243*75efa57dSDarrick J. Wong 	 */
244*75efa57dSDarrick J. Wong 	if (fsc->ifree > fsc->icount) {
245*75efa57dSDarrick J. Wong 		if (tries--)
246*75efa57dSDarrick J. Wong 			goto retry;
247*75efa57dSDarrick J. Wong 		xchk_set_incomplete(sc);
248*75efa57dSDarrick J. Wong 		return 0;
249*75efa57dSDarrick J. Wong 	}
250*75efa57dSDarrick J. Wong 
251*75efa57dSDarrick J. Wong 	return 0;
252*75efa57dSDarrick J. Wong }
253*75efa57dSDarrick J. Wong 
254*75efa57dSDarrick J. Wong /*
255*75efa57dSDarrick J. Wong  * Is the @counter reasonably close to the @expected value?
256*75efa57dSDarrick J. Wong  *
257*75efa57dSDarrick J. Wong  * We neither locked nor froze anything in the filesystem while aggregating the
258*75efa57dSDarrick J. Wong  * per-AG data to compute the @expected value, which means that the counter
259*75efa57dSDarrick J. Wong  * could have changed.  We know the @old_value of the summation of the counter
260*75efa57dSDarrick J. Wong  * before the aggregation, and we re-sum the counter now.  If the expected
261*75efa57dSDarrick J. Wong  * value falls between the two summations, we're ok.
262*75efa57dSDarrick J. Wong  *
263*75efa57dSDarrick J. Wong  * Otherwise, we /might/ have a problem.  If the change in the summations is
264*75efa57dSDarrick J. Wong  * more than we want to tolerate, the filesystem is probably busy and we should
265*75efa57dSDarrick J. Wong  * just send back INCOMPLETE and see if userspace will try again.
266*75efa57dSDarrick J. Wong  */
267*75efa57dSDarrick J. Wong static inline bool
268*75efa57dSDarrick J. Wong xchk_fscount_within_range(
269*75efa57dSDarrick J. Wong 	struct xfs_scrub	*sc,
270*75efa57dSDarrick J. Wong 	const int64_t		old_value,
271*75efa57dSDarrick J. Wong 	struct percpu_counter	*counter,
272*75efa57dSDarrick J. Wong 	uint64_t		expected)
273*75efa57dSDarrick J. Wong {
274*75efa57dSDarrick J. Wong 	int64_t			min_value, max_value;
275*75efa57dSDarrick J. Wong 	int64_t			curr_value = percpu_counter_sum(counter);
276*75efa57dSDarrick J. Wong 
277*75efa57dSDarrick J. Wong 	trace_xchk_fscounters_within_range(sc->mp, expected, curr_value,
278*75efa57dSDarrick J. Wong 			old_value);
279*75efa57dSDarrick J. Wong 
280*75efa57dSDarrick J. Wong 	/* Negative values are always wrong. */
281*75efa57dSDarrick J. Wong 	if (curr_value < 0)
282*75efa57dSDarrick J. Wong 		return false;
283*75efa57dSDarrick J. Wong 
284*75efa57dSDarrick J. Wong 	/* Exact matches are always ok. */
285*75efa57dSDarrick J. Wong 	if (curr_value == expected)
286*75efa57dSDarrick J. Wong 		return true;
287*75efa57dSDarrick J. Wong 
288*75efa57dSDarrick J. Wong 	min_value = min(old_value, curr_value);
289*75efa57dSDarrick J. Wong 	max_value = max(old_value, curr_value);
290*75efa57dSDarrick J. Wong 
291*75efa57dSDarrick J. Wong 	/* Within the before-and-after range is ok. */
292*75efa57dSDarrick J. Wong 	if (expected >= min_value && expected <= max_value)
293*75efa57dSDarrick J. Wong 		return true;
294*75efa57dSDarrick J. Wong 
295*75efa57dSDarrick J. Wong 	/*
296*75efa57dSDarrick J. Wong 	 * If the difference between the two summations is too large, the fs
297*75efa57dSDarrick J. Wong 	 * might just be busy and so we'll mark the scrub incomplete.  Return
298*75efa57dSDarrick J. Wong 	 * true here so that we don't mark the counter corrupt.
299*75efa57dSDarrick J. Wong 	 *
300*75efa57dSDarrick J. Wong 	 * XXX: In the future when userspace can grant scrub permission to
301*75efa57dSDarrick J. Wong 	 * quiesce the filesystem to solve the outsized variance problem, this
302*75efa57dSDarrick J. Wong 	 * check should be moved up and the return code changed to signal to
303*75efa57dSDarrick J. Wong 	 * userspace that we need quiesce permission.
304*75efa57dSDarrick J. Wong 	 */
305*75efa57dSDarrick J. Wong 	if (max_value - min_value >= XCHK_FSCOUNT_MIN_VARIANCE) {
306*75efa57dSDarrick J. Wong 		xchk_set_incomplete(sc);
307*75efa57dSDarrick J. Wong 		return true;
308*75efa57dSDarrick J. Wong 	}
309*75efa57dSDarrick J. Wong 
310*75efa57dSDarrick J. Wong 	return false;
311*75efa57dSDarrick J. Wong }
312*75efa57dSDarrick J. Wong 
313*75efa57dSDarrick J. Wong /* Check the superblock counters. */
314*75efa57dSDarrick J. Wong int
315*75efa57dSDarrick J. Wong xchk_fscounters(
316*75efa57dSDarrick J. Wong 	struct xfs_scrub	*sc)
317*75efa57dSDarrick J. Wong {
318*75efa57dSDarrick J. Wong 	struct xfs_mount	*mp = sc->mp;
319*75efa57dSDarrick J. Wong 	struct xchk_fscounters	*fsc = sc->buf;
320*75efa57dSDarrick J. Wong 	int64_t			icount, ifree, fdblocks;
321*75efa57dSDarrick J. Wong 	int			error;
322*75efa57dSDarrick J. Wong 
323*75efa57dSDarrick J. Wong 	/* Snapshot the percpu counters. */
324*75efa57dSDarrick J. Wong 	icount = percpu_counter_sum(&mp->m_icount);
325*75efa57dSDarrick J. Wong 	ifree = percpu_counter_sum(&mp->m_ifree);
326*75efa57dSDarrick J. Wong 	fdblocks = percpu_counter_sum(&mp->m_fdblocks);
327*75efa57dSDarrick J. Wong 
328*75efa57dSDarrick J. Wong 	/* No negative values, please! */
329*75efa57dSDarrick J. Wong 	if (icount < 0 || ifree < 0 || fdblocks < 0)
330*75efa57dSDarrick J. Wong 		xchk_set_corrupt(sc);
331*75efa57dSDarrick J. Wong 
332*75efa57dSDarrick J. Wong 	/* See if icount is obviously wrong. */
333*75efa57dSDarrick J. Wong 	if (icount < fsc->icount_min || icount > fsc->icount_max)
334*75efa57dSDarrick J. Wong 		xchk_set_corrupt(sc);
335*75efa57dSDarrick J. Wong 
336*75efa57dSDarrick J. Wong 	/* See if fdblocks is obviously wrong. */
337*75efa57dSDarrick J. Wong 	if (fdblocks > mp->m_sb.sb_dblocks)
338*75efa57dSDarrick J. Wong 		xchk_set_corrupt(sc);
339*75efa57dSDarrick J. Wong 
340*75efa57dSDarrick J. Wong 	/*
341*75efa57dSDarrick J. Wong 	 * If ifree exceeds icount by more than the minimum variance then
342*75efa57dSDarrick J. Wong 	 * something's probably wrong with the counters.
343*75efa57dSDarrick J. Wong 	 */
344*75efa57dSDarrick J. Wong 	if (ifree > icount && ifree - icount > XCHK_FSCOUNT_MIN_VARIANCE)
345*75efa57dSDarrick J. Wong 		xchk_set_corrupt(sc);
346*75efa57dSDarrick J. Wong 
347*75efa57dSDarrick J. Wong 	/* Walk the incore AG headers to calculate the expected counters. */
348*75efa57dSDarrick J. Wong 	error = xchk_fscount_aggregate_agcounts(sc, fsc);
349*75efa57dSDarrick J. Wong 	if (!xchk_process_error(sc, 0, XFS_SB_BLOCK(mp), &error))
350*75efa57dSDarrick J. Wong 		return error;
351*75efa57dSDarrick J. Wong 	if (sc->sm->sm_flags & XFS_SCRUB_OFLAG_INCOMPLETE)
352*75efa57dSDarrick J. Wong 		return 0;
353*75efa57dSDarrick J. Wong 
354*75efa57dSDarrick J. Wong 	/* Compare the in-core counters with whatever we counted. */
355*75efa57dSDarrick J. Wong 	if (!xchk_fscount_within_range(sc, icount, &mp->m_icount, fsc->icount))
356*75efa57dSDarrick J. Wong 		xchk_set_corrupt(sc);
357*75efa57dSDarrick J. Wong 
358*75efa57dSDarrick J. Wong 	if (!xchk_fscount_within_range(sc, ifree, &mp->m_ifree, fsc->ifree))
359*75efa57dSDarrick J. Wong 		xchk_set_corrupt(sc);
360*75efa57dSDarrick J. Wong 
361*75efa57dSDarrick J. Wong 	if (!xchk_fscount_within_range(sc, fdblocks, &mp->m_fdblocks,
362*75efa57dSDarrick J. Wong 			fsc->fdblocks))
363*75efa57dSDarrick J. Wong 		xchk_set_corrupt(sc);
364*75efa57dSDarrick J. Wong 
365*75efa57dSDarrick J. Wong 	return 0;
366*75efa57dSDarrick J. Wong }
367