Skip to content

perf tool: Assorted fixes

Jiri Olsa lkml 3 versions – Applied

Status from this message: Thanks, applied.

Get series with b4
b4 am [email protected]

Versions newest first · linked by author and title

  1. v3 Latest9 patches 10 repliesApplied
  2. v210 patches 2 repliesSuperseded
  3. v117 patches 10 repliesApplied

Changes in v3 from the cover letter

- reworked machine__create_kernel_maps to take into account kernel modules based on Namhyung's comments
- fixed dso__load changes to take into account namespace switch
- add perf test 1 fix

Patches in v3 Applied

#Subject and filesSizeRepliesApplied
1/9tools lib symbol: Skip non-address kallsyms linetools/lib/symbol/kallsyms.c+4 -00 replies
2/9perf tools: Check if we read regular file in dso__loadtools/perf/util/symbol.c+4 -60 replies
3/9perf tools: Free root_dir in machine__init error pathtools/perf/util/machine.c+7 -10 replies
4/9perf tools: Move kernel mmap name into struct machinetools/perf/util/build-id.c tools/perf/util/event.c tools/perf/util/machine.c +2 more+39 -490 replies
5/9perf tools: Generalize machine__set_kernel_mmap functiontools/perf/util/machine.c+7 -60 replies
6/9perf tools: Don't search for active kernel start in __machine__create_kernel_mapstools/perf/util/machine.c tools/perf/util/machine.h+26 -3013 replies
7/9perf tools: Remove machine__load_kallsyms functiontools/perf/tests/vmlinux-kallsyms.c tools/perf/util/machine.c tools/perf/util/machine.h+5 -130 replies
8/9perf tools: Do not create kernel maps in sample__resolvetools/perf/util/event.c+0 -110 replies
9/9perf tests: Use arch__compare_symbol_names to compare symbolstools/perf/tests/vmlinux-kallsyms.c+1 -13 replies

Reviews trailers per patch

1 of 9 patches has a Reviewed-by or Acked-by.

PatchNaveen N. RaoJiri Olsa
1/9
2/9
3/9
4/9
5/9
6/9A-b
7/9
8/9
9/9Acked-and-Tested-by

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v3 thread

Cover letter [PATCHv3 0/9] perf tool: Assorted fixes

hi,
sending assorted general fixes that queued
up in my other branches.

v3 changes:
- reworked machine__create_kernel_maps to take into account kernel modules based on Namhyung's comments
- fixed dso__load changes to take into account namespace switch
- add perf test 1 fix

v2 changes:
- rebased on current perf/core
- detailed changelog for patch 1
- using zfree instead of free in patch 2
- using machine__set_mmap_name function name

Also available in here:
https://git.kernel.org/pub/scm/linux/kernel/git/jolsa/perf.git
perf/fixes

thanks,
jirka

---
Jiri Olsa (9):
tools lib symbol: Skip non-address kallsyms line
perf tools: Check if we read regular file in dso__load
perf tools: Free root_dir in machine__init error path
perf tools: Move kernel mmap name into struct machine
perf tools: Generalize machine__set_kernel_mmap function
perf tools: Don't search for active kernel start in __machine__create_kernel_maps perf tools: Remove machine__load_kallsyms function
perf tools: Do not create kernel maps in sample__resolve
perf tests: Use arch__compare_symbol_names to compare symbols

 tools/lib/symbol/kallsyms.c         |   4 ++++
 tools/perf/tests/vmlinux-kallsyms.c |   4 ++--
 tools/perf/util/build-id.c          |  10 +++------
 tools/perf/util/event.c             |  16 +-------------
 tools/perf/util/machine.c           | 145 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++----------------------------------------------------------------
 tools/perf/util/machine.h           |   6 +-----
 tools/perf/util/symbol.c            |  13 +++++-------
 7 files changed, 87 insertions(+), 111 deletions(-)