New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 632809 link

Starred by 3 users

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Aug 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug



Sign in to add a comment

Enable atrace for battor benchmark on Android

Project Member Reported by nedngu...@google.com, Jul 29 2016

Issue description

We should enable atrace for battor benchmark on Android to get more data to understand what cause the noise of power measurement on Android. Assign to Charlie to triage this.
 
Owner: rnep...@chromium.org
Me and charlie talked earlier this week and I'm going to be looking into enabling atrace for battor runs.
Project Member

Comment 2 by bugdroid1@chromium.org, Jul 29 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/e3cd5640a78c8adad2d84ab4b9b63f67f0f48de5

commit e3cd5640a78c8adad2d84ab4b9b63f67f0f48de5
Author: rnephew <rnephew@chromium.org>
Date: Fri Jul 29 21:36:55 2016

[Android][Telemetry] Enable ATrace tracing for BattOr Tests.

BUG= 632809 
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.perf:android_s5_perf_cq;master.tryserver.chromium.perf:linux_perf_cq;master.tryserver.chromium.perf:mac_retina_perf_cq;master.tryserver.chromium.perf:winx64_10_perf_cq

Review-Url: https://codereview.chromium.org/2198523003
Cr-Commit-Position: refs/heads/master@{#408773}

[modify] https://crrev.com/e3cd5640a78c8adad2d84ab4b9b63f67f0f48de5/tools/perf/benchmarks/battor.py

Project Member

Comment 3 by sheriffbot@chromium.org, Jul 30 2016

Labels: Hotlist-Google
Project Member

Comment 4 by bugdroid1@chromium.org, Aug 17 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/c4f0f2771056e1b05ca87d772af6a49d5123a0e6

commit c4f0f2771056e1b05ca87d772af6a49d5123a0e6
Author: rnephew <rnephew@chromium.org>
Date: Wed Aug 17 21:29:06 2016

[BattOr][Android] Re-enable atrace tracing for BattOr tests.

BUG= 632809 
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.perf:android_s5_perf_cq;master.tryserver.chromium.perf:linux_perf_cq;master.tryserver.chromium.perf:mac_retina_perf_cq;master.tryserver.chromium.perf:winx64_10_perf_cq

Review-Url: https://codereview.chromium.org/2241153004
Cr-Commit-Position: refs/heads/master@{#412645}

[modify] https://crrev.com/c4f0f2771056e1b05ca87d772af6a49d5123a0e6/tools/perf/benchmarks/battor.py

Status: Fixed (was: Untriaged)
Atrace is now enabled by default for battor tests using the category filter of 'sched'. 


https://chromeperf.appspot.com/report?sid=1da09af5a84a7c3e7d0f2865823eba18ad574dbecf06a3397674c62845fe280d&rev=413427
This looks really cool, thanks!
Wonderful! Thanks Randy & Charlie for making this possible.

Sign in to add a comment