xfs: simplify rt bitmap/summary block accessor functions

Source kernel commit: e2cf427c91494ea0d1173a911090c39665c5fdef

Simplify the calling convention of these functions since the
xfs_rtalloc_args structure contains the parameters we need.

Signed-off-by: Darrick J. Wong <djwong@kernel.org>
Reviewed-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Carlos Maiolino <cem@kernel.org>

diff --git a/libxfs/xfs_rtbitmap.c b/libxfs/xfs_rtbitmap.c
index bc3ce7bce29cfb9f84648ceaac1554512f33f612..f5d7e14abf91206b715f03fbee5c536fd59e1a18 100644 (file)
--- a/libxfs/xfs_rtbitmap.c
+++ b/libxfs/xfs_rtbitmap.c
@@ -171,7 +171,7 @@ xfs_rtfind_back(
         * Compute match value, based on the bit at start: if 1 (free)
         * then all-ones, else all-zeroes.
         */
-       incore = xfs_rtbitmap_getword(args->rbmbp, word);
+       incore = xfs_rtbitmap_getword(args, word);
        want = (incore & ((xfs_rtword_t)1 << bit)) ? -1 : 0;
        /*
         * If the starting position is not word-aligned, deal with the
@@ -226,7 +226,7 @@ xfs_rtfind_back(
                /*
                 * Compute difference between actual and desired value.
                 */
-               incore = xfs_rtbitmap_getword(args->rbmbp, word);
+               incore = xfs_rtbitmap_getword(args, word);
                if ((wdiff = incore ^ want)) {
                        /*
                         * Different, mark where we are and return.
@@ -265,7 +265,7 @@ xfs_rtfind_back(
                /*
                 * Compute difference between actual and desired value.
                 */
-               incore = xfs_rtbitmap_getword(args->rbmbp, word);
+               incore = xfs_rtbitmap_getword(args, word);
                if ((wdiff = (incore ^ want) & mask)) {
                        /*
                         * Different, mark where we are and return.
@@ -325,7 +325,7 @@ xfs_rtfind_forw(
         * Compute match value, based on the bit at start: if 1 (free)
         * then all-ones, else all-zeroes.
         */
-       incore = xfs_rtbitmap_getword(args->rbmbp, word);
+       incore = xfs_rtbitmap_getword(args, word);
        want = (incore & ((xfs_rtword_t)1 << bit)) ? -1 : 0;
        /*
         * If the starting position is not word-aligned, deal with the
@@ -379,7 +379,7 @@ xfs_rtfind_forw(
                /*
                 * Compute difference between actual and desired value.
                 */
-               incore = xfs_rtbitmap_getword(args->rbmbp, word);
+               incore = xfs_rtbitmap_getword(args, word);
                if ((wdiff = incore ^ want)) {
                        /*
                         * Different, mark where we are and return.
@@ -416,7 +416,7 @@ xfs_rtfind_forw(
                /*
                 * Compute difference between actual and desired value.
                 */
-               incore = xfs_rtbitmap_getword(args->rbmbp, word);
+               incore = xfs_rtbitmap_getword(args, word);
                if ((wdiff = (incore ^ want) & mask)) {
                        /*
                         * Different, mark where we are and return.
@@ -437,16 +437,16 @@ xfs_rtfind_forw(
 /* Log rtsummary counter at @infoword. */
 static inline void
 xfs_trans_log_rtsummary(
-       struct xfs_trans        *tp,
-       struct xfs_buf          *bp,
+       struct xfs_rtalloc_args *args,
        unsigned int            infoword)
 {
+       struct xfs_buf          *bp = args->sumbp;
        size_t                  first, last;
 
-       first = (void *)xfs_rsumblock_infoptr(bp, infoword) - bp->b_addr;
+       first = (void *)xfs_rsumblock_infoptr(args, infoword) - bp->b_addr;
        last = first + sizeof(xfs_suminfo_t) - 1;
 
-       xfs_trans_log_buf(tp, bp, first, last);
+       xfs_trans_log_buf(args->tp, bp, first, last);
 }
 
 /*
@@ -490,8 +490,7 @@ xfs_rtmodify_summary_int(
         */
        infoword = xfs_rtsumoffs_to_infoword(mp, so);
        if (delta) {
-               xfs_suminfo_t   val = xfs_suminfo_add(args->sumbp, infoword,
-                                                     delta);
+               xfs_suminfo_t   val = xfs_suminfo_add(args, infoword, delta);
 
                if (mp->m_rsum_cache) {
                        if (val == 0 && log == mp->m_rsum_cache[bbno])
@@ -499,11 +498,11 @@ xfs_rtmodify_summary_int(
                        if (val != 0 && log < mp->m_rsum_cache[bbno])
                                mp->m_rsum_cache[bbno] = log;
                }
-               xfs_trans_log_rtsummary(args->tp, args->sumbp, infoword);
+               xfs_trans_log_rtsummary(args, infoword);
                if (sum)
                        *sum = val;
        } else if (sum) {
-               *sum = xfs_suminfo_get(args->sumbp, infoword);
+               *sum = xfs_suminfo_get(args, infoword);
        }
        return 0;
 }
@@ -521,17 +520,17 @@ xfs_rtmodify_summary(
 /* Log rtbitmap block from the word @from to the byte before @next. */
 static inline void
 xfs_trans_log_rtbitmap(
-       struct xfs_trans        *tp,
-       struct xfs_buf          *bp,
+       struct xfs_rtalloc_args *args,
        unsigned int            from,
        unsigned int            next)
 {
+       struct xfs_buf          *bp = args->rbmbp;
        size_t                  first, last;
 
-       first = (void *)xfs_rbmblock_wordptr(bp, from) - bp->b_addr;
-       last = ((void *)xfs_rbmblock_wordptr(bp, next) - 1) - bp->b_addr;
+       first = (void *)xfs_rbmblock_wordptr(args, from) - bp->b_addr;
+       last = ((void *)xfs_rbmblock_wordptr(args, next) - 1) - bp->b_addr;
 
-       xfs_trans_log_buf(tp, bp, first, last);
+       xfs_trans_log_buf(args->tp, bp, first, last);
 }
 
 /*
@@ -589,12 +588,12 @@ xfs_rtmodify_range(
                /*
                 * Set/clear the active bits.
                 */
-               incore = xfs_rtbitmap_getword(args->rbmbp, word);
+               incore = xfs_rtbitmap_getword(args, word);
                if (val)
                        incore |= mask;
                else
                        incore &= ~mask;
-               xfs_rtbitmap_setword(args->rbmbp, word, incore);
+               xfs_rtbitmap_setword(args, word, incore);
                i = lastbit - bit;
                /*
                 * Go on to the next block if that's where the next word is
@@ -605,8 +604,7 @@ xfs_rtmodify_range(
                         * Log the changed part of this block.
                         * Get the next one.
                         */
-                       xfs_trans_log_rtbitmap(args->tp, args->rbmbp, firstword,
-                                       word);
+                       xfs_trans_log_rtbitmap(args, firstword, word);
                        error = xfs_rtbitmap_read_buf(args, ++block);
                        if (error)
                                return error;
@@ -627,7 +625,7 @@ xfs_rtmodify_range(
                /*
                 * Set the word value correctly.
                 */
-               xfs_rtbitmap_setword(args->rbmbp, word, val);
+               xfs_rtbitmap_setword(args, word, val);
                i += XFS_NBWORD;
                /*
                 * Go on to the next block if that's where the next word is
@@ -638,8 +636,7 @@ xfs_rtmodify_range(
                         * Log the changed part of this block.
                         * Get the next one.
                         */
-                       xfs_trans_log_rtbitmap(args->tp, args->rbmbp, firstword,
-                                       word);
+                       xfs_trans_log_rtbitmap(args, firstword, word);
                        error = xfs_rtbitmap_read_buf(args, ++block);
                        if (error)
                                return error;
@@ -659,19 +656,19 @@ xfs_rtmodify_range(
                /*
                 * Set/clear the active bits.
                 */
-               incore = xfs_rtbitmap_getword(args->rbmbp, word);
+               incore = xfs_rtbitmap_getword(args, word);
                if (val)
                        incore |= mask;
                else
                        incore &= ~mask;
-               xfs_rtbitmap_setword(args->rbmbp, word, incore);
+               xfs_rtbitmap_setword(args, word, incore);
                word++;
        }
        /*
         * Log any remaining changed bytes.
         */
        if (word > firstword)
-               xfs_trans_log_rtbitmap(args->tp, args->rbmbp, firstword, word);
+               xfs_trans_log_rtbitmap(args, firstword, word);
        return 0;
 }
 
@@ -808,7 +805,7 @@ xfs_rtcheck_range(
                /*
                 * Compute difference between actual and desired value.
                 */
-               incore = xfs_rtbitmap_getword(args->rbmbp, word);
+               incore = xfs_rtbitmap_getword(args, word);
                if ((wdiff = (incore ^ val) & mask)) {
                        /*
                         * Different, compute first wrong bit and return.
@@ -847,7 +844,7 @@ xfs_rtcheck_range(
                /*
                 * Compute difference between actual and desired value.
                 */
-               incore = xfs_rtbitmap_getword(args->rbmbp, word);
+               incore = xfs_rtbitmap_getword(args, word);
                if ((wdiff = incore ^ val)) {
                        /*
                         * Different, compute first wrong bit and return.
@@ -885,7 +882,7 @@ xfs_rtcheck_range(
                /*
                 * Compute difference between actual and desired value.
                 */
-               incore = xfs_rtbitmap_getword(args->rbmbp, word);
+               incore = xfs_rtbitmap_getword(args, word);
                if ((wdiff = (incore ^ val) & mask)) {
                        /*
                         * Different, compute first wrong bit and return.

diff --git a/libxfs/xfs_rtbitmap.h b/libxfs/xfs_rtbitmap.h
index b5505da7aa58ae6421dee7d11b2595571d30535d..db2f8c924b05031b35cac9a8a6642fdc3f6e06b0 100644 (file)
--- a/libxfs/xfs_rtbitmap.h
+++ b/libxfs/xfs_rtbitmap.h
@@ -175,10 +175,10 @@ xfs_rbmblock_to_rtx(
 /* Return a pointer to a bitmap word within a rt bitmap block. */
 static inline union xfs_rtword_raw *
 xfs_rbmblock_wordptr(
-       struct xfs_buf          *bp,
+       struct xfs_rtalloc_args *args,
        unsigned int            index)
 {
-       union xfs_rtword_raw    *words = bp->b_addr;
+       union xfs_rtword_raw    *words = args->rbmbp->b_addr;
 
        return words + index;
 }
@@ -186,10 +186,10 @@ xfs_rbmblock_wordptr(
 /* Convert an ondisk bitmap word to its incore representation. */
 static inline xfs_rtword_t
 xfs_rtbitmap_getword(
-       struct xfs_buf          *bp,
+       struct xfs_rtalloc_args *args,
        unsigned int            index)
 {
-       union xfs_rtword_raw    *word = xfs_rbmblock_wordptr(bp, index);
+       union xfs_rtword_raw    *word = xfs_rbmblock_wordptr(args, index);
 
        return word->old;
 }
@@ -197,11 +197,11 @@ xfs_rtbitmap_getword(
 /* Set an ondisk bitmap word from an incore representation. */
 static inline void
 xfs_rtbitmap_setword(
-       struct xfs_buf          *bp,
+       struct xfs_rtalloc_args *args,
        unsigned int            index,
        xfs_rtword_t            value)
 {
-       union xfs_rtword_raw    *word = xfs_rbmblock_wordptr(bp, index);
+       union xfs_rtword_raw    *word = xfs_rbmblock_wordptr(args, index);
 
        word->old = value;
 }
@@ -248,10 +248,10 @@ xfs_rtsumoffs_to_infoword(
 /* Return a pointer to a summary info word within a rt summary block. */
 static inline union xfs_suminfo_raw *
 xfs_rsumblock_infoptr(
-       struct xfs_buf          *bp,
+       struct xfs_rtalloc_args *args,
        unsigned int            index)
 {
-       union xfs_suminfo_raw   *info = bp->b_addr;
+       union xfs_suminfo_raw   *info = args->sumbp->b_addr;
 
        return info + index;
 }
@@ -259,10 +259,10 @@ xfs_rsumblock_infoptr(
 /* Get the current value of a summary counter. */
 static inline xfs_suminfo_t
 xfs_suminfo_get(
-       struct xfs_buf          *bp,
+       struct xfs_rtalloc_args *args,
        unsigned int            index)
 {
-       union xfs_suminfo_raw   *info = xfs_rsumblock_infoptr(bp, index);
+       union xfs_suminfo_raw   *info = xfs_rsumblock_infoptr(args, index);
 
        return info->old;
 }
@@ -270,11 +270,11 @@ xfs_suminfo_get(
 /* Add to the current value of a summary counter and return the new value. */
 static inline xfs_suminfo_t
 xfs_suminfo_add(
-       struct xfs_buf          *bp,
+       struct xfs_rtalloc_args *args,
        unsigned int            index,
        int                     delta)
 {
-       union xfs_suminfo_raw   *info = xfs_rsumblock_infoptr(bp, index);
+       union xfs_suminfo_raw   *info = xfs_rsumblock_infoptr(args, index);
 
        info->old += delta;
        return info->old;