diff options
author | Tvrtko Ursulin <tvrtko.ursulin@intel.com> | 2017-05-17 12:27:09 +0100 |
---|---|---|
committer | Tvrtko Ursulin <tvrtko.ursulin@intel.com> | 2017-05-17 12:37:00 +0100 |
commit | c2a15894eae56b02849008fafc8f6785783d12b1 (patch) | |
tree | 6e816f2557afd855d32ac0093bd0d8bab62c76ff /scripts | |
parent | 36d97824afb1256ecfa8381a135b6cba11dcb04f (diff) |
media-bench: Flag workloads which failed to balance
Split out the flagging logic so to make it easier to read and so
that the complete failure to balance is declared a failure
instead of a warning.
Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Diffstat (limited to 'scripts')
-rwxr-xr-x | scripts/media-bench.pl | 19 |
1 files changed, 14 insertions, 5 deletions
diff --git a/scripts/media-bench.pl b/scripts/media-bench.pl index fff0d1bd..78f44522 100755 --- a/scripts/media-bench.pl +++ b/scripts/media-bench.pl @@ -482,11 +482,20 @@ foreach my $wrk (@workloads) { if $engines->{$key} < $idle_tolerance_pct; } - if ($saturated == 0 or - ($saturated == 1 and - ($engines->{'2'} < $idle_tolerance_pct or - $engines->{'3'} < $idle_tolerance_pct))) { - $result = $saturated == 0 ? 'FAIL' : 'WARN'; + if ($saturated == 0) { + # Not a single saturated engine + $result = 'FAIL'; + } elsif (not exists $engines->{'2'} or not exists $engines->{'3'}) { + # VCS1 and VCS2 not present in a balancing workload + $result = 'FAIL'; + } elsif ($saturated == 1 and + ($engines->{'2'} < $idle_tolerance_pct or + $engines->{'3'} < $idle_tolerance_pct)) { + # Only one VCS saturated + $result = 'WARN'; + } + + if ($result ne 'Pass') { $problem{'c'} = $c; $problem{'r'} = $r; $problem{'stats'} = $engines; |