Re: [PATCH] Btrfs: use next_state in find_first_extent_bit

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




On 2018/8/23 上午3:14, Liu Bo wrote:
> As next_state() is already defined to get the next state, update us to use
> the helper.
> 
> No functional changes.
> 
> Signed-off-by: Liu Bo <bo.liu@xxxxxxxxxxxxxxxxx>

Reviewed-by: Qu Wenruo <wqu@xxxxxxxx>

Thanks,
Qu

> ---
>  fs/btrfs/extent_io.c | 7 +------
>  1 file changed, 1 insertion(+), 6 deletions(-)
> 
> diff --git a/fs/btrfs/extent_io.c b/fs/btrfs/extent_io.c
> index 628f1aef34b0..44397fa909c2 100644
> --- a/fs/btrfs/extent_io.c
> +++ b/fs/btrfs/extent_io.c
> @@ -1424,20 +1424,15 @@ int find_first_extent_bit(struct extent_io_tree *tree, u64 start,
>  			  struct extent_state **cached_state)
>  {
>  	struct extent_state *state;
> -	struct rb_node *n;
>  	int ret = 1;
>  
>  	spin_lock(&tree->lock);
>  	if (cached_state && *cached_state) {
>  		state = *cached_state;
>  		if (state->end == start - 1 && extent_state_in_tree(state)) {
> -			n = rb_next(&state->rb_node);
> -			while (n) {
> -				state = rb_entry(n, struct extent_state,
> -						 rb_node);
> +			while ((state = next_state(state)) != NULL) {
>  				if (state->state & bits)
>  					goto got_it;
> -				n = rb_next(n);
>  			}
>  			free_extent_state(*cached_state);
>  			*cached_state = NULL;
> 

Attachment: signature.asc
Description: OpenPGP digital signature


[Index of Archives]     [Linux Filesystem Development]     [Linux NFS]     [Linux NILFS]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux