Thread (7 messages) 7 messages, 2 authors, 10d ago

Re: [PATCH 1/3] tracing/user_events: Simplify data output in user_seq_show()

From: Steven Rostedt <rostedt@goodmis.org>
Date: 2026-06-09 16:13:53
Also in: kernel-janitors, lkml

On Thu, 4 Jun 2026 14:07:45 +0200
Markus Elfring [off-list ref] wrote:
quoted hunk ↗ jump to hunk
--- a/kernel/trace/trace_events_user.c
+++ b/kernel/trace/trace_events_user.c
@@ -2800,8 +2800,7 @@ static int user_seq_show(struct seq_file *m, void *p)
 
 	mutex_unlock(&group->reg_mutex);
 
-	seq_puts(m, "\n");
-	seq_printf(m, "Active: %d\n", active);
+	seq_printf(m, "\nActive: %d\n", active);
 	seq_printf(m, "Busy: %d\n", busy);
This isn't a critical section and I find the original way easier to
read. But the other two patches are fine.

-- Steve
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help