From: "Darrick J. Wong" <djwong@kernel.org>
To: Dave Chinner <david@fromorbit.com>
Cc: linux-xfs@vger.kernel.org
Subject: Re: [PATCH 12/22] xfs: convert rmap btree cursor to using a perag
Date: Wed, 12 May 2021 20:54:25 -0700 [thread overview]
Message-ID: <20210513035425.GR8582@magnolia> (raw)
In-Reply-To: <20210512224512.GI8582@magnolia>
On Wed, May 12, 2021 at 03:45:12PM -0700, Darrick J. Wong wrote:
> On Thu, May 06, 2021 at 05:20:44PM +1000, Dave Chinner wrote:
> > From: Dave Chinner <dchinner@redhat.com>
> >
> > Signed-off-by: Dave Chinner <dchinner@redhat.com>
> > ---
> > fs/xfs/libxfs/xfs_ag.c | 2 +-
> > fs/xfs/libxfs/xfs_alloc.c | 7 ++++---
> > fs/xfs/libxfs/xfs_rmap.c | 10 ++++-----
> > fs/xfs/libxfs/xfs_rmap.h | 6 ++++--
> > fs/xfs/libxfs/xfs_rmap_btree.c | 37 +++++++++++++++-------------------
> > fs/xfs/libxfs/xfs_rmap_btree.h | 4 ++--
> > fs/xfs/scrub/agheader_repair.c | 6 ++----
> > fs/xfs/scrub/bmap.c | 2 +-
> > fs/xfs/scrub/common.c | 2 +-
> > fs/xfs/scrub/repair.c | 10 ++++-----
> > fs/xfs/xfs_fsmap.c | 2 +-
> > 11 files changed, 42 insertions(+), 46 deletions(-)
> >
> > diff --git a/fs/xfs/libxfs/xfs_ag.c b/fs/xfs/libxfs/xfs_ag.c
> > index 14d8b866dc6d..44f2787f3556 100644
> > --- a/fs/xfs/libxfs/xfs_ag.c
> > +++ b/fs/xfs/libxfs/xfs_ag.c
> > @@ -914,7 +914,7 @@ xfs_ag_extend_space(
> > * XFS_RMAP_OINFO_SKIP_UPDATE is used here to tell the rmap btree that
> > * this doesn't actually exist in the rmap btree.
> > */
> > - error = xfs_rmap_free(tp, bp, id->agno,
> > + error = xfs_rmap_free(tp, bp, bp->b_pag,
>
> Same question here as in the last patch -- so long as we have to pass
> the AGF buffer pointer into the xfs_rmap functions, why is it even
> necessary to pass both the AGF and the perag when we could get the
> second from the first?
>
> I suspect I'm going to whine about this for the next three patches, but
> the mechanical conversion looks correct to me.
/me changes his mind and
Reviewed-by: Darrick J. Wong <djwong@kernel.org>
--D
>
> --D
>
> > be32_to_cpu(agf->agf_length) - len,
> > len, &XFS_RMAP_OINFO_SKIP_UPDATE);
> > if (error)
> > diff --git a/fs/xfs/libxfs/xfs_alloc.c b/fs/xfs/libxfs/xfs_alloc.c
> > index 7ec4af6bf494..10747cc4d8f6 100644
> > --- a/fs/xfs/libxfs/xfs_alloc.c
> > +++ b/fs/xfs/libxfs/xfs_alloc.c
> > @@ -1092,7 +1092,7 @@ xfs_alloc_ag_vextent_small(
> > * If we're feeding an AGFL block to something that doesn't live in the
> > * free space, we need to clear out the OWN_AG rmap.
> > */
> > - error = xfs_rmap_free(args->tp, args->agbp, args->agno, fbno, 1,
> > + error = xfs_rmap_free(args->tp, args->agbp, args->pag, fbno, 1,
> > &XFS_RMAP_OINFO_AG);
> > if (error)
> > goto error;
> > @@ -1169,7 +1169,7 @@ xfs_alloc_ag_vextent(
> >
> > /* if not file data, insert new block into the reverse map btree */
> > if (!xfs_rmap_should_skip_owner_update(&args->oinfo)) {
> > - error = xfs_rmap_alloc(args->tp, args->agbp, args->agno,
> > + error = xfs_rmap_alloc(args->tp, args->agbp, args->pag,
> > args->agbno, args->len, &args->oinfo);
> > if (error)
> > return error;
> > @@ -1899,12 +1899,13 @@ xfs_free_ag_extent(
> > int haveright; /* have a right neighbor */
> > int i;
> > int error;
> > + struct xfs_perag *pag = agbp->b_pag;
> >
> > bno_cur = cnt_cur = NULL;
> > mp = tp->t_mountp;
> >
> > if (!xfs_rmap_should_skip_owner_update(oinfo)) {
> > - error = xfs_rmap_free(tp, agbp, agno, bno, len, oinfo);
> > + error = xfs_rmap_free(tp, agbp, pag, bno, len, oinfo);
> > if (error)
> > goto error0;
> > }
> > diff --git a/fs/xfs/libxfs/xfs_rmap.c b/fs/xfs/libxfs/xfs_rmap.c
> > index 0d7a6997120c..b23f949ee15c 100644
> > --- a/fs/xfs/libxfs/xfs_rmap.c
> > +++ b/fs/xfs/libxfs/xfs_rmap.c
> > @@ -696,7 +696,7 @@ int
> > xfs_rmap_free(
> > struct xfs_trans *tp,
> > struct xfs_buf *agbp,
> > - xfs_agnumber_t agno,
> > + struct xfs_perag *pag,
> > xfs_agblock_t bno,
> > xfs_extlen_t len,
> > const struct xfs_owner_info *oinfo)
> > @@ -708,7 +708,7 @@ xfs_rmap_free(
> > if (!xfs_sb_version_hasrmapbt(&mp->m_sb))
> > return 0;
> >
> > - cur = xfs_rmapbt_init_cursor(mp, tp, agbp, agno, NULL);
> > + cur = xfs_rmapbt_init_cursor(mp, tp, agbp, pag);
> >
> > error = xfs_rmap_unmap(cur, bno, len, false, oinfo);
> >
> > @@ -950,7 +950,7 @@ int
> > xfs_rmap_alloc(
> > struct xfs_trans *tp,
> > struct xfs_buf *agbp,
> > - xfs_agnumber_t agno,
> > + struct xfs_perag *pag,
> > xfs_agblock_t bno,
> > xfs_extlen_t len,
> > const struct xfs_owner_info *oinfo)
> > @@ -962,7 +962,7 @@ xfs_rmap_alloc(
> > if (!xfs_sb_version_hasrmapbt(&mp->m_sb))
> > return 0;
> >
> > - cur = xfs_rmapbt_init_cursor(mp, tp, agbp, agno, NULL);
> > + cur = xfs_rmapbt_init_cursor(mp, tp, agbp, pag);
> > error = xfs_rmap_map(cur, bno, len, false, oinfo);
> >
> > xfs_btree_del_cursor(cur, error);
> > @@ -2408,7 +2408,7 @@ xfs_rmap_finish_one(
> > goto out_drop;
> > }
> >
> > - rcur = xfs_rmapbt_init_cursor(mp, tp, agbp, pag->pag_agno, pag);
> > + rcur = xfs_rmapbt_init_cursor(mp, tp, agbp, pag);
> > }
> > *pcur = rcur;
> >
> > diff --git a/fs/xfs/libxfs/xfs_rmap.h b/fs/xfs/libxfs/xfs_rmap.h
> > index abe633403fd1..f2423cf7f1e2 100644
> > --- a/fs/xfs/libxfs/xfs_rmap.h
> > +++ b/fs/xfs/libxfs/xfs_rmap.h
> > @@ -6,6 +6,8 @@
> > #ifndef __XFS_RMAP_H__
> > #define __XFS_RMAP_H__
> >
> > +struct xfs_perag;
> > +
> > static inline void
> > xfs_rmap_ino_bmbt_owner(
> > struct xfs_owner_info *oi,
> > @@ -113,10 +115,10 @@ xfs_owner_info_pack(
> > }
> >
> > int xfs_rmap_alloc(struct xfs_trans *tp, struct xfs_buf *agbp,
> > - xfs_agnumber_t agno, xfs_agblock_t bno, xfs_extlen_t len,
> > + struct xfs_perag *pag, xfs_agblock_t bno, xfs_extlen_t len,
> > const struct xfs_owner_info *oinfo);
> > int xfs_rmap_free(struct xfs_trans *tp, struct xfs_buf *agbp,
> > - xfs_agnumber_t agno, xfs_agblock_t bno, xfs_extlen_t len,
> > + struct xfs_perag *pag, xfs_agblock_t bno, xfs_extlen_t len,
> > const struct xfs_owner_info *oinfo);
> >
> > int xfs_rmap_lookup_le(struct xfs_btree_cur *cur, xfs_agblock_t bno,
> > diff --git a/fs/xfs/libxfs/xfs_rmap_btree.c b/fs/xfs/libxfs/xfs_rmap_btree.c
> > index 7bef8feeded1..cafe181bc92d 100644
> > --- a/fs/xfs/libxfs/xfs_rmap_btree.c
> > +++ b/fs/xfs/libxfs/xfs_rmap_btree.c
> > @@ -52,7 +52,7 @@ xfs_rmapbt_dup_cursor(
> > struct xfs_btree_cur *cur)
> > {
> > return xfs_rmapbt_init_cursor(cur->bc_mp, cur->bc_tp,
> > - cur->bc_ag.agbp, cur->bc_ag.agno, cur->bc_ag.pag);
> > + cur->bc_ag.agbp, cur->bc_ag.pag);
> > }
> >
> > STATIC void
> > @@ -64,13 +64,12 @@ xfs_rmapbt_set_root(
> > struct xfs_buf *agbp = cur->bc_ag.agbp;
> > struct xfs_agf *agf = agbp->b_addr;
> > int btnum = cur->bc_btnum;
> > - struct xfs_perag *pag = agbp->b_pag;
> >
> > ASSERT(ptr->s != 0);
> >
> > agf->agf_roots[btnum] = ptr->s;
> > be32_add_cpu(&agf->agf_levels[btnum], inc);
> > - pag->pagf_levels[btnum] += inc;
> > + cur->bc_ag.pag->pagf_levels[btnum] += inc;
> >
> > xfs_alloc_log_agf(cur->bc_tp, agbp, XFS_AGF_ROOTS | XFS_AGF_LEVELS);
> > }
> > @@ -84,6 +83,7 @@ xfs_rmapbt_alloc_block(
> > {
> > struct xfs_buf *agbp = cur->bc_ag.agbp;
> > struct xfs_agf *agf = agbp->b_addr;
> > + struct xfs_perag *pag = cur->bc_ag.pag;
> > int error;
> > xfs_agblock_t bno;
> >
> > @@ -93,20 +93,19 @@ xfs_rmapbt_alloc_block(
> > if (error)
> > return error;
> >
> > - trace_xfs_rmapbt_alloc_block(cur->bc_mp, cur->bc_ag.agno,
> > - bno, 1);
> > + trace_xfs_rmapbt_alloc_block(cur->bc_mp, pag->pag_agno, bno, 1);
> > if (bno == NULLAGBLOCK) {
> > *stat = 0;
> > return 0;
> > }
> >
> > - xfs_extent_busy_reuse(cur->bc_mp, agbp->b_pag, bno, 1, false);
> > + xfs_extent_busy_reuse(cur->bc_mp, pag, bno, 1, false);
> >
> > new->s = cpu_to_be32(bno);
> > be32_add_cpu(&agf->agf_rmap_blocks, 1);
> > xfs_alloc_log_agf(cur->bc_tp, agbp, XFS_AGF_RMAP_BLOCKS);
> >
> > - xfs_ag_resv_rmapbt_alloc(cur->bc_mp, cur->bc_ag.agno);
> > + xfs_ag_resv_rmapbt_alloc(cur->bc_mp, pag->pag_agno);
> >
> > *stat = 1;
> > return 0;
> > @@ -119,12 +118,12 @@ xfs_rmapbt_free_block(
> > {
> > struct xfs_buf *agbp = cur->bc_ag.agbp;
> > struct xfs_agf *agf = agbp->b_addr;
> > - struct xfs_perag *pag;
> > + struct xfs_perag *pag = cur->bc_ag.pag;
> > xfs_agblock_t bno;
> > int error;
> >
> > bno = xfs_daddr_to_agbno(cur->bc_mp, XFS_BUF_ADDR(bp));
> > - trace_xfs_rmapbt_free_block(cur->bc_mp, cur->bc_ag.agno,
> > + trace_xfs_rmapbt_free_block(cur->bc_mp, pag->pag_agno,
> > bno, 1);
> > be32_add_cpu(&agf->agf_rmap_blocks, -1);
> > xfs_alloc_log_agf(cur->bc_tp, agbp, XFS_AGF_RMAP_BLOCKS);
> > @@ -132,7 +131,6 @@ xfs_rmapbt_free_block(
> > if (error)
> > return error;
> >
> > - pag = cur->bc_ag.agbp->b_pag;
> > xfs_extent_busy_insert(cur->bc_tp, pag, bno, 1,
> > XFS_EXTENT_BUSY_SKIP_DISCARD);
> >
> > @@ -214,7 +212,7 @@ xfs_rmapbt_init_ptr_from_cur(
> > {
> > struct xfs_agf *agf = cur->bc_ag.agbp->b_addr;
> >
> > - ASSERT(cur->bc_ag.agno == be32_to_cpu(agf->agf_seqno));
> > + ASSERT(cur->bc_ag.pag->pag_agno == be32_to_cpu(agf->agf_seqno));
> >
> > ptr->s = agf->agf_roots[cur->bc_btnum];
> > }
> > @@ -449,7 +447,6 @@ static struct xfs_btree_cur *
> > xfs_rmapbt_init_common(
> > struct xfs_mount *mp,
> > struct xfs_trans *tp,
> > - xfs_agnumber_t agno,
> > struct xfs_perag *pag)
> > {
> > struct xfs_btree_cur *cur;
> > @@ -462,13 +459,12 @@ xfs_rmapbt_init_common(
> > cur->bc_flags = XFS_BTREE_CRC_BLOCKS | XFS_BTREE_OVERLAPPING;
> > cur->bc_blocklog = mp->m_sb.sb_blocklog;
> > cur->bc_statoff = XFS_STATS_CALC_INDEX(xs_rmap_2);
> > - cur->bc_ag.agno = agno;
> > cur->bc_ops = &xfs_rmapbt_ops;
> > - if (pag) {
> > - /* take a reference for the cursor */
> > - atomic_inc(&pag->pag_ref);
> > - }
> > +
> > + /* take a reference for the cursor */
> > + atomic_inc(&pag->pag_ref);
> > cur->bc_ag.pag = pag;
> > + cur->bc_ag.agno = pag->pag_agno;
> >
> > return cur;
> > }
> > @@ -479,13 +475,12 @@ xfs_rmapbt_init_cursor(
> > struct xfs_mount *mp,
> > struct xfs_trans *tp,
> > struct xfs_buf *agbp,
> > - xfs_agnumber_t agno,
> > struct xfs_perag *pag)
> > {
> > struct xfs_agf *agf = agbp->b_addr;
> > struct xfs_btree_cur *cur;
> >
> > - cur = xfs_rmapbt_init_common(mp, tp, agno, pag);
> > + cur = xfs_rmapbt_init_common(mp, tp, pag);
> > cur->bc_nlevels = be32_to_cpu(agf->agf_levels[XFS_BTNUM_RMAP]);
> > cur->bc_ag.agbp = agbp;
> > return cur;
> > @@ -496,11 +491,11 @@ struct xfs_btree_cur *
> > xfs_rmapbt_stage_cursor(
> > struct xfs_mount *mp,
> > struct xbtree_afakeroot *afake,
> > - xfs_agnumber_t agno)
> > + struct xfs_perag *pag)
> > {
> > struct xfs_btree_cur *cur;
> >
> > - cur = xfs_rmapbt_init_common(mp, NULL, agno, NULL);
> > + cur = xfs_rmapbt_init_common(mp, NULL, pag);
> > xfs_btree_stage_afakeroot(cur, afake);
> > return cur;
> > }
> > diff --git a/fs/xfs/libxfs/xfs_rmap_btree.h b/fs/xfs/libxfs/xfs_rmap_btree.h
> > index c94f418cc06b..88d8d18788a2 100644
> > --- a/fs/xfs/libxfs/xfs_rmap_btree.h
> > +++ b/fs/xfs/libxfs/xfs_rmap_btree.h
> > @@ -43,9 +43,9 @@ struct xbtree_afakeroot;
> >
> > struct xfs_btree_cur *xfs_rmapbt_init_cursor(struct xfs_mount *mp,
> > struct xfs_trans *tp, struct xfs_buf *bp,
> > - xfs_agnumber_t agno, struct xfs_perag *pag);
> > + struct xfs_perag *pag);
> > struct xfs_btree_cur *xfs_rmapbt_stage_cursor(struct xfs_mount *mp,
> > - struct xbtree_afakeroot *afake, xfs_agnumber_t agno);
> > + struct xbtree_afakeroot *afake, struct xfs_perag *pag);
> > void xfs_rmapbt_commit_staged_btree(struct xfs_btree_cur *cur,
> > struct xfs_trans *tp, struct xfs_buf *agbp);
> > int xfs_rmapbt_maxrecs(int blocklen, int leaf);
> > diff --git a/fs/xfs/scrub/agheader_repair.c b/fs/xfs/scrub/agheader_repair.c
> > index 5dd91bf04c18..981c689e3d95 100644
> > --- a/fs/xfs/scrub/agheader_repair.c
> > +++ b/fs/xfs/scrub/agheader_repair.c
> > @@ -269,8 +269,7 @@ xrep_agf_calc_from_btrees(
> > btreeblks += blocks - 1;
> >
> > /* Update the AGF counters from the rmapbt. */
> > - cur = xfs_rmapbt_init_cursor(mp, sc->tp, agf_bp, sc->sa.agno,
> > - sc->sa.pag);
> > + cur = xfs_rmapbt_init_cursor(mp, sc->tp, agf_bp, sc->sa.pag);
> > error = xfs_btree_count_blocks(cur, &blocks);
> > if (error)
> > goto err;
> > @@ -491,8 +490,7 @@ xrep_agfl_collect_blocks(
> > xbitmap_init(&ra.agmetablocks);
> >
> > /* Find all space used by the free space btrees & rmapbt. */
> > - cur = xfs_rmapbt_init_cursor(mp, sc->tp, agf_bp, sc->sa.agno,
> > - sc->sa.pag);
> > + cur = xfs_rmapbt_init_cursor(mp, sc->tp, agf_bp, sc->sa.pag);
> > error = xfs_rmap_query_all(cur, xrep_agfl_walk_rmap, &ra);
> > if (error)
> > goto err;
> > diff --git a/fs/xfs/scrub/bmap.c b/fs/xfs/scrub/bmap.c
> > index a792d9ffd61e..b20c3f03b3c5 100644
> > --- a/fs/xfs/scrub/bmap.c
> > +++ b/fs/xfs/scrub/bmap.c
> > @@ -556,7 +556,7 @@ xchk_bmap_check_ag_rmaps(
> > if (error)
> > return error;
> >
> > - cur = xfs_rmapbt_init_cursor(sc->mp, sc->tp, agf, agno, NULL);
> > + cur = xfs_rmapbt_init_cursor(sc->mp, sc->tp, agf, sc->sa.pag);
> >
> > sbcri.sc = sc;
> > sbcri.whichfork = whichfork;
> > diff --git a/fs/xfs/scrub/common.c b/fs/xfs/scrub/common.c
> > index 50768559fb60..48381c1adeed 100644
> > --- a/fs/xfs/scrub/common.c
> > +++ b/fs/xfs/scrub/common.c
> > @@ -493,7 +493,7 @@ xchk_ag_btcur_init(
> > if (sa->agf_bp && xfs_sb_version_hasrmapbt(&mp->m_sb) &&
> > xchk_ag_btree_healthy_enough(sc, sa->pag, XFS_BTNUM_RMAP)) {
> > sa->rmap_cur = xfs_rmapbt_init_cursor(mp, sc->tp, sa->agf_bp,
> > - agno, sa->pag);
> > + sa->pag);
> > }
> >
> > /* Set up a refcountbt cursor for cross-referencing. */
> > diff --git a/fs/xfs/scrub/repair.c b/fs/xfs/scrub/repair.c
> > index 862dc56fd8cd..5cf1c3707b6a 100644
> > --- a/fs/xfs/scrub/repair.c
> > +++ b/fs/xfs/scrub/repair.c
> > @@ -509,7 +509,7 @@ xrep_put_freelist(
> > * create an rmap for the block prior to merging it or else other
> > * parts will break.
> > */
> > - error = xfs_rmap_alloc(sc->tp, sc->sa.agf_bp, sc->sa.agno, agbno, 1,
> > + error = xfs_rmap_alloc(sc->tp, sc->sa.agf_bp, sc->sa.pag, agbno, 1,
> > &XFS_RMAP_OINFO_AG);
> > if (error)
> > return error;
> > @@ -555,7 +555,7 @@ xrep_reap_block(
> > } else {
> > agf_bp = sc->sa.agf_bp;
> > }
> > - cur = xfs_rmapbt_init_cursor(sc->mp, sc->tp, agf_bp, agno, sc->sa.pag);
> > + cur = xfs_rmapbt_init_cursor(sc->mp, sc->tp, agf_bp, sc->sa.pag);
> >
> > /* Can we find any other rmappings? */
> > error = xfs_rmap_has_other_keys(cur, agbno, 1, oinfo, &has_other_rmap);
> > @@ -577,7 +577,8 @@ xrep_reap_block(
> > * to run xfs_repair.
> > */
> > if (has_other_rmap)
> > - error = xfs_rmap_free(sc->tp, agf_bp, agno, agbno, 1, oinfo);
> > + error = xfs_rmap_free(sc->tp, agf_bp, sc->sa.pag, agbno,
> > + 1, oinfo);
> > else if (resv == XFS_AG_RESV_AGFL)
> > error = xrep_put_freelist(sc, agbno);
> > else
> > @@ -892,8 +893,7 @@ xrep_find_ag_btree_roots(
> > fab->height = 0;
> > }
> >
> > - cur = xfs_rmapbt_init_cursor(mp, sc->tp, agf_bp, sc->sa.agno,
> > - sc->sa.pag);
> > + cur = xfs_rmapbt_init_cursor(mp, sc->tp, agf_bp, sc->sa.pag);
> > error = xfs_rmap_query_all(cur, xrep_findroot_rmap, &ri);
> > xfs_btree_del_cursor(cur, error);
> >
> > diff --git a/fs/xfs/xfs_fsmap.c b/fs/xfs/xfs_fsmap.c
> > index b654a2bf9a9f..7bfe9ea35de0 100644
> > --- a/fs/xfs/xfs_fsmap.c
> > +++ b/fs/xfs/xfs_fsmap.c
> > @@ -708,7 +708,7 @@ xfs_getfsmap_datadev_rmapbt_query(
> >
> > /* Allocate cursor for this AG and query_range it. */
> > *curpp = xfs_rmapbt_init_cursor(tp->t_mountp, tp, info->agf_bp,
> > - info->pag->pag_agno, info->pag);
> > + info->pag);
> > return xfs_rmap_query_range(*curpp, &info->low, &info->high,
> > xfs_getfsmap_datadev_helper, info);
> > }
> > --
> > 2.31.1
> >
next prev parent reply other threads:[~2021-05-13 3:54 UTC|newest]
Thread overview: 87+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-05-06 7:20 [RFC 00/22] xfs: initial agnumber -> perag conversions for shrink Dave Chinner
2021-05-06 7:20 ` [PATCH 01/22] xfs: move xfs_perag_get/put to xfs_ag.[ch] Dave Chinner
2021-05-10 12:52 ` Brian Foster
2021-05-11 7:18 ` Dave Chinner
2021-05-10 22:28 ` Darrick J. Wong
2021-05-06 7:20 ` [PATCH 02/22] xfs: prepare for moving perag definitions and support to libxfs Dave Chinner
2021-05-10 12:53 ` Brian Foster
2021-05-11 7:19 ` Dave Chinner
2021-05-06 7:20 ` [PATCH 03/22] xfs: move perag structure and setup to libxfs/xfs_ag.[ch] Dave Chinner
2021-05-10 22:26 ` Darrick J. Wong
2021-05-10 23:38 ` Dave Chinner
2021-05-06 7:20 ` [PATCH 04/22] xfs: make for_each_perag... a first class citizen Dave Chinner
2021-05-10 12:53 ` Brian Foster
2021-05-11 7:35 ` Dave Chinner
2021-05-11 12:29 ` Brian Foster
2021-05-11 21:33 ` Dave Chinner
2021-05-12 21:58 ` Darrick J. Wong
2021-05-06 7:20 ` [PATCH 05/22] xfs: convert raw ag walks to use for_each_perag Dave Chinner
2021-05-10 12:54 ` Brian Foster
2021-05-06 7:20 ` [PATCH 06/22] xfs: convert xfs_iwalk to use perag references Dave Chinner
2021-05-10 13:41 ` Brian Foster
2021-05-12 22:08 ` Darrick J. Wong
2021-05-06 7:20 ` [PATCH 07/22] xfs: convert secondary superblock walk to use perags Dave Chinner
2021-05-10 13:41 ` Brian Foster
2021-05-12 22:09 ` Darrick J. Wong
2021-05-06 7:20 ` [PATCH 08/22] xfs: pass perags through to the busy extent code Dave Chinner
2021-05-11 12:29 ` Brian Foster
2021-05-12 22:13 ` Darrick J. Wong
2021-05-06 7:20 ` [PATCH 09/22] xfs: push perags through the ag reservation callouts Dave Chinner
2021-05-11 12:29 ` Brian Foster
2021-05-13 0:29 ` Dave Chinner
2021-05-12 22:16 ` Darrick J. Wong
2021-05-06 7:20 ` [PATCH 10/22] xfs: pass perags around in fsmap data dev functions Dave Chinner
2021-05-11 12:30 ` Brian Foster
2021-05-12 22:23 ` Darrick J. Wong
2021-05-18 1:00 ` Dave Chinner
2021-05-06 7:20 ` [PATCH 11/22] xfs: add a perag to the btree cursor Dave Chinner
2021-05-11 12:30 ` Brian Foster
2021-05-11 20:51 ` Darrick J. Wong
2021-05-11 21:52 ` Dave Chinner
2021-05-12 12:49 ` Brian Foster
2021-05-12 22:41 ` Darrick J. Wong
2021-05-12 22:40 ` Darrick J. Wong
2021-05-13 0:12 ` Dave Chinner
2021-05-13 0:55 ` Darrick J. Wong
2021-05-13 1:07 ` Dave Chinner
2021-05-13 3:49 ` Darrick J. Wong
2021-05-06 7:20 ` [PATCH 12/22] xfs: convert rmap btree cursor to using a perag Dave Chinner
2021-05-11 12:30 ` Brian Foster
2021-05-12 22:45 ` Darrick J. Wong
2021-05-13 3:54 ` Darrick J. Wong [this message]
2021-05-06 7:20 ` [PATCH 13/22] xfs: convert refcount btree cursor to use perags Dave Chinner
2021-05-11 12:30 ` Brian Foster
2021-05-12 22:47 ` Darrick J. Wong
2021-05-06 7:20 ` [PATCH 14/22] xfs: convert allocbt cursors " Dave Chinner
2021-05-11 12:30 ` Brian Foster
2021-05-13 3:55 ` Darrick J. Wong
2021-05-06 7:20 ` [PATCH 15/22] xfs: use perag for ialloc btree cursors Dave Chinner
2021-05-11 12:30 ` Brian Foster
2021-05-13 3:55 ` Darrick J. Wong
2021-05-06 7:20 ` [PATCH 16/22] xfs: remove agno from btree cursor Dave Chinner
2021-05-11 12:34 ` Brian Foster
2021-05-11 22:02 ` Dave Chinner
2021-05-12 22:52 ` Darrick J. Wong
2021-05-06 7:20 ` [PATCH 17/22] xfs: simplify xfs_dialloc_select_ag() return values Dave Chinner
2021-05-12 12:49 ` Brian Foster
2021-05-12 22:55 ` Darrick J. Wong
2021-05-06 7:20 ` [PATCH 18/22] xfs: collapse AG selection for inode allocation Dave Chinner
2021-05-12 12:52 ` Brian Foster
2021-05-18 1:21 ` Dave Chinner
2021-05-12 23:11 ` Darrick J. Wong
2021-05-18 1:29 ` Dave Chinner
2021-05-06 7:20 ` [PATCH 19/22] xfs: get rid of xfs_dir_ialloc() Dave Chinner
2021-05-06 11:26 ` kernel test robot
2021-05-06 11:26 ` kernel test robot
2021-05-06 11:26 ` [RFC PATCH] xfs: xfs_dialloc_ag can be static kernel test robot
2021-05-06 11:26 ` kernel test robot
2021-05-12 12:52 ` [PATCH 19/22] xfs: get rid of xfs_dir_ialloc() Brian Foster
2021-05-12 23:19 ` Darrick J. Wong
2021-05-06 7:20 ` [PATCH 20/22] xfs: inode allocation can use a single perag instance Dave Chinner
2021-05-12 12:52 ` Brian Foster
2021-05-12 23:19 ` Darrick J. Wong
2021-05-06 7:20 ` [PATCH 21/22] xfs: clean up and simplify xfs_dialloc() Dave Chinner
2021-05-12 21:49 ` Darrick J. Wong
2021-05-18 1:46 ` Dave Chinner
2021-05-06 7:20 ` [PATCH 22/22] xfs: use perag through unlink processing Dave Chinner
2021-05-12 21:37 ` 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=20210513035425.GR8582@magnolia \
--to=djwong@kernel.org \
--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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.