btrfs: Small cleanup for get index_srcdev loop

1: Adjust condition in loop to make less TAB
2: Move btrfs_put_bbio()'s line for combine, and makes logic clean.

Signed-off-by: Zhao Lei <zhaolei@cn.fujitsu.com>
Signed-off-by: Chris Mason <clm@fb.com>

1 files changed, 20 insertions, 22 deletions

diff --git a/fs/btrfs/volumes.c b/fs/btrfs/volumes.c
index 91bd948d600b..7a769ca04f3f 100644
--- a/fs/btrfs/volumes.c
+++ b/fs/btrfs/volumes.c
@@ -5379,35 +5379,33 @@ static int __btrfs_map_block(struct btrfs_fs_info *fs_info, int rw,
                  * target drive.
                  */
                 for (i = 0; i < tmp_num_stripes; i++) {
-                        if (tmp_bbio->stripes[i].dev->devid == srcdev_devid) {
-                                /*
-                                 * In case of DUP, in order to keep it
-                                 * simple, only add the mirror with the
-                                 * lowest physical address
-                                 */
-                                if (found &&
-                                    physical_of_found <=
-                                     tmp_bbio->stripes[i].physical)
-                                        continue;
-                                index_srcdev = i;
-                                found = 1;
-                                physical_of_found =
-                                        tmp_bbio->stripes[i].physical;
-                        }
+                        if (tmp_bbio->stripes[i].dev->devid != srcdev_devid)
+                                continue;
+
+                        /*
+                         * In case of DUP, in order to keep it simple, only add
+                         * the mirror with the lowest physical address
+                         */
+                        if (found &&
+                            physical_of_found <= tmp_bbio->stripes[i].physical)
+                                continue;
+
+                        index_srcdev = i;
+                        found = 1;
+                        physical_of_found = tmp_bbio->stripes[i].physical;
                 }
 
-                if (found) {
-                        mirror_num = index_srcdev + 1;
-                        patch_the_first_stripe_for_dev_replace = 1;
-                        physical_to_patch_in_first_stripe = physical_of_found;
-                } else {
+                btrfs_put_bbio(tmp_bbio);
+
+                if (!found) {
                         WARN_ON(1);
                         ret = -EIO;
-                        btrfs_put_bbio(tmp_bbio);
                         goto out;
                 }
 
-                btrfs_put_bbio(tmp_bbio);
+                mirror_num = index_srcdev + 1;
+                patch_the_first_stripe_for_dev_replace = 1;
+                physical_to_patch_in_first_stripe = physical_of_found;
         } else if (mirror_num > map->num_stripes) {
                 mirror_num = 0;
         }