diff options
| author | Pan Chuang <panchuang@vivo.com> | 2025-05-16 11:03:32 +0800 |
|---|---|---|
| committer | David Sterba <dsterba@suse.com> | 2025-07-21 23:53:26 +0200 |
| commit | c52ea14d0544cfcd3c76ac9e3ce8ca16832cc033 (patch) | |
| tree | 49163b4ad604f670a966ed13bcd5d39027af6c82 /fs/btrfs/misc.h | |
| parent | fbec9a5d3e98513ce796194fe8604cade1aa3188 (diff) | |
btrfs: pass struct rb_simple_node pointer directly in rb_simple_insert()
Replace struct embedding with union to enable safe type conversion in
btrfs_backref_node, tree_block and mapping_node.
Adjust function calls to use the new unified API, eliminating redundant
parameters.
Signed-off-by: Pan Chuang <panchuang@vivo.com>
Reviewed-by: David Sterba <dsterba@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
Diffstat (limited to 'fs/btrfs/misc.h')
| -rw-r--r-- | fs/btrfs/misc.h | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/fs/btrfs/misc.h b/fs/btrfs/misc.h index 0d599fd847c9..e28bca1b3de5 100644 --- a/fs/btrfs/misc.h +++ b/fs/btrfs/misc.h @@ -119,8 +119,8 @@ static inline struct rb_node *rb_simple_search_first(const struct rb_root *root, return ret; } -static inline struct rb_node *rb_simple_insert(struct rb_root *root, u64 bytenr, - struct rb_node *node) +static inline struct rb_node *rb_simple_insert(struct rb_root *root, + struct rb_simple_node *simple_node) { struct rb_node **p = &root->rb_node; struct rb_node *parent = NULL; @@ -130,16 +130,16 @@ static inline struct rb_node *rb_simple_insert(struct rb_root *root, u64 bytenr, parent = *p; entry = rb_entry(parent, struct rb_simple_node, rb_node); - if (bytenr < entry->bytenr) + if (simple_node->bytenr < entry->bytenr) p = &(*p)->rb_left; - else if (bytenr > entry->bytenr) + else if (simple_node->bytenr > entry->bytenr) p = &(*p)->rb_right; else return parent; } - rb_link_node(node, parent, p); - rb_insert_color(node, root); + rb_link_node(&simple_node->rb_node, parent, p); + rb_insert_color(&simple_node->rb_node, root); return NULL; } |