aboutsummaryrefslogtreecommitdiff
path: root/crosperf/experiment_factory_unittest.py
diff options
context:
space:
mode:
authorZhizhou Yang <zhizhouy@google.com>2018-11-09 11:44:10 -0800
committerchrome-bot <chrome-bot@chromium.org>2018-11-13 03:32:49 -0800
commit1a199b1827d125b89c8cf66c8d5680bf14a1b857 (patch)
treefd963d10454b318e4c83f9627c2be3546a3a6a15 /crosperf/experiment_factory_unittest.py
parentaa8d023f42fa991587e291e067820b2bbd6031a3 (diff)
downloadtoolchain-utils-1a199b1827d125b89c8cf66c8d5680bf14a1b857.tar.gz
crosperf: introduce cwp_dso and weight for CWP approximation
This patch does initial work to support CWP approximation in crosperf. It introduceds a global option cwp_dso, which is used to specify the type of DSO to use for approximation; an option weight in benchmark setting, which decides the weight of each benchmark in test. This patch also contains some value checks for these two options. BUG=chromium:902785 TEST=Passed all unit tests and sample tests locally. Change-Id: I565c4baf0630ce6c1b62ad0398d05a5f336aac6e Reviewed-on: https://chromium-review.googlesource.com/1323473 Commit-Ready: Zhizhou Yang <zhizhouy@google.com> Tested-by: Zhizhou Yang <zhizhouy@google.com> Reviewed-by: Caroline Tice <cmtice@chromium.org>
Diffstat (limited to 'crosperf/experiment_factory_unittest.py')
-rwxr-xr-xcrosperf/experiment_factory_unittest.py98
1 files changed, 98 insertions, 0 deletions
diff --git a/crosperf/experiment_factory_unittest.py b/crosperf/experiment_factory_unittest.py
index 74521c7c..372dfbe6 100755
--- a/crosperf/experiment_factory_unittest.py
+++ b/crosperf/experiment_factory_unittest.py
@@ -43,6 +43,30 @@ EXPERIMENT_FILE_1 = """
}
"""
+EXPERIMENT_FILE_2 = """
+ board: x86-alex
+ remote: chromeos-alex3
+
+ cwp_dso: kallsyms
+
+ benchmark: Octane {
+ iterations: 1
+ suite: telemetry_Crosperf
+ weight: 0.8
+ }
+
+ benchmark: Kraken {
+ iterations: 1
+ suite: telemetry_Crosperf
+ weight: 0.2
+ }
+
+ image1 {
+ chromeos_image: /usr/local/google/cros_image1.bin
+ }
+ """
+
+
# pylint: disable=too-many-function-args
@@ -71,6 +95,15 @@ class ExperimentFactoryTest(unittest.TestCase):
'/usr/local/google/cros_image1.bin')
self.assertEqual(exp.labels[0].board, 'x86-alex')
+ def testLoadExperimentFile2CWP(self):
+ experiment_file = ExperimentFile(StringIO.StringIO(EXPERIMENT_FILE_2))
+ exp = ExperimentFactory().GetExperiment(
+ experiment_file, working_directory='', log_dir='')
+ self.assertEqual(exp.cwp_dso, 'kallsyms')
+ self.assertEqual(len(exp.benchmarks), 2)
+ self.assertEqual(exp.benchmarks[0].weight, 0.8)
+ self.assertEqual(exp.benchmarks[1].weight, 0.2)
+
def testDuplecateBenchmark(self):
mock_experiment_file = ExperimentFile(StringIO.StringIO(''))
mock_experiment_file.all_settings = []
@@ -83,6 +116,71 @@ class ExperimentFactoryTest(unittest.TestCase):
ef = ExperimentFactory()
ef.GetExperiment(mock_experiment_file, '', '')
+ def testCWPExceptions(self):
+ mock_experiment_file = ExperimentFile(StringIO.StringIO(''))
+ mock_experiment_file.all_settings = []
+ global_settings = settings_factory.GlobalSettings('test_name')
+
+ # Test 1: DSO type not supported
+ global_settings.SetField('cwp_dso', 'test')
+ self.assertEqual(global_settings.GetField('cwp_dso'), 'test')
+ mock_experiment_file.global_settings = global_settings
+ with self.assertRaises(RuntimeError) as msg:
+ ef = ExperimentFactory()
+ ef.GetExperiment(mock_experiment_file, '', '')
+ self.assertEqual('The DSO specified is not supported',
+ str(msg.exception))
+
+ # Test 2: No weight after DSO specified
+ global_settings.SetField('cwp_dso', 'kallsyms')
+ mock_experiment_file.global_settings = global_settings
+ benchmark_settings = settings_factory.BenchmarkSettings('name')
+ mock_experiment_file.all_settings.append(benchmark_settings)
+ with self.assertRaises(RuntimeError) as msg:
+ ef = ExperimentFactory()
+ ef.GetExperiment(mock_experiment_file, '', '')
+ self.assertEqual('With DSO specified, each benchmark should have a weight',
+ str(msg.exception))
+
+ # Test 3: Weight is set, but no dso specified
+ global_settings.SetField('cwp_dso', '')
+ mock_experiment_file.global_settings = global_settings
+ benchmark_settings = settings_factory.BenchmarkSettings('name')
+ benchmark_settings.SetField('weight', '0.8')
+ mock_experiment_file.all_settings = []
+ mock_experiment_file.all_settings.append(benchmark_settings)
+ with self.assertRaises(RuntimeError) as msg:
+ ef = ExperimentFactory()
+ ef.GetExperiment(mock_experiment_file, '', '')
+ self.assertEqual('Weight can only be set when DSO specified',
+ str(msg.exception))
+
+ # Test 4: cwp_dso only works for telemetry_Crosperf benchmarks
+ global_settings.SetField('cwp_dso', 'kallsyms')
+ mock_experiment_file.global_settings = global_settings
+ benchmark_settings = settings_factory.BenchmarkSettings('name')
+ benchmark_settings.SetField('weight', '0.8')
+ mock_experiment_file.all_settings = []
+ mock_experiment_file.all_settings.append(benchmark_settings)
+ with self.assertRaises(RuntimeError) as msg:
+ ef = ExperimentFactory()
+ ef.GetExperiment(mock_experiment_file, '', '')
+ self.assertEqual('CWP approximation weight only works with '
+ 'telemetry_Crosperf suite',
+ str(msg.exception))
+
+ # Test 5: weight should be float between 0 and 1
+ benchmark_settings = settings_factory.BenchmarkSettings('name')
+ benchmark_settings.SetField('weight', '1.2')
+ benchmark_settings.SetField('suite', 'telemetry_Crosperf')
+ mock_experiment_file.all_settings = []
+ mock_experiment_file.all_settings.append(benchmark_settings)
+ with self.assertRaises(RuntimeError) as msg:
+ ef = ExperimentFactory()
+ ef.GetExperiment(mock_experiment_file, '', '')
+ self.assertEqual('Weight should be a float between 0 and 1',
+ str(msg.exception))
+
def test_append_benchmark_set(self):
ef = ExperimentFactory()