Commit f5f3bd90 authored by Jaegeuk Kim's avatar Jaegeuk Kim
Browse files

f2fs: set the default compress_level on ioctl 



Otherwise, we'll get a broken inode.

 # touch $FILE
 # f2fs_io setflags compression $FILE
 # f2fs_io set_coption 2 8 $FILE

[  112.227612] F2FS-fs (dm-51): sanity_check_compress_inode: inode (ino=8d3fe) has unsupported compress level: 0, run fsck to fix

Cc: stable@vger.kernel.org
Signed-off-by: default avatarJaegeuk Kim <jaegeuk@kernel.org>
parent 943f7c6f

fs/f2fs/file.c

+9 −0
Original line number Diff line number Diff line
@@ -4005,6 +4005,15 @@ static int f2fs_ioc_set_compress_option(struct file *filp, unsigned long arg) 
	F2FS_I(inode)->i_compress_algorithm = option.algorithm;

	F2FS_I(inode)->i_log_cluster_size = option.log_cluster_size;

	F2FS_I(inode)->i_cluster_size = BIT(option.log_cluster_size);

	/* Set default level */

	if (F2FS_I(inode)->i_compress_algorithm == COMPRESS_ZSTD)

		F2FS_I(inode)->i_compress_level = F2FS_ZSTD_DEFAULT_CLEVEL;

	else

		F2FS_I(inode)->i_compress_level = 0;

	/* Adjust mount option level */

	if (option.algorithm == F2FS_OPTION(sbi).compress_algorithm &&

	    F2FS_OPTION(sbi).compress_level)

		F2FS_I(inode)->i_compress_level = F2FS_OPTION(sbi).compress_level;

	f2fs_mark_inode_dirty_sync(inode, true);



	if (!f2fs_is_compress_backend_ready(inode))