xfs: add a xfs_iext_update_extent helper

Source kernel commit: 67e4e69cb2a7afbffdefd1a0a23a94d1d706c38f

This helper is used to update an extent record based on the extent index,
and can be used to provide a level of abstractions between callers that
want to modify in-core extent records and the details of the extent list
implementation.

Also switch all users of the xfs_bmbt_set_all(xfs_iext_get_ext(...))
pattern to this new helper.

Signed-off-by: Christoph Hellwig <hch@lst.de>
Reviewed-by: Darrick J. Wong <darrick.wong@oracle.com>
Signed-off-by: Darrick J. Wong <darrick.wong@oracle.com>
Signed-off-by: Eric Sandeen <sandeen@sandeen.net>

diff --git a/libxfs/xfs_bmap.c b/libxfs/xfs_bmap.c
index a4d402a00fc3289105537baf8a58838fef25322d..ac904a6f0ab876ab1d97a26c46056afc3502ebe6 100644 (file)
--- a/libxfs/xfs_bmap.c
+++ b/libxfs/xfs_bmap.c
@@ -4909,7 +4909,7 @@ xfs_bmap_del_extent_delay(
                da_new = XFS_FILBLKS_MIN(xfs_bmap_worst_indlen(ip,
                                got->br_blockcount), da_old);
                got->br_startblock = nullstartblock((int)da_new);
-               xfs_bmbt_set_all(xfs_iext_get_ext(ifp, *idx), got);
+               xfs_iext_update_extent(ifp, *idx, got);
                trace_xfs_bmap_post_update(ip, *idx, state, _THIS_IP_);
                break;
        case BMAP_RIGHT_CONTIG:
@@ -4921,7 +4921,7 @@ xfs_bmap_del_extent_delay(
                da_new = XFS_FILBLKS_MIN(xfs_bmap_worst_indlen(ip,
                                got->br_blockcount), da_old);
                got->br_startblock = nullstartblock((int)da_new);
-               xfs_bmbt_set_all(xfs_iext_get_ext(ifp, *idx), got);
+               xfs_iext_update_extent(ifp, *idx, got);
                trace_xfs_bmap_post_update(ip, *idx, state, _THIS_IP_);
                break;
        case 0:
@@ -4947,7 +4947,7 @@ xfs_bmap_del_extent_delay(
                                                       del->br_blockcount);
 
                got->br_startblock = nullstartblock((int)got_indlen);
-               xfs_bmbt_set_all(xfs_iext_get_ext(ifp, *idx), got);
+               xfs_iext_update_extent(ifp, *idx, got);
                trace_xfs_bmap_post_update(ip, *idx, 0, _THIS_IP_);
 
                new.br_startoff = del_endoff;
@@ -5017,7 +5017,7 @@ xfs_bmap_del_extent_cow(
                got->br_startoff = del_endoff;
                got->br_blockcount -= del->br_blockcount;
                got->br_startblock = del->br_startblock + del->br_blockcount;
-               xfs_bmbt_set_all(xfs_iext_get_ext(ifp, *idx), got);
+               xfs_iext_update_extent(ifp, *idx, got);
                trace_xfs_bmap_post_update(ip, *idx, state, _THIS_IP_);
                break;
        case BMAP_RIGHT_CONTIG:
@@ -5026,7 +5026,7 @@ xfs_bmap_del_extent_cow(
                 */
                trace_xfs_bmap_pre_update(ip, *idx, state, _THIS_IP_);
                got->br_blockcount -= del->br_blockcount;
-               xfs_bmbt_set_all(xfs_iext_get_ext(ifp, *idx), got);
+               xfs_iext_update_extent(ifp, *idx, got);
                trace_xfs_bmap_post_update(ip, *idx, state, _THIS_IP_);
                break;
        case 0:
@@ -5035,7 +5035,7 @@ xfs_bmap_del_extent_cow(
                 */
                trace_xfs_bmap_pre_update(ip, *idx, state, _THIS_IP_);
                got->br_blockcount = del->br_startoff - got->br_startoff;
-               xfs_bmbt_set_all(xfs_iext_get_ext(ifp, *idx), got);
+               xfs_iext_update_extent(ifp, *idx, got);
                trace_xfs_bmap_post_update(ip, *idx, state, _THIS_IP_);
 
                new.br_startoff = del_endoff;

diff --git a/libxfs/xfs_inode_fork.c b/libxfs/xfs_inode_fork.c
index 0ef989d7f90811ae57fc38abe112c4ddec930041..c8778bec3ebd3d789f46bd696bf21f25dbf736ec 100644 (file)
--- a/libxfs/xfs_inode_fork.c
+++ b/libxfs/xfs_inode_fork.c
@@ -2022,3 +2022,15 @@ xfs_iext_get_extent(
        xfs_bmbt_get_all(xfs_iext_get_ext(ifp, idx), gotp);
        return true;
 }
+
+void
+xfs_iext_update_extent(
+       struct xfs_ifork        *ifp,
+       xfs_extnum_t            idx,
+       struct xfs_bmbt_irec    *gotp)
+{
+       ASSERT(idx >= 0);
+       ASSERT(idx < xfs_iext_count(ifp));
+
+       xfs_bmbt_set_all(xfs_iext_get_ext(ifp, idx), gotp);
+}

diff --git a/libxfs/xfs_inode_fork.h b/libxfs/xfs_inode_fork.h
index 7fb8365326d1a745583c4f133bc5a63668316b33..11af705219f6d7dacace149dd8d6b00f2a794ef3 100644 (file)
--- a/libxfs/xfs_inode_fork.h
+++ b/libxfs/xfs_inode_fork.h
@@ -187,6 +187,8 @@ bool                xfs_iext_lookup_extent(struct xfs_inode *ip,
                        xfs_extnum_t *idxp, struct xfs_bmbt_irec *gotp);
 bool           xfs_iext_get_extent(struct xfs_ifork *ifp, xfs_extnum_t idx,
                        struct xfs_bmbt_irec *gotp);
+void           xfs_iext_update_extent(struct xfs_ifork *ifp, xfs_extnum_t idx,
+                       struct xfs_bmbt_irec *gotp);
 
 extern struct kmem_zone        *xfs_ifork_zone;