diff options
| author | Linus Torvalds <torvalds@linux-foundation.org> | 2012-05-10 20:06:18 +0200 |
|---|---|---|
| committer | Linus Torvalds <torvalds@linux-foundation.org> | 2012-05-11 04:52:35 +0200 |
| commit | 8c01a529b861ba97c7d78368e6a5d4d42e946f75 (patch) | |
| tree | 933b5a95604674d0534b0290d8c3633425989f62 | |
| parent | vfs: clean up __d_lookup_rcu() and dentry_cmp() interfaces (diff) | |
| download | linux-8c01a529b861ba97c7d78368e6a5d4d42e946f75.tar.xz linux-8c01a529b861ba97c7d78368e6a5d4d42e946f75.zip | |
vfs: remove unnecessary d_unhashed() check from __d_lookup_rcu
The check for d_unhashed() is not strictly incorrect, but at the same
time it is also not sensible. The actual dentry removal from the dentry
hash chains is totally asynchronous to the __d_lookup_rcu() logic, and
we depend on __d_drop() updating the sequence number to invalidate any
lookup of an unhashed dentry.
So checking d_unhashed() is not incorrect, but it's not useful either:
the code has to work correctly even without it. So just remove it.
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
| -rw-r--r-- | fs/dcache.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/fs/dcache.c b/fs/dcache.c index 539943eb442c..c4d2ff8b4912 100644 --- a/fs/dcache.c +++ b/fs/dcache.c @@ -1871,8 +1871,6 @@ seqretry: seq = raw_seqcount_begin(&dentry->d_seq); if (dentry->d_parent != parent) continue; - if (d_unhashed(dentry)) - continue; *seqp = seq; if (unlikely(parent->d_flags & DCACHE_OP_COMPARE)) { |