Extent type can only be regular/prealloc/inline. The main branch of the
'if' already handles the first two, leaving the 'else' to handle inline.
Furthermore, tree-checker ensures that leaf items are correct.

Signed-off-by: Nikolay Borisov <nbori...@suse.com>
---
 fs/btrfs/inode.c | 10 +++-------
 1 file changed, 3 insertions(+), 7 deletions(-)

diff --git a/fs/btrfs/inode.c b/fs/btrfs/inode.c
index 8e24b7641247..6c3f9f3a7ed1 100644
--- a/fs/btrfs/inode.c
+++ b/fs/btrfs/inode.c
@@ -1502,18 +1502,14 @@ static noinline int run_delalloc_nocow(struct inode 
*inode,
                        if (!btrfs_inc_nocow_writers(fs_info, disk_bytenr))
                                goto out_check;
                        nocow = true;
-               } else if (extent_type == BTRFS_FILE_EXTENT_INLINE) {
-                       extent_end = found_key.offset +
-                               btrfs_file_extent_ram_bytes(leaf, fi);
-                       extent_end = ALIGN(extent_end,
-                                          fs_info->sectorsize);
+               } else {
+                       extent_end = found_key.offset + ram_bytes;
+                       extent_end = ALIGN(extent_end, fs_info->sectorsize);
                        /* Skip extents outside of our requested range */
                        if (extent_end <= start) {
                                path->slots[0]++;
                                goto next_slot;
                        }
-               } else {
-                       BUG();
                }
 out_check:
                /*
-- 
2.17.1

Reply via email to