tests/benchmarks: remove redefinition of benchmarkFinalize
What benchmarkFinalize currently only does is to pause for an amount of
time before proceding with the next iteration. Since we now do the same
_before_ each iteration (with --iterations-pause) the redefinition is
redundant.
Remove it from all benchmarks.
Signed-off-by: Juri Lelli <juri.lelli@arm.com>
diff --git a/tests/benchmarks/android_youtube.py b/tests/benchmarks/android_youtube.py
index 3afa217..3822965 100755
--- a/tests/benchmarks/android_youtube.py
+++ b/tests/benchmarks/android_youtube.py
@@ -67,18 +67,10 @@
self.setupWorkload()
self.setupGovernor()
- def benchmarkFinalize(self):
- if self.delay_after_s:
- self._log.info("Waiting %d[s] before to continue...",
- self.delay_after_s)
- sleep(self.delay_after_s)
-
- def __init__(self, governor, video_url, video_duration_s,
- delay_after_s=0):
+ def __init__(self, governor, video_url, video_duration_s):
self.governor = governor
self.video_url = video_url
self.video_duration_s = video_duration_s
- self.delay_after_s = delay_after_s
super(YouTubeTest, self).__init__()
def setupWorkload(self):
@@ -168,10 +160,8 @@
for governor in governors:
for url in video_urls:
tests_remaining -= 1
- delay_after_s = 30 if tests_remaining else 0
try:
- YouTubeTest(governor, url, video_duration_s,
- delay_after_s)
+ YouTubeTest(governor, url, video_duration_s)
tests_completed += 1
except:
# A test configuraion failed, continue with other tests