Closed
Bug 1138390
Opened 10 years ago
Closed 10 years ago
runOffThreadScript() can hang if incremental GC is in progress
Categories
(Core :: JavaScript: GC, defect)
Core
JavaScript: GC
Tracking
()
RESOLVED
FIXED
mozilla39
People
(Reporter: jonco, Assigned: jonco)
References
Details
Attachments
(1 file)
2.34 KB,
patch
|
terrence
:
review+
lsblakk
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
The testcase for bug 1137341 hangs in opt builds with the fix applied.
The cause is that the runOffThreadScript() shell function waits for the off-thread compilation to finish without taking into account that it might have been deferred due to an ongoing GC of the atoms zone.
The testcase only causes this on opt builds because it tries to schedule a zone GC, but schedulegc() is not present on opt builds but is replaced by a dummy function, so a full GC including the atoms zone is triggered instead.
Assignee | ||
Comment 1•10 years ago
|
||
This fix is to wait for any ongoing incremental GC to finish in runOffThreadScript().
Attachment #8571440 -
Flags: review?(terrence)
Updated•10 years ago
|
Attachment #8571440 -
Flags: review?(terrence) → review+
Assignee | ||
Comment 2•10 years ago
|
||
Status: NEW → RESOLVED
Closed: 10 years ago
status-firefox39:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla39
Assignee | ||
Comment 4•10 years ago
|
||
Comment on attachment 8571440 [details] [diff] [review]
bug1138390-runOffThreadScript
Approval Request Comment
[Feature/regressing bug #]: Requesting this as a test dependency for the uplift of bug 1137341.
[User impact if declined]: None
[Describe test coverage new/current, TreeHerder]: On central for over a week.
[Risks and why]: None, shell-only changes.
[String/UUID change made/needed]: None.
Attachment #8571440 -
Flags: approval-mozilla-aurora?
Updated•10 years ago
|
Attachment #8571440 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Updated•10 years ago
|
status-firefox38:
--- → affected
Assignee | ||
Comment 5•10 years ago
|
||
Updated•10 years ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•