* [PATCH] xfs_repair: remove gratuitous code block in phase5
@ 2020-05-11 19:20 Eric Sandeen
2020-05-11 20:32 ` Eric Sandeen
2020-05-14 20:45 ` Darrick J. Wong
0 siblings, 2 replies; 4+ messages in thread
From: Eric Sandeen @ 2020-05-11 19:20 UTC (permalink / raw)
To: linux-xfs
A commit back in 2008 removed a "for" loop ahead of this code block, but
left the indented code block in place. Remove it for clarity and reflow
comments & lines as needed.
Signed-off-by: Eric Sandeen <sandeen@redhat.com>
---
(Probably easiest to review this by applying it and then doing
git show -w $COMMIT to see only the non-whitespace differences)
diff --git a/repair/phase5.c b/repair/phase5.c
index 677297fe..84c05a13 100644
--- a/repair/phase5.c
+++ b/repair/phase5.c
@@ -2313,201 +2313,186 @@ phase5_func(
if (verbose)
do_log(_(" - agno = %d\n"), agno);
- {
- /*
- * build up incore bno and bcnt extent btrees
- */
- num_extents = mk_incore_fstree(mp, agno);
+ /*
+ * build up incore bno and bcnt extent btrees
+ */
+ num_extents = mk_incore_fstree(mp, agno);
#ifdef XR_BLD_FREE_TRACE
- fprintf(stderr, "# of bno extents is %d\n",
- count_bno_extents(agno));
+ fprintf(stderr, "# of bno extents is %d\n", count_bno_extents(agno));
#endif
- if (num_extents == 0) {
- /*
- * XXX - what we probably should do here is pick an
- * inode for a regular file in the allocation group
- * that has space allocated and shoot it by traversing
- * the bmap list and putting all its extents on the
- * incore freespace trees, clearing the inode,
- * and clearing the in-use bit in the incore inode
- * tree. Then try mk_incore_fstree() again.
- */
- do_error(_("unable to rebuild AG %u. "
- "Not enough free space in on-disk AG.\n"),
- agno);
- }
-
+ if (num_extents == 0) {
/*
- * ok, now set up the btree cursors for the
- * on-disk btrees (includs pre-allocating all
- * required blocks for the trees themselves)
+ * XXX - what we probably should do here is pick an inode for
+ * a regular file in the allocation group that has space
+ * allocated and shoot it by traversing the bmap list and
+ * putting all its extents on the incore freespace trees,
+ * clearing the inode, and clearing the in-use bit in the
+ * incore inode tree. Then try mk_incore_fstree() again.
*/
- init_ino_cursor(mp, agno, &ino_btree_curs, &num_inos,
- &num_free_inos, 0);
+ do_error(
+_("unable to rebuild AG %u. Not enough free space in on-disk AG.\n"),
+ agno);
+ }
- if (xfs_sb_version_hasfinobt(&mp->m_sb))
- init_ino_cursor(mp, agno, &fino_btree_curs,
- &finobt_num_inos, &finobt_num_free_inos,
- 1);
+ /*
+ * ok, now set up the btree cursors for the on-disk btrees (includes
+ * pre-allocating all required blocks for the trees themselves)
+ */
+ init_ino_cursor(mp, agno, &ino_btree_curs, &num_inos,
+ &num_free_inos, 0);
- sb_icount_ag[agno] += num_inos;
- sb_ifree_ag[agno] += num_free_inos;
+ if (xfs_sb_version_hasfinobt(&mp->m_sb))
+ init_ino_cursor(mp, agno, &fino_btree_curs, &finobt_num_inos,
+ &finobt_num_free_inos, 1);
- /*
- * Set up the btree cursors for the on-disk rmap btrees,
- * which includes pre-allocating all required blocks.
- */
- init_rmapbt_cursor(mp, agno, &rmap_btree_curs);
+ sb_icount_ag[agno] += num_inos;
+ sb_ifree_ag[agno] += num_free_inos;
- /*
- * Set up the btree cursors for the on-disk refcount btrees,
- * which includes pre-allocating all required blocks.
- */
- init_refc_cursor(mp, agno, &refcnt_btree_curs);
+ /*
+ * Set up the btree cursors for the on-disk rmap btrees, which includes
+ * pre-allocating all required blocks.
+ */
+ init_rmapbt_cursor(mp, agno, &rmap_btree_curs);
- num_extents = count_bno_extents_blocks(agno, &num_freeblocks);
+ /*
+ * Set up the btree cursors for the on-disk refcount btrees,
+ * which includes pre-allocating all required blocks.
+ */
+ init_refc_cursor(mp, agno, &refcnt_btree_curs);
+
+ num_extents = count_bno_extents_blocks(agno, &num_freeblocks);
+ /*
+ * lose two blocks per AG -- the space tree roots are counted as
+ * allocated since the space trees always have roots
+ */
+ sb_fdblocks_ag[agno] += num_freeblocks - 2;
+
+ if (num_extents == 0) {
/*
- * lose two blocks per AG -- the space tree roots
- * are counted as allocated since the space trees
- * always have roots
+ * XXX - what we probably should do here is pick an inode for
+ * a regular file in the allocation group that has space
+ * allocated and shoot it by traversing the bmap list and
+ * putting all its extents on the incore freespace trees,
+ * clearing the inode, and clearing the in-use bit in the
+ * incore inode tree. Then try mk_incore_fstree() again.
*/
- sb_fdblocks_ag[agno] += num_freeblocks - 2;
-
- if (num_extents == 0) {
- /*
- * XXX - what we probably should do here is pick an
- * inode for a regular file in the allocation group
- * that has space allocated and shoot it by traversing
- * the bmap list and putting all its extents on the
- * incore freespace trees, clearing the inode,
- * and clearing the in-use bit in the incore inode
- * tree. Then try mk_incore_fstree() again.
- */
- do_error(
- _("unable to rebuild AG %u. No free space.\n"), agno);
- }
+ do_error(_("unable to rebuild AG %u. No free space.\n"), agno);
+ }
#ifdef XR_BLD_FREE_TRACE
- fprintf(stderr, "# of bno extents is %d\n", num_extents);
+ fprintf(stderr, "# of bno extents is %d\n", num_extents);
#endif
- /*
- * track blocks that we might really lose
- */
- extra_blocks = calculate_freespace_cursor(mp, agno,
- &num_extents, &bno_btree_curs);
+ /*
+ * track blocks that we might really lose
+ */
+ extra_blocks = calculate_freespace_cursor(mp, agno,
+ &num_extents, &bno_btree_curs);
- /*
- * freespace btrees live in the "free space" but
- * the filesystem treats AGFL blocks as allocated
- * since they aren't described by the freespace trees
- */
+ /*
+ * freespace btrees live in the "free space" but the filesystem treats
+ * AGFL blocks as allocated since they aren't described by the
+ * freespace trees
+ */
- /*
- * see if we can fit all the extra blocks into the AGFL
- */
- extra_blocks = (extra_blocks - libxfs_agfl_size(mp) > 0)
- ? extra_blocks - libxfs_agfl_size(mp)
- : 0;
+ /*
+ * see if we can fit all the extra blocks into the AGFL
+ */
+ extra_blocks = (extra_blocks - libxfs_agfl_size(mp) > 0) ?
+ extra_blocks - libxfs_agfl_size(mp) : 0;
- if (extra_blocks > 0)
- sb_fdblocks_ag[agno] -= extra_blocks;
+ if (extra_blocks > 0)
+ sb_fdblocks_ag[agno] -= extra_blocks;
- bcnt_btree_curs = bno_btree_curs;
+ bcnt_btree_curs = bno_btree_curs;
- bno_btree_curs.owner = XFS_RMAP_OWN_AG;
- bcnt_btree_curs.owner = XFS_RMAP_OWN_AG;
- setup_cursor(mp, agno, &bno_btree_curs);
- setup_cursor(mp, agno, &bcnt_btree_curs);
+ bno_btree_curs.owner = XFS_RMAP_OWN_AG;
+ bcnt_btree_curs.owner = XFS_RMAP_OWN_AG;
+ setup_cursor(mp, agno, &bno_btree_curs);
+ setup_cursor(mp, agno, &bcnt_btree_curs);
#ifdef XR_BLD_FREE_TRACE
- fprintf(stderr, "# of bno extents is %d\n",
- count_bno_extents(agno));
- fprintf(stderr, "# of bcnt extents is %d\n",
- count_bcnt_extents(agno));
+ fprintf(stderr, "# of bno extents is %d\n", count_bno_extents(agno));
+ fprintf(stderr, "# of bcnt extents is %d\n", count_bcnt_extents(agno));
#endif
- /*
- * now rebuild the freespace trees
- */
- freeblks1 = build_freespace_tree(mp, agno,
+ /*
+ * now rebuild the freespace trees
+ */
+ freeblks1 = build_freespace_tree(mp, agno,
&bno_btree_curs, XFS_BTNUM_BNO);
#ifdef XR_BLD_FREE_TRACE
- fprintf(stderr, "# of free blocks == %d\n", freeblks1);
+ fprintf(stderr, "# of free blocks == %d\n", freeblks1);
#endif
- write_cursor(&bno_btree_curs);
+ write_cursor(&bno_btree_curs);
#ifdef DEBUG
- freeblks2 = build_freespace_tree(mp, agno,
- &bcnt_btree_curs, XFS_BTNUM_CNT);
+ freeblks2 = build_freespace_tree(mp, agno,
+ &bcnt_btree_curs, XFS_BTNUM_CNT);
#else
- (void) build_freespace_tree(mp, agno,
- &bcnt_btree_curs, XFS_BTNUM_CNT);
+ (void) build_freespace_tree(mp, agno, &bcnt_btree_curs, XFS_BTNUM_CNT);
#endif
- write_cursor(&bcnt_btree_curs);
+ write_cursor(&bcnt_btree_curs);
- ASSERT(freeblks1 == freeblks2);
+ ASSERT(freeblks1 == freeblks2);
- if (xfs_sb_version_hasrmapbt(&mp->m_sb)) {
- build_rmap_tree(mp, agno, &rmap_btree_curs);
- write_cursor(&rmap_btree_curs);
- sb_fdblocks_ag[agno] += (rmap_btree_curs.num_tot_blocks -
- rmap_btree_curs.num_free_blocks) - 1;
- }
+ if (xfs_sb_version_hasrmapbt(&mp->m_sb)) {
+ build_rmap_tree(mp, agno, &rmap_btree_curs);
+ write_cursor(&rmap_btree_curs);
+ sb_fdblocks_ag[agno] += (rmap_btree_curs.num_tot_blocks -
+ rmap_btree_curs.num_free_blocks) - 1;
+ }
- if (xfs_sb_version_hasreflink(&mp->m_sb)) {
- build_refcount_tree(mp, agno, &refcnt_btree_curs);
- write_cursor(&refcnt_btree_curs);
- }
+ if (xfs_sb_version_hasreflink(&mp->m_sb)) {
+ build_refcount_tree(mp, agno, &refcnt_btree_curs);
+ write_cursor(&refcnt_btree_curs);
+ }
- /*
- * set up agf and agfl
- */
- build_agf_agfl(mp, agno, &bno_btree_curs,
- &bcnt_btree_curs, freeblks1, extra_blocks,
- &rmap_btree_curs, &refcnt_btree_curs, lost_fsb);
- /*
- * build inode allocation tree.
- */
- build_ino_tree(mp, agno, &ino_btree_curs, XFS_BTNUM_INO,
- &agi_stat);
- write_cursor(&ino_btree_curs);
+ /*
+ * set up agf and agfl
+ */
+ build_agf_agfl(mp, agno, &bno_btree_curs,
+ &bcnt_btree_curs, freeblks1, extra_blocks,
+ &rmap_btree_curs, &refcnt_btree_curs, lost_fsb);
+ /*
+ * build inode allocation tree.
+ */
+ build_ino_tree(mp, agno, &ino_btree_curs, XFS_BTNUM_INO, &agi_stat);
+ write_cursor(&ino_btree_curs);
- /*
- * build free inode tree
- */
- if (xfs_sb_version_hasfinobt(&mp->m_sb)) {
- build_ino_tree(mp, agno, &fino_btree_curs,
- XFS_BTNUM_FINO, NULL);
- write_cursor(&fino_btree_curs);
- }
+ /*
+ * build free inode tree
+ */
+ if (xfs_sb_version_hasfinobt(&mp->m_sb)) {
+ build_ino_tree(mp, agno, &fino_btree_curs,
+ XFS_BTNUM_FINO, NULL);
+ write_cursor(&fino_btree_curs);
+ }
- /* build the agi */
- build_agi(mp, agno, &ino_btree_curs, &fino_btree_curs,
- &agi_stat);
+ /* build the agi */
+ build_agi(mp, agno, &ino_btree_curs, &fino_btree_curs, &agi_stat);
- /*
- * tear down cursors
- */
- finish_cursor(&bno_btree_curs);
- finish_cursor(&ino_btree_curs);
- if (xfs_sb_version_hasrmapbt(&mp->m_sb))
- finish_cursor(&rmap_btree_curs);
- if (xfs_sb_version_hasreflink(&mp->m_sb))
- finish_cursor(&refcnt_btree_curs);
- if (xfs_sb_version_hasfinobt(&mp->m_sb))
- finish_cursor(&fino_btree_curs);
- finish_cursor(&bcnt_btree_curs);
+ /*
+ * tear down cursors
+ */
+ finish_cursor(&bno_btree_curs);
+ finish_cursor(&ino_btree_curs);
+ if (xfs_sb_version_hasrmapbt(&mp->m_sb))
+ finish_cursor(&rmap_btree_curs);
+ if (xfs_sb_version_hasreflink(&mp->m_sb))
+ finish_cursor(&refcnt_btree_curs);
+ if (xfs_sb_version_hasfinobt(&mp->m_sb))
+ finish_cursor(&fino_btree_curs);
+ finish_cursor(&bcnt_btree_curs);
- /*
- * release the incore per-AG bno/bcnt trees so
- * the extent nodes can be recycled
- */
- release_agbno_extent_tree(agno);
- release_agbcnt_extent_tree(agno);
- }
+ /*
+ * release the incore per-AG bno/bcnt trees so the extent nodes
+ * can be recycled
+ */
+ release_agbno_extent_tree(agno);
+ release_agbcnt_extent_tree(agno);
PROG_RPT_INC(prog_rpt_done[agno], 1);
}
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] xfs_repair: remove gratuitous code block in phase5
2020-05-11 19:20 [PATCH] xfs_repair: remove gratuitous code block in phase5 Eric Sandeen
@ 2020-05-11 20:32 ` Eric Sandeen
2020-05-14 19:23 ` Darrick J. Wong
2020-05-14 20:45 ` Darrick J. Wong
1 sibling, 1 reply; 4+ messages in thread
From: Eric Sandeen @ 2020-05-11 20:32 UTC (permalink / raw)
To: linux-xfs
On 5/11/20 2:20 PM, Eric Sandeen wrote:
> A commit back in 2008 removed a "for" loop ahead of this code block, but
> left the indented code block in place. Remove it for clarity and reflow
> comments & lines as needed.
>
> Signed-off-by: Eric Sandeen <sandeen@redhat.com>
Never mind, Darrick /just/ did this in his other series. Weird. I didn't
see it there! ;)
So ignore this patch.
-Eric
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] xfs_repair: remove gratuitous code block in phase5
2020-05-11 20:32 ` Eric Sandeen
@ 2020-05-14 19:23 ` Darrick J. Wong
0 siblings, 0 replies; 4+ messages in thread
From: Darrick J. Wong @ 2020-05-14 19:23 UTC (permalink / raw)
To: Eric Sandeen; +Cc: linux-xfs
On Mon, May 11, 2020 at 03:32:42PM -0500, Eric Sandeen wrote:
> On 5/11/20 2:20 PM, Eric Sandeen wrote:
> > A commit back in 2008 removed a "for" loop ahead of this code block, but
> > left the indented code block in place. Remove it for clarity and reflow
> > comments & lines as needed.
> >
> > Signed-off-by: Eric Sandeen <sandeen@redhat.com>
>
> Never mind, Darrick /just/ did this in his other series. Weird. I didn't
> see it there! ;)
>
> So ignore this patch.
Actually, I see that you fixed all the 80col stuff that I didn't.
Maybe I want this patch over yours after all. :P
--D
> -Eric
>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] xfs_repair: remove gratuitous code block in phase5
2020-05-11 19:20 [PATCH] xfs_repair: remove gratuitous code block in phase5 Eric Sandeen
2020-05-11 20:32 ` Eric Sandeen
@ 2020-05-14 20:45 ` Darrick J. Wong
1 sibling, 0 replies; 4+ messages in thread
From: Darrick J. Wong @ 2020-05-14 20:45 UTC (permalink / raw)
To: Eric Sandeen; +Cc: linux-xfs
On Mon, May 11, 2020 at 02:20:56PM -0500, Eric Sandeen wrote:
> A commit back in 2008 removed a "for" loop ahead of this code block, but
> left the indented code block in place. Remove it for clarity and reflow
> comments & lines as needed.
>
> Signed-off-by: Eric Sandeen <sandeen@redhat.com>
Uh, so you reflowed all the comments and whatnot whereas I didn't, so:
Reviewed-by: Darrick J. Wong <darrick.wong@oracle.com>
--D
> ---
>
> (Probably easiest to review this by applying it and then doing
> git show -w $COMMIT to see only the non-whitespace differences)
>
> diff --git a/repair/phase5.c b/repair/phase5.c
> index 677297fe..84c05a13 100644
> --- a/repair/phase5.c
> +++ b/repair/phase5.c
> @@ -2313,201 +2313,186 @@ phase5_func(
> if (verbose)
> do_log(_(" - agno = %d\n"), agno);
>
> - {
> - /*
> - * build up incore bno and bcnt extent btrees
> - */
> - num_extents = mk_incore_fstree(mp, agno);
> + /*
> + * build up incore bno and bcnt extent btrees
> + */
> + num_extents = mk_incore_fstree(mp, agno);
>
> #ifdef XR_BLD_FREE_TRACE
> - fprintf(stderr, "# of bno extents is %d\n",
> - count_bno_extents(agno));
> + fprintf(stderr, "# of bno extents is %d\n", count_bno_extents(agno));
> #endif
>
> - if (num_extents == 0) {
> - /*
> - * XXX - what we probably should do here is pick an
> - * inode for a regular file in the allocation group
> - * that has space allocated and shoot it by traversing
> - * the bmap list and putting all its extents on the
> - * incore freespace trees, clearing the inode,
> - * and clearing the in-use bit in the incore inode
> - * tree. Then try mk_incore_fstree() again.
> - */
> - do_error(_("unable to rebuild AG %u. "
> - "Not enough free space in on-disk AG.\n"),
> - agno);
> - }
> -
> + if (num_extents == 0) {
> /*
> - * ok, now set up the btree cursors for the
> - * on-disk btrees (includs pre-allocating all
> - * required blocks for the trees themselves)
> + * XXX - what we probably should do here is pick an inode for
> + * a regular file in the allocation group that has space
> + * allocated and shoot it by traversing the bmap list and
> + * putting all its extents on the incore freespace trees,
> + * clearing the inode, and clearing the in-use bit in the
> + * incore inode tree. Then try mk_incore_fstree() again.
> */
> - init_ino_cursor(mp, agno, &ino_btree_curs, &num_inos,
> - &num_free_inos, 0);
> + do_error(
> +_("unable to rebuild AG %u. Not enough free space in on-disk AG.\n"),
> + agno);
> + }
>
> - if (xfs_sb_version_hasfinobt(&mp->m_sb))
> - init_ino_cursor(mp, agno, &fino_btree_curs,
> - &finobt_num_inos, &finobt_num_free_inos,
> - 1);
> + /*
> + * ok, now set up the btree cursors for the on-disk btrees (includes
> + * pre-allocating all required blocks for the trees themselves)
> + */
> + init_ino_cursor(mp, agno, &ino_btree_curs, &num_inos,
> + &num_free_inos, 0);
>
> - sb_icount_ag[agno] += num_inos;
> - sb_ifree_ag[agno] += num_free_inos;
> + if (xfs_sb_version_hasfinobt(&mp->m_sb))
> + init_ino_cursor(mp, agno, &fino_btree_curs, &finobt_num_inos,
> + &finobt_num_free_inos, 1);
>
> - /*
> - * Set up the btree cursors for the on-disk rmap btrees,
> - * which includes pre-allocating all required blocks.
> - */
> - init_rmapbt_cursor(mp, agno, &rmap_btree_curs);
> + sb_icount_ag[agno] += num_inos;
> + sb_ifree_ag[agno] += num_free_inos;
>
> - /*
> - * Set up the btree cursors for the on-disk refcount btrees,
> - * which includes pre-allocating all required blocks.
> - */
> - init_refc_cursor(mp, agno, &refcnt_btree_curs);
> + /*
> + * Set up the btree cursors for the on-disk rmap btrees, which includes
> + * pre-allocating all required blocks.
> + */
> + init_rmapbt_cursor(mp, agno, &rmap_btree_curs);
>
> - num_extents = count_bno_extents_blocks(agno, &num_freeblocks);
> + /*
> + * Set up the btree cursors for the on-disk refcount btrees,
> + * which includes pre-allocating all required blocks.
> + */
> + init_refc_cursor(mp, agno, &refcnt_btree_curs);
> +
> + num_extents = count_bno_extents_blocks(agno, &num_freeblocks);
> + /*
> + * lose two blocks per AG -- the space tree roots are counted as
> + * allocated since the space trees always have roots
> + */
> + sb_fdblocks_ag[agno] += num_freeblocks - 2;
> +
> + if (num_extents == 0) {
> /*
> - * lose two blocks per AG -- the space tree roots
> - * are counted as allocated since the space trees
> - * always have roots
> + * XXX - what we probably should do here is pick an inode for
> + * a regular file in the allocation group that has space
> + * allocated and shoot it by traversing the bmap list and
> + * putting all its extents on the incore freespace trees,
> + * clearing the inode, and clearing the in-use bit in the
> + * incore inode tree. Then try mk_incore_fstree() again.
> */
> - sb_fdblocks_ag[agno] += num_freeblocks - 2;
> -
> - if (num_extents == 0) {
> - /*
> - * XXX - what we probably should do here is pick an
> - * inode for a regular file in the allocation group
> - * that has space allocated and shoot it by traversing
> - * the bmap list and putting all its extents on the
> - * incore freespace trees, clearing the inode,
> - * and clearing the in-use bit in the incore inode
> - * tree. Then try mk_incore_fstree() again.
> - */
> - do_error(
> - _("unable to rebuild AG %u. No free space.\n"), agno);
> - }
> + do_error(_("unable to rebuild AG %u. No free space.\n"), agno);
> + }
>
> #ifdef XR_BLD_FREE_TRACE
> - fprintf(stderr, "# of bno extents is %d\n", num_extents);
> + fprintf(stderr, "# of bno extents is %d\n", num_extents);
> #endif
>
> - /*
> - * track blocks that we might really lose
> - */
> - extra_blocks = calculate_freespace_cursor(mp, agno,
> - &num_extents, &bno_btree_curs);
> + /*
> + * track blocks that we might really lose
> + */
> + extra_blocks = calculate_freespace_cursor(mp, agno,
> + &num_extents, &bno_btree_curs);
>
> - /*
> - * freespace btrees live in the "free space" but
> - * the filesystem treats AGFL blocks as allocated
> - * since they aren't described by the freespace trees
> - */
> + /*
> + * freespace btrees live in the "free space" but the filesystem treats
> + * AGFL blocks as allocated since they aren't described by the
> + * freespace trees
> + */
>
> - /*
> - * see if we can fit all the extra blocks into the AGFL
> - */
> - extra_blocks = (extra_blocks - libxfs_agfl_size(mp) > 0)
> - ? extra_blocks - libxfs_agfl_size(mp)
> - : 0;
> + /*
> + * see if we can fit all the extra blocks into the AGFL
> + */
> + extra_blocks = (extra_blocks - libxfs_agfl_size(mp) > 0) ?
> + extra_blocks - libxfs_agfl_size(mp) : 0;
>
> - if (extra_blocks > 0)
> - sb_fdblocks_ag[agno] -= extra_blocks;
> + if (extra_blocks > 0)
> + sb_fdblocks_ag[agno] -= extra_blocks;
>
> - bcnt_btree_curs = bno_btree_curs;
> + bcnt_btree_curs = bno_btree_curs;
>
> - bno_btree_curs.owner = XFS_RMAP_OWN_AG;
> - bcnt_btree_curs.owner = XFS_RMAP_OWN_AG;
> - setup_cursor(mp, agno, &bno_btree_curs);
> - setup_cursor(mp, agno, &bcnt_btree_curs);
> + bno_btree_curs.owner = XFS_RMAP_OWN_AG;
> + bcnt_btree_curs.owner = XFS_RMAP_OWN_AG;
> + setup_cursor(mp, agno, &bno_btree_curs);
> + setup_cursor(mp, agno, &bcnt_btree_curs);
>
> #ifdef XR_BLD_FREE_TRACE
> - fprintf(stderr, "# of bno extents is %d\n",
> - count_bno_extents(agno));
> - fprintf(stderr, "# of bcnt extents is %d\n",
> - count_bcnt_extents(agno));
> + fprintf(stderr, "# of bno extents is %d\n", count_bno_extents(agno));
> + fprintf(stderr, "# of bcnt extents is %d\n", count_bcnt_extents(agno));
> #endif
>
> - /*
> - * now rebuild the freespace trees
> - */
> - freeblks1 = build_freespace_tree(mp, agno,
> + /*
> + * now rebuild the freespace trees
> + */
> + freeblks1 = build_freespace_tree(mp, agno,
> &bno_btree_curs, XFS_BTNUM_BNO);
> #ifdef XR_BLD_FREE_TRACE
> - fprintf(stderr, "# of free blocks == %d\n", freeblks1);
> + fprintf(stderr, "# of free blocks == %d\n", freeblks1);
> #endif
> - write_cursor(&bno_btree_curs);
> + write_cursor(&bno_btree_curs);
>
> #ifdef DEBUG
> - freeblks2 = build_freespace_tree(mp, agno,
> - &bcnt_btree_curs, XFS_BTNUM_CNT);
> + freeblks2 = build_freespace_tree(mp, agno,
> + &bcnt_btree_curs, XFS_BTNUM_CNT);
> #else
> - (void) build_freespace_tree(mp, agno,
> - &bcnt_btree_curs, XFS_BTNUM_CNT);
> + (void) build_freespace_tree(mp, agno, &bcnt_btree_curs, XFS_BTNUM_CNT);
> #endif
> - write_cursor(&bcnt_btree_curs);
> + write_cursor(&bcnt_btree_curs);
>
> - ASSERT(freeblks1 == freeblks2);
> + ASSERT(freeblks1 == freeblks2);
>
> - if (xfs_sb_version_hasrmapbt(&mp->m_sb)) {
> - build_rmap_tree(mp, agno, &rmap_btree_curs);
> - write_cursor(&rmap_btree_curs);
> - sb_fdblocks_ag[agno] += (rmap_btree_curs.num_tot_blocks -
> - rmap_btree_curs.num_free_blocks) - 1;
> - }
> + if (xfs_sb_version_hasrmapbt(&mp->m_sb)) {
> + build_rmap_tree(mp, agno, &rmap_btree_curs);
> + write_cursor(&rmap_btree_curs);
> + sb_fdblocks_ag[agno] += (rmap_btree_curs.num_tot_blocks -
> + rmap_btree_curs.num_free_blocks) - 1;
> + }
>
> - if (xfs_sb_version_hasreflink(&mp->m_sb)) {
> - build_refcount_tree(mp, agno, &refcnt_btree_curs);
> - write_cursor(&refcnt_btree_curs);
> - }
> + if (xfs_sb_version_hasreflink(&mp->m_sb)) {
> + build_refcount_tree(mp, agno, &refcnt_btree_curs);
> + write_cursor(&refcnt_btree_curs);
> + }
>
> - /*
> - * set up agf and agfl
> - */
> - build_agf_agfl(mp, agno, &bno_btree_curs,
> - &bcnt_btree_curs, freeblks1, extra_blocks,
> - &rmap_btree_curs, &refcnt_btree_curs, lost_fsb);
> - /*
> - * build inode allocation tree.
> - */
> - build_ino_tree(mp, agno, &ino_btree_curs, XFS_BTNUM_INO,
> - &agi_stat);
> - write_cursor(&ino_btree_curs);
> + /*
> + * set up agf and agfl
> + */
> + build_agf_agfl(mp, agno, &bno_btree_curs,
> + &bcnt_btree_curs, freeblks1, extra_blocks,
> + &rmap_btree_curs, &refcnt_btree_curs, lost_fsb);
> + /*
> + * build inode allocation tree.
> + */
> + build_ino_tree(mp, agno, &ino_btree_curs, XFS_BTNUM_INO, &agi_stat);
> + write_cursor(&ino_btree_curs);
>
> - /*
> - * build free inode tree
> - */
> - if (xfs_sb_version_hasfinobt(&mp->m_sb)) {
> - build_ino_tree(mp, agno, &fino_btree_curs,
> - XFS_BTNUM_FINO, NULL);
> - write_cursor(&fino_btree_curs);
> - }
> + /*
> + * build free inode tree
> + */
> + if (xfs_sb_version_hasfinobt(&mp->m_sb)) {
> + build_ino_tree(mp, agno, &fino_btree_curs,
> + XFS_BTNUM_FINO, NULL);
> + write_cursor(&fino_btree_curs);
> + }
>
> - /* build the agi */
> - build_agi(mp, agno, &ino_btree_curs, &fino_btree_curs,
> - &agi_stat);
> + /* build the agi */
> + build_agi(mp, agno, &ino_btree_curs, &fino_btree_curs, &agi_stat);
>
> - /*
> - * tear down cursors
> - */
> - finish_cursor(&bno_btree_curs);
> - finish_cursor(&ino_btree_curs);
> - if (xfs_sb_version_hasrmapbt(&mp->m_sb))
> - finish_cursor(&rmap_btree_curs);
> - if (xfs_sb_version_hasreflink(&mp->m_sb))
> - finish_cursor(&refcnt_btree_curs);
> - if (xfs_sb_version_hasfinobt(&mp->m_sb))
> - finish_cursor(&fino_btree_curs);
> - finish_cursor(&bcnt_btree_curs);
> + /*
> + * tear down cursors
> + */
> + finish_cursor(&bno_btree_curs);
> + finish_cursor(&ino_btree_curs);
> + if (xfs_sb_version_hasrmapbt(&mp->m_sb))
> + finish_cursor(&rmap_btree_curs);
> + if (xfs_sb_version_hasreflink(&mp->m_sb))
> + finish_cursor(&refcnt_btree_curs);
> + if (xfs_sb_version_hasfinobt(&mp->m_sb))
> + finish_cursor(&fino_btree_curs);
> + finish_cursor(&bcnt_btree_curs);
>
> - /*
> - * release the incore per-AG bno/bcnt trees so
> - * the extent nodes can be recycled
> - */
> - release_agbno_extent_tree(agno);
> - release_agbcnt_extent_tree(agno);
> - }
> + /*
> + * release the incore per-AG bno/bcnt trees so the extent nodes
> + * can be recycled
> + */
> + release_agbno_extent_tree(agno);
> + release_agbcnt_extent_tree(agno);
> PROG_RPT_INC(prog_rpt_done[agno], 1);
> }
>
>
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2020-05-14 20:47 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-05-11 19:20 [PATCH] xfs_repair: remove gratuitous code block in phase5 Eric Sandeen
2020-05-11 20:32 ` Eric Sandeen
2020-05-14 19:23 ` Darrick J. Wong
2020-05-14 20:45 ` Darrick J. Wong
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.