Release/protect arenas sooner in compacting GC

RESOLVED FIXED in Firefox 42

Status

()

defect
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: jonco, Assigned: jonco)

Tracking

(Blocks 1 bug)

unspecified
mozilla42
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox42 fixed)

Details

Attachments

(1 attachment)

Although compacting GC is incremental, we currently wait till the end before we release or protect and arenas we reclaimed (protection is to catch stray accessed by the mutator to things it should no longer have pointers to).  It would be better to do this when we have finished compacting a zone.
Comment on attachment 8640552 [details] [diff] [review]
release-relocated-arenas-earlier

Review of attachment 8640552 [details] [diff] [review]:
-----------------------------------------------------------------

::: js/src/jsgc.cpp
@@ +2087,5 @@
> +static inline bool
> +ShouldProtectRelocatedArenas(JS::gcreason::Reason reason)
> +{
> +    // For zeal mode collections we don't release the relocated arenas
> +    // immediately.  Instead we protect them and keep them around until the next

2 spaces.
Attachment #8640552 - Flags: review?(terrence) → review+
https://hg.mozilla.org/mozilla-central/rev/cad7572d811e
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla42
You need to log in before you can comment on or make changes to this bug.