Hi, Chao
On 2018/3/9 20:32, Chao Yu wrote:
On 2018/3/9 15:53, Sheng Yong wrote:
This patch introduces a new mount option `test_dummy_encryption' to
allow fscrypt to create a fake fscrypt context. This is used by xfstests.
It needs to add doc for this new mount option.
Oh. Right, I'll update the doc.
[...]
static int f2fs_tmpfile(struct inode *dir, struct dentry *dentry, umode_t
mode)
{
- if (unlikely(f2fs_cp_error(F2FS_I_SB(dir))))
+ struct f2fs_sb_info *sbi = F2FS_I_SB(dir);
+
+ if (unlikely(f2fs_cp_error(sbi)))
return -EIO;
- if (f2fs_encrypted_inode(dir)) {
+ if (f2fs_encrypted_inode(dir) || DUMMY_ENCRYPTION_ENABLED(sbi)) {
Why we need to add this condition here?
I think it's both OK if we add the check or not. But since commit 304eecc3462e
("f2fs crypto:
check encryption for tmpfile) add f2fs_encrypt_inode() to check encryption
earlier, IMO, we
should do the same check if test_dummy_encryption is enabled according to the
semantic.
int err = fscrypt_get_encryption_info(dir);
if (err)
return err;
[...]
@@ -2620,6 +2642,15 @@ static int f2fs_fill_super(struct super_block *sb, void
*data, int silent)
if (err)
goto free_options;
+ if (DUMMY_ENCRYPTION_ENABLED(sbi) &&
+ (sb_rdonly(sb) || !f2fs_sb_has_encrypt(sb))) {
Use f2fs_readonly() here which can wrap sb_rdonly()?
OK.
+ f2fs_msg(sb, KERN_ERR,
+ "Encrypt feature is off or filesystem is read-only");
+ err = -EINVAL;
+ retry = false;
+ goto free_options;
+ }
How about moving this check into parse_options(), so we can check this condition
in remount_fs() too.
OK.
Thanks,
Sheng
Thanks,
+
sbi->max_file_blocks = max_file_blocks();
sb->s_maxbytes = sbi->max_file_blocks <<
le32_to_cpu(raw_super->log_blocksize);
--
To unsubscribe from this list: send the line "unsubscribe linux-fscrypt" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
.
------------------------------------------------------------------------------
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
