linux-kernel - [PATCH v3 0/3] tracing: Changes to saved

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [thread-next>] [day] [month] [year] [list]

Message-ID: <20240216210047.584712062@goodmis.org>
Date: Fri, 16 Feb 2024 16:00:47 -0500
From: Steven Rostedt <rostedt@...dmis.org>
To: linux-kernel@...r.kernel.org,
 linux-trace-kernel@...r.kernel.org
Cc: Masami Hiramatsu <mhiramat@...nel.org>,
 Mark Rutland <mark.rutland@....com>,
 Mathieu Desnoyers <mathieu.desnoyers@...icios.com>,
 Andrew Morton <akpm@...ux-foundation.org>,
 Tim Chen <tim.c.chen@...ux.intel.com>,
 Vincent Donnefort <vdonnefort@...gle.com>,
 Sven Schnelle <svens@...ux.ibm.com>,
 Mete Durlu <meted@...ux.ibm.com>
Subject: [PATCH v3 0/3] tracing: Changes to saved_cmdlines

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/)
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(-)