diff mbox

[i-g-t,1/2] scripts/trace.pl: Support class:instance engine tracepoints

Message ID 20180605165051.29136-1-tvrtko.ursulin@linux.intel.com (mailing list archive)
State New, archived
Headers show

Commit Message

Tvrtko Ursulin June 5, 2018, 4:50 p.m. UTC
From: Tvrtko Ursulin <tvrtko.ursulin@intel.com>

New way of describing engines needs the tool to be adapted to understand it.

Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
---
 scripts/trace.pl | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

Comments

Lionel Landwerlin June 5, 2018, 5:15 p.m. UTC | #1
Reviewed-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com>

On 05/06/18 17:50, Tvrtko Ursulin wrote:
> From: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
>
> New way of describing engines needs the tool to be adapted to understand it.
>
> Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
> ---
>   scripts/trace.pl | 4 +++-
>   1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/scripts/trace.pl b/scripts/trace.pl
> index 068eee68b30c..ea6c667696f4 100755
> --- a/scripts/trace.pl
> +++ b/scripts/trace.pl
> @@ -378,6 +378,8 @@ while (<>) {
>   		$k = 'global' if $k eq 'global_seqno';
>   		chop $v if substr($v, -1, 1) eq ',';
>   		$tp{$k} = $v;
> +
> +		$tp{'ring'} = $tp{'engine'} if $k eq 'engine';
>   	}
>   
>   	next if exists $tp{'ring'} and exists $ignore_ring{$tp{'ring'}};
> @@ -631,7 +633,7 @@ foreach my $gid (sort keys %rings) {
>   
>   	# Extract all GPU busy intervals and sort them.
>   	foreach my $key (@sorted_keys) {
> -		next unless $db{$key}->{'ring'} == $ring;
> +		next unless $db{$key}->{'ring'} eq $ring;
>   		push @s_, $db{$key}->{'start'};
>   		push @e_, $db{$key}->{'end'};
>   		die if $db{$key}->{'start'} > $db{$key}->{'end'};
diff mbox

Patch

diff --git a/scripts/trace.pl b/scripts/trace.pl
index 068eee68b30c..ea6c667696f4 100755
--- a/scripts/trace.pl
+++ b/scripts/trace.pl
@@ -378,6 +378,8 @@  while (<>) {
 		$k = 'global' if $k eq 'global_seqno';
 		chop $v if substr($v, -1, 1) eq ',';
 		$tp{$k} = $v;
+
+		$tp{'ring'} = $tp{'engine'} if $k eq 'engine';
 	}
 
 	next if exists $tp{'ring'} and exists $ignore_ring{$tp{'ring'}};
@@ -631,7 +633,7 @@  foreach my $gid (sort keys %rings) {
 
 	# Extract all GPU busy intervals and sort them.
 	foreach my $key (@sorted_keys) {
-		next unless $db{$key}->{'ring'} == $ring;
+		next unless $db{$key}->{'ring'} eq $ring;
 		push @s_, $db{$key}->{'start'};
 		push @e_, $db{$key}->{'end'};
 		die if $db{$key}->{'start'} > $db{$key}->{'end'};