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, |
