diff options
author | Mathieu Chartier <mathieuc@google.com> | 2014-03-19 17:08:17 -0700 |
---|---|---|
committer | Mathieu Chartier <mathieuc@google.com> | 2014-03-20 09:17:02 -0700 |
commit | 1ad2784ad9f311ebf9fe0677d33818648f423f9c (patch) | |
tree | 3dbc930a653615e074cc4965ded9e0cc6ed07a45 /runtime/gc/heap.cc | |
parent | 66e4c3e96dccdec7423d673ad6bbf7821a776651 (diff) | |
download | art-1ad2784ad9f311ebf9fe0677d33818648f423f9c.zip art-1ad2784ad9f311ebf9fe0677d33818648f423f9c.tar.gz art-1ad2784ad9f311ebf9fe0677d33818648f423f9c.tar.bz2 |
Add soft reference pre processing.
Soft reference pre-processing does soft reference preservation with
mutators running. After this is done, it does another pass with
mutators paused in the ProcessReference code. This helps lower pauses
since most preserved soft references have their referents recursive
marked outside the pause.
Changed ergonomics to have non sticky collectors always clear the
soft references.
Maps pauses ~10ms -> ~3ms on Nexus 4.
Bug: 13421927
Change-Id: I1370f7bb6934034869aa5afca0c377876267aa8e
Diffstat (limited to 'runtime/gc/heap.cc')
-rw-r--r-- | runtime/gc/heap.cc | 30 |
1 files changed, 21 insertions, 9 deletions
diff --git a/runtime/gc/heap.cc b/runtime/gc/heap.cc index e8ee62f..a763e37 100644 --- a/runtime/gc/heap.cc +++ b/runtime/gc/heap.cc @@ -751,22 +751,31 @@ mirror::Object* Heap::PreserveSoftReferenceCallback(mirror::Object* obj, void* a return args->mark_callback_(obj, args->arg_); } -// Process reference class instances and schedule finalizations. -void Heap::ProcessReferences(TimingLogger& timings, bool clear_soft, - IsMarkedCallback* is_marked_callback, - MarkObjectCallback* mark_object_callback, - ProcessMarkStackCallback* process_mark_stack_callback, void* arg) { - // Unless we are in the zygote or required to clear soft references with white references, - // preserve some white referents. - if (!clear_soft && !Runtime::Current()->IsZygote()) { +void Heap::ProcessSoftReferences(TimingLogger& timings, bool clear_soft, + IsMarkedCallback* is_marked_callback, + MarkObjectCallback* mark_object_callback, + ProcessMarkStackCallback* process_mark_stack_callback, void* arg) { + // Unless required to clear soft references with white references, preserve some white referents. + if (!clear_soft) { + // Don't clear for sticky GC. SoftReferenceArgs soft_reference_args; soft_reference_args.is_marked_callback_ = is_marked_callback; soft_reference_args.mark_callback_ = mark_object_callback; soft_reference_args.arg_ = arg; + // References with a marked referent are removed from the list. soft_reference_queue_.PreserveSomeSoftReferences(&PreserveSoftReferenceCallback, &soft_reference_args); process_mark_stack_callback(arg); } +} + +// Process reference class instances and schedule finalizations. +void Heap::ProcessReferences(TimingLogger& timings, bool clear_soft, + IsMarkedCallback* is_marked_callback, + MarkObjectCallback* mark_object_callback, + ProcessMarkStackCallback* process_mark_stack_callback, void* arg) { + ProcessSoftReferences(timings, clear_soft, is_marked_callback, mark_object_callback, + process_mark_stack_callback, arg); timings.StartSplit("(Paused)ProcessReferences"); // Clear all remaining soft and weak references with white referents. soft_reference_queue_.ClearWhiteReferences(cleared_references_, is_marked_callback, arg); @@ -1813,7 +1822,10 @@ collector::GcType Heap::CollectGarbageInternal(collector::GcType gc_type, GcCaus << "Could not find garbage collector with concurrent=" << concurrent_gc_ << " and type=" << gc_type; ATRACE_BEGIN(StringPrintf("%s %s GC", PrettyCause(gc_cause), collector->GetName()).c_str()); - collector->Run(gc_cause, clear_soft_references); + if (!clear_soft_references) { + clear_soft_references = gc_type != collector::kGcTypeSticky; // TODO: GSS? + } + collector->Run(gc_cause, clear_soft_references || Runtime::Current()->IsZygote()); total_objects_freed_ever_ += collector->GetFreedObjects(); total_bytes_freed_ever_ += collector->GetFreedBytes(); RequestHeapTrim(); |