fix-collapse-range-only-works-on-extent-based-files

   1 ext4: COLLAPSE_RANGE only works on extent-based files
   2
   3 Unfortunately, we weren't checking to make sure of this the inode was
   4 extent-based before attempt operate on it.  Hilarity ensues.
   5
   6 Signed-off-by: "Theodore Ts'o" <tytso@mit.edu>
   7 Cc: Namjae Jeon <namjae.jeon@samsung.com>
   8 ---
   9  fs/ext4/extents.c | 6 +++---
  10  1 file changed, 3 insertions(+), 3 deletions(-)
  11
  12 diff --git a/fs/ext4/extents.c b/fs/ext4/extents.c
  13 index f24ef86..96e0a4b 100644
  14 --- a/fs/ext4/extents.c
  15 +++ b/fs/ext4/extents.c
  16 @@ -4878,9 +4878,6 @@ long ext4_fallocate(struct file *file, int mode, loff_t offset, loff_t len)
  17         if (mode & FALLOC_FL_PUNCH_HOLE)
  18                 return ext4_punch_hole(inode, offset, len);
  19
  20 -       if (mode & FALLOC_FL_COLLAPSE_RANGE)
  21 -               return ext4_collapse_range(inode, offset, len);
  22 -
  23         ret = ext4_convert_inline_data(inode);
  24         if (ret)
  25                 return ret;
  26 @@ -4892,6 +4889,9 @@ long ext4_fallocate(struct file *file, int mode, loff_t offset, loff_t len)
  27         if (!(ext4_test_inode_flag(inode, EXT4_INODE_EXTENTS)))
  28                 return -EOPNOTSUPP;
  29
  30 +       if (mode & FALLOC_FL_COLLAPSE_RANGE)
  31 +               return ext4_collapse_range(inode, offset, len);
  32 +
  33         if (mode & FALLOC_FL_ZERO_RANGE)
  34                 return ext4_zero_range(file, offset, len, mode);
  35