xfs: rename on-disk dquot counter zap functions
authorDarrick J. Wong <darrick.wong@oracle.com>
Fri, 4 May 2018 22:31:20 +0000 (15:31 -0700)
committerDarrick J. Wong <darrick.wong@oracle.com>
Thu, 10 May 2018 15:56:48 +0000 (08:56 -0700)
The function 'xfs_qm_dqiterate' doesn't iterate dquots at all, it
iterates all dquot blocks of a quota inode and clears the counters.
Therefore, change the name to something more descriptive so that we can
introduce a real dquot iterator later.

Signed-off-by: Darrick J. Wong <darrick.wong@oracle.com>
Reviewed-by: Brian Foster <bfoster@redhat.com>
Reviewed-by: Christoph Hellwig <hch@lst.de>
fs/xfs/xfs_qm.c

index 50ba0ea..f927b7d 100644 (file)
@@ -884,7 +884,7 @@ xfs_qm_reset_dqcounts(
 }
 
 STATIC int
-xfs_qm_dqiter_bufs(
+xfs_qm_reset_dqcounts_all(
        struct xfs_mount        *mp,
        xfs_dqid_t              firstid,
        xfs_fsblock_t           bno,
@@ -952,11 +952,11 @@ xfs_qm_dqiter_bufs(
 }
 
 /*
- * Iterate over all allocated USR/GRP/PRJ dquots in the system, calling a
- * caller supplied function for every chunk of dquots that we find.
+ * Iterate over all allocated dquot blocks in this quota inode, zeroing all
+ * counters for every chunk of dquots that we find.
  */
 STATIC int
-xfs_qm_dqiterate(
+xfs_qm_reset_dqcounts_buf(
        struct xfs_mount        *mp,
        struct xfs_inode        *qip,
        uint                    flags,
@@ -1032,7 +1032,7 @@ xfs_qm_dqiterate(
                         * Iterate thru all the blks in the extent and
                         * reset the counters of all the dquots inside them.
                         */
-                       error = xfs_qm_dqiter_bufs(mp, firstid,
+                       error = xfs_qm_reset_dqcounts_all(mp, firstid,
                                                   map[i].br_startblock,
                                                   map[i].br_blockcount,
                                                   flags, buffer_list);
@@ -1293,7 +1293,7 @@ xfs_qm_quotacheck(
         * We don't log our changes till later.
         */
        if (uip) {
-               error = xfs_qm_dqiterate(mp, uip, XFS_QMOPT_UQUOTA,
+               error = xfs_qm_reset_dqcounts_buf(mp, uip, XFS_QMOPT_UQUOTA,
                                         &buffer_list);
                if (error)
                        goto error_return;
@@ -1301,7 +1301,7 @@ xfs_qm_quotacheck(
        }
 
        if (gip) {
-               error = xfs_qm_dqiterate(mp, gip, XFS_QMOPT_GQUOTA,
+               error = xfs_qm_reset_dqcounts_buf(mp, gip, XFS_QMOPT_GQUOTA,
                                         &buffer_list);
                if (error)
                        goto error_return;
@@ -1309,7 +1309,7 @@ xfs_qm_quotacheck(
        }
 
        if (pip) {
-               error = xfs_qm_dqiterate(mp, pip, XFS_QMOPT_PQUOTA,
+               error = xfs_qm_reset_dqcounts_buf(mp, pip, XFS_QMOPT_PQUOTA,
                                         &buffer_list);
                if (error)
                        goto error_return;