This is a note to let you know that I've just added the patch titled
Revert "configfs: fix a race in configfs_lookup()"
to the 5.10-stable tree which can be found at:
http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary
The filename of the patch is:
revert-configfs-fix-a-race-in-configfs_lookup.patch
and it can be found in the queue-5.10 subdirectory.
If you, or anyone else, feels it should not be added to the stable tree,
please let <stable@xxxxxxxxxxxxxxx> know about it.
>From ef42517c4b0abdd6e2f62072c4730732a1eb949d Mon Sep 17 00:00:00 2001
From: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx>
Date: Thu, 21 Sep 2023 09:14:16 +0200
Subject: Revert "configfs: fix a race in configfs_lookup()"
From: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx>
This reverts commit c709c7ca020a52ef6530733a9a5f014bd6b71937 which is
commit c42dd069be8dfc9b2239a5c89e73bbd08ab35de0 upstream.
It was backported incorrectly, and will come back later in a way that
actually works.
Cc: Sishuai Gong <sishuai@xxxxxxxxxx>
Cc: Christoph Hellwig <hch@xxxxxx>
Cc: Kyle Zeng <zengyhkyle@xxxxxxxxx>
Reported-by: Guenter Roeck <linux@xxxxxxxxxxxx>
Link: https://lore.kernel.org/r/27f978f3-11ca-6054-5160-ab66d4c918bc@xxxxxxxxxxxx
Signed-off-by: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx>
---
fs/configfs/dir.c | 2 --
1 file changed, 2 deletions(-)
--- a/fs/configfs/dir.c
+++ b/fs/configfs/dir.c
@@ -479,7 +479,6 @@ static struct dentry * configfs_lookup(s
if (!configfs_dirent_is_ready(parent_sd))
goto out;
- spin_lock(&configfs_dirent_lock);
list_for_each_entry(sd, &parent_sd->s_children, s_sibling) {
if (sd->s_type & CONFIGFS_NOT_PINNED) {
const unsigned char * name = configfs_get_name(sd);
@@ -492,7 +491,6 @@ static struct dentry * configfs_lookup(s
break;
}
}
- spin_unlock(&configfs_dirent_lock);
if (!found) {
/*
Patches currently in stable-queue which might be from gregkh@xxxxxxxxxxxxxxxxxxx are
queue-5.10/revert-configfs-fix-a-race-in-configfs_lookup.patch
