Closed
Bug 1380398
Opened 7 years ago
Closed 7 years ago
10.98 - 12.74% ts_paint_webext (osx-10-10, windows10-64, windows7-32) regression on push 9dd2156225d334fc9c164017b7c5068273bfb7ca (Tue Jul 11 2017)
Categories
(WebExtensions :: General, defect, P1)
Tracking
(firefox-esr52 unaffected, firefox54 unaffected, firefox55 unaffected, firefox56 fixed)
RESOLVED
FIXED
mozilla56
Tracking | Status | |
---|---|---|
firefox-esr52 | --- | unaffected |
firefox54 | --- | unaffected |
firefox55 | --- | unaffected |
firefox56 | --- | fixed |
People
(Reporter: jmaher, Assigned: kmag)
References
Details
(Keywords: perf, regression, talos-regression, Whiteboard: triaged)
Talos has detected a Firefox performance regression from push:
https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?changeset=9dd2156225d334fc9c164017b7c5068273bfb7ca
As author of one of the patches included in that push, we need your help to address this regression.
Regressions:
13% ts_paint_webext windows10-64 opt e10s 848.92 -> 957.08
12% ts_paint_webext windows7-32 opt e10s 981.25 -> 1,103.50
11% ts_paint_webext osx-10-10 opt e10s 1,388.00 -> 1,540.42
You can find links to graphs and comparison views for each of the above tests at: https://treeherder.mozilla.org/perf.html#/alerts?id=7861
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 authored the patches in bug 1364768, can you take a look at this regression which appears to be on the webext only test.
Flags: needinfo?(kmaglione+bmo)
Comment 2•7 years ago
|
||
This also landed an improvement, alongside regressions from above:
== Change summary for alert #7861 (as of July 11 2017 23:20 UTC) ==
Improvements:
78% tp5n nonmain_startup_fileio windows7-32 opt e10s 29,614.42 -> 6,390.25
Assignee | ||
Comment 3•7 years ago
|
||
This is a timing issue. We were previously (accidentally) delaying most of WebExtension startup until after first paint because the IndexedDB queries took so long. Now, startup happens earlier, which is the intended behavior, but is less performant.
I haven't decided exactly what to do about this yet, but I'm probably going to tweak the last unlanded patch for that bug to delay most of initialization for all extensions except the ones with the webRequestBlocking permission.
Flags: needinfo?(kmaglione+bmo)
Updated•7 years ago
|
Whiteboard: triaged
Assignee | ||
Comment 5•7 years ago
|
||
Yes, bug 1381687 and bug 1382329 should take care of most of the regression.
I'm looking into other potential optimizations before I consider delaying most of the extension load process again, since our actual startup performance overhead is much easier to track this way, and we won't ever be able to completely delay these things when extensions with blocking webRequest permissions are installed.
Flags: needinfo?(kmaglione+bmo)
Comment 6•7 years ago
|
||
Bug 1381687 already brought considerable, but still partial improvements on ts_paint_webext. Will wait for bug 1382329.
Reporter | ||
Comment 7•7 years ago
|
||
it looks like bug 1382329 has all r+ patches, we have 1 week until merge day, possibly we can have this landed and confirmed before then.
Updated•7 years ago
|
Priority: -- → P1
Comment 8•7 years ago
|
||
This regression in ts_paint_webext is fixed. The current results are lower than before the July 11th regressionn.
https://treeherder.mozilla.org/perf.html#/graphs?timerange=2592000&series=%5Bmozilla-inbound,91d0cee6b8905897868871a4aa2a60775208c95c,1%5D&series=%5Bautoland,91d0cee6b8905897868871a4aa2a60775208c95c,0%5D&selected=%5Bmozilla-inbound,91d0cee6b8905897868871a4aa2a60775208c95c%5D
https://treeherder.mozilla.org/perf.html#/graphs?timerange=2592000&series=%5Bmozilla-inbound,43b86e9d30eb1c24ef91a2f7f3610cb4ac75c647,1%5D&series=%5Bautoland,43b86e9d30eb1c24ef91a2f7f3610cb4ac75c647,0%5D&selected=%5Bmozilla-inbound,43b86e9d30eb1c24ef91a2f7f3610cb4ac75c647%5D
https://treeherder.mozilla.org/perf.html#/graphs?timerange=2592000&series=%5Bmozilla-inbound,e3e3940e5854ff1bded84fd00263b7d11059ef79,1%5D&series=%5Bautoland,e3e3940e5854ff1bded84fd00263b7d11059ef79,0%5D&selected=%5Bmozilla-inbound,e3e3940e5854ff1bded84fd00263b7d11059ef79%5D
Assignee: nobody → kmaglione+bmo
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Comment 9•7 years ago
|
||
To be clear the numbers improved after the 26th which is when 1382329 landed.
Updated•7 years ago
|
status-firefox54:
--- → unaffected
status-firefox55:
--- → unaffected
status-firefox56:
--- → fixed
status-firefox-esr52:
--- → unaffected
Depends on: 1382329
Target Milestone: --- → mozilla56
Version: 53 Branch → 56 Branch
Updated•6 years ago
|
Product: Toolkit → WebExtensions
You need to log in
before you can comment on or make changes to this bug.
Description
•