New issue
Advanced search Search tips

Issue 781475 link

Starred by 3 users

Issue metadata

Status: WontFix
Owner:
Closed: Dec 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Mac
Pri: 2
Type: Bug



Sign in to add a comment

We use more power now than reference build for playing video

Project Member Reported by crouleau@chromium.org, Nov 4 2017

Issue description

Specifically the test case is crowd1080.webm video.

See https://chromeperf.appspot.com/report?sid=1c809f9702c0a0ed6dd297b47269a6f11431ead2aedf23677db3d6c4957f6f3f&start_rev=505521&end_rev=513864

There's a ton of noise, but what I see is that we consistently use about .15 watts more power in ToT than in Reference. Looks like current reference build is 62.0.3194.0 (updated to that in August, but omahaproxy can't find revision number for that. see issue 781466) and current mac canary is 64.0.3257.0 (revision 513642). I'm going to try to put off a bisection just to see what we can find.
 
๐Ÿ“ Pinpoint job started.
https://pinpoint-dot-chromeperf.appspot.com/job/1699eb32f80000
๐Ÿ˜ฟ Pinpoint job stopped with an error.
https://pinpoint-dot-chromeperf.appspot.com/job/1699eb32f80000
Pinpoint failed in an RPC call to appengine :(
๐Ÿ“ Pinpoint job started.
https://pinpoint-dot-chromeperf.appspot.com/job/1106222ef80000
Blockedon: -781466
Project Member

Comment 6 by 42576172...@developer.gserviceaccount.com, Nov 10 2017

Cc: johnchen@chromium.org nedngu...@google.com
Owner: johnchen@chromium.org
๐Ÿ“ Found a significant difference after 1 commit.
https://pinpoint-dot-chromeperf.appspot.com/job/1106222ef80000

Rename TBMv2 media benchmarks
By johnchen@chromium.org ยท Mon Oct 09 19:59:40 2017
chromium @ b451847e6fc207b7f39c986b74712bfb33132874

Understanding performance regressions:
  http://g.co/ChromePerformanceRegressions
Status: WontFix (was: Assigned)
Argggg! We should never rename benchmarks again. Totally not worth it given that the tooling is broken.
Cc: dtu@chromium.org
+Dave: is it possible to tweak pinpoint so it can understand benchmark rename, metric rename?
Thanks Ned!

Also note that this isn't the only time that the rename has caused us problems. We had many bisections that we did right after the rename that were all unsuccessful.
Caleb: wdyt about doing bisect from "Start --> Before rename" & "After rename --> now"?
Owner: crouleau@chromium.org
Status: Assigned (was: WontFix)
Sure. that would be great. Good idea. But I don't know how to kick off a bisect against the old name of a benchmark. I always use the dashboard UI to kick off the bisect. Is there a commandline tool?
Cc: simonhatch@chromium.org
+Simon & Dave
There isn't a commandline tool no, we'd have to do a one-off where we force the name in Dashboard.
Status: WontFix (was: Assigned)

Sign in to add a comment