lkml.org 
[lkml]   [2022]   [Aug]   [15]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 5.18 0954/1095] f2fs: allow compression for mmap files in compress_mode=user
    Date
    From: Sungjong Seo <sj1557.seo@samsung.com>

    [ Upstream commit 66d34fcbbe63ebd8584b792e0d741f6648100894 ]

    Since commit e3c548323d32 ("f2fs: let's allow compression for mmap files"),
    it has been allowed to compress mmap files. However, in compress_mode=user,
    it is not allowed yet. To keep the same concept in both compress_modes,
    f2fs_ioc_(de)compress_file() should also allow it.

    Let's remove checking mmap files in f2fs_ioc_(de)compress_file() so that
    the compression for mmap files is also allowed in compress_mode=user.

    Signed-off-by: Sungjong Seo <sj1557.seo@samsung.com>
    Reviewed-by: Chao Yu <chao@kernel.org>
    Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>
    Signed-off-by: Sasha Levin <sashal@kernel.org>
    ---
    fs/f2fs/file.c | 10 ----------
    1 file changed, 10 deletions(-)

    diff --git a/fs/f2fs/file.c b/fs/f2fs/file.c
    index 863518695ea6..9a676ea080e4 100644
    --- a/fs/f2fs/file.c
    +++ b/fs/f2fs/file.c
    @@ -3914,11 +3914,6 @@ static int f2fs_ioc_decompress_file(struct file *filp, unsigned long arg)
    goto out;
    }

    - if (f2fs_is_mmap_file(inode)) {
    - ret = -EBUSY;
    - goto out;
    - }
    -
    ret = filemap_write_and_wait_range(inode->i_mapping, 0, LLONG_MAX);
    if (ret)
    goto out;
    @@ -3986,11 +3981,6 @@ static int f2fs_ioc_compress_file(struct file *filp, unsigned long arg)
    goto out;
    }

    - if (f2fs_is_mmap_file(inode)) {
    - ret = -EBUSY;
    - goto out;
    - }
    -
    ret = filemap_write_and_wait_range(inode->i_mapping, 0, LLONG_MAX);
    if (ret)
    goto out;
    --
    2.35.1


    \
     
     \ /
      Last update: 2022-09-17 16:26    [W:2.981 / U:0.788 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site