Hi, Chao

On 2018/7/26 7:19, Chao Yu wrote:
From: Chao Yu <yuch...@huawei.com>

Like quota_ino feature, we need to reject mounting RDWR with image
which enables project_quota feature when there is no CONFIG_QUOTA
be set in kernel.

Signed-off-by: Chao Yu <yuch...@huawei.com>
---
  fs/f2fs/super.c | 6 ++++++
  1 file changed, 6 insertions(+)

diff --git a/fs/f2fs/super.c b/fs/f2fs/super.c
index dbc1cb53581f..bc7c14a055bb 100644
--- a/fs/f2fs/super.c
+++ b/fs/f2fs/super.c
@@ -775,6 +775,12 @@ static int parse_options(struct super_block *sb, char 
*options)
                         "without CONFIG_QUOTA");
                return -EINVAL;
        }

I think the following should be checked within !CONFIG_QUOTA, like patch "f2fs: 
quota: do
not mount as RDWR without QUOTA if quota feature enabled" :)

thanks

+       if (f2fs_sb_has_project_quota(sbi->sb) && !f2fs_readonly(sbi->sb)) {
+               f2fs_msg(sb, KERN_ERR,
+                       "Filesystem with project quota feature cannot be "
+                       "mounted RDWR without CONFIG_QUOTA");
+               return -EINVAL;
+       }
  #endif
if (F2FS_IO_SIZE_BITS(sbi) && !test_opt(sbi, LFS)) {



------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
Linux-f2fs-devel mailing list
Linux-f2fs-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel

Reply via email to