From: Brian Foster <bfoster@redhat.com>
To: Dave Chinner <david@fromorbit.com>
Cc: linux-xfs@vger.kernel.org
Subject: Re: [PATCH 05/23] xfs: convert raw ag walks to use for_each_perag
Date: Wed, 26 May 2021 08:33:58 -0400 [thread overview]
Message-ID: <YK5ANsvJkEBWSD0K@bfoster> (raw)
In-Reply-To: <20210519012102.450926-6-david@fromorbit.com>
On Wed, May 19, 2021 at 11:20:44AM +1000, Dave Chinner wrote:
> From: Dave Chinner <dchinner@redhat.com>
>
> Convert the raw walks to an iterator, pulling the current AG out of
> pag->pag_agno instead of the loop iterator variable.
>
> Signed-off-by: Dave Chinner <dchinner@redhat.com>
> ---
Reviewed-by: Brian Foster <bfoster@redhat.com>
> fs/xfs/libxfs/xfs_types.c | 4 ++-
> fs/xfs/scrub/bmap.c | 13 +++++----
> fs/xfs/xfs_log_recover.c | 55 ++++++++++++++++++---------------------
> fs/xfs/xfs_reflink.c | 9 ++++---
> 4 files changed, 43 insertions(+), 38 deletions(-)
>
> diff --git a/fs/xfs/libxfs/xfs_types.c b/fs/xfs/libxfs/xfs_types.c
> index 04801362e1a7..e8f4abee7892 100644
> --- a/fs/xfs/libxfs/xfs_types.c
> +++ b/fs/xfs/libxfs/xfs_types.c
> @@ -11,6 +11,7 @@
> #include "xfs_trans_resv.h"
> #include "xfs_bit.h"
> #include "xfs_mount.h"
> +#include "xfs_ag.h"
>
> /* Find the size of the AG, in blocks. */
> inline xfs_agblock_t
> @@ -222,12 +223,13 @@ xfs_icount_range(
> unsigned long long *max)
> {
> unsigned long long nr_inos = 0;
> + struct xfs_perag *pag;
> xfs_agnumber_t agno;
>
> /* root, rtbitmap, rtsum all live in the first chunk */
> *min = XFS_INODES_PER_CHUNK;
>
> - for (agno = 0; agno < mp->m_sb.sb_agcount; agno++) {
> + for_each_perag(mp, agno, pag) {
> xfs_agino_t first, last;
>
> xfs_agino_range(mp, agno, &first, &last);
> diff --git a/fs/xfs/scrub/bmap.c b/fs/xfs/scrub/bmap.c
> index b5ebf1d1b4db..e457c086887f 100644
> --- a/fs/xfs/scrub/bmap.c
> +++ b/fs/xfs/scrub/bmap.c
> @@ -22,6 +22,7 @@
> #include "scrub/scrub.h"
> #include "scrub/common.h"
> #include "scrub/btree.h"
> +#include "xfs_ag.h"
>
> /* Set us up with an inode's bmap. */
> int
> @@ -575,6 +576,7 @@ xchk_bmap_check_rmaps(
> int whichfork)
> {
> struct xfs_ifork *ifp = XFS_IFORK_PTR(sc->ip, whichfork);
> + struct xfs_perag *pag;
> xfs_agnumber_t agno;
> bool zero_size;
> int error;
> @@ -607,15 +609,16 @@ xchk_bmap_check_rmaps(
> (zero_size || ifp->if_nextents > 0))
> return 0;
>
> - for (agno = 0; agno < sc->mp->m_sb.sb_agcount; agno++) {
> - error = xchk_bmap_check_ag_rmaps(sc, whichfork, agno);
> + for_each_perag(sc->mp, agno, pag) {
> + error = xchk_bmap_check_ag_rmaps(sc, whichfork, pag->pag_agno);
> if (error)
> - return error;
> + break;
> if (sc->sm->sm_flags & XFS_SCRUB_OFLAG_CORRUPT)
> break;
> }
> -
> - return 0;
> + if (pag)
> + xfs_perag_put(pag);
> + return error;
> }
>
> /*
> diff --git a/fs/xfs/xfs_log_recover.c b/fs/xfs/xfs_log_recover.c
> index fee2a4e80241..1227503d2246 100644
> --- a/fs/xfs/xfs_log_recover.c
> +++ b/fs/xfs/xfs_log_recover.c
> @@ -2742,21 +2742,17 @@ STATIC void
> xlog_recover_process_iunlinks(
> struct xlog *log)
> {
> - xfs_mount_t *mp;
> - xfs_agnumber_t agno;
> - xfs_agi_t *agi;
> - struct xfs_buf *agibp;
> - xfs_agino_t agino;
> - int bucket;
> - int error;
> -
> - mp = log->l_mp;
> + struct xfs_mount *mp = log->l_mp;
> + struct xfs_perag *pag;
> + xfs_agnumber_t agno;
> + struct xfs_agi *agi;
> + struct xfs_buf *agibp;
> + xfs_agino_t agino;
> + int bucket;
> + int error;
>
> - for (agno = 0; agno < mp->m_sb.sb_agcount; agno++) {
> - /*
> - * Find the agi for this ag.
> - */
> - error = xfs_read_agi(mp, NULL, agno, &agibp);
> + for_each_perag(mp, agno, pag) {
> + error = xfs_read_agi(mp, NULL, pag->pag_agno, &agibp);
> if (error) {
> /*
> * AGI is b0rked. Don't process it.
> @@ -2782,7 +2778,7 @@ xlog_recover_process_iunlinks(
> agino = be32_to_cpu(agi->agi_unlinked[bucket]);
> while (agino != NULLAGINO) {
> agino = xlog_recover_process_one_iunlink(mp,
> - agno, agino, bucket);
> + pag->pag_agno, agino, bucket);
> cond_resched();
> }
> }
> @@ -3494,27 +3490,28 @@ xlog_recover_cancel(
> */
> STATIC void
> xlog_recover_check_summary(
> - struct xlog *log)
> + struct xlog *log)
> {
> - xfs_mount_t *mp;
> - struct xfs_buf *agfbp;
> - struct xfs_buf *agibp;
> - xfs_agnumber_t agno;
> - uint64_t freeblks;
> - uint64_t itotal;
> - uint64_t ifree;
> - int error;
> + struct xfs_mount *mp = log->l_mp;
> + struct xfs_perag *pag;
> + struct xfs_buf *agfbp;
> + struct xfs_buf *agibp;
> + xfs_agnumber_t agno;
> + uint64_t freeblks;
> + uint64_t itotal;
> + uint64_t ifree;
> + int error;
>
> mp = log->l_mp;
>
> freeblks = 0LL;
> itotal = 0LL;
> ifree = 0LL;
> - for (agno = 0; agno < mp->m_sb.sb_agcount; agno++) {
> - error = xfs_read_agf(mp, NULL, agno, 0, &agfbp);
> + for_each_perag(mp, agno, pag) {
> + error = xfs_read_agf(mp, NULL, pag->pag_agno, 0, &agfbp);
> if (error) {
> xfs_alert(mp, "%s agf read failed agno %d error %d",
> - __func__, agno, error);
> + __func__, pag->pag_agno, error);
> } else {
> struct xfs_agf *agfp = agfbp->b_addr;
>
> @@ -3523,10 +3520,10 @@ xlog_recover_check_summary(
> xfs_buf_relse(agfbp);
> }
>
> - error = xfs_read_agi(mp, NULL, agno, &agibp);
> + error = xfs_read_agi(mp, NULL, pag->pag_agno, &agibp);
> if (error) {
> xfs_alert(mp, "%s agi read failed agno %d error %d",
> - __func__, agno, error);
> + __func__, pag->pag_agno, error);
> } else {
> struct xfs_agi *agi = agibp->b_addr;
>
> diff --git a/fs/xfs/xfs_reflink.c b/fs/xfs/xfs_reflink.c
> index f297d68a931b..0e430b0c1b16 100644
> --- a/fs/xfs/xfs_reflink.c
> +++ b/fs/xfs/xfs_reflink.c
> @@ -755,16 +755,19 @@ int
> xfs_reflink_recover_cow(
> struct xfs_mount *mp)
> {
> + struct xfs_perag *pag;
> xfs_agnumber_t agno;
> int error = 0;
>
> if (!xfs_sb_version_hasreflink(&mp->m_sb))
> return 0;
>
> - for (agno = 0; agno < mp->m_sb.sb_agcount; agno++) {
> - error = xfs_refcount_recover_cow_leftovers(mp, agno);
> - if (error)
> + for_each_perag(mp, agno, pag) {
> + error = xfs_refcount_recover_cow_leftovers(mp, pag->pag_agno);
> + if (error) {
> + xfs_perag_put(pag);
> break;
> + }
> }
>
> return error;
> --
> 2.31.1
>
next prev parent reply other threads:[~2021-05-26 12:34 UTC|newest]
Thread overview: 49+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-05-19 1:20 [PATCH 00/23] xfs: initial agnumber -> perag conversions for shrink Dave Chinner
2021-05-19 1:20 ` [PATCH 01/23] xfs: move xfs_perag_get/put to xfs_ag.[ch] Dave Chinner
2021-05-19 1:20 ` [PATCH 02/23] xfs: prepare for moving perag definitions and support to libxfs Dave Chinner
2021-05-26 12:33 ` Brian Foster
2021-05-27 22:02 ` Darrick J. Wong
2021-05-19 1:20 ` [PATCH 03/23] xfs: move perag structure and setup to libxfs/xfs_ag.[ch] Dave Chinner
2021-05-26 12:33 ` Brian Foster
2021-05-27 22:07 ` Darrick J. Wong
2021-05-19 1:20 ` [PATCH 04/23] xfs: make for_each_perag... a first class citizen Dave Chinner
2021-05-26 12:33 ` Brian Foster
2021-05-27 22:10 ` Darrick J. Wong
2021-05-19 1:20 ` [PATCH 05/23] xfs: convert raw ag walks to use for_each_perag Dave Chinner
2021-05-26 12:33 ` Brian Foster [this message]
2021-05-27 22:12 ` Darrick J. Wong
2021-05-19 1:20 ` [PATCH 06/23] xfs: convert xfs_iwalk to use perag references Dave Chinner
2021-05-27 22:16 ` Darrick J. Wong
2021-06-01 22:00 ` [PATCH 06/23 V2] " Dave Chinner
2021-06-01 23:32 ` Darrick J. Wong
2021-06-02 0:34 ` Dave Chinner
2021-05-19 1:20 ` [PATCH 07/23] xfs: convert secondary superblock walk to use perags Dave Chinner
2021-05-19 1:20 ` [PATCH 08/23] xfs: pass perags through to the busy extent code Dave Chinner
2021-05-19 1:20 ` [PATCH 09/23] xfs: push perags through the ag reservation callouts Dave Chinner
2021-05-26 12:34 ` Brian Foster
2021-05-19 1:20 ` [PATCH 10/23] xfs: pass perags around in fsmap data dev functions Dave Chinner
2021-05-27 22:19 ` Darrick J. Wong
2021-05-19 1:20 ` [PATCH 11/23] xfs: add a perag to the btree cursor Dave Chinner
2021-05-26 12:34 ` Brian Foster
2021-05-19 1:20 ` [PATCH 12/23] xfs: convert rmap btree cursor to using a perag Dave Chinner
2021-05-19 1:20 ` [PATCH 13/23] xfs: convert refcount btree cursor to use perags Dave Chinner
2021-05-27 22:22 ` Darrick J. Wong
2021-05-19 1:20 ` [PATCH 14/23] xfs: convert allocbt cursors " Dave Chinner
2021-05-19 1:20 ` [PATCH 15/23] xfs: use perag for ialloc btree cursors Dave Chinner
2021-05-19 1:20 ` [PATCH 16/23] xfs: remove agno from btree cursor Dave Chinner
2021-05-19 1:20 ` [PATCH 17/23] xfs: simplify xfs_dialloc_select_ag() return values Dave Chinner
2021-05-19 1:20 ` [PATCH 18/23] xfs: collapse AG selection for inode allocation Dave Chinner
2021-05-27 11:19 ` Brian Foster
2021-06-01 22:40 ` [PATCH 18/23 V2] " Dave Chinner
2021-06-01 23:27 ` Darrick J. Wong
2021-05-27 22:33 ` [PATCH 18/23] " Darrick J. Wong
2021-05-19 1:20 ` [PATCH 19/23] xfs: get rid of xfs_dir_ialloc() Dave Chinner
2021-05-19 1:20 ` [PATCH 20/23] xfs: inode allocation can use a single perag instance Dave Chinner
2021-05-19 1:21 ` [PATCH 21/23] xfs: clean up and simplify xfs_dialloc() Dave Chinner
2021-05-27 11:20 ` Brian Foster
2021-05-27 22:39 ` Darrick J. Wong
2021-05-19 1:21 ` [PATCH 22/23] xfs: use perag through unlink processing Dave Chinner
2021-05-27 11:20 ` Brian Foster
2021-05-19 1:21 ` [PATCH 23/23] xfs: remove xfs_perag_t Dave Chinner
2021-05-27 11:20 ` Brian Foster
2021-05-27 22:06 ` Darrick J. Wong
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=YK5ANsvJkEBWSD0K@bfoster \
--to=bfoster@redhat.com \
--cc=david@fromorbit.com \
--cc=linux-xfs@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).