Closed
Bug 659839
Opened 13 years ago
Closed 12 years ago
IonMonkey: Implement OOM Ballast
Categories
(Core :: JavaScript Engine, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: dvander, Unassigned)
References
(Blocks 1 open bug)
Details
As of bug 659782 the allocator is treated as infallible for small, fix-sized structures. The ballast isn't actually implemented though, so at some point we'll have to modify IonAllocPolicy::ensureBallast to reserve space in the arena.
Comment 1•13 years ago
|
||
A tip from TM: be careful where you put the ballast. TM has some ballast (for nanojit) in every compartment, and you can have a lot of compartments, so that's just wasted space. It's much better if you can put it in the JSRuntime, say. Though I guess you then have locking considerations. Hmm. Ballast kinda sucks.
Reporter | ||
Comment 2•13 years ago
|
||
Would the threaddata be okay in terms of memory use? The compiler won't re-enter and the ballast could live and die with the compiler, if need be.
Comment 3•13 years ago
|
||
(In reply to comment #2) > Would the threaddata be okay in terms of memory use? The compiler won't > re-enter and the ballast could live and die with the compiler, if need be. Sounds ideal. Do you mean ThreadData in jscntxt.h? I wonder if the tracer ballast could be moved in there as well.
Reporter | ||
Comment 4•12 years ago
|
||
This got fixed a while ago.
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•