Closed
Bug 1407984
Opened 7 years ago
Closed 7 years ago
1.87 - 4.77% tp5o_webext (linux64, windows10-64, windows7-32) regression on push 7a4a9b65e2e773ea74d5485e6ddbf97ab3160252 (Thu Oct 12 2017)
Categories
(WebExtensions :: Request Handling, defect)
WebExtensions
Request Handling
Tracking
(Not tracked)
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/integration/mozilla-inbound/pushloghtml?changeset=7a4a9b65e2e773ea74d5485e6ddbf97ab3160252 As author of one of the patches included in that push, we need your help to address this regression. Regressions: 5% tp5o_webext summary windows10-64 pgo e10s 282.10 -> 295.54 4% tp5o_webext summary windows7-32 pgo e10s 278.97 -> 291.17 2% tp5o_webext summary linux64 pgo e10s 349.09 -> 356.45 2% tp5o_webext summary windows7-32 opt e10s 377.99 -> 385.07 You can find links to graphs and comparison views for each of the above tests at: https://treeherder.mozilla.org/perf.html#/alerts?id=9975 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 → WebExtensions: General
Product: Firefox → Toolkit
Reporter | ||
Comment 1•7 years ago
|
||
:kmag I see you own bug 1404652. Can we fix these performance regressions? Or should we consider backout/accepting them?
Flags: needinfo?(kmaglione+bmo)
Comment 2•7 years ago
|
||
This is somewhat expected. We're essentially trading better responsiveness for somewhat higher request latency. The latency shows up in the tp5o numbers, but is generally not visible to users. The difference in responsiveness, however, can be significant. So I suggest we accept these regressions.
Flags: needinfo?(kmaglione+bmo)
Updated•7 years ago
|
Component: WebExtensions: General → WebExtensions: Request Handling
Reporter | ||
Updated•7 years ago
|
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → WONTFIX
Updated•6 years ago
|
Product: Toolkit → WebExtensions
You need to log in
before you can comment on or make changes to this bug.
Description
•