diff options
| author | Filipe Manana <fdmanana@suse.com> | 2026-07-06 12:41:33 +0100 |
|---|---|---|
| committer | David Sterba <dsterba@suse.com> | 2026-08-07 19:17:16 +0200 |
| commit | a503aa0610177b0c0098546b68a7ecad69c6448b (patch) | |
| tree | 80e2273186d092fdc1721488dc2994e8a13922aa | |
| parent | 3f950867c307c5413d628a153ac44915bd117ffd (diff) | |
| download | linux-a503aa0610177b0c0098546b68a7ecad69c6448b.tar.gz linux-a503aa0610177b0c0098546b68a7ecad69c6448b.zip | |
btrfs: get rid of useless label in btrfs_create_dio_extent()
There's no point in having a label where under it we do nothing but return
a variable. So remove it and directly return where we used to goto.
Reviewed-by: Johannes Thumshirn <johannes.thumshirn@wdc.com>
Reviewed-by: Qu Wenruo <wqu@suse.com>
Signed-off-by: Filipe Manana <fdmanana@suse.com>
Reviewed-by: David Sterba <dsterba@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
| -rw-r--r-- | fs/btrfs/direct-io.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/fs/btrfs/direct-io.c b/fs/btrfs/direct-io.c index 27d227f5bdc0..ed1779ccb4de 100644 --- a/fs/btrfs/direct-io.c +++ b/fs/btrfs/direct-io.c @@ -150,7 +150,7 @@ static struct extent_map *btrfs_create_dio_extent(struct btrfs_inode *inode, if (type != BTRFS_ORDERED_NOCOW) { em = btrfs_create_io_em(inode, start, file_extent, type); if (IS_ERR(em)) - goto out; + return em; } ordered = btrfs_alloc_ordered_extent(inode, start, file_extent, @@ -167,7 +167,6 @@ static struct extent_map *btrfs_create_dio_extent(struct btrfs_inode *inode, ASSERT(!dio_data->ordered); dio_data->ordered = ordered; } - out: return em; } |