Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(171)

Side by Side Diff: build/android/pylib/perf/perf_control_unittest.py

Issue 338233003: Telemetry: Set scaling governor on all cpus (on all devices) (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 # Copyright 2014 The Chromium Authors. All rights reserved. 1 # Copyright 2014 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 4
5 import os 5 import os
6 import sys 6 import sys
7 import unittest 7 import unittest
8 8
9 sys.path.append(os.path.join(os.path.dirname(__file__), os.pardir, os.pardir)) 9 sys.path.append(os.path.join(os.path.dirname(__file__), os.pardir, os.pardir))
10 10
11 from pylib import android_commands 11 from pylib import android_commands
12 from pylib.device import device_utils 12 from pylib.device import device_utils
13 from pylib.perf import perf_control 13 from pylib.perf import perf_control
14 14
15 15
16 class TestPerfControl(unittest.TestCase): 16 class TestPerfControl(unittest.TestCase):
17 def setUp(self): 17 def setUp(self):
18 if not os.getenv('BUILDTYPE'): 18 if not os.getenv('BUILDTYPE'):
19 os.environ['BUILDTYPE'] = 'Debug' 19 os.environ['BUILDTYPE'] = 'Debug'
20 20
21 devices = android_commands.GetAttachedDevices() 21 devices = android_commands.GetAttachedDevices()
22 self.assertGreater(len(devices), 0, 'No device attached!') 22 self.assertGreater(len(devices), 0, 'No device attached!')
23 self._device = device_utils.DeviceUtils( 23 self._device = device_utils.DeviceUtils(
24 android_commands.AndroidCommands(device=devices[0])) 24 android_commands.AndroidCommands(device=devices[0]))
25 25
26 def testForceAllCpusOnline(self): 26 def testHighPerfMode(self):
27 perf = perf_control.PerfControl(self._device) 27 perf = perf_control.PerfControl(self._device)
28 cpu_online_files = self._device.old_interface.RunShellCommand( 28
29 'ls -d /sys/devices/system/cpu/cpu[0-9]*/online')
30 try: 29 try:
31 perf.ForceAllCpusOnline(True) 30 perf.SetHighPerfMode()
31 cpu_online_files = self._device.old_interface.RunShellCommand(
32 'ls -d /sys/devices/system/cpu/cpu[0-9]*/online')
32 for path in cpu_online_files: 33 for path in cpu_online_files:
33 self.assertEquals('1', 34 self.assertEquals('1',
34 self._device.old_interface.GetFileContents(path)[0]) 35 self._device.old_interface.GetFileContents(path)[0])
35 mode = self._device.old_interface.RunShellCommand('ls -l %s' % path)[0] 36 cpu_governor_files = self._device.old_interface.RunShellCommand(
36 self.assertEquals('-r--r--r--', mode[:10]) 37 'ls -d /sys/devices/system/cpu/cpu[0-9]*/cpufreq/cpuinfo_cur_freq')
Sami 2014/06/17 04:16:44 Should we be checking scaling_governor instead of
epenner 2014/06/17 22:10:56 Thanks! As you can see I haven't run this yet. I w
38 for path in cpu_governor_files:
39 self.assertEquals('performance',
40 self._device.old_interface.GetFileContents(path)[0])
41
37 finally: 42 finally:
38 perf.ForceAllCpusOnline(False) 43 perf.SetDefaultPerfMode()
39
40 for path in cpu_online_files:
41 mode = self._device.old_interface.RunShellCommand('ls -l %s' % path)[0]
42 self.assertEquals('-rw-r--r--', mode[:10])
43
44 44
45 if __name__ == '__main__': 45 if __name__ == '__main__':
46 unittest.main() 46 unittest.main()
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698