564060 - TM: Implement arena pool chunk time-deferred "soft release"

Status: RESOLVED DUPLICATE of bug 684039

(Core :: JavaScript Engine, enhancement, P5)

Description

[Chris Leary [:cdleary] (not checking bugmail)]

Never filed a bug for this -- an interesting experiment stemming from bug 561286:

"""you can use time locality to let the arenas sit and age until
the next garbage collection. For imacro_asm, this seems particularly useful in
regular expression evaluation."""

Comment 1

[Chris Leary [:cdleary] (not checking bugmail)]

Adding reference to regexp pool fix bug. I've actually implemented and tested a single-big-chunk arena strategy that shows no speedup in the shell, but still has to be tested in browser. Will post patches for that tomorrow.

Comment 2

[Chris Leary [:cdleary] (not checking bugmail)]

Attachment: Experiment: Fat arena.

Make an arena template class in arenang.h that just uses a huge primary chunk. For this part of the experiment I just made it sufficiently huge that secondary chunks are unnecessary for benchmark runs.

Comment 3

[Chris Leary [:cdleary] (not checking bugmail)]

Attachment: Experiment: Fat arena.

It helps when you add new files...

Comment 4

[Chris Leary [:cdleary] (not checking bugmail)]

Attachment: Experiment: Fat arena.

Rebase.

Comment 5

[Chris Leary [:cdleary] (not checking bugmail)]

To do: we can also eliminate all the emit checks with a sufficiently large single primary chunk.

Comment 6

[Chris Leary [:cdleary] (not checking bugmail)]

My observation was that this wouldn't give us much of a speedup, so I'm downgrading to my "cleanup" priority. :-)