Received-SPF: pass (google.com: domain of linux-kernel+bounces-195479-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) client-ip=139.178.88.99;
Date: Thu, 30 May 2024 09:59:53 -0400
From: Steven Rostedt <rostedt@goodmis.org>
To: Ilkka =?UTF-8?B?TmF1bGFww6TDpA==?= <digirigawa@gmail.com>
Cc: "Linux regression tracking (Thorsten Leemhuis)"
 <regressions@leemhuis.info>, Linux regressions mailing list
 <regressions@lists.linux.dev>, stable@vger.kernel.org,
 linux-kernel@vger.kernel.org, linux-trace-kernel@vger.kernel.org
Subject: Re: Bug in Kernel 6.8.x, 6.9.x Causing Trace/Panic During
 Shutdown/Reboot
Message-ID: <20240530095953.0020dff9@rorschach.local.home>
In-Reply-To: <CAE4VaRGRwsp+KuEWtsUCxjEtgv1FO+_Ey1-A9xr-o+chaUeteg@mail.gmail.com>
References: <CAE4VaREzY+a2PvQJYJbfh8DwB4OP7kucZG-e28H22xyWob1w_A@mail.gmail.com>
	<5b79732b-087c-411f-a477-9b837566673e@leemhuis.info>
	<20240527183139.42b6123c@rorschach.local.home>
	<CAE4VaRHaijpV1CC9Jo_Lg4tNQb_+=LTHwygOp5Bm2z5ErVzeow@mail.gmail.com>
	<20240528144743.149e351b@rorschach.local.home>
	<CAE4VaRE3_MYVt+=BGs+WVCmKUiQv0VSKE2NT+JmUPKG0UF+Juw@mail.gmail.com>
	<20240529144757.79d09eeb@rorschach.local.home>
	<20240529154824.2db8133a@rorschach.local.home>
	<CAE4VaRGRwsp+KuEWtsUCxjEtgv1FO+_Ey1-A9xr-o+chaUeteg@mail.gmail.com>
Precedence: bulk
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable

On Thu, 30 May 2024 16:02:37 +0300
Ilkka Naulap=C3=A4=C3=A4 <digirigawa@gmail.com> wrote:

> applied your patch and here's the output.
>=20

Unfortunately, it doesn't give me any new information. I added one more
BUG on, want to try this? Otherwise, I'm pretty much at a lost. :-/

-- Steve

diff --git a/fs/tracefs/inode.c b/fs/tracefs/inode.c
index de5b72216b1a..a090495e78c9 100644
--- a/fs/tracefs/inode.c
+++ b/fs/tracefs/inode.c
@@ -39,13 +39,17 @@ static struct inode *tracefs_alloc_inode(struct super_b=
lock *sb)
 		return NULL;
=20
 	ti->flags =3D 0;
+	ti->magic =3D 20240823;
=20
 	return &ti->vfs_inode;
 }
=20
 static void tracefs_free_inode(struct inode *inode)
 {
-	kmem_cache_free(tracefs_inode_cachep, get_tracefs(inode));
+	struct tracefs_inode *ti =3D get_tracefs(inode);
+
+	BUG_ON(ti->magic !=3D 20240823);
+	kmem_cache_free(tracefs_inode_cachep, ti);
 }
=20
 static ssize_t default_read_file(struct file *file, char __user *buf,
@@ -147,16 +151,6 @@ static const struct inode_operations tracefs_dir_inode=
_operations =3D {
 	.rmdir		=3D tracefs_syscall_rmdir,
 };
=20
-struct inode *tracefs_get_inode(struct super_block *sb)
-{
-	struct inode *inode =3D new_inode(sb);
-	if (inode) {
-		inode->i_ino =3D get_next_ino();
-		inode->i_atime =3D inode->i_mtime =3D inode_set_ctime_current(inode);
-	}
-	return inode;
-}
-
 struct tracefs_mount_opts {
 	kuid_t uid;
 	kgid_t gid;
@@ -384,6 +378,7 @@ static void tracefs_dentry_iput(struct dentry *dentry, =
struct inode *inode)
 		return;
=20
 	ti =3D get_tracefs(inode);
+	BUG_ON(ti->magic !=3D 20240823);
 	if (ti && ti->flags & TRACEFS_EVENT_INODE)
 		eventfs_set_ef_status_free(dentry);
 	iput(inode);
@@ -568,6 +563,18 @@ struct dentry *eventfs_end_creating(struct dentry *den=
try)
 	return dentry;
 }
=20
+struct inode *tracefs_get_inode(struct super_block *sb)
+{
+	struct inode *inode =3D new_inode(sb);
+
+	BUG_ON(sb->s_op !=3D &tracefs_super_operations);
+	if (inode) {
+		inode->i_ino =3D get_next_ino();
+		inode->i_atime =3D inode->i_mtime =3D inode_set_ctime_current(inode);
+	}
+	return inode;
+}
+
 /**
  * tracefs_create_file - create a file in the tracefs filesystem
  * @name: a pointer to a string containing the name of the file to create.
diff --git a/fs/tracefs/internal.h b/fs/tracefs/internal.h
index 69c2b1d87c46..9059b8b11bb6 100644
--- a/fs/tracefs/internal.h
+++ b/fs/tracefs/internal.h
@@ -9,12 +9,15 @@ enum {
 struct tracefs_inode {
 	unsigned long           flags;
 	void                    *private;
+	unsigned long		magic;
 	struct inode            vfs_inode;
 };
=20
 static inline struct tracefs_inode *get_tracefs(const struct inode *inode)
 {
-	return container_of(inode, struct tracefs_inode, vfs_inode);
+	struct tracefs_inode *ti =3D container_of(inode, struct tracefs_inode, vf=
s_inode);
+	BUG_ON(ti->magic !=3D 20240823);
+	return ti;
 }
=20
 struct dentry *tracefs_start_creating(const char *name, struct dentry *par=
ent);