UbiquitousOS
/
OpenCloudOS-Kernel
mirror of https://gitee.com/OpenCloudOS/OpenCloudOS-Kernel.git
11
0
Fork 0
Code Issues Projects Releases Wiki Activity

btrfs: switch return_bigger to bool in find_ref_head 

Using bool is more suitable than int here, and add the comment about the
return_bigger.

Signed-off-by: Lu Fengqi <lufq.fnst@cn.fujitsu.com>
Reviewed-by: Nikolay Borisov <nborisov@suse.com>
Reviewed-by: David Sterba <dsterba@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
This commit is contained in:
Lu Fengqi 2018-10-11 13:40:38 +08:00 committed by David Sterba
parent 7c8616278b
commit d9352794da
1 changed files with 6 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
fs/btrfs/delayed-ref.c
View File

@ -168,11 +168,12 @@ static struct btrfs_delayed_ref_node* tree_insert(struct rb_root_cached *root,
* find an head entry based on bytenr. This returns the delayed ref * find an head entry based on bytenr. This returns the delayed ref
* head if it was able to find one, or NULL if nothing was in that spot. * head if it was able to find one, or NULL if nothing was in that spot.
* If return_bigger is given, the next bigger entry is returned if no exact * If return_bigger is given, the next bigger entry is returned if no exact
* match is found. * match is found. But if no bigger one is found then the first node of the
* ref head tree will be returned.
*/ */
static struct btrfs_delayed_ref_head* find_ref_head( static struct btrfs_delayed_ref_head* find_ref_head(
struct btrfs_delayed_ref_root *dr, u64 bytenr, struct btrfs_delayed_ref_root *dr, u64 bytenr,
int return_bigger) bool return_bigger)
{ {
struct rb_root *root = &dr->href_root.rb_root; struct rb_root *root = &dr->href_root.rb_root;
struct rb_node *n; struct rb_node *n;
@ -359,12 +360,12 @@ struct btrfs_delayed_ref_head *btrfs_select_ref_head(
again: again:
start = delayed_refs->run_delayed_start; start = delayed_refs->run_delayed_start;
head = find_ref_head(delayed_refs, start, 1); head = find_ref_head(delayed_refs, start, true);
if (!head && !loop) { if (!head && !loop) {
delayed_refs->run_delayed_start = 0; delayed_refs->run_delayed_start = 0;
start = 0; start = 0;
loop = true; loop = true;
head = find_ref_head(delayed_refs, start, 1); head = find_ref_head(delayed_refs, start, true);
if (!head) if (!head)
return NULL; return NULL;
} else if (!head && loop) { } else if (!head && loop) {
@ -905,7 +906,7 @@ int btrfs_add_delayed_extent_op(struct btrfs_fs_info *fs_info,
struct btrfs_delayed_ref_head * struct btrfs_delayed_ref_head *
btrfs_find_delayed_ref_head(struct btrfs_delayed_ref_root *delayed_refs, u64 bytenr) btrfs_find_delayed_ref_head(struct btrfs_delayed_ref_root *delayed_refs, u64 bytenr)
{ {
return find_ref_head(delayed_refs, bytenr, 0); return find_ref_head(delayed_refs, bytenr, false);
} }
void __cold btrfs_delayed_ref_exit(void) void __cold btrfs_delayed_ref_exit(void)