]> git.baikalelectronics.ru Git - kernel.git/commitdiff
btrfs: open code rbtree search in split_state
authorDavid Sterba <dsterba@suse.com>
Thu, 25 Jun 2020 14:49:48 +0000 (16:49 +0200)
committerDavid Sterba <dsterba@suse.com>
Mon, 25 Jul 2022 15:45:35 +0000 (17:45 +0200)
Preparatory work to remove tree_insert from extent_io.c, the rbtree
search loop is a known and simple so it can be open coded.

Signed-off-by: David Sterba <dsterba@suse.com>
fs/btrfs/extent_io.c

index 3395474d51a3f77e5e3a050fb98ef8bd19dff789..4b3004874a964de0bdda7b20bda11c146370c34f 100644 (file)
@@ -607,7 +607,8 @@ static int insert_state(struct extent_io_tree *tree,
 static int split_state(struct extent_io_tree *tree, struct extent_state *orig,
                       struct extent_state *prealloc, u64 split)
 {
-       struct rb_node *node;
+       struct rb_node *parent = NULL;
+       struct rb_node **node;
 
        if (tree->private_data && is_data_inode(tree->private_data))
                btrfs_split_delalloc_extent(tree->private_data, orig, split);
@@ -617,12 +618,27 @@ static int split_state(struct extent_io_tree *tree, struct extent_state *orig,
        prealloc->state = orig->state;
        orig->start = split;
 
-       node = tree_insert(&tree->state, &orig->rb_node, prealloc->end,
-                          &prealloc->rb_node, NULL, NULL);
-       if (node) {
-               free_extent_state(prealloc);
-               return -EEXIST;
+       parent = &orig->rb_node;
+       node = &parent;
+       while (*node) {
+               struct tree_entry *entry;
+
+               parent = *node;
+               entry = rb_entry(parent, struct tree_entry, rb_node);
+
+               if (prealloc->end < entry->start) {
+                       node = &(*node)->rb_left;
+               } else if (prealloc->end > entry->end) {
+                       node = &(*node)->rb_right;
+               } else {
+                       free_extent_state(prealloc);
+                       return -EEXIST;
+               }
        }
+
+       rb_link_node(&prealloc->rb_node, parent, node);
+       rb_insert_color(&prealloc->rb_node, &tree->state);
+
        return 0;
 }