FF creates 150MB of dictionary (shapes+tables) for 30MB of objects, Chrome doesn't

RESOLVED INCOMPLETE

Status

()

defect
RESOLVED INCOMPLETE
7 years ago
6 years ago

People

(Reporter: luke, Unassigned)

Tracking

(Depends on 1 bug, Blocks 1 bug)

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [js:p2][MemShrink:P2][games:p2], )

Reporter

Description

7 years ago
Brendan saw this in a JSC bug (bugs.webkit.org/show_bug.cgi?id=89622).  This game:
  http://bigsword.sinaapp.com/game.html
seems to create 114MB of dictionary shapes and 32MB of dictionary tables for 30MB of objects.  v8 seems to create 34MB total so it would seem like there is some specific thing SM is missing that is preventing a ton of objects from reusing shapes in the property tree.
Whiteboard: [MemShrink]
Reporter

Comment 1

7 years ago
A quick glance at Chrome's memory snapshot tool seems to indicate most of the memory is in arrays.  That would perhaps indicate that our lack of named properties on dense arrays is what is killing us.

Waldo: I cannot for the life of me get bugzilla to give me the dense/sparse slots bug.  I'd like to block this on it, pending confirmation that slow arrays are actually the problem here.
Depends on: 586842
Whiteboard: [MemShrink] → [js:p2][MemShrink]
Whiteboard: [js:p2][MemShrink] → [js:p2][MemShrink:P2]
Whiteboard: [js:p2][MemShrink:P2] → [js:p2][MemShrink:P2][games:p2]
Bug 827490 just landed.  Does it help?
The URL is no longer valid.  I got a warning screen in Chinese, which Google Translate translates to:

  You access the website failure
  You visit the site exceeds the quota, may be caused by the following reasons:
  1 the site Yundou depleted;
  The site of more than a minute quota;
  3. "Budget set" too low. 

After a few seconds it redirected me to http://yunshangdian.com/?c=wiki&page=intro.

I will mark this INCOMPLETE.  We have numerous similar bugs blocking bug 586842, so we don't really need this one.
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → INCOMPLETE
You need to log in before you can comment on or make changes to this bug.