Closed
Bug 1467619
Opened 6 years ago
Closed 6 years ago
Fix our motionmark_animometer Design score
Categories
(Core :: Graphics: Layers, enhancement)
Core
Graphics: Layers
Tracking
()
RESOLVED
FIXED
mozilla62
Tracking | Status | |
---|---|---|
firefox62 | --- | fixed |
People
(Reporter: dthayer, Assigned: dthayer)
References
(Regressed 1 open bug)
Details
Attachments
(1 file)
The motionmark_animometer Design test creates a bunch of overlapping text rectangles which move around and change color in a requestAnimationFrame(...) cycle. Our performance on this test is really bad because we create a bunch of layers for no benefit, since they have to be repainted every frame anyway. We should fix that.
Here is a talos run with the behavior changed:
https://treeherder.mozilla.org/perf.html#/comparesubtest?originalProject=mozilla-central&newProject=try&newRevision=6efd3d75ffbbde28d5e6cf641d91ec5352e5ea80&originalSignature=cfbf3a9ac76b5de64bb3a493290639205ff28244&newSignature=cfbf3a9ac76b5de64bb3a493290639205ff28244&framework=1&selectedTimeRange=172800
Comment hidden (mozreview-request) |
Assignee | ||
Comment 2•6 years ago
|
||
Quick note: this caps out our performance on the test at 60fps. The real impact of this change is much larger. We should consider bumping the complexity that we run the test at from 17 to 200 at some point to compensate and ensure we're tracking regressions.
Comment 3•6 years ago
|
||
mozreview-review |
Comment on attachment 8984320 [details]
Bug 1467619 - Track and adjust layerization on repaint triggers
https://reviewboard.mozilla.org/r/250136/#review256442
Looks good! We should definitely increase the test complexity, both to show off this win, and to prevent regressions.
Attachment #8984320 -
Flags: review?(matt.woodrow) → review+
Assignee | ||
Comment 4•6 years ago
|
||
mozreview-review-reply |
Comment on attachment 8984320 [details]
Bug 1467619 - Track and adjust layerization on repaint triggers
https://reviewboard.mozilla.org/r/250136/#review256442
Okeydoke. I'm going to put it into a follow-up bug, so that we can get clean numbers after landing this for at least a few days, if that sounds all right.
Pushed by dothayer@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/7eb872d2f548
Track and adjust layerization on repaint triggers r=mattwoodrow
Comment 6•6 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
status-firefox62:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla62
Yay, this fix has really reduced GPU power as well as improving Motionmark scores. For your test case....
FPS GPU power, Watts Total machine power @ 50% brightness, Watts
Firefox 9 ~16 W ~35 W
Nightly 2018-08-08 58 ~4 W ~20 W
It's also helped with the Reddit r/firefox load times & power, bug 1459875. It's not a complete fix but it helps. I think the reduction in iGPU power allows the CPU to un-throttle a bit, it now gets 7 W up from 5 W (though it would like to go up to around 12 W so it's probably still only at half speed).
time to load,s GPU power, Watts CPU power, Watts
Firefox 8 s ~18 W ~5 W
Nightly 2018-08-08 7 s ~14 W ~7 W
MacBookPro11,1 @1280x800
It's also fixed my CSS animation test case https://codepen.io/davidhc/pen/nLpJk from bug 1422090 comment 37. When zoomed up to 150% this was killing my GPU, no longer.
GPU power, Watts
Firefox ~15 W
Nightly 2018-08-08 ~4 W
(Sorry to drone on about GPU power so much)
You need to log in
before you can comment on or make changes to this bug.
Description
•