btrfs: remove done label in writepage_delalloc
authorNikolay Borisov <nborisov@suse.com>
Thu, 16 Jul 2020 15:17:19 +0000 (18:17 +0300)
committerDavid Sterba <dsterba@suse.com>
Mon, 27 Jul 2020 10:55:45 +0000 (12:55 +0200)
Since there is not common cleanup run after the label it makes it
somewhat redundant.

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/extent_io.c

index e37950a..73c9c59 100644 (file)
@@ -3446,8 +3446,7 @@ static noinline_for_stack int writepage_delalloc(struct btrfs_inode *inode,
                         * started, so we don't want to return > 0 unless
                         * things are going well.
                         */
-                       ret = ret < 0 ? ret : -EIO;
-                       goto done;
+                       return ret < 0 ? ret : -EIO;
                }
                /*
                 * delalloc_end is already one less than the total length, so
@@ -3479,10 +3478,7 @@ static noinline_for_stack int writepage_delalloc(struct btrfs_inode *inode,
                return 1;
        }
 
-       ret = 0;
-
-done:
-       return ret;
+       return 0;
 }
 
 /*