KDAB / hotspot

The Linux perf GUI for performance analysis.
4.16k stars 257 forks source link

Tracepoint #508

Open lievenhey opened 1 year ago

lievenhey commented 1 year ago

Show tracepoints in TimeLineWidget

lievenhey commented 1 year ago

image

lievenhey commented 1 year ago

somehow making sure that the favorite rows are always shown, independent of the vertical current scroll position - potentially through a second view?

using a second view is most likely the easiest option since QWidgets don't have stuff like qmls header or footer

milianw commented 1 year ago

somehow making sure that the favorite rows are always shown, independent of the vertical current scroll position - potentially through a second view?

using a second view is most likely the easiest option since QWidgets don't have stuff like qmls header or footer

Agreed, we just need to sync the views and show just a single header somehow. But this should/ could be done in a follow-up patch and doesn't necessarily block integration of the work here as long as the other important issues are resolved.

GitMensch commented 1 year ago

Is this still in a draft state? It seems that new testcases are now available (but failing) are there still missing ones?

lievenhey commented 1 year ago

@milianw I finally found some time to work in this. Can you check if it to your satisfaction?

milianw commented 11 months ago

are you sure you pushed the correct work? I don't think you attended the issue I raised above, namely:

the biggest issue is that all tracepoints for all threads and processes are lumped together into a single row - that is pretty useless.

I just checked out your branch, rebased it on master and ran it on data I obtained with

perf record -z --call-graph dwarf -e cycles -e raw_syscalls:sys_enter -e raw_syscalls:sys_exit ./tests/test-clients/cpp-parallel/cpp-parallel

The result is still a single list of tracepoints across all threads, which is not useful. What's worse, some threads are suddenly showing up multiple times in the timelines above?

Screenshot_20231210_124332

lievenhey commented 10 months ago

Ok, fixed that. Now all events are shown in one line (before there was one line per even type). That is still somewhat wip since now it is really hard to differentiate different event types since only the tooltip reveals the type. I propose the following:

lievenhey commented 1 week ago

Currently it looks like this: grafik

lievenhey commented 1 week ago

waiting for https://github.com/KDAB/perfparser/pull/37