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

Steven Rostedt <[email protected]>
Newsgroups gmane.linux.kernel,gmane.linux.kernel.janitors
Message-ID <20260609121348.303ca675@fedora>
On Thu, 4 Jun 2026 14:07:45 +0200
Markus Elfring <[email protected]> wrote:

> --- 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
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.