simple_lookup(): just set DCACHE_DONTCACHE
authorAl Viro <viro@zeniv.linux.org.uk>
Mon, 24 Feb 2025 01:04:30 +0000 (20:04 -0500)
committerAl Viro <viro@zeniv.linux.org.uk>
Wed, 11 Jun 2025 17:34:51 +0000 (13:34 -0400)
commita97dc087da06b69ae976765d59810ca705e8dae1
tree75f57b4f443ffcb910d89f979c2c4bf2c4d2208a
parentd9b13cdad80dc11d74408cf201939a946e9303a6
simple_lookup(): just set DCACHE_DONTCACHE

No need to mess with ->d_op at all.  Note that ->d_delete that always
returns 1 is equivalent to having DCACHE_DONTCACHE in ->d_flags.
Later the same thing will be placed into ->s_d_flags of the filesystems
where we want that behaviour for all dentries; then the check in
simple_lookup() will at least get unlikely() slapped on it.

NOTE: there are only two filesystems where
* simple_lookup() might be called
* default ->d_op is non-NULL
* its ->d_delete() doesn't always return 1
If not for those, we could have simple_lookup() just set DCACHE_DONTCACHE
without even looking at ->d_op.  Filesystems in question are btrfs
and tracefs; both have ->d_delete() returning 1 on anything fed to
simple_lookup(), so both would be fine with simple_lookup() setting
DCACHE_DONTCACHE regardless of ->d_op.

IOW, we might want to drop the check for ->d_op in simple_lookup();
it's definitely a separate story, though.

Reviewed-by: Christian Brauner <brauner@kernel.org>
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
fs/libfs.c