Patch "btrfs: don't create or leak aliased root while cleaning up orphans" has been added to the 4.7-stable tree
Subsystems (2)
This is a note to let you know that I've just added the patch titled
btrfs: don't create or leak aliased root while cleaning up orphans
to the 4.7-stable tree which can be found at:
http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary
The filename of the patch is:
btrfs-don-t-create-or-leak-aliased-root-while-cleaning-up-orphans.patch
and it can be found in the queue-4.7 subdirectory.
If you, or anyone else, feels it should not be added to the stable tree, please let <[email protected]> know about it.
Quoted 1 line
From
35bbb97fc898aeb874cb7c8b746f091caa359994Mon Sep 17 00:00:00 2001
From: Jeff Mahoney <[email protected]>
Date: Wed, 17 Aug 2016 21:58:33 -0400
Subject: btrfs: don't create or leak aliased root while cleaning up orphans
From: Jeff Mahoney <[email protected]>
commit 35bbb97fc898aeb874cb7c8b746f091caa359994 upstream.
commit 909c3a22da3 (Btrfs: fix loading of orphan roots leading to BUG_ON) avoids the BUG_ON but can add an aliased root to the dead_roots list or leak the root.
Since we've already been loading roots into the radix tree, we should use it before looking the root up on disk.
--- fs/btrfs/disk-io.c | 4 ++-- fs/btrfs/disk-io.h | 2 ++ fs/btrfs/root-tree.c | 27 ++++++++++++++++++--------- 3 files changed, 22 insertions(+), 11 deletions(-)
fs/btrfs/disk-io.c +2 -2
@@ -1626,8 +1626,8 @@ fail:return ret; } -static struct btrfs_root *btrfs_lookup_fs_root (struct btrfs_fs_info *fs_info, -u64 root_id) +struct btrfs_root *btrfs_lookup_fs_root (struct btrfs_fs_info *fs_info, +u64 root_id) {struct btrfs_root *root;
fs/btrfs/disk-io.h +2 -0
@@ -68,6 +68,8 @@ struct extent_buffer *btrfs_find_tree_blstruct btrfs_root *btrfs_read_fs_root (struct btrfs_root *tree_root,struct btrfs_key *location);int btrfs_init_fs_root (struct btrfs_root *root); +struct btrfs_root *btrfs_lookup_fs_root (struct btrfs_fs_info *fs_info, +u64 root_id);int btrfs_insert_fs_root (struct btrfs_fs_info *fs_info,struct btrfs_root *root);void btrfs_free_fs_roots (struct btrfs_fs_info *fs_info);
fs/btrfs/root-tree.c +18 -9
@@ -272,6 +272,23 @@ int btrfs_find_orphan_roots(struct btrfs root_key.objectid = key.offset; key.offset++; +/* +* The root might have been inserted already, as before we look +* for orphan roots, log replay might have happened, which +* triggers a transaction commit and qgroup accounting, which +* in turn reads and inserts fs roots while doing backref +* walking. +*/ + root =btrfs_lookup_fs_root (tree_root->fs_info, + root_key.objectid); +if (root) { +WARN_ON (!test_bit (BTRFS_ROOT_ORPHAN_ITEM_INSERTED , + &root->state)); +if (btrfs_root_refs (&root->root_item) ==0 ) +btrfs_add_dead_root (root); +continue ; + } + root =btrfs_read_fs_root (tree_root, &root_key); err =PTR_ERR_OR_ZERO (root);if (err && err != -ENOENT ) {
@@ -310,16 +327,8 @@ int btrfs_find_orphan_roots(struct btrfsset_bit (BTRFS_ROOT_ORPHAN_ITEM_INSERTED , &root->state); err =btrfs_insert_fs_root (root->fs_info, root); -/* -* The root might have been inserted already, as before we look -* for orphan roots, log replay might have happened, which -* triggers a transaction commit and qgroup accounting, which -* in turn reads and inserts fs roots while doing backref -* walking. -*/ -if (err == -EEXIST ) - err =0 ;if (err) { +BUG_ON (err == -EEXIST );btrfs_free_fs_root (root);break ; }
Patches currently in stable-queue which might be from [email protected] are
queue-4.7/btrfs-don-t-create-or-leak-aliased-root-while-cleaning-up-orphans.patch queue-4.7/btrfs-waiting-on-qgroup-rescan-should-not-always-be-interruptible.patch queue-4.7/btrfs-properly-track-when-rescan-worker-is-running.patch