Changeset 120841 in webkit for trunk/Source/JavaScriptCore/heap/IncrementalSweeper.h

Timestamp:

Jun 20, 2012, 11:24:02 AM (13 years ago)

Author:

[email protected]

Message:

Made the incremental sweeper more aggressive
​https://bugs.webkit.org/show_bug.cgi?id=89527

Reviewed by Oliver Hunt.

This is a pre-requisite to getting rid of "berzerker GC" because we need
the sweeper to reclaim memory in a timely fashion, or we'll see a memory
footprint regression.

• heap/IncrementalSweeper.h:
• heap/IncrementalSweeper.cpp:

(JSC::IncrementalSweeper::scheduleTimer): Since the time slice is predictable,
no need to use a data member to record it.

(JSC::IncrementalSweeper::doSweep): Sweep as many blocks as we can in a
small time slice. This is better than sweeping only one block per timer
fire because that strategy has a heavy timer overhead, and artificially
delays memory reclamation.

File:

• trunk/Source/JavaScriptCore/heap/IncrementalSweeper.h (modified) (1 diff)

trunk/Source/JavaScriptCore/heap/IncrementalSweeper.h

@@ -28 +28 @@
     
     unsigned m_currentBlockToSweepIndex;
-    double m_lengthOfLastSweepIncrement; 
     Vector<MarkedBlock*> m_blocksToSweep;
 #else