New issue
Advanced search Search tips

Issue 640398 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Nov 2016
Components:
EstimatedDays: ----
NextAction: ----
OS: Mac
Pri: 3
Type: Bug



Sign in to add a comment

Investigate the noise of CPU power consumption for WPR-replayed web pages.

Project Member Reported by erikc...@chromium.org, Aug 23 2016

Issue description

Design doc: https://docs.google.com/document/d/1QlX7BJDjDmHH2BuysnwYA5QqqGOW24bj6OpQzqWHFeo/edit#

If WPR provides low-noise results, we should use real web pages, rather than deconstructed examples.
 
power.trivial_pages has very low noise (1-2%), as expected:


Screen Shot 2016-08-26 at 6.04.51 PM.png
99 KB View Download
Components: OS>Kernel>Power
Project Member

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

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

commit 9c3c0a38de1c83ac9e6a9f683bad1291e12e2885
Author: erikchen <erikchen@chromium.org>
Date: Wed Aug 31 16:31:07 2016

Add a new benchmark for cpu power measurements on steady state sites.

The site in question is known to have high CPU usage, even though there appears
to be no visual changes to the site.

BUG= 640398 ,  638365 
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/2271793003
Cr-Commit-Position: refs/heads/master@{#415654}

[modify] https://crrev.com/9c3c0a38de1c83ac9e6a9f683bad1291e12e2885/tools/perf/benchmarks/power.py
[add] https://crrev.com/9c3c0a38de1c83ac9e6a9f683bad1291e12e2885/tools/perf/page_sets/data/idle_after_loading_stories.json
[add] https://crrev.com/9c3c0a38de1c83ac9e6a9f683bad1291e12e2885/tools/perf/page_sets/data/idle_after_loading_stories_001.wpr.sha1
[add] https://crrev.com/9c3c0a38de1c83ac9e6a9f683bad1291e12e2885/tools/perf/page_sets/idle_after_loading_stories.py

Project Member

Comment 4 by bugdroid1@chromium.org, Sep 1 2016

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

commit 6380c710d7ee2d6739e563d2957fc7475d663f70
Author: rnephew <rnephew@chromium.org>
Date: Thu Sep 01 01:17:59 2016

[BattOr] Add mac BattOr test for steady state cases.

BUG= 640398 ,  638365 ,  639308 
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/2301653002
Cr-Commit-Position: refs/heads/master@{#415832}

[modify] https://crrev.com/6380c710d7ee2d6739e563d2957fc7475d663f70/tools/perf/benchmarks/battor.py
[modify] https://crrev.com/6380c710d7ee2d6739e563d2957fc7475d663f70/tools/perf/page_sets/idle_after_loading_stories.py

Status: Fixed (was: Assigned)
Project Member

Comment 7 by bugdroid1@chromium.org, Nov 15 2016

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

commit bcd8adf62ab7756b980d91bde0d25ccea2ea0f4a
Author: erikchen <erikchen@chromium.org>
Date: Tue Nov 15 00:46:41 2016

Reduce the list of idle_after_loading stories.

The excessive CPU usage of the removed stories stems from ads that fail to load
deterministically under WPR. This makes them poor candidates to measure and fix
improvements for.

https://docs.google.com/document/d/1YVO9FGRBF1RUsedU28H6P8lt9bDpOci0vi0CLSkFih8/

BUG= 640398 

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

[modify] https://crrev.com/bcd8adf62ab7756b980d91bde0d25ccea2ea0f4a/tools/perf/page_sets/idle_after_loading_stories.py

Sign in to add a comment