From 7bf0e3c3443702791833444ccf10ab0e8a618391 Mon Sep 17 00:00:00 2001 From: Sergei Trofimov Date: Mon, 31 Oct 2016 13:58:03 +0000 Subject: [PATCH] energy_model: only set "ui" parameter for ChomeOS Fixed a bug, where "ui" runtime parameter was being set regardless of the underlying platforms for "freq" iterations, causing the to fail on non-ChomeOS devices. --- wlauto/instrumentation/energy_model/__init__.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/wlauto/instrumentation/energy_model/__init__.py b/wlauto/instrumentation/energy_model/__init__.py index 904140cf..95056ece 100644 --- a/wlauto/instrumentation/energy_model/__init__.py +++ b/wlauto/instrumentation/energy_model/__init__.py @@ -745,7 +745,8 @@ class EnergyModelInstrument(Instrument): spec.runtime_parameters['{}_frequency'.format(core)] = freq if not self.no_hotplug: spec.runtime_parameters['{}_cores'.format(core)] = num_cpus - spec.runtime_parameters['ui'] = 'off' + if self.device.platform == 'chromeos': + spec.runtime_parameters['ui'] = 'off' spec.id = '{}_{}_{}'.format(cluster, num_cpus, freq) spec.label = 'freq_{}_{}'.format(cluster, spec.label) spec.workload_parameters['taskset_mask'] = list_to_mask(self.get_cpus(cluster))