diff options
author | Sun YangKai <sunk67188@gmail.com> | 2025-03-11 16:13:13 +0800 |
---|---|---|
committer | David Sterba <dsterba@suse.com> | 2025-03-18 20:35:52 +0100 |
commit | 10de00c7d4e3a12ffbb996e03aefcad0c107981d (patch) | |
tree | dda7af32a572d38638ddf20493f9c8791b9c566d | |
parent | 140ac522de14c1e44dde9ca69a4a1a853953c891 (diff) | |
download | linux-10de00c7d4e3a12ffbb996e03aefcad0c107981d.tar.gz linux-10de00c7d4e3a12ffbb996e03aefcad0c107981d.tar.bz2 linux-10de00c7d4e3a12ffbb996e03aefcad0c107981d.zip |
btrfs: remove unnecessary btrfs_key local variable in btrfs_search_forward()
The 'found_key' variable was only used to temporarily store the found key
before copying it to 'min_key' at the end of the function when returning
success.
Eliminate the 'found_key' variable, and directly store the key into
'min_key' at the exact loop exit points where ret=0 is set, maintaining
identical functionality.
Signed-off-by: Sun YangKai <sunk67188@gmail.com>
Reviewed-by: David Sterba <dsterba@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
-rw-r--r-- | fs/btrfs/ctree.c | 12 |
1 files changed, 5 insertions, 7 deletions
diff --git a/fs/btrfs/ctree.c b/fs/btrfs/ctree.c index 4d02227e9498..5322df012c29 100644 --- a/fs/btrfs/ctree.c +++ b/fs/btrfs/ctree.c @@ -4607,7 +4607,6 @@ int btrfs_search_forward(struct btrfs_root *root, struct btrfs_key *min_key, u64 min_trans) { struct extent_buffer *cur; - struct btrfs_key found_key; int slot; int sret; u32 nritems; @@ -4643,7 +4642,8 @@ again: goto find_next_key; ret = 0; path->slots[level] = slot; - btrfs_item_key_to_cpu(cur, &found_key, slot); + /* Save our key for returning back. */ + btrfs_item_key_to_cpu(cur, min_key, slot); goto out; } if (sret && slot > 0) @@ -4678,11 +4678,11 @@ find_next_key: goto out; } } - /* save our key for returning back */ - btrfs_node_key_to_cpu(cur, &found_key, slot); path->slots[level] = slot; if (level == path->lowest_level) { ret = 0; + /* Save our key for returning back. */ + btrfs_node_key_to_cpu(cur, min_key, slot); goto out; } cur = btrfs_read_node_slot(cur, slot); @@ -4699,10 +4699,8 @@ find_next_key: } out: path->keep_locks = keep_locks; - if (ret == 0) { + if (ret == 0) btrfs_unlock_up_safe(path, path->lowest_level + 1); - memcpy(min_key, &found_key, sizeof(found_key)); - } return ret; } |