]> 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)
commit153d8c2b10eec14864ea59917a319e3091725d18
tree8a4813c0adc05186e4fa45465a469400443a8e42
parent88c3e13950ca5bc9d4c7508bef93fc867229e030
Revert "vfs: add lookup_hash() helper"

This reverts commit c896636257d233e1fb54356f5974fb1a5286a26e.

As Miklos points out in commit dabf129e5ac2, 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