annotate src/share/vm/gc_implementation/concurrentMarkSweep/vmCMSOperations.cpp @ 6862:8a5ea0a9ccc4

7127708: G1: change task num types from int to uint in concurrent mark Summary: Change the type of various task num fields, parameters etc to unsigned and rename them to be more consistent with the other collectors. Code changes were also reviewed by Vitaly Davidovich. Reviewed-by: johnc Contributed-by: Kaushik Srenevasan <kaushik@twitter.com>
author johnc
date Sat, 06 Oct 2012 01:17:44 -0700
parents 9d679effd28c
children b9a9ed0f8eeb 4202510ee0fe
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
rev   line source
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1 /*
2365
a181f3a124dd 6987703: iCMS: Intermittent hang with gc/gctests/CallGC/CallGC01 and +ExplicitGCInvokesConcurrent
ysr
parents: 1972
diff changeset
2 * Copyright (c) 2005, 2011, Oracle and/or its affiliates. All rights reserved.
0
a61af66fc99e Initial load
duke
parents:
diff changeset
3 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
a61af66fc99e Initial load
duke
parents:
diff changeset
4 *
a61af66fc99e Initial load
duke
parents:
diff changeset
5 * This code is free software; you can redistribute it and/or modify it
a61af66fc99e Initial load
duke
parents:
diff changeset
6 * under the terms of the GNU General Public License version 2 only, as
a61af66fc99e Initial load
duke
parents:
diff changeset
7 * published by the Free Software Foundation.
a61af66fc99e Initial load
duke
parents:
diff changeset
8 *
a61af66fc99e Initial load
duke
parents:
diff changeset
9 * This code is distributed in the hope that it will be useful, but WITHOUT
a61af66fc99e Initial load
duke
parents:
diff changeset
10 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
a61af66fc99e Initial load
duke
parents:
diff changeset
11 * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License
a61af66fc99e Initial load
duke
parents:
diff changeset
12 * version 2 for more details (a copy is included in the LICENSE file that
a61af66fc99e Initial load
duke
parents:
diff changeset
13 * accompanied this code).
a61af66fc99e Initial load
duke
parents:
diff changeset
14 *
a61af66fc99e Initial load
duke
parents:
diff changeset
15 * You should have received a copy of the GNU General Public License version
a61af66fc99e Initial load
duke
parents:
diff changeset
16 * 2 along with this work; if not, write to the Free Software Foundation,
a61af66fc99e Initial load
duke
parents:
diff changeset
17 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
a61af66fc99e Initial load
duke
parents:
diff changeset
18 *
1552
c18cbe5936b8 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 1520
diff changeset
19 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
c18cbe5936b8 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 1520
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: 1520
diff changeset
21 * questions.
0
a61af66fc99e Initial load
duke
parents:
diff changeset
22 *
a61af66fc99e Initial load
duke
parents:
diff changeset
23 */
a61af66fc99e Initial load
duke
parents:
diff changeset
24
1972
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1656
diff changeset
25 #include "precompiled.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1656
diff changeset
26 #include "gc_implementation/concurrentMarkSweep/concurrentMarkSweepGeneration.inline.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1656
diff changeset
27 #include "gc_implementation/concurrentMarkSweep/concurrentMarkSweepThread.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1656
diff changeset
28 #include "gc_implementation/concurrentMarkSweep/vmCMSOperations.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1656
diff changeset
29 #include "gc_implementation/shared/isGCActiveMark.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1656
diff changeset
30 #include "memory/gcLocker.inline.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1656
diff changeset
31 #include "runtime/interfaceSupport.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1656
diff changeset
32 #include "utilities/dtrace.hpp"
4006
436b4a3231bf 7098194: integrate macosx-port changes
dcubed
parents: 2365
diff changeset
33
436b4a3231bf 7098194: integrate macosx-port changes
dcubed
parents: 2365
diff changeset
34
436b4a3231bf 7098194: integrate macosx-port changes
dcubed
parents: 2365
diff changeset
35 #ifndef USDT2
0
a61af66fc99e Initial load
duke
parents:
diff changeset
36 HS_DTRACE_PROBE_DECL(hs_private, cms__initmark__begin);
a61af66fc99e Initial load
duke
parents:
diff changeset
37 HS_DTRACE_PROBE_DECL(hs_private, cms__initmark__end);
a61af66fc99e Initial load
duke
parents:
diff changeset
38
a61af66fc99e Initial load
duke
parents:
diff changeset
39 HS_DTRACE_PROBE_DECL(hs_private, cms__remark__begin);
a61af66fc99e Initial load
duke
parents:
diff changeset
40 HS_DTRACE_PROBE_DECL(hs_private, cms__remark__end);
4006
436b4a3231bf 7098194: integrate macosx-port changes
dcubed
parents: 2365
diff changeset
41 #endif /* !USDT2 */
0
a61af66fc99e Initial load
duke
parents:
diff changeset
42
a61af66fc99e Initial load
duke
parents:
diff changeset
43 //////////////////////////////////////////////////////////
a61af66fc99e Initial load
duke
parents:
diff changeset
44 // Methods in abstract class VM_CMS_Operation
a61af66fc99e Initial load
duke
parents:
diff changeset
45 //////////////////////////////////////////////////////////
a61af66fc99e Initial load
duke
parents:
diff changeset
46 void VM_CMS_Operation::acquire_pending_list_lock() {
a61af66fc99e Initial load
duke
parents:
diff changeset
47 // The caller may block while communicating
a61af66fc99e Initial load
duke
parents:
diff changeset
48 // with the SLT thread in order to acquire/release the PLL.
a61af66fc99e Initial load
duke
parents:
diff changeset
49 ConcurrentMarkSweepThread::slt()->
a61af66fc99e Initial load
duke
parents:
diff changeset
50 manipulatePLL(SurrogateLockerThread::acquirePLL);
a61af66fc99e Initial load
duke
parents:
diff changeset
51 }
a61af66fc99e Initial load
duke
parents:
diff changeset
52
a61af66fc99e Initial load
duke
parents:
diff changeset
53 void VM_CMS_Operation::release_and_notify_pending_list_lock() {
a61af66fc99e Initial load
duke
parents:
diff changeset
54 // The caller may block while communicating
a61af66fc99e Initial load
duke
parents:
diff changeset
55 // with the SLT thread in order to acquire/release the PLL.
a61af66fc99e Initial load
duke
parents:
diff changeset
56 ConcurrentMarkSweepThread::slt()->
a61af66fc99e Initial load
duke
parents:
diff changeset
57 manipulatePLL(SurrogateLockerThread::releaseAndNotifyPLL);
a61af66fc99e Initial load
duke
parents:
diff changeset
58 }
a61af66fc99e Initial load
duke
parents:
diff changeset
59
a61af66fc99e Initial load
duke
parents:
diff changeset
60 void VM_CMS_Operation::verify_before_gc() {
a61af66fc99e Initial load
duke
parents:
diff changeset
61 if (VerifyBeforeGC &&
a61af66fc99e Initial load
duke
parents:
diff changeset
62 GenCollectedHeap::heap()->total_collections() >= VerifyGCStartAt) {
a61af66fc99e Initial load
duke
parents:
diff changeset
63 HandleMark hm;
a61af66fc99e Initial load
duke
parents:
diff changeset
64 FreelistLocker x(_collector);
a61af66fc99e Initial load
duke
parents:
diff changeset
65 MutexLockerEx y(_collector->bitMapLock(), Mutex::_no_safepoint_check_flag);
a61af66fc99e Initial load
duke
parents:
diff changeset
66 Universe::heap()->prepare_for_verify();
a61af66fc99e Initial load
duke
parents:
diff changeset
67 Universe::verify(true);
a61af66fc99e Initial load
duke
parents:
diff changeset
68 }
a61af66fc99e Initial load
duke
parents:
diff changeset
69 }
a61af66fc99e Initial load
duke
parents:
diff changeset
70
a61af66fc99e Initial load
duke
parents:
diff changeset
71 void VM_CMS_Operation::verify_after_gc() {
a61af66fc99e Initial load
duke
parents:
diff changeset
72 if (VerifyAfterGC &&
a61af66fc99e Initial load
duke
parents:
diff changeset
73 GenCollectedHeap::heap()->total_collections() >= VerifyGCStartAt) {
a61af66fc99e Initial load
duke
parents:
diff changeset
74 HandleMark hm;
a61af66fc99e Initial load
duke
parents:
diff changeset
75 FreelistLocker x(_collector);
a61af66fc99e Initial load
duke
parents:
diff changeset
76 MutexLockerEx y(_collector->bitMapLock(), Mutex::_no_safepoint_check_flag);
a61af66fc99e Initial load
duke
parents:
diff changeset
77 Universe::verify(true);
a61af66fc99e Initial load
duke
parents:
diff changeset
78 }
a61af66fc99e Initial load
duke
parents:
diff changeset
79 }
a61af66fc99e Initial load
duke
parents:
diff changeset
80
a61af66fc99e Initial load
duke
parents:
diff changeset
81 bool VM_CMS_Operation::lost_race() const {
a61af66fc99e Initial load
duke
parents:
diff changeset
82 if (CMSCollector::abstract_state() == CMSCollector::Idling) {
a61af66fc99e Initial load
duke
parents:
diff changeset
83 // We lost a race to a foreground collection
a61af66fc99e Initial load
duke
parents:
diff changeset
84 // -- there's nothing to do
a61af66fc99e Initial load
duke
parents:
diff changeset
85 return true;
a61af66fc99e Initial load
duke
parents:
diff changeset
86 }
a61af66fc99e Initial load
duke
parents:
diff changeset
87 assert(CMSCollector::abstract_state() == legal_state(),
a61af66fc99e Initial load
duke
parents:
diff changeset
88 "Inconsistent collector state?");
a61af66fc99e Initial load
duke
parents:
diff changeset
89 return false;
a61af66fc99e Initial load
duke
parents:
diff changeset
90 }
a61af66fc99e Initial load
duke
parents:
diff changeset
91
a61af66fc99e Initial load
duke
parents:
diff changeset
92 bool VM_CMS_Operation::doit_prologue() {
a61af66fc99e Initial load
duke
parents:
diff changeset
93 assert(Thread::current()->is_ConcurrentGC_thread(), "just checking");
a61af66fc99e Initial load
duke
parents:
diff changeset
94 assert(!CMSCollector::foregroundGCShouldWait(), "Possible deadlock");
a61af66fc99e Initial load
duke
parents:
diff changeset
95 assert(!ConcurrentMarkSweepThread::cms_thread_has_cms_token(),
a61af66fc99e Initial load
duke
parents:
diff changeset
96 "Possible deadlock");
a61af66fc99e Initial load
duke
parents:
diff changeset
97
a61af66fc99e Initial load
duke
parents:
diff changeset
98 if (needs_pll()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
99 acquire_pending_list_lock();
a61af66fc99e Initial load
duke
parents:
diff changeset
100 }
a61af66fc99e Initial load
duke
parents:
diff changeset
101 // Get the Heap_lock after the pending_list_lock.
a61af66fc99e Initial load
duke
parents:
diff changeset
102 Heap_lock->lock();
a61af66fc99e Initial load
duke
parents:
diff changeset
103 if (lost_race()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
104 assert(_prologue_succeeded == false, "Initialized in c'tor");
a61af66fc99e Initial load
duke
parents:
diff changeset
105 Heap_lock->unlock();
a61af66fc99e Initial load
duke
parents:
diff changeset
106 if (needs_pll()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
107 release_and_notify_pending_list_lock();
a61af66fc99e Initial load
duke
parents:
diff changeset
108 }
a61af66fc99e Initial load
duke
parents:
diff changeset
109 } else {
a61af66fc99e Initial load
duke
parents:
diff changeset
110 _prologue_succeeded = true;
a61af66fc99e Initial load
duke
parents:
diff changeset
111 }
a61af66fc99e Initial load
duke
parents:
diff changeset
112 return _prologue_succeeded;
a61af66fc99e Initial load
duke
parents:
diff changeset
113 }
a61af66fc99e Initial load
duke
parents:
diff changeset
114
a61af66fc99e Initial load
duke
parents:
diff changeset
115 void VM_CMS_Operation::doit_epilogue() {
a61af66fc99e Initial load
duke
parents:
diff changeset
116 assert(Thread::current()->is_ConcurrentGC_thread(), "just checking");
a61af66fc99e Initial load
duke
parents:
diff changeset
117 assert(!CMSCollector::foregroundGCShouldWait(), "Possible deadlock");
a61af66fc99e Initial load
duke
parents:
diff changeset
118 assert(!ConcurrentMarkSweepThread::cms_thread_has_cms_token(),
a61af66fc99e Initial load
duke
parents:
diff changeset
119 "Possible deadlock");
a61af66fc99e Initial load
duke
parents:
diff changeset
120
a61af66fc99e Initial load
duke
parents:
diff changeset
121 // Release the Heap_lock first.
a61af66fc99e Initial load
duke
parents:
diff changeset
122 Heap_lock->unlock();
a61af66fc99e Initial load
duke
parents:
diff changeset
123 if (needs_pll()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
124 release_and_notify_pending_list_lock();
a61af66fc99e Initial load
duke
parents:
diff changeset
125 }
a61af66fc99e Initial load
duke
parents:
diff changeset
126 }
a61af66fc99e Initial load
duke
parents:
diff changeset
127
a61af66fc99e Initial load
duke
parents:
diff changeset
128 //////////////////////////////////////////////////////////
a61af66fc99e Initial load
duke
parents:
diff changeset
129 // Methods in class VM_CMS_Initial_Mark
a61af66fc99e Initial load
duke
parents:
diff changeset
130 //////////////////////////////////////////////////////////
a61af66fc99e Initial load
duke
parents:
diff changeset
131 void VM_CMS_Initial_Mark::doit() {
a61af66fc99e Initial load
duke
parents:
diff changeset
132 if (lost_race()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
133 // Nothing to do.
a61af66fc99e Initial load
duke
parents:
diff changeset
134 return;
a61af66fc99e Initial load
duke
parents:
diff changeset
135 }
4006
436b4a3231bf 7098194: integrate macosx-port changes
dcubed
parents: 2365
diff changeset
136 #ifndef USDT2
0
a61af66fc99e Initial load
duke
parents:
diff changeset
137 HS_DTRACE_PROBE(hs_private, cms__initmark__begin);
4006
436b4a3231bf 7098194: integrate macosx-port changes
dcubed
parents: 2365
diff changeset
138 #else /* USDT2 */
436b4a3231bf 7098194: integrate macosx-port changes
dcubed
parents: 2365
diff changeset
139 HS_PRIVATE_CMS_INITMARK_BEGIN(
436b4a3231bf 7098194: integrate macosx-port changes
dcubed
parents: 2365
diff changeset
140 );
436b4a3231bf 7098194: integrate macosx-port changes
dcubed
parents: 2365
diff changeset
141 #endif /* USDT2 */
0
a61af66fc99e Initial load
duke
parents:
diff changeset
142
a61af66fc99e Initial load
duke
parents:
diff changeset
143 GenCollectedHeap* gch = GenCollectedHeap::heap();
a61af66fc99e Initial load
duke
parents:
diff changeset
144 GCCauseSetter gccs(gch, GCCause::_cms_initial_mark);
a61af66fc99e Initial load
duke
parents:
diff changeset
145
a61af66fc99e Initial load
duke
parents:
diff changeset
146 VM_CMS_Operation::verify_before_gc();
a61af66fc99e Initial load
duke
parents:
diff changeset
147
a61af66fc99e Initial load
duke
parents:
diff changeset
148 IsGCActiveMark x; // stop-world GC active
6064
9d679effd28c 7166894: Add gc cause to GC logging for all collectors
brutisso
parents: 4006
diff changeset
149 _collector->do_CMS_operation(CMSCollector::CMS_op_checkpointRootsInitial, gch->gc_cause());
0
a61af66fc99e Initial load
duke
parents:
diff changeset
150
a61af66fc99e Initial load
duke
parents:
diff changeset
151 VM_CMS_Operation::verify_after_gc();
4006
436b4a3231bf 7098194: integrate macosx-port changes
dcubed
parents: 2365
diff changeset
152 #ifndef USDT2
0
a61af66fc99e Initial load
duke
parents:
diff changeset
153 HS_DTRACE_PROBE(hs_private, cms__initmark__end);
4006
436b4a3231bf 7098194: integrate macosx-port changes
dcubed
parents: 2365
diff changeset
154 #else /* USDT2 */
436b4a3231bf 7098194: integrate macosx-port changes
dcubed
parents: 2365
diff changeset
155 HS_PRIVATE_CMS_INITMARK_END(
436b4a3231bf 7098194: integrate macosx-port changes
dcubed
parents: 2365
diff changeset
156 );
436b4a3231bf 7098194: integrate macosx-port changes
dcubed
parents: 2365
diff changeset
157 #endif /* USDT2 */
0
a61af66fc99e Initial load
duke
parents:
diff changeset
158 }
a61af66fc99e Initial load
duke
parents:
diff changeset
159
a61af66fc99e Initial load
duke
parents:
diff changeset
160 //////////////////////////////////////////////////////////
a61af66fc99e Initial load
duke
parents:
diff changeset
161 // Methods in class VM_CMS_Final_Remark_Operation
a61af66fc99e Initial load
duke
parents:
diff changeset
162 //////////////////////////////////////////////////////////
a61af66fc99e Initial load
duke
parents:
diff changeset
163 void VM_CMS_Final_Remark::doit() {
a61af66fc99e Initial load
duke
parents:
diff changeset
164 if (lost_race()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
165 // Nothing to do.
a61af66fc99e Initial load
duke
parents:
diff changeset
166 return;
a61af66fc99e Initial load
duke
parents:
diff changeset
167 }
4006
436b4a3231bf 7098194: integrate macosx-port changes
dcubed
parents: 2365
diff changeset
168 #ifndef USDT2
0
a61af66fc99e Initial load
duke
parents:
diff changeset
169 HS_DTRACE_PROBE(hs_private, cms__remark__begin);
4006
436b4a3231bf 7098194: integrate macosx-port changes
dcubed
parents: 2365
diff changeset
170 #else /* USDT2 */
436b4a3231bf 7098194: integrate macosx-port changes
dcubed
parents: 2365
diff changeset
171 HS_PRIVATE_CMS_REMARK_BEGIN(
436b4a3231bf 7098194: integrate macosx-port changes
dcubed
parents: 2365
diff changeset
172 );
436b4a3231bf 7098194: integrate macosx-port changes
dcubed
parents: 2365
diff changeset
173 #endif /* USDT2 */
0
a61af66fc99e Initial load
duke
parents:
diff changeset
174
a61af66fc99e Initial load
duke
parents:
diff changeset
175 GenCollectedHeap* gch = GenCollectedHeap::heap();
a61af66fc99e Initial load
duke
parents:
diff changeset
176 GCCauseSetter gccs(gch, GCCause::_cms_final_remark);
a61af66fc99e Initial load
duke
parents:
diff changeset
177
a61af66fc99e Initial load
duke
parents:
diff changeset
178 VM_CMS_Operation::verify_before_gc();
a61af66fc99e Initial load
duke
parents:
diff changeset
179
a61af66fc99e Initial load
duke
parents:
diff changeset
180 IsGCActiveMark x; // stop-world GC active
6064
9d679effd28c 7166894: Add gc cause to GC logging for all collectors
brutisso
parents: 4006
diff changeset
181 _collector->do_CMS_operation(CMSCollector::CMS_op_checkpointRootsFinal, gch->gc_cause());
0
a61af66fc99e Initial load
duke
parents:
diff changeset
182
a61af66fc99e Initial load
duke
parents:
diff changeset
183 VM_CMS_Operation::verify_after_gc();
4006
436b4a3231bf 7098194: integrate macosx-port changes
dcubed
parents: 2365
diff changeset
184 #ifndef USDT2
0
a61af66fc99e Initial load
duke
parents:
diff changeset
185 HS_DTRACE_PROBE(hs_private, cms__remark__end);
4006
436b4a3231bf 7098194: integrate macosx-port changes
dcubed
parents: 2365
diff changeset
186 #else /* USDT2 */
436b4a3231bf 7098194: integrate macosx-port changes
dcubed
parents: 2365
diff changeset
187 HS_PRIVATE_CMS_REMARK_END(
436b4a3231bf 7098194: integrate macosx-port changes
dcubed
parents: 2365
diff changeset
188 );
436b4a3231bf 7098194: integrate macosx-port changes
dcubed
parents: 2365
diff changeset
189 #endif /* USDT2 */
0
a61af66fc99e Initial load
duke
parents:
diff changeset
190 }
a61af66fc99e Initial load
duke
parents:
diff changeset
191
a61af66fc99e Initial load
duke
parents:
diff changeset
192 // VM operation to invoke a concurrent collection of a
a61af66fc99e Initial load
duke
parents:
diff changeset
193 // GenCollectedHeap heap.
a61af66fc99e Initial load
duke
parents:
diff changeset
194 void VM_GenCollectFullConcurrent::doit() {
a61af66fc99e Initial load
duke
parents:
diff changeset
195 assert(Thread::current()->is_VM_thread(), "Should be VM thread");
1520
bb843ebc7c55 6919638: CMS: ExplicitGCInvokesConcurrent misinteracts with gc locker
ysr
parents: 0
diff changeset
196 assert(GCLockerInvokesConcurrent || ExplicitGCInvokesConcurrent, "Unexpected");
0
a61af66fc99e Initial load
duke
parents:
diff changeset
197
a61af66fc99e Initial load
duke
parents:
diff changeset
198 GenCollectedHeap* gch = GenCollectedHeap::heap();
a61af66fc99e Initial load
duke
parents:
diff changeset
199 if (_gc_count_before == gch->total_collections()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
200 // The "full" of do_full_collection call below "forces"
a61af66fc99e Initial load
duke
parents:
diff changeset
201 // a collection; the second arg, 0, below ensures that
a61af66fc99e Initial load
duke
parents:
diff changeset
202 // only the young gen is collected. XXX In the future,
a61af66fc99e Initial load
duke
parents:
diff changeset
203 // we'll probably need to have something in this interface
a61af66fc99e Initial load
duke
parents:
diff changeset
204 // to say do this only if we are sure we will not bail
a61af66fc99e Initial load
duke
parents:
diff changeset
205 // out to a full collection in this attempt, but that's
a61af66fc99e Initial load
duke
parents:
diff changeset
206 // for the future.
a61af66fc99e Initial load
duke
parents:
diff changeset
207 assert(SafepointSynchronize::is_at_safepoint(),
a61af66fc99e Initial load
duke
parents:
diff changeset
208 "We can only be executing this arm of if at a safepoint");
a61af66fc99e Initial load
duke
parents:
diff changeset
209 GCCauseSetter gccs(gch, _gc_cause);
a61af66fc99e Initial load
duke
parents:
diff changeset
210 gch->do_full_collection(gch->must_clear_all_soft_refs(),
a61af66fc99e Initial load
duke
parents:
diff changeset
211 0 /* collect only youngest gen */);
a61af66fc99e Initial load
duke
parents:
diff changeset
212 } // Else no need for a foreground young gc
a61af66fc99e Initial load
duke
parents:
diff changeset
213 assert((_gc_count_before < gch->total_collections()) ||
a61af66fc99e Initial load
duke
parents:
diff changeset
214 (GC_locker::is_active() /* gc may have been skipped */
a61af66fc99e Initial load
duke
parents:
diff changeset
215 && (_gc_count_before == gch->total_collections())),
a61af66fc99e Initial load
duke
parents:
diff changeset
216 "total_collections() should be monotonically increasing");
a61af66fc99e Initial load
duke
parents:
diff changeset
217
a61af66fc99e Initial load
duke
parents:
diff changeset
218 MutexLockerEx x(FullGCCount_lock, Mutex::_no_safepoint_check_flag);
2365
a181f3a124dd 6987703: iCMS: Intermittent hang with gc/gctests/CallGC/CallGC01 and +ExplicitGCInvokesConcurrent
ysr
parents: 1972
diff changeset
219 assert(_full_gc_count_before <= gch->total_full_collections(), "Error");
0
a61af66fc99e Initial load
duke
parents:
diff changeset
220 if (gch->total_full_collections() == _full_gc_count_before) {
2365
a181f3a124dd 6987703: iCMS: Intermittent hang with gc/gctests/CallGC/CallGC01 and +ExplicitGCInvokesConcurrent
ysr
parents: 1972
diff changeset
221 // Disable iCMS until the full collection is done, and
a181f3a124dd 6987703: iCMS: Intermittent hang with gc/gctests/CallGC/CallGC01 and +ExplicitGCInvokesConcurrent
ysr
parents: 1972
diff changeset
222 // remember that we did so.
0
a61af66fc99e Initial load
duke
parents:
diff changeset
223 CMSCollector::disable_icms();
2365
a181f3a124dd 6987703: iCMS: Intermittent hang with gc/gctests/CallGC/CallGC01 and +ExplicitGCInvokesConcurrent
ysr
parents: 1972
diff changeset
224 _disabled_icms = true;
0
a61af66fc99e Initial load
duke
parents:
diff changeset
225 // In case CMS thread was in icms_wait(), wake it up.
a61af66fc99e Initial load
duke
parents:
diff changeset
226 CMSCollector::start_icms();
1520
bb843ebc7c55 6919638: CMS: ExplicitGCInvokesConcurrent misinteracts with gc locker
ysr
parents: 0
diff changeset
227 // Nudge the CMS thread to start a concurrent collection.
0
a61af66fc99e Initial load
duke
parents:
diff changeset
228 CMSCollector::request_full_gc(_full_gc_count_before);
a61af66fc99e Initial load
duke
parents:
diff changeset
229 } else {
2365
a181f3a124dd 6987703: iCMS: Intermittent hang with gc/gctests/CallGC/CallGC01 and +ExplicitGCInvokesConcurrent
ysr
parents: 1972
diff changeset
230 assert(_full_gc_count_before < gch->total_full_collections(), "Error");
0
a61af66fc99e Initial load
duke
parents:
diff changeset
231 FullGCCount_lock->notify_all(); // Inform the Java thread its work is done
a61af66fc99e Initial load
duke
parents:
diff changeset
232 }
a61af66fc99e Initial load
duke
parents:
diff changeset
233 }
a61af66fc99e Initial load
duke
parents:
diff changeset
234
a61af66fc99e Initial load
duke
parents:
diff changeset
235 bool VM_GenCollectFullConcurrent::evaluate_at_safepoint() const {
a61af66fc99e Initial load
duke
parents:
diff changeset
236 Thread* thr = Thread::current();
a61af66fc99e Initial load
duke
parents:
diff changeset
237 assert(thr != NULL, "Unexpected tid");
a61af66fc99e Initial load
duke
parents:
diff changeset
238 if (!thr->is_Java_thread()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
239 assert(thr->is_VM_thread(), "Expected to be evaluated by VM thread");
a61af66fc99e Initial load
duke
parents:
diff changeset
240 GenCollectedHeap* gch = GenCollectedHeap::heap();
a61af66fc99e Initial load
duke
parents:
diff changeset
241 if (_gc_count_before != gch->total_collections()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
242 // No need to do a young gc, we'll just nudge the CMS thread
a61af66fc99e Initial load
duke
parents:
diff changeset
243 // in the doit() method above, to be executed soon.
a61af66fc99e Initial load
duke
parents:
diff changeset
244 assert(_gc_count_before < gch->total_collections(),
a61af66fc99e Initial load
duke
parents:
diff changeset
245 "total_collections() should be monotnically increasing");
a61af66fc99e Initial load
duke
parents:
diff changeset
246 return false; // no need for foreground young gc
a61af66fc99e Initial load
duke
parents:
diff changeset
247 }
a61af66fc99e Initial load
duke
parents:
diff changeset
248 }
a61af66fc99e Initial load
duke
parents:
diff changeset
249 return true; // may still need foreground young gc
a61af66fc99e Initial load
duke
parents:
diff changeset
250 }
a61af66fc99e Initial load
duke
parents:
diff changeset
251
a61af66fc99e Initial load
duke
parents:
diff changeset
252
a61af66fc99e Initial load
duke
parents:
diff changeset
253 void VM_GenCollectFullConcurrent::doit_epilogue() {
a61af66fc99e Initial load
duke
parents:
diff changeset
254 Thread* thr = Thread::current();
a61af66fc99e Initial load
duke
parents:
diff changeset
255 assert(thr->is_Java_thread(), "just checking");
a61af66fc99e Initial load
duke
parents:
diff changeset
256 JavaThread* jt = (JavaThread*)thr;
a61af66fc99e Initial load
duke
parents:
diff changeset
257 // Release the Heap_lock first.
a61af66fc99e Initial load
duke
parents:
diff changeset
258 Heap_lock->unlock();
a61af66fc99e Initial load
duke
parents:
diff changeset
259 release_and_notify_pending_list_lock();
a61af66fc99e Initial load
duke
parents:
diff changeset
260
a61af66fc99e Initial load
duke
parents:
diff changeset
261 // It is fine to test whether completed collections has
a61af66fc99e Initial load
duke
parents:
diff changeset
262 // exceeded our request count without locking because
a61af66fc99e Initial load
duke
parents:
diff changeset
263 // the completion count is monotonically increasing;
a61af66fc99e Initial load
duke
parents:
diff changeset
264 // this will break for very long-running apps when the
a61af66fc99e Initial load
duke
parents:
diff changeset
265 // count overflows and wraps around. XXX fix me !!!
a61af66fc99e Initial load
duke
parents:
diff changeset
266 // e.g. at the rate of 1 full gc per ms, this could
a61af66fc99e Initial load
duke
parents:
diff changeset
267 // overflow in about 1000 years.
a61af66fc99e Initial load
duke
parents:
diff changeset
268 GenCollectedHeap* gch = GenCollectedHeap::heap();
1520
bb843ebc7c55 6919638: CMS: ExplicitGCInvokesConcurrent misinteracts with gc locker
ysr
parents: 0
diff changeset
269 if (_gc_cause != GCCause::_gc_locker &&
bb843ebc7c55 6919638: CMS: ExplicitGCInvokesConcurrent misinteracts with gc locker
ysr
parents: 0
diff changeset
270 gch->total_full_collections_completed() <= _full_gc_count_before) {
1656
4e5661ba9d98 6944166: G1: explicit GCs are not always handled correctly
tonyp
parents: 1552
diff changeset
271 // maybe we should change the condition to test _gc_cause ==
4e5661ba9d98 6944166: G1: explicit GCs are not always handled correctly
tonyp
parents: 1552
diff changeset
272 // GCCause::_java_lang_system_gc, instead of
4e5661ba9d98 6944166: G1: explicit GCs are not always handled correctly
tonyp
parents: 1552
diff changeset
273 // _gc_cause != GCCause::_gc_locker
4e5661ba9d98 6944166: G1: explicit GCs are not always handled correctly
tonyp
parents: 1552
diff changeset
274 assert(_gc_cause == GCCause::_java_lang_system_gc,
4e5661ba9d98 6944166: G1: explicit GCs are not always handled correctly
tonyp
parents: 1552
diff changeset
275 "the only way to get here if this was a System.gc()-induced GC");
1520
bb843ebc7c55 6919638: CMS: ExplicitGCInvokesConcurrent misinteracts with gc locker
ysr
parents: 0
diff changeset
276 assert(ExplicitGCInvokesConcurrent, "Error");
0
a61af66fc99e Initial load
duke
parents:
diff changeset
277 // Now, wait for witnessing concurrent gc cycle to complete,
a61af66fc99e Initial load
duke
parents:
diff changeset
278 // but do so in native mode, because we want to lock the
a61af66fc99e Initial load
duke
parents:
diff changeset
279 // FullGCEvent_lock, which may be needed by the VM thread
a61af66fc99e Initial load
duke
parents:
diff changeset
280 // or by the CMS thread, so we do not want to be suspended
a61af66fc99e Initial load
duke
parents:
diff changeset
281 // while holding that lock.
a61af66fc99e Initial load
duke
parents:
diff changeset
282 ThreadToNativeFromVM native(jt);
a61af66fc99e Initial load
duke
parents:
diff changeset
283 MutexLockerEx ml(FullGCCount_lock, Mutex::_no_safepoint_check_flag);
a61af66fc99e Initial load
duke
parents:
diff changeset
284 // Either a concurrent or a stop-world full gc is sufficient
a61af66fc99e Initial load
duke
parents:
diff changeset
285 // witness to our request.
a61af66fc99e Initial load
duke
parents:
diff changeset
286 while (gch->total_full_collections_completed() <= _full_gc_count_before) {
a61af66fc99e Initial load
duke
parents:
diff changeset
287 FullGCCount_lock->wait(Mutex::_no_safepoint_check_flag);
a61af66fc99e Initial load
duke
parents:
diff changeset
288 }
a61af66fc99e Initial load
duke
parents:
diff changeset
289 }
2365
a181f3a124dd 6987703: iCMS: Intermittent hang with gc/gctests/CallGC/CallGC01 and +ExplicitGCInvokesConcurrent
ysr
parents: 1972
diff changeset
290 // Enable iCMS back if we disabled it earlier.
a181f3a124dd 6987703: iCMS: Intermittent hang with gc/gctests/CallGC/CallGC01 and +ExplicitGCInvokesConcurrent
ysr
parents: 1972
diff changeset
291 if (_disabled_icms) {
a181f3a124dd 6987703: iCMS: Intermittent hang with gc/gctests/CallGC/CallGC01 and +ExplicitGCInvokesConcurrent
ysr
parents: 1972
diff changeset
292 CMSCollector::enable_icms();
a181f3a124dd 6987703: iCMS: Intermittent hang with gc/gctests/CallGC/CallGC01 and +ExplicitGCInvokesConcurrent
ysr
parents: 1972
diff changeset
293 }
0
a61af66fc99e Initial load
duke
parents:
diff changeset
294 }