]> git.baikalelectronics.ru Git - kernel.git/commit
Btrfs: Simplify the logic in alloc_extent_buffer() for existing extent buffer case
authorChandra Seetharaman <sekharan@us.ibm.com>
Mon, 7 Oct 2013 15:45:25 +0000 (10:45 -0500)
committerChris Mason <chris.mason@fusionio.com>
Tue, 12 Nov 2013 02:59:11 +0000 (21:59 -0500)
commit8e065564f87edf23997e2117ad7578494b060ef6
tree0747512f55c254c1102fd878c9fca20c0138ea33
parented9b50ad9b98dd1d7bc9ea0dbd32a9122a6285f4
Btrfs: Simplify the logic in alloc_extent_buffer() for existing extent buffer case

alloc_extent_buffer() uses radix_tree_lookup() when radix_tree_insert()
fails with EEXIST. That part of the code is very similar to the code in
find_extent_buffer(). This patch replaces radix_tree_lookup() and
surrounding code in alloc_extent_buffer() with find_extent_buffer().

Note that radix_tree_lookup() does not need to be protected by
tree->buffer_lock. It is protected by eb->refs.

While at it, this patch
  - changes the other usage of radix_tree_lookup() in alloc_extent_buffer()
    with find_extent_buffer() to reduce redundancy.
  - removes the unused argument 'len' to find_extent_buffer().

Signed-Off-by: Chandra Seetharaman <sekharan@us.ibm.com>
Reviewed-by: Zach Brown <zab@redhat.com>
Signed-off-by: Josef Bacik <jbacik@fusionio.com>
Signed-off-by: Chris Mason <chris.mason@fusionio.com>
fs/btrfs/disk-io.c
fs/btrfs/extent_io.c
fs/btrfs/extent_io.h