Closed Bug 1285605 Opened 8 years ago Closed 8 years ago

Report the number of Zones and Compartments swept in a GC

Categories

(Core :: JavaScript: GC, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla50
Tracking Status
firefox50 --- fixed

People

(Reporter: terrence, Assigned: terrence)

References

Details

Attachments

(1 file)

We currently tell you how many are collected, but not whether or not we freed any. The old stats look like: Compartments Collected: 122 of 1749 The new stats add the amount removed after in parentheses: Compartments Collected: 122 of 1749 (-42)
Attachment #8769280 - Flags: review?(sphink)
Comment on attachment 8769280 [details] [diff] [review] stats_on_swept_zones_compartments-v0.diff Review of attachment 8769280 [details] [diff] [review]: ----------------------------------------------------------------- Thanks. I think I've meant to do this a few times, but always got distracted. ::: js/src/gc/Statistics.cpp @@ +391,2 @@ > zoneStats.collectedCompartmentCount, zoneStats.compartmentCount, > + zoneStats.sweptCompartmentCount, Looks a little weird here, because the heap delta gets its own header but the compartments delta doesn't. But it looks like that's because the heap delta is in terms of chunks. So never mind, it's fine. I suppose you could change it to "Heap Chunks: %d (%+d); ", but whatever.
Attachment #8769280 - Flags: review?(sphink) → review+
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla50
Depends on: 1288566
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: