Closed
Bug 928056
Opened 8 years ago
Closed 8 years ago
Unbreak GGC on AWFY
Categories
(Core :: JavaScript Engine, defect)
Core
JavaScript Engine
Tracking
()
RESOLVED
FIXED
mozilla27
People
(Reporter: terrence, Assigned: terrence)
References
(Blocks 1 open bug)
Details
Attachments
(2 files)
9.70 KB,
patch
|
bhackett1024
:
review+
|
Details | Diff | Splinter Review |
1.39 KB,
patch
|
billm
:
review+
|
Details | Diff | Splinter Review |
When IonBuilder moved off main thread, we started blocking on completion of background threads in AutoTraceSession. For normal GC's this isn't such a big deal, but when it happens every minor collection, we get a catastrophic slowdown. This was slow enough that it took AWFY completely down last week. See bug 928050 for the complete fix: this is just a quick hack to make GGC work again.
Assignee | ||
Comment 1•8 years ago
|
||
This unbundles AutoPauseWorkersForGC from AutoTraceSession and renames it to AutoPauseWorkersForTracing. I then manually re-added the pause all the places it was before except for generational GC's. This takes my local time for running octane from 8m30s -> 30s.
Attachment #819062 -
Flags: review?(bhackett1024)
Updated•8 years ago
|
Attachment #819062 -
Flags: review?(bhackett1024) → review+
Assignee | ||
Comment 2•8 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/61f6c63bcb3d
Assignee | ||
Comment 3•8 years ago
|
||
This changed the ordering of swapping the state and pausing in AutoGCSession, which causes us to occasionally fire spurious assertions. This will be fixed by bug 928050, but in the meantime we should stop the intermittent orange by fixing the ordering.
Attachment #819139 -
Flags: review?(bhackett1024)
Assignee | ||
Updated•8 years ago
|
Attachment #819139 -
Flags: review?(bhackett1024) → review?(wmccloskey)
Attachment #819139 -
Flags: review?(wmccloskey) → review+
Assignee | ||
Comment 4•8 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/d6180c846c36
Assignee | ||
Comment 5•8 years ago
|
||
And backed out because I forgot about runtime and apparently can't keep my patches straight when testing. https://hg.mozilla.org/integration/mozilla-inbound/rev/3b459fed3fc6
Comment 6•8 years ago
|
||
Let's get the original landing while we're at it. https://hg.mozilla.org/integration/mozilla-inbound/rev/a46ff1a56160
Comment 7•8 years ago
|
||
BTW, these also showed up on your original push: https://tbpl.mozilla.org/php/getParsedLog.php?id=29327224&tree=Mozilla-Inbound
Assignee | ||
Comment 8•8 years ago
|
||
Once more unto the breach: https://hg.mozilla.org/integration/mozilla-inbound/rev/b1ce50371389
https://hg.mozilla.org/mozilla-central/rev/b1ce50371389
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla27
You need to log in
before you can comment on or make changes to this bug.
Description
•