Closed
Bug 1392573
Opened 7 years ago
Closed 7 years ago
2.04 - 22.01% cart / tart / tp5o Main_RSS / tp5o responsiveness / tp5o_webext Main_RSS / tp5o_webext responsiveness / tpaint / tsvg_static (linux64, osx-10-10, windows10-64, windows7-32) regression on push a9df4b7dd62f (Wed Aug 16 2017)
Categories
(Firefox :: Screenshots, defect)
Firefox
Screenshots
Tracking
()
RESOLVED
WONTFIX
People
(Reporter: igoldan, Unassigned)
References
Details
(Keywords: perf, regression, talos-regression)
Talos has detected a Firefox performance regression from push: https://hg.mozilla.org/releases/mozilla-beta/rev/a9df4b7dd62f As author of one of the patches included in that push, we need your help to address this regression. Regressions: 22% tp5o responsiveness windows10-64 opt e10s 2.60 -> 3.17 21% tp5o responsiveness linux64 opt e10s 2.85 -> 3.45 19% tsvg_static summary windows7-32 opt e10s 49.28 -> 58.70 11% tp5o_webext responsiveness linux64 opt e10s 3.20 -> 3.55 7% tsvg_static summary osx-10-10 opt e10s 54.77 -> 58.46 5% tp5o Main_RSS linux64 opt e10s 161,038,768.02 -> 169,385,608.16 5% tp5o Main_RSS windows7-32 opt e10s 112,824,700.39 -> 118,028,869.02 3% tpaint summary windows7-32 opt e10s 201.05 -> 208.06 3% tart summary windows7-32 opt e10s 4.96 -> 5.11 2% tp5o_webext Main_RSS linux64 opt e10s 167,098,178.34 -> 170,918,726.09 2% cart summary windows7-32 opt e10s 22.55 -> 23.01 You can find links to graphs and comparison views for each of the above tests at: https://treeherder.mozilla.org/perf.html#/alerts?id=8861 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
Reporter | ||
Updated•7 years ago
|
Component: Untriaged → Screenshots
Product: Firefox → Cloud Services
Reporter | ||
Comment 1•7 years ago
|
||
Jared, I see you are the owner of bug 1386333. Where these regressions expected, or should we look for an improvement/backout?
Flags: needinfo?(jhirsch)
Comment 2•7 years ago
|
||
as a note on August 1st this same pref landed on trunk: https://hg.mozilla.org/mozilla-central/rev/ba769a1d2f82
Comment 3•7 years ago
|
||
as a note, these seem to for the most part erase gains we saw while uplifting 56 to beta, put another way this is back to parity for many tests that we shipped firefox 55 with. There are a few exceptions: 5% tp5o Main_RSS windows7-32 opt e10s 112,824,700.39 -> 118,028,869.02 7% tsvg_static summary osx-10-10 opt e10s 54.77 -> 58.46 3% tpaint summary windows7-32 opt e10s 201.05 -> 208.06 2% cart summary windows7-32 opt e10s 22.55 -> 23.01 tsvg_static has gone up/down, I suspect this might be an artifact of pgo: https://treeherder.mozilla.org/perf.html#/graphs?timerange=7776000&series=mozilla-inbound,1455931,0,1&series=mozilla-beta,1533495,1,1&series=mozilla-inbound,1455996,1,1 as for tpaint, I am not sure, it is small and measures the time to create a new window. the cart regression is small, and this is probably ok. the tp5o main_rss regression, seems to be 100% related to this- at 5%, maybe that is the overhead for webext + screenshots? Given that we rely heavily on PGO for linux/windows, there is some [un]lucky draws we take when changing code- I think we should focus on the smaller list, specifically on the tsvg_static (os) and tp5o Main_RSS on windows 7.
Updated•7 years ago
|
Flags: needinfo?(jhirsch)
Comment 4•7 years ago
|
||
I didn't mean to unset the needinfo, I was just CCing some people. I'm looking at this now.
Comment 5•7 years ago
|
||
These are the same regressions we already decided to eat when we originally preffed this on in nightly. The responsiveness and tsvg regressions are unfortunately most likely the result of enabling screenshots shortly after startup, rather than before first paint. I suppose we could try to shove some of that work into idle slices, but it may be easier said than done, since most of it is async, and we wouldn't want to do that for the before-first-paint loads.
Comment 6•7 years ago
|
||
(In reply to Kris Maglione [:kmag] from comment #5) > These are the same regressions we already decided to eat when we originally > preffed this on in nightly. > > The responsiveness and tsvg regressions are unfortunately most likely the > result of enabling screenshots shortly after startup, rather than before > first paint. I suppose we could try to shove some of that work into idle > slices, but it may be easier said than done, since most of it is async, and > we wouldn't want to do that for the before-first-paint loads. Is that something that you think people would be comfortable uplifting to Beta at this point, or do you think it's too complex of a patch for that? Is there anything we can do in the add-on at this point? Any other ideas for improving this in the Beta timeline?
Flags: needinfo?(kmaglione+bmo)
Comment 7•7 years ago
|
||
I talked to Kris on IRC. He does not think the idle slice patch would be upliftable and it is not an add-on patch.
Flags: needinfo?(kmaglione+bmo)
Comment 8•7 years ago
|
||
As Kris pointed out, there is a lot of overlap with the regressions we accepted in bug 1361792. Kris has a bunch of performance improvements landing in Nightly, but they aren't good candidates to uplift to address this in Beta. I confirmed with Jeff and Dave that we should accept the same regressions we did in Nightly and let this bug ride in Beta.
Comment 9•7 years ago
|
||
Assuming these are regressions are ones we've already accepted in Nightly, we have sign off from Jeff G. and Dave C. to accept these in Beta as well.
Comment 10•7 years ago
|
||
Per comments 8/9.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → WONTFIX
Updated•7 years ago
|
Product: Cloud Services → Firefox
You need to log in
before you can comment on or make changes to this bug.
Description
•