Closed
Bug 1125221
Opened 9 years ago
Closed 9 years ago
4.66 - 21.5% animation test regression on mozilla-inbound (v.38) Jan 22nd from push ed9225015e22
Categories
(Core :: Layout, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: vaibhav1994, Unassigned)
References
Details
(Keywords: perf, regression, Whiteboard: [talos_regression])
There is a regression on linux32/linux64 and win platforms: http://alertmanager.allizom.org:8080/alerts.html?rev=ed9225015e22&showAll=1&testIndex=0&platIndex=0 for revision ed9225015e22
Updated•9 years ago
|
Blocks: 1122690
Summary: 4.66 - 21.5% animation test regression on mozilla-inbound Jan 22nd from push ed9225015e22 → 4.66 - 21.5% animation test regression on mozilla-inbound (v.38) Jan 22nd from push ed9225015e22
Updated•9 years ago
|
Blocks: CVE-2015-0826
Comment 1•9 years ago
|
||
you can see that we have regressions in CART/TART on a mix of windows/linux in the above link. It appears this is a fix for a security issue, so we might have to accept this as a regression. As this isn't a surprise, it is sort of expected, I would like to know: * is there any optimizations we can make to this to reduce the impact? * if there isn't, I would like to get something kind of conclusion that we understand this and won't fix it right away so we can, understand when we will ship with this and what versions.
Flags: needinfo?(cam)
Comment 2•9 years ago
|
||
this appears to have landed on beta/aurora- since this landed on beta, I want lsblakk to weigh in here on any concerns.
This patch disabled a major piece of recent performance work, so performance regressions are expected.
Comment 4•9 years ago
|
||
Thanks dbaron! I suspect this will be a wontfix bug due to the nature of it- if there is tracking work to re-enable this work, that would be good to somehow associate with this bug!
Depends on: 1125391
Comment 5•9 years ago
|
||
Thanks Joel - I'm tracking bug 1125391 and the bug it relies on so that we hopefully avoid shipping this perf hit in 36.
Flags: needinfo?(lsblakk)
Comment 6•9 years ago
|
||
The fix and re-enabling of the optimisation landed on inbound and Aurora, but had to be backed out of Aurora. I'll continue looking into that early next week.
Flags: needinfo?(cam)
Comment 7•9 years ago
|
||
keep in mind next week Aurora will uplift to Beta. That shouldn't change anything, but worth keeping in mind.
Updated•9 years ago
|
Keywords: perf,
regression
Whiteboard: [talos_regression]
Comment 8•9 years ago
|
||
With bug 1125391 landed, can we close this one?
Comment 9•9 years ago
|
||
yes, these graphs look fine on trunk as of Feb 22nd as well as aurora!
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•