Mercurial > hg > graal-jvmci-8
annotate src/share/vm/gc_implementation/g1/g1MarkSweep.cpp @ 22923:c04f46b4abe4
8068036: assert(is_available(index)) failed in G1 cset
Summary: Some verification code iterated over the heap using the region mapping array. This is not allowed. Changed to use the regular iteration method with closure.
Reviewed-by: jwilhelm, brutisso
| author | tschatzl |
|---|---|
| date | Tue, 31 Mar 2015 11:36:37 +0200 |
| parents | 38d6febe66af |
| children | dd9cc155639c |
| rev | line source |
|---|---|
| 342 | 1 /* |
| 17764 | 2 * Copyright (c) 2001, 2014, Oracle and/or its affiliates. All rights reserved. |
| 342 | 3 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. |
| 4 * | |
| 5 * This code is free software; you can redistribute it and/or modify it | |
| 6 * under the terms of the GNU General Public License version 2 only, as | |
| 7 * published by the Free Software Foundation. | |
| 8 * | |
| 9 * This code is distributed in the hope that it will be useful, but WITHOUT | |
| 10 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or | |
| 11 * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License | |
| 12 * version 2 for more details (a copy is included in the LICENSE file that | |
| 13 * accompanied this code). | |
| 14 * | |
| 15 * You should have received a copy of the GNU General Public License version | |
| 16 * 2 along with this work; if not, write to the Free Software Foundation, | |
| 17 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA. | |
| 18 * | |
|
1552
c18cbe5936b8
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
1387
diff
changeset
|
19 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA |
|
c18cbe5936b8
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
1387
diff
changeset
|
20 * or visit www.oracle.com if you need additional information or have any |
|
c18cbe5936b8
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
1387
diff
changeset
|
21 * questions. |
| 342 | 22 * |
| 23 */ | |
| 24 | |
| 1972 | 25 #include "precompiled.hpp" |
| 26 #include "classfile/javaClasses.hpp" | |
| 27 #include "classfile/symbolTable.hpp" | |
| 28 #include "classfile/systemDictionary.hpp" | |
| 29 #include "classfile/vmSymbols.hpp" | |
| 30 #include "code/codeCache.hpp" | |
| 31 #include "code/icBuffer.hpp" | |
|
6007
5c86f8211d1e
7160728: Introduce an extra logging level for G1 logging
brutisso
parents:
4872
diff
changeset
|
32 #include "gc_implementation/g1/g1Log.hpp" |
| 1972 | 33 #include "gc_implementation/g1/g1MarkSweep.hpp" |
|
22909
38d6febe66af
8075210: Refactor strong root processing in order to allow G1 to evolve separately from GenCollectedHeap
mgerdin
parents:
20417
diff
changeset
|
34 #include "gc_implementation/g1/g1RootProcessor.hpp" |
| 17764 | 35 #include "gc_implementation/g1/g1StringDedup.hpp" |
| 10405 | 36 #include "gc_implementation/shared/gcHeapSummary.hpp" |
| 37 #include "gc_implementation/shared/gcTimer.hpp" | |
| 38 #include "gc_implementation/shared/gcTrace.hpp" | |
| 39 #include "gc_implementation/shared/gcTraceTime.hpp" | |
| 1972 | 40 #include "memory/gcLocker.hpp" |
| 41 #include "memory/genCollectedHeap.hpp" | |
| 42 #include "memory/modRefBarrierSet.hpp" | |
| 43 #include "memory/referencePolicy.hpp" | |
| 44 #include "memory/space.hpp" | |
| 45 #include "oops/instanceRefKlass.hpp" | |
| 46 #include "oops/oop.inline.hpp" | |
| 47 #include "prims/jvmtiExport.hpp" | |
| 48 #include "runtime/biasedLocking.hpp" | |
| 49 #include "runtime/fprofiler.hpp" | |
| 50 #include "runtime/synchronizer.hpp" | |
| 51 #include "runtime/thread.hpp" | |
| 52 #include "runtime/vmThread.hpp" | |
| 53 #include "utilities/copy.hpp" | |
| 54 #include "utilities/events.hpp" | |
| 342 | 55 |
| 56 class HeapRegion; | |
| 57 | |
| 58 void G1MarkSweep::invoke_at_safepoint(ReferenceProcessor* rp, | |
| 59 bool clear_all_softrefs) { | |
| 60 assert(SafepointSynchronize::is_at_safepoint(), "must be at a safepoint"); | |
| 61 | |
|
1387
0bfd3fb24150
6858496: Clear all SoftReferences before an out-of-memory due to GC overhead limit.
jmasa
parents:
1311
diff
changeset
|
62 SharedHeap* sh = SharedHeap::heap(); |
|
0bfd3fb24150
6858496: Clear all SoftReferences before an out-of-memory due to GC overhead limit.
jmasa
parents:
1311
diff
changeset
|
63 #ifdef ASSERT |
|
0bfd3fb24150
6858496: Clear all SoftReferences before an out-of-memory due to GC overhead limit.
jmasa
parents:
1311
diff
changeset
|
64 if (sh->collector_policy()->should_clear_all_soft_refs()) { |
|
0bfd3fb24150
6858496: Clear all SoftReferences before an out-of-memory due to GC overhead limit.
jmasa
parents:
1311
diff
changeset
|
65 assert(clear_all_softrefs, "Policy should have been checked earler"); |
|
0bfd3fb24150
6858496: Clear all SoftReferences before an out-of-memory due to GC overhead limit.
jmasa
parents:
1311
diff
changeset
|
66 } |
|
0bfd3fb24150
6858496: Clear all SoftReferences before an out-of-memory due to GC overhead limit.
jmasa
parents:
1311
diff
changeset
|
67 #endif |
| 342 | 68 // hook up weak ref data so it can be used during Mark-Sweep |
| 69 assert(GenMarkSweep::ref_processor() == NULL, "no stomping"); | |
|
453
c96030fff130
6684579: SoftReference processing can be made more efficient
ysr
parents:
358
diff
changeset
|
70 assert(rp != NULL, "should be non-NULL"); |
|
3979
4dfb2df418f2
6484982: G1: process references during evacuation pauses
johnc
parents:
3772
diff
changeset
|
71 assert(rp == G1CollectedHeap::heap()->ref_processor_stw(), "Precondition"); |
|
4dfb2df418f2
6484982: G1: process references during evacuation pauses
johnc
parents:
3772
diff
changeset
|
72 |
| 342 | 73 GenMarkSweep::_ref_processor = rp; |
|
457
27a80744a83b
6778647: snap(), snap_policy() should be renamed setup(), setup_policy()
ysr
parents:
453
diff
changeset
|
74 rp->setup_policy(clear_all_softrefs); |
| 342 | 75 |
|
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6254
diff
changeset
|
76 // When collecting the permanent generation Method*s may be moving, |
| 342 | 77 // so we either have to flush all bcp data or convert it into bci. |
| 78 CodeCache::gc_prologue(); | |
| 79 Threads::gc_prologue(); | |
| 80 | |
| 81 bool marked_for_unloading = false; | |
| 82 | |
| 83 allocate_stacks(); | |
| 84 | |
|
358
8651a65ac4b4
6735416: G1: runThese javasoft.sqe.tests.lang.thrd011.thrd01101.thrd01101 fails
iveresov
parents:
342
diff
changeset
|
85 // We should save the marks of the currently locked biased monitors. |
|
8651a65ac4b4
6735416: G1: runThese javasoft.sqe.tests.lang.thrd011.thrd01101.thrd01101 fails
iveresov
parents:
342
diff
changeset
|
86 // The marking doesn't preserve the marks of biased objects. |
|
8651a65ac4b4
6735416: G1: runThese javasoft.sqe.tests.lang.thrd011.thrd01101.thrd01101 fails
iveresov
parents:
342
diff
changeset
|
87 BiasedLocking::preserve_marks(); |
|
8651a65ac4b4
6735416: G1: runThese javasoft.sqe.tests.lang.thrd011.thrd01101.thrd01101 fails
iveresov
parents:
342
diff
changeset
|
88 |
| 342 | 89 mark_sweep_phase1(marked_for_unloading, clear_all_softrefs); |
| 90 | |
| 91 mark_sweep_phase2(); | |
| 92 | |
| 93 // Don't add any more derived pointers during phase3 | |
| 94 COMPILER2_PRESENT(DerivedPointerTable::set_active(false)); | |
| 95 | |
| 96 mark_sweep_phase3(); | |
| 97 | |
| 98 mark_sweep_phase4(); | |
| 99 | |
| 100 GenMarkSweep::restore_marks(); | |
|
358
8651a65ac4b4
6735416: G1: runThese javasoft.sqe.tests.lang.thrd011.thrd01101.thrd01101 fails
iveresov
parents:
342
diff
changeset
|
101 BiasedLocking::restore_marks(); |
| 342 | 102 GenMarkSweep::deallocate_stacks(); |
| 103 | |
| 104 // "free at last gc" is calculated from these. | |
| 105 // CHF: cheating for now!!! | |
| 106 // Universe::set_heap_capacity_at_last_gc(Universe::heap()->capacity()); | |
| 107 // Universe::set_heap_used_at_last_gc(Universe::heap()->used()); | |
| 108 | |
| 109 Threads::gc_epilogue(); | |
| 110 CodeCache::gc_epilogue(); | |
|
2147
9afee0b9fc1d
7012505: BreakpointWithFullGC.sh fails with Internal Error (src/share/vm/oops/methodOop.cpp:220)
kamg
parents:
1972
diff
changeset
|
111 JvmtiExport::gc_epilogue(); |
| 342 | 112 |
| 113 // refs processing: clean slate | |
| 114 GenMarkSweep::_ref_processor = NULL; | |
| 115 } | |
| 116 | |
| 117 | |
| 118 void G1MarkSweep::allocate_stacks() { | |
| 119 GenMarkSweep::_preserved_count_max = 0; | |
| 120 GenMarkSweep::_preserved_marks = NULL; | |
| 121 GenMarkSweep::_preserved_count = 0; | |
| 122 } | |
| 123 | |
| 124 void G1MarkSweep::mark_sweep_phase1(bool& marked_for_unloading, | |
| 125 bool clear_all_softrefs) { | |
| 126 // Recursively traverse all live objects and mark them | |
|
20190
0982ec23da03
8043607: Add a GC id as a log decoration similar to PrintGCTimeStamps
brutisso
parents:
17764
diff
changeset
|
127 GCTraceTime tm("phase 1", G1Log::fine() && Verbose, true, gc_timer(), gc_tracer()->gc_id()); |
| 342 | 128 GenMarkSweep::trace(" 1"); |
| 129 | |
|
22909
38d6febe66af
8075210: Refactor strong root processing in order to allow G1 to evolve separately from GenCollectedHeap
mgerdin
parents:
20417
diff
changeset
|
130 G1CollectedHeap* g1h = G1CollectedHeap::heap(); |
| 342 | 131 |
|
20278
2c6ef90f030a
8049421: G1 Class Unloading after completing a concurrent mark cycle
stefank
parents:
20258
diff
changeset
|
132 // Need cleared claim bits for the roots processing |
|
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6254
diff
changeset
|
133 ClassLoaderDataGraph::clear_claimed_marks(); |
|
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6254
diff
changeset
|
134 |
|
20278
2c6ef90f030a
8049421: G1 Class Unloading after completing a concurrent mark cycle
stefank
parents:
20258
diff
changeset
|
135 MarkingCodeBlobClosure follow_code_closure(&GenMarkSweep::follow_root_closure, !CodeBlobToOopClosure::FixRelocations); |
|
22909
38d6febe66af
8075210: Refactor strong root processing in order to allow G1 to evolve separately from GenCollectedHeap
mgerdin
parents:
20417
diff
changeset
|
136 { |
|
38d6febe66af
8075210: Refactor strong root processing in order to allow G1 to evolve separately from GenCollectedHeap
mgerdin
parents:
20417
diff
changeset
|
137 G1RootProcessor root_processor(g1h); |
|
38d6febe66af
8075210: Refactor strong root processing in order to allow G1 to evolve separately from GenCollectedHeap
mgerdin
parents:
20417
diff
changeset
|
138 root_processor.process_strong_roots(&GenMarkSweep::follow_root_closure, |
|
38d6febe66af
8075210: Refactor strong root processing in order to allow G1 to evolve separately from GenCollectedHeap
mgerdin
parents:
20417
diff
changeset
|
139 &GenMarkSweep::follow_cld_closure, |
|
38d6febe66af
8075210: Refactor strong root processing in order to allow G1 to evolve separately from GenCollectedHeap
mgerdin
parents:
20417
diff
changeset
|
140 &follow_code_closure); |
|
38d6febe66af
8075210: Refactor strong root processing in order to allow G1 to evolve separately from GenCollectedHeap
mgerdin
parents:
20417
diff
changeset
|
141 } |
| 342 | 142 |
| 143 // Process reference objects found during marking | |
|
453
c96030fff130
6684579: SoftReference processing can be made more efficient
ysr
parents:
358
diff
changeset
|
144 ReferenceProcessor* rp = GenMarkSweep::ref_processor(); |
|
22909
38d6febe66af
8075210: Refactor strong root processing in order to allow G1 to evolve separately from GenCollectedHeap
mgerdin
parents:
20417
diff
changeset
|
145 assert(rp == g1h->ref_processor_stw(), "Sanity"); |
|
3979
4dfb2df418f2
6484982: G1: process references during evacuation pauses
johnc
parents:
3772
diff
changeset
|
146 |
|
457
27a80744a83b
6778647: snap(), snap_policy() should be renamed setup(), setup_policy()
ysr
parents:
453
diff
changeset
|
147 rp->setup_policy(clear_all_softrefs); |
| 10405 | 148 const ReferenceProcessorStats& stats = |
| 149 rp->process_discovered_references(&GenMarkSweep::is_alive, | |
| 150 &GenMarkSweep::keep_alive, | |
| 151 &GenMarkSweep::follow_stack_closure, | |
| 152 NULL, | |
|
20190
0982ec23da03
8043607: Add a GC id as a log decoration similar to PrintGCTimeStamps
brutisso
parents:
17764
diff
changeset
|
153 gc_timer(), |
|
0982ec23da03
8043607: Add a GC id as a log decoration similar to PrintGCTimeStamps
brutisso
parents:
17764
diff
changeset
|
154 gc_tracer()->gc_id()); |
| 10405 | 155 gc_tracer()->report_gc_reference_stats(stats); |
| 342 | 156 |
|
10188
2f50bc369470
8013160: NPG: Remove unnecessary mark stack draining after CodeCache::do_unloading
stefank
parents:
10186
diff
changeset
|
157 |
|
2f50bc369470
8013160: NPG: Remove unnecessary mark stack draining after CodeCache::do_unloading
stefank
parents:
10186
diff
changeset
|
158 // This is the point where the entire marking should have completed. |
|
2f50bc369470
8013160: NPG: Remove unnecessary mark stack draining after CodeCache::do_unloading
stefank
parents:
10186
diff
changeset
|
159 assert(GenMarkSweep::_marking_stack.is_empty(), "Marking should have completed"); |
|
2f50bc369470
8013160: NPG: Remove unnecessary mark stack draining after CodeCache::do_unloading
stefank
parents:
10186
diff
changeset
|
160 |
|
2f50bc369470
8013160: NPG: Remove unnecessary mark stack draining after CodeCache::do_unloading
stefank
parents:
10186
diff
changeset
|
161 // Unload classes and purge the SystemDictionary. |
| 342 | 162 bool purged_class = SystemDictionary::do_unloading(&GenMarkSweep::is_alive); |
| 163 | |
|
10188
2f50bc369470
8013160: NPG: Remove unnecessary mark stack draining after CodeCache::do_unloading
stefank
parents:
10186
diff
changeset
|
164 // Unload nmethods. |
|
6787
8966c2d65d96
7200470: KeepAliveClosure not needed in CodeCache::do_unloading
brutisso
parents:
6725
diff
changeset
|
165 CodeCache::do_unloading(&GenMarkSweep::is_alive, purged_class); |
| 342 | 166 |
|
10188
2f50bc369470
8013160: NPG: Remove unnecessary mark stack draining after CodeCache::do_unloading
stefank
parents:
10186
diff
changeset
|
167 // Prune dead klasses from subklass/sibling/implementor lists. |
|
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6254
diff
changeset
|
168 Klass::clean_weak_klass_links(&GenMarkSweep::is_alive); |
| 342 | 169 |
|
17634
5a32d2a3cc1e
8027476: Improve performance of Stringtable unlink
tschatzl
parents:
12343
diff
changeset
|
170 // Delete entries for dead interned string and clean up unreferenced symbols in symbol table. |
|
5a32d2a3cc1e
8027476: Improve performance of Stringtable unlink
tschatzl
parents:
12343
diff
changeset
|
171 G1CollectedHeap::heap()->unlink_string_and_symbol_table(&GenMarkSweep::is_alive); |
| 342 | 172 |
|
3772
6747fd0512e0
7004681: G1: Extend marking verification to Full GCs
johnc
parents:
2361
diff
changeset
|
173 if (VerifyDuringGC) { |
|
6747fd0512e0
7004681: G1: Extend marking verification to Full GCs
johnc
parents:
2361
diff
changeset
|
174 HandleMark hm; // handle scope |
|
6747fd0512e0
7004681: G1: Extend marking verification to Full GCs
johnc
parents:
2361
diff
changeset
|
175 COMPILER2_PRESENT(DerivedPointerTableDeactivate dpt_deact); |
|
6747fd0512e0
7004681: G1: Extend marking verification to Full GCs
johnc
parents:
2361
diff
changeset
|
176 Universe::heap()->prepare_for_verify(); |
|
6747fd0512e0
7004681: G1: Extend marking verification to Full GCs
johnc
parents:
2361
diff
changeset
|
177 // Note: we can verify only the heap here. When an object is |
|
6747fd0512e0
7004681: G1: Extend marking verification to Full GCs
johnc
parents:
2361
diff
changeset
|
178 // marked, the previous value of the mark word (including |
|
6747fd0512e0
7004681: G1: Extend marking verification to Full GCs
johnc
parents:
2361
diff
changeset
|
179 // identity hash values, ages, etc) is preserved, and the mark |
|
6747fd0512e0
7004681: G1: Extend marking verification to Full GCs
johnc
parents:
2361
diff
changeset
|
180 // word is set to markOop::marked_value - effectively removing |
|
6747fd0512e0
7004681: G1: Extend marking verification to Full GCs
johnc
parents:
2361
diff
changeset
|
181 // any hash values from the mark word. These hash values are |
|
6747fd0512e0
7004681: G1: Extend marking verification to Full GCs
johnc
parents:
2361
diff
changeset
|
182 // used when verifying the dictionaries and so removing them |
|
6747fd0512e0
7004681: G1: Extend marking verification to Full GCs
johnc
parents:
2361
diff
changeset
|
183 // from the mark word can make verification of the dictionaries |
|
6747fd0512e0
7004681: G1: Extend marking verification to Full GCs
johnc
parents:
2361
diff
changeset
|
184 // fail. At the end of the GC, the orginal mark word values |
|
6747fd0512e0
7004681: G1: Extend marking verification to Full GCs
johnc
parents:
2361
diff
changeset
|
185 // (including hash values) are restored to the appropriate |
|
6747fd0512e0
7004681: G1: Extend marking verification to Full GCs
johnc
parents:
2361
diff
changeset
|
186 // objects. |
|
10186
b06ac540229e
8013132: Add a flag to turn off the output of the verbose verification code
stefank
parents:
10179
diff
changeset
|
187 if (!VerifySilently) { |
|
b06ac540229e
8013132: Add a flag to turn off the output of the verbose verification code
stefank
parents:
10179
diff
changeset
|
188 gclog_or_tty->print(" VerifyDuringGC:(full)[Verifying "); |
|
b06ac540229e
8013132: Add a flag to turn off the output of the verbose verification code
stefank
parents:
10179
diff
changeset
|
189 } |
|
b06ac540229e
8013132: Add a flag to turn off the output of the verbose verification code
stefank
parents:
10179
diff
changeset
|
190 Universe::heap()->verify(VerifySilently, VerifyOption_G1UseMarkWord); |
|
b06ac540229e
8013132: Add a flag to turn off the output of the verbose verification code
stefank
parents:
10179
diff
changeset
|
191 if (!VerifySilently) { |
|
b06ac540229e
8013132: Add a flag to turn off the output of the verbose verification code
stefank
parents:
10179
diff
changeset
|
192 gclog_or_tty->print_cr("]"); |
|
b06ac540229e
8013132: Add a flag to turn off the output of the verbose verification code
stefank
parents:
10179
diff
changeset
|
193 } |
|
3772
6747fd0512e0
7004681: G1: Extend marking verification to Full GCs
johnc
parents:
2361
diff
changeset
|
194 } |
| 10405 | 195 |
| 196 gc_tracer()->report_object_count_after_gc(&GenMarkSweep::is_alive); | |
| 342 | 197 } |
| 198 | |
| 677 | 199 |
| 342 | 200 void G1MarkSweep::mark_sweep_phase2() { |
| 201 // Now all live objects are marked, compute the new object addresses. | |
| 202 | |
| 203 // It is not required that we traverse spaces in the same order in | |
| 204 // phase2, phase3 and phase4, but the ValidateMarkSweep live oops | |
| 205 // tracking expects us to do so. See comment under phase4. | |
| 206 | |
|
20190
0982ec23da03
8043607: Add a GC id as a log decoration similar to PrintGCTimeStamps
brutisso
parents:
17764
diff
changeset
|
207 GCTraceTime tm("phase 2", G1Log::fine() && Verbose, true, gc_timer(), gc_tracer()->gc_id()); |
| 342 | 208 GenMarkSweep::trace("2"); |
| 209 | |
| 20417 | 210 prepare_compaction(); |
| 342 | 211 } |
| 212 | |
| 213 class G1AdjustPointersClosure: public HeapRegionClosure { | |
| 214 public: | |
| 215 bool doHeapRegion(HeapRegion* r) { | |
| 216 if (r->isHumongous()) { | |
| 217 if (r->startsHumongous()) { | |
| 218 // We must adjust the pointers on the single H object. | |
| 219 oop obj = oop(r->bottom()); | |
| 220 // point all the oops to the new location | |
| 221 obj->adjust_pointers(); | |
| 222 } | |
| 223 } else { | |
| 224 // This really ought to be "as_CompactibleSpace"... | |
| 225 r->adjust_pointers(); | |
| 226 } | |
| 227 return false; | |
| 228 } | |
| 229 }; | |
| 230 | |
|
22909
38d6febe66af
8075210: Refactor strong root processing in order to allow G1 to evolve separately from GenCollectedHeap
mgerdin
parents:
20417
diff
changeset
|
231 class G1AlwaysTrueClosure: public BoolObjectClosure { |
|
38d6febe66af
8075210: Refactor strong root processing in order to allow G1 to evolve separately from GenCollectedHeap
mgerdin
parents:
20417
diff
changeset
|
232 public: |
|
38d6febe66af
8075210: Refactor strong root processing in order to allow G1 to evolve separately from GenCollectedHeap
mgerdin
parents:
20417
diff
changeset
|
233 bool do_object_b(oop p) { return true; } |
|
38d6febe66af
8075210: Refactor strong root processing in order to allow G1 to evolve separately from GenCollectedHeap
mgerdin
parents:
20417
diff
changeset
|
234 }; |
|
38d6febe66af
8075210: Refactor strong root processing in order to allow G1 to evolve separately from GenCollectedHeap
mgerdin
parents:
20417
diff
changeset
|
235 static G1AlwaysTrueClosure always_true; |
|
38d6febe66af
8075210: Refactor strong root processing in order to allow G1 to evolve separately from GenCollectedHeap
mgerdin
parents:
20417
diff
changeset
|
236 |
| 342 | 237 void G1MarkSweep::mark_sweep_phase3() { |
| 238 G1CollectedHeap* g1h = G1CollectedHeap::heap(); | |
| 239 | |
| 240 // Adjust the pointers to reflect the new locations | |
|
20190
0982ec23da03
8043607: Add a GC id as a log decoration similar to PrintGCTimeStamps
brutisso
parents:
17764
diff
changeset
|
241 GCTraceTime tm("phase 3", G1Log::fine() && Verbose, true, gc_timer(), gc_tracer()->gc_id()); |
| 342 | 242 GenMarkSweep::trace("3"); |
| 243 | |
|
20278
2c6ef90f030a
8049421: G1 Class Unloading after completing a concurrent mark cycle
stefank
parents:
20258
diff
changeset
|
244 // Need cleared claim bits for the roots processing |
|
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6254
diff
changeset
|
245 ClassLoaderDataGraph::clear_claimed_marks(); |
|
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6254
diff
changeset
|
246 |
|
20278
2c6ef90f030a
8049421: G1 Class Unloading after completing a concurrent mark cycle
stefank
parents:
20258
diff
changeset
|
247 CodeBlobToOopClosure adjust_code_closure(&GenMarkSweep::adjust_pointer_closure, CodeBlobToOopClosure::FixRelocations); |
|
22909
38d6febe66af
8075210: Refactor strong root processing in order to allow G1 to evolve separately from GenCollectedHeap
mgerdin
parents:
20417
diff
changeset
|
248 { |
|
38d6febe66af
8075210: Refactor strong root processing in order to allow G1 to evolve separately from GenCollectedHeap
mgerdin
parents:
20417
diff
changeset
|
249 G1RootProcessor root_processor(g1h); |
|
38d6febe66af
8075210: Refactor strong root processing in order to allow G1 to evolve separately from GenCollectedHeap
mgerdin
parents:
20417
diff
changeset
|
250 root_processor.process_all_roots(&GenMarkSweep::adjust_pointer_closure, |
|
38d6febe66af
8075210: Refactor strong root processing in order to allow G1 to evolve separately from GenCollectedHeap
mgerdin
parents:
20417
diff
changeset
|
251 &GenMarkSweep::adjust_cld_closure, |
|
38d6febe66af
8075210: Refactor strong root processing in order to allow G1 to evolve separately from GenCollectedHeap
mgerdin
parents:
20417
diff
changeset
|
252 &adjust_code_closure); |
|
38d6febe66af
8075210: Refactor strong root processing in order to allow G1 to evolve separately from GenCollectedHeap
mgerdin
parents:
20417
diff
changeset
|
253 } |
| 342 | 254 |
|
3979
4dfb2df418f2
6484982: G1: process references during evacuation pauses
johnc
parents:
3772
diff
changeset
|
255 assert(GenMarkSweep::ref_processor() == g1h->ref_processor_stw(), "Sanity"); |
|
10179
a08c80e9e1e5
8012687: Remove unused is_root checks and closures
stefank
parents:
7448
diff
changeset
|
256 g1h->ref_processor_stw()->weak_oops_do(&GenMarkSweep::adjust_pointer_closure); |
| 342 | 257 |
| 258 // Now adjust pointers in remaining weak roots. (All of which should | |
| 259 // have been cleared if they pointed to non-surviving objects.) | |
|
22909
38d6febe66af
8075210: Refactor strong root processing in order to allow G1 to evolve separately from GenCollectedHeap
mgerdin
parents:
20417
diff
changeset
|
260 JNIHandles::weak_oops_do(&always_true, &GenMarkSweep::adjust_pointer_closure); |
| 342 | 261 |
| 17764 | 262 if (G1StringDedup::is_enabled()) { |
| 263 G1StringDedup::oops_do(&GenMarkSweep::adjust_pointer_closure); | |
| 264 } | |
| 265 | |
| 342 | 266 GenMarkSweep::adjust_marks(); |
| 267 | |
| 268 G1AdjustPointersClosure blk; | |
| 269 g1h->heap_region_iterate(&blk); | |
| 270 } | |
| 271 | |
| 272 class G1SpaceCompactClosure: public HeapRegionClosure { | |
| 273 public: | |
| 274 G1SpaceCompactClosure() {} | |
| 275 | |
| 276 bool doHeapRegion(HeapRegion* hr) { | |
| 277 if (hr->isHumongous()) { | |
| 278 if (hr->startsHumongous()) { | |
| 279 oop obj = oop(hr->bottom()); | |
| 280 if (obj->is_gc_marked()) { | |
| 281 obj->init_mark(); | |
| 282 } else { | |
| 283 assert(hr->is_empty(), "Should have been cleared in phase 2."); | |
| 284 } | |
| 285 hr->reset_during_compaction(); | |
| 286 } | |
| 287 } else { | |
| 288 hr->compact(); | |
| 289 } | |
| 290 return false; | |
| 291 } | |
| 292 }; | |
| 293 | |
| 294 void G1MarkSweep::mark_sweep_phase4() { | |
| 295 // All pointers are now adjusted, move objects accordingly | |
| 296 | |
| 297 // The ValidateMarkSweep live oops tracking expects us to traverse spaces | |
| 298 // in the same order in phase2, phase3 and phase4. We don't quite do that | |
|
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6254
diff
changeset
|
299 // here (code and comment not fixed for perm removal), so we tell the validate code |
| 342 | 300 // to use a higher index (saved from phase2) when verifying perm_gen. |
| 301 G1CollectedHeap* g1h = G1CollectedHeap::heap(); | |
| 302 | |
|
20190
0982ec23da03
8043607: Add a GC id as a log decoration similar to PrintGCTimeStamps
brutisso
parents:
17764
diff
changeset
|
303 GCTraceTime tm("phase 4", G1Log::fine() && Verbose, true, gc_timer(), gc_tracer()->gc_id()); |
| 342 | 304 GenMarkSweep::trace("4"); |
| 305 | |
| 306 G1SpaceCompactClosure blk; | |
| 307 g1h->heap_region_iterate(&blk); | |
| 308 | |
| 309 } | |
| 20417 | 310 |
| 311 void G1MarkSweep::prepare_compaction_work(G1PrepareCompactClosure* blk) { | |
| 312 G1CollectedHeap* g1h = G1CollectedHeap::heap(); | |
| 313 g1h->heap_region_iterate(blk); | |
| 314 blk->update_sets(); | |
| 315 } | |
| 316 | |
| 317 void G1PrepareCompactClosure::free_humongous_region(HeapRegion* hr) { | |
| 318 HeapWord* end = hr->end(); | |
| 319 FreeRegionList dummy_free_list("Dummy Free List for G1MarkSweep"); | |
| 320 | |
| 321 assert(hr->startsHumongous(), | |
| 322 "Only the start of a humongous region should be freed."); | |
| 323 | |
| 324 hr->set_containing_set(NULL); | |
| 325 _humongous_regions_removed.increment(1u, hr->capacity()); | |
| 326 | |
| 327 _g1h->free_humongous_region(hr, &dummy_free_list, false /* par */); | |
| 328 prepare_for_compaction(hr, end); | |
| 329 dummy_free_list.remove_all(); | |
| 330 } | |
| 331 | |
| 332 void G1PrepareCompactClosure::prepare_for_compaction(HeapRegion* hr, HeapWord* end) { | |
| 333 // If this is the first live region that we came across which we can compact, | |
| 334 // initialize the CompactPoint. | |
| 335 if (!is_cp_initialized()) { | |
| 336 _cp.space = hr; | |
| 337 _cp.threshold = hr->initialize_threshold(); | |
| 338 } | |
| 339 prepare_for_compaction_work(&_cp, hr, end); | |
| 340 } | |
| 341 | |
| 342 void G1PrepareCompactClosure::prepare_for_compaction_work(CompactPoint* cp, | |
| 343 HeapRegion* hr, | |
| 344 HeapWord* end) { | |
| 345 hr->prepare_for_compaction(cp); | |
| 346 // Also clear the part of the card table that will be unused after | |
| 347 // compaction. | |
| 348 _mrbs->clear(MemRegion(hr->compaction_top(), end)); | |
| 349 } | |
| 350 | |
| 351 void G1PrepareCompactClosure::update_sets() { | |
| 352 // We'll recalculate total used bytes and recreate the free list | |
| 353 // at the end of the GC, so no point in updating those values here. | |
| 354 HeapRegionSetCount empty_set; | |
| 355 _g1h->remove_from_old_sets(empty_set, _humongous_regions_removed); | |
| 356 } | |
| 357 | |
| 358 bool G1PrepareCompactClosure::doHeapRegion(HeapRegion* hr) { | |
| 359 if (hr->isHumongous()) { | |
| 360 if (hr->startsHumongous()) { | |
| 361 oop obj = oop(hr->bottom()); | |
| 362 if (obj->is_gc_marked()) { | |
| 363 obj->forward_to(obj); | |
| 364 } else { | |
| 365 free_humongous_region(hr); | |
| 366 } | |
| 367 } else { | |
| 368 assert(hr->continuesHumongous(), "Invalid humongous."); | |
| 369 } | |
| 370 } else { | |
| 371 prepare_for_compaction(hr, hr->end()); | |
| 372 } | |
| 373 return false; | |
| 374 } |