]> git.baikalelectronics.ru Git - kernel.git/commit
Revert "vfs: add lookup_hash() helper"
authorLinus Torvalds <torvalds@linux-foundation.org>
Fri, 29 Jul 2016 19:17:52 +0000 (12:17 -0700)
committerLinus Torvalds <torvalds@linux-foundation.org>
Fri, 29 Jul 2016 19:17:52 +0000 (12:17 -0700)
commitec8fb26e9694619489b9d92d7b5c38e56081c0eb
tree8a4813c0adc05186e4fa45465a469400443a8e42
parent7012cc9c166cad91f26452a25420e6bdc54f6980
Revert "vfs: add lookup_hash() helper"

This reverts commit dd84b2a99620c856575ce031a42e5f683340fe7b.

As Miklos points out in commit b790915368c7, the "lookup_hash()" helper
is now unused, and in fact, with the hash salting changes, since the
hash of a dentry name now depends on the directory dentry it is in, the
helper function isn't even really likely to be useful.

So rather than keep it around in case somebody else might end up finding
a use for it, let's just remove the helper and not trick people into
thinking it might be a useful thing.

For example, I had obviously completely missed how the helper didn't
follow the normal dentry hashing patterns, and how the hash salting
patch broke overlayfs.  Things would quietly build and look sane, but
not work.

Suggested-by: Miklos Szeredi <mszeredi@redhat.com>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
fs/namei.c
include/linux/namei.h