btrfs: use for loop in prealloc_file_extent_cluster
authorNikolay Borisov <nborisov@suse.com>
Wed, 17 Jun 2020 09:10:44 +0000 (12:10 +0300)
committerDavid Sterba <dsterba@suse.com>
Mon, 27 Jul 2020 10:55:27 +0000 (12:55 +0200)
This function iterates all extents in the extent cluster, make this
intention obvious by using a for loop. No functional chanes.

Reviewed-by: Johannes Thumshirn <johannes.thumshirn@wdc.com>
Signed-off-by: Nikolay Borisov <nborisov@suse.com>
Reviewed-by: David Sterba <dsterba@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
fs/btrfs/relocation.c

index f7feada..e3f3e2e 100644 (file)
@@ -2580,7 +2580,7 @@ int prealloc_file_extent_cluster(struct inode *inode,
        u64 end;
        u64 offset = BTRFS_I(inode)->index_cnt;
        u64 num_bytes;
-       int nr = 0;
+       int nr;
        int ret = 0;
        u64 prealloc_start = cluster->start - offset;
        u64 prealloc_end = cluster->end - offset;
@@ -2593,7 +2593,7 @@ int prealloc_file_extent_cluster(struct inode *inode,
                return ret;
 
        inode_lock(inode);
-       while (nr < cluster->nr) {
+       for (nr = 0; nr < cluster->nr; nr++) {
                start = cluster->boundary[nr] - offset;
                if (nr + 1 < cluster->nr)
                        end = cluster->boundary[nr + 1] - 1 - offset;
@@ -2609,7 +2609,6 @@ int prealloc_file_extent_cluster(struct inode *inode,
                unlock_extent(&BTRFS_I(inode)->io_tree, start, end);
                if (ret)
                        break;
-               nr++;
        }
        inode_unlock(inode);