btrfs: move ino_cache_inode dropping out of btrfs_free_fs_root
authorJosef Bacik <josef@toxicpanda.com>
Fri, 14 Feb 2020 21:11:41 +0000 (16:11 -0500)
committerDavid Sterba <dsterba@suse.com>
Mon, 23 Mar 2020 16:01:58 +0000 (17:01 +0100)
We are going to make root life be controlled soley by refcounting, and
inodes will be one of the things that hold a ref on the root.  This
means we need to handle dropping the ino_cache_inode outside of the root
freeing logic, so move it into btrfs_drop_and_free_fs_root() so it is
cleaned up properly on unmount.

Signed-off-by: Josef Bacik <josef@toxicpanda.com>
Reviewed-by: David Sterba <dsterba@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
fs/btrfs/disk-io.c
fs/btrfs/transaction.c

index 56cad6a51a7d06462371172596618f99e0005c36..37418d1d082067a4f4ec7c980fe2daf1e0765ec8 100644 (file)
@@ -3912,12 +3912,15 @@ void btrfs_drop_and_free_fs_root(struct btrfs_fs_info *fs_info,
                __btrfs_remove_free_space_cache(root->free_ino_pinned);
        if (root->free_ino_ctl)
                __btrfs_remove_free_space_cache(root->free_ino_ctl);
+       if (root->ino_cache_inode) {
+               iput(root->ino_cache_inode);
+               root->ino_cache_inode = NULL;
+       }
        btrfs_free_fs_root(root);
 }
 
 void btrfs_free_fs_root(struct btrfs_root *root)
 {
-       iput(root->ino_cache_inode);
        WARN_ON(!RB_EMPTY_ROOT(&root->inode_tree));
        if (root->anon_dev)
                free_anon_bdev(root->anon_dev);
index 5939bca9d5ebd6b480adf51119c884f193a94fdf..d2c667465fd4561e5f78dbbf7b49ade06de2a78e 100644 (file)
@@ -2424,6 +2424,10 @@ int btrfs_clean_one_deleted_snapshot(struct btrfs_root *root)
        btrfs_debug(fs_info, "cleaner removing %llu", root->root_key.objectid);
 
        btrfs_kill_all_delayed_nodes(root);
+       if (root->ino_cache_inode) {
+               iput(root->ino_cache_inode);
+               root->ino_cache_inode = NULL;
+       }
 
        if (btrfs_header_backref_rev(root->node) <
                        BTRFS_MIXED_BACKREF_REV)