Perform scavenge in idle tasks.

src/heap/heap.h

 // Copyright 2012 the V8 project authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef V8_HEAP_HEAP_H_
 #define V8_HEAP_HEAP_H_
 
 #include <cmath>
 #include <map>
 
 #include "src/allocation.h"
 #include "src/assert-scope.h"
 #include "src/atomic-utils.h"
 #include "src/globals.h"
 #include "src/heap/gc-idle-time-handler.h"
 #include "src/heap/incremental-marking.h"
 #include "src/heap/mark-compact.h"
+#include "src/heap/scavenge-job.h"

[Hannes Payer (out of office), 2015/09/21 09:00:57]

Move this include to heap.cc

 #include "src/heap/spaces.h"
 #include "src/heap/store-buffer.h"
 #include "src/list.h"
 
 namespace v8 {
 namespace internal {
 
 // Defines all the roots in Heap.
 #define STRONG_ROOT_LIST(V)                                                    \
   V(Map, byte_array_map, ByteArrayMap)                                         \
@@ skipping 1152 matching lines @@
   // Inline allocation. ========================================================
   // ===========================================================================
 
   // Indicates whether inline bump-pointer allocation has been disabled.
   bool inline_allocation_disabled() { return inline_allocation_disabled_; }
 
   // Switch whether inline bump-pointer allocation should be used.
   void EnableInlineAllocation();
   void DisableInlineAllocation();
 
+  // Update the inline allocation limit.
+  void LowerInlineAllocationLimit(intptr_t step);
+  void ResetInlineAllocationLimit();

[Hannes Payer (out of office), 2015/09/21 09:00:57]

We should not expose them on the public heap API.

[ulan, 2015/09/23 09:58:38]

Incremental marker calls these functions.

[Hannes Payer (out of office), 2015/09/23 14:16:39]

As discussed offline, we make them friends and do not expose them publicly.

+
   // ===========================================================================
   // Methods triggering GCs. ===================================================
   // ===========================================================================
 
   // Performs garbage collection operation.
   // Returns whether there is a chance that another major GC could
   // collect more garbage.
   inline bool CollectGarbage(
       AllocationSpace space, const char* gc_reason = NULL,
       const GCCallbackFlags gc_callback_flags = kNoGCCallbackFlags);
 
   // Performs a full garbage collection.  If (flags & kMakeHeapIterableMask) is
   // non-zero, then the slower precise sweeper is used, which leaves the heap
   // in a state where we can iterate over the heap visiting all objects.
   void CollectAllGarbage(
       int flags = kFinalizeIncrementalMarkingMask, const char* gc_reason = NULL,
       const GCCallbackFlags gc_callback_flags = kNoGCCallbackFlags);
 
   // Last hope GC, should try to squeeze as much as possible.
   void CollectAllAvailableGarbage(const char* gc_reason = NULL);
 
   // Invoked when GC was requested via the stack guard.
   void HandleGCRequest();
 
+  // Schedule idle scavenge job.
+  void ScheduleIdleScavenge(int bytes_allocated);

[Hannes Payer (out of office), 2015/09/21 09:00:57]

This one should be private as well.

[ulan, 2015/09/23 09:58:38]

This is called by NewSpace on interrupt.

+
   // ===========================================================================
   // Iterators. ================================================================
   // ===========================================================================
 
   // Iterates over all roots in the heap.
   void IterateRoots(ObjectVisitor* v, VisitMode mode);
   // Iterates over all strong roots in the heap.
   void IterateStrongRoots(ObjectVisitor* v, VisitMode mode);
   // Iterates over entries in the smi roots list.  Only interesting to the
   // serializer/deserializer, since GC does not care about smis.
@@ skipping 1034 matching lines @@
   StoreBuffer store_buffer_;
 
   IncrementalMarking incremental_marking_;
 
   GCIdleTimeHandler gc_idle_time_handler_;
 
   MemoryReducer* memory_reducer_;
 
   ObjectStats* object_stats_;
 
+  ScavengeJob scavenge_job_;

[Hannes Payer (out of office), 2015/09/21 09:00:57]

If you make ScavengeJob a ScavengeJob*, you don't have to include it in heap.h

[ulan, 2015/09/23 09:58:38]

Done.

+
   // These two counters are monotomically increasing and never reset.
   size_t full_codegen_bytes_generated_;
   size_t crankshaft_codegen_bytes_generated_;
 
   // This counter is increased before each GC and never reset.
   // To account for the bytes allocated since the last GC, use the
   // NewSpaceAllocationCounter() function.
   size_t new_space_allocation_counter_;
 
   // This counter is increased before each GC and never reset. To
@@ skipping 421 matching lines @@
   DisallowHeapAllocation no_allocation;  // i.e. no gc allowed.
 
  private:
   DISALLOW_IMPLICIT_CONSTRUCTORS(PathTracer);
 };
 #endif  // DEBUG
 }
 }  // namespace v8::internal
 
 #endif  // V8_HEAP_HEAP_H_