annotate src/share/vm/gc_implementation/concurrentMarkSweep/vmCMSOperations.cpp @ 14380:e6195383bcaf

8034080: Remove the USDT1 dtrace code from Hotspot Reviewed-by: dholmes, coleenp
author sla
date Tue, 18 Feb 2014 09:15:40 +0100
parents e5d78f318aec
children 4ca6dc0799b6
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
rev   line source
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1 /*
10405
f2110083203d 8005849: JEP 167: Event-Based JVM Tracing
sla
parents: 6866
diff changeset
2 * Copyright (c) 2005, 2013, 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"
10405
f2110083203d 8005849: JEP 167: Event-Based JVM Tracing
sla
parents: 6866
diff changeset
29 #include "gc_implementation/shared/gcTimer.hpp"
f2110083203d 8005849: JEP 167: Event-Based JVM Tracing
sla
parents: 6866
diff changeset
30 #include "gc_implementation/shared/gcTraceTime.hpp"
1972
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1656
diff changeset
31 #include "gc_implementation/shared/isGCActiveMark.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1656
diff changeset
32 #include "memory/gcLocker.inline.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1656
diff changeset
33 #include "runtime/interfaceSupport.hpp"
10405
f2110083203d 8005849: JEP 167: Event-Based JVM Tracing
sla
parents: 6866
diff changeset
34 #include "runtime/os.hpp"
1972
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1656
diff changeset
35 #include "utilities/dtrace.hpp"
4006
436b4a3231bf 7098194: integrate macosx-port changes
dcubed
parents: 2365
diff changeset
36
436b4a3231bf 7098194: integrate macosx-port changes
dcubed
parents: 2365
diff changeset
37
0
a61af66fc99e Initial load
duke
parents:
diff changeset
38 //////////////////////////////////////////////////////////
a61af66fc99e Initial load
duke
parents:
diff changeset
39 // Methods in abstract class VM_CMS_Operation
a61af66fc99e Initial load
duke
parents:
diff changeset
40 //////////////////////////////////////////////////////////
a61af66fc99e Initial load
duke
parents:
diff changeset
41 void VM_CMS_Operation::acquire_pending_list_lock() {
a61af66fc99e Initial load
duke
parents:
diff changeset
42 // The caller may block while communicating
a61af66fc99e Initial load
duke
parents:
diff changeset
43 // with the SLT thread in order to acquire/release the PLL.
a61af66fc99e Initial load
duke
parents:
diff changeset
44 ConcurrentMarkSweepThread::slt()->
a61af66fc99e Initial load
duke
parents:
diff changeset
45 manipulatePLL(SurrogateLockerThread::acquirePLL);
a61af66fc99e Initial load
duke
parents:
diff changeset
46 }
a61af66fc99e Initial load
duke
parents:
diff changeset
47
a61af66fc99e Initial load
duke
parents:
diff changeset
48 void VM_CMS_Operation::release_and_notify_pending_list_lock() {
a61af66fc99e Initial load
duke
parents:
diff changeset
49 // The caller may block while communicating
a61af66fc99e Initial load
duke
parents:
diff changeset
50 // with the SLT thread in order to acquire/release the PLL.
a61af66fc99e Initial load
duke
parents:
diff changeset
51 ConcurrentMarkSweepThread::slt()->
a61af66fc99e Initial load
duke
parents:
diff changeset
52 manipulatePLL(SurrogateLockerThread::releaseAndNotifyPLL);
a61af66fc99e Initial load
duke
parents:
diff changeset
53 }
a61af66fc99e Initial load
duke
parents:
diff changeset
54
a61af66fc99e Initial load
duke
parents:
diff changeset
55 void VM_CMS_Operation::verify_before_gc() {
a61af66fc99e Initial load
duke
parents:
diff changeset
56 if (VerifyBeforeGC &&
a61af66fc99e Initial load
duke
parents:
diff changeset
57 GenCollectedHeap::heap()->total_collections() >= VerifyGCStartAt) {
10405
f2110083203d 8005849: JEP 167: Event-Based JVM Tracing
sla
parents: 6866
diff changeset
58 GCTraceTime tm("Verify Before", false, false, _collector->_gc_timer_cm);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
59 HandleMark hm;
a61af66fc99e Initial load
duke
parents:
diff changeset
60 FreelistLocker x(_collector);
a61af66fc99e Initial load
duke
parents:
diff changeset
61 MutexLockerEx y(_collector->bitMapLock(), Mutex::_no_safepoint_check_flag);
a61af66fc99e Initial load
duke
parents:
diff changeset
62 Universe::heap()->prepare_for_verify();
6865
4202510ee0fe 8000831: Heap verification output incorrect/incomplete
johnc
parents: 6064
diff changeset
63 Universe::verify();
0
a61af66fc99e Initial load
duke
parents:
diff changeset
64 }
a61af66fc99e Initial load
duke
parents:
diff changeset
65 }
a61af66fc99e Initial load
duke
parents:
diff changeset
66
a61af66fc99e Initial load
duke
parents:
diff changeset
67 void VM_CMS_Operation::verify_after_gc() {
a61af66fc99e Initial load
duke
parents:
diff changeset
68 if (VerifyAfterGC &&
a61af66fc99e Initial load
duke
parents:
diff changeset
69 GenCollectedHeap::heap()->total_collections() >= VerifyGCStartAt) {
10405
f2110083203d 8005849: JEP 167: Event-Based JVM Tracing
sla
parents: 6866
diff changeset
70 GCTraceTime tm("Verify After", false, false, _collector->_gc_timer_cm);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
71 HandleMark hm;
a61af66fc99e Initial load
duke
parents:
diff changeset
72 FreelistLocker x(_collector);
a61af66fc99e Initial load
duke
parents:
diff changeset
73 MutexLockerEx y(_collector->bitMapLock(), Mutex::_no_safepoint_check_flag);
6865
4202510ee0fe 8000831: Heap verification output incorrect/incomplete
johnc
parents: 6064
diff changeset
74 Universe::verify();
0
a61af66fc99e Initial load
duke
parents:
diff changeset
75 }
a61af66fc99e Initial load
duke
parents:
diff changeset
76 }
a61af66fc99e Initial load
duke
parents:
diff changeset
77
a61af66fc99e Initial load
duke
parents:
diff changeset
78 bool VM_CMS_Operation::lost_race() const {
a61af66fc99e Initial load
duke
parents:
diff changeset
79 if (CMSCollector::abstract_state() == CMSCollector::Idling) {
a61af66fc99e Initial load
duke
parents:
diff changeset
80 // We lost a race to a foreground collection
a61af66fc99e Initial load
duke
parents:
diff changeset
81 // -- there's nothing to do
a61af66fc99e Initial load
duke
parents:
diff changeset
82 return true;
a61af66fc99e Initial load
duke
parents:
diff changeset
83 }
a61af66fc99e Initial load
duke
parents:
diff changeset
84 assert(CMSCollector::abstract_state() == legal_state(),
a61af66fc99e Initial load
duke
parents:
diff changeset
85 "Inconsistent collector state?");
a61af66fc99e Initial load
duke
parents:
diff changeset
86 return false;
a61af66fc99e Initial load
duke
parents:
diff changeset
87 }
a61af66fc99e Initial load
duke
parents:
diff changeset
88
a61af66fc99e Initial load
duke
parents:
diff changeset
89 bool VM_CMS_Operation::doit_prologue() {
a61af66fc99e Initial load
duke
parents:
diff changeset
90 assert(Thread::current()->is_ConcurrentGC_thread(), "just checking");
a61af66fc99e Initial load
duke
parents:
diff changeset
91 assert(!CMSCollector::foregroundGCShouldWait(), "Possible deadlock");
a61af66fc99e Initial load
duke
parents:
diff changeset
92 assert(!ConcurrentMarkSweepThread::cms_thread_has_cms_token(),
a61af66fc99e Initial load
duke
parents:
diff changeset
93 "Possible deadlock");
a61af66fc99e Initial load
duke
parents:
diff changeset
94
a61af66fc99e Initial load
duke
parents:
diff changeset
95 if (needs_pll()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
96 acquire_pending_list_lock();
a61af66fc99e Initial load
duke
parents:
diff changeset
97 }
a61af66fc99e Initial load
duke
parents:
diff changeset
98 // Get the Heap_lock after the pending_list_lock.
a61af66fc99e Initial load
duke
parents:
diff changeset
99 Heap_lock->lock();
a61af66fc99e Initial load
duke
parents:
diff changeset
100 if (lost_race()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
101 assert(_prologue_succeeded == false, "Initialized in c'tor");
a61af66fc99e Initial load
duke
parents:
diff changeset
102 Heap_lock->unlock();
a61af66fc99e Initial load
duke
parents:
diff changeset
103 if (needs_pll()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
104 release_and_notify_pending_list_lock();
a61af66fc99e Initial load
duke
parents:
diff changeset
105 }
a61af66fc99e Initial load
duke
parents:
diff changeset
106 } else {
a61af66fc99e Initial load
duke
parents:
diff changeset
107 _prologue_succeeded = true;
a61af66fc99e Initial load
duke
parents:
diff changeset
108 }
a61af66fc99e Initial load
duke
parents:
diff changeset
109 return _prologue_succeeded;
a61af66fc99e Initial load
duke
parents:
diff changeset
110 }
a61af66fc99e Initial load
duke
parents:
diff changeset
111
a61af66fc99e Initial load
duke
parents:
diff changeset
112 void VM_CMS_Operation::doit_epilogue() {
a61af66fc99e Initial load
duke
parents:
diff changeset
113 assert(Thread::current()->is_ConcurrentGC_thread(), "just checking");
a61af66fc99e Initial load
duke
parents:
diff changeset
114 assert(!CMSCollector::foregroundGCShouldWait(), "Possible deadlock");
a61af66fc99e Initial load
duke
parents:
diff changeset
115 assert(!ConcurrentMarkSweepThread::cms_thread_has_cms_token(),
a61af66fc99e Initial load
duke
parents:
diff changeset
116 "Possible deadlock");
a61af66fc99e Initial load
duke
parents:
diff changeset
117
a61af66fc99e Initial load
duke
parents:
diff changeset
118 // Release the Heap_lock first.
a61af66fc99e Initial load
duke
parents:
diff changeset
119 Heap_lock->unlock();
a61af66fc99e Initial load
duke
parents:
diff changeset
120 if (needs_pll()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
121 release_and_notify_pending_list_lock();
a61af66fc99e Initial load
duke
parents:
diff changeset
122 }
a61af66fc99e Initial load
duke
parents:
diff changeset
123 }
a61af66fc99e Initial load
duke
parents:
diff changeset
124
a61af66fc99e Initial load
duke
parents:
diff changeset
125 //////////////////////////////////////////////////////////
a61af66fc99e Initial load
duke
parents:
diff changeset
126 // Methods in class VM_CMS_Initial_Mark
a61af66fc99e Initial load
duke
parents:
diff changeset
127 //////////////////////////////////////////////////////////
a61af66fc99e Initial load
duke
parents:
diff changeset
128 void VM_CMS_Initial_Mark::doit() {
a61af66fc99e Initial load
duke
parents:
diff changeset
129 if (lost_race()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
130 // Nothing to do.
a61af66fc99e Initial load
duke
parents:
diff changeset
131 return;
a61af66fc99e Initial load
duke
parents:
diff changeset
132 }
14278
a3ba776d6ab6 8030812: Change the solaris DTrace implementation to use USDT2 instead of USDT1
sla
parents: 13400
diff changeset
133 HS_PRIVATE_CMS_INITMARK_BEGIN();
0
a61af66fc99e Initial load
duke
parents:
diff changeset
134
13400
86e6d691f2e1 8028128: Add a type safe alternative for working with counter based data
mgronlun
parents: 10405
diff changeset
135 _collector->_gc_timer_cm->register_gc_pause_start("Initial Mark");
10405
f2110083203d 8005849: JEP 167: Event-Based JVM Tracing
sla
parents: 6866
diff changeset
136
0
a61af66fc99e Initial load
duke
parents:
diff changeset
137 GenCollectedHeap* gch = GenCollectedHeap::heap();
a61af66fc99e Initial load
duke
parents:
diff changeset
138 GCCauseSetter gccs(gch, GCCause::_cms_initial_mark);
a61af66fc99e Initial load
duke
parents:
diff changeset
139
a61af66fc99e Initial load
duke
parents:
diff changeset
140 VM_CMS_Operation::verify_before_gc();
a61af66fc99e Initial load
duke
parents:
diff changeset
141
a61af66fc99e Initial load
duke
parents:
diff changeset
142 IsGCActiveMark x; // stop-world GC active
6064
9d679effd28c 7166894: Add gc cause to GC logging for all collectors
brutisso
parents: 4006
diff changeset
143 _collector->do_CMS_operation(CMSCollector::CMS_op_checkpointRootsInitial, gch->gc_cause());
0
a61af66fc99e Initial load
duke
parents:
diff changeset
144
a61af66fc99e Initial load
duke
parents:
diff changeset
145 VM_CMS_Operation::verify_after_gc();
10405
f2110083203d 8005849: JEP 167: Event-Based JVM Tracing
sla
parents: 6866
diff changeset
146
13400
86e6d691f2e1 8028128: Add a type safe alternative for working with counter based data
mgronlun
parents: 10405
diff changeset
147 _collector->_gc_timer_cm->register_gc_pause_end();
10405
f2110083203d 8005849: JEP 167: Event-Based JVM Tracing
sla
parents: 6866
diff changeset
148
14278
a3ba776d6ab6 8030812: Change the solaris DTrace implementation to use USDT2 instead of USDT1
sla
parents: 13400
diff changeset
149 HS_PRIVATE_CMS_INITMARK_END();
0
a61af66fc99e Initial load
duke
parents:
diff changeset
150 }
a61af66fc99e Initial load
duke
parents:
diff changeset
151
a61af66fc99e Initial load
duke
parents:
diff changeset
152 //////////////////////////////////////////////////////////
a61af66fc99e Initial load
duke
parents:
diff changeset
153 // Methods in class VM_CMS_Final_Remark_Operation
a61af66fc99e Initial load
duke
parents:
diff changeset
154 //////////////////////////////////////////////////////////
a61af66fc99e Initial load
duke
parents:
diff changeset
155 void VM_CMS_Final_Remark::doit() {
a61af66fc99e Initial load
duke
parents:
diff changeset
156 if (lost_race()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
157 // Nothing to do.
a61af66fc99e Initial load
duke
parents:
diff changeset
158 return;
a61af66fc99e Initial load
duke
parents:
diff changeset
159 }
14278
a3ba776d6ab6 8030812: Change the solaris DTrace implementation to use USDT2 instead of USDT1
sla
parents: 13400
diff changeset
160 HS_PRIVATE_CMS_REMARK_BEGIN();
0
a61af66fc99e Initial load
duke
parents:
diff changeset
161
13400
86e6d691f2e1 8028128: Add a type safe alternative for working with counter based data
mgronlun
parents: 10405
diff changeset
162 _collector->_gc_timer_cm->register_gc_pause_start("Final Mark");
10405
f2110083203d 8005849: JEP 167: Event-Based JVM Tracing
sla
parents: 6866
diff changeset
163
0
a61af66fc99e Initial load
duke
parents:
diff changeset
164 GenCollectedHeap* gch = GenCollectedHeap::heap();
a61af66fc99e Initial load
duke
parents:
diff changeset
165 GCCauseSetter gccs(gch, GCCause::_cms_final_remark);
a61af66fc99e Initial load
duke
parents:
diff changeset
166
a61af66fc99e Initial load
duke
parents:
diff changeset
167 VM_CMS_Operation::verify_before_gc();
a61af66fc99e Initial load
duke
parents:
diff changeset
168
a61af66fc99e Initial load
duke
parents:
diff changeset
169 IsGCActiveMark x; // stop-world GC active
6064
9d679effd28c 7166894: Add gc cause to GC logging for all collectors
brutisso
parents: 4006
diff changeset
170 _collector->do_CMS_operation(CMSCollector::CMS_op_checkpointRootsFinal, gch->gc_cause());
0
a61af66fc99e Initial load
duke
parents:
diff changeset
171
a61af66fc99e Initial load
duke
parents:
diff changeset
172 VM_CMS_Operation::verify_after_gc();
10405
f2110083203d 8005849: JEP 167: Event-Based JVM Tracing
sla
parents: 6866
diff changeset
173
f2110083203d 8005849: JEP 167: Event-Based JVM Tracing
sla
parents: 6866
diff changeset
174 _collector->save_heap_summary();
13400
86e6d691f2e1 8028128: Add a type safe alternative for working with counter based data
mgronlun
parents: 10405
diff changeset
175 _collector->_gc_timer_cm->register_gc_pause_end();
10405
f2110083203d 8005849: JEP 167: Event-Based JVM Tracing
sla
parents: 6866
diff changeset
176
14278
a3ba776d6ab6 8030812: Change the solaris DTrace implementation to use USDT2 instead of USDT1
sla
parents: 13400
diff changeset
177 HS_PRIVATE_CMS_REMARK_END();
0
a61af66fc99e Initial load
duke
parents:
diff changeset
178 }
a61af66fc99e Initial load
duke
parents:
diff changeset
179
a61af66fc99e Initial load
duke
parents:
diff changeset
180 // VM operation to invoke a concurrent collection of a
a61af66fc99e Initial load
duke
parents:
diff changeset
181 // GenCollectedHeap heap.
a61af66fc99e Initial load
duke
parents:
diff changeset
182 void VM_GenCollectFullConcurrent::doit() {
a61af66fc99e Initial load
duke
parents:
diff changeset
183 assert(Thread::current()->is_VM_thread(), "Should be VM thread");
1520
bb843ebc7c55 6919638: CMS: ExplicitGCInvokesConcurrent misinteracts with gc locker
ysr
parents: 0
diff changeset
184 assert(GCLockerInvokesConcurrent || ExplicitGCInvokesConcurrent, "Unexpected");
0
a61af66fc99e Initial load
duke
parents:
diff changeset
185
a61af66fc99e Initial load
duke
parents:
diff changeset
186 GenCollectedHeap* gch = GenCollectedHeap::heap();
a61af66fc99e Initial load
duke
parents:
diff changeset
187 if (_gc_count_before == gch->total_collections()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
188 // The "full" of do_full_collection call below "forces"
a61af66fc99e Initial load
duke
parents:
diff changeset
189 // a collection; the second arg, 0, below ensures that
a61af66fc99e Initial load
duke
parents:
diff changeset
190 // only the young gen is collected. XXX In the future,
a61af66fc99e Initial load
duke
parents:
diff changeset
191 // we'll probably need to have something in this interface
a61af66fc99e Initial load
duke
parents:
diff changeset
192 // to say do this only if we are sure we will not bail
a61af66fc99e Initial load
duke
parents:
diff changeset
193 // out to a full collection in this attempt, but that's
a61af66fc99e Initial load
duke
parents:
diff changeset
194 // for the future.
a61af66fc99e Initial load
duke
parents:
diff changeset
195 assert(SafepointSynchronize::is_at_safepoint(),
a61af66fc99e Initial load
duke
parents:
diff changeset
196 "We can only be executing this arm of if at a safepoint");
a61af66fc99e Initial load
duke
parents:
diff changeset
197 GCCauseSetter gccs(gch, _gc_cause);
a61af66fc99e Initial load
duke
parents:
diff changeset
198 gch->do_full_collection(gch->must_clear_all_soft_refs(),
a61af66fc99e Initial load
duke
parents:
diff changeset
199 0 /* collect only youngest gen */);
a61af66fc99e Initial load
duke
parents:
diff changeset
200 } // Else no need for a foreground young gc
a61af66fc99e Initial load
duke
parents:
diff changeset
201 assert((_gc_count_before < gch->total_collections()) ||
a61af66fc99e Initial load
duke
parents:
diff changeset
202 (GC_locker::is_active() /* gc may have been skipped */
a61af66fc99e Initial load
duke
parents:
diff changeset
203 && (_gc_count_before == gch->total_collections())),
a61af66fc99e Initial load
duke
parents:
diff changeset
204 "total_collections() should be monotonically increasing");
a61af66fc99e Initial load
duke
parents:
diff changeset
205
a61af66fc99e Initial load
duke
parents:
diff changeset
206 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
207 assert(_full_gc_count_before <= gch->total_full_collections(), "Error");
0
a61af66fc99e Initial load
duke
parents:
diff changeset
208 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
209 // 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
210 // remember that we did so.
0
a61af66fc99e Initial load
duke
parents:
diff changeset
211 CMSCollector::disable_icms();
2365
a181f3a124dd 6987703: iCMS: Intermittent hang with gc/gctests/CallGC/CallGC01 and +ExplicitGCInvokesConcurrent
ysr
parents: 1972
diff changeset
212 _disabled_icms = true;
0
a61af66fc99e Initial load
duke
parents:
diff changeset
213 // In case CMS thread was in icms_wait(), wake it up.
a61af66fc99e Initial load
duke
parents:
diff changeset
214 CMSCollector::start_icms();
1520
bb843ebc7c55 6919638: CMS: ExplicitGCInvokesConcurrent misinteracts with gc locker
ysr
parents: 0
diff changeset
215 // Nudge the CMS thread to start a concurrent collection.
10405
f2110083203d 8005849: JEP 167: Event-Based JVM Tracing
sla
parents: 6866
diff changeset
216 CMSCollector::request_full_gc(_full_gc_count_before, _gc_cause);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
217 } else {
2365
a181f3a124dd 6987703: iCMS: Intermittent hang with gc/gctests/CallGC/CallGC01 and +ExplicitGCInvokesConcurrent
ysr
parents: 1972
diff changeset
218 assert(_full_gc_count_before < gch->total_full_collections(), "Error");
0
a61af66fc99e Initial load
duke
parents:
diff changeset
219 FullGCCount_lock->notify_all(); // Inform the Java thread its work is done
a61af66fc99e Initial load
duke
parents:
diff changeset
220 }
a61af66fc99e Initial load
duke
parents:
diff changeset
221 }
a61af66fc99e Initial load
duke
parents:
diff changeset
222
a61af66fc99e Initial load
duke
parents:
diff changeset
223 bool VM_GenCollectFullConcurrent::evaluate_at_safepoint() const {
a61af66fc99e Initial load
duke
parents:
diff changeset
224 Thread* thr = Thread::current();
a61af66fc99e Initial load
duke
parents:
diff changeset
225 assert(thr != NULL, "Unexpected tid");
a61af66fc99e Initial load
duke
parents:
diff changeset
226 if (!thr->is_Java_thread()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
227 assert(thr->is_VM_thread(), "Expected to be evaluated by VM thread");
a61af66fc99e Initial load
duke
parents:
diff changeset
228 GenCollectedHeap* gch = GenCollectedHeap::heap();
a61af66fc99e Initial load
duke
parents:
diff changeset
229 if (_gc_count_before != gch->total_collections()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
230 // No need to do a young gc, we'll just nudge the CMS thread
a61af66fc99e Initial load
duke
parents:
diff changeset
231 // in the doit() method above, to be executed soon.
a61af66fc99e Initial load
duke
parents:
diff changeset
232 assert(_gc_count_before < gch->total_collections(),
14358
e5d78f318aec 8026849: Fix typos in the GC code, part 2
jwilhelm
parents: 14278
diff changeset
233 "total_collections() should be monotonically increasing");
0
a61af66fc99e Initial load
duke
parents:
diff changeset
234 return false; // no need for foreground young gc
a61af66fc99e Initial load
duke
parents:
diff changeset
235 }
a61af66fc99e Initial load
duke
parents:
diff changeset
236 }
a61af66fc99e Initial load
duke
parents:
diff changeset
237 return true; // may still need foreground young gc
a61af66fc99e Initial load
duke
parents:
diff changeset
238 }
a61af66fc99e Initial load
duke
parents:
diff changeset
239
a61af66fc99e Initial load
duke
parents:
diff changeset
240
a61af66fc99e Initial load
duke
parents:
diff changeset
241 void VM_GenCollectFullConcurrent::doit_epilogue() {
a61af66fc99e Initial load
duke
parents:
diff changeset
242 Thread* thr = Thread::current();
a61af66fc99e Initial load
duke
parents:
diff changeset
243 assert(thr->is_Java_thread(), "just checking");
a61af66fc99e Initial load
duke
parents:
diff changeset
244 JavaThread* jt = (JavaThread*)thr;
a61af66fc99e Initial load
duke
parents:
diff changeset
245 // Release the Heap_lock first.
a61af66fc99e Initial load
duke
parents:
diff changeset
246 Heap_lock->unlock();
a61af66fc99e Initial load
duke
parents:
diff changeset
247 release_and_notify_pending_list_lock();
a61af66fc99e Initial load
duke
parents:
diff changeset
248
a61af66fc99e Initial load
duke
parents:
diff changeset
249 // It is fine to test whether completed collections has
a61af66fc99e Initial load
duke
parents:
diff changeset
250 // exceeded our request count without locking because
a61af66fc99e Initial load
duke
parents:
diff changeset
251 // the completion count is monotonically increasing;
a61af66fc99e Initial load
duke
parents:
diff changeset
252 // this will break for very long-running apps when the
a61af66fc99e Initial load
duke
parents:
diff changeset
253 // count overflows and wraps around. XXX fix me !!!
a61af66fc99e Initial load
duke
parents:
diff changeset
254 // e.g. at the rate of 1 full gc per ms, this could
a61af66fc99e Initial load
duke
parents:
diff changeset
255 // overflow in about 1000 years.
a61af66fc99e Initial load
duke
parents:
diff changeset
256 GenCollectedHeap* gch = GenCollectedHeap::heap();
1520
bb843ebc7c55 6919638: CMS: ExplicitGCInvokesConcurrent misinteracts with gc locker
ysr
parents: 0
diff changeset
257 if (_gc_cause != GCCause::_gc_locker &&
bb843ebc7c55 6919638: CMS: ExplicitGCInvokesConcurrent misinteracts with gc locker
ysr
parents: 0
diff changeset
258 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
259 // 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
260 // GCCause::_java_lang_system_gc, instead of
4e5661ba9d98 6944166: G1: explicit GCs are not always handled correctly
tonyp
parents: 1552
diff changeset
261 // _gc_cause != GCCause::_gc_locker
4e5661ba9d98 6944166: G1: explicit GCs are not always handled correctly
tonyp
parents: 1552
diff changeset
262 assert(_gc_cause == GCCause::_java_lang_system_gc,
4e5661ba9d98 6944166: G1: explicit GCs are not always handled correctly
tonyp
parents: 1552
diff changeset
263 "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
264 assert(ExplicitGCInvokesConcurrent, "Error");
0
a61af66fc99e Initial load
duke
parents:
diff changeset
265 // Now, wait for witnessing concurrent gc cycle to complete,
a61af66fc99e Initial load
duke
parents:
diff changeset
266 // but do so in native mode, because we want to lock the
a61af66fc99e Initial load
duke
parents:
diff changeset
267 // FullGCEvent_lock, which may be needed by the VM thread
a61af66fc99e Initial load
duke
parents:
diff changeset
268 // or by the CMS thread, so we do not want to be suspended
a61af66fc99e Initial load
duke
parents:
diff changeset
269 // while holding that lock.
a61af66fc99e Initial load
duke
parents:
diff changeset
270 ThreadToNativeFromVM native(jt);
a61af66fc99e Initial load
duke
parents:
diff changeset
271 MutexLockerEx ml(FullGCCount_lock, Mutex::_no_safepoint_check_flag);
a61af66fc99e Initial load
duke
parents:
diff changeset
272 // Either a concurrent or a stop-world full gc is sufficient
a61af66fc99e Initial load
duke
parents:
diff changeset
273 // witness to our request.
a61af66fc99e Initial load
duke
parents:
diff changeset
274 while (gch->total_full_collections_completed() <= _full_gc_count_before) {
a61af66fc99e Initial load
duke
parents:
diff changeset
275 FullGCCount_lock->wait(Mutex::_no_safepoint_check_flag);
a61af66fc99e Initial load
duke
parents:
diff changeset
276 }
a61af66fc99e Initial load
duke
parents:
diff changeset
277 }
2365
a181f3a124dd 6987703: iCMS: Intermittent hang with gc/gctests/CallGC/CallGC01 and +ExplicitGCInvokesConcurrent
ysr
parents: 1972
diff changeset
278 // 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
279 if (_disabled_icms) {
a181f3a124dd 6987703: iCMS: Intermittent hang with gc/gctests/CallGC/CallGC01 and +ExplicitGCInvokesConcurrent
ysr
parents: 1972
diff changeset
280 CMSCollector::enable_icms();
a181f3a124dd 6987703: iCMS: Intermittent hang with gc/gctests/CallGC/CallGC01 and +ExplicitGCInvokesConcurrent
ysr
parents: 1972
diff changeset
281 }
0
a61af66fc99e Initial load
duke
parents:
diff changeset
282 }