Commit 01f6afd0 authored by Chao Yu's avatar Chao Yu Committed by Jaegeuk Kim

f2fs: compress: fix to set zstd compress level correctly

As 5kft reported in [1]:

set_compress_context() should set compress level into .i_compress_flag
for zstd as well as lz4hc, otherwise, zstd compressor will still use
default zstd compress level during compression, fix it.

[1] https://lore.kernel.org/linux-f2fs-devel/8e29f52b-6b0d-45ec-9520-e63eb254287a@www.fastmail.com/T/#u

Fixes: 3fde13f8 ("f2fs: compress: support compress level")
Reported-by: default avatar5kft <5kft@5kft.org>
Signed-off-by: default avatarChao Yu <chao@kernel.org>
Signed-off-by: default avatarJaegeuk Kim <jaegeuk@kernel.org>
parent 07c6b593
...@@ -4142,7 +4142,8 @@ static inline void set_compress_context(struct inode *inode) ...@@ -4142,7 +4142,8 @@ static inline void set_compress_context(struct inode *inode)
1 << COMPRESS_CHKSUM : 0; 1 << COMPRESS_CHKSUM : 0;
F2FS_I(inode)->i_cluster_size = F2FS_I(inode)->i_cluster_size =
1 << F2FS_I(inode)->i_log_cluster_size; 1 << F2FS_I(inode)->i_log_cluster_size;
if (F2FS_I(inode)->i_compress_algorithm == COMPRESS_LZ4 && if ((F2FS_I(inode)->i_compress_algorithm == COMPRESS_LZ4 ||
F2FS_I(inode)->i_compress_algorithm == COMPRESS_ZSTD) &&
F2FS_OPTION(sbi).compress_level) F2FS_OPTION(sbi).compress_level)
F2FS_I(inode)->i_compress_flag |= F2FS_I(inode)->i_compress_flag |=
F2FS_OPTION(sbi).compress_level << F2FS_OPTION(sbi).compress_level <<
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment