Commit e17a5c6f authored by Christoph Hellwig's avatar Christoph Hellwig Committed by Darrick J. Wong
Browse files

xfs: rewrite xfs_bmap_count_leaves using xfs_iext_get_extent 



This avoids poking into the internals of the extent list.  Also return
the number of extents as the return value instead of an additional
by reference argument, and make it available to callers outside of
xfs_bmap_util.c

Signed-off-by: default avatarChristoph Hellwig <hch@lst.de>
Reviewed-by: default avatarDarrick J. Wong <darrick.wong@oracle.com>
Signed-off-by: default avatarDarrick J. Wong <darrick.wong@oracle.com>
parent 4c35445b

fs/xfs/xfs_bmap_util.c

+10 −11
Original line number Diff line number Diff line
@@ -222,22 +222,21 @@ xfs_bmap_eof( 
 * Count leaf blocks given a range of extent records.  Delayed allocation

 * extents are not counted towards the totals.

 */

STATIC void

xfs_extnum_t

xfs_bmap_count_leaves(

	struct xfs_ifork	*ifp,

	xfs_extnum_t		*numrecs,

	xfs_filblks_t		*count)

{

	xfs_extnum_t		i;

	xfs_extnum_t		nr_exts = xfs_iext_count(ifp);

	struct xfs_bmbt_irec	got;

	xfs_extnum_t		numrecs = 0, i = 0;



	for (i = 0; i < nr_exts; i++) {

		xfs_bmbt_rec_host_t *frp = xfs_iext_get_ext(ifp, i);

		if (!isnullstartblock(xfs_bmbt_get_startblock(frp))) {

			(*numrecs)++;

			*count += xfs_bmbt_get_blockcount(frp);

	while (xfs_iext_get_extent(ifp, i++, &got)) {

		if (!isnullstartblock(got.br_startblock)) {

			*count += got.br_blockcount;

			numrecs++;

		}

	}

	return numrecs;

}



/*
@@ -370,7 +369,7 @@ xfs_bmap_count_blocks( 


	switch (XFS_IFORK_FORMAT(ip, whichfork)) {

	case XFS_DINODE_FMT_EXTENTS:

		xfs_bmap_count_leaves(ifp, nextents, count);

		*nextents = xfs_bmap_count_leaves(ifp, count);

		return 0;

	case XFS_DINODE_FMT_BTREE:

		if (!(ifp->if_flags & XFS_IFEXTENTS)) {

fs/xfs/xfs_bmap_util.h

+1 −0
Original line number Diff line number Diff line
@@ -70,6 +70,7 @@ int xfs_swap_extents(struct xfs_inode *ip, struct xfs_inode *tip, 


xfs_daddr_t xfs_fsb_to_db(struct xfs_inode *ip, xfs_fsblock_t fsb);



xfs_extnum_t xfs_bmap_count_leaves(struct xfs_ifork *ifp, xfs_filblks_t *count);

int xfs_bmap_count_blocks(struct xfs_trans *tp, struct xfs_inode *ip,

			  int whichfork, xfs_extnum_t *nextents,

			  xfs_filblks_t *count);

CRA Git | Maintained and supported by SUSTech CRA and CCSE