]> www.infradead.org Git - users/jedix/linux-maple.git/commit
smb: Log an error when close_all_cached_dirs fails
authorPaul Aurich <paul@darkrain42.org>
Wed, 20 Nov 2024 16:01:54 +0000 (08:01 -0800)
committerSteve French <stfrench@microsoft.com>
Sat, 21 Jun 2025 16:03:25 +0000 (11:03 -0500)
commita2182743a8b4969481f64aec4908ff162e8a206c
tree4404fe8eb454e2ca171d73f5a4a29bf01c3f69a1
parent2c4fd3d141465ef1afc8318c95e7b916d9a8c49c
smb: Log an error when close_all_cached_dirs fails

Under low-memory conditions, close_all_cached_dirs() can't move the
dentries to a separate list to dput() them once the locks are dropped.
This will result in a "Dentry still in use" error, so add an error
message that makes it clear this is what happened:

[  495.281119] CIFS: VFS: \\otters.example.com\share Out of memory while dropping dentries
[  495.281595] ------------[ cut here ]------------
[  495.281887] BUG: Dentry ffff888115531138{i=78,n=/}  still in use (2) [unmount of cifs cifs]
[  495.282391] WARNING: CPU: 1 PID: 2329 at fs/dcache.c:1536 umount_check+0xc8/0xf0

Also, bail out of looping through all tcons as soon as a single
allocation fails, since we're already in trouble, and kmalloc() attempts
for subseqeuent tcons are likely to fail just like the first one did.

Signed-off-by: Paul Aurich <paul@darkrain42.org>
Acked-by: Bharath SM <bharathsm@microsoft.com>
Suggested-by: Ruben Devos <rdevos@oxya.com>
Cc: stable@vger.kernel.org
Signed-off-by: Steve French <stfrench@microsoft.com>
fs/smb/client/cached_dir.c