Re: [PATCH v3 20/21] __dentry_kill(): new locking scheme

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

To: Al Viro <viro@xxxxxxxxxxxxxxxxxx>
Subject: Re: [PATCH v3 20/21] __dentry_kill(): new locking scheme
From: Max Kellermann <max.kellermann@xxxxxxxxx>
Date: Mon, 7 Jul 2025 20:11:43 +0200
Cc: linux-fsdevel@xxxxxxxxxxxxxxx, Christian Brauner <brauner@xxxxxxxxxx>, linux-kernel@xxxxxxxxxxxxxxx
In-reply-to: <20250707180026.GG1880847@ZenIV>

On Mon, Jul 7, 2025 at 8:00 PM Al Viro <viro@xxxxxxxxxxxxxxxxxx> wrote:
> Well, it tries to evict all non-busy dentries, along with the ones that
> are only busy due to (non-busy) children, etc.

But why did you add code that keeps looping if a dead/killed dentry
was found, even though there is no code to do anything with such a
dentry?

Follow-Ups:
- Re: [PATCH v3 20/21] __dentry_kill(): new locking scheme
  - From: Al Viro

References:
- [RFC][PATCHSET v3] simplifying fast_dput(), dentry_kill() et.al.
  - From: Al Viro
- [PATCH v3 01/21] switch nfsd_client_rmdir() to use of simple_recursive_removal()
  - From: Al Viro
- [PATCH v3 20/21] __dentry_kill(): new locking scheme
  - From: Al Viro
- Re: [PATCH v3 20/21] __dentry_kill(): new locking scheme
  - From: Max Kellermann
- Re: [PATCH v3 20/21] __dentry_kill(): new locking scheme
  - From: Al Viro
- Re: [PATCH v3 20/21] __dentry_kill(): new locking scheme
  - From: Max Kellermann
- Re: [PATCH v3 20/21] __dentry_kill(): new locking scheme
  - From: Al Viro

Prev by Date: Re: [RFC PATCH v2 18/51] mm: hugetlb: Cleanup interpretation of map_chg_state within alloc_hugetlb_folio()
Next by Date: Re: [PATCH v4 00/15] kunit: Introduce UAPI testing framework
Previous by thread: Re: [PATCH v3 20/21] __dentry_kill(): new locking scheme
Next by thread: Re: [PATCH v3 20/21] __dentry_kill(): new locking scheme
Index(es):
- Date
- Thread

[Index of Archives] [Linux Ext4 Filesystem] [Union Filesystem] [Filesystem Testing] [Ceph Users] [Ecryptfs] [NTFS 3] [AutoFS] [Kernel Newbies] [Share Photos] [Security] [Netfilter] [Bugtraq] [Yosemite News] [MIPS Linux] [ARM Linux] [Linux Security] [Linux Cachefs] [Reiser Filesystem] [Linux RAID] [NTFS 3] [Samba] [Device Mapper] [CEPH Development]