Closed Bug 1397622 Opened 7 years ago Closed 7 years ago

2.25 - 3.16% JS (windows10-64) regression on push 55e08f65ac23b380ca995319032f07121d46bf71 (Mon Sep 4 2017)

Categories

(Core :: JavaScript: GC, defect)

defect
Not set
normal

Tracking

()

RESOLVED DUPLICATE of bug 1397314
Tracking Status
firefox55 --- unaffected
firefox56 --- unaffected
firefox57 --- fixed

People

(Reporter: igoldan, Assigned: pbone)

References

Details

(Keywords: perf, regression)

We have detected an awsy regression from push:

https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?changeset=55e08f65ac23b380ca995319032f07121d46bf71

As author of one of the patches included in that push, we need your help to address this regression.

Regressions:

  3%  JS summary windows10-64 pgo      123,164,055.50 -> 127,051,493.46
  2%  JS summary windows10-64 opt      123,982,386.48 -> 126,769,852.04


You can find links to graphs and comparison views for each of the above tests at: https://treeherder.mozilla.org/perf.html#/alerts?id=9266

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 jobs in a pushlog format.

To learn more about the regressing test(s), please see: https://developer.mozilla.org/en-US/docs/Mozilla/Performance/AWSY
Paul, these are the AWSY regressions we observed, alongside the Talos one, mentioned in bug 1397314.
Flags: needinfo?(pbone)
Component: Untriaged → JavaScript: GC
Product: Firefox → Core
I'm working on this already, same comment as bug 1397314

It's plausable that my change created this regression.

My change can affect the tuning of the JavaScript GC Nursery size which will affect this metric. I'll do some testing and see if I can create a better policy/heuristic for setting the nursery size.
Assignee: nobody → pbone
Status: NEW → ASSIGNED
Flags: needinfo?(pbone)
This was resolved with the fix to Bug 1397314
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → DUPLICATE
Mirror the status of bug 1397314.
You need to log in before you can comment on or make changes to this bug.