aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorPeter Zijlstra <a.p.zijlstra@chello.nl>2007-10-15 08:51:31 -0400
committerPeter Zijlstra <a.p.zijlstra@chello.nl>2007-10-15 08:51:31 -0400
commitd475fd428ce77aa2a8bc650d230e17663a4f49c3 (patch)
tree19ba4a1b688a758b2e674f792e1d605896419cc8
parent58dfe883d3bc3b4c08c53a7f39e2ca3ec84f089e (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.c12
-rw-r--r--include/linux/fs.h5
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
1309extern int get_sb_bdev(struct file_system_type *fs_type, 1314extern int get_sb_bdev(struct file_system_type *fs_type,