Report the number of Zones and Compartments swept in a GC

RESOLVED FIXED in Firefox 50

Status

()

defect
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: terrence, Assigned: terrence)

Tracking

Trunk
mozilla50
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox50 fixed)

Details

Attachments

(1 attachment)

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+
https://hg.mozilla.org/mozilla-central/rev/c686dd1366a3
Status: ASSIGNED → RESOLVED
Closed: 3 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.