Closed Bug 1452761 Opened 7 years ago Closed 7 years ago

2.84% kraken (linux64) regression on push 81ef11104ebb7871c6efe1794a40d4505a80c8dd (Sun Apr 8 2018)

Categories

(Core :: JavaScript Engine: JIT, defect)

defect
Not set
normal

Tracking

()

RESOLVED WONTFIX
Tracking Status
firefox-esr52 --- unaffected
firefox59 --- unaffected
firefox60 --- unaffected
firefox61 - wontfix

People

(Reporter: jmaher, Unassigned)

References

Details

(Keywords: perf, regression, talos-regression)

Talos has detected a Firefox performance regression from push: https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?changeset=81ef11104ebb7871c6efe1794a40d4505a80c8dd As author of one of the patches included in that push, we need your help to address this regression. Regressions: 3% kraken linux64 opt e10s stylo 898.79 -> 924.30 You can find links to graphs and comparison views for each of the above tests at: https://treeherder.mozilla.org/perf.html#/alerts?id=12588 On the page above you can see an alert for each affected platform as well as a link to a graph showing the history of scores for this test. There is also a link to a treeherder page showing the Talos jobs in a pushlog format. To learn more about the regressing test(s), please see: https://wiki.mozilla.org/Buildbot/Talos/Tests For information on reproducing and debugging the regression, either on try or locally, see: https://wiki.mozilla.org/Buildbot/Talos/Running *** Please let us know your plans within 3 business days, or the offending patch(es) will be backed out! *** Our wiki page outlines the common responses and expectations: https://wiki.mozilla.org/Buildbot/Talos/RegressionBugsHandling
Component: General → JavaScript Engine: JIT
Flags: needinfo?(jdemooij)
Product: Firefox → Core
(In reply to Ryan VanderMeulen [:RyanVM] from comment #2) > https://bugzilla.mozilla.org/show_bug.cgi?id=1448887#c10 makes it sound like > they were expected. Yeah it's unfortunate but bug 1448887 fixes a crash (bug 1446907), removes some really complicated (and platform-specific) code, and should improve performance in some other cases (workers) so we decided landing that was the right trade-off.
Flags: needinfo?(jdemooij)
Resolving as wontfix as it was an expected change with trade-offs weighed and accepted.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → WONTFIX
Version: 53 Branch → Trunk
You need to log in before you can comment on or make changes to this bug.