diff options
author | Artem Bityutskiy <artem.bityutskiy@linux.intel.com> | 2012-07-12 10:28:44 -0400 |
---|---|---|
committer | Al Viro <viro@zeniv.linux.org.uk> | 2012-07-22 15:58:05 -0400 |
commit | 715189d836ab276b3d0fc114681f12b423686ffa (patch) | |
tree | a78c44275d3094363bf6055f114f6da0c909f85f /fs/hfs/super.c | |
parent | 9e6c5829b07c9ba6668807631914efc557fab059 (diff) |
hfs: push lock_super down
HFS uses 'lock_super()'/'unlock_super()' around 'hfs_mdb_commit()' in order
to serialize MDB (Master Directory Block) changes. Push it down to
'hfs_mdb_commit()' in order to simplify the code a bit.
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/super.c')
-rw-r--r-- | fs/hfs/super.c | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/fs/hfs/super.c b/fs/hfs/super.c index 7b4c537d6e13..f7c06bbf33bc 100644 --- a/fs/hfs/super.c +++ b/fs/hfs/super.c | |||
@@ -50,21 +50,17 @@ MODULE_LICENSE("GPL"); | |||
50 | */ | 50 | */ |
51 | static void hfs_write_super(struct super_block *sb) | 51 | static void hfs_write_super(struct super_block *sb) |
52 | { | 52 | { |
53 | lock_super(sb); | ||
54 | sb->s_dirt = 0; | 53 | sb->s_dirt = 0; |
55 | 54 | ||
56 | /* sync everything to the buffers */ | 55 | /* sync everything to the buffers */ |
57 | if (!(sb->s_flags & MS_RDONLY)) | 56 | if (!(sb->s_flags & MS_RDONLY)) |
58 | hfs_mdb_commit(sb); | 57 | hfs_mdb_commit(sb); |
59 | unlock_super(sb); | ||
60 | } | 58 | } |
61 | 59 | ||
62 | static int hfs_sync_fs(struct super_block *sb, int wait) | 60 | static int hfs_sync_fs(struct super_block *sb, int wait) |
63 | { | 61 | { |
64 | lock_super(sb); | ||
65 | hfs_mdb_commit(sb); | 62 | hfs_mdb_commit(sb); |
66 | sb->s_dirt = 0; | 63 | sb->s_dirt = 0; |
67 | unlock_super(sb); | ||
68 | 64 | ||
69 | return 0; | 65 | return 0; |
70 | } | 66 | } |