Commit 50bb44c2 authored by Christoph Hellwig's avatar Christoph Hellwig Committed by Darrick J. Wong

xfs: switch xfs_bmap_local_to_extents to use xfs_iext_insert

Use the helper instead of open coding it, to provide a better abstraction
for the scalable extent list work.  This also gets an additional assert
and trace point for free.
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 67e4e69c
...@@ -880,7 +880,7 @@ xfs_bmap_local_to_extents( ...@@ -880,7 +880,7 @@ xfs_bmap_local_to_extents(
xfs_ifork_t *ifp; /* inode fork pointer */ xfs_ifork_t *ifp; /* inode fork pointer */
xfs_alloc_arg_t args; /* allocation arguments */ xfs_alloc_arg_t args; /* allocation arguments */
xfs_buf_t *bp; /* buffer for extent block */ xfs_buf_t *bp; /* buffer for extent block */
xfs_bmbt_rec_host_t *ep; /* extent record pointer */ struct xfs_bmbt_irec rec;
/* /*
* We don't want to deal with the case of keeping inode data inline yet. * We don't want to deal with the case of keeping inode data inline yet.
...@@ -943,9 +943,12 @@ xfs_bmap_local_to_extents( ...@@ -943,9 +943,12 @@ xfs_bmap_local_to_extents(
xfs_bmap_local_to_extents_empty(ip, whichfork); xfs_bmap_local_to_extents_empty(ip, whichfork);
flags |= XFS_ILOG_CORE; flags |= XFS_ILOG_CORE;
xfs_iext_add(ifp, 0, 1); rec.br_startoff = 0;
ep = xfs_iext_get_ext(ifp, 0); rec.br_startblock = args.fsbno;
xfs_bmbt_set_allf(ep, 0, args.fsbno, 1, XFS_EXT_NORM); rec.br_blockcount = 1;
rec.br_state = XFS_EXT_NORM;
xfs_iext_insert(ip, 0, 1, &rec, 0);
trace_xfs_bmap_post_update(ip, 0, trace_xfs_bmap_post_update(ip, 0,
whichfork == XFS_ATTR_FORK ? BMAP_ATTRFORK : 0, whichfork == XFS_ATTR_FORK ? BMAP_ATTRFORK : 0,
_THIS_IP_); _THIS_IP_);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment