]> git.ipfire.org Git - thirdparty/kernel/stable.git/commit
tracing: Do not take trace_event_sem in print_event_fields()
authorSteven Rostedt <rostedt@goodmis.org>
Fri, 2 May 2025 02:41:28 +0000 (22:41 -0400)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 9 May 2025 07:43:52 +0000 (09:43 +0200)
commit8daa71c694b3d1e187e44f8a1acb18a73e2ed324
tree545abc7e890117adf300774f18d5a7afef137ac9
parent75fda0e591680f863ba33bc8e639e5840d1caca6
tracing: Do not take trace_event_sem in print_event_fields()

commit 0a8f11f8569e7ed16cbcedeb28c4350f6378fea6 upstream.

On some paths in print_event_fields() it takes the trace_event_sem for
read, even though it should always be held when the function is called.

Remove the taking of that mutex and add a lockdep_assert_held_read() to
make sure the trace_event_sem is held when print_event_fields() is called.

Cc: stable@vger.kernel.org
Cc: Masami Hiramatsu <mhiramat@kernel.org>
Cc: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
Link: https://lore.kernel.org/20250501224128.0b1f0571@batman.local.home
Fixes: 80a76994b2d88 ("tracing: Add "fields" option to show raw trace event fields")
Reported-by: syzbot+441582c1592938fccf09@syzkaller.appspotmail.com
Closes: https://lore.kernel.org/all/6813ff5e.050a0220.14dd7d.001b.GAE@google.com/
Signed-off-by: Steven Rostedt (Google) <rostedt@goodmis.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
kernel/trace/trace_output.c