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

Unified Diff: tools/telemetry/telemetry/core/platform/profiler/perf_profiler.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: Address feedback. 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
Index: tools/telemetry/telemetry/core/platform/profiler/perf_profiler.py
diff --git a/tools/telemetry/telemetry/core/platform/profiler/perf_profiler.py b/tools/telemetry/telemetry/core/platform/profiler/perf_profiler.py
index 6b460d98f65cee035a7ccfcbd2305aa7e0ee64d7..29aba7cccaf20a0754782af4512f47a78e003b0e 100644
--- a/tools/telemetry/telemetry/core/platform/profiler/perf_profiler.py
+++ b/tools/telemetry/telemetry/core/platform/profiler/perf_profiler.py
@@ -172,7 +172,7 @@ class PerfProfiler(profiler.Profiler):
device = browser_backend.adb.device()
perf_binary = android_profiling_helper.PrepareDeviceForPerf(device)
self._perf_control = perf_control.PerfControl(device)
- self._perf_control.ForceAllCpusOnline(True)
+ self._perf_control.ForceHighPerfMode()
else:
_PrepareHostForPerf()
@@ -185,7 +185,7 @@ class PerfProfiler(profiler.Profiler):
perf_binary, perfhost_binary))
except:
if self._is_android:
- self._perf_control.ForceAllCpusOnline(False)
+ self._perf_control.SetDefaultPerfMode()
raise
@classmethod
@@ -209,7 +209,7 @@ class PerfProfiler(profiler.Profiler):
def CollectProfile(self):
if self._is_android:
- self._perf_control.ForceAllCpusOnline(False)
+ self._perf_control.SetDefaultPerfMode()
output_files = []
for single_process in self._process_profilers:
output_files.append(single_process.CollectProfile())

Powered by Google App Engine
This is Rietveld 408576698