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