diff options
| author | Sun YangKai <sunk67188@gmail.com> | 2025-03-11 16:13:14 +0800 |
|---|---|---|
| committer | David Sterba <dsterba@suse.com> | 2025-03-18 20:35:52 +0100 |
| commit | 0aaaf10ae9aef82bf6589ced2510ff4e249cb3af (patch) | |
| tree | 34e5fa238176851c2ccbc466bd318fb2d036dfa1 /fs/btrfs | |
| parent | 10de00c7d4e3a12ffbb996e03aefcad0c107981d (diff) | |
btrfs: avoid redundant path slot assignment in btrfs_search_forward()
Move path slot assignment before the condition check to prevent
duplicate assignment. Previously, the slot was set both inside and after
the 'slot >= nritems' block with no change in its value, which is
unnecessary.
Signed-off-by: Sun YangKai <sunk67188@gmail.com>
Reviewed-by: David Sterba <dsterba@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
Diffstat (limited to 'fs/btrfs')
| -rw-r--r-- | fs/btrfs/ctree.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/fs/btrfs/ctree.c b/fs/btrfs/ctree.c index 5322df012c29..a2e7979372cc 100644 --- a/fs/btrfs/ctree.c +++ b/fs/btrfs/ctree.c @@ -4667,8 +4667,8 @@ find_next_key: * we didn't find a candidate key in this node, walk forward * and find another one */ + path->slots[level] = slot; if (slot >= nritems) { - path->slots[level] = slot; sret = btrfs_find_next_key(root, path, min_key, level, min_trans); if (sret == 0) { @@ -4678,7 +4678,6 @@ find_next_key: goto out; } } - path->slots[level] = slot; if (level == path->lowest_level) { ret = 0; /* Save our key for returning back. */ |