src/hotspot/share/runtime/safepoint.cpp
author coleenp
Fri, 23 Feb 2018 14:13:06 -0500
changeset 49061 a6b6a428c915
parent 48472 de2e4ff493bf
child 49169 af8578e25d17
permissions -rw-r--r--
8198635: Remove unused safepoint message functions and ShowSafepointMsgs Reviewed-by: eosterlund, gtriantafill
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
     1
/*
49061
a6b6a428c915 8198635: Remove unused safepoint message functions and ShowSafepointMsgs
coleenp
parents: 48472
diff changeset
     2
 * Copyright (c) 1997, 2018, Oracle and/or its affiliates. All rights reserved.
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
     3
 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
489c9b5090e2 Initial load
duke
parents:
diff changeset
     4
 *
489c9b5090e2 Initial load
duke
parents:
diff changeset
     5
 * This code is free software; you can redistribute it and/or modify it
489c9b5090e2 Initial load
duke
parents:
diff changeset
     6
 * under the terms of the GNU General Public License version 2 only, as
489c9b5090e2 Initial load
duke
parents:
diff changeset
     7
 * published by the Free Software Foundation.
489c9b5090e2 Initial load
duke
parents:
diff changeset
     8
 *
489c9b5090e2 Initial load
duke
parents:
diff changeset
     9
 * This code is distributed in the hope that it will be useful, but WITHOUT
489c9b5090e2 Initial load
duke
parents:
diff changeset
    10
 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
489c9b5090e2 Initial load
duke
parents:
diff changeset
    11
 * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
489c9b5090e2 Initial load
duke
parents:
diff changeset
    12
 * version 2 for more details (a copy is included in the LICENSE file that
489c9b5090e2 Initial load
duke
parents:
diff changeset
    13
 * accompanied this code).
489c9b5090e2 Initial load
duke
parents:
diff changeset
    14
 *
489c9b5090e2 Initial load
duke
parents:
diff changeset
    15
 * You should have received a copy of the GNU General Public License version
489c9b5090e2 Initial load
duke
parents:
diff changeset
    16
 * 2 along with this work; if not, write to the Free Software Foundation,
489c9b5090e2 Initial load
duke
parents:
diff changeset
    17
 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
489c9b5090e2 Initial load
duke
parents:
diff changeset
    18
 *
5547
f4b087cbb361 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 5533
diff changeset
    19
 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
f4b087cbb361 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 5533
diff changeset
    20
 * or visit www.oracle.com if you need additional information or have any
f4b087cbb361 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 5533
diff changeset
    21
 * questions.
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    22
 *
489c9b5090e2 Initial load
duke
parents:
diff changeset
    23
 */
489c9b5090e2 Initial load
duke
parents:
diff changeset
    24
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 7106
diff changeset
    25
#include "precompiled.hpp"
47774
69c081ca110a 8184765: Dynamically resize SystemDictionary
gziemski
parents: 47637
diff changeset
    26
#include "classfile/classLoaderData.hpp"
24426
0a69c8cdfca9 8038654: Separate SymbolTable and StringTable code
gziemski
parents: 24096
diff changeset
    27
#include "classfile/stringTable.hpp"
34666
1c7168ea0034 8140485: Class load and creation cleanup
mgronlun
parents: 33763
diff changeset
    28
#include "classfile/symbolTable.hpp"
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 7106
diff changeset
    29
#include "classfile/systemDictionary.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 7106
diff changeset
    30
#include "code/codeCache.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 7106
diff changeset
    31
#include "code/icBuffer.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 7106
diff changeset
    32
#include "code/nmethod.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 7106
diff changeset
    33
#include "code/pcDesc.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 7106
diff changeset
    34
#include "code/scopeDesc.hpp"
30764
fec48bf5a827 8079792: GC directory structure cleanup
pliden
parents: 27662
diff changeset
    35
#include "gc/shared/collectedHeap.hpp"
fec48bf5a827 8079792: GC directory structure cleanup
pliden
parents: 27662
diff changeset
    36
#include "gc/shared/gcLocker.inline.hpp"
46702
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
    37
#include "gc/shared/strongRootsScope.hpp"
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
    38
#include "gc/shared/workgroup.hpp"
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 7106
diff changeset
    39
#include "interpreter/interpreter.hpp"
33763
cec2333f839c 8140348: Convert TraceSafepoint to Unified Logging
rprotacio
parents: 33589
diff changeset
    40
#include "logging/log.hpp"
46701
f559541c0daa 8181917: Refactor UL LogStreams to avoid using resource area
stuefe
parents: 46630
diff changeset
    41
#include "logging/logStream.hpp"
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 7106
diff changeset
    42
#include "memory/resourceArea.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 7106
diff changeset
    43
#include "memory/universe.inline.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 7106
diff changeset
    44
#include "oops/oop.inline.hpp"
8076
96d498ec7ae1 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 7397
diff changeset
    45
#include "oops/symbol.hpp"
40655
9f644073d3a0 8157907: Incorrect inclusion of atomic.hpp instead of atomic.inline.hpp
dholmes
parents: 38151
diff changeset
    46
#include "runtime/atomic.hpp"
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 7106
diff changeset
    47
#include "runtime/compilationPolicy.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 7106
diff changeset
    48
#include "runtime/deoptimization.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 7106
diff changeset
    49
#include "runtime/frame.inline.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 7106
diff changeset
    50
#include "runtime/interfaceSupport.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 7106
diff changeset
    51
#include "runtime/mutexLocker.hpp"
24351
61b33cc6d3cf 8042195: Introduce umbrella header orderAccess.inline.hpp.
goetz
parents: 24096
diff changeset
    52
#include "runtime/orderAccess.inline.hpp"
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 7106
diff changeset
    53
#include "runtime/osThread.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 7106
diff changeset
    54
#include "runtime/safepoint.hpp"
47881
0ce0ac68ace7 8189941: Implementation JEP 312: Thread-local handshake
rehn
parents: 47774
diff changeset
    55
#include "runtime/safepointMechanism.inline.hpp"
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 7106
diff changeset
    56
#include "runtime/signature.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 7106
diff changeset
    57
#include "runtime/stubCodeGenerator.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 7106
diff changeset
    58
#include "runtime/stubRoutines.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 7106
diff changeset
    59
#include "runtime/sweeper.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 7106
diff changeset
    60
#include "runtime/synchronizer.hpp"
14583
d70ee55535f4 8003935: Simplify the needed includes for using Thread::current()
stefank
parents: 13195
diff changeset
    61
#include "runtime/thread.inline.hpp"
48105
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
    62
#include "runtime/threadSMR.hpp"
37161
e881f320966e 8150015: Integrate TraceTime with Unified Logging more seamlessly
rehn
parents: 37135
diff changeset
    63
#include "runtime/timerTrace.hpp"
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 7106
diff changeset
    64
#include "services/runtimeService.hpp"
37176
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
    65
#include "trace/tracing.hpp"
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
    66
#include "trace/traceMacros.hpp"
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 7106
diff changeset
    67
#include "utilities/events.hpp"
15482
470d0b0c09f1 8005915: Unify SERIALGC and INCLUDE_ALTERNATE_GCS
jprovino
parents: 14583
diff changeset
    68
#include "utilities/macros.hpp"
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 7106
diff changeset
    69
#ifdef COMPILER1
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 7106
diff changeset
    70
#include "c1/c1_globals.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 7106
diff changeset
    71
#endif
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    72
489c9b5090e2 Initial load
duke
parents:
diff changeset
    73
// --------------------------------------------------------------------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
    74
// Implementation of Safepoint begin/end
489c9b5090e2 Initial load
duke
parents:
diff changeset
    75
489c9b5090e2 Initial load
duke
parents:
diff changeset
    76
SafepointSynchronize::SynchronizeState volatile SafepointSynchronize::_state = SafepointSynchronize::_not_synchronized;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    77
volatile int  SafepointSynchronize::_waiting_to_block = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    78
volatile int SafepointSynchronize::_safepoint_counter = 0;
11631
33813f69207b 7129164: JNI Get/ReleasePrimitiveArrayCritical doesn't scale
never
parents: 10565
diff changeset
    79
int SafepointSynchronize::_current_jni_active_count = 0;
5042
f86707fd195a 6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents: 4006
diff changeset
    80
long  SafepointSynchronize::_end_of_last_safepoint = 0;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    81
static volatile int PageArmed = 0 ;        // safepoint polling page is RO|RW vs PROT_NONE
489c9b5090e2 Initial load
duke
parents:
diff changeset
    82
static volatile int TryingToBlock = 0 ;    // proximate value -- for advisory use only
489c9b5090e2 Initial load
duke
parents:
diff changeset
    83
static bool timeout_error_printed = false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    84
489c9b5090e2 Initial load
duke
parents:
diff changeset
    85
// Roll all threads forward to a safepoint and suspend them all
489c9b5090e2 Initial load
duke
parents:
diff changeset
    86
void SafepointSynchronize::begin() {
37176
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
    87
  EventSafepointBegin begin_event;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    88
  Thread* myThread = Thread::current();
489c9b5090e2 Initial load
duke
parents:
diff changeset
    89
  assert(myThread->is_VM_thread(), "Only VM thread may execute a safepoint");
489c9b5090e2 Initial load
duke
parents:
diff changeset
    90
5042
f86707fd195a 6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents: 4006
diff changeset
    91
  if (PrintSafepointStatistics || PrintSafepointStatisticsTimeout > 0) {
f86707fd195a 6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents: 4006
diff changeset
    92
    _safepoint_begin_time = os::javaTimeNanos();
f86707fd195a 6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents: 4006
diff changeset
    93
    _ts_of_current_safepoint = tty->time_stamp().seconds();
f86707fd195a 6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents: 4006
diff changeset
    94
  }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    95
47636
52d46d7e3454 8188919: Generalize GC thread suspend/resume at safepoints
eosterlund
parents: 47629
diff changeset
    96
  Universe::heap()->safepoint_synchronize_begin();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    97
489c9b5090e2 Initial load
duke
parents:
diff changeset
    98
  // By getting the Threads_lock, we assure that no threads are about to start or
489c9b5090e2 Initial load
duke
parents:
diff changeset
    99
  // exit. It is released again in SafepointSynchronize::end().
489c9b5090e2 Initial load
duke
parents:
diff changeset
   100
  Threads_lock->lock();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   101
489c9b5090e2 Initial load
duke
parents:
diff changeset
   102
  assert( _state == _not_synchronized, "trying to safepoint synchronize with wrong state");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   103
489c9b5090e2 Initial load
duke
parents:
diff changeset
   104
  int nof_threads = Threads::number_of_threads();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   105
48472
de2e4ff493bf 8152957: Improve specificity of safepoint logging to print safepoint type
coleenp
parents: 48105
diff changeset
   106
  log_debug(safepoint)("Safepoint synchronization initiated. (%d threads)", nof_threads);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   107
489c9b5090e2 Initial load
duke
parents:
diff changeset
   108
  RuntimeService::record_safepoint_begin();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   109
489c9b5090e2 Initial load
duke
parents:
diff changeset
   110
  MutexLocker mu(Safepoint_lock);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   111
11631
33813f69207b 7129164: JNI Get/ReleasePrimitiveArrayCritical doesn't scale
never
parents: 10565
diff changeset
   112
  // Reset the count of active JNI critical threads
33813f69207b 7129164: JNI Get/ReleasePrimitiveArrayCritical doesn't scale
never
parents: 10565
diff changeset
   113
  _current_jni_active_count = 0;
33813f69207b 7129164: JNI Get/ReleasePrimitiveArrayCritical doesn't scale
never
parents: 10565
diff changeset
   114
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   115
  // Set number of threads to wait for, before we initiate the callbacks
489c9b5090e2 Initial load
duke
parents:
diff changeset
   116
  _waiting_to_block = nof_threads;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   117
  TryingToBlock     = 0 ;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   118
  int still_running = nof_threads;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   119
489c9b5090e2 Initial load
duke
parents:
diff changeset
   120
  // Save the starting time, so that it can be compared to see if this has taken
489c9b5090e2 Initial load
duke
parents:
diff changeset
   121
  // too long to complete.
33589
7cbd1b2c139b 8139040: Fix initializations before ShouldNotReachHere() etc. and enable -Wuninitialized on linux.
goetz
parents: 33222
diff changeset
   122
  jlong safepoint_limit_time = 0;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   123
  timeout_error_printed = false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   124
4006
7be2d5b3b15c 6880029: JDK 1.6.0_u14p Application crashed very early
xlu
parents: 3826
diff changeset
   125
  // PrintSafepointStatisticsTimeout can be specified separately. When
7be2d5b3b15c 6880029: JDK 1.6.0_u14p Application crashed very early
xlu
parents: 3826
diff changeset
   126
  // specified, PrintSafepointStatistics will be set to true in
7be2d5b3b15c 6880029: JDK 1.6.0_u14p Application crashed very early
xlu
parents: 3826
diff changeset
   127
  // deferred_initialize_stat method. The initialization has to be done
7be2d5b3b15c 6880029: JDK 1.6.0_u14p Application crashed very early
xlu
parents: 3826
diff changeset
   128
  // early enough to avoid any races. See bug 6880029 for details.
7be2d5b3b15c 6880029: JDK 1.6.0_u14p Application crashed very early
xlu
parents: 3826
diff changeset
   129
  if (PrintSafepointStatistics || PrintSafepointStatisticsTimeout > 0) {
7be2d5b3b15c 6880029: JDK 1.6.0_u14p Application crashed very early
xlu
parents: 3826
diff changeset
   130
    deferred_initialize_stat();
7be2d5b3b15c 6880029: JDK 1.6.0_u14p Application crashed very early
xlu
parents: 3826
diff changeset
   131
  }
7be2d5b3b15c 6880029: JDK 1.6.0_u14p Application crashed very early
xlu
parents: 3826
diff changeset
   132
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   133
  // Begin the process of bringing the system to a safepoint.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   134
  // Java threads can be in several different states and are
489c9b5090e2 Initial load
duke
parents:
diff changeset
   135
  // stopped by different mechanisms:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   136
  //
489c9b5090e2 Initial load
duke
parents:
diff changeset
   137
  //  1. Running interpreted
22551
9bf46d16dcc6 8025856: Fix typos in the GC code
jwilhelm
parents: 20300
diff changeset
   138
  //     The interpreter dispatch table is changed to force it to
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   139
  //     check for a safepoint condition between bytecodes.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   140
  //  2. Running in native code
489c9b5090e2 Initial load
duke
parents:
diff changeset
   141
  //     When returning from the native code, a Java thread must check
489c9b5090e2 Initial load
duke
parents:
diff changeset
   142
  //     the safepoint _state to see if we must block.  If the
489c9b5090e2 Initial load
duke
parents:
diff changeset
   143
  //     VM thread sees a Java thread in native, it does
489c9b5090e2 Initial load
duke
parents:
diff changeset
   144
  //     not wait for this thread to block.  The order of the memory
489c9b5090e2 Initial load
duke
parents:
diff changeset
   145
  //     writes and reads of both the safepoint state and the Java
489c9b5090e2 Initial load
duke
parents:
diff changeset
   146
  //     threads state is critical.  In order to guarantee that the
489c9b5090e2 Initial load
duke
parents:
diff changeset
   147
  //     memory writes are serialized with respect to each other,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   148
  //     the VM thread issues a memory barrier instruction
489c9b5090e2 Initial load
duke
parents:
diff changeset
   149
  //     (on MP systems).  In order to avoid the overhead of issuing
489c9b5090e2 Initial load
duke
parents:
diff changeset
   150
  //     a memory barrier for each Java thread making native calls, each Java
489c9b5090e2 Initial load
duke
parents:
diff changeset
   151
  //     thread performs a write to a single memory page after changing
489c9b5090e2 Initial load
duke
parents:
diff changeset
   152
  //     the thread state.  The VM thread performs a sequence of
489c9b5090e2 Initial load
duke
parents:
diff changeset
   153
  //     mprotect OS calls which forces all previous writes from all
489c9b5090e2 Initial load
duke
parents:
diff changeset
   154
  //     Java threads to be serialized.  This is done in the
489c9b5090e2 Initial load
duke
parents:
diff changeset
   155
  //     os::serialize_thread_states() call.  This has proven to be
489c9b5090e2 Initial load
duke
parents:
diff changeset
   156
  //     much more efficient than executing a membar instruction
489c9b5090e2 Initial load
duke
parents:
diff changeset
   157
  //     on every call to native code.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   158
  //  3. Running compiled Code
489c9b5090e2 Initial load
duke
parents:
diff changeset
   159
  //     Compiled code reads a global (Safepoint Polling) page that
489c9b5090e2 Initial load
duke
parents:
diff changeset
   160
  //     is set to fault if we are trying to get to a safepoint.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   161
  //  4. Blocked
489c9b5090e2 Initial load
duke
parents:
diff changeset
   162
  //     A thread which is blocked will not be allowed to return from the
489c9b5090e2 Initial load
duke
parents:
diff changeset
   163
  //     block condition until the safepoint operation is complete.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   164
  //  5. In VM or Transitioning between states
489c9b5090e2 Initial load
duke
parents:
diff changeset
   165
  //     If a Java thread is currently running in the VM or transitioning
489c9b5090e2 Initial load
duke
parents:
diff changeset
   166
  //     between states, the safepointing code will wait for the thread to
489c9b5090e2 Initial load
duke
parents:
diff changeset
   167
  //     block itself when it attempts transitions to a new state.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   168
  //
37176
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   169
  {
40664
1ec65b303bb7 8164523: Clean up metadata for event based tracing
egahlin
parents: 40655
diff changeset
   170
    EventSafepointStateSynchronization sync_event;
37176
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   171
    int initial_running = 0;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   172
37176
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   173
    _state            = _synchronizing;
47881
0ce0ac68ace7 8189941: Implementation JEP 312: Thread-local handshake
rehn
parents: 47774
diff changeset
   174
0ce0ac68ace7 8189941: Implementation JEP 312: Thread-local handshake
rehn
parents: 47774
diff changeset
   175
    if (SafepointMechanism::uses_thread_local_poll()) {
0ce0ac68ace7 8189941: Implementation JEP 312: Thread-local handshake
rehn
parents: 47774
diff changeset
   176
      // Arming the per thread poll while having _state != _not_synchronized means safepointing
0ce0ac68ace7 8189941: Implementation JEP 312: Thread-local handshake
rehn
parents: 47774
diff changeset
   177
      log_trace(safepoint)("Setting thread local yield flag for threads");
48105
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   178
      for (JavaThreadIteratorWithHandle jtiwh; JavaThread *cur = jtiwh.next(); ) {
47881
0ce0ac68ace7 8189941: Implementation JEP 312: Thread-local handshake
rehn
parents: 47774
diff changeset
   179
        // Make sure the threads start polling, it is time to yield.
0ce0ac68ace7 8189941: Implementation JEP 312: Thread-local handshake
rehn
parents: 47774
diff changeset
   180
        SafepointMechanism::arm_local_poll(cur); // release store, global state -> local state
0ce0ac68ace7 8189941: Implementation JEP 312: Thread-local handshake
rehn
parents: 47774
diff changeset
   181
      }
0ce0ac68ace7 8189941: Implementation JEP 312: Thread-local handshake
rehn
parents: 47774
diff changeset
   182
    }
0ce0ac68ace7 8189941: Implementation JEP 312: Thread-local handshake
rehn
parents: 47774
diff changeset
   183
    OrderAccess::fence(); // storestore|storeload, global state -> local state
37176
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   184
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   185
    // Flush all thread states to memory
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   186
    if (!UseMembar) {
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   187
      os::serialize_thread_states();
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   188
    }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   189
47881
0ce0ac68ace7 8189941: Implementation JEP 312: Thread-local handshake
rehn
parents: 47774
diff changeset
   190
    if (SafepointMechanism::uses_global_page_poll()) {
0ce0ac68ace7 8189941: Implementation JEP 312: Thread-local handshake
rehn
parents: 47774
diff changeset
   191
      // Make interpreter safepoint aware
0ce0ac68ace7 8189941: Implementation JEP 312: Thread-local handshake
rehn
parents: 47774
diff changeset
   192
      Interpreter::notice_safepoints();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   193
47881
0ce0ac68ace7 8189941: Implementation JEP 312: Thread-local handshake
rehn
parents: 47774
diff changeset
   194
      if (DeferPollingPageLoopCount < 0) {
0ce0ac68ace7 8189941: Implementation JEP 312: Thread-local handshake
rehn
parents: 47774
diff changeset
   195
        // Make polling safepoint aware
0ce0ac68ace7 8189941: Implementation JEP 312: Thread-local handshake
rehn
parents: 47774
diff changeset
   196
        guarantee (PageArmed == 0, "invariant") ;
0ce0ac68ace7 8189941: Implementation JEP 312: Thread-local handshake
rehn
parents: 47774
diff changeset
   197
        PageArmed = 1 ;
0ce0ac68ace7 8189941: Implementation JEP 312: Thread-local handshake
rehn
parents: 47774
diff changeset
   198
        os::make_polling_page_unreadable();
0ce0ac68ace7 8189941: Implementation JEP 312: Thread-local handshake
rehn
parents: 47774
diff changeset
   199
      }
37176
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   200
    }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   201
37176
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   202
    // Consider using active_processor_count() ... but that call is expensive.
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   203
    int ncpus = os::processor_count() ;
48105
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   204
    unsigned int iterations = 0;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   205
48105
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   206
    {
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   207
      JavaThreadIteratorWithHandle jtiwh;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   208
#ifdef ASSERT
48105
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   209
      for (; JavaThread *cur = jtiwh.next(); ) {
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   210
        assert(cur->safepoint_state()->is_running(), "Illegal initial state");
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   211
        // Clear the visited flag to ensure that the critical counts are collected properly.
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   212
        cur->set_visited_for_critical_count(false);
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   213
      }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   214
#endif // ASSERT
489c9b5090e2 Initial load
duke
parents:
diff changeset
   215
48105
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   216
      if (SafepointTimeout)
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   217
        safepoint_limit_time = os::javaTimeNanos() + (jlong)SafepointTimeoutDelay * MICROUNITS;
37176
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   218
48105
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   219
      // Iterate through all threads until it have been determined how to stop them all at a safepoint
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   220
      int steps = 0 ;
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   221
      while(still_running > 0) {
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   222
        jtiwh.rewind();
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   223
        for (; JavaThread *cur = jtiwh.next(); ) {
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   224
          assert(!cur->is_ConcurrentGC_thread(), "A concurrent GC thread is unexpectly being suspended");
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   225
          ThreadSafepointState *cur_state = cur->safepoint_state();
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   226
          if (cur_state->is_running()) {
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   227
            cur_state->examine_state_of_thread();
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   228
            if (!cur_state->is_running()) {
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   229
              still_running--;
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   230
              // consider adjusting steps downward:
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   231
              //   steps = 0
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   232
              //   steps -= NNN
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   233
              //   steps >>= 1
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   234
              //   steps = MIN(steps, 2000-100)
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   235
              //   if (iterations != 0) steps -= NNN
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   236
            }
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   237
            LogTarget(Trace, safepoint) lt;
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   238
            if (lt.is_enabled()) {
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   239
              ResourceMark rm;
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   240
              LogStream ls(lt);
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   241
              cur_state->print_on(&ls);
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   242
            }
37176
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   243
          }
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   244
        }
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   245
48105
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   246
        if (iterations == 0) {
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   247
          initial_running = still_running;
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   248
          if (PrintSafepointStatistics) {
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   249
            begin_statistics(nof_threads, still_running);
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   250
          }
37176
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   251
        }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   252
48105
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   253
        if (still_running > 0) {
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   254
          // Check for if it takes to long
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   255
          if (SafepointTimeout && safepoint_limit_time < os::javaTimeNanos()) {
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   256
            print_safepoint_timeout(_spinning_timeout);
37176
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   257
          }
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   258
48105
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   259
          // Spin to avoid context switching.
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   260
          // There's a tension between allowing the mutators to run (and rendezvous)
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   261
          // vs spinning.  As the VM thread spins, wasting cycles, it consumes CPU that
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   262
          // a mutator might otherwise use profitably to reach a safepoint.  Excessive
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   263
          // spinning by the VM thread on a saturated system can increase rendezvous latency.
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   264
          // Blocking or yielding incur their own penalties in the form of context switching
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   265
          // and the resultant loss of $ residency.
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   266
          //
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   267
          // Further complicating matters is that yield() does not work as naively expected
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   268
          // on many platforms -- yield() does not guarantee that any other ready threads
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   269
          // will run.   As such we revert to naked_short_sleep() after some number of iterations.
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   270
          // nakes_short_sleep() is implemented as a short unconditional sleep.
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   271
          // Typical operating systems round a "short" sleep period up to 10 msecs, so sleeping
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   272
          // can actually increase the time it takes the VM thread to detect that a system-wide
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   273
          // stop-the-world safepoint has been reached.  In a pathological scenario such as that
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   274
          // described in CR6415670 the VMthread may sleep just before the mutator(s) become safe.
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   275
          // In that case the mutators will be stalled waiting for the safepoint to complete and the
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   276
          // the VMthread will be sleeping, waiting for the mutators to rendezvous.  The VMthread
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   277
          // will eventually wake up and detect that all mutators are safe, at which point
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   278
          // we'll again make progress.
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   279
          //
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   280
          // Beware too that that the VMThread typically runs at elevated priority.
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   281
          // Its default priority is higher than the default mutator priority.
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   282
          // Obviously, this complicates spinning.
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   283
          //
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   284
          // Note too that on Windows XP SwitchThreadTo() has quite different behavior than Sleep(0).
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   285
          // Sleep(0) will _not yield to lower priority threads, while SwitchThreadTo() will.
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   286
          //
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   287
          // See the comments in synchronizer.cpp for additional remarks on spinning.
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   288
          //
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   289
          // In the future we might:
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   290
          // 1. Modify the safepoint scheme to avoid potentially unbounded spinning.
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   291
          //    This is tricky as the path used by a thread exiting the JVM (say on
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   292
          //    on JNI call-out) simply stores into its state field.  The burden
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   293
          //    is placed on the VM thread, which must poll (spin).
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   294
          // 2. Find something useful to do while spinning.  If the safepoint is GC-related
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   295
          //    we might aggressively scan the stacks of threads that are already safe.
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   296
          // 3. Use Solaris schedctl to examine the state of the still-running mutators.
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   297
          //    If all the mutators are ONPROC there's no reason to sleep or yield.
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   298
          // 4. YieldTo() any still-running mutators that are ready but OFFPROC.
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   299
          // 5. Check system saturation.  If the system is not fully saturated then
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   300
          //    simply spin and avoid sleep/yield.
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   301
          // 6. As still-running mutators rendezvous they could unpark the sleeping
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   302
          //    VMthread.  This works well for still-running mutators that become
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   303
          //    safe.  The VMthread must still poll for mutators that call-out.
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   304
          // 7. Drive the policy on time-since-begin instead of iterations.
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   305
          // 8. Consider making the spin duration a function of the # of CPUs:
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   306
          //    Spin = (((ncpus-1) * M) + K) + F(still_running)
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   307
          //    Alternately, instead of counting iterations of the outer loop
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   308
          //    we could count the # of threads visited in the inner loop, above.
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   309
          // 9. On windows consider using the return value from SwitchThreadTo()
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   310
          //    to drive subsequent spin/SwitchThreadTo()/Sleep(N) decisions.
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   311
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   312
          if (SafepointMechanism::uses_global_page_poll() && int(iterations) == DeferPollingPageLoopCount) {
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   313
            guarantee (PageArmed == 0, "invariant") ;
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   314
            PageArmed = 1 ;
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   315
            os::make_polling_page_unreadable();
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   316
          }
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   317
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   318
          // Instead of (ncpus > 1) consider either (still_running < (ncpus + EPSILON)) or
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   319
          // ((still_running + _waiting_to_block - TryingToBlock)) < ncpus)
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   320
          ++steps ;
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   321
          if (ncpus > 1 && steps < SafepointSpinBeforeYield) {
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   322
            SpinPause() ;     // MP-Polite spin
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   323
          } else
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   324
            if (steps < DeferThrSuspendLoopCount) {
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   325
              os::naked_yield() ;
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   326
            } else {
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   327
              os::naked_short_sleep(1);
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   328
            }
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   329
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   330
          iterations ++ ;
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   331
        }
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   332
        assert(iterations < (uint)max_jint, "We have been iterating in the safepoint loop too long");
37176
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   333
      }
48105
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   334
    } // ThreadsListHandle destroyed here.
37176
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   335
    assert(still_running == 0, "sanity check");
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   336
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   337
    if (PrintSafepointStatistics) {
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   338
      update_statistics_on_spin_end();
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   339
    }
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   340
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   341
    if (sync_event.should_commit()) {
47629
494e1490b341 8173917: Safepoint ID is not consistent across event-based tracing events
egahlin
parents: 47216
diff changeset
   342
      // Group this event together with the ones committed after the counter is increased
494e1490b341 8173917: Safepoint ID is not consistent across event-based tracing events
egahlin
parents: 47216
diff changeset
   343
      sync_event.set_safepointId(safepoint_counter() + 1);
37176
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   344
      sync_event.set_initialThreadCount(initial_running);
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   345
      sync_event.set_runningThreadCount(_waiting_to_block);
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   346
      sync_event.set_iterations(iterations);
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   347
      sync_event.commit();
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   348
    }
48105
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   349
  } // EventSafepointStateSynchronization destroyed here.
37176
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   350
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   351
  // wait until all threads are stopped
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   352
  {
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   353
    EventSafepointWaitBlocked wait_blocked_event;
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   354
    int initial_waiting_to_block = _waiting_to_block;
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   355
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   356
    while (_waiting_to_block > 0) {
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   357
      log_debug(safepoint)("Waiting for %d thread(s) to block", _waiting_to_block);
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   358
      if (!SafepointTimeout || timeout_error_printed) {
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   359
        Safepoint_lock->wait(true);  // true, means with no safepoint checks
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   360
      } else {
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   361
        // Compute remaining time
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   362
        jlong remaining_time = safepoint_limit_time - os::javaTimeNanos();
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   363
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   364
        // If there is no remaining time, then there is an error
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   365
        if (remaining_time < 0 || Safepoint_lock->wait(true, remaining_time / MICROUNITS)) {
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   366
          print_safepoint_timeout(_blocking_timeout);
33763
cec2333f839c 8140348: Convert TraceSafepoint to Unified Logging
rprotacio
parents: 33589
diff changeset
   367
        }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   368
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   369
    }
37176
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   370
    assert(_waiting_to_block == 0, "sanity check");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   371
37176
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   372
#ifndef PRODUCT
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   373
    if (SafepointTimeout) {
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   374
      jlong current_time = os::javaTimeNanos();
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   375
      if (safepoint_limit_time < current_time) {
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   376
        tty->print_cr("# SafepointSynchronize: Finished after "
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   377
                      INT64_FORMAT_W(6) " ms",
46630
75aa3e39d02c 8182299: Enable disabled clang warnings, build on OSX 10 + Xcode 8
jwilhelm
parents: 46541
diff changeset
   378
                      (int64_t)((current_time - safepoint_limit_time) / MICROUNITS +
75aa3e39d02c 8182299: Enable disabled clang warnings, build on OSX 10 + Xcode 8
jwilhelm
parents: 46541
diff changeset
   379
                                (jlong)SafepointTimeoutDelay));
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   380
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   381
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   382
#endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
   383
37176
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   384
    assert((_safepoint_counter & 0x1) == 0, "must be even");
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   385
    assert(Threads_lock->owned_by_self(), "must hold Threads_lock");
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   386
    _safepoint_counter ++;
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   387
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   388
    // Record state
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   389
    _state = _synchronized;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   390
37176
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   391
    OrderAccess::fence();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   392
37176
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   393
    if (wait_blocked_event.should_commit()) {
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   394
      wait_blocked_event.set_safepointId(safepoint_counter());
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   395
      wait_blocked_event.set_runningThreadCount(initial_waiting_to_block);
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   396
      wait_blocked_event.commit();
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   397
    }
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   398
  } // EventSafepointWaitBlocked
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   399
12108
b2ee55606008 7150390: JFR test crashed on assert(_jni_lock_count == count) failed: must be equal
never
parents: 11637
diff changeset
   400
#ifdef ASSERT
48105
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   401
  // Make sure all the threads were visited.
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   402
  for (JavaThreadIteratorWithHandle jtiwh; JavaThread *cur = jtiwh.next(); ) {
12108
b2ee55606008 7150390: JFR test crashed on assert(_jni_lock_count == count) failed: must be equal
never
parents: 11637
diff changeset
   403
    assert(cur->was_visited_for_critical_count(), "missed a thread");
b2ee55606008 7150390: JFR test crashed on assert(_jni_lock_count == count) failed: must be equal
never
parents: 11637
diff changeset
   404
  }
b2ee55606008 7150390: JFR test crashed on assert(_jni_lock_count == count) failed: must be equal
never
parents: 11637
diff changeset
   405
#endif // ASSERT
b2ee55606008 7150390: JFR test crashed on assert(_jni_lock_count == count) failed: must be equal
never
parents: 11637
diff changeset
   406
11631
33813f69207b 7129164: JNI Get/ReleasePrimitiveArrayCritical doesn't scale
never
parents: 10565
diff changeset
   407
  // Update the count of active JNI critical regions
35492
c8c0273e6b91 8146690: Make all classes in GC follow the naming convention.
david
parents: 35061
diff changeset
   408
  GCLocker::set_jni_lock_count(_current_jni_active_count);
11631
33813f69207b 7129164: JNI Get/ReleasePrimitiveArrayCritical doesn't scale
never
parents: 10565
diff changeset
   409
48472
de2e4ff493bf 8152957: Improve specificity of safepoint logging to print safepoint type
coleenp
parents: 48105
diff changeset
   410
  log_info(safepoint)("Entering safepoint region: %s", VMThread::vm_safepoint_description());
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   411
489c9b5090e2 Initial load
duke
parents:
diff changeset
   412
  RuntimeService::record_safepoint_synchronized();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   413
  if (PrintSafepointStatistics) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   414
    update_statistics_on_sync_end(os::javaTimeNanos());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   415
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   416
489c9b5090e2 Initial load
duke
parents:
diff changeset
   417
  // Call stuff that needs to be run when a safepoint is just about to be completed
37176
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   418
  {
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   419
    EventSafepointCleanup cleanup_event;
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   420
    do_cleanup_tasks();
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   421
    if (cleanup_event.should_commit()) {
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   422
      cleanup_event.set_safepointId(safepoint_counter());
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   423
      cleanup_event.commit();
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   424
    }
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   425
  }
5042
f86707fd195a 6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents: 4006
diff changeset
   426
f86707fd195a 6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents: 4006
diff changeset
   427
  if (PrintSafepointStatistics) {
f86707fd195a 6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents: 4006
diff changeset
   428
    // Record how much time spend on the above cleanup tasks
f86707fd195a 6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents: 4006
diff changeset
   429
    update_statistics_on_cleanup_end(os::javaTimeNanos());
f86707fd195a 6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents: 4006
diff changeset
   430
  }
37176
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   431
  if (begin_event.should_commit()) {
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   432
    begin_event.set_safepointId(safepoint_counter());
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   433
    begin_event.set_totalThreadCount(nof_threads);
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   434
    begin_event.set_jniCriticalThreadCount(_current_jni_active_count);
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   435
    begin_event.commit();
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   436
  }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   437
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   438
489c9b5090e2 Initial load
duke
parents:
diff changeset
   439
// Wake up all threads, so they are ready to resume execution after the safepoint
489c9b5090e2 Initial load
duke
parents:
diff changeset
   440
// operation has been carried out
489c9b5090e2 Initial load
duke
parents:
diff changeset
   441
void SafepointSynchronize::end() {
37176
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   442
  EventSafepointEnd event;
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   443
  int safepoint_id = safepoint_counter(); // Keep the odd counter as "id"
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   444
489c9b5090e2 Initial load
duke
parents:
diff changeset
   445
  assert(Threads_lock->owned_by_self(), "must hold Threads_lock");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   446
  assert((_safepoint_counter & 0x1) == 1, "must be odd");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   447
  _safepoint_counter ++;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   448
  // memory fence isn't required here since an odd _safepoint_counter
489c9b5090e2 Initial load
duke
parents:
diff changeset
   449
  // value can do no harm and a fence is issued below anyway.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   450
489c9b5090e2 Initial load
duke
parents:
diff changeset
   451
  DEBUG_ONLY(Thread* myThread = Thread::current();)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   452
  assert(myThread->is_VM_thread(), "Only VM thread can execute a safepoint");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   453
489c9b5090e2 Initial load
duke
parents:
diff changeset
   454
  if (PrintSafepointStatistics) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   455
    end_statistics(os::javaTimeNanos());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   456
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   457
48105
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   458
  {
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   459
    JavaThreadIteratorWithHandle jtiwh;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   460
#ifdef ASSERT
48105
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   461
    // A pending_exception cannot be installed during a safepoint.  The threads
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   462
    // may install an async exception after they come back from a safepoint into
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   463
    // pending_exception after they unblock.  But that should happen later.
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   464
    for (; JavaThread *cur = jtiwh.next(); ) {
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   465
      assert (!(cur->has_pending_exception() &&
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   466
                cur->safepoint_state()->is_at_poll_safepoint()),
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   467
              "safepoint installed a pending exception");
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   468
    }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   469
#endif // ASSERT
489c9b5090e2 Initial load
duke
parents:
diff changeset
   470
48105
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   471
    if (PageArmed) {
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   472
      assert(SafepointMechanism::uses_global_page_poll(), "sanity");
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   473
      // Make polling safepoint aware
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   474
      os::make_polling_page_readable();
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   475
      PageArmed = 0 ;
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   476
    }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   477
48105
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   478
    if (SafepointMechanism::uses_global_page_poll()) {
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   479
      // Remove safepoint check from interpreter
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   480
      Interpreter::ignore_safepoints();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   481
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   482
48105
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   483
    {
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   484
      MutexLocker mu(Safepoint_lock);
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   485
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   486
      assert(_state == _synchronized, "must be synchronized before ending safepoint synchronization");
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   487
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   488
      if (SafepointMechanism::uses_thread_local_poll()) {
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   489
        _state = _not_synchronized;
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   490
        OrderAccess::storestore(); // global state -> local state
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   491
        jtiwh.rewind();
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   492
        for (; JavaThread *current = jtiwh.next(); ) {
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   493
          ThreadSafepointState* cur_state = current->safepoint_state();
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   494
          cur_state->restart(); // TSS _running
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   495
          SafepointMechanism::disarm_local_poll(current); // release store, local state -> polling page
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   496
        }
48472
de2e4ff493bf 8152957: Improve specificity of safepoint logging to print safepoint type
coleenp
parents: 48105
diff changeset
   497
        log_info(safepoint)("Leaving safepoint region");
48105
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   498
      } else {
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   499
        // Set to not synchronized, so the threads will not go into the signal_thread_blocked method
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   500
        // when they get restarted.
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   501
        _state = _not_synchronized;
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   502
        OrderAccess::fence();
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   503
48472
de2e4ff493bf 8152957: Improve specificity of safepoint logging to print safepoint type
coleenp
parents: 48105
diff changeset
   504
        log_info(safepoint)("Leaving safepoint region");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   505
48105
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   506
        // Start suspended threads
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   507
        jtiwh.rewind();
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   508
        for (; JavaThread *current = jtiwh.next(); ) {
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   509
          // A problem occurring on Solaris is when attempting to restart threads
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   510
          // the first #cpus - 1 go well, but then the VMThread is preempted when we get
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   511
          // to the next one (since it has been running the longest).  We then have
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   512
          // to wait for a cpu to become available before we can continue restarting
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   513
          // threads.
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   514
          // FIXME: This causes the performance of the VM to degrade when active and with
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   515
          // large numbers of threads.  Apparently this is due to the synchronous nature
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   516
          // of suspending threads.
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   517
          //
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   518
          // TODO-FIXME: the comments above are vestigial and no longer apply.
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   519
          // Furthermore, using solaris' schedctl in this particular context confers no benefit
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   520
          if (VMThreadHintNoPreempt) {
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   521
            os::hint_no_preempt();
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   522
          }
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   523
          ThreadSafepointState* cur_state = current->safepoint_state();
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   524
          assert(cur_state->type() != ThreadSafepointState::_running, "Thread not suspended at safepoint");
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   525
          cur_state->restart();
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   526
          assert(cur_state->is_running(), "safepoint state has not been reset");
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   527
        }
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   528
      }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   529
48105
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   530
      RuntimeService::record_safepoint_end();
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   531
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   532
      // Release threads lock, so threads can be created/destroyed again.
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   533
      // It will also release all threads blocked in signal_thread_blocked.
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   534
      Threads_lock->unlock();
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   535
    }
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   536
  } // ThreadsListHandle destroyed here.
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   537
47636
52d46d7e3454 8188919: Generalize GC thread suspend/resume at safepoints
eosterlund
parents: 47629
diff changeset
   538
  Universe::heap()->safepoint_synchronize_end();
22551
9bf46d16dcc6 8025856: Fix typos in the GC code
jwilhelm
parents: 20300
diff changeset
   539
  // record this time so VMThread can keep track how much time has elapsed
5042
f86707fd195a 6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents: 4006
diff changeset
   540
  // since last safepoint.
f86707fd195a 6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents: 4006
diff changeset
   541
  _end_of_last_safepoint = os::javaTimeMillis();
37176
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   542
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   543
  if (event.should_commit()) {
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   544
    event.set_safepointId(safepoint_id);
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   545
    event.commit();
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   546
  }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   547
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   548
489c9b5090e2 Initial load
duke
parents:
diff changeset
   549
bool SafepointSynchronize::is_cleanup_needed() {
46541
d20828de9e39 8181859: Monitor deflation is not checked in cleanup path
rehn
parents: 46496
diff changeset
   550
  // Need a safepoint if there are many monitors to deflate.
d20828de9e39 8181859: Monitor deflation is not checked in cleanup path
rehn
parents: 46496
diff changeset
   551
  if (ObjectSynchronizer::is_cleanup_needed()) return true;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   552
  // Need a safepoint if some inline cache buffers is non-empty
489c9b5090e2 Initial load
duke
parents:
diff changeset
   553
  if (!InlineCacheBuffer::is_empty()) return true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   554
  return false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   555
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   556
37176
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   557
static void event_safepoint_cleanup_task_commit(EventSafepointCleanupTask& event, const char* name) {
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   558
  if (event.should_commit()) {
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   559
    event.set_safepointId(SafepointSynchronize::safepoint_counter());
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   560
    event.set_name(name);
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   561
    event.commit();
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   562
  }
663bdc7d0b86 8149023: Event based tracing should cover safepoint begin and end
dsimms
parents: 37161
diff changeset
   563
}
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   564
46702
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   565
class ParallelSPCleanupThreadClosure : public ThreadClosure {
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   566
private:
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   567
  CodeBlobClosure* _nmethod_cl;
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   568
  DeflateMonitorCounters* _counters;
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   569
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   570
public:
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   571
  ParallelSPCleanupThreadClosure(DeflateMonitorCounters* counters) :
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   572
    _counters(counters),
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   573
    _nmethod_cl(NMethodSweeper::prepare_mark_active_nmethods()) {}
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   574
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   575
  void do_thread(Thread* thread) {
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   576
    ObjectSynchronizer::deflate_thread_local_monitors(thread, _counters);
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   577
    if (_nmethod_cl != NULL && thread->is_Java_thread() &&
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   578
        ! thread->is_Code_cache_sweeper_thread()) {
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   579
      JavaThread* jt = (JavaThread*) thread;
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   580
      jt->nmethods_do(_nmethod_cl);
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   581
    }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   582
  }
46702
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   583
};
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   584
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   585
class ParallelSPCleanupTask : public AbstractGangTask {
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   586
private:
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   587
  SubTasksDone _subtasks;
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   588
  ParallelSPCleanupThreadClosure _cleanup_threads_cl;
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   589
  uint _num_workers;
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   590
  DeflateMonitorCounters* _counters;
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   591
public:
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   592
  ParallelSPCleanupTask(uint num_workers, DeflateMonitorCounters* counters) :
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   593
    AbstractGangTask("Parallel Safepoint Cleanup"),
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   594
    _cleanup_threads_cl(ParallelSPCleanupThreadClosure(counters)),
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   595
    _num_workers(num_workers),
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   596
    _subtasks(SubTasksDone(SafepointSynchronize::SAFEPOINT_CLEANUP_NUM_TASKS)),
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   597
    _counters(counters) {}
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   598
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   599
  void work(uint worker_id) {
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   600
    // All threads deflate monitors and mark nmethods (if necessary).
47637
87141990dde5 8185580: Refactor Threads::possibly_parallel_oops_do() to use Threads::parallel_java_threads_do()
rkennke
parents: 47636
diff changeset
   601
    Threads::possibly_parallel_threads_do(true, &_cleanup_threads_cl);
46702
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   602
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   603
    if (!_subtasks.is_task_claimed(SafepointSynchronize::SAFEPOINT_CLEANUP_DEFLATE_MONITORS)) {
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   604
      const char* name = "deflating idle monitors";
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   605
      EventSafepointCleanupTask event;
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   606
      TraceTime timer(name, TRACETIME_LOG(Info, safepoint, cleanup));
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   607
      ObjectSynchronizer::deflate_idle_monitors(_counters);
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   608
      event_safepoint_cleanup_task_commit(event, name);
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   609
    }
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   610
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   611
    if (!_subtasks.is_task_claimed(SafepointSynchronize::SAFEPOINT_CLEANUP_UPDATE_INLINE_CACHES)) {
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   612
      const char* name = "updating inline caches";
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   613
      EventSafepointCleanupTask event;
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   614
      TraceTime timer(name, TRACETIME_LOG(Info, safepoint, cleanup));
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   615
      InlineCacheBuffer::update_inline_caches();
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   616
      event_safepoint_cleanup_task_commit(event, name);
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   617
    }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   618
46702
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   619
    if (!_subtasks.is_task_claimed(SafepointSynchronize::SAFEPOINT_CLEANUP_COMPILATION_POLICY)) {
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   620
      const char* name = "compilation policy safepoint handler";
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   621
      EventSafepointCleanupTask event;
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   622
      TraceTime timer(name, TRACETIME_LOG(Info, safepoint, cleanup));
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   623
      CompilationPolicy::policy()->do_safepoint_work();
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   624
      event_safepoint_cleanup_task_commit(event, name);
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   625
    }
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   626
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   627
    if (!_subtasks.is_task_claimed(SafepointSynchronize::SAFEPOINT_CLEANUP_SYMBOL_TABLE_REHASH)) {
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   628
      if (SymbolTable::needs_rehashing()) {
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   629
        const char* name = "rehashing symbol table";
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   630
        EventSafepointCleanupTask event;
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   631
        TraceTime timer(name, TRACETIME_LOG(Info, safepoint, cleanup));
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   632
        SymbolTable::rehash_table();
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   633
        event_safepoint_cleanup_task_commit(event, name);
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   634
      }
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   635
    }
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   636
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   637
    if (!_subtasks.is_task_claimed(SafepointSynchronize::SAFEPOINT_CLEANUP_STRING_TABLE_REHASH)) {
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   638
      if (StringTable::needs_rehashing()) {
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   639
        const char* name = "rehashing string table";
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   640
        EventSafepointCleanupTask event;
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   641
        TraceTime timer(name, TRACETIME_LOG(Info, safepoint, cleanup));
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   642
        StringTable::rehash_table();
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   643
        event_safepoint_cleanup_task_commit(event, name);
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   644
      }
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   645
    }
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   646
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   647
    if (!_subtasks.is_task_claimed(SafepointSynchronize::SAFEPOINT_CLEANUP_CLD_PURGE)) {
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   648
      // CMS delays purging the CLDG until the beginning of the next safepoint and to
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   649
      // make sure concurrent sweep is done
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   650
      const char* name = "purging class loader data graph";
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   651
      EventSafepointCleanupTask event;
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   652
      TraceTime timer(name, TRACETIME_LOG(Info, safepoint, cleanup));
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   653
      ClassLoaderDataGraph::purge_if_needed();
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   654
      event_safepoint_cleanup_task_commit(event, name);
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   655
    }
47774
69c081ca110a 8184765: Dynamically resize SystemDictionary
gziemski
parents: 47637
diff changeset
   656
69c081ca110a 8184765: Dynamically resize SystemDictionary
gziemski
parents: 47637
diff changeset
   657
    if (!_subtasks.is_task_claimed(SafepointSynchronize::SAFEPOINT_CLEANUP_SYSTEM_DICTIONARY_RESIZE)) {
69c081ca110a 8184765: Dynamically resize SystemDictionary
gziemski
parents: 47637
diff changeset
   658
      const char* name = "resizing system dictionaries";
69c081ca110a 8184765: Dynamically resize SystemDictionary
gziemski
parents: 47637
diff changeset
   659
      EventSafepointCleanupTask event;
69c081ca110a 8184765: Dynamically resize SystemDictionary
gziemski
parents: 47637
diff changeset
   660
      TraceTime timer(name, TRACETIME_LOG(Info, safepoint, cleanup));
69c081ca110a 8184765: Dynamically resize SystemDictionary
gziemski
parents: 47637
diff changeset
   661
      ClassLoaderDataGraph::resize_if_needed();
69c081ca110a 8184765: Dynamically resize SystemDictionary
gziemski
parents: 47637
diff changeset
   662
      event_safepoint_cleanup_task_commit(event, name);
69c081ca110a 8184765: Dynamically resize SystemDictionary
gziemski
parents: 47637
diff changeset
   663
    }
46702
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   664
    _subtasks.all_tasks_completed(_num_workers);
5042
f86707fd195a 6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents: 4006
diff changeset
   665
  }
46702
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   666
};
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   667
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   668
// Various cleaning tasks that should be done periodically at safepoints.
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   669
void SafepointSynchronize::do_cleanup_tasks() {
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   670
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   671
  TraceTime timer("safepoint cleanup tasks", TRACETIME_LOG(Info, safepoint, cleanup));
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   672
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   673
  // Prepare for monitor deflation.
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   674
  DeflateMonitorCounters deflate_counters;
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   675
  ObjectSynchronizer::prepare_deflate_idle_monitors(&deflate_counters);
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   676
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   677
  CollectedHeap* heap = Universe::heap();
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   678
  assert(heap != NULL, "heap not initialized yet?");
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   679
  WorkGang* cleanup_workers = heap->get_safepoint_workers();
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   680
  if (cleanup_workers != NULL) {
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   681
    // Parallel cleanup using GC provided thread pool.
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   682
    uint num_cleanup_workers = cleanup_workers->active_workers();
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   683
    ParallelSPCleanupTask cleanup(num_cleanup_workers, &deflate_counters);
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   684
    StrongRootsScope srs(num_cleanup_workers);
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   685
    cleanup_workers->run_task(&cleanup);
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   686
  } else {
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   687
    // Serial cleanup using VMThread.
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   688
    ParallelSPCleanupTask cleanup(1, &deflate_counters);
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   689
    StrongRootsScope srs(1);
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   690
    cleanup.work(0);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   691
  }
5042
f86707fd195a 6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents: 4006
diff changeset
   692
46702
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   693
  // Finish monitor deflation.
13ae789b982e 8180932: Parallelize safepoint cleanup
rkennke
parents: 46701
diff changeset
   694
  ObjectSynchronizer::finish_deflate_idle_monitors(&deflate_counters);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   695
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   696
489c9b5090e2 Initial load
duke
parents:
diff changeset
   697
489c9b5090e2 Initial load
duke
parents:
diff changeset
   698
bool SafepointSynchronize::safepoint_safe(JavaThread *thread, JavaThreadState state) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   699
  switch(state) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   700
  case _thread_in_native:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   701
    // native threads are safe if they have no java stack or have walkable stack
489c9b5090e2 Initial load
duke
parents:
diff changeset
   702
    return !thread->has_last_Java_frame() || thread->frame_anchor()->walkable();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   703
489c9b5090e2 Initial load
duke
parents:
diff changeset
   704
   // blocked threads should have already have walkable stack
489c9b5090e2 Initial load
duke
parents:
diff changeset
   705
  case _thread_blocked:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   706
    assert(!thread->has_last_Java_frame() || thread->frame_anchor()->walkable(), "blocked and not walkable");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   707
    return true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   708
489c9b5090e2 Initial load
duke
parents:
diff changeset
   709
  default:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   710
    return false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   711
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   712
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   713
489c9b5090e2 Initial load
duke
parents:
diff changeset
   714
11637
030466036615 7013347: allow crypto functions to be called inline to enhance performance
never
parents: 11631
diff changeset
   715
// See if the thread is running inside a lazy critical native and
030466036615 7013347: allow crypto functions to be called inline to enhance performance
never
parents: 11631
diff changeset
   716
// update the thread critical count if so.  Also set a suspend flag to
030466036615 7013347: allow crypto functions to be called inline to enhance performance
never
parents: 11631
diff changeset
   717
// cause the native wrapper to return into the JVM to do the unlock
030466036615 7013347: allow crypto functions to be called inline to enhance performance
never
parents: 11631
diff changeset
   718
// once the native finishes.
030466036615 7013347: allow crypto functions to be called inline to enhance performance
never
parents: 11631
diff changeset
   719
void SafepointSynchronize::check_for_lazy_critical_native(JavaThread *thread, JavaThreadState state) {
030466036615 7013347: allow crypto functions to be called inline to enhance performance
never
parents: 11631
diff changeset
   720
  if (state == _thread_in_native &&
030466036615 7013347: allow crypto functions to be called inline to enhance performance
never
parents: 11631
diff changeset
   721
      thread->has_last_Java_frame() &&
030466036615 7013347: allow crypto functions to be called inline to enhance performance
never
parents: 11631
diff changeset
   722
      thread->frame_anchor()->walkable()) {
030466036615 7013347: allow crypto functions to be called inline to enhance performance
never
parents: 11631
diff changeset
   723
    // This thread might be in a critical native nmethod so look at
030466036615 7013347: allow crypto functions to be called inline to enhance performance
never
parents: 11631
diff changeset
   724
    // the top of the stack and increment the critical count if it
030466036615 7013347: allow crypto functions to be called inline to enhance performance
never
parents: 11631
diff changeset
   725
    // is.
030466036615 7013347: allow crypto functions to be called inline to enhance performance
never
parents: 11631
diff changeset
   726
    frame wrapper_frame = thread->last_frame();
030466036615 7013347: allow crypto functions to be called inline to enhance performance
never
parents: 11631
diff changeset
   727
    CodeBlob* stub_cb = wrapper_frame.cb();
030466036615 7013347: allow crypto functions to be called inline to enhance performance
never
parents: 11631
diff changeset
   728
    if (stub_cb != NULL &&
030466036615 7013347: allow crypto functions to be called inline to enhance performance
never
parents: 11631
diff changeset
   729
        stub_cb->is_nmethod() &&
030466036615 7013347: allow crypto functions to be called inline to enhance performance
never
parents: 11631
diff changeset
   730
        stub_cb->as_nmethod_or_null()->is_lazy_critical_native()) {
030466036615 7013347: allow crypto functions to be called inline to enhance performance
never
parents: 11631
diff changeset
   731
      // A thread could potentially be in a critical native across
030466036615 7013347: allow crypto functions to be called inline to enhance performance
never
parents: 11631
diff changeset
   732
      // more than one safepoint, so only update the critical state on
030466036615 7013347: allow crypto functions to be called inline to enhance performance
never
parents: 11631
diff changeset
   733
      // the first one.  When it returns it will perform the unlock.
030466036615 7013347: allow crypto functions to be called inline to enhance performance
never
parents: 11631
diff changeset
   734
      if (!thread->do_critical_native_unlock()) {
030466036615 7013347: allow crypto functions to be called inline to enhance performance
never
parents: 11631
diff changeset
   735
#ifdef ASSERT
030466036615 7013347: allow crypto functions to be called inline to enhance performance
never
parents: 11631
diff changeset
   736
        if (!thread->in_critical()) {
35492
c8c0273e6b91 8146690: Make all classes in GC follow the naming convention.
david
parents: 35061
diff changeset
   737
          GCLocker::increment_debug_jni_lock_count();
11637
030466036615 7013347: allow crypto functions to be called inline to enhance performance
never
parents: 11631
diff changeset
   738
        }
030466036615 7013347: allow crypto functions to be called inline to enhance performance
never
parents: 11631
diff changeset
   739
#endif
030466036615 7013347: allow crypto functions to be called inline to enhance performance
never
parents: 11631
diff changeset
   740
        thread->enter_critical();
030466036615 7013347: allow crypto functions to be called inline to enhance performance
never
parents: 11631
diff changeset
   741
        // Make sure the native wrapper calls back on return to
030466036615 7013347: allow crypto functions to be called inline to enhance performance
never
parents: 11631
diff changeset
   742
        // perform the needed critical unlock.
030466036615 7013347: allow crypto functions to be called inline to enhance performance
never
parents: 11631
diff changeset
   743
        thread->set_critical_native_unlock();
030466036615 7013347: allow crypto functions to be called inline to enhance performance
never
parents: 11631
diff changeset
   744
      }
030466036615 7013347: allow crypto functions to be called inline to enhance performance
never
parents: 11631
diff changeset
   745
    }
030466036615 7013347: allow crypto functions to be called inline to enhance performance
never
parents: 11631
diff changeset
   746
  }
030466036615 7013347: allow crypto functions to be called inline to enhance performance
never
parents: 11631
diff changeset
   747
}
030466036615 7013347: allow crypto functions to be called inline to enhance performance
never
parents: 11631
diff changeset
   748
030466036615 7013347: allow crypto functions to be called inline to enhance performance
never
parents: 11631
diff changeset
   749
030466036615 7013347: allow crypto functions to be called inline to enhance performance
never
parents: 11631
diff changeset
   750
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   751
// -------------------------------------------------------------------------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   752
// Implementation of Safepoint callback point
489c9b5090e2 Initial load
duke
parents:
diff changeset
   753
489c9b5090e2 Initial load
duke
parents:
diff changeset
   754
void SafepointSynchronize::block(JavaThread *thread) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   755
  assert(thread != NULL, "thread must be set");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   756
  assert(thread->is_Java_thread(), "not a Java thread");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   757
489c9b5090e2 Initial load
duke
parents:
diff changeset
   758
  // Threads shouldn't block if they are in the middle of printing, but...
489c9b5090e2 Initial load
duke
parents:
diff changeset
   759
  ttyLocker::break_tty_lock_for_safepoint(os::current_thread_id());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   760
489c9b5090e2 Initial load
duke
parents:
diff changeset
   761
  // Only bail from the block() call if the thread is gone from the
489c9b5090e2 Initial load
duke
parents:
diff changeset
   762
  // thread list; starting to exit should still block.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   763
  if (thread->is_terminated()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   764
     // block current thread if we come here from native code when VM is gone
489c9b5090e2 Initial load
duke
parents:
diff changeset
   765
     thread->block_if_vm_exited();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   766
489c9b5090e2 Initial load
duke
parents:
diff changeset
   767
     // otherwise do nothing
489c9b5090e2 Initial load
duke
parents:
diff changeset
   768
     return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   769
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   770
489c9b5090e2 Initial load
duke
parents:
diff changeset
   771
  JavaThreadState state = thread->thread_state();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   772
  thread->frame_anchor()->make_walkable(thread);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   773
489c9b5090e2 Initial load
duke
parents:
diff changeset
   774
  // Check that we have a valid thread_state at this point
489c9b5090e2 Initial load
duke
parents:
diff changeset
   775
  switch(state) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   776
    case _thread_in_vm_trans:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   777
    case _thread_in_Java:        // From compiled code
489c9b5090e2 Initial load
duke
parents:
diff changeset
   778
489c9b5090e2 Initial load
duke
parents:
diff changeset
   779
      // We are highly likely to block on the Safepoint_lock. In order to avoid blocking in this case,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   780
      // we pretend we are still in the VM.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   781
      thread->set_thread_state(_thread_in_vm);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   782
489c9b5090e2 Initial load
duke
parents:
diff changeset
   783
      if (is_synchronizing()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   784
         Atomic::inc (&TryingToBlock) ;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   785
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   786
489c9b5090e2 Initial load
duke
parents:
diff changeset
   787
      // We will always be holding the Safepoint_lock when we are examine the state
489c9b5090e2 Initial load
duke
parents:
diff changeset
   788
      // of a thread. Hence, the instructions between the Safepoint_lock->lock() and
489c9b5090e2 Initial load
duke
parents:
diff changeset
   789
      // Safepoint_lock->unlock() are happening atomic with regards to the safepoint code
489c9b5090e2 Initial load
duke
parents:
diff changeset
   790
      Safepoint_lock->lock_without_safepoint_check();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   791
      if (is_synchronizing()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   792
        // Decrement the number of threads to wait for and signal vm thread
489c9b5090e2 Initial load
duke
parents:
diff changeset
   793
        assert(_waiting_to_block > 0, "sanity check");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   794
        _waiting_to_block--;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   795
        thread->safepoint_state()->set_has_called_back(true);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   796
12108
b2ee55606008 7150390: JFR test crashed on assert(_jni_lock_count == count) failed: must be equal
never
parents: 11637
diff changeset
   797
        DEBUG_ONLY(thread->set_visited_for_critical_count(true));
11631
33813f69207b 7129164: JNI Get/ReleasePrimitiveArrayCritical doesn't scale
never
parents: 10565
diff changeset
   798
        if (thread->in_critical()) {
33813f69207b 7129164: JNI Get/ReleasePrimitiveArrayCritical doesn't scale
never
parents: 10565
diff changeset
   799
          // Notice that this thread is in a critical section
33813f69207b 7129164: JNI Get/ReleasePrimitiveArrayCritical doesn't scale
never
parents: 10565
diff changeset
   800
          increment_jni_active_count();
33813f69207b 7129164: JNI Get/ReleasePrimitiveArrayCritical doesn't scale
never
parents: 10565
diff changeset
   801
        }
33813f69207b 7129164: JNI Get/ReleasePrimitiveArrayCritical doesn't scale
never
parents: 10565
diff changeset
   802
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   803
        // Consider (_waiting_to_block < 2) to pipeline the wakeup of the VM thread
489c9b5090e2 Initial load
duke
parents:
diff changeset
   804
        if (_waiting_to_block == 0) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   805
          Safepoint_lock->notify_all();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   806
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   807
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   808
489c9b5090e2 Initial load
duke
parents:
diff changeset
   809
      // We transition the thread to state _thread_blocked here, but
489c9b5090e2 Initial load
duke
parents:
diff changeset
   810
      // we can't do our usual check for external suspension and then
489c9b5090e2 Initial load
duke
parents:
diff changeset
   811
      // self-suspend after the lock_without_safepoint_check() call
489c9b5090e2 Initial load
duke
parents:
diff changeset
   812
      // below because we are often called during transitions while
489c9b5090e2 Initial load
duke
parents:
diff changeset
   813
      // we hold different locks. That would leave us suspended while
489c9b5090e2 Initial load
duke
parents:
diff changeset
   814
      // holding a resource which results in deadlocks.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   815
      thread->set_thread_state(_thread_blocked);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   816
      Safepoint_lock->unlock();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   817
489c9b5090e2 Initial load
duke
parents:
diff changeset
   818
      // We now try to acquire the threads lock. Since this lock is hold by the VM thread during
489c9b5090e2 Initial load
duke
parents:
diff changeset
   819
      // the entire safepoint, the threads will all line up here during the safepoint.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   820
      Threads_lock->lock_without_safepoint_check();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   821
      // restore original state. This is important if the thread comes from compiled code, so it
489c9b5090e2 Initial load
duke
parents:
diff changeset
   822
      // will continue to execute with the _thread_in_Java state.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   823
      thread->set_thread_state(state);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   824
      Threads_lock->unlock();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   825
      break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   826
489c9b5090e2 Initial load
duke
parents:
diff changeset
   827
    case _thread_in_native_trans:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   828
    case _thread_blocked_trans:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   829
    case _thread_new_trans:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   830
      if (thread->safepoint_state()->type() == ThreadSafepointState::_call_back) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   831
        thread->print_thread_state();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   832
        fatal("Deadlock in safepoint code.  "
489c9b5090e2 Initial load
duke
parents:
diff changeset
   833
              "Should have called back to the VM before blocking.");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   834
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   835
489c9b5090e2 Initial load
duke
parents:
diff changeset
   836
      // We transition the thread to state _thread_blocked here, but
489c9b5090e2 Initial load
duke
parents:
diff changeset
   837
      // we can't do our usual check for external suspension and then
489c9b5090e2 Initial load
duke
parents:
diff changeset
   838
      // self-suspend after the lock_without_safepoint_check() call
489c9b5090e2 Initial load
duke
parents:
diff changeset
   839
      // below because we are often called during transitions while
489c9b5090e2 Initial load
duke
parents:
diff changeset
   840
      // we hold different locks. That would leave us suspended while
489c9b5090e2 Initial load
duke
parents:
diff changeset
   841
      // holding a resource which results in deadlocks.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   842
      thread->set_thread_state(_thread_blocked);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   843
489c9b5090e2 Initial load
duke
parents:
diff changeset
   844
      // It is not safe to suspend a thread if we discover it is in _thread_in_native_trans. Hence,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   845
      // the safepoint code might still be waiting for it to block. We need to change the state here,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   846
      // so it can see that it is at a safepoint.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   847
489c9b5090e2 Initial load
duke
parents:
diff changeset
   848
      // Block until the safepoint operation is completed.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   849
      Threads_lock->lock_without_safepoint_check();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   850
489c9b5090e2 Initial load
duke
parents:
diff changeset
   851
      // Restore state
489c9b5090e2 Initial load
duke
parents:
diff changeset
   852
      thread->set_thread_state(state);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   853
489c9b5090e2 Initial load
duke
parents:
diff changeset
   854
      Threads_lock->unlock();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   855
      break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   856
489c9b5090e2 Initial load
duke
parents:
diff changeset
   857
    default:
33105
294e48b4f704 8080775: Better argument formatting for assert() and friends
david
parents: 31592
diff changeset
   858
     fatal("Illegal threadstate encountered: %d", state);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   859
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   860
489c9b5090e2 Initial load
duke
parents:
diff changeset
   861
  // Check for pending. async. exceptions or suspends - except if the
489c9b5090e2 Initial load
duke
parents:
diff changeset
   862
  // thread was blocked inside the VM. has_special_runtime_exit_condition()
489c9b5090e2 Initial load
duke
parents:
diff changeset
   863
  // is called last since it grabs a lock and we only want to do that when
489c9b5090e2 Initial load
duke
parents:
diff changeset
   864
  // we must.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   865
  //
489c9b5090e2 Initial load
duke
parents:
diff changeset
   866
  // Note: we never deliver an async exception at a polling point as the
489c9b5090e2 Initial load
duke
parents:
diff changeset
   867
  // compiler may not have an exception handler for it. The polling
489c9b5090e2 Initial load
duke
parents:
diff changeset
   868
  // code will notice the async and deoptimize and the exception will
489c9b5090e2 Initial load
duke
parents:
diff changeset
   869
  // be delivered. (Polling at a return point is ok though). Sure is
489c9b5090e2 Initial load
duke
parents:
diff changeset
   870
  // a lot of bother for a deprecated feature...
489c9b5090e2 Initial load
duke
parents:
diff changeset
   871
  //
489c9b5090e2 Initial load
duke
parents:
diff changeset
   872
  // We don't deliver an async exception if the thread state is
489c9b5090e2 Initial load
duke
parents:
diff changeset
   873
  // _thread_in_native_trans so JNI functions won't be called with
489c9b5090e2 Initial load
duke
parents:
diff changeset
   874
  // a surprising pending exception. If the thread state is going back to java,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   875
  // async exception is checked in check_special_condition_for_native_trans().
489c9b5090e2 Initial load
duke
parents:
diff changeset
   876
489c9b5090e2 Initial load
duke
parents:
diff changeset
   877
  if (state != _thread_blocked_trans &&
489c9b5090e2 Initial load
duke
parents:
diff changeset
   878
      state != _thread_in_vm_trans &&
489c9b5090e2 Initial load
duke
parents:
diff changeset
   879
      thread->has_special_runtime_exit_condition()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   880
    thread->handle_special_runtime_exit_condition(
489c9b5090e2 Initial load
duke
parents:
diff changeset
   881
      !thread->is_at_poll_safepoint() && (state != _thread_in_native_trans));
489c9b5090e2 Initial load
duke
parents:
diff changeset
   882
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   883
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   884
489c9b5090e2 Initial load
duke
parents:
diff changeset
   885
// ------------------------------------------------------------------------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   886
// Exception handlers
489c9b5090e2 Initial load
duke
parents:
diff changeset
   887
489c9b5090e2 Initial load
duke
parents:
diff changeset
   888
489c9b5090e2 Initial load
duke
parents:
diff changeset
   889
void SafepointSynchronize::handle_polling_page_exception(JavaThread *thread) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   890
  assert(thread->is_Java_thread(), "polling reference encountered by VM thread");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   891
  assert(thread->thread_state() == _thread_in_Java, "should come from Java code");
47881
0ce0ac68ace7 8189941: Implementation JEP 312: Thread-local handshake
rehn
parents: 47774
diff changeset
   892
  if (!ThreadLocalHandshakes) {
0ce0ac68ace7 8189941: Implementation JEP 312: Thread-local handshake
rehn
parents: 47774
diff changeset
   893
    assert(SafepointSynchronize::is_synchronizing(), "polling encountered outside safepoint synchronization");
0ce0ac68ace7 8189941: Implementation JEP 312: Thread-local handshake
rehn
parents: 47774
diff changeset
   894
  }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   895
489c9b5090e2 Initial load
duke
parents:
diff changeset
   896
  if (PrintSafepointStatistics) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   897
    inc_page_trap_count();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   898
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   899
489c9b5090e2 Initial load
duke
parents:
diff changeset
   900
  ThreadSafepointState* state = thread->safepoint_state();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   901
489c9b5090e2 Initial load
duke
parents:
diff changeset
   902
  state->handle_polling_page_exception();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   903
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   904
489c9b5090e2 Initial load
duke
parents:
diff changeset
   905
489c9b5090e2 Initial load
duke
parents:
diff changeset
   906
void SafepointSynchronize::print_safepoint_timeout(SafepointTimeoutReason reason) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   907
  if (!timeout_error_printed) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   908
    timeout_error_printed = true;
22551
9bf46d16dcc6 8025856: Fix typos in the GC code
jwilhelm
parents: 20300
diff changeset
   909
    // Print out the thread info which didn't reach the safepoint for debugging
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   910
    // purposes (useful when there are lots of threads in the debugger).
24424
2658d7834c6e 8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents: 24351
diff changeset
   911
    tty->cr();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   912
    tty->print_cr("# SafepointSynchronize::begin: Timeout detected:");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   913
    if (reason ==  _spinning_timeout) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   914
      tty->print_cr("# SafepointSynchronize::begin: Timed out while spinning to reach a safepoint.");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   915
    } else if (reason == _blocking_timeout) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   916
      tty->print_cr("# SafepointSynchronize::begin: Timed out while waiting for threads to stop.");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   917
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   918
489c9b5090e2 Initial load
duke
parents:
diff changeset
   919
    tty->print_cr("# SafepointSynchronize::begin: Threads which did not reach the safepoint:");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   920
    ThreadSafepointState *cur_state;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   921
    ResourceMark rm;
48105
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   922
    for (JavaThreadIteratorWithHandle jtiwh; JavaThread *cur_thread = jtiwh.next(); ) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   923
      cur_state = cur_thread->safepoint_state();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   924
489c9b5090e2 Initial load
duke
parents:
diff changeset
   925
      if (cur_thread->thread_state() != _thread_blocked &&
48105
8d15b1369c7a 8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents: 47881
diff changeset
   926
        ((reason == _spinning_timeout && cur_state->is_running()) ||
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   927
           (reason == _blocking_timeout && !cur_state->has_called_back()))) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   928
        tty->print("# ");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   929
        cur_thread->print();
24424
2658d7834c6e 8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents: 24351
diff changeset
   930
        tty->cr();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   931
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   932
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   933
    tty->print_cr("# SafepointSynchronize::begin: (End of list)");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   934
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   935
489c9b5090e2 Initial load
duke
parents:
diff changeset
   936
  // To debug the long safepoint, specify both DieOnSafepointTimeout &
489c9b5090e2 Initial load
duke
parents:
diff changeset
   937
  // ShowMessageBoxOnError.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   938
  if (DieOnSafepointTimeout) {
33105
294e48b4f704 8080775: Better argument formatting for assert() and friends
david
parents: 31592
diff changeset
   939
    fatal("Safepoint sync time longer than " INTX_FORMAT "ms detected when executing %s.",
46496
76ed99d51a67 8152955: Many safepoints of "no vm operation" kind
rehn
parents: 46473
diff changeset
   940
          SafepointTimeoutDelay, VMThread::vm_safepoint_description());
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   941
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   942
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   943
489c9b5090e2 Initial load
duke
parents:
diff changeset
   944
489c9b5090e2 Initial load
duke
parents:
diff changeset
   945
// -------------------------------------------------------------------------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   946
// Implementation of ThreadSafepointState
489c9b5090e2 Initial load
duke
parents:
diff changeset
   947
489c9b5090e2 Initial load
duke
parents:
diff changeset
   948
ThreadSafepointState::ThreadSafepointState(JavaThread *thread) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   949
  _thread = thread;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   950
  _type   = _running;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   951
  _has_called_back = false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   952
  _at_poll_safepoint = false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   953
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   954
489c9b5090e2 Initial load
duke
parents:
diff changeset
   955
void ThreadSafepointState::create(JavaThread *thread) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   956
  ThreadSafepointState *state = new ThreadSafepointState(thread);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   957
  thread->set_safepoint_state(state);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   958
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   959
489c9b5090e2 Initial load
duke
parents:
diff changeset
   960
void ThreadSafepointState::destroy(JavaThread *thread) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   961
  if (thread->safepoint_state()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   962
    delete(thread->safepoint_state());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   963
    thread->set_safepoint_state(NULL);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   964
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   965
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   966
489c9b5090e2 Initial load
duke
parents:
diff changeset
   967
void ThreadSafepointState::examine_state_of_thread() {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   968
  assert(is_running(), "better be running or just have hit safepoint poll");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   969
489c9b5090e2 Initial load
duke
parents:
diff changeset
   970
  JavaThreadState state = _thread->thread_state();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   971
6269
10e06287c0b0 6975006: assert(check.is_deoptimized_frame()) failed: missed deopt
never
parents: 5547
diff changeset
   972
  // Save the state at the start of safepoint processing.
10e06287c0b0 6975006: assert(check.is_deoptimized_frame()) failed: missed deopt
never
parents: 5547
diff changeset
   973
  _orig_thread_state = state;
10e06287c0b0 6975006: assert(check.is_deoptimized_frame()) failed: missed deopt
never
parents: 5547
diff changeset
   974
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   975
  // Check for a thread that is suspended. Note that thread resume tries
489c9b5090e2 Initial load
duke
parents:
diff changeset
   976
  // to grab the Threads_lock which we own here, so a thread cannot be
489c9b5090e2 Initial load
duke
parents:
diff changeset
   977
  // resumed during safepoint synchronization.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   978
3826
67b89f5a5cac 6876794: 4/4 sp07t002 hangs very intermittently
dcubed
parents: 3262
diff changeset
   979
  // We check to see if this thread is suspended without locking to
67b89f5a5cac 6876794: 4/4 sp07t002 hangs very intermittently
dcubed
parents: 3262
diff changeset
   980
  // avoid deadlocking with a third thread that is waiting for this
67b89f5a5cac 6876794: 4/4 sp07t002 hangs very intermittently
dcubed
parents: 3262
diff changeset
   981
  // thread to be suspended. The third thread can notice the safepoint
67b89f5a5cac 6876794: 4/4 sp07t002 hangs very intermittently
dcubed
parents: 3262
diff changeset
   982
  // that we're trying to start at the beginning of its SR_lock->wait()
67b89f5a5cac 6876794: 4/4 sp07t002 hangs very intermittently
dcubed
parents: 3262
diff changeset
   983
  // call. If that happens, then the third thread will block on the
67b89f5a5cac 6876794: 4/4 sp07t002 hangs very intermittently
dcubed
parents: 3262
diff changeset
   984
  // safepoint while still holding the underlying SR_lock. We won't be
67b89f5a5cac 6876794: 4/4 sp07t002 hangs very intermittently
dcubed
parents: 3262
diff changeset
   985
  // able to get the SR_lock and we'll deadlock.
67b89f5a5cac 6876794: 4/4 sp07t002 hangs very intermittently
dcubed
parents: 3262
diff changeset
   986
  //
67b89f5a5cac 6876794: 4/4 sp07t002 hangs very intermittently
dcubed
parents: 3262
diff changeset
   987
  // We don't need to grab the SR_lock here for two reasons:
67b89f5a5cac 6876794: 4/4 sp07t002 hangs very intermittently
dcubed
parents: 3262
diff changeset
   988
  // 1) The suspend flags are both volatile and are set with an
67b89f5a5cac 6876794: 4/4 sp07t002 hangs very intermittently
dcubed
parents: 3262
diff changeset
   989
  //    Atomic::cmpxchg() call so we should see the suspended
67b89f5a5cac 6876794: 4/4 sp07t002 hangs very intermittently
dcubed
parents: 3262
diff changeset
   990
  //    state right away.
67b89f5a5cac 6876794: 4/4 sp07t002 hangs very intermittently
dcubed
parents: 3262
diff changeset
   991
  // 2) We're being called from the safepoint polling loop; if
67b89f5a5cac 6876794: 4/4 sp07t002 hangs very intermittently
dcubed
parents: 3262
diff changeset
   992
  //    we don't see the suspended state on this iteration, then
67b89f5a5cac 6876794: 4/4 sp07t002 hangs very intermittently
dcubed
parents: 3262
diff changeset
   993
  //    we'll come around again.
67b89f5a5cac 6876794: 4/4 sp07t002 hangs very intermittently
dcubed
parents: 3262
diff changeset
   994
  //
67b89f5a5cac 6876794: 4/4 sp07t002 hangs very intermittently
dcubed
parents: 3262
diff changeset
   995
  bool is_suspended = _thread->is_ext_suspended();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   996
  if (is_suspended) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   997
    roll_forward(_at_safepoint);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   998
    return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   999
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1000
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1001
  // Some JavaThread states have an initial safepoint state of
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1002
  // running, but are actually at a safepoint. We will happily
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1003
  // agree and update the safepoint state here.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1004
  if (SafepointSynchronize::safepoint_safe(_thread, state)) {
12108
b2ee55606008 7150390: JFR test crashed on assert(_jni_lock_count == count) failed: must be equal
never
parents: 11637
diff changeset
  1005
    SafepointSynchronize::check_for_lazy_critical_native(_thread, state);
11631
33813f69207b 7129164: JNI Get/ReleasePrimitiveArrayCritical doesn't scale
never
parents: 10565
diff changeset
  1006
    roll_forward(_at_safepoint);
33813f69207b 7129164: JNI Get/ReleasePrimitiveArrayCritical doesn't scale
never
parents: 10565
diff changeset
  1007
    return;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1008
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1009
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1010
  if (state == _thread_in_vm) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1011
    roll_forward(_call_back);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1012
    return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1013
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1014
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1015
  // All other thread states will continue to run until they
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1016
  // transition and self-block in state _blocked
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1017
  // Safepoint polling in compiled code causes the Java threads to do the same.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1018
  // Note: new threads may require a malloc so they must be allowed to finish
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1019
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1020
  assert(is_running(), "examine_state_of_thread on non-running thread");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1021
  return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1022
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1023
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1024
// Returns true is thread could not be rolled forward at present position.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1025
void ThreadSafepointState::roll_forward(suspend_type type) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1026
  _type = type;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1027
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1028
  switch(_type) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1029
    case _at_safepoint:
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1030
      SafepointSynchronize::signal_thread_at_safepoint();
12108
b2ee55606008 7150390: JFR test crashed on assert(_jni_lock_count == count) failed: must be equal
never
parents: 11637
diff changeset
  1031
      DEBUG_ONLY(_thread->set_visited_for_critical_count(true));
b2ee55606008 7150390: JFR test crashed on assert(_jni_lock_count == count) failed: must be equal
never
parents: 11637
diff changeset
  1032
      if (_thread->in_critical()) {
b2ee55606008 7150390: JFR test crashed on assert(_jni_lock_count == count) failed: must be equal
never
parents: 11637
diff changeset
  1033
        // Notice that this thread is in a critical section
b2ee55606008 7150390: JFR test crashed on assert(_jni_lock_count == count) failed: must be equal
never
parents: 11637
diff changeset
  1034
        SafepointSynchronize::increment_jni_active_count();
b2ee55606008 7150390: JFR test crashed on assert(_jni_lock_count == count) failed: must be equal
never
parents: 11637
diff changeset
  1035
      }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1036
      break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1037
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1038
    case _call_back:
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1039
      set_has_called_back(false);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1040
      break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1041
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1042
    case _running:
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1043
    default:
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1044
      ShouldNotReachHere();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1045
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1046
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1047
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1048
void ThreadSafepointState::restart() {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1049
  switch(type()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1050
    case _at_safepoint:
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1051
    case _call_back:
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1052
      break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1053
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1054
    case _running:
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1055
    default:
31592
43f48e165466 8081202: Hotspot compile warning: "Invalid suffix on literal; C++11 requires a space between literal and identifier"
bpittore
parents: 30764
diff changeset
  1056
       tty->print_cr("restart thread " INTPTR_FORMAT " with state %d",
33148
68fa8b6c4340 8042893: compiler: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents: 33105
diff changeset
  1057
                     p2i(_thread), _type);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1058
       _thread->print();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1059
      ShouldNotReachHere();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1060
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1061
  _type = _running;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1062
  set_has_called_back(false);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1063
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1064
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1065
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1066
void ThreadSafepointState::print_on(outputStream *st) const {
33589
7cbd1b2c139b 8139040: Fix initializations before ShouldNotReachHere() etc. and enable -Wuninitialized on linux.
goetz
parents: 33222
diff changeset
  1067
  const char *s = NULL;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1068
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1069
  switch(_type) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1070
    case _running                : s = "_running";              break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1071
    case _at_safepoint           : s = "_at_safepoint";         break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1072
    case _call_back              : s = "_call_back";            break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1073
    default:
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1074
      ShouldNotReachHere();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1075
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1076
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1077
  st->print_cr("Thread: " INTPTR_FORMAT
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1078
              "  [0x%2x] State: %s _has_called_back %d _at_poll_safepoint %d",
33148
68fa8b6c4340 8042893: compiler: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents: 33105
diff changeset
  1079
               p2i(_thread), _thread->osthread()->thread_id(), s, _has_called_back,
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1080
               _at_poll_safepoint);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1081
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1082
  _thread->print_thread_state_on(st);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1083
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1084
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1085
// ---------------------------------------------------------------------------------------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1086
47881
0ce0ac68ace7 8189941: Implementation JEP 312: Thread-local handshake
rehn
parents: 47774
diff changeset
  1087
// Block the thread at poll or poll return for safepoint/handshake.
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1088
void ThreadSafepointState::handle_polling_page_exception() {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1089
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1090
  // Check state.  block() will set thread state to thread_in_vm which will
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1091
  // cause the safepoint state _type to become _call_back.
47881
0ce0ac68ace7 8189941: Implementation JEP 312: Thread-local handshake
rehn
parents: 47774
diff changeset
  1092
  suspend_type t = type();
0ce0ac68ace7 8189941: Implementation JEP 312: Thread-local handshake
rehn
parents: 47774
diff changeset
  1093
  assert(!SafepointMechanism::uses_global_page_poll() || t == ThreadSafepointState::_running,
0ce0ac68ace7 8189941: Implementation JEP 312: Thread-local handshake
rehn
parents: 47774
diff changeset
  1094
         "polling page exception on thread not running state: %u", uint(t));
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1095
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1096
  // Step 1: Find the nmethod from the return address
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1097
  address real_return_addr = thread()->saved_exception_pc();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1098
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1099
  CodeBlob *cb = CodeCache::find_blob(real_return_addr);
38133
78b95467b9f1 8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents: 37242
diff changeset
  1100
  assert(cb != NULL && cb->is_compiled(), "return address should be in nmethod");
78b95467b9f1 8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents: 37242
diff changeset
  1101
  CompiledMethod* nm = (CompiledMethod*)cb;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1102
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1103
  // Find frame of caller
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1104
  frame stub_fr = thread()->last_frame();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1105
  CodeBlob* stub_cb = stub_fr.cb();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1106
  assert(stub_cb->is_safepoint_stub(), "must be a safepoint stub");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1107
  RegisterMap map(thread(), true);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1108
  frame caller_fr = stub_fr.sender(&map);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1109
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1110
  // Should only be poll_return or poll
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1111
  assert( nm->is_at_poll_or_poll_return(real_return_addr), "should not be at call" );
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1112
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1113
  // This is a poll immediately before a return. The exception handling code
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1114
  // has already had the effect of causing the return to occur, so the execution
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1115
  // will continue immediately after the call. In addition, the oopmap at the
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1116
  // return point does not mark the return value as an oop (if it is), so
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1117
  // it needs a handle here to be updated.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1118
  if( nm->is_at_poll_return(real_return_addr) ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1119
    // See if return type is an oop.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1120
    bool return_oop = nm->method()->is_returning_oop();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1121
    Handle return_value;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1122
    if (return_oop) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1123
      // The oop result has been saved on the stack together with all
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1124
      // the other registers. In order to preserve it over GCs we need
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1125
      // to keep it in a handle.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1126
      oop result = caller_fr.saved_oop_result(&map);
46968
9119841280f4 8160399: is_oop_or_null involves undefined behavior
coleenp
parents: 46702
diff changeset
  1127
      assert(oopDesc::is_oop_or_null(result), "must be oop");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1128
      return_value = Handle(thread(), result);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1129
      assert(Universe::heap()->is_in_or_null(result), "must be heap pointer");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1130
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1131
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1132
    // Block the thread
47881
0ce0ac68ace7 8189941: Implementation JEP 312: Thread-local handshake
rehn
parents: 47774
diff changeset
  1133
    SafepointMechanism::block_if_requested(thread());
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1134
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1135
    // restore oop result, if any
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1136
    if (return_oop) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1137
      caller_fr.set_saved_oop_result(&map, return_value());
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1138
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1139
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1140
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1141
  // This is a safepoint poll. Verify the return address and block.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1142
  else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1143
    set_at_poll_safepoint(true);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1144
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1145
    // verify the blob built the "return address" correctly
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1146
    assert(real_return_addr == caller_fr.pc(), "must match");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1147
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1148
    // Block the thread
47881
0ce0ac68ace7 8189941: Implementation JEP 312: Thread-local handshake
rehn
parents: 47774
diff changeset
  1149
    SafepointMechanism::block_if_requested(thread());
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1150
    set_at_poll_safepoint(false);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1151
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1152
    // If we have a pending async exception deoptimize the frame
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1153
    // as otherwise we may never deliver it.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1154
    if (thread()->has_async_condition()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1155
      ThreadInVMfromJavaNoAsyncException __tiv(thread());
7106
867c9d296c6b 6968367: can_post_on_exceptions is still using VM_DeoptimizeFrame in some places
never
parents: 6453
diff changeset
  1156
      Deoptimization::deoptimize_frame(thread(), caller_fr.id());
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1157
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1158
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1159
    // If an exception has been installed we must check for a pending deoptimization
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1160
    // Deoptimize frame if exception has been thrown.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1161
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1162
    if (thread()->has_pending_exception() ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1163
      RegisterMap map(thread(), true);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1164
      frame caller_fr = stub_fr.sender(&map);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1165
      if (caller_fr.is_deoptimized_frame()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1166
        // The exception patch will destroy registers that are still
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1167
        // live and will be needed during deoptimization. Defer the
22551
9bf46d16dcc6 8025856: Fix typos in the GC code
jwilhelm
parents: 20300
diff changeset
  1168
        // Async exception should have deferred the exception until the
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1169
        // next safepoint which will be detected when we get into
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1170
        // the interpreter so if we have an exception now things
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1171
        // are messed up.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1172
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1173
        fatal("Exception installed and deoptimization is pending");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1174
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1175
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1176
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1177
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1178
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1179
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1180
//
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1181
//                     Statistics & Instrumentations
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1182
//
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1183
SafepointSynchronize::SafepointStats*  SafepointSynchronize::_safepoint_stats = NULL;
5042
f86707fd195a 6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents: 4006
diff changeset
  1184
jlong  SafepointSynchronize::_safepoint_begin_time = 0;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1185
int    SafepointSynchronize::_cur_stat_index = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1186
julong SafepointSynchronize::_safepoint_reasons[VM_Operation::VMOp_Terminating];
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1187
julong SafepointSynchronize::_coalesced_vmop_count = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1188
jlong  SafepointSynchronize::_max_sync_time = 0;
5042
f86707fd195a 6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents: 4006
diff changeset
  1189
jlong  SafepointSynchronize::_max_vmop_time = 0;
f86707fd195a 6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents: 4006
diff changeset
  1190
float  SafepointSynchronize::_ts_of_current_safepoint = 0.0f;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1191
5042
f86707fd195a 6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents: 4006
diff changeset
  1192
static jlong  cleanup_end_time = 0;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1193
static bool   need_to_track_page_armed_status = false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1194
static bool   init_done = false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1195
5042
f86707fd195a 6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents: 4006
diff changeset
  1196
// Helper method to print the header.
f86707fd195a 6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents: 4006
diff changeset
  1197
static void print_header() {
46473
d43055e806c9 8180482: Reformat -XX:+PrintSafepointStatistics table
shade
parents: 40664
diff changeset
  1198
  // The number of spaces is significant here, and should match the format
d43055e806c9 8180482: Reformat -XX:+PrintSafepointStatistics table
shade
parents: 40664
diff changeset
  1199
  // specifiers in print_statistics().
d43055e806c9 8180482: Reformat -XX:+PrintSafepointStatistics table
shade
parents: 40664
diff changeset
  1200
d43055e806c9 8180482: Reformat -XX:+PrintSafepointStatistics table
shade
parents: 40664
diff changeset
  1201
  tty->print("          vmop                            "
d43055e806c9 8180482: Reformat -XX:+PrintSafepointStatistics table
shade
parents: 40664
diff changeset
  1202
             "[ threads:    total initially_running wait_to_block ]"
d43055e806c9 8180482: Reformat -XX:+PrintSafepointStatistics table
shade
parents: 40664
diff changeset
  1203
             "[ time:    spin   block    sync cleanup    vmop ] ");
5042
f86707fd195a 6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents: 4006
diff changeset
  1204
f86707fd195a 6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents: 4006
diff changeset
  1205
  // no page armed status printed out if it is always armed.
f86707fd195a 6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents: 4006
diff changeset
  1206
  if (need_to_track_page_armed_status) {
f86707fd195a 6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents: 4006
diff changeset
  1207
    tty->print("page_armed ");
f86707fd195a 6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents: 4006
diff changeset
  1208
  }
f86707fd195a 6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents: 4006
diff changeset
  1209
f86707fd195a 6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents: 4006
diff changeset
  1210
  tty->print_cr("page_trap_count");
f86707fd195a 6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents: 4006
diff changeset
  1211
}
f86707fd195a 6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents: 4006
diff changeset
  1212
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1213
void SafepointSynchronize::deferred_initialize_stat() {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1214
  if (init_done) return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1215
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1216
  // If PrintSafepointStatisticsTimeout is specified, the statistics data will
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1217
  // be printed right away, in which case, _safepoint_stats will regress to
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1218
  // a single element array. Otherwise, it is a circular ring buffer with default
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1219
  // size of PrintSafepointStatisticsCount.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1220
  int stats_array_size;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1221
  if (PrintSafepointStatisticsTimeout > 0) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1222
    stats_array_size = 1;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1223
    PrintSafepointStatistics = true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1224
  } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1225
    stats_array_size = PrintSafepointStatisticsCount;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1226
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1227
  _safepoint_stats = (SafepointStats*)os::malloc(stats_array_size
13195
be27e1b6a4b9 6995781: Native Memory Tracking (Phase 1)
zgu
parents: 13087
diff changeset
  1228
                                                 * sizeof(SafepointStats), mtInternal);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1229
  guarantee(_safepoint_stats != NULL,
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1230
            "not enough memory for safepoint instrumentation data");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1231
27662
222e6578695a 8064749: -XX:-UseCompilerSafepoints breaks safepoint rendezvous
shade
parents: 25946
diff changeset
  1232
  if (DeferPollingPageLoopCount >= 0) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1233
    need_to_track_page_armed_status = true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1234
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1235
  init_done = true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1236
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1237
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1238
void SafepointSynchronize::begin_statistics(int nof_threads, int nof_running) {
4006
7be2d5b3b15c 6880029: JDK 1.6.0_u14p Application crashed very early
xlu
parents: 3826
diff changeset
  1239
  assert(init_done, "safepoint statistics array hasn't been initialized");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1240
  SafepointStats *spstat = &_safepoint_stats[_cur_stat_index];
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1241
5042
f86707fd195a 6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents: 4006
diff changeset
  1242
  spstat->_time_stamp = _ts_of_current_safepoint;
f86707fd195a 6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents: 4006
diff changeset
  1243
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1244
  VM_Operation *op = VMThread::vm_operation();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1245
  spstat->_vmop_type = (op != NULL ? op->type() : -1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1246
  if (op != NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1247
    _safepoint_reasons[spstat->_vmop_type]++;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1248
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1249
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1250
  spstat->_nof_total_threads = nof_threads;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1251
  spstat->_nof_initial_running_threads = nof_running;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1252
  spstat->_nof_threads_hit_page_trap = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1253
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1254
  // Records the start time of spinning. The real time spent on spinning
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1255
  // will be adjusted when spin is done. Same trick is applied for time
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1256
  // spent on waiting for threads to block.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1257
  if (nof_running != 0) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1258
    spstat->_time_to_spin = os::javaTimeNanos();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1259
  }  else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1260
    spstat->_time_to_spin = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1261
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1262
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1263
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1264
void SafepointSynchronize::update_statistics_on_spin_end() {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1265
  SafepointStats *spstat = &_safepoint_stats[_cur_stat_index];
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1266
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1267
  jlong cur_time = os::javaTimeNanos();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1268
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1269
  spstat->_nof_threads_wait_to_block = _waiting_to_block;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1270
  if (spstat->_nof_initial_running_threads != 0) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1271
    spstat->_time_to_spin = cur_time - spstat->_time_to_spin;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1272
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1273
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1274
  if (need_to_track_page_armed_status) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1275
    spstat->_page_armed = (PageArmed == 1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1276
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1277
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1278
  // Records the start time of waiting for to block. Updated when block is done.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1279
  if (_waiting_to_block != 0) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1280
    spstat->_time_to_wait_to_block = cur_time;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1281
  } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1282
    spstat->_time_to_wait_to_block = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1283
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1284
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1285
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1286
void SafepointSynchronize::update_statistics_on_sync_end(jlong end_time) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1287
  SafepointStats *spstat = &_safepoint_stats[_cur_stat_index];
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1288
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1289
  if (spstat->_nof_threads_wait_to_block != 0) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1290
    spstat->_time_to_wait_to_block = end_time -
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1291
      spstat->_time_to_wait_to_block;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1292
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1293
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1294
  // Records the end time of sync which will be used to calculate the total
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1295
  // vm operation time. Again, the real time spending in syncing will be deducted
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1296
  // from the start of the sync time later when end_statistics is called.
5042
f86707fd195a 6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents: 4006
diff changeset
  1297
  spstat->_time_to_sync = end_time - _safepoint_begin_time;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1298
  if (spstat->_time_to_sync > _max_sync_time) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1299
    _max_sync_time = spstat->_time_to_sync;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1300
  }
5042
f86707fd195a 6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents: 4006
diff changeset
  1301
f86707fd195a 6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents: 4006
diff changeset
  1302
  spstat->_time_to_do_cleanups = end_time;
f86707fd195a 6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents: 4006
diff changeset
  1303
}
f86707fd195a 6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents: 4006
diff changeset
  1304
f86707fd195a 6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents: 4006
diff changeset
  1305
void SafepointSynchronize::update_statistics_on_cleanup_end(jlong end_time) {
f86707fd195a 6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents: 4006
diff changeset
  1306
  SafepointStats *spstat = &_safepoint_stats[_cur_stat_index];
f86707fd195a 6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents: 4006
diff changeset
  1307
f86707fd195a 6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents: 4006
diff changeset
  1308
  // Record how long spent in cleanup tasks.
f86707fd195a 6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents: 4006
diff changeset
  1309
  spstat->_time_to_do_cleanups = end_time - spstat->_time_to_do_cleanups;
f86707fd195a 6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents: 4006
diff changeset
  1310
f86707fd195a 6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents: 4006
diff changeset
  1311
  cleanup_end_time = end_time;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1312
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1313
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1314
void SafepointSynchronize::end_statistics(jlong vmop_end_time) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1315
  SafepointStats *spstat = &_safepoint_stats[_cur_stat_index];
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1316
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1317
  // Update the vm operation time.
5042
f86707fd195a 6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents: 4006
diff changeset
  1318
  spstat->_time_to_exec_vmop = vmop_end_time -  cleanup_end_time;
f86707fd195a 6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents: 4006
diff changeset
  1319
  if (spstat->_time_to_exec_vmop > _max_vmop_time) {
f86707fd195a 6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents: 4006
diff changeset
  1320
    _max_vmop_time = spstat->_time_to_exec_vmop;
f86707fd195a 6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents: 4006
diff changeset
  1321
  }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1322
  // Only the sync time longer than the specified
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1323
  // PrintSafepointStatisticsTimeout will be printed out right away.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1324
  // By default, it is -1 meaning all samples will be put into the list.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1325
  if ( PrintSafepointStatisticsTimeout > 0) {
33222
e0a340f4ab6e 8078556: Runtime: implement ranges (optionally constraints) for those flags that have them missing.
gziemski
parents: 33148
diff changeset
  1326
    if (spstat->_time_to_sync > (jlong)PrintSafepointStatisticsTimeout * MICROUNITS) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1327
      print_statistics();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1328
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1329
  } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1330
    // The safepoint statistics will be printed out when the _safepoin_stats
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1331
    // array fills up.
5042
f86707fd195a 6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents: 4006
diff changeset
  1332
    if (_cur_stat_index == PrintSafepointStatisticsCount - 1) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1333
      print_statistics();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1334
      _cur_stat_index = 0;
5042
f86707fd195a 6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents: 4006
diff changeset
  1335
    } else {
f86707fd195a 6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents: 4006
diff changeset
  1336
      _cur_stat_index++;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1337
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1338
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1339
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1340
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1341
void SafepointSynchronize::print_statistics() {
5042
f86707fd195a 6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents: 4006
diff changeset
  1342
  for (int index = 0; index <= _cur_stat_index; index++) {
f86707fd195a 6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents: 4006
diff changeset
  1343
    if (index % 30 == 0) {
f86707fd195a 6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents: 4006
diff changeset
  1344
      print_header();
f86707fd195a 6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents: 4006
diff changeset
  1345
    }
46473
d43055e806c9 8180482: Reformat -XX:+PrintSafepointStatistics table
shade
parents: 40664
diff changeset
  1346
    SafepointStats* sstats = &_safepoint_stats[index];
d43055e806c9 8180482: Reformat -XX:+PrintSafepointStatistics table
shade
parents: 40664
diff changeset
  1347
    tty->print("%8.3f: ", sstats->_time_stamp);
d43055e806c9 8180482: Reformat -XX:+PrintSafepointStatistics table
shade
parents: 40664
diff changeset
  1348
    tty->print("%-30s  [          "
d43055e806c9 8180482: Reformat -XX:+PrintSafepointStatistics table
shade
parents: 40664
diff changeset
  1349
               INT32_FORMAT_W(8) " " INT32_FORMAT_W(17) " " INT32_FORMAT_W(13) " "
d43055e806c9 8180482: Reformat -XX:+PrintSafepointStatistics table
shade
parents: 40664
diff changeset
  1350
               "]",
d43055e806c9 8180482: Reformat -XX:+PrintSafepointStatistics table
shade
parents: 40664
diff changeset
  1351
               (sstats->_vmop_type == -1 ? "no vm operation" : VM_Operation::name(sstats->_vmop_type)),
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1352
               sstats->_nof_total_threads,
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1353
               sstats->_nof_initial_running_threads,
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1354
               sstats->_nof_threads_wait_to_block);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1355
    // "/ MICROUNITS " is to convert the unit from nanos to millis.
46473
d43055e806c9 8180482: Reformat -XX:+PrintSafepointStatistics table
shade
parents: 40664
diff changeset
  1356
    tty->print("[       "
d43055e806c9 8180482: Reformat -XX:+PrintSafepointStatistics table
shade
parents: 40664
diff changeset
  1357
               INT64_FORMAT_W(7) " " INT64_FORMAT_W(7) " "
d43055e806c9 8180482: Reformat -XX:+PrintSafepointStatistics table
shade
parents: 40664
diff changeset
  1358
               INT64_FORMAT_W(7) " " INT64_FORMAT_W(7) " "
d43055e806c9 8180482: Reformat -XX:+PrintSafepointStatistics table
shade
parents: 40664
diff changeset
  1359
               INT64_FORMAT_W(7) " ] ",
46630
75aa3e39d02c 8182299: Enable disabled clang warnings, build on OSX 10 + Xcode 8
jwilhelm
parents: 46541
diff changeset
  1360
               (int64_t)(sstats->_time_to_spin / MICROUNITS),
75aa3e39d02c 8182299: Enable disabled clang warnings, build on OSX 10 + Xcode 8
jwilhelm
parents: 46541
diff changeset
  1361
               (int64_t)(sstats->_time_to_wait_to_block / MICROUNITS),
75aa3e39d02c 8182299: Enable disabled clang warnings, build on OSX 10 + Xcode 8
jwilhelm
parents: 46541
diff changeset
  1362
               (int64_t)(sstats->_time_to_sync / MICROUNITS),
75aa3e39d02c 8182299: Enable disabled clang warnings, build on OSX 10 + Xcode 8
jwilhelm
parents: 46541
diff changeset
  1363
               (int64_t)(sstats->_time_to_do_cleanups / MICROUNITS),
75aa3e39d02c 8182299: Enable disabled clang warnings, build on OSX 10 + Xcode 8
jwilhelm
parents: 46541
diff changeset
  1364
               (int64_t)(sstats->_time_to_exec_vmop / MICROUNITS));
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1365
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1366
    if (need_to_track_page_armed_status) {
46473
d43055e806c9 8180482: Reformat -XX:+PrintSafepointStatistics table
shade
parents: 40664
diff changeset
  1367
      tty->print(INT32_FORMAT_W(10) " ", sstats->_page_armed);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1368
    }
46473
d43055e806c9 8180482: Reformat -XX:+PrintSafepointStatistics table
shade
parents: 40664
diff changeset
  1369
    tty->print_cr(INT32_FORMAT_W(15) " ", sstats->_nof_threads_hit_page_trap);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1370
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1371
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1372
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1373
// This method will be called when VM exits. It will first call
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1374
// print_statistics to print out the rest of the sampling.  Then
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1375
// it tries to summarize the sampling.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1376
void SafepointSynchronize::print_stat_on_exit() {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1377
  if (_safepoint_stats == NULL) return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1378
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1379
  SafepointStats *spstat = &_safepoint_stats[_cur_stat_index];
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1380
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1381
  // During VM exit, end_statistics may not get called and in that
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1382
  // case, if the sync time is less than PrintSafepointStatisticsTimeout,
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1383
  // don't print it out.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1384
  // Approximate the vm op time.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1385
  _safepoint_stats[_cur_stat_index]._time_to_exec_vmop =
5042
f86707fd195a 6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents: 4006
diff changeset
  1386
    os::javaTimeNanos() - cleanup_end_time;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1387
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1388
  if ( PrintSafepointStatisticsTimeout < 0 ||
33222
e0a340f4ab6e 8078556: Runtime: implement ranges (optionally constraints) for those flags that have them missing.
gziemski
parents: 33148
diff changeset
  1389
       spstat->_time_to_sync > (jlong)PrintSafepointStatisticsTimeout * MICROUNITS) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1390
    print_statistics();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1391
  }
24424
2658d7834c6e 8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents: 24351
diff changeset
  1392
  tty->cr();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1393
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1394
  // Print out polling page sampling status.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1395
  if (!need_to_track_page_armed_status) {
27662
222e6578695a 8064749: -XX:-UseCompilerSafepoints breaks safepoint rendezvous
shade
parents: 25946
diff changeset
  1396
    tty->print_cr("Polling page always armed");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1397
  } else {
33148
68fa8b6c4340 8042893: compiler: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents: 33105
diff changeset
  1398
    tty->print_cr("Defer polling page loop count = " INTX_FORMAT "\n",
68fa8b6c4340 8042893: compiler: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents: 33105
diff changeset
  1399
                  DeferPollingPageLoopCount);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1400
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1401
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1402
  for (int index = 0; index < VM_Operation::VMOp_Terminating; index++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1403
    if (_safepoint_reasons[index] != 0) {
31592
43f48e165466 8081202: Hotspot compile warning: "Invalid suffix on literal; C++11 requires a space between literal and identifier"
bpittore
parents: 30764
diff changeset
  1404
      tty->print_cr("%-26s" UINT64_FORMAT_W(10), VM_Operation::name(index),
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1405
                    _safepoint_reasons[index]);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1406
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1407
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1408
31592
43f48e165466 8081202: Hotspot compile warning: "Invalid suffix on literal; C++11 requires a space between literal and identifier"
bpittore
parents: 30764
diff changeset
  1409
  tty->print_cr(UINT64_FORMAT_W(5) " VM operations coalesced during safepoint",
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1410
                _coalesced_vmop_count);
31592
43f48e165466 8081202: Hotspot compile warning: "Invalid suffix on literal; C++11 requires a space between literal and identifier"
bpittore
parents: 30764
diff changeset
  1411
  tty->print_cr("Maximum sync time  " INT64_FORMAT_W(5) " ms",
46630
75aa3e39d02c 8182299: Enable disabled clang warnings, build on OSX 10 + Xcode 8
jwilhelm
parents: 46541
diff changeset
  1412
                (int64_t)(_max_sync_time / MICROUNITS));
5042
f86707fd195a 6933402: RFE: Improve PrintSafepointStatistics output to track cleanup time
xlu
parents: 4006
diff changeset
  1413
  tty->print_cr("Maximum vm operation time (except for Exit VM operation)  "
31592
43f48e165466 8081202: Hotspot compile warning: "Invalid suffix on literal; C++11 requires a space between literal and identifier"
bpittore
parents: 30764
diff changeset
  1414
                INT64_FORMAT_W(5) " ms",
46630
75aa3e39d02c 8182299: Enable disabled clang warnings, build on OSX 10 + Xcode 8
jwilhelm
parents: 46541
diff changeset
  1415
                (int64_t)(_max_vmop_time / MICROUNITS));
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1416
}