Closed
Bug 1465086
Opened 7 years ago
Closed 5 years ago
1.92 - 2.01% Heap Unclassified (windows10-64) regression on push a0414a6c423bd33ea684d97458eb7b85eb664afe (Mon May 28 2018)
Categories
(Testing :: General, defect, P3)
Tracking
(firefox62 ?, firefox63 ?)
RESOLVED
WONTFIX
People
(Reporter: igoldan, Unassigned)
References
Details
(Keywords: perf, regression)
We have detected an awsy regression from push:
https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?changeset=a0414a6c423bd33ea684d97458eb7b85eb664afe
As author of one of the patches included in that push, we need your help to address this regression.
Regressions:
2% Heap Unclassified windows10-64 pgo stylo 40,898,630.82 -> 41,721,545.96
2% Heap Unclassified windows10-64 opt stylo 40,588,143.01 -> 41,368,038.70
You can find links to graphs and comparison views for each of the above tests at: https://treeherder.mozilla.org/perf.html#/alerts?id=13504
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://wiki.mozilla.org/AWSY/Tests
Reporter | ||
Updated•7 years ago
|
Flags: needinfo?(matt.woodrow)
Comment 1•7 years ago
|
||
This patch enabled some debugging code that uses memory to do extra tracking. It's under MOZ_DIAGNOSTIC_ASSERT_ENABLED, so won't affect beta. I'll also back it out soon, once the investigation is complete.
Flags: needinfo?(matt.woodrow)
Updated•7 years ago
|
status-firefox62:
--- → affected
Reporter | ||
Comment 2•7 years ago
|
||
(In reply to Matt Woodrow (:mattwoodrow) from comment #1)
> This patch enabled some debugging code that uses memory to do extra
> tracking. It's under MOZ_DIAGNOSTIC_ASSERT_ENABLED, so won't affect beta.
> I'll also back it out soon, once the investigation is complete.
Can you approximate the end of the investigation?
Flags: needinfo?(matt.woodrow)
![]() |
||
Updated•7 years ago
|
Priority: -- → P3
Comment 3•7 years ago
|
||
Sounds this this should only affect nightly (currently 63).
status-firefox63:
--- → ?
Comment 4•5 years ago
|
||
Closing as WONTFIX based on comment 1 as it's difficult to follow up on this after almost 2 years.
Status: NEW → RESOLVED
Closed: 5 years ago
Flags: needinfo?(matt.woodrow)
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•