diff options
| author | Linus Torvalds <torvalds@linux-foundation.org> | 2012-05-10 21:19:19 +0200 |
|---|---|---|
| committer | Linus Torvalds <torvalds@linux-foundation.org> | 2012-05-11 04:54:09 +0200 |
| commit | 94753db5ed9ad97582ef453127d9626a7a2be602 (patch) | |
| tree | ebf9b6cce17c3f4348a598988cb398a9807b3db7 /fs/nfs | |
| parent | vfs: remove unnecessary d_unhashed() check from __d_lookup_rcu (diff) | |
| download | linux-94753db5ed9ad97582ef453127d9626a7a2be602.tar.xz linux-94753db5ed9ad97582ef453127d9626a7a2be602.zip | |
vfs: do the careful dentry name access for all dentry_cmp cases
Commit 12f8ad4b0533 ("vfs: clean up __d_lookup_rcu() and dentry_cmp()
interfaces") did the careful ACCESS_ONCE() of the dentry name only for
the word-at-a-time case, even though the issue is generic.
Admittedly I don't really see gcc ever reloading the value in the middle
of the loop, so the ACCESS_ONCE() protects us from a fairly theoretical
issue. But better safe than sorry.
Also, this consolidates the common parts of the word-at-a-time and
bytewise logic, which includes checking the length. We'll be changing
that later.
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'fs/nfs')
0 files changed, 0 insertions, 0 deletions