Thread (4 messages) 4 messages, 1 author, 2024-02-16

[PATCH v3 0/3] tracing: Changes to saved_cmdlines

DORMANTno replies

From: Steven Rostedt <rostedt@goodmis.org>
Date: 2024-02-16 21:00:13
Also in: lkml

Revision v3 of 2 in this series.

Revisions (2)
  1. v3 current
  2. v4 [diff vs current]
Repost of "tracing: Fix wasted memory in saved_cmdlines logic" (hence the v3:
v2 at: https://lore.kernel.org/linux-trace-kernel/20240213115232.5fd9e611@gandalf.local.home/ (local))
That patch was based on the two other patches in this series, but
I forgot to post those two other patches.

Instead of applying the update to the saved_cmdlines logic on top of the
two phantom patches, apply it first. This way it can be backported
nicely. I put a "Fixes" tag on it but not a Cc stable as it's more of
a nice to have than fixing any bug (besides wasting some memory).

The second two patches simply move code around. The goal is to keep
all the saved_cmdlines logic in one place. Currently it's in two files
(trace.c and trace_sched_switch.c). Since trace.c is awfully large,
move all the code to trace_sched_switch.c, as that's its only purpose
today anyway.

The first patch is the saved_cmdlines update to consolidate memory.

The second patch removes some open coded saved_cmdlines logic in trace.c
into a helper function to make it a cleaner move.

The last patch simply moves the code from trace.c into trace_sched_switch.c

Steven Rostedt (Google) (3):
      tracing: Have saved_cmdlines arrays all in one allocation
      tracing: Move open coded processing of tgid_map into helper function
      tracing: Move saved_cmdline code into trace_sched_switch.c

----
 kernel/trace/trace.c              | 509 +------------------------------------
 kernel/trace/trace.h              |  10 +
 kernel/trace/trace_sched_switch.c | 516 ++++++++++++++++++++++++++++++++++++++
 3 files changed, 529 insertions(+), 506 deletions(-)
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help