The default bug view has changed. See this FAQ.

Add memory reporter for the atoms table

RESOLVED FIXED in mozilla8

Status

()

Core
JavaScript Engine
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: njn, Assigned: njn)

Tracking

(Blocks: 1 bug)

unspecified
mozilla8
x86_64
Linux
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [MemShrink:P2])

Attachments

(1 attachment)

(Assignee)

Description

6 years ago
Created attachment 551685 [details] [diff] [review]
patch

DMD found that the atoms table is quite big.  E.g. after loading Gmail it's just over 2MB (1.5%) on my Linux64 box.

This patch also adds a "runtime" reporter, which is 282KB on my box.  Not that big, but doesn't hurt to include it.
Attachment #551685 - Flags: review?(luke)
(Assignee)

Updated

6 years ago
Blocks: 563700
Whiteboard: [MemShrink]

Comment 1

6 years ago
Comment on attachment 551685 [details] [diff] [review]
patch

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

On the subject of the large atom state, it would be interested to measure how many of those atoms have been "interned" since interned strings are never swept and thus could be a source of leaks.  This is pretty easy to derive (see js_TraceAtomState).
Attachment #551685 - Flags: review?(luke) → review+
(Assignee)

Updated

6 years ago
Whiteboard: [MemShrink] → [MemShrink:P2]
http://hg.mozilla.org/mozilla-central/rev/b391327ee19a
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla8
You need to log in before you can comment on or make changes to this bug.