Fix description of js-gc-heap-unused-fraction

RESOLVED FIXED in mozilla11

Status

()

Core
JavaScript Engine
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: terrence, Assigned: terrence)

Tracking

Trunk
mozilla11
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

6 years ago
Created attachment 574476 [details] [diff] [review]
v1: trivial

This did not get updated correctly in bug 670596.
Attachment #574476 - Flags: review?(nnethercote)
Comment on attachment 574476 [details] [diff] [review]
v1: trivial

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

::: js/xpconnect/src/XPCJSRuntime.cpp
@@ +2023,5 @@
>                                 data.gcHeapUnusedPercentage,
>                                 "Fraction of the garbage-collected JavaScript heap that is unused. "
>                                 "Computed as ('js-gc-heap-chunk-clean-unused' + "
> +                               "'js-gc-heap-chunk-dirty-unused' + 'js-gc-heap-decommitted' + "
> +                               "'js-gc-heap-arena-unused') / 'js-gc-heap'.",

I would have made 'js-gc-heap-decommitted' the 3rd operand in the sum to keep the two 'unused' reports together, but it's not a big deal.
Attachment #574476 - Flags: review?(nnethercote) → review+

Comment 2

6 years ago
Could we also remove the useless inner loop "while (chunk)" loop in DecommitFreePages? The outer loop over all chunks is enough to get all the chunks.
(Assignee)

Comment 3

6 years ago
https://tbpl.mozilla.org/?tree=Mozilla-Inbound&rev=2057f46bf229
http://hg.mozilla.org/integration/mozilla-inbound/rev/2057f46bf229

and 

https://bugzilla.mozilla.org/show_bug.cgi?id=702681

Comment 4

6 years ago
https://hg.mozilla.org/mozilla-central/rev/2057f46bf229
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla11
You need to log in before you can comment on or make changes to this bug.