]> git.ipfire.org Git - thirdparty/linux.git/commitdiff
xfs: make metabtree reservations global
authorChristoph Hellwig <hch@lst.de>
Sat, 15 Feb 2025 06:48:28 +0000 (07:48 +0100)
committerChristoph Hellwig <hch@lst.de>
Mon, 3 Mar 2025 15:16:43 +0000 (08:16 -0700)
Currently each metabtree inode has it's own space reservation to ensure
it can be expanded to the maximum size, mirroring what is done for the
AG-based btrees.  But unlike the AG-based btrees the metabtree inodes
aren't restricted to allocate from a single AG but can use free space
form the entire file system.  And unlike AG-based btrees where the
required reservation shrinks with the available free space due to this,
the metabtree reservations for the rtrmap and rtfreflink trees are not
bound in any way by the data device free space as they track RT extent
allocations.  This is not very efficient as it requires a large number
of blocks to be set aside that can't be used at all by other btrees.

Switch to a model that uses a global pool instead in preparation for
reducing the amount of reserved space, which now also removes the
overloading of the i_nblocks field for metabtree inodes, which would
create problems if metabtree inodes ever had a big enough xattr fork
to require xattr blocks outside the inode.

Signed-off-by: Christoph Hellwig <hch@lst.de>
Reviewed-by: "Darrick J. Wong" <djwong@kernel.org>
fs/xfs/libxfs/xfs_metafile.c
fs/xfs/libxfs/xfs_metafile.h
fs/xfs/scrub/repair.c
fs/xfs/xfs_fsops.c
fs/xfs/xfs_inode.h
fs/xfs/xfs_mount.h
fs/xfs/xfs_reflink.c
fs/xfs/xfs_rtalloc.c
fs/xfs/xfs_rtalloc.h
fs/xfs/xfs_super.c
fs/xfs/xfs_trace.h

index 7625e694eb8dffbdd39e009d5d9cb9d278c7e83a..88f011750add5c4efc7298a5ddfa5b6310d3c78d 100644 (file)
@@ -21,6 +21,9 @@
 #include "xfs_errortag.h"
 #include "xfs_error.h"
 #include "xfs_alloc.h"
+#include "xfs_rtgroup.h"
+#include "xfs_rtrmap_btree.h"
+#include "xfs_rtrefcount_btree.h"
 
 static const struct {
        enum xfs_metafile_type  mtype;
@@ -74,12 +77,11 @@ xfs_metafile_clear_iflag(
 }
 
 /*
- * Is the amount of space that could be allocated towards a given metadata
- * file at or beneath a certain threshold?
+ * Is the metafile reservations at or beneath a certain threshold?
  */
 static inline bool
 xfs_metafile_resv_can_cover(
-       struct xfs_inode        *ip,
+       struct xfs_mount        *mp,
        int64_t                 rhs)
 {
        /*
@@ -88,43 +90,38 @@ xfs_metafile_resv_can_cover(
         * global free block count.  Take care of the first case to avoid
         * touching the per-cpu counter.
         */
-       if (ip->i_delayed_blks >= rhs)
+       if (mp->m_metafile_resv_avail >= rhs)
                return true;
 
        /*
         * There aren't enough blocks left in the inode's reservation, but it
         * isn't critical unless there also isn't enough free space.
         */
-       return xfs_compare_freecounter(ip->i_mount, XC_FREE_BLOCKS,
-                       rhs - ip->i_delayed_blks, 2048) >= 0;
+       return xfs_compare_freecounter(mp, XC_FREE_BLOCKS,
+                       rhs - mp->m_metafile_resv_avail, 2048) >= 0;
 }
 
 /*
- * Is this metadata file critically low on blocks?  For now we'll define that
- * as the number of blocks we can get our hands on being less than 10% of what
- * we reserved or less than some arbitrary number (maximum btree height).
+ * Is the metafile reservation critically low on blocks?  For now we'll define
+ * that as the number of blocks we can get our hands on being less than 10% of
+ * what we reserved or less than some arbitrary number (maximum btree height).
  */
 bool
 xfs_metafile_resv_critical(
-       struct xfs_inode        *ip)
+       struct xfs_mount        *mp)
 {
-       uint64_t                asked_low_water;
+       ASSERT(xfs_has_metadir(mp));
 
-       if (!ip)
-               return false;
-
-       ASSERT(xfs_is_metadir_inode(ip));
-       trace_xfs_metafile_resv_critical(ip, 0);
+       trace_xfs_metafile_resv_critical(mp, 0);
 
-       if (!xfs_metafile_resv_can_cover(ip, ip->i_mount->m_rtbtree_maxlevels))
+       if (!xfs_metafile_resv_can_cover(mp, mp->m_rtbtree_maxlevels))
                return true;
 
-       asked_low_water = div_u64(ip->i_meta_resv_asked, 10);
-       if (!xfs_metafile_resv_can_cover(ip, asked_low_water))
+       if (!xfs_metafile_resv_can_cover(mp,
+                       div_u64(mp->m_metafile_resv_target, 10)))
                return true;
 
-       return XFS_TEST_ERROR(false, ip->i_mount,
-                       XFS_ERRTAG_METAFILE_RESV_CRITICAL);
+       return XFS_TEST_ERROR(false, mp, XFS_ERRTAG_METAFILE_RESV_CRITICAL);
 }
 
 /* Allocate a block from the metadata file's reservation. */
@@ -133,22 +130,24 @@ xfs_metafile_resv_alloc_space(
        struct xfs_inode        *ip,
        struct xfs_alloc_arg    *args)
 {
+       struct xfs_mount        *mp = ip->i_mount;
        int64_t                 len = args->len;
 
        ASSERT(xfs_is_metadir_inode(ip));
        ASSERT(args->resv == XFS_AG_RESV_METAFILE);
 
-       trace_xfs_metafile_resv_alloc_space(ip, args->len);
+       trace_xfs_metafile_resv_alloc_space(mp, args->len);
 
        /*
         * Allocate the blocks from the metadata inode's block reservation
         * and update the ondisk sb counter.
         */
-       if (ip->i_delayed_blks > 0) {
+       mutex_lock(&mp->m_metafile_resv_lock);
+       if (mp->m_metafile_resv_avail > 0) {
                int64_t         from_resv;
 
-               from_resv = min_t(int64_t, len, ip->i_delayed_blks);
-               ip->i_delayed_blks -= from_resv;
+               from_resv = min_t(int64_t, len, mp->m_metafile_resv_avail);
+               mp->m_metafile_resv_avail -= from_resv;
                xfs_mod_delalloc(ip, 0, -from_resv);
                xfs_trans_mod_sb(args->tp, XFS_TRANS_SB_RES_FDBLOCKS,
                                -from_resv);
@@ -175,6 +174,9 @@ xfs_metafile_resv_alloc_space(
                xfs_trans_mod_sb(args->tp, field, -len);
        }
 
+       mp->m_metafile_resv_used += args->len;
+       mutex_unlock(&mp->m_metafile_resv_lock);
+
        ip->i_nblocks += args->len;
        xfs_trans_log_inode(args->tp, ip, XFS_ILOG_CORE);
 }
@@ -186,26 +188,33 @@ xfs_metafile_resv_free_space(
        struct xfs_trans        *tp,
        xfs_filblks_t           len)
 {
+       struct xfs_mount        *mp = ip->i_mount;
        int64_t                 to_resv;
 
        ASSERT(xfs_is_metadir_inode(ip));
-       trace_xfs_metafile_resv_free_space(ip, len);
+
+       trace_xfs_metafile_resv_free_space(mp, len);
 
        ip->i_nblocks -= len;
        xfs_trans_log_inode(tp, ip, XFS_ILOG_CORE);
 
+       mutex_lock(&mp->m_metafile_resv_lock);
+       mp->m_metafile_resv_used -= len;
+
        /*
         * Add the freed blocks back into the inode's delalloc reservation
         * until it reaches the maximum size.  Update the ondisk fdblocks only.
         */
-       to_resv = ip->i_meta_resv_asked - (ip->i_nblocks + ip->i_delayed_blks);
+       to_resv = mp->m_metafile_resv_target -
+               (mp->m_metafile_resv_used + mp->m_metafile_resv_avail);
        if (to_resv > 0) {
                to_resv = min_t(int64_t, to_resv, len);
-               ip->i_delayed_blks += to_resv;
+               mp->m_metafile_resv_avail += to_resv;
                xfs_mod_delalloc(ip, 0, to_resv);
                xfs_trans_mod_sb(tp, XFS_TRANS_SB_RES_FDBLOCKS, to_resv);
                len -= to_resv;
        }
+       mutex_unlock(&mp->m_metafile_resv_lock);
 
        /*
         * Everything else goes back to the filesystem, so update the in-core
@@ -215,61 +224,96 @@ xfs_metafile_resv_free_space(
                xfs_trans_mod_sb(tp, XFS_TRANS_SB_FDBLOCKS, len);
 }
 
-/* Release a metadata file's space reservation. */
+static void
+__xfs_metafile_resv_free(
+       struct xfs_mount        *mp)
+{
+       if (mp->m_metafile_resv_avail) {
+               xfs_mod_sb_delalloc(mp, -(int64_t)mp->m_metafile_resv_avail);
+               xfs_add_fdblocks(mp, mp->m_metafile_resv_avail);
+       }
+       mp->m_metafile_resv_avail = 0;
+       mp->m_metafile_resv_used = 0;
+       mp->m_metafile_resv_target = 0;
+}
+
+/* Release unused metafile space reservation. */
 void
 xfs_metafile_resv_free(
-       struct xfs_inode        *ip)
+       struct xfs_mount        *mp)
 {
-       /* Non-btree metadata inodes don't need space reservations. */
-       if (!ip || !ip->i_meta_resv_asked)
+       if (!xfs_has_metadir(mp))
                return;
 
-       ASSERT(xfs_is_metadir_inode(ip));
-       trace_xfs_metafile_resv_free(ip, 0);
+       trace_xfs_metafile_resv_free(mp, 0);
 
-       if (ip->i_delayed_blks) {
-               xfs_mod_delalloc(ip, 0, -ip->i_delayed_blks);
-               xfs_add_fdblocks(ip->i_mount, ip->i_delayed_blks);
-               ip->i_delayed_blks = 0;
-       }
-       ip->i_meta_resv_asked = 0;
+       mutex_lock(&mp->m_metafile_resv_lock);
+       __xfs_metafile_resv_free(mp);
+       mutex_unlock(&mp->m_metafile_resv_lock);
 }
 
-/* Set up a metadata file's space reservation. */
+/* Set up a metafile space reservation. */
 int
 xfs_metafile_resv_init(
-       struct xfs_inode        *ip,
-       xfs_filblks_t           ask)
+       struct xfs_mount        *mp)
 {
+       struct xfs_rtgroup      *rtg = NULL;
+       xfs_filblks_t           used = 0, target = 0;
        xfs_filblks_t           hidden_space;
-       xfs_filblks_t           used;
-       int                     error;
+       int                     error = 0;
 
-       if (!ip || ip->i_meta_resv_asked > 0)
+       if (!xfs_has_metadir(mp))
                return 0;
 
-       ASSERT(xfs_is_metadir_inode(ip));
+       /*
+        * Free any previous reservation to have a clean slate.
+        */
+       mutex_lock(&mp->m_metafile_resv_lock);
+       __xfs_metafile_resv_free(mp);
+
+       /*
+        * Currently the only btree metafiles that require reservations are the
+        * rtrmap and the rtrefcount.  Anything new will have to be added here
+        * as well.
+        */
+       while ((rtg = xfs_rtgroup_next(mp, rtg))) {
+               if (xfs_has_rtrmapbt(mp)) {
+                       used += rtg_rmap(rtg)->i_nblocks;
+                       target += xfs_rtrmapbt_calc_reserves(mp);
+               }
+               if (xfs_has_rtreflink(mp)) {
+                       used += rtg_refcount(rtg)->i_nblocks;
+                       target += xfs_rtrefcountbt_calc_reserves(mp);
+               }
+       }
+
+       if (!target)
+               goto out_unlock;
 
        /*
-        * Space taken by all other metadata btrees are accounted on-disk as
+        * Space taken by the per-AG metadata btrees are accounted on-disk as
         * used space.  We therefore only hide the space that is reserved but
         * not used by the trees.
         */
-       used = ip->i_nblocks;
-       if (used > ask)
-               ask = used;
-       hidden_space = ask - used;
+       if (used > target)
+               target = used;
+       hidden_space = target - used;
 
-       error = xfs_dec_fdblocks(ip->i_mount, hidden_space, true);
+       error = xfs_dec_fdblocks(mp, hidden_space, true);
        if (error) {
-               trace_xfs_metafile_resv_init_error(ip, error, _RET_IP_);
-               return error;
+               trace_xfs_metafile_resv_init_error(mp, 0);
+               goto out_unlock;
        }
 
-       xfs_mod_delalloc(ip, 0, hidden_space);
-       ip->i_delayed_blks = hidden_space;
-       ip->i_meta_resv_asked = ask;
+       xfs_mod_sb_delalloc(mp, hidden_space);
+
+       mp->m_metafile_resv_target = target;
+       mp->m_metafile_resv_used = used;
+       mp->m_metafile_resv_avail = hidden_space;
+
+       trace_xfs_metafile_resv_init(mp, target);
 
-       trace_xfs_metafile_resv_init(ip, ask);
-       return 0;
+out_unlock:
+       mutex_unlock(&mp->m_metafile_resv_lock);
+       return error;
 }
index 95af4b52e5a75f8f2b6c228141e53f21f91cf4e2..ae6f9e779b988e3667cc83b723f5e3febf3921ee 100644 (file)
@@ -26,13 +26,13 @@ void xfs_metafile_clear_iflag(struct xfs_trans *tp, struct xfs_inode *ip);
 /* Space reservations for metadata inodes. */
 struct xfs_alloc_arg;
 
-bool xfs_metafile_resv_critical(struct xfs_inode *ip);
+bool xfs_metafile_resv_critical(struct xfs_mount *mp);
 void xfs_metafile_resv_alloc_space(struct xfs_inode *ip,
                struct xfs_alloc_arg *args);
 void xfs_metafile_resv_free_space(struct xfs_inode *ip, struct xfs_trans *tp,
                xfs_filblks_t len);
-void xfs_metafile_resv_free(struct xfs_inode *ip);
-int xfs_metafile_resv_init(struct xfs_inode *ip, xfs_filblks_t ask);
+void xfs_metafile_resv_free(struct xfs_mount *mp);
+int xfs_metafile_resv_init(struct xfs_mount *mp);
 
 /* Code specific to kernel/userspace; must be provided externally. */
 
index 3b5288d3ef4e341ff02e46228657c7a2512b762a..c2705c3cba0d7e3014137362cad36303b061a9ff 100644 (file)
@@ -1386,11 +1386,12 @@ int
 xrep_reset_metafile_resv(
        struct xfs_scrub        *sc)
 {
-       struct xfs_inode        *ip = sc->ip;
+       struct xfs_mount        *mp = sc->mp;
        int64_t                 delta;
        int                     error;
 
-       delta = ip->i_nblocks + ip->i_delayed_blks - ip->i_meta_resv_asked;
+       delta = mp->m_metafile_resv_used + mp->m_metafile_resv_avail -
+               mp->m_metafile_resv_target;
        if (delta == 0)
                return 0;
 
@@ -1401,11 +1402,11 @@ xrep_reset_metafile_resv(
        if (delta > 0) {
                int64_t         give_back;
 
-               give_back = min_t(uint64_t, delta, ip->i_delayed_blks);
+               give_back = min_t(uint64_t, delta, mp->m_metafile_resv_avail);
                if (give_back > 0) {
-                       xfs_mod_delalloc(ip, 0, -give_back);
-                       xfs_add_fdblocks(ip->i_mount, give_back);
-                       ip->i_delayed_blks -= give_back;
+                       xfs_mod_sb_delalloc(mp, -give_back);
+                       xfs_add_fdblocks(mp, give_back);
+                       mp->m_metafile_resv_avail -= give_back;
                }
 
                return 0;
@@ -1413,24 +1414,23 @@ xrep_reset_metafile_resv(
 
        /*
         * Not enough reservation; try to take some blocks from the filesystem
-        * to the metadata inode.  @delta is negative here, so invert the sign.
+        * to the metabtree reservation.
         */
-       delta = -delta;
-       error = xfs_dec_fdblocks(sc->mp, delta, true);
+       delta = -delta; /* delta is negative here, so invert the sign. */
+       error = xfs_dec_fdblocks(mp, delta, true);
        while (error == -ENOSPC) {
                delta--;
                if (delta == 0) {
                        xfs_warn(sc->mp,
-"Insufficient free space to reset space reservation for inode 0x%llx after repair.",
-                                       ip->i_ino);
+"Insufficient free space to reset metabtree reservation after repair.");
                        return 0;
                }
-               error = xfs_dec_fdblocks(sc->mp, delta, true);
+               error = xfs_dec_fdblocks(mp, delta, true);
        }
        if (error)
                return error;
 
-       xfs_mod_delalloc(ip, 0, delta);
-       ip->i_delayed_blks += delta;
+       xfs_mod_sb_delalloc(mp, delta);
+       mp->m_metafile_resv_avail += delta;
        return 0;
 }
index f055aebe4c7a13be1c4d1eff4a527868c0df8a75..b2c7337910116d2b26c0176e21e603d9f6343413 100644 (file)
@@ -24,6 +24,7 @@
 #include "xfs_rtalloc.h"
 #include "xfs_rtrmap_btree.h"
 #include "xfs_rtrefcount_btree.h"
+#include "xfs_metafile.h"
 
 /*
  * Write new AG headers to disk. Non-transactional, but need to be
@@ -561,15 +562,13 @@ xfs_fs_reserve_ag_blocks(
                return error;
        }
 
-       if (xfs_has_realtime(mp)) {
-               err2 = xfs_rt_resv_init(mp);
-               if (err2 && err2 != -ENOSPC) {
-                       xfs_warn(mp,
-               "Error %d reserving realtime metadata reserve pool.", err2);
-                       xfs_force_shutdown(mp, SHUTDOWN_CORRUPT_INCORE);
-               }
+       err2 = xfs_metafile_resv_init(mp);
+       if (err2 && err2 != -ENOSPC) {
+               xfs_warn(mp,
+       "Error %d reserving realtime metadata reserve pool.", err2);
+               xfs_force_shutdown(mp, SHUTDOWN_CORRUPT_INCORE);
 
-               if (err2 && !error)
+               if (!error)
                        error = err2;
        }
 
@@ -585,9 +584,7 @@ xfs_fs_unreserve_ag_blocks(
 {
        struct xfs_perag        *pag = NULL;
 
-       if (xfs_has_realtime(mp))
-               xfs_rt_resv_free(mp);
-
+       xfs_metafile_resv_free(mp);
        while ((pag = xfs_perag_next(mp, pag)))
                xfs_ag_resv_free(pag);
 }
index c08093a65352ecc99ead78ba6d9354334d67b3a0..1648dc5a806882278d7eec1cc7451cc9818332fa 100644 (file)
@@ -25,19 +25,9 @@ struct xfs_dquot;
 typedef struct xfs_inode {
        /* Inode linking and identification information. */
        struct xfs_mount        *i_mount;       /* fs mount struct ptr */
-       union {
-               struct {
-                       struct xfs_dquot *i_udquot;     /* user dquot */
-                       struct xfs_dquot *i_gdquot;     /* group dquot */
-                       struct xfs_dquot *i_pdquot;     /* project dquot */
-               };
-
-               /*
-                * Space that has been set aside to accomodate expansions of a
-                * metadata btree rooted in this file.
-                */
-               uint64_t        i_meta_resv_asked;
-       };
+       struct xfs_dquot        *i_udquot;      /* user dquot */
+       struct xfs_dquot        *i_gdquot;      /* group dquot */
+       struct xfs_dquot        *i_pdquot;      /* project dquot */
 
        /* Inode location stuff */
        xfs_ino_t               i_ino;          /* inode number (agno/agino)*/
index 579eaf09157dccf1a005bcc966d0ba11475daa22..bbcf01555947d32173618e369ebc02ce0eac5843 100644 (file)
@@ -273,6 +273,11 @@ typedef struct xfs_mount {
        atomic_t                m_agirotor;     /* last ag dir inode alloced */
        atomic_t                m_rtgrotor;     /* last rtgroup rtpicked */
 
+       struct mutex            m_metafile_resv_lock;
+       uint64_t                m_metafile_resv_target;
+       uint64_t                m_metafile_resv_used;
+       uint64_t                m_metafile_resv_avail;
+
        /* Memory shrinker to throttle and reprioritize inodegc */
        struct shrinker         *m_inodegc_shrinker;
        /*
@@ -747,5 +752,9 @@ int xfs_add_incompat_log_feature(struct xfs_mount *mp, uint32_t feature);
 bool xfs_clear_incompat_log_features(struct xfs_mount *mp);
 void xfs_mod_delalloc(struct xfs_inode *ip, int64_t data_delta,
                int64_t ind_delta);
+static inline void xfs_mod_sb_delalloc(struct xfs_mount *mp, int64_t delta)
+{
+       percpu_counter_add(&mp->m_delalloc_blks, delta);
+}
 
 #endif /* __XFS_MOUNT_H__ */
index 59f7fc16eb8093ba74a5b31e61372163d58808fe..b21cb0d36dd44b139ab16bb504b02c2903e62299 100644 (file)
@@ -1207,15 +1207,9 @@ xfs_reflink_ag_has_free_space(
        if (!xfs_has_rmapbt(mp))
                return 0;
        if (XFS_IS_REALTIME_INODE(ip)) {
-               struct xfs_rtgroup      *rtg;
-               xfs_rgnumber_t          rgno;
-
-               rgno = xfs_rtb_to_rgno(mp, fsb);
-               rtg = xfs_rtgroup_get(mp, rgno);
-               if (xfs_metafile_resv_critical(rtg_rmap(rtg)))
-                       error = -ENOSPC;
-               xfs_rtgroup_put(rtg);
-               return error;
+               if (xfs_metafile_resv_critical(mp))
+                       return -ENOSPC;
+               return 0;
        }
 
        agno = XFS_FSB_TO_AGNO(mp, fsb);
index 93caf4406402d71124584b41016b3ae46d5a66ef..15cc9c88a9f8a825f314d36e312524b4700b75f5 100644 (file)
@@ -1376,8 +1376,7 @@ xfs_growfs_rt(
                        error = error2;
 
                /* Reset the rt metadata btree space reservations. */
-               xfs_rt_resv_free(mp);
-               error2 = xfs_rt_resv_init(mp);
+               error2 = xfs_metafile_resv_init(mp);
                if (error2 && error2 != -ENOSPC)
                        error = error2;
        }
@@ -1523,46 +1522,6 @@ xfs_rtalloc_reinit_frextents(
        return 0;
 }
 
-/* Free space reservations for rt metadata inodes. */
-void
-xfs_rt_resv_free(
-       struct xfs_mount        *mp)
-{
-       struct xfs_rtgroup      *rtg = NULL;
-       unsigned int            i;
-
-       while ((rtg = xfs_rtgroup_next(mp, rtg))) {
-               for (i = 0; i < XFS_RTGI_MAX; i++)
-                       xfs_metafile_resv_free(rtg->rtg_inodes[i]);
-       }
-}
-
-/* Reserve space for rt metadata inodes' space expansion. */
-int
-xfs_rt_resv_init(
-       struct xfs_mount        *mp)
-{
-       struct xfs_rtgroup      *rtg = NULL;
-       xfs_filblks_t           ask;
-       int                     error = 0;
-
-       while ((rtg = xfs_rtgroup_next(mp, rtg))) {
-               int             err2;
-
-               ask = xfs_rtrmapbt_calc_reserves(mp);
-               err2 = xfs_metafile_resv_init(rtg_rmap(rtg), ask);
-               if (err2 && !error)
-                       error = err2;
-
-               ask = xfs_rtrefcountbt_calc_reserves(mp);
-               err2 = xfs_metafile_resv_init(rtg_refcount(rtg), ask);
-               if (err2 && !error)
-                       error = err2;
-       }
-
-       return error;
-}
-
 /*
  * Read in the bmbt of an rt metadata inode so that we never have to load them
  * at runtime.  This enables the use of shared ILOCKs for rtbitmap scans.  Use
index 0d95b29092c9f30c261faceec455275cf815661e..78a690b489ed59b3541f1b38977ac4f89bc34158 100644 (file)
@@ -34,9 +34,6 @@ int                                   /* error */
 xfs_rtmount_inodes(
        struct xfs_mount        *mp);   /* file system mount structure */
 
-void xfs_rt_resv_free(struct xfs_mount *mp);
-int xfs_rt_resv_init(struct xfs_mount *mp);
-
 /*
  * Grow the realtime area of the filesystem.
  */
@@ -65,8 +62,6 @@ xfs_rtmount_init(
 }
 # define xfs_rtmount_inodes(m)  (((mp)->m_sb.sb_rblocks == 0)? 0 : (-ENOSYS))
 # define xfs_rtunmount_inodes(m)
-# define xfs_rt_resv_free(mp)                          ((void)0)
-# define xfs_rt_resv_init(mp)                          (0)
 
 static inline int
 xfs_growfs_check_rtgeom(const struct xfs_mount *mp,
index 366837e71eebdbf4b72b6c76440a68a0ea97dd93..fc6aab21adc4ad8dc11365974aa6b99ce0496cb0 100644 (file)
@@ -2094,6 +2094,7 @@ xfs_init_fs_context(
        for (i = 0; i < XG_TYPE_MAX; i++)
                xa_init(&mp->m_groups[i].xa);
        mutex_init(&mp->m_growlock);
+       mutex_init(&mp->m_metafile_resv_lock);
        INIT_WORK(&mp->m_flush_inodes_work, xfs_flush_inodes_worker);
        INIT_DELAYED_WORK(&mp->m_reclaim_work, xfs_reclaim_worker);
        mp->m_kobj.kobject.kset = xfs_kset;
index dc4d3e913b172250c5550145f7ce9c3f8338e854..5ec45e5edeb5d4ebedb75e1646e82471fe2ba851 100644 (file)
@@ -5606,11 +5606,10 @@ DEFINE_METADIR_EVENT(xfs_metadir_lookup);
 /* metadata inode space reservations */
 
 DECLARE_EVENT_CLASS(xfs_metafile_resv_class,
-       TP_PROTO(struct xfs_inode *ip, xfs_filblks_t len),
-       TP_ARGS(ip, len),
+       TP_PROTO(struct xfs_mount *mp, xfs_filblks_t len),
+       TP_ARGS(mp, len),
        TP_STRUCT__entry(
                __field(dev_t, dev)
-               __field(xfs_ino_t, ino)
                __field(unsigned long long, freeblks)
                __field(unsigned long long, reserved)
                __field(unsigned long long, asked)
@@ -5618,19 +5617,15 @@ DECLARE_EVENT_CLASS(xfs_metafile_resv_class,
                __field(unsigned long long, len)
        ),
        TP_fast_assign(
-               struct xfs_mount *mp = ip->i_mount;
-
                __entry->dev = mp->m_super->s_dev;
-               __entry->ino = ip->i_ino;
                __entry->freeblks = xfs_sum_freecounter_raw(mp, XC_FREE_BLOCKS);
-               __entry->reserved = ip->i_delayed_blks;
-               __entry->asked = ip->i_meta_resv_asked;
-               __entry->used = ip->i_nblocks;
+               __entry->reserved = mp->m_metafile_resv_avail;
+               __entry->asked = mp->m_metafile_resv_target;
+               __entry->used = mp->m_metafile_resv_used;
                __entry->len = len;
        ),
-       TP_printk("dev %d:%d ino 0x%llx freeblks %llu resv %llu ask %llu used %llu len %llu",
+       TP_printk("dev %d:%d freeblks %llu resv %llu ask %llu used %llu len %llu",
                  MAJOR(__entry->dev), MINOR(__entry->dev),
-                 __entry->ino,
                  __entry->freeblks,
                  __entry->reserved,
                  __entry->asked,
@@ -5639,14 +5634,14 @@ DECLARE_EVENT_CLASS(xfs_metafile_resv_class,
 )
 #define DEFINE_METAFILE_RESV_EVENT(name) \
 DEFINE_EVENT(xfs_metafile_resv_class, name, \
-       TP_PROTO(struct xfs_inode *ip, xfs_filblks_t len), \
-       TP_ARGS(ip, len))
+       TP_PROTO(struct xfs_mount *mp, xfs_filblks_t len), \
+       TP_ARGS(mp, len))
 DEFINE_METAFILE_RESV_EVENT(xfs_metafile_resv_init);
 DEFINE_METAFILE_RESV_EVENT(xfs_metafile_resv_free);
 DEFINE_METAFILE_RESV_EVENT(xfs_metafile_resv_alloc_space);
 DEFINE_METAFILE_RESV_EVENT(xfs_metafile_resv_free_space);
 DEFINE_METAFILE_RESV_EVENT(xfs_metafile_resv_critical);
-DEFINE_INODE_ERROR_EVENT(xfs_metafile_resv_init_error);
+DEFINE_METAFILE_RESV_EVENT(xfs_metafile_resv_init_error);
 
 #ifdef CONFIG_XFS_RT
 TRACE_EVENT(xfs_growfs_check_rtgeom,