diff options
author | Chris Mason <chris.mason@oracle.com> | 2009-03-13 10:24:59 -0400 |
---|---|---|
committer | Chris Mason <chris.mason@oracle.com> | 2009-03-24 16:14:25 -0400 |
commit | 9fa8cfe706f9c20067c042a064999d5825a35330 (patch) | |
tree | 8e64281caffcdb3ad5a8ab8bf22bb90483e9e214 /fs/btrfs/transaction.c | |
parent | 8e0ee43bc2c3e19db56a4adaa9a9b04ce885cd84 (diff) |
Btrfs: don't preallocate metadata blocks during btrfs_search_slot
In order to avoid doing expensive extent management with tree locks held,
btrfs_search_slot will preallocate tree blocks for use by COW without
any tree locks held.
A later commit moves all of the extent allocation work for COW into
a delayed update mechanism, and this preallocation will no longer be
required.
Signed-off-by: Chris Mason <chris.mason@oracle.com>
Diffstat (limited to 'fs/btrfs/transaction.c')
-rw-r--r-- | fs/btrfs/transaction.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/fs/btrfs/transaction.c b/fs/btrfs/transaction.c index 4112d53d4f4d..d638c54d39e9 100644 --- a/fs/btrfs/transaction.c +++ b/fs/btrfs/transaction.c | |||
@@ -463,7 +463,7 @@ int btrfs_commit_tree_roots(struct btrfs_trans_handle *trans, | |||
463 | btrfs_extent_post_op(trans, fs_info->tree_root); | 463 | btrfs_extent_post_op(trans, fs_info->tree_root); |
464 | 464 | ||
465 | eb = btrfs_lock_root_node(fs_info->tree_root); | 465 | eb = btrfs_lock_root_node(fs_info->tree_root); |
466 | btrfs_cow_block(trans, fs_info->tree_root, eb, NULL, 0, &eb, 0); | 466 | btrfs_cow_block(trans, fs_info->tree_root, eb, NULL, 0, &eb); |
467 | btrfs_tree_unlock(eb); | 467 | btrfs_tree_unlock(eb); |
468 | free_extent_buffer(eb); | 468 | free_extent_buffer(eb); |
469 | 469 | ||
@@ -766,7 +766,7 @@ static noinline int create_pending_snapshot(struct btrfs_trans_handle *trans, | |||
766 | btrfs_set_key_type(&key, BTRFS_ROOT_ITEM_KEY); | 766 | btrfs_set_key_type(&key, BTRFS_ROOT_ITEM_KEY); |
767 | 767 | ||
768 | old = btrfs_lock_root_node(root); | 768 | old = btrfs_lock_root_node(root); |
769 | btrfs_cow_block(trans, root, old, NULL, 0, &old, 0); | 769 | btrfs_cow_block(trans, root, old, NULL, 0, &old); |
770 | 770 | ||
771 | btrfs_copy_root(trans, root, old, &tmp, objectid); | 771 | btrfs_copy_root(trans, root, old, &tmp, objectid); |
772 | btrfs_tree_unlock(old); | 772 | btrfs_tree_unlock(old); |