Telemetry: Pin Nexus 5 cores online. Limit CPU & GPU frequency.

build/android/pylib/perf/perf_control.py

 # Copyright 2013 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 import atexit
 import logging
 
 from pylib import android_commands
 from pylib.device import device_utils
 
@@ skipping 10 matching lines @@
       device = device_utils.DeviceUtils(device)
     self._device = device
     cpu_files = self._device.RunShellCommand(
       'ls -d /sys/devices/system/cpu/cpu[0-9]*')
     self._num_cpu_cores = len(cpu_files)
     assert self._num_cpu_cores > 0, 'Failed to detect CPUs.'
     logging.info('Number of CPUs: %d', self._num_cpu_cores)
     self._have_mpdecision = self._device.FileExists('/system/bin/mpdecision')
 
   def SetHighPerfMode(self):
-    """Sets the highest possible performance mode for the device."""
+    """Sets the highest stable performance mode for the device."""
     if not self._device.old_interface.IsRootEnabled():
       message = 'Need root for performance mode. Results may be NOISY!!'
       logging.warning(message)
       # Add an additional warning at exit, such that it's clear that any results
       # may be different/noisy (due to the lack of intended performance mode).
       atexit.register(logging.warning, message)
       return
+
+    product_model = self._device.old_interface.GetProductModel()
     # TODO(epenner): Enable on all devices (http://crbug.com/383566)
-    if 'Nexus 4' == self._device.old_interface.GetProductModel():
+    if 'Nexus 4' == product_model:
       self._ForceAllCpusOnline(True)
       if not self._AllCpusAreOnline():
         logging.warning('Failed to force CPUs online. Results may be NOISY!')
-    self._SetScalingGovernorInternal('performance')
+      self._SetScalingGovernorInternal('performance')
+    elif 'Nexus 5' == product_model:
+      self._ForceAllCpusOnline(True)
+      if not self._AllCpusAreOnline():
+        logging.warning('Failed to force CPUs online. Results may be NOISY!')
+      self._SetScalingGovernorInternal('performance')
+      self._SetScalingMaxFreq(1190400)
+      self._SetMaxGpuClock(200000000)
+    else:
+      self._SetScalingGovernorInternal('performance')
 
   def SetPerfProfilingMode(self):
     """Enables all cores for reliable perf profiling."""
     self._ForceAllCpusOnline(True)
     self._SetScalingGovernorInternal('performance')
     if not self._AllCpusAreOnline():
       if not self._device.old_interface.IsRootEnabled():
         raise RuntimeError('Need root to force CPUs online.')
       raise RuntimeError('Failed to force CPUs online.')
 
   def SetDefaultPerfMode(self):
     """Sets the performance mode for the device to its default mode."""
     if not self._device.old_interface.IsRootEnabled():
       return
-    product_model = self._device.GetProp('ro.product.model')
+    product_model = self._device.old_interface.GetProductModel()
+    if 'Nexus 5' == product_model:
+      if self._AllCpusAreOnline():
+        self._SetScalingMaxFreq(2265600)
+        self._SetMaxGpuClock(450000000)
+
     governor_mode = {
         'GT-I9300': 'pegasusq',
         'Galaxy Nexus': 'interactive',
         'Nexus 4': 'ondemand',
         'Nexus 7': 'interactive',
         'Nexus 10': 'interactive'
     }.get(product_model, 'ondemand')

[Sami, 2014/10/15 13:29:17]

Is this the right choice for Nexus 5? Please make it explicit if so.

[vmiura, 2014/10/15 20:15:01]

Done.

     self._SetScalingGovernorInternal(governor_mode)
     self._ForceAllCpusOnline(False)
 
   def _SetScalingGovernorInternal(self, value):
     cpu_cores = ' '.join([str(x) for x in range(self._num_cpu_cores)])
     script = ('for CPU in %s; do\n'
         '  FILE="/sys/devices/system/cpu/cpu$CPU/cpufreq/scaling_governor"\n'
         '  test -e $FILE && echo %s > $FILE\n'
         'done\n') % (cpu_cores, value)
     logging.info('Setting scaling governor mode: %s', value)
     self._device.RunShellCommand(script, as_root=True)
 
+  def _SetScalingMaxFreq(self, value):
+    cpu_cores = ' '.join([str(x) for x in range(self._num_cpu_cores)])
+    script = ('for CPU in %s; do\n'
+        '  FILE="/sys/devices/system/cpu/cpu$CPU/cpufreq/scaling_max_freq"\n'
+        '  test -e $FILE && echo %s > $FILE\n'

[Sami, 2014/10/15 13:29:17]

nit: %s should really be a %d here, right?

[vmiura, 2014/10/15 20:15:01]

Done.

+        'done\n') % (cpu_cores, str(value))

[Sami, 2014/10/15 13:29:17]

Remove str() if you agree with above.

[vmiura, 2014/10/15 20:15:01]

Done.

+    self._device.RunShellCommand(script, as_root=True)
+
+  def _SetMaxGpuClock(self, value):
+    self._device.WriteFile('/sys/class/kgsl/kgsl-3d0/max_gpuclk',
+                           str(value),
+                           as_root=True)
+
   def _AllCpusAreOnline(self):
     for cpu in range(1, self._num_cpu_cores):
       online_path = PerfControl._CPU_ONLINE_FMT % cpu
       # TODO(epenner): Investigate why file may be missing
       # (http://crbug.com/397118)
       if not self._device.FileExists(online_path) or \
             self._device.ReadFile(online_path)[0] == '0':
         return False
     return True
 
@@ skipping 17 matching lines @@
 
     if not self._have_mpdecision and not self._AllCpusAreOnline():
       logging.warning('Unexpected cpu hot plugging detected.')
 
     if not force_online:
       return
 
     for cpu in range(self._num_cpu_cores):
       online_path = PerfControl._CPU_ONLINE_FMT % cpu
       self._device.WriteFile(online_path, '1', as_root=True)
+