hotspot/src/share/vm/gc/g1/concurrentMarkThread.cpp
author brutisso
Tue, 29 Sep 2015 17:44:58 +0200
changeset 33106 20c533b9e167
parent 31597 f63dde8cc8f7
child 33107 77bf0d2069a3
permissions -rw-r--r--
8133706: Kitchensink hanged Reviewed-by: pliden, jmasa
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
25350
6423a57e5451 8043607: Add a GC id as a log decoration similar to PrintGCTimeStamps
brutisso
parents: 24331
diff changeset
     1
 /*
29464
02c245ad3ec9 8073545: Use shorter and more descriptive names for GC worker threads
david
parents: 27885
diff changeset
     2
 * Copyright (c) 2001, 2015, Oracle and/or its affiliates. All rights reserved.
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
     3
 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
     4
 *
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
     5
 * This code is free software; you can redistribute it and/or modify it
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
     6
 * under the terms of the GNU General Public License version 2 only, as
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
     7
 * published by the Free Software Foundation.
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
     8
 *
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
     9
 * This code is distributed in the hope that it will be useful, but WITHOUT
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    10
 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    11
 * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    12
 * version 2 for more details (a copy is included in the LICENSE file that
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    13
 * accompanied this code).
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    14
 *
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    15
 * You should have received a copy of the GNU General Public License version
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    16
 * 2 along with this work; if not, write to the Free Software Foundation,
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    17
 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    18
 *
5547
f4b087cbb361 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 4022
diff changeset
    19
 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
f4b087cbb361 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 4022
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: 4022
diff changeset
    21
 * questions.
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    22
 *
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    23
 */
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    24
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6766
diff changeset
    25
#include "precompiled.hpp"
30764
fec48bf5a827 8079792: GC directory structure cleanup
pliden
parents: 30613
diff changeset
    26
#include "gc/g1/concurrentMarkThread.inline.hpp"
fec48bf5a827 8079792: GC directory structure cleanup
pliden
parents: 30613
diff changeset
    27
#include "gc/g1/g1CollectedHeap.inline.hpp"
fec48bf5a827 8079792: GC directory structure cleanup
pliden
parents: 30613
diff changeset
    28
#include "gc/g1/g1CollectorPolicy.hpp"
fec48bf5a827 8079792: GC directory structure cleanup
pliden
parents: 30613
diff changeset
    29
#include "gc/g1/g1Log.hpp"
fec48bf5a827 8079792: GC directory structure cleanup
pliden
parents: 30613
diff changeset
    30
#include "gc/g1/g1MMUTracker.hpp"
30770
5ba2d9f2084d 8080585: concurrentGCThread.hpp should not include suspendibleThreadSet.hpp
pliden
parents: 30764
diff changeset
    31
#include "gc/g1/suspendibleThreadSet.hpp"
30764
fec48bf5a827 8079792: GC directory structure cleanup
pliden
parents: 30613
diff changeset
    32
#include "gc/g1/vm_operations_g1.hpp"
fec48bf5a827 8079792: GC directory structure cleanup
pliden
parents: 30613
diff changeset
    33
#include "gc/shared/gcTrace.hpp"
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6766
diff changeset
    34
#include "memory/resourceArea.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6766
diff changeset
    35
#include "runtime/vmThread.hpp"
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    36
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    37
// ======= Concurrent Mark Thread ========
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    38
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    39
// The CM thread is created when the G1 garbage collector is used
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    40
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    41
SurrogateLockerThread*
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    42
     ConcurrentMarkThread::_slt = NULL;
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    43
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    44
ConcurrentMarkThread::ConcurrentMarkThread(ConcurrentMark* cm) :
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    45
  ConcurrentGCThread(),
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    46
  _cm(cm),
31597
f63dde8cc8f7 8129626: G1: set_in_progress() and clear_started() needs a barrier on non-TSO platforms
brutisso
parents: 31397
diff changeset
    47
  _state(Idle),
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    48
  _vtime_accum(0.0),
11583
83a7383de44c 6484965: G1: piggy-back liveness accounting phase on marking
johnc
parents: 11175
diff changeset
    49
  _vtime_mark_accum(0.0) {
24331
c0bc7e5653fb 6885993: Named Thread: introduce print() and print_on(outputStream* st) methods
zgu
parents: 24094
diff changeset
    50
29464
02c245ad3ec9 8073545: Use shorter and more descriptive names for GC worker threads
david
parents: 27885
diff changeset
    51
  set_name("G1 Main Marker");
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    52
  create_and_start();
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    53
}
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    54
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    55
class CMCheckpointRootsFinalClosure: public VoidClosure {
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    56
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    57
  ConcurrentMark* _cm;
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    58
public:
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    59
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    60
  CMCheckpointRootsFinalClosure(ConcurrentMark* cm) :
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    61
    _cm(cm) {}
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    62
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    63
  void do_void(){
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    64
    _cm->checkpointRootsFinal(false); // !clear_all_soft_refs
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    65
  }
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    66
};
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    67
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    68
class CMCleanUp: public VoidClosure {
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    69
  ConcurrentMark* _cm;
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    70
public:
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    71
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    72
  CMCleanUp(ConcurrentMark* cm) :
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    73
    _cm(cm) {}
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    74
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    75
  void do_void(){
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    76
    _cm->cleanup();
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    77
  }
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    78
};
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    79
31397
c9cc3289b80c 8129549: G1: Make sure the concurrent thread does not mix its logging with the STW pauses
brutisso
parents: 31331
diff changeset
    80
// We want to avoid that the logging from the concurrent thread is mixed
c9cc3289b80c 8129549: G1: Make sure the concurrent thread does not mix its logging with the STW pauses
brutisso
parents: 31331
diff changeset
    81
// with the logging from a STW GC. So, if necessary join the STS to ensure
c9cc3289b80c 8129549: G1: Make sure the concurrent thread does not mix its logging with the STW pauses
brutisso
parents: 31331
diff changeset
    82
// that the logging is done either before or after the STW logging.
c9cc3289b80c 8129549: G1: Make sure the concurrent thread does not mix its logging with the STW pauses
brutisso
parents: 31331
diff changeset
    83
void ConcurrentMarkThread::cm_log(bool doit, bool join_sts, const char* fmt, ...) {
c9cc3289b80c 8129549: G1: Make sure the concurrent thread does not mix its logging with the STW pauses
brutisso
parents: 31331
diff changeset
    84
  if (doit) {
c9cc3289b80c 8129549: G1: Make sure the concurrent thread does not mix its logging with the STW pauses
brutisso
parents: 31331
diff changeset
    85
    SuspendibleThreadSetJoiner sts_joiner(join_sts);
c9cc3289b80c 8129549: G1: Make sure the concurrent thread does not mix its logging with the STW pauses
brutisso
parents: 31331
diff changeset
    86
    va_list args;
c9cc3289b80c 8129549: G1: Make sure the concurrent thread does not mix its logging with the STW pauses
brutisso
parents: 31331
diff changeset
    87
    va_start(args, fmt);
c9cc3289b80c 8129549: G1: Make sure the concurrent thread does not mix its logging with the STW pauses
brutisso
parents: 31331
diff changeset
    88
    gclog_or_tty->gclog_stamp(cm()->concurrent_gc_id());
c9cc3289b80c 8129549: G1: Make sure the concurrent thread does not mix its logging with the STW pauses
brutisso
parents: 31331
diff changeset
    89
    gclog_or_tty->vprint_cr(fmt, args);
c9cc3289b80c 8129549: G1: Make sure the concurrent thread does not mix its logging with the STW pauses
brutisso
parents: 31331
diff changeset
    90
    va_end(args);
c9cc3289b80c 8129549: G1: Make sure the concurrent thread does not mix its logging with the STW pauses
brutisso
parents: 31331
diff changeset
    91
  }
c9cc3289b80c 8129549: G1: Make sure the concurrent thread does not mix its logging with the STW pauses
brutisso
parents: 31331
diff changeset
    92
}
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    93
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    94
void ConcurrentMarkThread::run() {
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    95
  initialize_in_thread();
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    96
  _vtime_start = os::elapsedVTime();
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    97
  wait_for_universe_init();
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    98
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents: 7455
diff changeset
    99
  G1CollectedHeap* g1h = G1CollectedHeap::heap();
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents: 7455
diff changeset
   100
  G1CollectorPolicy* g1_policy = g1h->g1_policy();
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   101
  G1MMUTracker *mmu_tracker = g1_policy->mmu_tracker();
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   102
  Thread *current_thread = Thread::current();
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   103
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   104
  while (!_should_terminate) {
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   105
    // wait until started is set.
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   106
    sleepBeforeNextCycle();
24093
095cc0a63ed9 8037112: gc/g1/TestHumongousAllocInitialMark.java caused SIGSEGV
pliden
parents: 18025
diff changeset
   107
    if (_should_terminate) {
095cc0a63ed9 8037112: gc/g1/TestHumongousAllocInitialMark.java caused SIGSEGV
pliden
parents: 18025
diff changeset
   108
      break;
095cc0a63ed9 8037112: gc/g1/TestHumongousAllocInitialMark.java caused SIGSEGV
pliden
parents: 18025
diff changeset
   109
    }
095cc0a63ed9 8037112: gc/g1/TestHumongousAllocInitialMark.java caused SIGSEGV
pliden
parents: 18025
diff changeset
   110
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   111
    {
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   112
      ResourceMark rm;
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   113
      HandleMark   hm;
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   114
      double cycle_start = os::elapsedVTime();
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   115
11584
e1df4d08a1f4 7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents: 11583
diff changeset
   116
      // We have to ensure that we finish scanning the root regions
e1df4d08a1f4 7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents: 11583
diff changeset
   117
      // before the next GC takes place. To ensure this we have to
e1df4d08a1f4 7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents: 11583
diff changeset
   118
      // make sure that we do not join the STS until the root regions
e1df4d08a1f4 7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents: 11583
diff changeset
   119
      // have been scanned. If we did then it's possible that a
e1df4d08a1f4 7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents: 11583
diff changeset
   120
      // subsequent GC could block us from joining the STS and proceed
e1df4d08a1f4 7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents: 11583
diff changeset
   121
      // without the root regions have been scanned which would be a
e1df4d08a1f4 7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents: 11583
diff changeset
   122
      // correctness issue.
e1df4d08a1f4 7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents: 11583
diff changeset
   123
e1df4d08a1f4 7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents: 11583
diff changeset
   124
      if (!cm()->has_aborted()) {
e1df4d08a1f4 7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents: 11583
diff changeset
   125
        _cm->scanRootRegions();
e1df4d08a1f4 7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents: 11583
diff changeset
   126
      }
e1df4d08a1f4 7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents: 11583
diff changeset
   127
e1df4d08a1f4 7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents: 11583
diff changeset
   128
      double mark_start_sec = os::elapsedTime();
31397
c9cc3289b80c 8129549: G1: Make sure the concurrent thread does not mix its logging with the STW pauses
brutisso
parents: 31331
diff changeset
   129
      cm_log(G1Log::fine(), true, "[GC concurrent-mark-start]");
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   130
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   131
      int iter = 0;
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   132
      do {
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   133
        iter++;
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   134
        if (!cm()->has_aborted()) {
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   135
          _cm->markFromRoots();
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   136
        }
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   137
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   138
        double mark_end_time = os::elapsedVTime();
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   139
        double mark_end_sec = os::elapsedTime();
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   140
        _vtime_mark_accum += (mark_end_time - cycle_start);
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   141
        if (!cm()->has_aborted()) {
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   142
          if (g1_policy->adaptive_young_list_length()) {
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   143
            double now = os::elapsedTime();
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   144
            double remark_prediction_ms = g1_policy->predict_remark_time_ms();
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   145
            jlong sleep_time_ms = mmu_tracker->when_ms(now, remark_prediction_ms);
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   146
            os::sleep(current_thread, sleep_time_ms, false);
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   147
          }
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   148
31397
c9cc3289b80c 8129549: G1: Make sure the concurrent thread does not mix its logging with the STW pauses
brutisso
parents: 31331
diff changeset
   149
          cm_log(G1Log::fine(), true, "[GC concurrent-mark-end, %1.7lf secs]", mark_end_sec - mark_start_sec);
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   150
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   151
          CMCheckpointRootsFinalClosure final_cl(_cm);
18025
b7bcf7497f93 8005849: JEP 167: Event-Based JVM Tracing
sla
parents: 15804
diff changeset
   152
          VM_CGC_Operation op(&final_cl, "GC remark", true /* needs_pll */);
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   153
          VMThread::execute(&op);
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   154
        }
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   155
        if (cm()->restart_for_overflow()) {
31397
c9cc3289b80c 8129549: G1: Make sure the concurrent thread does not mix its logging with the STW pauses
brutisso
parents: 31331
diff changeset
   156
          cm_log(G1TraceMarkStackOverflow, true, "Restarting conc marking because of MS overflow in remark (restart #%d).", iter);
c9cc3289b80c 8129549: G1: Make sure the concurrent thread does not mix its logging with the STW pauses
brutisso
parents: 31331
diff changeset
   157
          cm_log(G1Log::fine(), true, "[GC concurrent-mark-restart-for-overflow]");
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   158
        }
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   159
      } while (cm()->restart_for_overflow());
10769
983d377770fd 7099824: G1: we should take the pending list lock before doing the remark pause
johnc
parents: 10745
diff changeset
   160
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   161
      double end_time = os::elapsedVTime();
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   162
      // Update the total virtual time before doing this, since it will try
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   163
      // to measure it to get the vtime for this marking.  We purposely
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   164
      // neglect the presumably-short "completeCleanup" phase here.
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   165
      _vtime_accum = (end_time - _vtime_start);
11583
83a7383de44c 6484965: G1: piggy-back liveness accounting phase on marking
johnc
parents: 11175
diff changeset
   166
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   167
      if (!cm()->has_aborted()) {
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   168
        if (g1_policy->adaptive_young_list_length()) {
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   169
          double now = os::elapsedTime();
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   170
          double cleanup_prediction_ms = g1_policy->predict_cleanup_time_ms();
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   171
          jlong sleep_time_ms = mmu_tracker->when_ms(now, cleanup_prediction_ms);
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   172
          os::sleep(current_thread, sleep_time_ms, false);
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   173
        }
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   174
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   175
        CMCleanUp cl_cl(_cm);
18025
b7bcf7497f93 8005849: JEP 167: Event-Based JVM Tracing
sla
parents: 15804
diff changeset
   176
        VM_CGC_Operation op(&cl_cl, "GC cleanup", false /* needs_pll */);
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   177
        VMThread::execute(&op);
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   178
      } else {
11175
7fde26aecbe5 7114303: G1: assert(_g1->mark_in_progress()) failed: shouldn't be here otherwise
johnc
parents: 10769
diff changeset
   179
        // We don't want to update the marking status if a GC pause
7fde26aecbe5 7114303: G1: assert(_g1->mark_in_progress()) failed: shouldn't be here otherwise
johnc
parents: 10769
diff changeset
   180
        // is already underway.
30613
a7815bb05ae2 8079579: Add SuspendibleThreadSetLeaver and make SuspendibleThreadSet::joint()/leave() private
pliden
parents: 29464
diff changeset
   181
        SuspendibleThreadSetJoiner sts_join;
31331
a7c714b6cfb3 7097567: G1: abstract and encapsulate collector phases and transitions between them
drwhite
parents: 30770
diff changeset
   182
        g1h->collector_state()->set_mark_in_progress(false);
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   183
      }
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   184
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents: 7455
diff changeset
   185
      // Check if cleanup set the free_regions_coming flag. If it
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents: 7455
diff changeset
   186
      // hasn't, we can just skip the next step.
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents: 7455
diff changeset
   187
      if (g1h->free_regions_coming()) {
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents: 7455
diff changeset
   188
        // The following will finish freeing up any regions that we
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents: 7455
diff changeset
   189
        // found to be empty during cleanup. We'll do this part
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents: 7455
diff changeset
   190
        // without joining the suspendible set. If an evacuation pause
8680
f1c414e16a4c 7014923: G1: code cleanup
tonyp
parents: 8100
diff changeset
   191
        // takes place, then we would carry on freeing regions in
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents: 7455
diff changeset
   192
        // case they are needed by the pause. If a Full GC takes
8680
f1c414e16a4c 7014923: G1: code cleanup
tonyp
parents: 8100
diff changeset
   193
        // place, it would wait for us to process the regions
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents: 7455
diff changeset
   194
        // reclaimed by cleanup.
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents: 7455
diff changeset
   195
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   196
        double cleanup_start_sec = os::elapsedTime();
33106
20c533b9e167 8133706: Kitchensink hanged
brutisso
parents: 31597
diff changeset
   197
        cm_log(G1Log::fine(), false, "[GC concurrent-cleanup-start]");
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   198
10745
6b33ec421509 7098085: G1: partially-young GCs not initiated under certain circumstances
tonyp
parents: 10280
diff changeset
   199
        // Now do the concurrent cleanup operation.
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   200
        _cm->completeCleanup();
10745
6b33ec421509 7098085: G1: partially-young GCs not initiated under certain circumstances
tonyp
parents: 10280
diff changeset
   201
8100
6d2b49640dd5 7014679: G1: deadlock during concurrent cleanup
tonyp
parents: 8071
diff changeset
   202
        // Notify anyone who's waiting that there are no more free
10745
6b33ec421509 7098085: G1: partially-young GCs not initiated under certain circumstances
tonyp
parents: 10280
diff changeset
   203
        // regions coming. We have to do this before we join the STS
6b33ec421509 7098085: G1: partially-young GCs not initiated under certain circumstances
tonyp
parents: 10280
diff changeset
   204
        // (in fact, we should not attempt to join the STS in the
6b33ec421509 7098085: G1: partially-young GCs not initiated under certain circumstances
tonyp
parents: 10280
diff changeset
   205
        // interval between finishing the cleanup pause and clearing
6b33ec421509 7098085: G1: partially-young GCs not initiated under certain circumstances
tonyp
parents: 10280
diff changeset
   206
        // the free_regions_coming flag) otherwise we might deadlock:
6b33ec421509 7098085: G1: partially-young GCs not initiated under certain circumstances
tonyp
parents: 10280
diff changeset
   207
        // a GC worker could be blocked waiting for the notification
6b33ec421509 7098085: G1: partially-young GCs not initiated under certain circumstances
tonyp
parents: 10280
diff changeset
   208
        // whereas this thread will be blocked for the pause to finish
6b33ec421509 7098085: G1: partially-young GCs not initiated under certain circumstances
tonyp
parents: 10280
diff changeset
   209
        // while it's trying to join the STS, which is conditional on
6b33ec421509 7098085: G1: partially-young GCs not initiated under certain circumstances
tonyp
parents: 10280
diff changeset
   210
        // the GC workers finishing.
8100
6d2b49640dd5 7014679: G1: deadlock during concurrent cleanup
tonyp
parents: 8071
diff changeset
   211
        g1h->reset_free_regions_coming();
6d2b49640dd5 7014679: G1: deadlock during concurrent cleanup
tonyp
parents: 8071
diff changeset
   212
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents: 7455
diff changeset
   213
        double cleanup_end_sec = os::elapsedTime();
31397
c9cc3289b80c 8129549: G1: Make sure the concurrent thread does not mix its logging with the STW pauses
brutisso
parents: 31331
diff changeset
   214
        cm_log(G1Log::fine(), true, "[GC concurrent-cleanup-end, %1.7lf secs]", cleanup_end_sec - cleanup_start_sec);
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   215
      }
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents: 7455
diff changeset
   216
      guarantee(cm()->cleanup_list_is_empty(),
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents: 7455
diff changeset
   217
                "at this point there should be no regions on the cleanup list");
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   218
10745
6b33ec421509 7098085: G1: partially-young GCs not initiated under certain circumstances
tonyp
parents: 10280
diff changeset
   219
      // There is a tricky race before recording that the concurrent
6b33ec421509 7098085: G1: partially-young GCs not initiated under certain circumstances
tonyp
parents: 10280
diff changeset
   220
      // cleanup has completed and a potential Full GC starting around
6b33ec421509 7098085: G1: partially-young GCs not initiated under certain circumstances
tonyp
parents: 10280
diff changeset
   221
      // the same time. We want to make sure that the Full GC calls
6b33ec421509 7098085: G1: partially-young GCs not initiated under certain circumstances
tonyp
parents: 10280
diff changeset
   222
      // abort() on concurrent mark after
6b33ec421509 7098085: G1: partially-young GCs not initiated under certain circumstances
tonyp
parents: 10280
diff changeset
   223
      // record_concurrent_mark_cleanup_completed(), since abort() is
6b33ec421509 7098085: G1: partially-young GCs not initiated under certain circumstances
tonyp
parents: 10280
diff changeset
   224
      // the method that will reset the concurrent mark state. If we
6b33ec421509 7098085: G1: partially-young GCs not initiated under certain circumstances
tonyp
parents: 10280
diff changeset
   225
      // end up calling record_concurrent_mark_cleanup_completed()
6b33ec421509 7098085: G1: partially-young GCs not initiated under certain circumstances
tonyp
parents: 10280
diff changeset
   226
      // after abort() then we might incorrectly undo some of the work
6b33ec421509 7098085: G1: partially-young GCs not initiated under certain circumstances
tonyp
parents: 10280
diff changeset
   227
      // abort() did. Checking the has_aborted() flag after joining
6b33ec421509 7098085: G1: partially-young GCs not initiated under certain circumstances
tonyp
parents: 10280
diff changeset
   228
      // the STS allows the correct ordering of the two methods. There
6b33ec421509 7098085: G1: partially-young GCs not initiated under certain circumstances
tonyp
parents: 10280
diff changeset
   229
      // are two scenarios:
6b33ec421509 7098085: G1: partially-young GCs not initiated under certain circumstances
tonyp
parents: 10280
diff changeset
   230
      //
6b33ec421509 7098085: G1: partially-young GCs not initiated under certain circumstances
tonyp
parents: 10280
diff changeset
   231
      // a) If we reach here before the Full GC, the fact that we have
6b33ec421509 7098085: G1: partially-young GCs not initiated under certain circumstances
tonyp
parents: 10280
diff changeset
   232
      // joined the STS means that the Full GC cannot start until we
6b33ec421509 7098085: G1: partially-young GCs not initiated under certain circumstances
tonyp
parents: 10280
diff changeset
   233
      // leave the STS, so record_concurrent_mark_cleanup_completed()
6b33ec421509 7098085: G1: partially-young GCs not initiated under certain circumstances
tonyp
parents: 10280
diff changeset
   234
      // will complete before abort() is called.
6b33ec421509 7098085: G1: partially-young GCs not initiated under certain circumstances
tonyp
parents: 10280
diff changeset
   235
      //
6b33ec421509 7098085: G1: partially-young GCs not initiated under certain circumstances
tonyp
parents: 10280
diff changeset
   236
      // b) If we reach here during the Full GC, we'll be held up from
6b33ec421509 7098085: G1: partially-young GCs not initiated under certain circumstances
tonyp
parents: 10280
diff changeset
   237
      // joining the STS until the Full GC is done, which means that
6b33ec421509 7098085: G1: partially-young GCs not initiated under certain circumstances
tonyp
parents: 10280
diff changeset
   238
      // abort() will have completed and has_aborted() will return
6b33ec421509 7098085: G1: partially-young GCs not initiated under certain circumstances
tonyp
parents: 10280
diff changeset
   239
      // true to prevent us from calling
6b33ec421509 7098085: G1: partially-young GCs not initiated under certain circumstances
tonyp
parents: 10280
diff changeset
   240
      // record_concurrent_mark_cleanup_completed() (and, in fact, it's
6b33ec421509 7098085: G1: partially-young GCs not initiated under certain circumstances
tonyp
parents: 10280
diff changeset
   241
      // not needed any more as the concurrent mark state has been
6b33ec421509 7098085: G1: partially-young GCs not initiated under certain circumstances
tonyp
parents: 10280
diff changeset
   242
      // already reset).
24094
5dbf1f44de18 8039147: Cleanup SuspendibleThreadSet
pliden
parents: 24093
diff changeset
   243
      {
30613
a7815bb05ae2 8079579: Add SuspendibleThreadSetLeaver and make SuspendibleThreadSet::joint()/leave() private
pliden
parents: 29464
diff changeset
   244
        SuspendibleThreadSetJoiner sts_join;
24094
5dbf1f44de18 8039147: Cleanup SuspendibleThreadSet
pliden
parents: 24093
diff changeset
   245
        if (!cm()->has_aborted()) {
5dbf1f44de18 8039147: Cleanup SuspendibleThreadSet
pliden
parents: 24093
diff changeset
   246
          g1_policy->record_concurrent_mark_cleanup_completed();
31397
c9cc3289b80c 8129549: G1: Make sure the concurrent thread does not mix its logging with the STW pauses
brutisso
parents: 31331
diff changeset
   247
        } else {
c9cc3289b80c 8129549: G1: Make sure the concurrent thread does not mix its logging with the STW pauses
brutisso
parents: 31331
diff changeset
   248
          cm_log(G1Log::fine(), false, "[GC concurrent-mark-abort]");
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   249
        }
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   250
      }
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   251
10745
6b33ec421509 7098085: G1: partially-young GCs not initiated under certain circumstances
tonyp
parents: 10280
diff changeset
   252
      // We now want to allow clearing of the marking bitmap to be
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   253
      // suspended by a collection pause.
25728
7f8a76e3baa8 8048085: Aborting marking just before remark results in useless additional clearing of the next mark bitmap
tschatzl
parents: 25350
diff changeset
   254
      // We may have aborted just before the remark. Do not bother clearing the
7f8a76e3baa8 8048085: Aborting marking just before remark results in useless additional clearing of the next mark bitmap
tschatzl
parents: 25350
diff changeset
   255
      // bitmap then, as it has been done during mark abort.
7f8a76e3baa8 8048085: Aborting marking just before remark results in useless additional clearing of the next mark bitmap
tschatzl
parents: 25350
diff changeset
   256
      if (!cm()->has_aborted()) {
24094
5dbf1f44de18 8039147: Cleanup SuspendibleThreadSet
pliden
parents: 24093
diff changeset
   257
        _cm->clearNextBitmap();
25728
7f8a76e3baa8 8048085: Aborting marking just before remark results in useless additional clearing of the next mark bitmap
tschatzl
parents: 25350
diff changeset
   258
      } else {
7f8a76e3baa8 8048085: Aborting marking just before remark results in useless additional clearing of the next mark bitmap
tschatzl
parents: 25350
diff changeset
   259
        assert(!G1VerifyBitmaps || _cm->nextMarkBitmapIsClear(), "Next mark bitmap must be clear");
24094
5dbf1f44de18 8039147: Cleanup SuspendibleThreadSet
pliden
parents: 24093
diff changeset
   260
      }
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   261
    }
6058
9c9aec6ab47d 6944166: G1: explicit GCs are not always handled correctly
tonyp
parents: 5547
diff changeset
   262
9c9aec6ab47d 6944166: G1: explicit GCs are not always handled correctly
tonyp
parents: 5547
diff changeset
   263
    // Update the number of full collections that have been
9c9aec6ab47d 6944166: G1: explicit GCs are not always handled correctly
tonyp
parents: 5547
diff changeset
   264
    // completed. This will also notify the FullGCCount_lock in case a
9c9aec6ab47d 6944166: G1: explicit GCs are not always handled correctly
tonyp
parents: 5547
diff changeset
   265
    // Java thread is waiting for a full GC to happen (e.g., it
9c9aec6ab47d 6944166: G1: explicit GCs are not always handled correctly
tonyp
parents: 5547
diff changeset
   266
    // called System.gc() with +ExplicitGCInvokesConcurrent).
24094
5dbf1f44de18 8039147: Cleanup SuspendibleThreadSet
pliden
parents: 24093
diff changeset
   267
    {
30613
a7815bb05ae2 8079579: Add SuspendibleThreadSetLeaver and make SuspendibleThreadSet::joint()/leave() private
pliden
parents: 29464
diff changeset
   268
      SuspendibleThreadSetJoiner sts_join;
24094
5dbf1f44de18 8039147: Cleanup SuspendibleThreadSet
pliden
parents: 24093
diff changeset
   269
      g1h->increment_old_marking_cycles_completed(true /* concurrent */);
5dbf1f44de18 8039147: Cleanup SuspendibleThreadSet
pliden
parents: 24093
diff changeset
   270
      g1h->register_concurrent_cycle_end();
5dbf1f44de18 8039147: Cleanup SuspendibleThreadSet
pliden
parents: 24093
diff changeset
   271
    }
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   272
  }
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   273
  assert(_should_terminate, "just checking");
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   274
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   275
  terminate();
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   276
}
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   277
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   278
void ConcurrentMarkThread::stop() {
24093
095cc0a63ed9 8037112: gc/g1/TestHumongousAllocInitialMark.java caused SIGSEGV
pliden
parents: 18025
diff changeset
   279
  {
095cc0a63ed9 8037112: gc/g1/TestHumongousAllocInitialMark.java caused SIGSEGV
pliden
parents: 18025
diff changeset
   280
    MutexLockerEx ml(Terminator_lock);
095cc0a63ed9 8037112: gc/g1/TestHumongousAllocInitialMark.java caused SIGSEGV
pliden
parents: 18025
diff changeset
   281
    _should_terminate = true;
095cc0a63ed9 8037112: gc/g1/TestHumongousAllocInitialMark.java caused SIGSEGV
pliden
parents: 18025
diff changeset
   282
  }
095cc0a63ed9 8037112: gc/g1/TestHumongousAllocInitialMark.java caused SIGSEGV
pliden
parents: 18025
diff changeset
   283
095cc0a63ed9 8037112: gc/g1/TestHumongousAllocInitialMark.java caused SIGSEGV
pliden
parents: 18025
diff changeset
   284
  {
095cc0a63ed9 8037112: gc/g1/TestHumongousAllocInitialMark.java caused SIGSEGV
pliden
parents: 18025
diff changeset
   285
    MutexLockerEx ml(CGC_lock, Mutex::_no_safepoint_check_flag);
095cc0a63ed9 8037112: gc/g1/TestHumongousAllocInitialMark.java caused SIGSEGV
pliden
parents: 18025
diff changeset
   286
    CGC_lock->notify_all();
095cc0a63ed9 8037112: gc/g1/TestHumongousAllocInitialMark.java caused SIGSEGV
pliden
parents: 18025
diff changeset
   287
  }
095cc0a63ed9 8037112: gc/g1/TestHumongousAllocInitialMark.java caused SIGSEGV
pliden
parents: 18025
diff changeset
   288
095cc0a63ed9 8037112: gc/g1/TestHumongousAllocInitialMark.java caused SIGSEGV
pliden
parents: 18025
diff changeset
   289
  {
095cc0a63ed9 8037112: gc/g1/TestHumongousAllocInitialMark.java caused SIGSEGV
pliden
parents: 18025
diff changeset
   290
    MutexLockerEx ml(Terminator_lock);
095cc0a63ed9 8037112: gc/g1/TestHumongousAllocInitialMark.java caused SIGSEGV
pliden
parents: 18025
diff changeset
   291
    while (!_has_terminated) {
095cc0a63ed9 8037112: gc/g1/TestHumongousAllocInitialMark.java caused SIGSEGV
pliden
parents: 18025
diff changeset
   292
      Terminator_lock->wait();
095cc0a63ed9 8037112: gc/g1/TestHumongousAllocInitialMark.java caused SIGSEGV
pliden
parents: 18025
diff changeset
   293
    }
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   294
  }
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   295
}
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   296
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   297
void ConcurrentMarkThread::sleepBeforeNextCycle() {
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   298
  // We join here because we don't want to do the "shouldConcurrentMark()"
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   299
  // below while the world is otherwise stopped.
6766
839211600ad0 6983311: G1: LoopTest hangs when run with -XX:+ExplicitInvokesConcurrent
johnc
parents: 6058
diff changeset
   300
  assert(!in_progress(), "should have been cleared");
839211600ad0 6983311: G1: LoopTest hangs when run with -XX:+ExplicitInvokesConcurrent
johnc
parents: 6058
diff changeset
   301
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   302
  MutexLockerEx x(CGC_lock, Mutex::_no_safepoint_check_flag);
24093
095cc0a63ed9 8037112: gc/g1/TestHumongousAllocInitialMark.java caused SIGSEGV
pliden
parents: 18025
diff changeset
   303
  while (!started() && !_should_terminate) {
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   304
    CGC_lock->wait(Mutex::_no_safepoint_check_flag);
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   305
  }
24093
095cc0a63ed9 8037112: gc/g1/TestHumongousAllocInitialMark.java caused SIGSEGV
pliden
parents: 18025
diff changeset
   306
095cc0a63ed9 8037112: gc/g1/TestHumongousAllocInitialMark.java caused SIGSEGV
pliden
parents: 18025
diff changeset
   307
  if (started()) {
095cc0a63ed9 8037112: gc/g1/TestHumongousAllocInitialMark.java caused SIGSEGV
pliden
parents: 18025
diff changeset
   308
    set_in_progress();
095cc0a63ed9 8037112: gc/g1/TestHumongousAllocInitialMark.java caused SIGSEGV
pliden
parents: 18025
diff changeset
   309
  }
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   310
}
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   311
10769
983d377770fd 7099824: G1: we should take the pending list lock before doing the remark pause
johnc
parents: 10745
diff changeset
   312
// Note: As is the case with CMS - this method, although exported
983d377770fd 7099824: G1: we should take the pending list lock before doing the remark pause
johnc
parents: 10745
diff changeset
   313
// by the ConcurrentMarkThread, which is a non-JavaThread, can only
983d377770fd 7099824: G1: we should take the pending list lock before doing the remark pause
johnc
parents: 10745
diff changeset
   314
// be called by a JavaThread. Currently this is done at vm creation
983d377770fd 7099824: G1: we should take the pending list lock before doing the remark pause
johnc
parents: 10745
diff changeset
   315
// time (post-vm-init) by the main/Primordial (Java)Thread.
983d377770fd 7099824: G1: we should take the pending list lock before doing the remark pause
johnc
parents: 10745
diff changeset
   316
// XXX Consider changing this in the future to allow the CM thread
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   317
// itself to create this thread?
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   318
void ConcurrentMarkThread::makeSurrogateLockerThread(TRAPS) {
10769
983d377770fd 7099824: G1: we should take the pending list lock before doing the remark pause
johnc
parents: 10745
diff changeset
   319
  assert(UseG1GC, "SLT thread needed only for concurrent GC");
983d377770fd 7099824: G1: we should take the pending list lock before doing the remark pause
johnc
parents: 10745
diff changeset
   320
  assert(THREAD->is_Java_thread(), "must be a Java thread");
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   321
  assert(_slt == NULL, "SLT already created");
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   322
  _slt = SurrogateLockerThread::make(THREAD);
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   323
}