diff options
| author |   Josef Bacik <jbacik@fb.com> | 2015-03-02 16:37:31 -0500 |
|---|---|---|
| committer | Josef Bacik <jbacik@fb.com> | 2015-03-17 10:56:55 -0400 |
| commit | dcdf7f6ddba006f3482ebee73dfa6b75aec5f07b (patch) | |
| tree | a91a5105b64f8d0c6ea5f653ea0000b295050b2a /fs/btrfs/transaction.c | |
| parent | Btrfs: fix ASSERT(list_empty(&cur_trans->dirty_bgs_list) (diff) | |
| download | linux-dev-dcdf7f6ddba006f3482ebee73dfa6b75aec5f07b.tar.xz linux-dev-dcdf7f6ddba006f3482ebee73dfa6b75aec5f07b.zip | |
Btrfs: prepare block group cache before writing
Writing the block group cache will modify the extent tree quite a bit because it
truncates the old space cache and pre-allocates new stuff. To try and cut down
on the churn lets do the setup dance first, then later on hopefully we can avoid
looping with newly dirtied roots. Thanks,
Signed-off-by: Josef Bacik <jbacik@fb.com>
Diffstat (limited to 'fs/btrfs/transaction.c')
| -rw-r--r-- | fs/btrfs/transaction.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/fs/btrfs/transaction.c b/fs/btrfs/transaction.c index 2fe3ef5e9de3..932709af5163 100644 --- a/fs/btrfs/transaction.c +++ b/fs/btrfs/transaction.c @@ -1025,7 +1025,6 @@ static int update_cowonly_root(struct btrfs_trans_handle *trans, struct btrfs_root *tree_root = root->fs_info->tree_root; old_root_used = btrfs_root_used(&root->root_item); - btrfs_write_dirty_block_groups(trans, root); while (1) { old_root_bytenr = btrfs_root_bytenr(&root->root_item); @@ -1085,6 +1084,10 @@ static noinline int commit_cowonly_roots(struct btrfs_trans_handle *trans, if (ret) return ret; + ret = btrfs_setup_space_cache(trans, root); + if (ret) + return ret; + /* run_qgroups might have added some more refs */ ret = btrfs_run_delayed_refs(trans, root, (unsigned long)-1); if (ret) |