From: Filipe Manana <fdmanana@suse.com>
Date: Wed, 2 Apr 2025 12:23:45 +0000 (+0100)
Subject: btrfs: remove redundant check at find_first_extent_bit_state()
X-Git-Tag: block-6.16-20250606~42^2~170
X-Git-Url: https://git.kernel.dk/?a=commitdiff_plain;h=c54c245f809aab1c81c626a4a667faba86b63adf;p=linux-2.6-block.git

btrfs: remove redundant check at find_first_extent_bit_state()

The tree_search() function always returns an entry that either contains
the search offset or the first entry in the tree that starts after the
offset. So checking at find_first_extent_bit_state() if the returned
entry ends at or after the search offset is pointless. Remove the check.

Signed-off-by: Filipe Manana <fdmanana@suse.com>
Reviewed-by: David Sterba <dsterba@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
---

diff --git a/fs/btrfs/extent-io-tree.c b/fs/btrfs/extent-io-tree.c
index 355c24449776..18b10e7ed815 100644
--- a/fs/btrfs/extent-io-tree.c
+++ b/fs/btrfs/extent-io-tree.c
@@ -877,7 +877,7 @@ static struct extent_state *find_first_extent_bit_state(struct extent_io_tree *t
 	 */
 	state = tree_search(tree, start);
 	while (state) {
-		if (state->end >= start && (state->state & bits))
+		if (state->state & bits)
 			return state;
 		state = next_state(state);
 	}