diff options
author | Peter Zijlstra <a.p.zijlstra@chello.nl> | 2007-10-15 08:51:31 -0400 |
---|---|---|
committer | Peter Zijlstra <a.p.zijlstra@chello.nl> | 2007-10-15 08:51:31 -0400 |
commit | d475fd428ce77aa2a8bc650d230e17663a4f49c3 (patch) | |
tree | 19ba4a1b688a758b2e674f792e1d605896419cc8 | |
parent | 58dfe883d3bc3b4c08c53a7f39e2ca3ec84f089e (diff) |
lockdep: per filesystem inode lock class
Give each filesystem its own inode lock class. The various filesystems have
different locking order wrt the inode locks; esp. the pseudo filesystems differ
from the rest.
Signed-off-by: Peter Zijlstra <a.p.zijlstra@chello.nl>
-rw-r--r-- | fs/inode.c | 12 | ||||
-rw-r--r-- | include/linux/fs.h | 5 |
2 files changed, 14 insertions, 3 deletions
diff --git a/fs/inode.c b/fs/inode.c index 29f5068f819b..bf6adf122c68 100644 --- a/fs/inode.c +++ b/fs/inode.c | |||
@@ -142,6 +142,15 @@ static struct inode *alloc_inode(struct super_block *sb) | |||
142 | return NULL; | 142 | return NULL; |
143 | } | 143 | } |
144 | 144 | ||
145 | spin_lock_init(&inode->i_lock); | ||
146 | lockdep_set_class(&inode->i_lock, &sb->s_type->i_lock_key); | ||
147 | |||
148 | mutex_init(&inode->i_mutex); | ||
149 | lockdep_set_class(&inode->i_mutex, &sb->s_type->i_mutex_key); | ||
150 | |||
151 | init_rwsem(&inode->i_alloc_sem); | ||
152 | lockdep_set_class(&inode->i_alloc_sem, &sb->s_type->i_alloc_sem_key); | ||
153 | |||
145 | mapping->a_ops = &empty_aops; | 154 | mapping->a_ops = &empty_aops; |
146 | mapping->host = inode; | 155 | mapping->host = inode; |
147 | mapping->flags = 0; | 156 | mapping->flags = 0; |
@@ -190,8 +199,6 @@ void inode_init_once(struct inode *inode) | |||
190 | INIT_HLIST_NODE(&inode->i_hash); | 199 | INIT_HLIST_NODE(&inode->i_hash); |
191 | INIT_LIST_HEAD(&inode->i_dentry); | 200 | INIT_LIST_HEAD(&inode->i_dentry); |
192 | INIT_LIST_HEAD(&inode->i_devices); | 201 | INIT_LIST_HEAD(&inode->i_devices); |
193 | mutex_init(&inode->i_mutex); | ||
194 | init_rwsem(&inode->i_alloc_sem); | ||
195 | INIT_RADIX_TREE(&inode->i_data.page_tree, GFP_ATOMIC); | 202 | INIT_RADIX_TREE(&inode->i_data.page_tree, GFP_ATOMIC); |
196 | rwlock_init(&inode->i_data.tree_lock); | 203 | rwlock_init(&inode->i_data.tree_lock); |
197 | spin_lock_init(&inode->i_data.i_mmap_lock); | 204 | spin_lock_init(&inode->i_data.i_mmap_lock); |
@@ -199,7 +206,6 @@ void inode_init_once(struct inode *inode) | |||
199 | spin_lock_init(&inode->i_data.private_lock); | 206 | spin_lock_init(&inode->i_data.private_lock); |
200 | INIT_RAW_PRIO_TREE_ROOT(&inode->i_data.i_mmap); | 207 | INIT_RAW_PRIO_TREE_ROOT(&inode->i_data.i_mmap); |
201 | INIT_LIST_HEAD(&inode->i_data.i_mmap_nonlinear); | 208 | INIT_LIST_HEAD(&inode->i_data.i_mmap_nonlinear); |
202 | spin_lock_init(&inode->i_lock); | ||
203 | i_size_ordered_init(inode); | 209 | i_size_ordered_init(inode); |
204 | #ifdef CONFIG_INOTIFY | 210 | #ifdef CONFIG_INOTIFY |
205 | INIT_LIST_HEAD(&inode->inotify_watches); | 211 | INIT_LIST_HEAD(&inode->inotify_watches); |
diff --git a/include/linux/fs.h b/include/linux/fs.h index 16421f662a7a..0cad20e12585 100644 --- a/include/linux/fs.h +++ b/include/linux/fs.h | |||
@@ -1302,8 +1302,13 @@ struct file_system_type { | |||
1302 | struct module *owner; | 1302 | struct module *owner; |
1303 | struct file_system_type * next; | 1303 | struct file_system_type * next; |
1304 | struct list_head fs_supers; | 1304 | struct list_head fs_supers; |
1305 | |||
1305 | struct lock_class_key s_lock_key; | 1306 | struct lock_class_key s_lock_key; |
1306 | struct lock_class_key s_umount_key; | 1307 | struct lock_class_key s_umount_key; |
1308 | |||
1309 | struct lock_class_key i_lock_key; | ||
1310 | struct lock_class_key i_mutex_key; | ||
1311 | struct lock_class_key i_alloc_sem_key; | ||
1307 | }; | 1312 | }; |
1308 | 1313 | ||
1309 | extern int get_sb_bdev(struct file_system_type *fs_type, | 1314 | extern int get_sb_bdev(struct file_system_type *fs_type, |