Mercurial > hg > graal-jvmci-8
annotate src/share/vm/gc_implementation/g1/g1MarkSweep.cpp @ 1084:5f932a151fd4
6895788: G1: SATB and update buffer allocation code allocates too much space
Summary: The type in the NEW_C_HEAP_ARRRY and FREE_C_HEAP_ARRAY calls in the buffer allocation code was changed from void* to char as the size argument had already been mulitipled by the byte size of an object pointer.
Reviewed-by: ysr, tonyp
author | johnc |
---|---|
date | Fri, 06 Nov 2009 11:10:05 -0800 |
parents | 54b3b351d6f9 |
children | 2a1472c30599 |
rev | line source |
---|---|
342 | 1 /* |
844 | 2 * Copyright 2001-2009 Sun Microsystems, Inc. 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 * | |
19 * Please contact Sun Microsystems, Inc., 4150 Network Circle, Santa Clara, | |
20 * CA 95054 USA or visit www.sun.com if you need additional information or | |
21 * have any questions. | |
22 * | |
23 */ | |
24 | |
25 #include "incls/_precompiled.incl" | |
26 #include "incls/_g1MarkSweep.cpp.incl" | |
27 | |
28 class HeapRegion; | |
29 | |
30 void G1MarkSweep::invoke_at_safepoint(ReferenceProcessor* rp, | |
31 bool clear_all_softrefs) { | |
32 assert(SafepointSynchronize::is_at_safepoint(), "must be at a safepoint"); | |
33 | |
34 // hook up weak ref data so it can be used during Mark-Sweep | |
35 assert(GenMarkSweep::ref_processor() == NULL, "no stomping"); | |
453
c96030fff130
6684579: SoftReference processing can be made more efficient
ysr
parents:
358
diff
changeset
|
36 assert(rp != NULL, "should be non-NULL"); |
342 | 37 GenMarkSweep::_ref_processor = rp; |
457
27a80744a83b
6778647: snap(), snap_policy() should be renamed setup(), setup_policy()
ysr
parents:
453
diff
changeset
|
38 rp->setup_policy(clear_all_softrefs); |
342 | 39 |
40 // When collecting the permanent generation methodOops may be moving, | |
41 // so we either have to flush all bcp data or convert it into bci. | |
42 CodeCache::gc_prologue(); | |
43 Threads::gc_prologue(); | |
44 | |
45 // Increment the invocation count for the permanent generation, since it is | |
46 // implicitly collected whenever we do a full mark sweep collection. | |
47 SharedHeap* sh = SharedHeap::heap(); | |
48 sh->perm_gen()->stat_record()->invocations++; | |
49 | |
50 bool marked_for_unloading = false; | |
51 | |
52 allocate_stacks(); | |
53 | |
358
8651a65ac4b4
6735416: G1: runThese javasoft.sqe.tests.lang.thrd011.thrd01101.thrd01101 fails
iveresov
parents:
342
diff
changeset
|
54 // 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
|
55 // 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
|
56 BiasedLocking::preserve_marks(); |
8651a65ac4b4
6735416: G1: runThese javasoft.sqe.tests.lang.thrd011.thrd01101.thrd01101 fails
iveresov
parents:
342
diff
changeset
|
57 |
342 | 58 mark_sweep_phase1(marked_for_unloading, clear_all_softrefs); |
59 | |
751 | 60 if (VerifyDuringGC) { |
342 | 61 G1CollectedHeap* g1h = G1CollectedHeap::heap(); |
62 g1h->checkConcurrentMark(); | |
63 } | |
64 | |
65 mark_sweep_phase2(); | |
66 | |
67 // Don't add any more derived pointers during phase3 | |
68 COMPILER2_PRESENT(DerivedPointerTable::set_active(false)); | |
69 | |
70 mark_sweep_phase3(); | |
71 | |
72 mark_sweep_phase4(); | |
73 | |
74 GenMarkSweep::restore_marks(); | |
358
8651a65ac4b4
6735416: G1: runThese javasoft.sqe.tests.lang.thrd011.thrd01101.thrd01101 fails
iveresov
parents:
342
diff
changeset
|
75 BiasedLocking::restore_marks(); |
342 | 76 GenMarkSweep::deallocate_stacks(); |
77 | |
78 // We must invalidate the perm-gen rs, so that it gets rebuilt. | |
79 GenRemSet* rs = sh->rem_set(); | |
80 rs->invalidate(sh->perm_gen()->used_region(), true /*whole_heap*/); | |
81 | |
82 // "free at last gc" is calculated from these. | |
83 // CHF: cheating for now!!! | |
84 // Universe::set_heap_capacity_at_last_gc(Universe::heap()->capacity()); | |
85 // Universe::set_heap_used_at_last_gc(Universe::heap()->used()); | |
86 | |
87 Threads::gc_epilogue(); | |
88 CodeCache::gc_epilogue(); | |
89 | |
90 // refs processing: clean slate | |
91 GenMarkSweep::_ref_processor = NULL; | |
92 } | |
93 | |
94 | |
95 void G1MarkSweep::allocate_stacks() { | |
96 GenMarkSweep::_preserved_count_max = 0; | |
97 GenMarkSweep::_preserved_marks = NULL; | |
98 GenMarkSweep::_preserved_count = 0; | |
99 GenMarkSweep::_preserved_mark_stack = NULL; | |
100 GenMarkSweep::_preserved_oop_stack = NULL; | |
101 | |
102 GenMarkSweep::_marking_stack = | |
103 new (ResourceObj::C_HEAP) GrowableArray<oop>(4000, true); | |
104 | |
941 | 105 int size = SystemDictionary::number_of_classes() * 2; |
342 | 106 GenMarkSweep::_revisit_klass_stack = |
941 | 107 new (ResourceObj::C_HEAP) GrowableArray<Klass*>(size, true); |
108 // (#klass/k)^2 for k ~ 10 appears a better fit, but this will have to do | |
109 // for now until we have a chance to work out a more optimal setting. | |
110 GenMarkSweep::_revisit_mdo_stack = | |
111 new (ResourceObj::C_HEAP) GrowableArray<DataLayout*>(size*2, true); | |
112 | |
342 | 113 } |
114 | |
115 void G1MarkSweep::mark_sweep_phase1(bool& marked_for_unloading, | |
116 bool clear_all_softrefs) { | |
117 // Recursively traverse all live objects and mark them | |
118 EventMark m("1 mark object"); | |
119 TraceTime tm("phase 1", PrintGC && Verbose, true, gclog_or_tty); | |
120 GenMarkSweep::trace(" 1"); | |
121 | |
122 SharedHeap* sh = SharedHeap::heap(); | |
123 | |
989
148e5441d916
6863023: need non-perm oops in code cache for JSR 292
jrose
parents:
844
diff
changeset
|
124 sh->process_strong_roots(true, // activeate StrongRootsScope |
148e5441d916
6863023: need non-perm oops in code cache for JSR 292
jrose
parents:
844
diff
changeset
|
125 true, // Collecting permanent generation. |
342 | 126 SharedHeap::SO_SystemClasses, |
127 &GenMarkSweep::follow_root_closure, | |
989
148e5441d916
6863023: need non-perm oops in code cache for JSR 292
jrose
parents:
844
diff
changeset
|
128 &GenMarkSweep::follow_code_root_closure, |
342 | 129 &GenMarkSweep::follow_root_closure); |
130 | |
131 // Process reference objects found during marking | |
453
c96030fff130
6684579: SoftReference processing can be made more efficient
ysr
parents:
358
diff
changeset
|
132 ReferenceProcessor* rp = GenMarkSweep::ref_processor(); |
457
27a80744a83b
6778647: snap(), snap_policy() should be renamed setup(), setup_policy()
ysr
parents:
453
diff
changeset
|
133 rp->setup_policy(clear_all_softrefs); |
453
c96030fff130
6684579: SoftReference processing can be made more efficient
ysr
parents:
358
diff
changeset
|
134 rp->process_discovered_references(&GenMarkSweep::is_alive, |
c96030fff130
6684579: SoftReference processing can be made more efficient
ysr
parents:
358
diff
changeset
|
135 &GenMarkSweep::keep_alive, |
c96030fff130
6684579: SoftReference processing can be made more efficient
ysr
parents:
358
diff
changeset
|
136 &GenMarkSweep::follow_stack_closure, |
c96030fff130
6684579: SoftReference processing can be made more efficient
ysr
parents:
358
diff
changeset
|
137 NULL); |
342 | 138 |
139 // Follow system dictionary roots and unload classes | |
140 bool purged_class = SystemDictionary::do_unloading(&GenMarkSweep::is_alive); | |
141 assert(GenMarkSweep::_marking_stack->is_empty(), | |
142 "stack should be empty by now"); | |
143 | |
144 // Follow code cache roots (has to be done after system dictionary, | |
145 // assumes all live klasses are marked) | |
146 CodeCache::do_unloading(&GenMarkSweep::is_alive, | |
147 &GenMarkSweep::keep_alive, | |
148 purged_class); | |
941 | 149 GenMarkSweep::follow_stack(); |
342 | 150 |
151 // Update subklass/sibling/implementor links of live klasses | |
152 GenMarkSweep::follow_weak_klass_links(); | |
153 assert(GenMarkSweep::_marking_stack->is_empty(), | |
154 "stack should be empty by now"); | |
155 | |
941 | 156 // Visit memoized MDO's and clear any unmarked weak refs |
157 GenMarkSweep::follow_mdo_weak_refs(); | |
158 assert(GenMarkSweep::_marking_stack->is_empty(), "just drained"); | |
159 | |
160 | |
342 | 161 // Visit symbol and interned string tables and delete unmarked oops |
162 SymbolTable::unlink(&GenMarkSweep::is_alive); | |
163 StringTable::unlink(&GenMarkSweep::is_alive); | |
164 | |
165 assert(GenMarkSweep::_marking_stack->is_empty(), | |
166 "stack should be empty by now"); | |
167 } | |
168 | |
169 class G1PrepareCompactClosure: public HeapRegionClosure { | |
170 ModRefBarrierSet* _mrbs; | |
171 CompactPoint _cp; | |
172 | |
173 void free_humongous_region(HeapRegion* hr) { | |
174 HeapWord* bot = hr->bottom(); | |
175 HeapWord* end = hr->end(); | |
176 assert(hr->startsHumongous(), | |
177 "Only the start of a humongous region should be freed."); | |
178 G1CollectedHeap::heap()->free_region(hr); | |
179 hr->prepare_for_compaction(&_cp); | |
180 // Also clear the part of the card table that will be unused after | |
181 // compaction. | |
182 _mrbs->clear(MemRegion(hr->compaction_top(), hr->end())); | |
183 } | |
184 | |
185 public: | |
677 | 186 G1PrepareCompactClosure(CompactibleSpace* cs) : |
342 | 187 _cp(NULL, cs, cs->initialize_threshold()), |
677 | 188 _mrbs(G1CollectedHeap::heap()->mr_bs()) |
342 | 189 {} |
190 bool doHeapRegion(HeapRegion* hr) { | |
191 if (hr->isHumongous()) { | |
192 if (hr->startsHumongous()) { | |
193 oop obj = oop(hr->bottom()); | |
194 if (obj->is_gc_marked()) { | |
195 obj->forward_to(obj); | |
196 } else { | |
197 free_humongous_region(hr); | |
198 } | |
199 } else { | |
200 assert(hr->continuesHumongous(), "Invalid humongous."); | |
201 } | |
202 } else { | |
203 hr->prepare_for_compaction(&_cp); | |
204 // Also clear the part of the card table that will be unused after | |
205 // compaction. | |
206 _mrbs->clear(MemRegion(hr->compaction_top(), hr->end())); | |
207 } | |
208 return false; | |
209 } | |
210 }; | |
677 | 211 |
212 // Finds the first HeapRegion. | |
342 | 213 class FindFirstRegionClosure: public HeapRegionClosure { |
214 HeapRegion* _a_region; | |
215 public: | |
677 | 216 FindFirstRegionClosure() : _a_region(NULL) {} |
342 | 217 bool doHeapRegion(HeapRegion* r) { |
677 | 218 _a_region = r; |
219 return true; | |
342 | 220 } |
221 HeapRegion* result() { return _a_region; } | |
222 }; | |
223 | |
224 void G1MarkSweep::mark_sweep_phase2() { | |
225 // Now all live objects are marked, compute the new object addresses. | |
226 | |
227 // It is imperative that we traverse perm_gen LAST. If dead space is | |
228 // allowed a range of dead object may get overwritten by a dead int | |
229 // array. If perm_gen is not traversed last a klassOop may get | |
230 // overwritten. This is fine since it is dead, but if the class has dead | |
231 // instances we have to skip them, and in order to find their size we | |
232 // need the klassOop! | |
233 // | |
234 // It is not required that we traverse spaces in the same order in | |
235 // phase2, phase3 and phase4, but the ValidateMarkSweep live oops | |
236 // tracking expects us to do so. See comment under phase4. | |
237 | |
238 G1CollectedHeap* g1h = G1CollectedHeap::heap(); | |
239 Generation* pg = g1h->perm_gen(); | |
240 | |
241 EventMark m("2 compute new addresses"); | |
242 TraceTime tm("phase 2", PrintGC && Verbose, true, gclog_or_tty); | |
243 GenMarkSweep::trace("2"); | |
244 | |
677 | 245 FindFirstRegionClosure cl; |
342 | 246 g1h->heap_region_iterate(&cl); |
247 HeapRegion *r = cl.result(); | |
248 CompactibleSpace* sp = r; | |
249 if (r->isHumongous() && oop(r->bottom())->is_gc_marked()) { | |
250 sp = r->next_compaction_space(); | |
251 } | |
252 | |
677 | 253 G1PrepareCompactClosure blk(sp); |
342 | 254 g1h->heap_region_iterate(&blk); |
255 | |
256 CompactPoint perm_cp(pg, NULL, NULL); | |
257 pg->prepare_for_compaction(&perm_cp); | |
258 } | |
259 | |
260 class G1AdjustPointersClosure: public HeapRegionClosure { | |
261 public: | |
262 bool doHeapRegion(HeapRegion* r) { | |
263 if (r->isHumongous()) { | |
264 if (r->startsHumongous()) { | |
265 // We must adjust the pointers on the single H object. | |
266 oop obj = oop(r->bottom()); | |
267 debug_only(GenMarkSweep::track_interior_pointers(obj)); | |
268 // point all the oops to the new location | |
269 obj->adjust_pointers(); | |
270 debug_only(GenMarkSweep::check_interior_pointers()); | |
271 } | |
272 } else { | |
273 // This really ought to be "as_CompactibleSpace"... | |
274 r->adjust_pointers(); | |
275 } | |
276 return false; | |
277 } | |
278 }; | |
279 | |
280 void G1MarkSweep::mark_sweep_phase3() { | |
281 G1CollectedHeap* g1h = G1CollectedHeap::heap(); | |
282 Generation* pg = g1h->perm_gen(); | |
283 | |
284 // Adjust the pointers to reflect the new locations | |
285 EventMark m("3 adjust pointers"); | |
286 TraceTime tm("phase 3", PrintGC && Verbose, true, gclog_or_tty); | |
287 GenMarkSweep::trace("3"); | |
288 | |
289 SharedHeap* sh = SharedHeap::heap(); | |
290 | |
989
148e5441d916
6863023: need non-perm oops in code cache for JSR 292
jrose
parents:
844
diff
changeset
|
291 sh->process_strong_roots(true, // activate StrongRootsScope |
148e5441d916
6863023: need non-perm oops in code cache for JSR 292
jrose
parents:
844
diff
changeset
|
292 true, // Collecting permanent generation. |
342 | 293 SharedHeap::SO_AllClasses, |
294 &GenMarkSweep::adjust_root_pointer_closure, | |
989
148e5441d916
6863023: need non-perm oops in code cache for JSR 292
jrose
parents:
844
diff
changeset
|
295 NULL, // do not touch code cache here |
342 | 296 &GenMarkSweep::adjust_pointer_closure); |
297 | |
298 g1h->ref_processor()->weak_oops_do(&GenMarkSweep::adjust_root_pointer_closure); | |
299 | |
300 // Now adjust pointers in remaining weak roots. (All of which should | |
301 // have been cleared if they pointed to non-surviving objects.) | |
302 g1h->g1_process_weak_roots(&GenMarkSweep::adjust_root_pointer_closure, | |
303 &GenMarkSweep::adjust_pointer_closure); | |
304 | |
305 GenMarkSweep::adjust_marks(); | |
306 | |
307 G1AdjustPointersClosure blk; | |
308 g1h->heap_region_iterate(&blk); | |
309 pg->adjust_pointers(); | |
310 } | |
311 | |
312 class G1SpaceCompactClosure: public HeapRegionClosure { | |
313 public: | |
314 G1SpaceCompactClosure() {} | |
315 | |
316 bool doHeapRegion(HeapRegion* hr) { | |
317 if (hr->isHumongous()) { | |
318 if (hr->startsHumongous()) { | |
319 oop obj = oop(hr->bottom()); | |
320 if (obj->is_gc_marked()) { | |
321 obj->init_mark(); | |
322 } else { | |
323 assert(hr->is_empty(), "Should have been cleared in phase 2."); | |
324 } | |
325 hr->reset_during_compaction(); | |
326 } | |
327 } else { | |
328 hr->compact(); | |
329 } | |
330 return false; | |
331 } | |
332 }; | |
333 | |
334 void G1MarkSweep::mark_sweep_phase4() { | |
335 // All pointers are now adjusted, move objects accordingly | |
336 | |
337 // It is imperative that we traverse perm_gen first in phase4. All | |
338 // classes must be allocated earlier than their instances, and traversing | |
339 // perm_gen first makes sure that all klassOops have moved to their new | |
340 // location before any instance does a dispatch through it's klass! | |
341 | |
342 // The ValidateMarkSweep live oops tracking expects us to traverse spaces | |
343 // in the same order in phase2, phase3 and phase4. We don't quite do that | |
344 // here (perm_gen first rather than last), so we tell the validate code | |
345 // to use a higher index (saved from phase2) when verifying perm_gen. | |
346 G1CollectedHeap* g1h = G1CollectedHeap::heap(); | |
347 Generation* pg = g1h->perm_gen(); | |
348 | |
349 EventMark m("4 compact heap"); | |
350 TraceTime tm("phase 4", PrintGC && Verbose, true, gclog_or_tty); | |
351 GenMarkSweep::trace("4"); | |
352 | |
353 pg->compact(); | |
354 | |
355 G1SpaceCompactClosure blk; | |
356 g1h->heap_region_iterate(&blk); | |
357 | |
358 } | |
359 | |
360 // Local Variables: *** | |
361 // c-indentation-style: gnu *** | |
362 // End: *** |