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

Issue 616153 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Closed: Aug 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Feature



Sign in to add a comment

power: publish percent_cpuidle_C0_time to perf results for power_LoadTest

Project Member Reported by tbroch@chromium.org, May 31 2016

Issue description

power: publish percent_cpuidle_C0_time to perf results for power_LoadTest

This key's value often provides direct relationship to software regressions that lead to lower battery life.

So lets publish it as a perf value.

I plan to modify PLT to do the following:

1. publish this value as a perf result ( results-chart.json )
2. create a test that runs 1hr on wifi with ACOK ( control.1hour_wifi_acok )
3. run that new control file as a part of dynamic suite 'daily'


 
Project Member

Comment 1 by sheriffbot@chromium.org, Jun 1 2016

Labels: -M-52 M-53 MovedFrom-52
Moving this nonessential bug to the next milestone.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Project Member

Comment 2 by bugdroid1@chromium.org, Jun 14 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/e2ad5763d4db741f6ffdcb89479a97c6fbd628f5

commit e2ad5763d4db741f6ffdcb89479a97c6fbd628f5
Author: Todd Broch <tbroch@chromium.org>
Date: Tue May 31 23:52:15 2016

power_LoadTest: publish percent_cpuidle_C0_time perf value.

CPU C0 utilization can provide useful metric for tracking battery life
regressions in the absence of reliable battery drain data.  Lets track
it.

BUG= chromium:616153 
TEST=test_that power_LoadTest.fast_gaia

Contents of results-chart.json,

{
  "percent_cpuidle_C0_time": {
    "summary": {
      "units": "percent",
      "type": "scalar",
      "value": 43.173375902954845,
      "improvement_direction": "down"
    }
  }

Change-Id: I8615a1a872ae2bedde5ed059223c064765163d85
Reviewed-on: https://chromium-review.googlesource.com/349370
Commit-Ready: Todd Broch <tbroch@chromium.org>
Tested-by: Todd Broch <tbroch@chromium.org>
Reviewed-by: Sameer Nanda <snanda@chromium.org>

[add] https://crrev.com/e2ad5763d4db741f6ffdcb89479a97c6fbd628f5/client/site_tests/power_LoadTest/control.fast_gaia
[modify] https://crrev.com/e2ad5763d4db741f6ffdcb89479a97c6fbd628f5/client/site_tests/power_LoadTest/power_LoadTest.py

Project Member

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

Labels: -M-53 MovedFrom-53
This issue has been moved once and is lower than Pri-1. Removing the milestone.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Cc: -mshe...@chromium.org
Project Member

Comment 5 by bugdroid1@chromium.org, Aug 3 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/9dc11aca827f25093691d831a9adb638dd74213a

commit 9dc11aca827f25093691d831a9adb638dd74213a
Author: Todd Broch <tbroch@chromium.org>
Date: Fri Jun 03 00:16:45 2016

power_LoadTest: Add 1hour wifi ac_ok run to power_daily suite.

BUG= chromium:616153 
TEST=test_that power_LoadTest.1hour_wifi_acok

Change-Id: If5941d3de949cbcf8eb8f5f6f6a05a00212a3199
Reviewed-on: https://chromium-review.googlesource.com/349371
Commit-Ready: Todd Broch <tbroch@chromium.org>
Tested-by: Todd Broch <tbroch@chromium.org>
Reviewed-by: Derek Basehore <dbasehore@chromium.org>

[add] https://crrev.com/9dc11aca827f25093691d831a9adb638dd74213a/client/site_tests/power_LoadTest/control.1hour_wifi_acok

Status: Fixed (was: Started)
Project Member

Comment 7 by bugdroid1@chromium.org, Aug 25 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/0fdc5cbe064448e3b6da5f4539040fc08621da27

commit 0fdc5cbe064448e3b6da5f4539040fc08621da27
Author: Todd Broch <tbroch@chromium.org>
Date: Wed Aug 17 22:23:47 2016

power_LoadTest: Change 1hour acok to wired.

Can not necessarily guarantee wificell access in the lab and with
ethernet connected the test is always failing w/ 'Ethernet interface is
active. Please remove Ethernet cable.'

Changing to ignore network as motivation for this test is to start
regularly getting c-state utilization in the absence of battery
discharge results.

BUG= chromium:616153 
TEST=test_that power_LoadTest.WIRED_1hr_acok

Change-Id: I5f5da3f3fdf54ecc203ee3b076ce351cb0c4c920
Reviewed-on: https://chromium-review.googlesource.com/372198
Commit-Ready: Todd Broch <tbroch@chromium.org>
Tested-by: Todd Broch <tbroch@chromium.org>
Reviewed-by: Ravi Chandra Sadineni <ravisadineni@chromium.org>

[rename] https://crrev.com/0fdc5cbe064448e3b6da5f4539040fc08621da27/client/site_tests/power_LoadTest/control.WIRED_1hr_acok

Labels: VerifyIn-54

Comment 9 by ka...@chromium.org, Sep 2 2016

Labels: Bulk-Verified
Status: Verified (was: Fixed)

Sign in to add a comment