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)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla39
Tracking Status
firefox38 --- fixed
firefox39 --- fixed

People

(Reporter: jonco, Assigned: jonco)

References

Details

Attachments

(1 file)

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.
Blocks: 1137341
This fix is to wait for any ongoing incremental GC to finish in runOffThreadScript().
Attachment #8571440 - Flags: review?(terrence)
Attachment #8571440 - Flags: review?(terrence) → review+
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla39
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?
Attachment #8571440 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: