diff options
author | Nick Piggin <npiggin@kernel.dk> | 2011-01-07 01:49:33 -0500 |
---|---|---|
committer | Nick Piggin <npiggin@kernel.dk> | 2011-01-07 01:50:21 -0500 |
commit | da5029563a0a026c64821b09e8e7b4fd81d3fe1b (patch) | |
tree | 5d5618e0cb382390073377b1be7d0aa76879ac54 /fs/ocfs2 | |
parent | b7ab39f631f505edc2bbdb86620d5493f995c9da (diff) |
fs: dcache scale d_unhashed
Protect d_unhashed(dentry) condition with d_lock. This means keeping
DCACHE_UNHASHED bit in synch with hash manipulations.
Signed-off-by: Nick Piggin <npiggin@kernel.dk>
Diffstat (limited to 'fs/ocfs2')
-rw-r--r-- | fs/ocfs2/dcache.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/fs/ocfs2/dcache.c b/fs/ocfs2/dcache.c index 895532ac4d98..35e5f5a9ef59 100644 --- a/fs/ocfs2/dcache.c +++ b/fs/ocfs2/dcache.c | |||
@@ -174,13 +174,16 @@ struct dentry *ocfs2_find_local_alias(struct inode *inode, | |||
174 | list_for_each(p, &inode->i_dentry) { | 174 | list_for_each(p, &inode->i_dentry) { |
175 | dentry = list_entry(p, struct dentry, d_alias); | 175 | dentry = list_entry(p, struct dentry, d_alias); |
176 | 176 | ||
177 | spin_lock(&dentry->d_lock); | ||
177 | if (ocfs2_match_dentry(dentry, parent_blkno, skip_unhashed)) { | 178 | if (ocfs2_match_dentry(dentry, parent_blkno, skip_unhashed)) { |
178 | mlog(0, "dentry found: %.*s\n", | 179 | mlog(0, "dentry found: %.*s\n", |
179 | dentry->d_name.len, dentry->d_name.name); | 180 | dentry->d_name.len, dentry->d_name.name); |
180 | 181 | ||
181 | dget_locked(dentry); | 182 | dget_locked_dlock(dentry); |
183 | spin_unlock(&dentry->d_lock); | ||
182 | break; | 184 | break; |
183 | } | 185 | } |
186 | spin_unlock(&dentry->d_lock); | ||
184 | 187 | ||
185 | dentry = NULL; | 188 | dentry = NULL; |
186 | } | 189 | } |