Closed Bug 779172 Opened 12 years ago Closed 12 years ago

GC: Incremental zeal modes interact badly with BudgetIncrementalGC

Categories

(Core :: JavaScript Engine, defect)

defect
Not set
minor

Tracking

()

RESOLVED FIXED
mozilla17

People

(Reporter: jonco, Unassigned)

Details

Attachments

(1 file)

Currently IncrementalCollectSlice() determines whether it should do an incremental collection or not by checking whether the slice budget is unlimited, unless any of the incremental zeal modes are in effect in which case it will always perform an incremental slice as determined by the zeal mode.

BudgetIncrementalGC() sets the budget to unlimited to prevent an incremental GC in situations where it is not safe to do so.  This is therefore ignored when these zeal modes are in effect.

The solution is to pass a dummy slice budget that is not unlimited for the zeal mode GCs.  If the budget is later set to unlimited by BudgetIncrementalGC this will then be respectected.
QA Contact: jcoppeard
Attached patch Proposed fixSplinter Review
Attachment #647573 - Flags: review?(wmccloskey)
Attachment #647573 - Flags: review?(wmccloskey) → review+
https://hg.mozilla.org/mozilla-central/rev/053116224bc9
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla17
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: