diff options
author | Ian Rogers <irogers@google.com> | 2025-03-07 05:39:04 +0300 |
---|---|---|
committer | Namhyung Kim <namhyung@kernel.org> | 2025-03-12 05:00:50 +0300 |
commit | 9a1c57fe26f23c4c996c51c5948d727068f306de (patch) | |
tree | 097d64e02f0a2b5836acc4c047b2849369ce6336 /tools/perf/scripts/python/task-analyzer.py | |
parent | b74683b3bb224eccb644cf260753dfc82e802d92 (diff) | |
download | linux-9a1c57fe26f23c4c996c51c5948d727068f306de.tar.xz |
perf parse-events: Corrections to topdown sorting
In the case of '{instructions,slots},faults,topdown-retiring' the
first event that must be grouped, slots, is ignored causing the
topdown-retiring event not to be adjacent to the group it needs to be
inserted into. Don't ignore the group members when computing the
force_grouped_index.
Make the force_grouped_index be for the leader of the group it is
within and always use it first rather than a group leader index so
that topdown events may be sorted from one group into another.
As the PMU name comparison applies to moving events in the same group
ensure the name ordering is always respected.
Change the group splitting logic to not group if there are no other
topdown events and to fix cases where the force group leader wasn't
being grouped with the other members of its group.
Reported-by: Dapeng Mi <dapeng1.mi@linux.intel.com>
Closes: https://lore.kernel.org/lkml/20250224083306.71813-2-dapeng1.mi@linux.intel.com/
Closes: https://lore.kernel.org/lkml/f7e4f7e8-748c-4ec7-9088-0e844392c11a@linux.intel.com/
Signed-off-by: Ian Rogers <irogers@google.com>
Tested-by: Dapeng Mi <dapeng1.mi@linux.intel.com>
Link: https://lore.kernel.org/r/20250307023906.1135613-3-irogers@google.com
Signed-off-by: Namhyung Kim <namhyung@kernel.org>
Diffstat (limited to 'tools/perf/scripts/python/task-analyzer.py')
0 files changed, 0 insertions, 0 deletions