1 // SPDX-License-Identifier: GPL-2.0-or-later
2 /*
3 * Copyright (C) 2017-2023 Oracle. All Rights Reserved.
4 * Author: Darrick J. Wong <djwong@kernel.org>
5 */
6 #include "xfs_platform.h"
7 #include "xfs_fs.h"
8 #include "xfs_shared.h"
9 #include "xfs_format.h"
10 #include "xfs_log_format.h"
11 #include "xfs_trans_resv.h"
12 #include "xfs_mount.h"
13 #include "xfs_trans.h"
14 #include "xfs_ag.h"
15 #include "xfs_btree.h"
16 #include "xfs_rmap.h"
17 #include "xfs_refcount.h"
18 #include "scrub/scrub.h"
19 #include "scrub/common.h"
20 #include "scrub/btree.h"
21 #include "scrub/trace.h"
22 #include "scrub/repair.h"
23
24 /*
25 * Set us up to scrub reference count btrees.
26 */
27 int
xchk_setup_ag_refcountbt(struct xfs_scrub * sc)28 xchk_setup_ag_refcountbt(
29 struct xfs_scrub *sc)
30 {
31 if (xchk_need_intent_drain(sc))
32 xchk_fsgates_enable(sc, XCHK_FSGATES_DRAIN);
33
34 if (xchk_could_repair(sc)) {
35 int error;
36
37 error = xrep_setup_ag_refcountbt(sc);
38 if (error)
39 return error;
40 }
41
42 return xchk_setup_ag_btree(sc, false);
43 }
44
45 /* Reference count btree scrubber. */
46
47 /*
48 * Confirming Reference Counts via Reverse Mappings
49 *
50 * We want to count the reverse mappings overlapping a refcount record
51 * (bno, len, refcount), allowing for the possibility that some of the
52 * overlap may come from smaller adjoining reverse mappings, while some
53 * comes from single extents which overlap the range entirely. The
54 * outer loop is as follows:
55 *
56 * 1. For all reverse mappings overlapping the refcount extent,
57 * a. If a given rmap completely overlaps, mark it as seen.
58 * b. Otherwise, record the fragment (in agbno order) for later
59 * processing.
60 *
61 * Once we've seen all the rmaps, we know that for all blocks in the
62 * refcount record we want to find $refcount owners and we've already
63 * visited $seen extents that overlap all the blocks. Therefore, we
64 * need to find ($refcount - $seen) owners for every block in the
65 * extent; call that quantity $target_nr. Proceed as follows:
66 *
67 * 2. Pull the first $target_nr fragments from the list; all of them
68 * should start at or before the start of the extent.
69 * Call this subset of fragments the working set.
70 * 3. Until there are no more unprocessed fragments,
71 * a. Find the shortest fragments in the set and remove them.
72 * b. Note the block number of the end of these fragments.
73 * c. Pull the same number of fragments from the list. All of these
74 * fragments should start at the block number recorded in the
75 * previous step.
76 * d. Put those fragments in the set.
77 * 4. Check that there are $target_nr fragments remaining in the list,
78 * and that they all end at or beyond the end of the refcount extent.
79 *
80 * If the refcount is correct, all the check conditions in the algorithm
81 * should always hold true. If not, the refcount is incorrect.
82 */
83 struct xchk_refcnt_frag {
84 struct list_head list;
85 struct xfs_rmap_irec rm;
86 };
87
88 struct xchk_refcnt_check {
89 struct xfs_scrub *sc;
90 struct list_head fragments;
91
92 /* refcount extent we're examining */
93 xfs_agblock_t bno;
94 xfs_extlen_t len;
95 xfs_nlink_t refcount;
96
97 /* number of owners seen */
98 xfs_nlink_t seen;
99 };
100
101 /*
102 * Decide if the given rmap is large enough that we can redeem it
103 * towards refcount verification now, or if it's a fragment, in
104 * which case we'll hang onto it in the hopes that we'll later
105 * discover that we've collected exactly the correct number of
106 * fragments as the refcountbt says we should have.
107 */
108 STATIC int
xchk_refcountbt_rmap_check(struct xfs_btree_cur * cur,const struct xfs_rmap_irec * rec,void * priv)109 xchk_refcountbt_rmap_check(
110 struct xfs_btree_cur *cur,
111 const struct xfs_rmap_irec *rec,
112 void *priv)
113 {
114 struct xchk_refcnt_check *refchk = priv;
115 struct xchk_refcnt_frag *frag;
116 xfs_agblock_t rm_last;
117 xfs_agblock_t rc_last;
118 int error = 0;
119
120 if (xchk_should_terminate(refchk->sc, &error))
121 return error;
122
123 rm_last = rec->rm_startblock + rec->rm_blockcount - 1;
124 rc_last = refchk->bno + refchk->len - 1;
125
126 /* Confirm that a single-owner refc extent is a CoW stage. */
127 if (refchk->refcount == 1 && rec->rm_owner != XFS_RMAP_OWN_COW) {
128 xchk_btree_xref_set_corrupt(refchk->sc, cur, 0);
129 return 0;
130 }
131
132 if (rec->rm_startblock <= refchk->bno && rm_last >= rc_last) {
133 /*
134 * The rmap overlaps the refcount record, so we can confirm
135 * one refcount owner seen.
136 */
137 refchk->seen++;
138 } else {
139 /*
140 * This rmap covers only part of the refcount record, so
141 * save the fragment for later processing. If the rmapbt
142 * is healthy each rmap_irec we see will be in agbno order
143 * so we don't need insertion sort here.
144 */
145 frag = kmalloc_obj(struct xchk_refcnt_frag, XCHK_GFP_FLAGS);
146 if (!frag)
147 return -ENOMEM;
148 memcpy(&frag->rm, rec, sizeof(frag->rm));
149 list_add_tail(&frag->list, &refchk->fragments);
150 }
151
152 return 0;
153 }
154
155 /*
156 * Given a bunch of rmap fragments, iterate through them, keeping
157 * a running tally of the refcount. If this ever deviates from
158 * what we expect (which is the refcountbt's refcount minus the
159 * number of extents that totally covered the refcountbt extent),
160 * we have a refcountbt error.
161 */
162 STATIC void
xchk_refcountbt_process_rmap_fragments(struct xchk_refcnt_check * refchk)163 xchk_refcountbt_process_rmap_fragments(
164 struct xchk_refcnt_check *refchk)
165 {
166 struct list_head worklist;
167 struct xchk_refcnt_frag *frag;
168 struct xchk_refcnt_frag *n;
169 xfs_agblock_t bno;
170 xfs_agblock_t rbno;
171 xfs_agblock_t next_rbno;
172 xfs_nlink_t nr;
173 xfs_nlink_t target_nr;
174
175 target_nr = refchk->refcount - refchk->seen;
176 if (target_nr == 0)
177 return;
178
179 /*
180 * There are (refchk->rc.rc_refcount - refchk->nr refcount)
181 * references we haven't found yet. Pull that many off the
182 * fragment list and figure out where the smallest rmap ends
183 * (and therefore the next rmap should start). All the rmaps
184 * we pull off should start at or before the beginning of the
185 * refcount record's range.
186 */
187 INIT_LIST_HEAD(&worklist);
188 rbno = NULLAGBLOCK;
189
190 /* Make sure the fragments actually /are/ in agbno order. */
191 bno = 0;
192 list_for_each_entry(frag, &refchk->fragments, list) {
193 if (frag->rm.rm_startblock < bno)
194 goto done;
195 bno = frag->rm.rm_startblock;
196 }
197
198 /*
199 * Find all the rmaps that start at or before the refc extent,
200 * and put them on the worklist.
201 */
202 nr = 0;
203 list_for_each_entry_safe(frag, n, &refchk->fragments, list) {
204 if (frag->rm.rm_startblock > refchk->bno || nr > target_nr)
205 break;
206 bno = frag->rm.rm_startblock + frag->rm.rm_blockcount;
207 if (bno < rbno)
208 rbno = bno;
209 list_move_tail(&frag->list, &worklist);
210 nr++;
211 }
212
213 /*
214 * We should have found exactly $target_nr rmap fragments starting
215 * at or before the refcount extent.
216 */
217 if (nr != target_nr)
218 goto done;
219
220 while (!list_empty(&refchk->fragments)) {
221 /* Discard any fragments ending at rbno from the worklist. */
222 nr = 0;
223 next_rbno = NULLAGBLOCK;
224 list_for_each_entry_safe(frag, n, &worklist, list) {
225 bno = frag->rm.rm_startblock + frag->rm.rm_blockcount;
226 if (bno != rbno) {
227 if (bno < next_rbno)
228 next_rbno = bno;
229 continue;
230 }
231 list_del(&frag->list);
232 kfree(frag);
233 nr++;
234 }
235
236 /* Try to add nr rmaps starting at rbno to the worklist. */
237 list_for_each_entry_safe(frag, n, &refchk->fragments, list) {
238 bno = frag->rm.rm_startblock + frag->rm.rm_blockcount;
239 if (frag->rm.rm_startblock != rbno)
240 goto done;
241 list_move_tail(&frag->list, &worklist);
242 if (next_rbno > bno)
243 next_rbno = bno;
244 nr--;
245 if (nr == 0)
246 break;
247 }
248
249 /*
250 * If we get here and nr > 0, this means that we added fewer
251 * items to the worklist than we discarded because the fragment
252 * list ran out of items. Therefore, we cannot maintain the
253 * required refcount. Something is wrong, so we're done.
254 */
255 if (nr)
256 goto done;
257
258 rbno = next_rbno;
259 }
260
261 /*
262 * Make sure the last extent we processed ends at or beyond
263 * the end of the refcount extent.
264 */
265 if (rbno < refchk->bno + refchk->len)
266 goto done;
267
268 /* Actually record us having seen the remaining refcount. */
269 refchk->seen = refchk->refcount;
270 done:
271 /* Delete fragments and work list. */
272 list_for_each_entry_safe(frag, n, &worklist, list) {
273 list_del(&frag->list);
274 kfree(frag);
275 }
276 list_for_each_entry_safe(frag, n, &refchk->fragments, list) {
277 list_del(&frag->list);
278 kfree(frag);
279 }
280 }
281
282 /* Use the rmap entries covering this extent to verify the refcount. */
283 STATIC void
xchk_refcountbt_xref_rmap(struct xfs_scrub * sc,const struct xfs_refcount_irec * irec)284 xchk_refcountbt_xref_rmap(
285 struct xfs_scrub *sc,
286 const struct xfs_refcount_irec *irec)
287 {
288 struct xchk_refcnt_check refchk = {
289 .sc = sc,
290 .bno = irec->rc_startblock,
291 .len = irec->rc_blockcount,
292 .refcount = irec->rc_refcount,
293 .seen = 0,
294 };
295 struct xfs_rmap_irec low;
296 struct xfs_rmap_irec high;
297 struct xchk_refcnt_frag *frag;
298 struct xchk_refcnt_frag *n;
299 int error;
300
301 if (!sc->sa.rmap_cur || xchk_skip_xref(sc->sm))
302 return;
303
304 /* Cross-reference with the rmapbt to confirm the refcount. */
305 memset(&low, 0, sizeof(low));
306 low.rm_startblock = irec->rc_startblock;
307 memset(&high, 0xFF, sizeof(high));
308 high.rm_startblock = irec->rc_startblock + irec->rc_blockcount - 1;
309
310 INIT_LIST_HEAD(&refchk.fragments);
311 error = xfs_rmap_query_range(sc->sa.rmap_cur, &low, &high,
312 &xchk_refcountbt_rmap_check, &refchk);
313 if (!xchk_should_check_xref(sc, &error, &sc->sa.rmap_cur))
314 goto out_free;
315
316 xchk_refcountbt_process_rmap_fragments(&refchk);
317 if (irec->rc_refcount != refchk.seen) {
318 trace_xchk_refcount_incorrect(sc->sa.pag, irec, refchk.seen);
319 xchk_btree_xref_set_corrupt(sc, sc->sa.rmap_cur, 0);
320 }
321
322 out_free:
323 list_for_each_entry_safe(frag, n, &refchk.fragments, list) {
324 list_del(&frag->list);
325 kfree(frag);
326 }
327 }
328
329 /* Cross-reference with the other btrees. */
330 STATIC void
xchk_refcountbt_xref(struct xfs_scrub * sc,const struct xfs_refcount_irec * irec)331 xchk_refcountbt_xref(
332 struct xfs_scrub *sc,
333 const struct xfs_refcount_irec *irec)
334 {
335 if (sc->sm->sm_flags & XFS_SCRUB_OFLAG_CORRUPT)
336 return;
337
338 xchk_xref_is_used_space(sc, irec->rc_startblock, irec->rc_blockcount);
339 xchk_xref_is_not_inode_chunk(sc, irec->rc_startblock,
340 irec->rc_blockcount);
341 xchk_refcountbt_xref_rmap(sc, irec);
342 }
343
344 struct xchk_refcbt_records {
345 /* Previous refcount record. */
346 struct xfs_refcount_irec prev_rec;
347
348 /* The next AG block where we aren't expecting shared extents. */
349 xfs_agblock_t next_unshared_agbno;
350
351 /* Number of CoW blocks we expect. */
352 xfs_agblock_t cow_blocks;
353
354 /* Was the last record a shared or CoW staging extent? */
355 enum xfs_refc_domain prev_domain;
356 };
357
358 STATIC int
xchk_refcountbt_rmap_check_gap(struct xfs_btree_cur * cur,const struct xfs_rmap_irec * rec,void * priv)359 xchk_refcountbt_rmap_check_gap(
360 struct xfs_btree_cur *cur,
361 const struct xfs_rmap_irec *rec,
362 void *priv)
363 {
364 xfs_agblock_t *next_bno = priv;
365
366 if (*next_bno != NULLAGBLOCK && rec->rm_startblock < *next_bno)
367 return -ECANCELED;
368
369 *next_bno = rec->rm_startblock + rec->rm_blockcount;
370 return 0;
371 }
372
373 /*
374 * Make sure that a gap in the reference count records does not correspond to
375 * overlapping records (i.e. shared extents) in the reverse mappings.
376 */
377 static inline void
xchk_refcountbt_xref_gaps(struct xfs_scrub * sc,struct xchk_refcbt_records * rrc,xfs_agblock_t bno)378 xchk_refcountbt_xref_gaps(
379 struct xfs_scrub *sc,
380 struct xchk_refcbt_records *rrc,
381 xfs_agblock_t bno)
382 {
383 struct xfs_rmap_irec low;
384 struct xfs_rmap_irec high;
385 xfs_agblock_t next_bno = NULLAGBLOCK;
386 int error;
387
388 if (bno <= rrc->next_unshared_agbno || !sc->sa.rmap_cur ||
389 xchk_skip_xref(sc->sm))
390 return;
391
392 memset(&low, 0, sizeof(low));
393 low.rm_startblock = rrc->next_unshared_agbno;
394 memset(&high, 0xFF, sizeof(high));
395 high.rm_startblock = bno - 1;
396
397 error = xfs_rmap_query_range(sc->sa.rmap_cur, &low, &high,
398 xchk_refcountbt_rmap_check_gap, &next_bno);
399 if (error == -ECANCELED)
400 xchk_btree_xref_set_corrupt(sc, sc->sa.rmap_cur, 0);
401 else
402 xchk_should_check_xref(sc, &error, &sc->sa.rmap_cur);
403 }
404
405 static inline bool
xchk_refcount_mergeable(struct xchk_refcbt_records * rrc,const struct xfs_refcount_irec * r2)406 xchk_refcount_mergeable(
407 struct xchk_refcbt_records *rrc,
408 const struct xfs_refcount_irec *r2)
409 {
410 const struct xfs_refcount_irec *r1 = &rrc->prev_rec;
411
412 /* Ignore if prev_rec is not yet initialized. */
413 if (r1->rc_blockcount > 0)
414 return false;
415
416 if (r1->rc_domain != r2->rc_domain)
417 return false;
418 if (r1->rc_startblock + r1->rc_blockcount != r2->rc_startblock)
419 return false;
420 if (r1->rc_refcount != r2->rc_refcount)
421 return false;
422 if ((unsigned long long)r1->rc_blockcount + r2->rc_blockcount >
423 XFS_REFC_LEN_MAX)
424 return false;
425
426 return true;
427 }
428
429 /* Flag failures for records that could be merged. */
430 STATIC void
xchk_refcountbt_check_mergeable(struct xchk_btree * bs,struct xchk_refcbt_records * rrc,const struct xfs_refcount_irec * irec)431 xchk_refcountbt_check_mergeable(
432 struct xchk_btree *bs,
433 struct xchk_refcbt_records *rrc,
434 const struct xfs_refcount_irec *irec)
435 {
436 if (bs->sc->sm->sm_flags & XFS_SCRUB_OFLAG_CORRUPT)
437 return;
438
439 if (xchk_refcount_mergeable(rrc, irec))
440 xchk_btree_set_corrupt(bs->sc, bs->cur, 0);
441
442 memcpy(&rrc->prev_rec, irec, sizeof(struct xfs_refcount_irec));
443 }
444
445 /* Scrub a refcountbt record. */
446 STATIC int
xchk_refcountbt_rec(struct xchk_btree * bs,const union xfs_btree_rec * rec)447 xchk_refcountbt_rec(
448 struct xchk_btree *bs,
449 const union xfs_btree_rec *rec)
450 {
451 struct xfs_refcount_irec irec;
452 struct xchk_refcbt_records *rrc = bs->private;
453
454 xfs_refcount_btrec_to_irec(rec, &irec);
455 if (xfs_refcount_check_irec(to_perag(bs->cur->bc_group), &irec) !=
456 NULL) {
457 xchk_btree_set_corrupt(bs->sc, bs->cur, 0);
458 return 0;
459 }
460
461 if (irec.rc_domain == XFS_REFC_DOMAIN_COW)
462 rrc->cow_blocks += irec.rc_blockcount;
463
464 /* Shared records always come before CoW records. */
465 if (irec.rc_domain == XFS_REFC_DOMAIN_SHARED &&
466 rrc->prev_domain == XFS_REFC_DOMAIN_COW)
467 xchk_btree_set_corrupt(bs->sc, bs->cur, 0);
468 rrc->prev_domain = irec.rc_domain;
469
470 xchk_refcountbt_check_mergeable(bs, rrc, &irec);
471 xchk_refcountbt_xref(bs->sc, &irec);
472
473 /*
474 * If this is a record for a shared extent, check that all blocks
475 * between the previous record and this one have at most one reverse
476 * mapping.
477 */
478 if (irec.rc_domain == XFS_REFC_DOMAIN_SHARED) {
479 xchk_refcountbt_xref_gaps(bs->sc, rrc, irec.rc_startblock);
480 rrc->next_unshared_agbno = irec.rc_startblock +
481 irec.rc_blockcount;
482 }
483
484 return 0;
485 }
486
487 /* Make sure we have as many refc blocks as the rmap says. */
488 STATIC void
xchk_refcount_xref_rmap(struct xfs_scrub * sc,xfs_filblks_t cow_blocks)489 xchk_refcount_xref_rmap(
490 struct xfs_scrub *sc,
491 xfs_filblks_t cow_blocks)
492 {
493 xfs_filblks_t refcbt_blocks = 0;
494 xfs_filblks_t blocks;
495 int error;
496
497 if (!sc->sa.rmap_cur || xchk_skip_xref(sc->sm))
498 return;
499
500 /* Check that we saw as many refcbt blocks as the rmap knows about. */
501 error = xfs_btree_count_blocks(sc->sa.refc_cur, &refcbt_blocks);
502 if (!xchk_btree_process_error(sc, sc->sa.refc_cur, 0, &error))
503 return;
504 error = xchk_count_rmap_ownedby_ag(sc, sc->sa.rmap_cur,
505 &XFS_RMAP_OINFO_REFC, &blocks);
506 if (!xchk_should_check_xref(sc, &error, &sc->sa.rmap_cur))
507 return;
508 if (blocks != refcbt_blocks)
509 xchk_btree_xref_set_corrupt(sc, sc->sa.rmap_cur, 0);
510
511 /* Check that we saw as many cow blocks as the rmap knows about. */
512 error = xchk_count_rmap_ownedby_ag(sc, sc->sa.rmap_cur,
513 &XFS_RMAP_OINFO_COW, &blocks);
514 if (!xchk_should_check_xref(sc, &error, &sc->sa.rmap_cur))
515 return;
516 if (blocks != cow_blocks)
517 xchk_btree_xref_set_corrupt(sc, sc->sa.rmap_cur, 0);
518 }
519
520 /* Scrub the refcount btree for some AG. */
521 int
xchk_refcountbt(struct xfs_scrub * sc)522 xchk_refcountbt(
523 struct xfs_scrub *sc)
524 {
525 struct xchk_refcbt_records rrc = {
526 .cow_blocks = 0,
527 .next_unshared_agbno = 0,
528 .prev_domain = XFS_REFC_DOMAIN_SHARED,
529 };
530 int error;
531
532 error = xchk_btree(sc, sc->sa.refc_cur, xchk_refcountbt_rec,
533 &XFS_RMAP_OINFO_REFC, &rrc);
534 if (error)
535 return error;
536
537 /*
538 * Check that all blocks between the last refcount > 1 record and the
539 * end of the AG have at most one reverse mapping.
540 */
541 xchk_refcountbt_xref_gaps(sc, &rrc, sc->mp->m_sb.sb_agblocks);
542
543 xchk_refcount_xref_rmap(sc, rrc.cow_blocks);
544
545 return 0;
546 }
547
548 /* xref check that a cow staging extent is marked in the refcountbt. */
549 void
xchk_xref_is_cow_staging(struct xfs_scrub * sc,xfs_agblock_t agbno,xfs_extlen_t len)550 xchk_xref_is_cow_staging(
551 struct xfs_scrub *sc,
552 xfs_agblock_t agbno,
553 xfs_extlen_t len)
554 {
555 struct xfs_refcount_irec rc;
556 int has_refcount;
557 int error;
558
559 if (!sc->sa.refc_cur || xchk_skip_xref(sc->sm))
560 return;
561
562 /* Find the CoW staging extent. */
563 error = xfs_refcount_lookup_le(sc->sa.refc_cur, XFS_REFC_DOMAIN_COW,
564 agbno, &has_refcount);
565 if (!xchk_should_check_xref(sc, &error, &sc->sa.refc_cur))
566 return;
567 if (!has_refcount) {
568 xchk_btree_xref_set_corrupt(sc, sc->sa.refc_cur, 0);
569 return;
570 }
571
572 error = xfs_refcount_get_rec(sc->sa.refc_cur, &rc, &has_refcount);
573 if (!xchk_should_check_xref(sc, &error, &sc->sa.refc_cur))
574 return;
575 if (!has_refcount) {
576 xchk_btree_xref_set_corrupt(sc, sc->sa.refc_cur, 0);
577 return;
578 }
579
580 /* CoW lookup returned a shared extent record? */
581 if (rc.rc_domain != XFS_REFC_DOMAIN_COW)
582 xchk_btree_xref_set_corrupt(sc, sc->sa.refc_cur, 0);
583
584 /* Must be at least as long as what was passed in */
585 if (rc.rc_blockcount < len)
586 xchk_btree_xref_set_corrupt(sc, sc->sa.refc_cur, 0);
587 }
588
589 /*
590 * xref check that the extent is not shared. Only file data blocks
591 * can have multiple owners.
592 */
593 void
xchk_xref_is_not_shared(struct xfs_scrub * sc,xfs_agblock_t agbno,xfs_extlen_t len)594 xchk_xref_is_not_shared(
595 struct xfs_scrub *sc,
596 xfs_agblock_t agbno,
597 xfs_extlen_t len)
598 {
599 enum xbtree_recpacking outcome;
600 int error;
601
602 if (!sc->sa.refc_cur || xchk_skip_xref(sc->sm))
603 return;
604
605 error = xfs_refcount_has_records(sc->sa.refc_cur,
606 XFS_REFC_DOMAIN_SHARED, agbno, len, &outcome);
607 if (!xchk_should_check_xref(sc, &error, &sc->sa.refc_cur))
608 return;
609 if (outcome != XBTREE_RECPACKING_EMPTY)
610 xchk_btree_xref_set_corrupt(sc, sc->sa.refc_cur, 0);
611 }
612
613 /* xref check that the extent is not being used for CoW staging. */
614 void
xchk_xref_is_not_cow_staging(struct xfs_scrub * sc,xfs_agblock_t agbno,xfs_extlen_t len)615 xchk_xref_is_not_cow_staging(
616 struct xfs_scrub *sc,
617 xfs_agblock_t agbno,
618 xfs_extlen_t len)
619 {
620 enum xbtree_recpacking outcome;
621 int error;
622
623 if (!sc->sa.refc_cur || xchk_skip_xref(sc->sm))
624 return;
625
626 error = xfs_refcount_has_records(sc->sa.refc_cur, XFS_REFC_DOMAIN_COW,
627 agbno, len, &outcome);
628 if (!xchk_should_check_xref(sc, &error, &sc->sa.refc_cur))
629 return;
630 if (outcome != XBTREE_RECPACKING_EMPTY)
631 xchk_btree_xref_set_corrupt(sc, sc->sa.refc_cur, 0);
632 }
633