xfs: fold xfs_mount-alloc() into xfs_init_fs_context()
authorIan Kent <raven@themaw.net>
Mon, 4 Nov 2019 21:58:48 +0000 (13:58 -0800)
committerDarrick J. Wong <darrick.wong@oracle.com>
Tue, 5 Nov 2019 16:28:27 +0000 (08:28 -0800)
After switching to use the mount-api the only remaining caller of
xfs_mount_alloc() is xfs_init_fs_context(), so fold xfs_mount_alloc()
into it.

Signed-off-by: Ian Kent <raven@themaw.net>
Reviewed-by: Christoph Hellwig <hch@lst.de>
Reviewed-by: Darrick J. Wong <darrick.wong@oracle.com>
Signed-off-by: Darrick J. Wong <darrick.wong@oracle.com>
fs/xfs/xfs_super.c

index d42c2317db66e8c3803f2901b52e397807874e40..b3188ea49413867df0f6bb315425d4152e38c8b4 100644 (file)
@@ -1096,35 +1096,6 @@ static const struct super_operations xfs_super_operations = {
        .free_cached_objects    = xfs_fs_free_cached_objects,
 };
 
-static struct xfs_mount *
-xfs_mount_alloc(void)
-{
-       struct xfs_mount        *mp;
-
-       mp = kmem_alloc(sizeof(struct xfs_mount), KM_ZERO);
-       if (!mp)
-               return NULL;
-
-       spin_lock_init(&mp->m_sb_lock);
-       spin_lock_init(&mp->m_agirotor_lock);
-       INIT_RADIX_TREE(&mp->m_perag_tree, GFP_ATOMIC);
-       spin_lock_init(&mp->m_perag_lock);
-       mutex_init(&mp->m_growlock);
-       atomic_set(&mp->m_active_trans, 0);
-       INIT_DELAYED_WORK(&mp->m_reclaim_work, xfs_reclaim_worker);
-       INIT_DELAYED_WORK(&mp->m_eofblocks_work, xfs_eofblocks_worker);
-       INIT_DELAYED_WORK(&mp->m_cowblocks_work, xfs_cowblocks_worker);
-       mp->m_kobj.kobject.kset = xfs_kset;
-       /*
-        * We don't create the finobt per-ag space reservation until after log
-        * recovery, so we must set this to true so that an ifree transaction
-        * started during log recovery will not depend on space reservations
-        * for finobt expansion.
-        */
-       mp->m_finobt_nores = true;
-       return mp;
-}
-
 static int
 suffix_kstrtoint(
        const char      *s,
@@ -1768,10 +1739,28 @@ static int xfs_init_fs_context(
 {
        struct xfs_mount        *mp;
 
-       mp = xfs_mount_alloc();
+       mp = kmem_alloc(sizeof(struct xfs_mount), KM_ZERO);
        if (!mp)
                return -ENOMEM;
 
+       spin_lock_init(&mp->m_sb_lock);
+       spin_lock_init(&mp->m_agirotor_lock);
+       INIT_RADIX_TREE(&mp->m_perag_tree, GFP_ATOMIC);
+       spin_lock_init(&mp->m_perag_lock);
+       mutex_init(&mp->m_growlock);
+       atomic_set(&mp->m_active_trans, 0);
+       INIT_DELAYED_WORK(&mp->m_reclaim_work, xfs_reclaim_worker);
+       INIT_DELAYED_WORK(&mp->m_eofblocks_work, xfs_eofblocks_worker);
+       INIT_DELAYED_WORK(&mp->m_cowblocks_work, xfs_cowblocks_worker);
+       mp->m_kobj.kobject.kset = xfs_kset;
+       /*
+        * We don't create the finobt per-ag space reservation until after log
+        * recovery, so we must set this to true so that an ifree transaction
+        * started during log recovery will not depend on space reservations
+        * for finobt expansion.
+        */
+       mp->m_finobt_nores = true;
+
        /*
         * These can be overridden by the mount option parsing.
         */