]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
tracing: Align synth event print fmt
authorDouglas Raillard <douglas.raillard@arm.com>
Wed, 19 Mar 2025 21:50:27 +0000 (21:50 +0000)
committerSteven Rostedt (Google) <rostedt@goodmis.org>
Sun, 23 Mar 2025 12:34:31 +0000 (08:34 -0400)
The vast majority of ftrace event print fmt consist of a space-separated
field=value pair. Synthetic event currently use a comma-separated
field=value pair, which sticks out from events created via more
classical means.

Align the format of synth events so they look just like any other event,
for better consistency and less headache when doing crude text-based
data processing.

Cc: Masami Hiramatsu <mhiramat@kernel.org>
Cc: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
Link: https://lore.kernel.org/20250319215028.1680278-1-douglas.raillard@arm.com
Signed-off-by: Douglas Raillard <douglas.raillard@arm.com>
Signed-off-by: Steven Rostedt (Google) <rostedt@goodmis.org>
kernel/trace/trace_events_synth.c

index e3f7d09e5512084fc045cf7d6637b4792f7c4fbf..07ff8be8267e9e5898188568805e9b1c592c537f 100644 (file)
@@ -612,7 +612,7 @@ static int __set_synth_event_print_fmt(struct synth_event *event,
                fmt = synth_field_fmt(event->fields[i]->type);
                pos += snprintf(buf + pos, LEN_OR_ZERO, "%s=%s%s",
                                event->fields[i]->name, fmt,
-                               i == event->n_fields - 1 ? "" : ", ");
+                               i == event->n_fields - 1 ? "" : " ");
        }
        pos += snprintf(buf + pos, LEN_OR_ZERO, "\"");