diff options
author | Nick Piggin <npiggin@kernel.dk> | 2011-01-07 01:49:27 -0500 |
---|---|---|
committer | Nick Piggin <npiggin@kernel.dk> | 2011-01-07 01:50:19 -0500 |
commit | 621e155a3591962420eacdd39f6f0aa29ceb221e (patch) | |
tree | 387a9fb396f1bf24514b712c294182e36ba51076 /fs/dcache.c | |
parent | fb2d5b86aff355a27ebfc132d3c99f4a940cc3fe (diff) |
fs: change d_compare for rcu-walk
Change d_compare so it may be called from lock-free RCU lookups. This
does put significant restrictions on what may be done from the callback,
however there don't seem to have been any problems with in-tree fses.
If some strange use case pops up that _really_ cannot cope with the
rcu-walk rules, we can just add new rcu-unaware callbacks, which would
cause name lookup to drop out of rcu-walk mode.
For in-tree filesystems, this is just a mechanical change.
Signed-off-by: Nick Piggin <npiggin@kernel.dk>
Diffstat (limited to 'fs/dcache.c')
-rw-r--r-- | fs/dcache.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/fs/dcache.c b/fs/dcache.c index 814e5f491e9c..7075555fbb04 100644 --- a/fs/dcache.c +++ b/fs/dcache.c | |||
@@ -1437,7 +1437,9 @@ struct dentry * __d_lookup(struct dentry * parent, struct qstr * name) | |||
1437 | */ | 1437 | */ |
1438 | qstr = &dentry->d_name; | 1438 | qstr = &dentry->d_name; |
1439 | if (parent->d_op && parent->d_op->d_compare) { | 1439 | if (parent->d_op && parent->d_op->d_compare) { |
1440 | if (parent->d_op->d_compare(parent, qstr, name)) | 1440 | if (parent->d_op->d_compare(parent, parent->d_inode, |
1441 | dentry, dentry->d_inode, | ||
1442 | qstr->len, qstr->name, name)) | ||
1441 | goto next; | 1443 | goto next; |
1442 | } else { | 1444 | } else { |
1443 | if (qstr->len != len) | 1445 | if (qstr->len != len) |