Skip to content

Commit

Permalink
i915-perf: don't skip events into the events
Browse files Browse the repository at this point in the history
We would like to show GPU frequency data in the timeline. This
requires to be able to find idle events so we don't have gaps in the
timeline.

This doesn't add much more events in the timeline and we can still
filter in the rendering code.
  • Loading branch information
llandwerlin-intel authored and mikesart committed Dec 30, 2020
1 parent c768731 commit bd3efcd
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
4 changes: 4 additions & 0 deletions src/gpuvis_graph.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -2501,6 +2501,10 @@ uint32_t TraceWin::graph_render_i915_perf_events( graph_info_t &gi )
}
const trace_event_t &event = get_event( eventid );

// Skip idle events
if ( event.pid == 0xffffffff )
continue;

if ( eventid > gi.eventend )
break;
else if ( gi.graph_only_filtered && event.is_filtered_out )
Expand Down
4 changes: 0 additions & 4 deletions src/i915-perf/i915-perf-read.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -75,10 +75,6 @@ int read_i915_perf_file( const char *file, StrPool &strpool, trace_info_t &trace

for (uint32_t i = 0; i < reader->n_timelines; i++)
{
// Skip the idle time
if ( reader->timelines[i].hw_id == 0xffffffff )
continue;

trace_event_t event;

if ( reader->timelines[i].cpu_ts_start < trace_info.min_file_ts )
Expand Down

0 comments on commit bd3efcd

Please sign in to comment.