src/share/vm/memory/gcLocker.hpp

Wed, 31 Jul 2019 14:28:51 -0400

author
kbarrett
date
Wed, 31 Jul 2019 14:28:51 -0400
changeset 9787
9f28a4cac6d9
parent 6619
9c3dc501b5eb
child 9806
758c07667682
permissions
-rw-r--r--

8048556: Unnecessary GCLocker-initiated young GCs
Summary: Fixed recognition of unnecessary GCLocker collections.
Reviewed-by: pliden, tschatzl
Contributed-by: johnc@azul.com

duke@435 1 /*
mikael@6198 2 * Copyright (c) 1997, 2013, Oracle and/or its affiliates. All rights reserved.
duke@435 3 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
duke@435 4 *
duke@435 5 * This code is free software; you can redistribute it and/or modify it
duke@435 6 * under the terms of the GNU General Public License version 2 only, as
duke@435 7 * published by the Free Software Foundation.
duke@435 8 *
duke@435 9 * This code is distributed in the hope that it will be useful, but WITHOUT
duke@435 10 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
duke@435 11 * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License
duke@435 12 * version 2 for more details (a copy is included in the LICENSE file that
duke@435 13 * accompanied this code).
duke@435 14 *
duke@435 15 * You should have received a copy of the GNU General Public License version
duke@435 16 * 2 along with this work; if not, write to the Free Software Foundation,
duke@435 17 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
duke@435 18 *
trims@1907 19 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
trims@1907 20 * or visit www.oracle.com if you need additional information or have any
trims@1907 21 * questions.
duke@435 22 *
duke@435 23 */
duke@435 24
stefank@2314 25 #ifndef SHARE_VM_MEMORY_GCLOCKER_HPP
stefank@2314 26 #define SHARE_VM_MEMORY_GCLOCKER_HPP
stefank@2314 27
stefank@2314 28 #include "gc_interface/collectedHeap.hpp"
kbarrett@9787 29 #include "gc_interface/gcCause.hpp"
stefank@2314 30 #include "memory/genCollectedHeap.hpp"
stefank@2314 31 #include "memory/universe.hpp"
stefank@2314 32 #include "oops/oop.hpp"
stefank@4299 33 #include "runtime/thread.inline.hpp"
stefank@2314 34 #ifdef TARGET_OS_FAMILY_linux
stefank@2314 35 # include "os_linux.inline.hpp"
stefank@2314 36 #endif
stefank@2314 37 #ifdef TARGET_OS_FAMILY_solaris
stefank@2314 38 # include "os_solaris.inline.hpp"
stefank@2314 39 #endif
stefank@2314 40 #ifdef TARGET_OS_FAMILY_windows
stefank@2314 41 # include "os_windows.inline.hpp"
stefank@2314 42 #endif
never@3156 43 #ifdef TARGET_OS_FAMILY_bsd
never@3156 44 # include "os_bsd.inline.hpp"
never@3156 45 #endif
stefank@2314 46
duke@435 47 // The direct lock/unlock calls do not force a collection if an unlock
duke@435 48 // decrements the count to zero. Avoid calling these if at all possible.
duke@435 49
duke@435 50 class GC_locker: public AllStatic {
duke@435 51 private:
never@3494 52 // The _jni_lock_count keeps track of the number of threads that are
never@3494 53 // currently in a critical region. It's only kept up to date when
never@3494 54 // _needs_gc is true. The current value is computed during
never@3494 55 // safepointing and decremented during the slow path of GC_locker
never@3494 56 // unlocking.
never@3494 57 static volatile jint _jni_lock_count; // number of jni active instances.
duke@435 58 static volatile bool _needs_gc; // heap is filling, we need a GC
duke@435 59 // note: bool is typedef'd as jint
duke@435 60 static volatile bool _doing_gc; // unlock_critical() is doing a GC
kbarrett@9787 61 static uint _total_collections; // value for _gc_locker collection
duke@435 62
never@3494 63 #ifdef ASSERT
never@3494 64 // This lock count is updated for all operations and is used to
never@3494 65 // validate the jni_lock_count that is computed during safepoints.
never@3494 66 static volatile jint _debug_jni_lock_count;
never@3494 67 #endif
never@3494 68
never@3494 69 // At a safepoint, visit all threads and count the number of active
never@3494 70 // critical sections. This is used to ensure that all active
never@3494 71 // critical sections are exited before a new one is started.
never@3494 72 static void verify_critical_count() NOT_DEBUG_RETURN;
duke@435 73
never@3494 74 static void jni_lock(JavaThread* thread);
never@3494 75 static void jni_unlock(JavaThread* thread);
duke@435 76
never@3576 77 static bool is_active_internal() {
never@3576 78 verify_critical_count();
sjohanss@6619 79 return _jni_lock_count > 0;
never@3576 80 }
never@3576 81
duke@435 82 public:
duke@435 83 // Accessors
never@3576 84 static bool is_active() {
stefank@5769 85 assert(SafepointSynchronize::is_at_safepoint(), "only read at safepoint");
never@3576 86 return is_active_internal();
never@3576 87 }
duke@435 88 static bool needs_gc() { return _needs_gc; }
never@3494 89
duke@435 90 // Shorthand
never@3576 91 static bool is_active_and_needs_gc() {
never@3576 92 // Use is_active_internal since _needs_gc can change from true to
never@3576 93 // false outside of a safepoint, triggering the assert in
never@3576 94 // is_active.
never@3576 95 return needs_gc() && is_active_internal();
never@3576 96 }
duke@435 97
never@3494 98 // In debug mode track the locking state at all times
never@3494 99 static void increment_debug_jni_lock_count() {
never@3494 100 #ifdef ASSERT
never@3494 101 assert(_debug_jni_lock_count >= 0, "bad value");
never@3494 102 Atomic::inc(&_debug_jni_lock_count);
never@3494 103 #endif
never@3494 104 }
never@3494 105 static void decrement_debug_jni_lock_count() {
never@3494 106 #ifdef ASSERT
never@3494 107 assert(_debug_jni_lock_count > 0, "bad value");
never@3494 108 Atomic::dec(&_debug_jni_lock_count);
never@3494 109 #endif
never@3494 110 }
never@3494 111
never@3494 112 // Set the current lock count
never@3494 113 static void set_jni_lock_count(int count) {
never@3494 114 _jni_lock_count = count;
never@3494 115 verify_critical_count();
never@3494 116 }
never@3494 117
never@3494 118 // Sets _needs_gc if is_active() is true. Returns is_active().
duke@435 119 static bool check_active_before_gc();
duke@435 120
kbarrett@9787 121 // Return true if the designated collection is a GCLocker request
kbarrett@9787 122 // that should be discarded. Returns true if cause == GCCause::_gc_locker
kbarrett@9787 123 // and the given total collection value indicates a collection has been
kbarrett@9787 124 // done since the GCLocker request was made.
kbarrett@9787 125 static bool should_discard(GCCause::Cause cause, uint total_collections);
kbarrett@9787 126
duke@435 127 // Stalls the caller (who should not be in a jni critical section)
duke@435 128 // until needs_gc() clears. Note however that needs_gc() may be
duke@435 129 // set at a subsequent safepoint and/or cleared under the
duke@435 130 // JNICritical_lock, so the caller may not safely assert upon
duke@435 131 // return from this method that "!needs_gc()" since that is
duke@435 132 // not a stable predicate.
duke@435 133 static void stall_until_clear();
duke@435 134
duke@435 135 // The following two methods are used for JNI critical regions.
duke@435 136 // If we find that we failed to perform a GC because the GC_locker
duke@435 137 // was active, arrange for one as soon as possible by allowing
duke@435 138 // all threads in critical regions to complete, but not allowing
duke@435 139 // other critical regions to be entered. The reasons for that are:
duke@435 140 // 1) a GC request won't be starved by overlapping JNI critical
duke@435 141 // region activities, which can cause unnecessary OutOfMemory errors.
duke@435 142 // 2) even if allocation requests can still be satisfied before GC locker
duke@435 143 // becomes inactive, for example, in tenured generation possibly with
duke@435 144 // heap expansion, those allocations can trigger lots of safepointing
duke@435 145 // attempts (ineffective GC attempts) and require Heap_lock which
duke@435 146 // slow down allocations tremendously.
duke@435 147 //
duke@435 148 // Note that critical regions can be nested in a single thread, so
duke@435 149 // we must allow threads already in critical regions to continue.
duke@435 150 //
duke@435 151 // JNI critical regions are the only participants in this scheme
duke@435 152 // because they are, by spec, well bounded while in a critical region.
duke@435 153 //
never@3494 154 // Each of the following two method is split into a fast path and a
never@3494 155 // slow path. JNICritical_lock is only grabbed in the slow path.
duke@435 156 // _needs_gc is initially false and every java thread will go
never@3494 157 // through the fast path, which simply increments or decrements the
never@3494 158 // current thread's critical count. When GC happens at a safepoint,
never@3494 159 // GC_locker::is_active() is checked. Since there is no safepoint in
never@3494 160 // the fast path of lock_critical() and unlock_critical(), there is
never@3494 161 // no race condition between the fast path and GC. After _needs_gc
never@3494 162 // is set at a safepoint, every thread will go through the slow path
never@3494 163 // after the safepoint. Since after a safepoint, each of the
never@3494 164 // following two methods is either entered from the method entry and
never@3494 165 // falls into the slow path, or is resumed from the safepoints in
never@3494 166 // the method, which only exist in the slow path. So when _needs_gc
never@3494 167 // is set, the slow path is always taken, till _needs_gc is cleared.
duke@435 168 static void lock_critical(JavaThread* thread);
duke@435 169 static void unlock_critical(JavaThread* thread);
never@3500 170
never@3500 171 static address needs_gc_address() { return (address) &_needs_gc; }
duke@435 172 };
duke@435 173
duke@435 174
duke@435 175 // A No_GC_Verifier object can be placed in methods where one assumes that
duke@435 176 // no garbage collection will occur. The destructor will verify this property
duke@435 177 // unless the constructor is called with argument false (not verifygc).
duke@435 178 //
duke@435 179 // The check will only be done in debug mode and if verifygc true.
duke@435 180
duke@435 181 class No_GC_Verifier: public StackObj {
duke@435 182 friend class Pause_No_GC_Verifier;
duke@435 183
duke@435 184 protected:
duke@435 185 bool _verifygc;
duke@435 186 unsigned int _old_invocations;
duke@435 187
duke@435 188 public:
duke@435 189 #ifdef ASSERT
duke@435 190 No_GC_Verifier(bool verifygc = true);
duke@435 191 ~No_GC_Verifier();
duke@435 192 #else
duke@435 193 No_GC_Verifier(bool verifygc = true) {}
duke@435 194 ~No_GC_Verifier() {}
duke@435 195 #endif
duke@435 196 };
duke@435 197
duke@435 198 // A Pause_No_GC_Verifier is used to temporarily pause the behavior
duke@435 199 // of a No_GC_Verifier object. If we are not in debug mode or if the
duke@435 200 // No_GC_Verifier object has a _verifygc value of false, then there
duke@435 201 // is nothing to do.
duke@435 202
duke@435 203 class Pause_No_GC_Verifier: public StackObj {
duke@435 204 private:
duke@435 205 No_GC_Verifier * _ngcv;
duke@435 206
duke@435 207 public:
duke@435 208 #ifdef ASSERT
duke@435 209 Pause_No_GC_Verifier(No_GC_Verifier * ngcv);
duke@435 210 ~Pause_No_GC_Verifier();
duke@435 211 #else
duke@435 212 Pause_No_GC_Verifier(No_GC_Verifier * ngcv) {}
duke@435 213 ~Pause_No_GC_Verifier() {}
duke@435 214 #endif
duke@435 215 };
duke@435 216
duke@435 217
duke@435 218 // A No_Safepoint_Verifier object will throw an assertion failure if
duke@435 219 // the current thread passes a possible safepoint while this object is
duke@435 220 // instantiated. A safepoint, will either be: an oop allocation, blocking
duke@435 221 // on a Mutex or JavaLock, or executing a VM operation.
duke@435 222 //
duke@435 223 // If StrictSafepointChecks is turned off, it degrades into a No_GC_Verifier
duke@435 224 //
duke@435 225 class No_Safepoint_Verifier : public No_GC_Verifier {
duke@435 226 friend class Pause_No_Safepoint_Verifier;
duke@435 227
duke@435 228 private:
duke@435 229 bool _activated;
duke@435 230 Thread *_thread;
duke@435 231 public:
duke@435 232 #ifdef ASSERT
never@533 233 No_Safepoint_Verifier(bool activated = true, bool verifygc = true ) :
never@533 234 No_GC_Verifier(verifygc),
never@533 235 _activated(activated) {
duke@435 236 _thread = Thread::current();
duke@435 237 if (_activated) {
duke@435 238 _thread->_allow_allocation_count++;
duke@435 239 _thread->_allow_safepoint_count++;
duke@435 240 }
duke@435 241 }
duke@435 242
duke@435 243 ~No_Safepoint_Verifier() {
duke@435 244 if (_activated) {
duke@435 245 _thread->_allow_allocation_count--;
duke@435 246 _thread->_allow_safepoint_count--;
duke@435 247 }
duke@435 248 }
duke@435 249 #else
duke@435 250 No_Safepoint_Verifier(bool activated = true, bool verifygc = true) : No_GC_Verifier(verifygc){}
duke@435 251 ~No_Safepoint_Verifier() {}
duke@435 252 #endif
duke@435 253 };
duke@435 254
duke@435 255 // A Pause_No_Safepoint_Verifier is used to temporarily pause the
duke@435 256 // behavior of a No_Safepoint_Verifier object. If we are not in debug
duke@435 257 // mode then there is nothing to do. If the No_Safepoint_Verifier
duke@435 258 // object has an _activated value of false, then there is nothing to
duke@435 259 // do for safepoint and allocation checking, but there may still be
duke@435 260 // something to do for the underlying No_GC_Verifier object.
duke@435 261
duke@435 262 class Pause_No_Safepoint_Verifier : public Pause_No_GC_Verifier {
duke@435 263 private:
duke@435 264 No_Safepoint_Verifier * _nsv;
duke@435 265
duke@435 266 public:
duke@435 267 #ifdef ASSERT
duke@435 268 Pause_No_Safepoint_Verifier(No_Safepoint_Verifier * nsv)
duke@435 269 : Pause_No_GC_Verifier(nsv) {
duke@435 270
duke@435 271 _nsv = nsv;
duke@435 272 if (_nsv->_activated) {
duke@435 273 _nsv->_thread->_allow_allocation_count--;
duke@435 274 _nsv->_thread->_allow_safepoint_count--;
duke@435 275 }
duke@435 276 }
duke@435 277
duke@435 278 ~Pause_No_Safepoint_Verifier() {
duke@435 279 if (_nsv->_activated) {
duke@435 280 _nsv->_thread->_allow_allocation_count++;
duke@435 281 _nsv->_thread->_allow_safepoint_count++;
duke@435 282 }
duke@435 283 }
duke@435 284 #else
duke@435 285 Pause_No_Safepoint_Verifier(No_Safepoint_Verifier * nsv)
duke@435 286 : Pause_No_GC_Verifier(nsv) {}
duke@435 287 ~Pause_No_Safepoint_Verifier() {}
duke@435 288 #endif
duke@435 289 };
duke@435 290
ysr@1241 291 // A SkipGCALot object is used to elide the usual effect of gc-a-lot
ysr@1241 292 // over a section of execution by a thread. Currently, it's used only to
ysr@1241 293 // prevent re-entrant calls to GC.
ysr@1241 294 class SkipGCALot : public StackObj {
ysr@1241 295 private:
ysr@1241 296 bool _saved;
ysr@1241 297 Thread* _t;
ysr@1241 298
ysr@1241 299 public:
ysr@1241 300 #ifdef ASSERT
ysr@1241 301 SkipGCALot(Thread* t) : _t(t) {
ysr@1241 302 _saved = _t->skip_gcalot();
ysr@1241 303 _t->set_skip_gcalot(true);
ysr@1241 304 }
ysr@1241 305
ysr@1241 306 ~SkipGCALot() {
ysr@1241 307 assert(_t->skip_gcalot(), "Save-restore protocol invariant");
ysr@1241 308 _t->set_skip_gcalot(_saved);
ysr@1241 309 }
ysr@1241 310 #else
ysr@1241 311 SkipGCALot(Thread* t) { }
ysr@1241 312 ~SkipGCALot() { }
ysr@1241 313 #endif
ysr@1241 314 };
ysr@1241 315
duke@435 316 // JRT_LEAF currently can be called from either _thread_in_Java or
duke@435 317 // _thread_in_native mode. In _thread_in_native, it is ok
duke@435 318 // for another thread to trigger GC. The rest of the JRT_LEAF
duke@435 319 // rules apply.
duke@435 320 class JRT_Leaf_Verifier : public No_Safepoint_Verifier {
duke@435 321 static bool should_verify_GC();
duke@435 322 public:
duke@435 323 #ifdef ASSERT
duke@435 324 JRT_Leaf_Verifier();
duke@435 325 ~JRT_Leaf_Verifier();
duke@435 326 #else
duke@435 327 JRT_Leaf_Verifier() {}
duke@435 328 ~JRT_Leaf_Verifier() {}
duke@435 329 #endif
duke@435 330 };
duke@435 331
duke@435 332 // A No_Alloc_Verifier object can be placed in methods where one assumes that
duke@435 333 // no allocation will occur. The destructor will verify this property
duke@435 334 // unless the constructor is called with argument false (not activated).
duke@435 335 //
duke@435 336 // The check will only be done in debug mode and if activated.
duke@435 337 // Note: this only makes sense at safepoints (otherwise, other threads may
duke@435 338 // allocate concurrently.)
duke@435 339
duke@435 340 class No_Alloc_Verifier : public StackObj {
duke@435 341 private:
duke@435 342 bool _activated;
duke@435 343
duke@435 344 public:
duke@435 345 #ifdef ASSERT
duke@435 346 No_Alloc_Verifier(bool activated = true) {
duke@435 347 _activated = activated;
duke@435 348 if (_activated) Thread::current()->_allow_allocation_count++;
duke@435 349 }
duke@435 350
duke@435 351 ~No_Alloc_Verifier() {
duke@435 352 if (_activated) Thread::current()->_allow_allocation_count--;
duke@435 353 }
duke@435 354 #else
duke@435 355 No_Alloc_Verifier(bool activated = true) {}
duke@435 356 ~No_Alloc_Verifier() {}
duke@435 357 #endif
duke@435 358 };
stefank@2314 359
stefank@2314 360 #endif // SHARE_VM_MEMORY_GCLOCKER_HPP

mercurial