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