BACKPORT: f2fs: do not set compression bit if kernel doesn't support
If kernel doesn't have CONFIG_F2FS_FS_COMPRESSION, a file having FS_COMPR_FL via ioctl(FS_IOC_SETFLAGS) is unaccessible due to f2fs_is_compress_backend_ready(). Let's avoid it. Bug: 240921972 Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org> (cherry picked from commit 376523b97c005fa57bb2f8e70f61ab965f16e160) Change-Id: Ieb0f8945175ea5ccb0060690e882f054360fb8f0
This commit is contained in:

committed by
Treehugger Robot

parent
b0988144b0
commit
f3a311b456
@@ -4210,8 +4210,9 @@ static inline void f2fs_invalidate_compress_pages(struct f2fs_sb_info *sbi,
|
|||||||
#define inc_compr_inode_stat(inode) do { } while (0)
|
#define inc_compr_inode_stat(inode) do { } while (0)
|
||||||
#endif
|
#endif
|
||||||
|
|
||||||
static inline void set_compress_context(struct inode *inode)
|
static inline int set_compress_context(struct inode *inode)
|
||||||
{
|
{
|
||||||
|
#ifdef CONFIG_F2FS_FS_COMPRESSION
|
||||||
struct f2fs_sb_info *sbi = F2FS_I_SB(inode);
|
struct f2fs_sb_info *sbi = F2FS_I_SB(inode);
|
||||||
|
|
||||||
F2FS_I(inode)->i_compress_algorithm =
|
F2FS_I(inode)->i_compress_algorithm =
|
||||||
@@ -4233,6 +4234,10 @@ static inline void set_compress_context(struct inode *inode)
|
|||||||
stat_inc_compr_inode(inode);
|
stat_inc_compr_inode(inode);
|
||||||
inc_compr_inode_stat(inode);
|
inc_compr_inode_stat(inode);
|
||||||
f2fs_mark_inode_dirty_sync(inode, true);
|
f2fs_mark_inode_dirty_sync(inode, true);
|
||||||
|
return 0;
|
||||||
|
#else
|
||||||
|
return -EOPNOTSUPP;
|
||||||
|
#endif
|
||||||
}
|
}
|
||||||
|
|
||||||
static inline bool f2fs_disable_compressed_file(struct inode *inode)
|
static inline bool f2fs_disable_compressed_file(struct inode *inode)
|
||||||
|
@@ -1843,8 +1843,8 @@ static int f2fs_setflags_common(struct inode *inode, u32 iflags, u32 mask)
|
|||||||
return -EINVAL;
|
return -EINVAL;
|
||||||
if (S_ISREG(inode->i_mode) && inode->i_size)
|
if (S_ISREG(inode->i_mode) && inode->i_size)
|
||||||
return -EINVAL;
|
return -EINVAL;
|
||||||
|
if (set_compress_context(inode))
|
||||||
set_compress_context(inode);
|
return -EOPNOTSUPP;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
if ((iflags ^ masked_flags) & F2FS_NOCOMP_FL) {
|
if ((iflags ^ masked_flags) & F2FS_NOCOMP_FL) {
|
||||||
|
Reference in New Issue
Block a user