On 2018/9/18 14:23, Chengguang Xu wrote:
> Currently we set default value to options before parsing remount options,
> it will cause unexpected change of options which were specified in first
> mount.
Can you check below commit? It looks w/o it we may lose default option after
remount.
498c5e9fcd10 ("f2fs: add default mount options to remount")
Thanks,
>
> Signed-off-by: Chengguang Xu <cgxu...@gmx.com>
> ---
> fs/f2fs/super.c | 2 --
> 1 file changed, 2 deletions(-)
>
> diff --git a/fs/f2fs/super.c b/fs/f2fs/super.c
> index 00d512cd4bf2..89ea19466e80 100644
> --- a/fs/f2fs/super.c
> +++ b/fs/f2fs/super.c
> @@ -1499,8 +1499,6 @@ static int f2fs_remount(struct super_block *sb, int
> *flags, char *data)
> clear_sbi_flag(sbi, SBI_NEED_SB_WRITE);
> }
>
> - default_options(sbi);
> -
> /* parse mount options */
> err = parse_options(sb, data);
> if (err)
>
_______________________________________________
Linux-f2fs-devel mailing list
Linux-f2fs-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel
