Skip to content

Commit

Permalink
btrfs: use reada direction enum instead of constant value in populate…
Browse files Browse the repository at this point in the history
…_free_space_tree

populate_free_space_tree calls function btrfs_search_slot_for_read with
parameter int find_higher = 1, it means that, if no exact match is
found, then use the next higher item.  So in function
populate_free_space_tree, use READA_FORWARD to read forward ahead.

This also changes the value from READA_BACK to READA_FORWARD, since
according to the logic, it should reada_for_search forward, not
backward.

Signed-off-by: Gu JinXiang <gujx@cn.fujitsu.com>
Reviewed-by: Nikolay Borisov <nborisov@suse.com>
[ update changelog ]
Signed-off-by: David Sterba <dsterba@suse.com>
  • Loading branch information
Gu Jinxiang authored and David Sterba committed Mar 26, 2018
1 parent c1c3fac commit 019599a
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion fs/btrfs/free-space-tree.c
Original file line number Diff line number Diff line change
Expand Up @@ -1071,7 +1071,7 @@ static int populate_free_space_tree(struct btrfs_trans_handle *trans,
path = btrfs_alloc_path();
if (!path)
return -ENOMEM;
path->reada = 1;
path->reada = READA_FORWARD;

path2 = btrfs_alloc_path();
if (!path2) {
Expand Down

0 comments on commit 019599a

Please sign in to comment.