truffle: src/share/vm/gc_implementation/g1/concurrentMarkThread.cpp annotate

annotate src/share/vm/gc_implementation/g1/concurrentMarkThread.cpp @ 12233:40136aa2cdb1

8010722: assert: failed: heap size is too big for compressed oops Summary: Use conservative assumptions of required alignment for the various garbage collector components into account when determining the maximum heap size that supports compressed oops. Using this conservative value avoids several circular dependencies in the calculation. Reviewed-by: stefank, dholmes

author	tschatzl
date	Wed, 11 Sep 2013 16:25:02 +0200
parents	f2110083203d
children	1772223a25a2

rev	line source
342 37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	1 /*
7450 d275c3dc73e6 8004816: G1: Kitchensink failures after marking stack changes johnc parents: 6120 diff changeset	2 * Copyright (c) 2001, 2013, Oracle and/or its affiliates. All rights reserved.
342 37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	3 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	4 *
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	5 * This code is free software; you can redistribute it and/or modify it
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	6 * under the terms of the GNU General Public License version 2 only, as
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	7 * published by the Free Software Foundation.
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	8 *
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	9 * This code is distributed in the hope that it will be useful, but WITHOUT
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	10 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	11 * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	12 * version 2 for more details (a copy is included in the LICENSE file that
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	13 * accompanied this code).
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	14 *
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	15 * You should have received a copy of the GNU General Public License version
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	16 * 2 along with this work; if not, write to the Free Software Foundation,
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	17 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	18 *
1552 c18cbe5936b8 6941466: Oracle rebranding changes for Hotspot repositories trims parents: 1019 diff changeset	19 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
c18cbe5936b8 6941466: Oracle rebranding changes for Hotspot repositories trims parents: 1019 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: 1019 diff changeset	21 * questions.
342 37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	22 *
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	23 */
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	24
1972 f95d63e2154a 6989984: Use standard include model for Hospot stefank parents: 1840 diff changeset	25 #include "precompiled.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot stefank parents: 1840 diff changeset	26 #include "gc_implementation/g1/concurrentMarkThread.inline.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot stefank parents: 1840 diff changeset	27 #include "gc_implementation/g1/g1CollectedHeap.inline.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot stefank parents: 1840 diff changeset	28 #include "gc_implementation/g1/g1CollectorPolicy.hpp"
6007 5c86f8211d1e 7160728: Introduce an extra logging level for G1 logging brutisso parents: 5963 diff changeset	29 #include "gc_implementation/g1/g1Log.hpp"
1972 f95d63e2154a 6989984: Use standard include model for Hospot stefank parents: 1840 diff changeset	30 #include "gc_implementation/g1/g1MMUTracker.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot stefank parents: 1840 diff changeset	31 #include "gc_implementation/g1/vm_operations_g1.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot stefank parents: 1840 diff changeset	32 #include "memory/resourceArea.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot stefank parents: 1840 diff changeset	33 #include "runtime/vmThread.hpp"
342 37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	34
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	35 // ======= Concurrent Mark Thread ========
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	36
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	37 // The CM thread is created when the G1 garbage collector is used
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	38
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	39 SurrogateLockerThread*
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	40 ConcurrentMarkThread::_slt = NULL;
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	41
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	42 ConcurrentMarkThread::ConcurrentMarkThread(ConcurrentMark* cm) :
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	43 ConcurrentGCThread(),
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	44 _cm(cm),
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	45 _started(false),
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	46 _in_progress(false),
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	47 _vtime_accum(0.0),
4836 d30fa85f9994 6484965: G1: piggy-back liveness accounting phase on marking johnc parents: 4096 diff changeset	48 _vtime_mark_accum(0.0) {
342 37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	49 create_and_start();
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	50 }
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	51
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	52 class CMCheckpointRootsFinalClosure: public VoidClosure {
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	53
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	54 ConcurrentMark* _cm;
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	55 public:
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	56
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	57 CMCheckpointRootsFinalClosure(ConcurrentMark* cm) :
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	58 _cm(cm) {}
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	59
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	60 void do_void(){
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	61 _cm->checkpointRootsFinal(false); // !clear_all_soft_refs
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	62 }
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	63 };
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	64
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	65 class CMCleanUp: public VoidClosure {
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	66 ConcurrentMark* _cm;
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	67 public:
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	68
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	69 CMCleanUp(ConcurrentMark* cm) :
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	70 _cm(cm) {}
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	71
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	72 void do_void(){
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	73 _cm->cleanup();
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	74 }
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	75 };
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	76
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	77
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	78
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	79 void ConcurrentMarkThread::run() {
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	80 initialize_in_thread();
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	81 _vtime_start = os::elapsedVTime();
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	82 wait_for_universe_init();
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	83
2152 0fa27f37d4d4 6977804: G1: remove the zero-filling thread tonyp parents: 2030 diff changeset	84 G1CollectedHeap* g1h = G1CollectedHeap::heap();
0fa27f37d4d4 6977804: G1: remove the zero-filling thread tonyp parents: 2030 diff changeset	85 G1CollectorPolicy* g1_policy = g1h->g1_policy();
342 37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	86 G1MMUTracker *mmu_tracker = g1_policy->mmu_tracker();
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	87 Thread *current_thread = Thread::current();
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	88
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	89 while (!_should_terminate) {
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	90 // wait until started is set.
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	91 sleepBeforeNextCycle();
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	92 {
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	93 ResourceMark rm;
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	94 HandleMark hm;
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	95 double cycle_start = os::elapsedVTime();
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	96
4837 eff609af17d7 7127706: G1: re-enable survivors during the initial-mark pause tonyp parents: 4836 diff changeset	97 // We have to ensure that we finish scanning the root regions
eff609af17d7 7127706: G1: re-enable survivors during the initial-mark pause tonyp parents: 4836 diff changeset	98 // before the next GC takes place. To ensure this we have to
eff609af17d7 7127706: G1: re-enable survivors during the initial-mark pause tonyp parents: 4836 diff changeset	99 // make sure that we do not join the STS until the root regions
eff609af17d7 7127706: G1: re-enable survivors during the initial-mark pause tonyp parents: 4836 diff changeset	100 // have been scanned. If we did then it's possible that a
eff609af17d7 7127706: G1: re-enable survivors during the initial-mark pause tonyp parents: 4836 diff changeset	101 // subsequent GC could block us from joining the STS and proceed
eff609af17d7 7127706: G1: re-enable survivors during the initial-mark pause tonyp parents: 4836 diff changeset	102 // without the root regions have been scanned which would be a
eff609af17d7 7127706: G1: re-enable survivors during the initial-mark pause tonyp parents: 4836 diff changeset	103 // correctness issue.
eff609af17d7 7127706: G1: re-enable survivors during the initial-mark pause tonyp parents: 4836 diff changeset	104
eff609af17d7 7127706: G1: re-enable survivors during the initial-mark pause tonyp parents: 4836 diff changeset	105 double scan_start = os::elapsedTime();
eff609af17d7 7127706: G1: re-enable survivors during the initial-mark pause tonyp parents: 4836 diff changeset	106 if (!cm()->has_aborted()) {
6007 5c86f8211d1e 7160728: Introduce an extra logging level for G1 logging brutisso parents: 5963 diff changeset	107 if (G1Log::fine()) {
4837 eff609af17d7 7127706: G1: re-enable survivors during the initial-mark pause tonyp parents: 4836 diff changeset	108 gclog_or_tty->date_stamp(PrintGCDateStamps);
eff609af17d7 7127706: G1: re-enable survivors during the initial-mark pause tonyp parents: 4836 diff changeset	109 gclog_or_tty->stamp(PrintGCTimeStamps);
eff609af17d7 7127706: G1: re-enable survivors during the initial-mark pause tonyp parents: 4836 diff changeset	110 gclog_or_tty->print_cr("[GC concurrent-root-region-scan-start]");
eff609af17d7 7127706: G1: re-enable survivors during the initial-mark pause tonyp parents: 4836 diff changeset	111 }
eff609af17d7 7127706: G1: re-enable survivors during the initial-mark pause tonyp parents: 4836 diff changeset	112
eff609af17d7 7127706: G1: re-enable survivors during the initial-mark pause tonyp parents: 4836 diff changeset	113 _cm->scanRootRegions();
eff609af17d7 7127706: G1: re-enable survivors during the initial-mark pause tonyp parents: 4836 diff changeset	114
eff609af17d7 7127706: G1: re-enable survivors during the initial-mark pause tonyp parents: 4836 diff changeset	115 double scan_end = os::elapsedTime();
6007 5c86f8211d1e 7160728: Introduce an extra logging level for G1 logging brutisso parents: 5963 diff changeset	116 if (G1Log::fine()) {
4837 eff609af17d7 7127706: G1: re-enable survivors during the initial-mark pause tonyp parents: 4836 diff changeset	117 gclog_or_tty->date_stamp(PrintGCDateStamps);
eff609af17d7 7127706: G1: re-enable survivors during the initial-mark pause tonyp parents: 4836 diff changeset	118 gclog_or_tty->stamp(PrintGCTimeStamps);
8109 96c885895d22 8007221: G1: concurrent phase durations do not state the time units ("secs") johnc parents: 7450 diff changeset	119 gclog_or_tty->print_cr("[GC concurrent-root-region-scan-end, %1.7lf secs]",
4837 eff609af17d7 7127706: G1: re-enable survivors during the initial-mark pause tonyp parents: 4836 diff changeset	120 scan_end - scan_start);
eff609af17d7 7127706: G1: re-enable survivors during the initial-mark pause tonyp parents: 4836 diff changeset	121 }
eff609af17d7 7127706: G1: re-enable survivors during the initial-mark pause tonyp parents: 4836 diff changeset	122 }
eff609af17d7 7127706: G1: re-enable survivors during the initial-mark pause tonyp parents: 4836 diff changeset	123
eff609af17d7 7127706: G1: re-enable survivors during the initial-mark pause tonyp parents: 4836 diff changeset	124 double mark_start_sec = os::elapsedTime();
6007 5c86f8211d1e 7160728: Introduce an extra logging level for G1 logging brutisso parents: 5963 diff changeset	125 if (G1Log::fine()) {
342 37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	126 gclog_or_tty->date_stamp(PrintGCDateStamps);
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	127 gclog_or_tty->stamp(PrintGCTimeStamps);
619 7ea5ca260b28 6814467: G1: small fixes related to concurrent marking verboseness tonyp parents: 342 diff changeset	128 gclog_or_tty->print_cr("[GC concurrent-mark-start]");
342 37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	129 }
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	130
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	131 int iter = 0;
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	132 do {
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	133 iter++;
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	134 if (!cm()->has_aborted()) {
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	135 _cm->markFromRoots();
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	136 }
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	137
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	138 double mark_end_time = os::elapsedVTime();
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	139 double mark_end_sec = os::elapsedTime();
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	140 _vtime_mark_accum += (mark_end_time - cycle_start);
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	141 if (!cm()->has_aborted()) {
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	142 if (g1_policy->adaptive_young_list_length()) {
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	143 double now = os::elapsedTime();
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	144 double remark_prediction_ms = g1_policy->predict_remark_time_ms();
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	145 jlong sleep_time_ms = mmu_tracker->when_ms(now, remark_prediction_ms);
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	146 os::sleep(current_thread, sleep_time_ms, false);
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	147 }
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	148
6007 5c86f8211d1e 7160728: Introduce an extra logging level for G1 logging brutisso parents: 5963 diff changeset	149 if (G1Log::fine()) {
342 37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	150 gclog_or_tty->date_stamp(PrintGCDateStamps);
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	151 gclog_or_tty->stamp(PrintGCTimeStamps);
8109 96c885895d22 8007221: G1: concurrent phase durations do not state the time units ("secs") johnc parents: 7450 diff changeset	152 gclog_or_tty->print_cr("[GC concurrent-mark-end, %1.7lf secs]",
342 37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	153 mark_end_sec - mark_start_sec);
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	154 }
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	155
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	156 CMCheckpointRootsFinalClosure final_cl(_cm);
10405 f2110083203d 8005849: JEP 167: Event-Based JVM Tracing sla parents: 8109 diff changeset	157 VM_CGC_Operation op(&final_cl, "GC remark", true /* needs_pll */);
342 37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	158 VMThread::execute(&op);
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	159 }
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	160 if (cm()->restart_for_overflow()) {
7450 d275c3dc73e6 8004816: G1: Kitchensink failures after marking stack changes johnc parents: 6120 diff changeset	161 if (G1TraceMarkStackOverflow) {
d275c3dc73e6 8004816: G1: Kitchensink failures after marking stack changes johnc parents: 6120 diff changeset	162 gclog_or_tty->print_cr("Restarting conc marking because of MS overflow "
d275c3dc73e6 8004816: G1: Kitchensink failures after marking stack changes johnc parents: 6120 diff changeset	163 "in remark (restart #%d).", iter);
d275c3dc73e6 8004816: G1: Kitchensink failures after marking stack changes johnc parents: 6120 diff changeset	164 }
6007 5c86f8211d1e 7160728: Introduce an extra logging level for G1 logging brutisso parents: 5963 diff changeset	165 if (G1Log::fine()) {
342 37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	166 gclog_or_tty->date_stamp(PrintGCDateStamps);
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	167 gclog_or_tty->stamp(PrintGCTimeStamps);
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	168 gclog_or_tty->print_cr("[GC concurrent-mark-restart-for-overflow]");
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	169 }
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	170 }
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	171 } while (cm()->restart_for_overflow());
4022 db89aa49298f 7099824: G1: we should take the pending list lock before doing the remark pause johnc parents: 4012 diff changeset	172
342 37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	173 double end_time = os::elapsedVTime();
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	174 // Update the total virtual time before doing this, since it will try
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	175 // to measure it to get the vtime for this marking. We purposely
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	176 // neglect the presumably-short "completeCleanup" phase here.
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	177 _vtime_accum = (end_time - _vtime_start);
4836 d30fa85f9994 6484965: G1: piggy-back liveness accounting phase on marking johnc parents: 4096 diff changeset	178
342 37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	179 if (!cm()->has_aborted()) {
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	180 if (g1_policy->adaptive_young_list_length()) {
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	181 double now = os::elapsedTime();
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	182 double cleanup_prediction_ms = g1_policy->predict_cleanup_time_ms();
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	183 jlong sleep_time_ms = mmu_tracker->when_ms(now, cleanup_prediction_ms);
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	184 os::sleep(current_thread, sleep_time_ms, false);
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	185 }
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	186
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	187 CMCleanUp cl_cl(_cm);
10405 f2110083203d 8005849: JEP 167: Event-Based JVM Tracing sla parents: 8109 diff changeset	188 VM_CGC_Operation op(&cl_cl, "GC cleanup", false /* needs_pll */);
342 37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	189 VMThread::execute(&op);
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	190 } else {
4096 00dd86e542eb 7114303: G1: assert(_g1->mark_in_progress()) failed: shouldn't be here otherwise johnc parents: 4022 diff changeset	191 // We don't want to update the marking status if a GC pause
00dd86e542eb 7114303: G1: assert(_g1->mark_in_progress()) failed: shouldn't be here otherwise johnc parents: 4022 diff changeset	192 // is already underway.
00dd86e542eb 7114303: G1: assert(_g1->mark_in_progress()) failed: shouldn't be here otherwise johnc parents: 4022 diff changeset	193 _sts.join();
2152 0fa27f37d4d4 6977804: G1: remove the zero-filling thread tonyp parents: 2030 diff changeset	194 g1h->set_marking_complete();
4096 00dd86e542eb 7114303: G1: assert(_g1->mark_in_progress()) failed: shouldn't be here otherwise johnc parents: 4022 diff changeset	195 _sts.leave();
342 37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	196 }
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	197
2152 0fa27f37d4d4 6977804: G1: remove the zero-filling thread tonyp parents: 2030 diff changeset	198 // Check if cleanup set the free_regions_coming flag. If it
0fa27f37d4d4 6977804: G1: remove the zero-filling thread tonyp parents: 2030 diff changeset	199 // hasn't, we can just skip the next step.
0fa27f37d4d4 6977804: G1: remove the zero-filling thread tonyp parents: 2030 diff changeset	200 if (g1h->free_regions_coming()) {
0fa27f37d4d4 6977804: G1: remove the zero-filling thread tonyp parents: 2030 diff changeset	201 // The following will finish freeing up any regions that we
0fa27f37d4d4 6977804: G1: remove the zero-filling thread tonyp parents: 2030 diff changeset	202 // found to be empty during cleanup. We'll do this part
0fa27f37d4d4 6977804: G1: remove the zero-filling thread tonyp parents: 2030 diff changeset	203 // without joining the suspendible set. If an evacuation pause
2361 1216415d8e35 7014923: G1: code cleanup tonyp parents: 2185 diff changeset	204 // takes place, then we would carry on freeing regions in
2152 0fa27f37d4d4 6977804: G1: remove the zero-filling thread tonyp parents: 2030 diff changeset	205 // case they are needed by the pause. If a Full GC takes
2361 1216415d8e35 7014923: G1: code cleanup tonyp parents: 2185 diff changeset	206 // place, it would wait for us to process the regions
2152 0fa27f37d4d4 6977804: G1: remove the zero-filling thread tonyp parents: 2030 diff changeset	207 // reclaimed by cleanup.
0fa27f37d4d4 6977804: G1: remove the zero-filling thread tonyp parents: 2030 diff changeset	208
342 37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	209 double cleanup_start_sec = os::elapsedTime();
6007 5c86f8211d1e 7160728: Introduce an extra logging level for G1 logging brutisso parents: 5963 diff changeset	210 if (G1Log::fine()) {
342 37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	211 gclog_or_tty->date_stamp(PrintGCDateStamps);
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	212 gclog_or_tty->stamp(PrintGCTimeStamps);
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	213 gclog_or_tty->print_cr("[GC concurrent-cleanup-start]");
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	214 }
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	215
4012 ec4b032a4977 7098085: G1: partially-young GCs not initiated under certain circumstances tonyp parents: 3867 diff changeset	216 // Now do the concurrent cleanup operation.
342 37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	217 _cm->completeCleanup();
4012 ec4b032a4977 7098085: G1: partially-young GCs not initiated under certain circumstances tonyp parents: 3867 diff changeset	218
2185 b7a938236e43 7014679: G1: deadlock during concurrent cleanup tonyp parents: 2172 diff changeset	219 // Notify anyone who's waiting that there are no more free
4012 ec4b032a4977 7098085: G1: partially-young GCs not initiated under certain circumstances tonyp parents: 3867 diff changeset	220 // regions coming. We have to do this before we join the STS
ec4b032a4977 7098085: G1: partially-young GCs not initiated under certain circumstances tonyp parents: 3867 diff changeset	221 // (in fact, we should not attempt to join the STS in the
ec4b032a4977 7098085: G1: partially-young GCs not initiated under certain circumstances tonyp parents: 3867 diff changeset	222 // interval between finishing the cleanup pause and clearing
ec4b032a4977 7098085: G1: partially-young GCs not initiated under certain circumstances tonyp parents: 3867 diff changeset	223 // the free_regions_coming flag) otherwise we might deadlock:
ec4b032a4977 7098085: G1: partially-young GCs not initiated under certain circumstances tonyp parents: 3867 diff changeset	224 // a GC worker could be blocked waiting for the notification
ec4b032a4977 7098085: G1: partially-young GCs not initiated under certain circumstances tonyp parents: 3867 diff changeset	225 // whereas this thread will be blocked for the pause to finish
ec4b032a4977 7098085: G1: partially-young GCs not initiated under certain circumstances tonyp parents: 3867 diff changeset	226 // while it's trying to join the STS, which is conditional on
ec4b032a4977 7098085: G1: partially-young GCs not initiated under certain circumstances tonyp parents: 3867 diff changeset	227 // the GC workers finishing.
2185 b7a938236e43 7014679: G1: deadlock during concurrent cleanup tonyp parents: 2172 diff changeset	228 g1h->reset_free_regions_coming();
b7a938236e43 7014679: G1: deadlock during concurrent cleanup tonyp parents: 2172 diff changeset	229
2152 0fa27f37d4d4 6977804: G1: remove the zero-filling thread tonyp parents: 2030 diff changeset	230 double cleanup_end_sec = os::elapsedTime();
6007 5c86f8211d1e 7160728: Introduce an extra logging level for G1 logging brutisso parents: 5963 diff changeset	231 if (G1Log::fine()) {
2152 0fa27f37d4d4 6977804: G1: remove the zero-filling thread tonyp parents: 2030 diff changeset	232 gclog_or_tty->date_stamp(PrintGCDateStamps);
0fa27f37d4d4 6977804: G1: remove the zero-filling thread tonyp parents: 2030 diff changeset	233 gclog_or_tty->stamp(PrintGCTimeStamps);
8109 96c885895d22 8007221: G1: concurrent phase durations do not state the time units ("secs") johnc parents: 7450 diff changeset	234 gclog_or_tty->print_cr("[GC concurrent-cleanup-end, %1.7lf secs]",
2152 0fa27f37d4d4 6977804: G1: remove the zero-filling thread tonyp parents: 2030 diff changeset	235 cleanup_end_sec - cleanup_start_sec);
342 37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	236 }
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	237 }
2152 0fa27f37d4d4 6977804: G1: remove the zero-filling thread tonyp parents: 2030 diff changeset	238 guarantee(cm()->cleanup_list_is_empty(),
0fa27f37d4d4 6977804: G1: remove the zero-filling thread tonyp parents: 2030 diff changeset	239 "at this point there should be no regions on the cleanup list");
342 37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	240
4012 ec4b032a4977 7098085: G1: partially-young GCs not initiated under certain circumstances tonyp parents: 3867 diff changeset	241 // There is a tricky race before recording that the concurrent
ec4b032a4977 7098085: G1: partially-young GCs not initiated under certain circumstances tonyp parents: 3867 diff changeset	242 // cleanup has completed and a potential Full GC starting around
ec4b032a4977 7098085: G1: partially-young GCs not initiated under certain circumstances tonyp parents: 3867 diff changeset	243 // the same time. We want to make sure that the Full GC calls
ec4b032a4977 7098085: G1: partially-young GCs not initiated under certain circumstances tonyp parents: 3867 diff changeset	244 // abort() on concurrent mark after
ec4b032a4977 7098085: G1: partially-young GCs not initiated under certain circumstances tonyp parents: 3867 diff changeset	245 // record_concurrent_mark_cleanup_completed(), since abort() is
ec4b032a4977 7098085: G1: partially-young GCs not initiated under certain circumstances tonyp parents: 3867 diff changeset	246 // the method that will reset the concurrent mark state. If we
ec4b032a4977 7098085: G1: partially-young GCs not initiated under certain circumstances tonyp parents: 3867 diff changeset	247 // end up calling record_concurrent_mark_cleanup_completed()
ec4b032a4977 7098085: G1: partially-young GCs not initiated under certain circumstances tonyp parents: 3867 diff changeset	248 // after abort() then we might incorrectly undo some of the work
ec4b032a4977 7098085: G1: partially-young GCs not initiated under certain circumstances tonyp parents: 3867 diff changeset	249 // abort() did. Checking the has_aborted() flag after joining
ec4b032a4977 7098085: G1: partially-young GCs not initiated under certain circumstances tonyp parents: 3867 diff changeset	250 // the STS allows the correct ordering of the two methods. There
ec4b032a4977 7098085: G1: partially-young GCs not initiated under certain circumstances tonyp parents: 3867 diff changeset	251 // are two scenarios:
ec4b032a4977 7098085: G1: partially-young GCs not initiated under certain circumstances tonyp parents: 3867 diff changeset	252 //
ec4b032a4977 7098085: G1: partially-young GCs not initiated under certain circumstances tonyp parents: 3867 diff changeset	253 // a) If we reach here before the Full GC, the fact that we have
ec4b032a4977 7098085: G1: partially-young GCs not initiated under certain circumstances tonyp parents: 3867 diff changeset	254 // joined the STS means that the Full GC cannot start until we
ec4b032a4977 7098085: G1: partially-young GCs not initiated under certain circumstances tonyp parents: 3867 diff changeset	255 // leave the STS, so record_concurrent_mark_cleanup_completed()
ec4b032a4977 7098085: G1: partially-young GCs not initiated under certain circumstances tonyp parents: 3867 diff changeset	256 // will complete before abort() is called.
ec4b032a4977 7098085: G1: partially-young GCs not initiated under certain circumstances tonyp parents: 3867 diff changeset	257 //
ec4b032a4977 7098085: G1: partially-young GCs not initiated under certain circumstances tonyp parents: 3867 diff changeset	258 // b) If we reach here during the Full GC, we'll be held up from
ec4b032a4977 7098085: G1: partially-young GCs not initiated under certain circumstances tonyp parents: 3867 diff changeset	259 // joining the STS until the Full GC is done, which means that
ec4b032a4977 7098085: G1: partially-young GCs not initiated under certain circumstances tonyp parents: 3867 diff changeset	260 // abort() will have completed and has_aborted() will return
ec4b032a4977 7098085: G1: partially-young GCs not initiated under certain circumstances tonyp parents: 3867 diff changeset	261 // true to prevent us from calling
ec4b032a4977 7098085: G1: partially-young GCs not initiated under certain circumstances tonyp parents: 3867 diff changeset	262 // record_concurrent_mark_cleanup_completed() (and, in fact, it's
ec4b032a4977 7098085: G1: partially-young GCs not initiated under certain circumstances tonyp parents: 3867 diff changeset	263 // not needed any more as the concurrent mark state has been
ec4b032a4977 7098085: G1: partially-young GCs not initiated under certain circumstances tonyp parents: 3867 diff changeset	264 // already reset).
ec4b032a4977 7098085: G1: partially-young GCs not initiated under certain circumstances tonyp parents: 3867 diff changeset	265 _sts.join();
ec4b032a4977 7098085: G1: partially-young GCs not initiated under certain circumstances tonyp parents: 3867 diff changeset	266 if (!cm()->has_aborted()) {
ec4b032a4977 7098085: G1: partially-young GCs not initiated under certain circumstances tonyp parents: 3867 diff changeset	267 g1_policy->record_concurrent_mark_cleanup_completed();
ec4b032a4977 7098085: G1: partially-young GCs not initiated under certain circumstances tonyp parents: 3867 diff changeset	268 }
ec4b032a4977 7098085: G1: partially-young GCs not initiated under certain circumstances tonyp parents: 3867 diff changeset	269 _sts.leave();
ec4b032a4977 7098085: G1: partially-young GCs not initiated under certain circumstances tonyp parents: 3867 diff changeset	270
342 37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	271 if (cm()->has_aborted()) {
6007 5c86f8211d1e 7160728: Introduce an extra logging level for G1 logging brutisso parents: 5963 diff changeset	272 if (G1Log::fine()) {
342 37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	273 gclog_or_tty->date_stamp(PrintGCDateStamps);
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	274 gclog_or_tty->stamp(PrintGCTimeStamps);
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	275 gclog_or_tty->print_cr("[GC concurrent-mark-abort]");
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	276 }
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	277 }
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	278
4012 ec4b032a4977 7098085: G1: partially-young GCs not initiated under certain circumstances tonyp parents: 3867 diff changeset	279 // We now want to allow clearing of the marking bitmap to be
342 37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	280 // suspended by a collection pause.
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	281 _sts.join();
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	282 _cm->clearNextBitmap();
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	283 _sts.leave();
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	284 }
1656 4e5661ba9d98 6944166: G1: explicit GCs are not always handled correctly tonyp parents: 1552 diff changeset	285
4e5661ba9d98 6944166: G1: explicit GCs are not always handled correctly tonyp parents: 1552 diff changeset	286 // Update the number of full collections that have been
4e5661ba9d98 6944166: G1: explicit GCs are not always handled correctly tonyp parents: 1552 diff changeset	287 // completed. This will also notify the FullGCCount_lock in case a
4e5661ba9d98 6944166: G1: explicit GCs are not always handled correctly tonyp parents: 1552 diff changeset	288 // Java thread is waiting for a full GC to happen (e.g., it
4e5661ba9d98 6944166: G1: explicit GCs are not always handled correctly tonyp parents: 1552 diff changeset	289 // called System.gc() with +ExplicitGCInvokesConcurrent).
2030 fb712ff22571 7000559: G1: assertion failure !outer \|\| (full_collections_started == _full_collections_completed + 1) tonyp parents: 1972 diff changeset	290 _sts.join();
6120 37552638d24a 7172388: G1: _total_full_collections should not be incremented for concurrent cycles brutisso parents: 6007 diff changeset	291 g1h->increment_old_marking_cycles_completed(true /* concurrent */);
10405 f2110083203d 8005849: JEP 167: Event-Based JVM Tracing sla parents: 8109 diff changeset	292 g1h->register_concurrent_cycle_end();
2030 fb712ff22571 7000559: G1: assertion failure !outer \|\| (full_collections_started == _full_collections_completed + 1) tonyp parents: 1972 diff changeset	293 _sts.leave();
342 37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	294 }
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	295 assert(_should_terminate, "just checking");
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	296
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	297 terminate();
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	298 }
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	299
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	300
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	301 void ConcurrentMarkThread::yield() {
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	302 _sts.yield("Concurrent Mark");
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	303 }
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	304
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	305 void ConcurrentMarkThread::stop() {
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	306 // it is ok to take late safepoints here, if needed
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	307 MutexLockerEx mu(Terminator_lock);
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	308 _should_terminate = true;
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	309 while (!_has_terminated) {
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	310 Terminator_lock->wait();
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	311 }
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	312 }
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	313
1019 035d2e036a9b 6885041: G1: inconsistent thread dump tonyp parents: 936 diff changeset	314 void ConcurrentMarkThread::print() const {
035d2e036a9b 6885041: G1: inconsistent thread dump tonyp parents: 936 diff changeset	315 print_on(tty);
035d2e036a9b 6885041: G1: inconsistent thread dump tonyp parents: 936 diff changeset	316 }
035d2e036a9b 6885041: G1: inconsistent thread dump tonyp parents: 936 diff changeset	317
035d2e036a9b 6885041: G1: inconsistent thread dump tonyp parents: 936 diff changeset	318 void ConcurrentMarkThread::print_on(outputStream* st) const {
035d2e036a9b 6885041: G1: inconsistent thread dump tonyp parents: 936 diff changeset	319 st->print("\"G1 Main Concurrent Mark GC Thread\" ");
035d2e036a9b 6885041: G1: inconsistent thread dump tonyp parents: 936 diff changeset	320 Thread::print_on(st);
035d2e036a9b 6885041: G1: inconsistent thread dump tonyp parents: 936 diff changeset	321 st->cr();
342 37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	322 }
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	323
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	324 void ConcurrentMarkThread::sleepBeforeNextCycle() {
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	325 // We join here because we don't want to do the "shouldConcurrentMark()"
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	326 // below while the world is otherwise stopped.
1840 4e0094bc41fa 6983311: G1: LoopTest hangs when run with -XX:+ExplicitInvokesConcurrent johnc parents: 1656 diff changeset	327 assert(!in_progress(), "should have been cleared");
4e0094bc41fa 6983311: G1: LoopTest hangs when run with -XX:+ExplicitInvokesConcurrent johnc parents: 1656 diff changeset	328
342 37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	329 MutexLockerEx x(CGC_lock, Mutex::_no_safepoint_check_flag);
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	330 while (!started()) {
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	331 CGC_lock->wait(Mutex::_no_safepoint_check_flag);
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	332 }
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	333 set_in_progress();
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	334 clear_started();
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	335 }
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	336
4022 db89aa49298f 7099824: G1: we should take the pending list lock before doing the remark pause johnc parents: 4012 diff changeset	337 // Note: As is the case with CMS - this method, although exported
db89aa49298f 7099824: G1: we should take the pending list lock before doing the remark pause johnc parents: 4012 diff changeset	338 // by the ConcurrentMarkThread, which is a non-JavaThread, can only
db89aa49298f 7099824: G1: we should take the pending list lock before doing the remark pause johnc parents: 4012 diff changeset	339 // be called by a JavaThread. Currently this is done at vm creation
db89aa49298f 7099824: G1: we should take the pending list lock before doing the remark pause johnc parents: 4012 diff changeset	340 // time (post-vm-init) by the main/Primordial (Java)Thread.
db89aa49298f 7099824: G1: we should take the pending list lock before doing the remark pause johnc parents: 4012 diff changeset	341 // XXX Consider changing this in the future to allow the CM thread
342 37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	342 // itself to create this thread?
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	343 void ConcurrentMarkThread::makeSurrogateLockerThread(TRAPS) {
4022 db89aa49298f 7099824: G1: we should take the pending list lock before doing the remark pause johnc parents: 4012 diff changeset	344 assert(UseG1GC, "SLT thread needed only for concurrent GC");
db89aa49298f 7099824: G1: we should take the pending list lock before doing the remark pause johnc parents: 4012 diff changeset	345 assert(THREAD->is_Java_thread(), "must be a Java thread");
342 37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	346 assert(_slt == NULL, "SLT already created");
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	347 _slt = SurrogateLockerThread::make(THREAD);
37f87013dfd8 6711316: Open source the Garbage-First garbage collector ysr parents: diff changeset	348 }

Mercurial > hg > truffle

annotate src/share/vm/gc_implementation/g1/concurrentMarkThread.cpp @ 12233:40136aa2cdb1