power: Automate measuing power impact of enabling bluetooth radio. |
|||
Issue descriptionpower: Measure power impact of enabling bluetooth radio. While manual measurements shows little power delta between BT radio on (no paired devices) and off it would be good to automate to capture regressions. I'd probably just add this functionality to the power_Idle test by doing the following: 1. set backlight / display off and BT radio off. Sample power for baseline 2. Enable radio and get another sample to compare. If the BT is nearly zero power (<10mW) its likely the battery's gas gauge measurement won't provide enough resolution to show so for starters I think just adding the new keyvals (bt_off/on) to the test will suffice.
,
Jul 17 2017
Right now power_Idle samples a series of data over 2min and report the statistics. I like its current way of collection data and my plan is the repeat the same data collection process after BT is turned on. What are the measurements that are relevant to BT power delta? Should this be measured with AC or on battery?
,
Jul 17 2017
Power should be measured only on battery as gas gauge is our only consistent way to measure power on all systems and when on AC it really tracks charge going into the battery so its not a reliable source of system power consumption.
,
Aug 10 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromiumos/third_party/autotest/+/c8c5d46e949660230cadfae8b7834feaa339804d commit c8c5d46e949660230cadfae8b7834feaa339804d Author: Mengqi Guo <mqg@chromium.org> Date: Thu Aug 10 22:39:47 2017 power_idle: get power stats while bluetooth adapter is on Get the power stats while bluetooth adapter is on, so we can compare with power stats while bluetooth adapter is off. We want to make sure that bluetooth adapter on/off has no major effect on power. BUG= chromium:743164 TEST=test_that Change-Id: I510cbb8af1e74719d8f923685d4a87dc4d87b23d Reviewed-on: https://chromium-review.googlesource.com/585730 Commit-Ready: Mengqi Guo <mqg@chromium.org> Tested-by: Mengqi Guo <mqg@chromium.org> Reviewed-by: Todd Broch <tbroch@chromium.org> [modify] https://crrev.com/c8c5d46e949660230cadfae8b7834feaa339804d/client/site_tests/power_Idle/power_Idle.py
,
Aug 18 2017
,
Aug 24 2017
,
Aug 30
The following revision refers to this bug: https://chromium.googlesource.com/chromiumos/third_party/autotest/+/362e3184180c12887db4e4374d86d33a9e760e8e commit 362e3184180c12887db4e4374d86d33a9e760e8e Author: Todd Broch <tbroch@chromium.org> Date: Thu Aug 30 04:05:40 2018 power_Idle: verify bluetooth powered status at test start. Should explicitly test whether bluetooth is powered at start and if it is turn it off. BUG= chromium:743164 TEST=power_Idle Change-Id: I5fd0c1c3ec28daa73f6440972f0ff4e9e944ac1e Reviewed-on: https://chromium-review.googlesource.com/1185226 Commit-Ready: Todd Broch <tbroch@chromium.org> Tested-by: Todd Broch <tbroch@chromium.org> Reviewed-by: Puthikorn Voravootivat <puthik@chromium.org> Reviewed-by: Mengqi Guo <mqg@chromium.org> [modify] https://crrev.com/362e3184180c12887db4e4374d86d33a9e760e8e/client/site_tests/power_Idle/power_Idle.py
,
Aug 30
The following revision refers to this bug: https://chromium.googlesource.com/chromiumos/third_party/autotest/+/b6c937ab1b83342b66603b4c5682daa25634f7e8 commit b6c937ab1b83342b66603b4c5682daa25634f7e8 Author: Todd Broch <tbroch@chromium.org> Date: Thu Aug 30 04:05:41 2018 service_stopper: Remove bluetoothd from power draw list. Bluetooth daemon should be left up for power tests to more accurately portray any impact it has on platform power. BUG= chromium:743164 TEST=power_Idle Change-Id: Ib64abd1b8f04e0693f1fc8a3ec12ad5721b18694 Reviewed-on: https://chromium-review.googlesource.com/1185227 Commit-Ready: Todd Broch <tbroch@chromium.org> Tested-by: Todd Broch <tbroch@chromium.org> Reviewed-by: Puthikorn Voravootivat <puthik@chromium.org> Reviewed-by: Mengqi Guo <mqg@chromium.org> [modify] https://crrev.com/b6c937ab1b83342b66603b4c5682daa25634f7e8/client/cros/service_stopper.py |
|||
►
Sign in to add a comment |
|||
Comment 1 by tbroch@chromium.org
, Jul 14 2017