diff options
author | Nick Piggin <npiggin@kernel.dk> | 2011-01-07 01:50:06 -0500 |
---|---|---|
committer | Nick Piggin <npiggin@kernel.dk> | 2011-01-07 01:50:31 -0500 |
commit | 873feea09ebc980cbd3631b767356ce1eee65ec1 (patch) | |
tree | 59a8fce9b138086abee7cf845f62ff70a390cf81 /fs/ocfs2/dcache.c | |
parent | ceb5bdc2d246f6d81cf61ed70f325308a11821d2 (diff) |
fs: dcache per-inode inode alias locking
dcache_inode_lock can be replaced with per-inode locking. Use existing
inode->i_lock for this. This is slightly non-trivial because we sometimes
need to find the inode from the dentry, which requires d_inode to be
stabilised (either with refcount or d_lock).
Signed-off-by: Nick Piggin <npiggin@kernel.dk>
Diffstat (limited to 'fs/ocfs2/dcache.c')
-rw-r--r-- | fs/ocfs2/dcache.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/fs/ocfs2/dcache.c b/fs/ocfs2/dcache.c index 0310b16a7238..6d80ecc7834f 100644 --- a/fs/ocfs2/dcache.c +++ b/fs/ocfs2/dcache.c | |||
@@ -175,7 +175,7 @@ struct dentry *ocfs2_find_local_alias(struct inode *inode, | |||
175 | struct list_head *p; | 175 | struct list_head *p; |
176 | struct dentry *dentry = NULL; | 176 | struct dentry *dentry = NULL; |
177 | 177 | ||
178 | spin_lock(&dcache_inode_lock); | 178 | spin_lock(&inode->i_lock); |
179 | list_for_each(p, &inode->i_dentry) { | 179 | list_for_each(p, &inode->i_dentry) { |
180 | dentry = list_entry(p, struct dentry, d_alias); | 180 | dentry = list_entry(p, struct dentry, d_alias); |
181 | 181 | ||
@@ -193,7 +193,7 @@ struct dentry *ocfs2_find_local_alias(struct inode *inode, | |||
193 | dentry = NULL; | 193 | dentry = NULL; |
194 | } | 194 | } |
195 | 195 | ||
196 | spin_unlock(&dcache_inode_lock); | 196 | spin_unlock(&inode->i_lock); |
197 | 197 | ||
198 | return dentry; | 198 | return dentry; |
199 | } | 199 | } |