diff options
| author | Al Viro <viro@zeniv.linux.org.uk> | 2024-02-04 05:53:05 +0100 |
|---|---|---|
| committer | Al Viro <viro@zeniv.linux.org.uk> | 2024-02-10 05:31:16 +0100 |
| commit | 7e4a205fe56b9092f0143dad6aa5fee081139b09 (patch) | |
| tree | 494c45bb1a3783b9bebb2ae87ebb879e1b6fc55b /fs/dcache.c | |
| parent | Linux 6.8-rc1 (diff) | |
| download | linux-7e4a205fe56b9092f0143dad6aa5fee081139b09.tar.xz linux-7e4a205fe56b9092f0143dad6aa5fee081139b09.zip | |
Revert "get rid of DCACHE_GENOCIDE"
This reverts commit 57851607326a2beef21e67f83f4f53a90df8445a.
Unfortunately, while we only call that thing once, the callback
*can* be called more than once for the same dentry - all it
takes is rename_lock being touched while we are in d_walk().
For now let's revert it.
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Diffstat (limited to 'fs/dcache.c')
| -rw-r--r-- | fs/dcache.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/fs/dcache.c b/fs/dcache.c index b813528fb147..6ebccba33336 100644 --- a/fs/dcache.c +++ b/fs/dcache.c @@ -3061,7 +3061,10 @@ static enum d_walk_ret d_genocide_kill(void *data, struct dentry *dentry) if (d_unhashed(dentry) || !dentry->d_inode) return D_WALK_SKIP; - dentry->d_lockref.count--; + if (!(dentry->d_flags & DCACHE_GENOCIDE)) { + dentry->d_flags |= DCACHE_GENOCIDE; + dentry->d_lockref.count--; + } } return D_WALK_CONTINUE; } |