New issue
Advanced search Search tips

Issue 838694 link

Starred by 3 users

Issue metadata

Status: Fixed
Owner:
Closed: Aug 3
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug



Sign in to add a comment

power_Display: create a test to measure various power consumption scenarios of built-in display.

Project Member Reported by tbroch@chromium.org, May 1 2018

Issue description

Create a test that focuses on measuring power consumption of built-in display.

At a minimum it should cover set of known patterns that typical display vendors use to characterize their power consumption.

In addition it may want to test:

- backlight levels and measurements.  Could be covered by power_Backlight test already.
- panel-self refresh
- content adaptive backlight control (CABC aka DBC or DCR)
- frame buffer compression (FBC)
 
Project Member

Comment 1 by bugdroid1@chromium.org, May 11 2018

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

commit 124cd531977352cc4ce169a9de23e84cc0c698e7
Author: Todd Broch <tbroch@chromium.org>
Date: Fri May 11 02:40:38 2018

power: create power_test class & power_Display to measure panel.

Create some test cases to measure display power.  Initially fullscreen
pages for white, black, red, green, blue & a checkerboard.

CL also introduces a base class to simplify various power results and
monitoring.  Future CLs will refactor appropriate power_* tests to use
the new base class.

BUG= chromium:838694 
TEST=power_Display

See keyvals for various power related results in particular
system_pwr_avg when on battery and values have low stdev (~25mW) while
still showing panel's power range across the test images.  For eve,

  power_Display_black_system_pwr_avg{perf}=2.26347666667
  power_Display_blue_system_pwr_avg{perf}=2.61279
  power_Display_checker1_system_pwr_avg{perf}=2.34846333333
  power_Display_green_system_pwr_avg{perf}=2.25831
  power_Display_red_system_pwr_avg{perf}=2.67503
  power_Display_white_system_pwr_avg{perf}=2.41540666667

Change-Id: Ia869be30aa68e10f24ac5aeaaf992ff468d55449
Reviewed-on: https://chromium-review.googlesource.com/1040539
Commit-Ready: Todd Broch <tbroch@chromium.org>
Tested-by: Todd Broch <tbroch@chromium.org>
Reviewed-by: Puthikorn Voravootivat <puthik@chromium.org>

[add] https://crrev.com/124cd531977352cc4ce169a9de23e84cc0c698e7/client/site_tests/power_Display/html/black.html
[add] https://crrev.com/124cd531977352cc4ce169a9de23e84cc0c698e7/client/site_tests/power_Display/html/blue.html
[add] https://crrev.com/124cd531977352cc4ce169a9de23e84cc0c698e7/client/site_tests/power_Display/html/checker1.html
[add] https://crrev.com/124cd531977352cc4ce169a9de23e84cc0c698e7/client/site_tests/power_Display/control
[add] https://crrev.com/124cd531977352cc4ce169a9de23e84cc0c698e7/client/site_tests/power_Display/html/green.html
[add] https://crrev.com/124cd531977352cc4ce169a9de23e84cc0c698e7/client/site_tests/power_Display/html/red.html
[add] https://crrev.com/124cd531977352cc4ce169a9de23e84cc0c698e7/client/site_tests/power_Display/html/white.html
[add] https://crrev.com/124cd531977352cc4ce169a9de23e84cc0c698e7/client/cros/power/power_test.py
[add] https://crrev.com/124cd531977352cc4ce169a9de23e84cc0c698e7/client/site_tests/power_Display/power_Display.py

Project Member

Comment 2 by bugdroid1@chromium.org, May 11 2018

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

commit a6f9cb1e6f32735b103f421809113ef402d3ee1b
Author: Todd Broch <tbroch@chromium.org>
Date: Fri May 11 02:40:38 2018

power_Idle: refactor to use power_test base class.

Differences I note from diff of keyval and power_log.json before
and after are:

temperature keyvals were only logged once before and without prefix.
Now they're logged twice and with prefixes.

BUG= chromium:838694 
TEST=power_Idle

Change-Id: I26303cbce6983d36ded75f5ec3ebb5520bfa9923
Reviewed-on: https://chromium-review.googlesource.com/1048044
Commit-Ready: Todd Broch <tbroch@chromium.org>
Tested-by: Todd Broch <tbroch@chromium.org>
Reviewed-by: Puthikorn Voravootivat <puthik@chromium.org>

[modify] https://crrev.com/a6f9cb1e6f32735b103f421809113ef402d3ee1b/client/site_tests/power_Idle/power_Idle.py

Project Member

Comment 3 by bugdroid1@chromium.org, May 12 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/chromiumos-overlay/+/567202d92a16af139b48cc64ca7990dcbf3e131f

commit 567202d92a16af139b48cc64ca7990dcbf3e131f
Author: Todd Broch <tbroch@chromium.org>
Date: Sat May 12 06:22:08 2018

chromiumos-overlay: Add power_Display to autotest-chrome ebuild.

Test uses telemetry to login and display web pages so add it here.

BUG= chromium:838694 
TEST=emerge-${BOARD} autotest-chrome

Change-Id: I3c33c1dbf4d7e9991750996a95ed2629526ceb6a
Reviewed-on: https://chromium-review.googlesource.com/1055673
Commit-Ready: Todd Broch <tbroch@chromium.org>
Tested-by: Todd Broch <tbroch@chromium.org>
Reviewed-by: Puthikorn Voravootivat <puthik@chromium.org>

[modify] https://crrev.com/567202d92a16af139b48cc64ca7990dcbf3e131f/chromeos-base/autotest-chrome/autotest-chrome-9999.ebuild

Project Member

Comment 4 by bugdroid1@chromium.org, May 12 2018

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

commit 040dbce2037a9d95ce25c6e74790d5c68906b0fd
Author: Todd Broch <tbroch@chromium.org>
Date: Sat May 12 06:22:09 2018

power_Display: add to power_daily suite.

CQ-DEPEND=CL:1055673
BUG= chromium:838694 
TEST=none

Change-Id: Ice62d44c8db7f2869da5e02f195332d95b916971
Reviewed-on: https://chromium-review.googlesource.com/1055674
Commit-Ready: Todd Broch <tbroch@chromium.org>
Tested-by: Todd Broch <tbroch@chromium.org>
Reviewed-by: Puthikorn Voravootivat <puthik@chromium.org>

[modify] https://crrev.com/040dbce2037a9d95ce25c6e74790d5c68906b0fd/client/site_tests/power_Display/control

Status: Fixed (was: Assigned)
Test is running and passing regularly.  Closing.

Sign in to add a comment