diff options
author | Nick Piggin <npiggin@kernel.dk> | 2011-01-07 01:49:35 -0500 |
---|---|---|
committer | Nick Piggin <npiggin@kernel.dk> | 2011-01-07 01:50:22 -0500 |
commit | b23fb0a60379a95e10c671f646b259ea2558421e (patch) | |
tree | 7c3644b91241d32fda502a7be0b78e4c225f8091 /fs/exportfs | |
parent | 2fd6b7f50797f2e993eea59e0a0b8c6399c811dc (diff) |
fs: scale inode alias list
Add a new lock, dcache_inode_lock, to protect the inode's i_dentry list
from concurrent modification. d_alias is also protected by d_lock.
Signed-off-by: Nick Piggin <npiggin@kernel.dk>
Diffstat (limited to 'fs/exportfs')
-rw-r--r-- | fs/exportfs/expfs.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/fs/exportfs/expfs.c b/fs/exportfs/expfs.c index 51b304056f10..84b8c460a781 100644 --- a/fs/exportfs/expfs.c +++ b/fs/exportfs/expfs.c | |||
@@ -48,8 +48,10 @@ find_acceptable_alias(struct dentry *result, | |||
48 | return result; | 48 | return result; |
49 | 49 | ||
50 | spin_lock(&dcache_lock); | 50 | spin_lock(&dcache_lock); |
51 | spin_lock(&dcache_inode_lock); | ||
51 | list_for_each_entry(dentry, &result->d_inode->i_dentry, d_alias) { | 52 | list_for_each_entry(dentry, &result->d_inode->i_dentry, d_alias) { |
52 | dget_locked(dentry); | 53 | dget_locked(dentry); |
54 | spin_unlock(&dcache_inode_lock); | ||
53 | spin_unlock(&dcache_lock); | 55 | spin_unlock(&dcache_lock); |
54 | if (toput) | 56 | if (toput) |
55 | dput(toput); | 57 | dput(toput); |
@@ -58,8 +60,10 @@ find_acceptable_alias(struct dentry *result, | |||
58 | return dentry; | 60 | return dentry; |
59 | } | 61 | } |
60 | spin_lock(&dcache_lock); | 62 | spin_lock(&dcache_lock); |
63 | spin_lock(&dcache_inode_lock); | ||
61 | toput = dentry; | 64 | toput = dentry; |
62 | } | 65 | } |
66 | spin_unlock(&dcache_inode_lock); | ||
63 | spin_unlock(&dcache_lock); | 67 | spin_unlock(&dcache_lock); |
64 | 68 | ||
65 | if (toput) | 69 | if (toput) |