diff options
| author |   Nikolay Borisov <nborisov@suse.com> | 2018-06-27 16:38:22 +0300 |
|---|---|---|
| committer |   David Sterba <dsterba@suse.com> | 2018-08-06 13:12:54 +0200 |
| commit | d64766fdf9cd172f8416d898cc9d5219ed933b96 (patch) | |
| tree | 0c73282c2f98a96b7e89750be2ffcd639357cd9b /lib/test_xarray.c | |
| parent | btrfs: Reword dodgy comments in alloc_extent_buffer (diff) | |
| download | linux-dev-d64766fdf9cd172f8416d898cc9d5219ed933b96.tar.xz linux-dev-d64766fdf9cd172f8416d898cc9d5219ed933b96.zip | |
btrfs: Refactor loop in btrfs_release_extent_buffer_page
The purpose of the function is to free all the pages comprising an
extent buffer. This can be achieved with a simple for loop rather than
the slightly more involved 'do {} while' construct. So rewrite the
loop using a 'for' construct. Additionally we can never have an
extent_buffer that has 0 pages so remove the check for index == 0. No
functional changes.
The reversed order used to have a meaning in the past where the first
page served as a blocking point for several callers. See eg
4f2de97acee6532b36dd6e99 ("Btrfs: set page->private to the eb").
Signed-off-by: Nikolay Borisov <nborisov@suse.com>
Reviewed-by: David Sterba <dsterba@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
Diffstat (limited to 'lib/test_xarray.c')
0 files changed, 0 insertions, 0 deletions