diff options
| author | Artem Bityutskiy <Artem.Bityutskiy@nokia.com> | 2011-04-21 15:11:33 +0300 |
|---|---|---|
| committer | Artem Bityutskiy <Artem.Bityutskiy@nokia.com> | 2011-05-13 19:23:55 +0300 |
| commit | 3b2f9a019e655f3407e4e69cdbaf8b75699b79a4 (patch) | |
| tree | 6cb40aec6dd430b1af32a4d20ffbe346ea6b9b3f | |
| parent | 1a29af8bd740b2720caeafbffece1c0be6a9d06d (diff) | |
| download | op-kernel-dev-3b2f9a019e655f3407e4e69cdbaf8b75699b79a4.zip op-kernel-dev-3b2f9a019e655f3407e4e69cdbaf8b75699b79a4.tar.gz | |
UBIFS: use ro_mount instead of MS_RDONLY
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>
| -rw-r--r-- | fs/ubifs/file.c | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/fs/ubifs/file.c b/fs/ubifs/file.c index 3594aae..adce3b7 100644 --- a/fs/ubifs/file.c +++ b/fs/ubifs/file.c @@ -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; /* |
