Commit a619b3c7 authored by Robbie Ko's avatar Robbie Ko Committed by David Sterba

btrfs: speedup dead root detection during orphan cleanup

When mounting, we handle deleted subvolume and orphan items.  First,
find add orphan roots, then add them to fs_root radix tree.  Second, in
tree-root, process each orphan item, skip if it is dead root.

The original algorithm is based on the list of dead_roots, one by one to
visit and check whether the objectid is consistent, the time complexity
is O (n ^ 2).  When processing 50000 deleted subvols, it takes about
120s.

Because btrfs_find_orphan_roots has already ran before us, and added
deleted subvol to fs_roots radix tree.

The fs root will only be removed from the fs_roots radix tree after the
cleaner process is started, and the cleaner will only start execution
after the mount is complete.

btrfs_orphan_cleanup can be called during the whole filesystem mount
lifetime, but only "tree root" will be used in this section of code, and
only mount time will be brought into tree root.

So we can quickly check whether the orphan item is dead root through the
fs_roots radix tree.
Reviewed-by: default avatarFilipe Manana <fdmanana@suse.com>
Signed-off-by: default avatarRobbie Ko <robbieko@synology.com>
Signed-off-by: default avatarDavid Sterba <dsterba@suse.com>
parent eec5b6e0
...@@ -2997,18 +2997,16 @@ int btrfs_orphan_cleanup(struct btrfs_root *root) ...@@ -2997,18 +2997,16 @@ int btrfs_orphan_cleanup(struct btrfs_root *root)
* orphan must not get deleted. * orphan must not get deleted.
* find_dead_roots already ran before us, so if this * find_dead_roots already ran before us, so if this
* is a snapshot deletion, we should find the root * is a snapshot deletion, we should find the root
* in the dead_roots list * in the fs_roots radix tree.
*/ */
spin_lock(&fs_info->trans_lock);
list_for_each_entry(dead_root, &fs_info->dead_roots, spin_lock(&fs_info->fs_roots_radix_lock);
root_list) { dead_root = radix_tree_lookup(&fs_info->fs_roots_radix,
if (dead_root->root_key.objectid == (unsigned long)found_key.objectid);
found_key.objectid) { if (dead_root && btrfs_root_refs(&dead_root->root_item) == 0)
is_dead_root = 1; is_dead_root = 1;
break; spin_unlock(&fs_info->fs_roots_radix_lock);
}
}
spin_unlock(&fs_info->trans_lock);
if (is_dead_root) { if (is_dead_root) {
/* prevent this orphan from being found again */ /* prevent this orphan from being found again */
key.offset = found_key.objectid - 1; key.offset = found_key.objectid - 1;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment