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