Closed
Bug 766847
Opened 12 years ago
Closed 12 years ago
FF creates 150MB of dictionary (shapes+tables) for 30MB of objects, Chrome doesn't
Categories
(Core :: JavaScript Engine, defect)
Core
JavaScript Engine
Tracking
()
RESOLVED
INCOMPLETE
People
(Reporter: luke, Unassigned)
References
(Blocks 1 open bug, )
Details
(Whiteboard: [js:p2][MemShrink:P2][games:p2])
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.
Updated•12 years ago
|
Whiteboard: [MemShrink]
Reporter | ||
Comment 1•12 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.
Comment 2•12 years ago
|
||
bug 586842 maybe?
Updated•12 years ago
|
Whiteboard: [MemShrink] → [js:p2][MemShrink]
Updated•12 years ago
|
Whiteboard: [js:p2][MemShrink] → [js:p2][MemShrink:P2]
Updated•12 years ago
|
Whiteboard: [js:p2][MemShrink:P2] → [js:p2][MemShrink:P2][games:p2]
Comment 3•12 years ago
|
||
Bug 827490 just landed. Does it help?
Comment 4•12 years ago
|
||
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
Closed: 12 years ago
Resolution: --- → INCOMPLETE
You need to log in
before you can comment on or make changes to this bug.
Description
•