Thread (73 messages) flat view 73 messages, 7 authors, 2021-08-31
STALE1852d

Revision v1 of 3 in this series.

Revisions (3)
  1. v1 current
  2. v2 [diff vs current]
  3. v3 [diff vs current]

[PATCH 0/6] tr2: plug memory leaks + logic errors + Win32 & Linux feature parity

From: Ævar Arnfjörð Bjarmason <hidden>
Date: 2021-08-25 23:19:55

The just-landed 6f64eeab605 (Merge branch
'es/trace2-log-parent-process-name', 2021-08-24) added parent process
name logging, but under Linux we'd only log the immediate parent, and
the full process chain on Windows.

This brings the Linux implementation in parity with the Windows
implementation. As it turns out /proc/<PID>/stat is a bit of a pain to
parse.

This is preceded by some minor memory leak fixes to
es/trace2-log-parent-process-name, and the fixing of a bug where we'd
log the empty string as a parent if we didn't have procfs.

Ævar Arnfjörð Bjarmason (6):
  tr2: remove NEEDSWORK comment for "non-procfs" implementations
  tr2: clarify TRACE2_PROCESS_INFO_EXIT comment under Linux
  tr2: stop leaking "thread_name" memory
  tr2: fix memory leak & logic error in 2f732bf15e6
  tr2: do compiler enum check in trace2_collect_process_info()
  tr2: log N parent process names on Linux

 compat/linux/procinfo.c | 151 +++++++++++++++++++++++++++++++++-------
 trace2/tr2_tls.c        |   1 +
 2 files changed, 128 insertions(+), 24 deletions(-)

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