138.44% build times (linux64) regression on push 469d7894f720cc63692dd23c9e0e9ace73bb9fbf (Tue March 3 2020)
Categories
(Core :: Panning and Zooming, defect)
Tracking
()
Tracking | Status | |
---|---|---|
thunderbird_esr60 | --- | unaffected |
thunderbird_esr68 | --- | unaffected |
firefox-esr68 | --- | unaffected |
firefox74 | --- | unaffected |
firefox75 | --- | wontfix |
firefox76 | --- | wontfix |
People
(Reporter: marauder, Unassigned)
References
(Regression)
Details
(Keywords: perf-alert, regression)
A few notes here:
- the graph doesn't show all the data points, but in treeherder we can see all the jobs
- the regression occured on 469d7894f720cc63692dd23c9e0e9ace73bb9fbf [1]
where "build times tsan taskcluster-c5.4xlarge tsan-fuzzing" is 1286.72
- the data point d1cc91327f3f which is before the changeset from [1], has:
build times tsan taskcluster-c5d.4xlarge tsan-fuzzing: 483.63 - the data point 9cf4382f8882 which is after the changeset from [1], has:
build times tsan taskcluster-c5d.4xlarge tsan-fuzzing: 1054.38
Because the data point where the regression occurred is not visible on the graph, i could not create a new alert and reassign the one opened by perfherder for 9cf4382f88826.
We have detected a build metrics regression from push:
As author of one of the patches included in that push, we need your help to address this regression.
Regressions:
138% build times linux64 tsan taskcluster-c5d.4xlarge tsan-fuzzing 498.40 -> 1,188.39
You can find links to graphs and comparison views for each of the above tests at: https://treeherder.mozilla.org/perf.html#/alerts?id=25418
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/Automated_Performance_Testing_and_Sheriffing/Build_Metrics
*** Please let us know your plans within 3 business days, or the offending patch(es) will be backed out! ***
Reporter | ||
Updated•5 years ago
|
Reporter | ||
Comment 1•5 years ago
|
||
A few updates:
Looking at the jobs in treeherder we can see that on the backout for 469d7894f720cc63692dd23c9e0e9ace73bb9fbf :
82e82612011b - Backed out changeset 469d7894f720 (Bug 1618545) for causing bustages in src/layout/base/PositionedEventTargeting.cpp
- the value for "build times tsan taskcluster-c5.4xlarge tsan-fuzzing" is 1137.52 , which is still high so it seems that the main changeset is not related to the regression.
Considering the noise from the graph it's also an option to close this bug as invalid/won't fix.
Reporter | ||
Updated•5 years ago
|
Updated•5 years ago
|
Comment 2•5 years ago
•
|
||
Yeah that graph looks really noisy. The patch in question mostly deleted a bunch of code so it's quite improbable that it caused an increase in compiler time. I think this one should get closed as invalid.
Updated•5 years ago
|
Description
•