[01/12] xfs_repair: drop lostblocks from build_agf_agfl
diff mbox series

Message ID 159107201957.315004.49440739053731951.stgit@magnolia
State New
Headers show
Series
  • xfs_repair: use btree bulk loading
Related show

Commit Message

Darrick J. Wong June 2, 2020, 4:26 a.m. UTC
From: Darrick J. Wong <darrick.wong@oracle.com>

We don't do anything with this parameter, so get rid of it.

Fixes: ef4332b8 ("xfs_repair: add freesp btree block overflow to the free space")
Signed-off-by: Darrick J. Wong <darrick.wong@oracle.com>
---
 repair/phase5.c |    7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

Comments

Brian Foster June 17, 2020, 12:09 p.m. UTC | #1
On Mon, Jun 01, 2020 at 09:26:59PM -0700, Darrick J. Wong wrote:
> From: Darrick J. Wong <darrick.wong@oracle.com>
> 
> We don't do anything with this parameter, so get rid of it.
> 
> Fixes: ef4332b8 ("xfs_repair: add freesp btree block overflow to the free space")
> Signed-off-by: Darrick J. Wong <darrick.wong@oracle.com>
> ---

Reviewed-by: Brian Foster <bfoster@redhat.com>

>  repair/phase5.c |    7 +++----
>  1 file changed, 3 insertions(+), 4 deletions(-)
> 
> 
> diff --git a/repair/phase5.c b/repair/phase5.c
> index 677297fe..c9b278bd 100644
> --- a/repair/phase5.c
> +++ b/repair/phase5.c
> @@ -2049,7 +2049,6 @@ build_agf_agfl(
>  	struct bt_status	*bno_bt,
>  	struct bt_status	*bcnt_bt,
>  	xfs_extlen_t		freeblks,	/* # free blocks in tree */
> -	int			lostblocks,	/* # blocks that will be lost */
>  	struct bt_status	*rmap_bt,
>  	struct bt_status	*refcnt_bt,
>  	struct xfs_slab		*lost_fsb)
> @@ -2465,9 +2464,9 @@ phase5_func(
>  		/*
>  		 * 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_agf_agfl(mp, agno, &bno_btree_curs, &bcnt_btree_curs,
> +				freeblks1, &rmap_btree_curs,
> +				&refcnt_btree_curs, lost_fsb);
>  		/*
>  		 * build inode allocation tree.
>  		 */
>

Patch
diff mbox series

diff --git a/repair/phase5.c b/repair/phase5.c
index 677297fe..c9b278bd 100644
--- a/repair/phase5.c
+++ b/repair/phase5.c
@@ -2049,7 +2049,6 @@  build_agf_agfl(
 	struct bt_status	*bno_bt,
 	struct bt_status	*bcnt_bt,
 	xfs_extlen_t		freeblks,	/* # free blocks in tree */
-	int			lostblocks,	/* # blocks that will be lost */
 	struct bt_status	*rmap_bt,
 	struct bt_status	*refcnt_bt,
 	struct xfs_slab		*lost_fsb)
@@ -2465,9 +2464,9 @@  phase5_func(
 		/*
 		 * 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_agf_agfl(mp, agno, &bno_btree_curs, &bcnt_btree_curs,
+				freeblks1, &rmap_btree_curs,
+				&refcnt_btree_curs, lost_fsb);
 		/*
 		 * build inode allocation tree.
 		 */