Re: [Cluster-devel] [PATCH] fs: push sync_filesystem() down to the file system's remount_fs()

2014-03-13 Thread Steven Whitehouse
Hi, On Thu, 2014-03-13 at 17:23 +0100, Jan Kara wrote: On Thu 13-03-14 10:20:56, Ted Tso wrote: Previously, the no-op mount -o mount /dev/xxx operation when the ^^remount file system is already mounted read-write causes an implied, unconditional

Re: [Cluster-devel] [PATCH] fs: push sync_filesystem() down to the file system's remount_fs()

2014-03-13 Thread Theodore Ts'o
On Thu, Mar 13, 2014 at 04:28:23PM +, Steven Whitehouse wrote: I guess the same is true for other file systems which are mounted ro too. So maybe a check for MS_RDONLY before doing the sync in those cases? My original patch moved the sync_filesystem into the check for MS_RDONLY in the