From 71cf10ce4562ed7b18f3bd44a4e2dfafd0e84c50 Mon Sep 17 00:00:00 2001 From: Al Viro Date: Tue, 26 Aug 2025 16:53:42 -0400 Subject: [PATCH] open_detached_copy(): don't bother with mount_lock_hash() we are holding namespace_sem and a reference to root of tree; iterating through that tree does not need mount_lock. Neither does the insertion into the rbtree of new namespace or incrementing the mount count of that namespace. Reviewed-by: Christian Brauner Signed-off-by: Al Viro --- fs/namespace.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/fs/namespace.c b/fs/namespace.c index 7489c9c716c62..468ae0cb475e8 100644 --- a/fs/namespace.c +++ b/fs/namespace.c @@ -3080,14 +3080,12 @@ static struct file *open_detached_copy(struct path *path, bool recursive) return ERR_CAST(mnt); } - lock_mount_hash(); for (p = mnt; p; p = next_mnt(p, mnt)) { mnt_add_to_ns(ns, p); ns->nr_mounts++; } ns->root = mnt; mntget(&mnt->mnt); - unlock_mount_hash(); namespace_unlock(); mntput(path->mnt); -- 2.47.3