xfs: store inode btree block counts in AGI header
authorDarrick J. Wong <darrick.wong@oracle.com>
Mon, 17 Aug 2020 16:58:01 +0000 (09:58 -0700)
committerDarrick J. Wong <darrick.wong@oracle.com>
Wed, 16 Sep 2020 03:52:39 +0000 (20:52 -0700)
Add a btree block usage counters for both inode btrees to the AGI header
so that we don't have to walk the entire finobt at mount time to create
the per-AG reservations.

Signed-off-by: Darrick J. Wong <darrick.wong@oracle.com>
Reviewed-by: Brian Foster <bfoster@redhat.com>
fs/xfs/libxfs/xfs_ag.c
fs/xfs/libxfs/xfs_format.h
fs/xfs/libxfs/xfs_ialloc.c
fs/xfs/libxfs/xfs_ialloc_btree.c
fs/xfs/xfs_ondisk.h
fs/xfs/xfs_super.c

index 8cf73fe4338e00c257cd5438d8e4addd37807c6c..9331f3516afa242aff225365923d28d0aef3eaf0 100644 (file)
@@ -333,6 +333,11 @@ xfs_agiblock_init(
        }
        for (bucket = 0; bucket < XFS_AGI_UNLINKED_BUCKETS; bucket++)
                agi->agi_unlinked[bucket] = cpu_to_be32(NULLAGINO);
+       if (xfs_sb_version_hasinobtcounts(&mp->m_sb)) {
+               agi->agi_iblocks = cpu_to_be32(1);
+               if (xfs_sb_version_hasfinobt(&mp->m_sb))
+                       agi->agi_fblocks = cpu_to_be32(1);
+       }
 }
 
 typedef void (*aghdr_init_work_f)(struct xfs_mount *mp, struct xfs_buf *bp,
index 31b7ece985bb231eb9354d7bffe8c535e2c71b1f..03cbedb7eafc7130f1cce7b5de67acb97130c887 100644 (file)
@@ -449,6 +449,7 @@ xfs_sb_has_compat_feature(
 #define XFS_SB_FEAT_RO_COMPAT_FINOBT   (1 << 0)                /* free inode btree */
 #define XFS_SB_FEAT_RO_COMPAT_RMAPBT   (1 << 1)                /* reverse map btree */
 #define XFS_SB_FEAT_RO_COMPAT_REFLINK  (1 << 2)                /* reflinked files */
+#define XFS_SB_FEAT_RO_COMPAT_INOBTCNT (1 << 3)                /* inobt block counts */
 #define XFS_SB_FEAT_RO_COMPAT_ALL \
                (XFS_SB_FEAT_RO_COMPAT_FINOBT | \
                 XFS_SB_FEAT_RO_COMPAT_RMAPBT | \
@@ -563,6 +564,17 @@ static inline bool xfs_sb_version_hasreflink(struct xfs_sb *sbp)
                (sbp->sb_features_ro_compat & XFS_SB_FEAT_RO_COMPAT_REFLINK);
 }
 
+/*
+ * Inode btree block counter.  We record the number of inobt and finobt blocks
+ * in the AGI header so that we can skip the finobt walk at mount time when
+ * setting up per-AG reservations.
+ */
+static inline bool xfs_sb_version_hasinobtcounts(struct xfs_sb *sbp)
+{
+       return XFS_SB_VERSION_NUM(sbp) == XFS_SB_VERSION_5 &&
+               (sbp->sb_features_ro_compat & XFS_SB_FEAT_RO_COMPAT_INOBTCNT);
+}
+
 /*
  * end of superblock version macros
  */
@@ -765,6 +777,9 @@ typedef struct xfs_agi {
        __be32          agi_free_root; /* root of the free inode btree */
        __be32          agi_free_level;/* levels in free inode btree */
 
+       __be32          agi_iblocks;    /* inobt blocks used */
+       __be32          agi_fblocks;    /* finobt blocks used */
+
        /* structure must be padded to 64 bit alignment */
 } xfs_agi_t;
 
@@ -785,7 +800,8 @@ typedef struct xfs_agi {
 #define        XFS_AGI_ALL_BITS_R1     ((1 << XFS_AGI_NUM_BITS_R1) - 1)
 #define        XFS_AGI_FREE_ROOT       (1 << 11)
 #define        XFS_AGI_FREE_LEVEL      (1 << 12)
-#define        XFS_AGI_NUM_BITS_R2     13
+#define        XFS_AGI_IBLOCKS         (1 << 13) /* both inobt/finobt block counters */
+#define        XFS_AGI_NUM_BITS_R2     14
 
 /* disk block (xfs_daddr_t) in the AG */
 #define XFS_AGI_DADDR(mp)      ((xfs_daddr_t)(2 << (mp)->m_sectbb_log))
index a6b37db55169c784b880bda8361c04602cf1394f..7c1ece5a5f059539e9604c5fd651af796d35fb8a 100644 (file)
@@ -2473,6 +2473,7 @@ xfs_ialloc_log_agi(
                offsetof(xfs_agi_t, agi_unlinked),
                offsetof(xfs_agi_t, agi_free_root),
                offsetof(xfs_agi_t, agi_free_level),
+               offsetof(xfs_agi_t, agi_iblocks),
                sizeof(xfs_agi_t)
        };
 #ifdef DEBUG
index 3c8aebc36e643fc2cba944c2ddcbebe9d0ce27c3..6c42a2421cce90be73250d83bb1b234eb2803d2a 100644 (file)
@@ -67,6 +67,25 @@ xfs_finobt_set_root(
                           XFS_AGI_FREE_ROOT | XFS_AGI_FREE_LEVEL);
 }
 
+/* Update the inode btree block counter for this btree. */
+static inline void
+xfs_inobt_mod_blockcount(
+       struct xfs_btree_cur    *cur,
+       int                     howmuch)
+{
+       struct xfs_buf          *agbp = cur->bc_ag.agbp;
+       struct xfs_agi          *agi = agbp->b_addr;
+
+       if (!xfs_sb_version_hasinobtcounts(&cur->bc_mp->m_sb))
+               return;
+
+       if (cur->bc_btnum == XFS_BTNUM_FINO)
+               be32_add_cpu(&agi->agi_fblocks, howmuch);
+       else if (cur->bc_btnum == XFS_BTNUM_INO)
+               be32_add_cpu(&agi->agi_iblocks, howmuch);
+       xfs_ialloc_log_agi(cur->bc_tp, agbp, XFS_AGI_IBLOCKS);
+}
+
 STATIC int
 __xfs_inobt_alloc_block(
        struct xfs_btree_cur    *cur,
@@ -102,6 +121,7 @@ __xfs_inobt_alloc_block(
 
        new->s = cpu_to_be32(XFS_FSB_TO_AGBNO(args.mp, args.fsbno));
        *stat = 1;
+       xfs_inobt_mod_blockcount(cur, 1);
        return 0;
 }
 
@@ -134,6 +154,7 @@ __xfs_inobt_free_block(
        struct xfs_buf          *bp,
        enum xfs_ag_resv_type   resv)
 {
+       xfs_inobt_mod_blockcount(cur, -1);
        return xfs_free_extent(cur->bc_tp,
                        XFS_DADDR_TO_FSB(cur->bc_mp, XFS_BUF_ADDR(bp)), 1,
                        &XFS_RMAP_OINFO_INOBT, resv);
index 5f04d8a5ab2a94280bf1e29428cb70cbf28226a6..acb9b737fe6bf5e9c790e630c7dfe4f134d0ecc5 100644 (file)
@@ -23,7 +23,7 @@ xfs_check_ondisk_structs(void)
        XFS_CHECK_STRUCT_SIZE(struct xfs_acl_entry,             12);
        XFS_CHECK_STRUCT_SIZE(struct xfs_agf,                   224);
        XFS_CHECK_STRUCT_SIZE(struct xfs_agfl,                  36);
-       XFS_CHECK_STRUCT_SIZE(struct xfs_agi,                   336);
+       XFS_CHECK_STRUCT_SIZE(struct xfs_agi,                   344);
        XFS_CHECK_STRUCT_SIZE(struct xfs_bmbt_key,              8);
        XFS_CHECK_STRUCT_SIZE(struct xfs_bmbt_rec,              16);
        XFS_CHECK_STRUCT_SIZE(struct xfs_bmdr_block,            4);
index 68ec8db12cc75c4635686593b8f29e210eac286c..e61199a4691c49fe7f550784353a54d536f67a9c 100644 (file)
@@ -1549,6 +1549,10 @@ xfs_fc_fill_super(
                goto out_filestream_unmount;
        }
 
+       if (xfs_sb_version_hasinobtcounts(&mp->m_sb))
+               xfs_warn(mp,
+ "EXPERIMENTAL inode btree counters feature in use. Use at your own risk!");
+
        error = xfs_mountfs(mp);
        if (error)
                goto out_filestream_unmount;