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>
This commit is contained in:
parent
a3742d4828
commit
4527440d5d
|
@ -646,8 +646,7 @@ static int hfs_file_fsync(struct file *filp, loff_t start, loff_t end,
|
|||
sb = inode->i_sb;
|
||||
if (sb->s_dirt) {
|
||||
sb->s_dirt = 0;
|
||||
if (!(sb->s_flags & MS_RDONLY))
|
||||
hfs_mdb_commit(sb);
|
||||
hfs_mdb_commit(sb);
|
||||
}
|
||||
/* .. finally sync the buffers to disk */
|
||||
err = sync_blockdev(sb->s_bdev);
|
||||
|
|
|
@ -260,6 +260,9 @@ void hfs_mdb_commit(struct super_block *sb)
|
|||
{
|
||||
struct hfs_mdb *mdb = HFS_SB(sb)->mdb;
|
||||
|
||||
if (sb->s_flags & MS_RDONLY)
|
||||
return;
|
||||
|
||||
lock_buffer(HFS_SB(sb)->mdb_bh);
|
||||
if (test_and_clear_bit(HFS_FLG_MDB_DIRTY, &HFS_SB(sb)->flags)) {
|
||||
/* These parameters may have been modified, so write them back */
|
||||
|
|
|
@ -53,8 +53,7 @@ static void hfs_write_super(struct super_block *sb)
|
|||
sb->s_dirt = 0;
|
||||
|
||||
/* sync everything to the buffers */
|
||||
if (!(sb->s_flags & MS_RDONLY))
|
||||
hfs_mdb_commit(sb);
|
||||
hfs_mdb_commit(sb);
|
||||
}
|
||||
|
||||
static int hfs_sync_fs(struct super_block *sb, int wait)
|
||||
|
|
Loading…
Reference in New Issue