]> git.openfabrics.org - ~emulex/infiniband.git/commitdiff
UBIFS: use ro_mount instead of MS_RDONLY
authorArtem Bityutskiy <Artem.Bityutskiy@nokia.com>
Thu, 21 Apr 2011 12:11:33 +0000 (15:11 +0300)
committerArtem Bityutskiy <Artem.Bityutskiy@nokia.com>
Fri, 13 May 2011 16:23:55 +0000 (19:23 +0300)
We have our own flags indicating R/O mode, and c->ro_mode is equivalent
to MS_RDONLY. Let's be consistent and use UBIFS flags everywhere.
This patch is just a minor cleanup.

Additionally, add a comment that we are surprised with VFS behavior -
as a reminder to look at this some day.

Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>
fs/ubifs/file.c

index 3594aae12a88e5324165cfc5343a026c484cdfe9..adce3b7b5a3ed3b2cffcc9e3c64a6f18f373a0b3 100644 (file)
@@ -1312,7 +1312,11 @@ int ubifs_fsync(struct file *file, int datasync)
 
        dbg_gen("syncing inode %lu", inode->i_ino);
 
-       if (inode->i_sb->s_flags & MS_RDONLY)
+       if (c->ro_mount)
+               /*
+                * For some really strange reasons VFS does not filter out
+                * 'fsync()' for R/O mounted file-systems as per 2.6.39.
+                */
                return 0;
 
        /*