Commit b7a7eb6e authored by Alexander Viro's avatar Alexander Viro Committed by Linus Torvalds

[PATCH] remount: fs/jffs2

 - jff2->remount_fs() was buggy - it played with sb->s_flags instead of
   doing modifications to *flags (->s_flags will be overwritten using
   *flags right after the call of ->remount_fs()).  Moreover, it tried
   to do the wrong thing - it should just enforce noatime and be done
   with that.  Fixed, ACKed by maintainer.
parent f3745470
...@@ -350,7 +350,7 @@ int jffs2_remount_fs (struct super_block *sb, int *flags, char *data) ...@@ -350,7 +350,7 @@ int jffs2_remount_fs (struct super_block *sb, int *flags, char *data)
if (!(*flags & MS_RDONLY)) if (!(*flags & MS_RDONLY))
jffs2_start_garbage_collect_thread(c); jffs2_start_garbage_collect_thread(c);
sb->s_flags = (sb->s_flags & ~MS_RDONLY)|(*flags & MS_RDONLY); *flags |= MS_NOATIME;
return 0; return 0;
} }
......
...@@ -129,7 +129,7 @@ static struct super_block *jffs2_get_sb_mtd(struct file_system_type *fs_type, ...@@ -129,7 +129,7 @@ static struct super_block *jffs2_get_sb_mtd(struct file_system_type *fs_type,
mtd->index, mtd->name)); mtd->index, mtd->name));
sb->s_op = &jffs2_super_operations; sb->s_op = &jffs2_super_operations;
sb->s_flags |= MS_NODIRATIME; sb->s_flags |= MS_NOATIME;
ret = jffs2_do_fill_super(sb, data, (flags&MS_VERBOSE)?1:0); ret = jffs2_do_fill_super(sb, data, (flags&MS_VERBOSE)?1:0);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment