On Fri, Jan 12, 2018 at 07:41:29PM +0530, Chandan Rajendra wrote: > Now that we have all the code to support encryption for block size less > than page size scenario, this commit removes the conditional check in > filesystem mount code. > > Signed-off-by: Chandan Rajendra <chandan@xxxxxxxxxxxxxxxxxx> > --- > fs/ext4/super.c | 9 +-------- > 1 file changed, 1 insertion(+), 8 deletions(-) > > diff --git a/fs/ext4/super.c b/fs/ext4/super.c > index ebb7edb..5a52c98 100644 > --- a/fs/ext4/super.c > +++ b/fs/ext4/super.c > @@ -4138,14 +4138,7 @@ static int ext4_fill_super(struct super_block *sb, void *data, int silent) > } > } > > - if ((DUMMY_ENCRYPTION_ENABLED(sbi) || ext4_has_feature_encrypt(sb)) && > - (blocksize != PAGE_SIZE)) { > - ext4_msg(sb, KERN_ERR, > - "Unsupported blocksize for fs encryption"); > - goto failed_mount_wq; > - } > - > - if (DUMMY_ENCRYPTION_ENABLED(sbi) && !sb_rdonly(sb) && > + if (DUMMY_ENCRYPTION_ENABLED(sbi) && !(sb->s_flags & MS_RDONLY) && > !ext4_has_feature_encrypt(sb)) { > ext4_set_feature_encrypt(sb); > ext4_commit_super(sb, 1); Why change sb_rdonly() to 'sb->s_flags & MS_RDONLY'? Eric