Open
Bug 1022834
Opened 10 years ago
Updated 2 years ago
Telemetry reports shutdown time doubled in Nightly 32
Categories
(Core :: General, defect)
Tracking
()
NEW
People
(Reporter: vladan, Unassigned)
References
Details
According to Telemetry, the median shutdown time in Nightly 32 has more than doubled, starting with the May 30th Nightly.
http://telemetry.mozilla.org/#filter=nightly%2F32%2FSIMPLE_MEASURES_SHUTDOWNDURATION!nightly%2F31%2FSIMPLE_MEASURES_SHUTDOWNDURATION!nightly%2F30%2FSIMPLE_MEASURES_SHUTDOWNDURATION!nightly%2F29%2FSIMPLE_MEASURES_SHUTDOWNDURATION!nightly%2F28%2FSIMPLE_MEASURES_SHUTDOWNDURATION!nightly%2F27%2FSIMPLE_MEASURES_SHUTDOWNDURATION&aggregates=median&evoOver=Builds&locked=true&sanitize=true&renderhistogram=Table
Comment 1•10 years ago
|
||
In some of the newer slow shutdown profiles I had seen that we were doing more expensive more NukeCrossCompartement. Last year killing those was one of our biggest win with shutdown performance. I'd suggestion first investigating that this could did not get brought back on shutdown.
Comment 2•10 years ago
|
||
Do we have a regression range for May 30th?
Reporter | ||
Comment 3•10 years ago
|
||
(In reply to David Rajchenbach Teller [:Yoric] from comment #2)
> Do we have a regression range for May 30th?
No, we don't. The Talos shutdown test is not useful (bug 1022824) :(
Comment 4•10 years ago
|
||
Fwiw, May 30th appears to match to https://tbpl.mozilla.org/?fromchange=1e712b724d17&tochange=e6f113c83095
Comment 5•10 years ago
|
||
On a hunch, possible suspects:
- Asaf Romano – Bug 1016831 - make sure the shared, readonly-db-connection is closed on shutdown. r=mak, brain=yorik
- Kyle Huey – Bug 965309: Shut down the sync loop, if any, when we shut down the XHR. r=bent
- Shian-Yow Wu – Bug 1010784: Null out sync loop target to avoid assertions when reusing sync XHR. r=khuey
- Dan Gohman – Bug 1005113 - IonMonkey: Mark more LUses as AtStart on x86/x64. r=bhackett
- David Rajchenbach-Teller – Bug 917883 - Use AsyncShutdown instead of spinning the event loop in healthreporter.jsm. r=gps, r=rnewman
Updated•2 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•