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

Unified Diff: build/android/chrome_profiler/perf_controller.py

Issue 316143002: telemetry: Improve perf profiler (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Poll quicker. 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | build/android/pylib/perf/perf_control.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: build/android/chrome_profiler/perf_controller.py
diff --git a/build/android/chrome_profiler/perf_controller.py b/build/android/chrome_profiler/perf_controller.py
index d5f3b8005f6c56a88a9ba5efc495fcfee221da0e..1c1522b4b4d720040714ce2db18e90fb99350067 100644
--- a/build/android/chrome_profiler/perf_controller.py
+++ b/build/android/chrome_profiler/perf_controller.py
@@ -12,6 +12,7 @@ from chrome_profiler import controllers
from pylib import android_commands
from pylib import constants
+from pylib.perf import perf_control
sys.path.append(os.path.join(constants.DIR_SOURCE_ROOT,
'tools',
@@ -55,6 +56,8 @@ class _PerfProfiler(object):
'--output', self._output_file.name] + _PERF_OPTIONS
if categories:
cmd += ['--event', ','.join(categories)]
+ self._perf_control = perf_control.PerfControl(self._device)
+ self._perf_control.ForceAllCpusOnline(True)
self._perf_process = subprocess.Popen(cmd,
stdout=self._log_file,
stderr=subprocess.STDOUT)
@@ -64,6 +67,7 @@ class _PerfProfiler(object):
self._device.old_interface.RunShellCommand(
'kill -SIGINT ' + ' '.join(perf_pids))
self._perf_process.wait()
+ self._perf_control.ForceAllCpusOnline(False)
def _FailWithLog(self, msg):
self._log_file.seek(0)
« no previous file with comments | « no previous file | build/android/pylib/perf/perf_control.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698