diff options
author | Artem Bityutskiy <artem.bityutskiy@linux.intel.com> | 2012-07-12 10:28:47 -0400 |
---|---|---|
committer | Al Viro <viro@zeniv.linux.org.uk> | 2012-07-22 15:58:07 -0400 |
commit | 4527440d5db8ff27ae8801de3f819843a1e6c6f6 (patch) | |
tree | 1e7ddac50528aba33e28941ee086818c35b9fb59 /fs/hfs/inode.c | |
parent | a3742d4828caeffab4cb075b562a4ed92d3e74d6 (diff) |
hfs: simplify a bit checking for R/O
We have the following pattern in 2 places in HFS
if (!RDONLY)
hfs_mdb_commit();
This patch pushes the RDONLY check down to 'hfs_mdb_commit()'. This will
make the following patches a bit simpler.
Signed-off-by: Artem Bityutskiy <artem.bityutskiy@linux.intel.com>
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Diffstat (limited to 'fs/hfs/inode.c')
-rw-r--r-- | fs/hfs/inode.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/fs/hfs/inode.c b/fs/hfs/inode.c index f2deefdb4066..90c1ccbff8e4 100644 --- a/fs/hfs/inode.c +++ b/fs/hfs/inode.c | |||
@@ -646,8 +646,7 @@ static int hfs_file_fsync(struct file *filp, loff_t start, loff_t end, | |||
646 | sb = inode->i_sb; | 646 | sb = inode->i_sb; |
647 | if (sb->s_dirt) { | 647 | if (sb->s_dirt) { |
648 | sb->s_dirt = 0; | 648 | sb->s_dirt = 0; |
649 | if (!(sb->s_flags & MS_RDONLY)) | 649 | hfs_mdb_commit(sb); |
650 | hfs_mdb_commit(sb); | ||
651 | } | 650 | } |
652 | /* .. finally sync the buffers to disk */ | 651 | /* .. finally sync the buffers to disk */ |
653 | err = sync_blockdev(sb->s_bdev); | 652 | err = sync_blockdev(sb->s_bdev); |