hotspot/src/share/vm/gc/g1/vm_operations_g1.cpp
author jprovino
Tue, 02 Aug 2016 16:39:33 -0400
changeset 40328 a2851f5f1cf6
parent 37985 539c597ee0fa
child 40892 330a02d935ad
permissions -rw-r--r--
8071770: G1 does not implement millis_since_last_gc which is needed by RMI GC Summary: G1 does not return a correct value for the CollectedHeap::millis_since_last_gc Reviewed-by: jmasa, tschatzl
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
     1
/*
37129
af29e306e50b 8151601: Cleanup locking of the Reference pending list
pliden
parents: 35061
diff changeset
     2
 * Copyright (c) 2001, 2016, 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: 4458
diff changeset
    19
 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
f4b087cbb361 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 4458
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: 4458
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: 6058
diff changeset
    25
#include "precompiled.hpp"
30764
fec48bf5a827 8079792: GC directory structure cleanup
pliden
parents: 30147
diff changeset
    26
#include "gc/g1/concurrentMarkThread.inline.hpp"
fec48bf5a827 8079792: GC directory structure cleanup
pliden
parents: 30147
diff changeset
    27
#include "gc/g1/g1CollectedHeap.inline.hpp"
37985
539c597ee0fa 8154154: Separate G1 specific policy code from the CollectorPolicy class hierarchy
mgerdin
parents: 37235
diff changeset
    28
#include "gc/g1/g1Policy.hpp"
33107
77bf0d2069a3 8134953: Make the GC ID available in a central place
brutisso
parents: 33105
diff changeset
    29
#include "gc/shared/gcId.hpp"
30764
fec48bf5a827 8079792: GC directory structure cleanup
pliden
parents: 30147
diff changeset
    30
#include "gc/g1/vm_operations_g1.hpp"
fec48bf5a827 8079792: GC directory structure cleanup
pliden
parents: 30147
diff changeset
    31
#include "gc/shared/gcTimer.hpp"
35061
be6025ebffea 8145092: Use Unified Logging for the GC logging
brutisso
parents: 33107
diff changeset
    32
#include "gc/shared/gcTraceTime.inline.hpp"
30764
fec48bf5a827 8079792: GC directory structure cleanup
pliden
parents: 30147
diff changeset
    33
#include "gc/shared/isGCActiveMark.hpp"
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6058
diff changeset
    34
#include "runtime/interfaceSupport.hpp"
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    35
29078
3b7dd035c20b 8072621: Clean up around VM_GC_Operations
mlarsson
parents: 27900
diff changeset
    36
VM_G1CollectForAllocation::VM_G1CollectForAllocation(uint gc_count_before,
3b7dd035c20b 8072621: Clean up around VM_GC_Operations
mlarsson
parents: 27900
diff changeset
    37
                                                     size_t word_size)
12227
371690c4f281 7147724: G1: hang in SurrogateLockerThread::manipulatePLL
johnc
parents: 11576
diff changeset
    38
  : VM_G1OperationWithAllocRequest(gc_count_before, word_size,
371690c4f281 7147724: G1: hang in SurrogateLockerThread::manipulatePLL
johnc
parents: 11576
diff changeset
    39
                                   GCCause::_allocation_failure) {
29079
832d35c45d0b 8066771: Refactor VM GC operations caused by allocation failure
mlarsson
parents: 29078
diff changeset
    40
  guarantee(word_size != 0, "An allocation should always be requested with this operation.");
7398
e4aa6d9bda09 6974966: G1: unnecessary direct-to-old allocations
tonyp
parents: 7397
diff changeset
    41
}
e4aa6d9bda09 6974966: G1: unnecessary direct-to-old allocations
tonyp
parents: 7397
diff changeset
    42
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    43
void VM_G1CollectForAllocation::doit() {
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    44
  G1CollectedHeap* g1h = G1CollectedHeap::heap();
12628
bd8e0e8f8e5c 7168294: G1: Some Full GCs incorrectly report GC cause as "No GC"
johnc
parents: 12378
diff changeset
    45
  GCCauseSetter x(g1h, _gc_cause);
26837
72a43d3841e7 8057536: Refactor G1 to allow context specific allocations
sjohanss
parents: 25350
diff changeset
    46
72a43d3841e7 8057536: Refactor G1 to allow context specific allocations
sjohanss
parents: 25350
diff changeset
    47
  _result = g1h->satisfy_failed_allocation(_word_size, allocation_context(), &_pause_succeeded);
7398
e4aa6d9bda09 6974966: G1: unnecessary direct-to-old allocations
tonyp
parents: 7397
diff changeset
    48
  assert(_result == NULL || _pause_succeeded,
e4aa6d9bda09 6974966: G1: unnecessary direct-to-old allocations
tonyp
parents: 7397
diff changeset
    49
         "if we get back a result, the pause should have succeeded");
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    50
}
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    51
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    52
void VM_G1CollectFull::doit() {
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    53
  G1CollectedHeap* g1h = G1CollectedHeap::heap();
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    54
  GCCauseSetter x(g1h, _gc_cause);
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    55
  g1h->do_full_collection(false /* clear_all_soft_refs */);
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
29078
3b7dd035c20b 8072621: Clean up around VM_GC_Operations
mlarsson
parents: 27900
diff changeset
    58
VM_G1IncCollectionPause::VM_G1IncCollectionPause(uint           gc_count_before,
3b7dd035c20b 8072621: Clean up around VM_GC_Operations
mlarsson
parents: 27900
diff changeset
    59
                                                 size_t         word_size,
3b7dd035c20b 8072621: Clean up around VM_GC_Operations
mlarsson
parents: 27900
diff changeset
    60
                                                 bool           should_initiate_conc_mark,
3b7dd035c20b 8072621: Clean up around VM_GC_Operations
mlarsson
parents: 27900
diff changeset
    61
                                                 double         target_pause_time_ms,
3b7dd035c20b 8072621: Clean up around VM_GC_Operations
mlarsson
parents: 27900
diff changeset
    62
                                                 GCCause::Cause gc_cause)
12227
371690c4f281 7147724: G1: hang in SurrogateLockerThread::manipulatePLL
johnc
parents: 11576
diff changeset
    63
  : VM_G1OperationWithAllocRequest(gc_count_before, word_size, gc_cause),
7398
e4aa6d9bda09 6974966: G1: unnecessary direct-to-old allocations
tonyp
parents: 7397
diff changeset
    64
    _should_initiate_conc_mark(should_initiate_conc_mark),
e4aa6d9bda09 6974966: G1: unnecessary direct-to-old allocations
tonyp
parents: 7397
diff changeset
    65
    _target_pause_time_ms(target_pause_time_ms),
12227
371690c4f281 7147724: G1: hang in SurrogateLockerThread::manipulatePLL
johnc
parents: 11576
diff changeset
    66
    _should_retry_gc(false),
12934
f9bc0e664918 7172388: G1: _total_full_collections should not be incremented for concurrent cycles
brutisso
parents: 12628
diff changeset
    67
    _old_marking_cycles_completed_before(0) {
7398
e4aa6d9bda09 6974966: G1: unnecessary direct-to-old allocations
tonyp
parents: 7397
diff changeset
    68
  guarantee(target_pause_time_ms > 0.0,
33105
294e48b4f704 8080775: Better argument formatting for assert() and friends
david
parents: 32185
diff changeset
    69
            "target_pause_time_ms = %1.6lf should be positive",
294e48b4f704 8080775: Better argument formatting for assert() and friends
david
parents: 32185
diff changeset
    70
            target_pause_time_ms);
7398
e4aa6d9bda09 6974966: G1: unnecessary direct-to-old allocations
tonyp
parents: 7397
diff changeset
    71
  _gc_cause = gc_cause;
e4aa6d9bda09 6974966: G1: unnecessary direct-to-old allocations
tonyp
parents: 7397
diff changeset
    72
}
e4aa6d9bda09 6974966: G1: unnecessary direct-to-old allocations
tonyp
parents: 7397
diff changeset
    73
12227
371690c4f281 7147724: G1: hang in SurrogateLockerThread::manipulatePLL
johnc
parents: 11576
diff changeset
    74
bool VM_G1IncCollectionPause::doit_prologue() {
29079
832d35c45d0b 8066771: Refactor VM GC operations caused by allocation failure
mlarsson
parents: 29078
diff changeset
    75
  bool res = VM_G1OperationWithAllocRequest::doit_prologue();
12227
371690c4f281 7147724: G1: hang in SurrogateLockerThread::manipulatePLL
johnc
parents: 11576
diff changeset
    76
  if (!res) {
371690c4f281 7147724: G1: hang in SurrogateLockerThread::manipulatePLL
johnc
parents: 11576
diff changeset
    77
    if (_should_initiate_conc_mark) {
371690c4f281 7147724: G1: hang in SurrogateLockerThread::manipulatePLL
johnc
parents: 11576
diff changeset
    78
      // The prologue can fail for a couple of reasons. The first is that another GC
371690c4f281 7147724: G1: hang in SurrogateLockerThread::manipulatePLL
johnc
parents: 11576
diff changeset
    79
      // got scheduled and prevented the scheduling of the initial mark GC. The
371690c4f281 7147724: G1: hang in SurrogateLockerThread::manipulatePLL
johnc
parents: 11576
diff changeset
    80
      // second is that the GC locker may be active and the heap can't be expanded.
371690c4f281 7147724: G1: hang in SurrogateLockerThread::manipulatePLL
johnc
parents: 11576
diff changeset
    81
      // In both cases we want to retry the GC so that the initial mark pause is
371690c4f281 7147724: G1: hang in SurrogateLockerThread::manipulatePLL
johnc
parents: 11576
diff changeset
    82
      // actually scheduled. In the second case, however, we should stall until
371690c4f281 7147724: G1: hang in SurrogateLockerThread::manipulatePLL
johnc
parents: 11576
diff changeset
    83
      // until the GC locker is no longer active and then retry the initial mark GC.
371690c4f281 7147724: G1: hang in SurrogateLockerThread::manipulatePLL
johnc
parents: 11576
diff changeset
    84
      _should_retry_gc = true;
371690c4f281 7147724: G1: hang in SurrogateLockerThread::manipulatePLL
johnc
parents: 11576
diff changeset
    85
    }
371690c4f281 7147724: G1: hang in SurrogateLockerThread::manipulatePLL
johnc
parents: 11576
diff changeset
    86
  }
371690c4f281 7147724: G1: hang in SurrogateLockerThread::manipulatePLL
johnc
parents: 11576
diff changeset
    87
  return res;
371690c4f281 7147724: G1: hang in SurrogateLockerThread::manipulatePLL
johnc
parents: 11576
diff changeset
    88
}
371690c4f281 7147724: G1: hang in SurrogateLockerThread::manipulatePLL
johnc
parents: 11576
diff changeset
    89
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    90
void VM_G1IncCollectionPause::doit() {
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    91
  G1CollectedHeap* g1h = G1CollectedHeap::heap();
27900
d24cd7a1ed38 8065579: WB method to start G1 concurrent mark cycle should be introduced
tschatzl
parents: 27887
diff changeset
    92
  assert(!_should_initiate_conc_mark || g1h->should_do_concurrent_full_gc(_gc_cause),
d24cd7a1ed38 8065579: WB method to start G1 concurrent mark cycle should be introduced
tschatzl
parents: 27887
diff changeset
    93
      "only a GC locker, a System.gc(), stats update, whitebox, or a hum allocation induced GC should start a cycle");
6058
9c9aec6ab47d 6944166: G1: explicit GCs are not always handled correctly
tonyp
parents: 5547
diff changeset
    94
7398
e4aa6d9bda09 6974966: G1: unnecessary direct-to-old allocations
tonyp
parents: 7397
diff changeset
    95
  if (_word_size > 0) {
e4aa6d9bda09 6974966: G1: unnecessary direct-to-old allocations
tonyp
parents: 7397
diff changeset
    96
    // An allocation has been requested. So, try to do that first.
32185
49a57ff2c3cb 8073052: Rename and clean up the allocation manager hierarchy in g1Allocator.?pp
tschatzl
parents: 31032
diff changeset
    97
    _result = g1h->attempt_allocation_at_safepoint(_word_size,
49a57ff2c3cb 8073052: Rename and clean up the allocation manager hierarchy in g1Allocator.?pp
tschatzl
parents: 31032
diff changeset
    98
                                                   allocation_context(),
49a57ff2c3cb 8073052: Rename and clean up the allocation manager hierarchy in g1Allocator.?pp
tschatzl
parents: 31032
diff changeset
    99
                                                   false /* expect_null_cur_alloc_region */);
7398
e4aa6d9bda09 6974966: G1: unnecessary direct-to-old allocations
tonyp
parents: 7397
diff changeset
   100
    if (_result != NULL) {
e4aa6d9bda09 6974966: G1: unnecessary direct-to-old allocations
tonyp
parents: 7397
diff changeset
   101
      // If we can successfully allocate before we actually do the
e4aa6d9bda09 6974966: G1: unnecessary direct-to-old allocations
tonyp
parents: 7397
diff changeset
   102
      // pause then we will consider this pause successful.
e4aa6d9bda09 6974966: G1: unnecessary direct-to-old allocations
tonyp
parents: 7397
diff changeset
   103
      _pause_succeeded = true;
e4aa6d9bda09 6974966: G1: unnecessary direct-to-old allocations
tonyp
parents: 7397
diff changeset
   104
      return;
e4aa6d9bda09 6974966: G1: unnecessary direct-to-old allocations
tonyp
parents: 7397
diff changeset
   105
    }
e4aa6d9bda09 6974966: G1: unnecessary direct-to-old allocations
tonyp
parents: 7397
diff changeset
   106
  }
e4aa6d9bda09 6974966: G1: unnecessary direct-to-old allocations
tonyp
parents: 7397
diff changeset
   107
4458
075a9ef4e467 6902303: G1: ScavengeALot should cause an incremental, rather than a full, collection
ysr
parents: 3261
diff changeset
   108
  GCCauseSetter x(g1h, _gc_cause);
6058
9c9aec6ab47d 6944166: G1: explicit GCs are not always handled correctly
tonyp
parents: 5547
diff changeset
   109
  if (_should_initiate_conc_mark) {
12934
f9bc0e664918 7172388: G1: _total_full_collections should not be incremented for concurrent cycles
brutisso
parents: 12628
diff changeset
   110
    // It's safer to read old_marking_cycles_completed() here, given
6058
9c9aec6ab47d 6944166: G1: explicit GCs are not always handled correctly
tonyp
parents: 5547
diff changeset
   111
    // that noone else will be updating it concurrently. Since we'll
9c9aec6ab47d 6944166: G1: explicit GCs are not always handled correctly
tonyp
parents: 5547
diff changeset
   112
    // only need it if we're initiating a marking cycle, no point in
9c9aec6ab47d 6944166: G1: explicit GCs are not always handled correctly
tonyp
parents: 5547
diff changeset
   113
    // setting it earlier.
12934
f9bc0e664918 7172388: G1: _total_full_collections should not be incremented for concurrent cycles
brutisso
parents: 12628
diff changeset
   114
    _old_marking_cycles_completed_before = g1h->old_marking_cycles_completed();
6058
9c9aec6ab47d 6944166: G1: explicit GCs are not always handled correctly
tonyp
parents: 5547
diff changeset
   115
9c9aec6ab47d 6944166: G1: explicit GCs are not always handled correctly
tonyp
parents: 5547
diff changeset
   116
    // At this point we are supposed to start a concurrent cycle. We
9c9aec6ab47d 6944166: G1: explicit GCs are not always handled correctly
tonyp
parents: 5547
diff changeset
   117
    // will do so if one is not already in progress.
10523
cdb54c167ab0 7050392: G1: Introduce flag to generate a log of the G1 ergonomic decisions
tonyp
parents: 9996
diff changeset
   118
    bool res = g1h->g1_policy()->force_initial_mark_if_outside_cycle(_gc_cause);
9996
3851e45f93e7 7045751: G1: +ExplicitGCInvokesConcurrent causes excessive single region evacuation pauses
johnc
parents: 8921
diff changeset
   119
3851e45f93e7 7045751: G1: +ExplicitGCInvokesConcurrent causes excessive single region evacuation pauses
johnc
parents: 8921
diff changeset
   120
    // The above routine returns true if we were able to force the
3851e45f93e7 7045751: G1: +ExplicitGCInvokesConcurrent causes excessive single region evacuation pauses
johnc
parents: 8921
diff changeset
   121
    // next GC pause to be an initial mark; it returns false if a
3851e45f93e7 7045751: G1: +ExplicitGCInvokesConcurrent causes excessive single region evacuation pauses
johnc
parents: 8921
diff changeset
   122
    // marking cycle is already in progress.
3851e45f93e7 7045751: G1: +ExplicitGCInvokesConcurrent causes excessive single region evacuation pauses
johnc
parents: 8921
diff changeset
   123
    //
12227
371690c4f281 7147724: G1: hang in SurrogateLockerThread::manipulatePLL
johnc
parents: 11576
diff changeset
   124
    // If a marking cycle is already in progress just return and skip the
371690c4f281 7147724: G1: hang in SurrogateLockerThread::manipulatePLL
johnc
parents: 11576
diff changeset
   125
    // pause below - if the reason for requesting this initial mark pause
371690c4f281 7147724: G1: hang in SurrogateLockerThread::manipulatePLL
johnc
parents: 11576
diff changeset
   126
    // was due to a System.gc() then the requesting thread should block in
371690c4f281 7147724: G1: hang in SurrogateLockerThread::manipulatePLL
johnc
parents: 11576
diff changeset
   127
    // doit_epilogue() until the marking cycle is complete.
371690c4f281 7147724: G1: hang in SurrogateLockerThread::manipulatePLL
johnc
parents: 11576
diff changeset
   128
    //
371690c4f281 7147724: G1: hang in SurrogateLockerThread::manipulatePLL
johnc
parents: 11576
diff changeset
   129
    // If this initial mark pause was requested as part of a humongous
371690c4f281 7147724: G1: hang in SurrogateLockerThread::manipulatePLL
johnc
parents: 11576
diff changeset
   130
    // allocation then we know that the marking cycle must just have
371690c4f281 7147724: G1: hang in SurrogateLockerThread::manipulatePLL
johnc
parents: 11576
diff changeset
   131
    // been started by another thread (possibly also allocating a humongous
371690c4f281 7147724: G1: hang in SurrogateLockerThread::manipulatePLL
johnc
parents: 11576
diff changeset
   132
    // object) as there was no active marking cycle when the requesting
371690c4f281 7147724: G1: hang in SurrogateLockerThread::manipulatePLL
johnc
parents: 11576
diff changeset
   133
    // thread checked before calling collect() in
371690c4f281 7147724: G1: hang in SurrogateLockerThread::manipulatePLL
johnc
parents: 11576
diff changeset
   134
    // attempt_allocation_humongous(). Retrying the GC, in this case,
371690c4f281 7147724: G1: hang in SurrogateLockerThread::manipulatePLL
johnc
parents: 11576
diff changeset
   135
    // will cause the requesting thread to spin inside collect() until the
371690c4f281 7147724: G1: hang in SurrogateLockerThread::manipulatePLL
johnc
parents: 11576
diff changeset
   136
    // just started marking cycle is complete - which may be a while. So
371690c4f281 7147724: G1: hang in SurrogateLockerThread::manipulatePLL
johnc
parents: 11576
diff changeset
   137
    // we do NOT retry the GC.
9996
3851e45f93e7 7045751: G1: +ExplicitGCInvokesConcurrent causes excessive single region evacuation pauses
johnc
parents: 8921
diff changeset
   138
    if (!res) {
12227
371690c4f281 7147724: G1: hang in SurrogateLockerThread::manipulatePLL
johnc
parents: 11576
diff changeset
   139
      assert(_word_size == 0, "Concurrent Full GC/Humongous Object IM shouldn't be allocating");
371690c4f281 7147724: G1: hang in SurrogateLockerThread::manipulatePLL
johnc
parents: 11576
diff changeset
   140
      if (_gc_cause != GCCause::_g1_humongous_allocation) {
371690c4f281 7147724: G1: hang in SurrogateLockerThread::manipulatePLL
johnc
parents: 11576
diff changeset
   141
        _should_retry_gc = true;
371690c4f281 7147724: G1: hang in SurrogateLockerThread::manipulatePLL
johnc
parents: 11576
diff changeset
   142
      }
9996
3851e45f93e7 7045751: G1: +ExplicitGCInvokesConcurrent causes excessive single region evacuation pauses
johnc
parents: 8921
diff changeset
   143
      return;
3851e45f93e7 7045751: G1: +ExplicitGCInvokesConcurrent causes excessive single region evacuation pauses
johnc
parents: 8921
diff changeset
   144
    }
6058
9c9aec6ab47d 6944166: G1: explicit GCs are not always handled correctly
tonyp
parents: 5547
diff changeset
   145
  }
7398
e4aa6d9bda09 6974966: G1: unnecessary direct-to-old allocations
tonyp
parents: 7397
diff changeset
   146
e4aa6d9bda09 6974966: G1: unnecessary direct-to-old allocations
tonyp
parents: 7397
diff changeset
   147
  _pause_succeeded =
e4aa6d9bda09 6974966: G1: unnecessary direct-to-old allocations
tonyp
parents: 7397
diff changeset
   148
    g1h->do_collection_pause_at_safepoint(_target_pause_time_ms);
e4aa6d9bda09 6974966: G1: unnecessary direct-to-old allocations
tonyp
parents: 7397
diff changeset
   149
  if (_pause_succeeded && _word_size > 0) {
e4aa6d9bda09 6974966: G1: unnecessary direct-to-old allocations
tonyp
parents: 7397
diff changeset
   150
    // An allocation had been requested.
32185
49a57ff2c3cb 8073052: Rename and clean up the allocation manager hierarchy in g1Allocator.?pp
tschatzl
parents: 31032
diff changeset
   151
    _result = g1h->attempt_allocation_at_safepoint(_word_size,
49a57ff2c3cb 8073052: Rename and clean up the allocation manager hierarchy in g1Allocator.?pp
tschatzl
parents: 31032
diff changeset
   152
                                                   allocation_context(),
49a57ff2c3cb 8073052: Rename and clean up the allocation manager hierarchy in g1Allocator.?pp
tschatzl
parents: 31032
diff changeset
   153
                                                   true /* expect_null_cur_alloc_region */);
7398
e4aa6d9bda09 6974966: G1: unnecessary direct-to-old allocations
tonyp
parents: 7397
diff changeset
   154
  } else {
e4aa6d9bda09 6974966: G1: unnecessary direct-to-old allocations
tonyp
parents: 7397
diff changeset
   155
    assert(_result == NULL, "invariant");
12227
371690c4f281 7147724: G1: hang in SurrogateLockerThread::manipulatePLL
johnc
parents: 11576
diff changeset
   156
    if (!_pause_succeeded) {
371690c4f281 7147724: G1: hang in SurrogateLockerThread::manipulatePLL
johnc
parents: 11576
diff changeset
   157
      // Another possible reason reason for the pause to not be successful
371690c4f281 7147724: G1: hang in SurrogateLockerThread::manipulatePLL
johnc
parents: 11576
diff changeset
   158
      // is that, again, the GC locker is active (and has become active
371690c4f281 7147724: G1: hang in SurrogateLockerThread::manipulatePLL
johnc
parents: 11576
diff changeset
   159
      // since the prologue was executed). In this case we should retry
371690c4f281 7147724: G1: hang in SurrogateLockerThread::manipulatePLL
johnc
parents: 11576
diff changeset
   160
      // the pause after waiting for the GC locker to become inactive.
371690c4f281 7147724: G1: hang in SurrogateLockerThread::manipulatePLL
johnc
parents: 11576
diff changeset
   161
      _should_retry_gc = true;
371690c4f281 7147724: G1: hang in SurrogateLockerThread::manipulatePLL
johnc
parents: 11576
diff changeset
   162
    }
7398
e4aa6d9bda09 6974966: G1: unnecessary direct-to-old allocations
tonyp
parents: 7397
diff changeset
   163
  }
6058
9c9aec6ab47d 6944166: G1: explicit GCs are not always handled correctly
tonyp
parents: 5547
diff changeset
   164
}
9c9aec6ab47d 6944166: G1: explicit GCs are not always handled correctly
tonyp
parents: 5547
diff changeset
   165
9c9aec6ab47d 6944166: G1: explicit GCs are not always handled correctly
tonyp
parents: 5547
diff changeset
   166
void VM_G1IncCollectionPause::doit_epilogue() {
29079
832d35c45d0b 8066771: Refactor VM GC operations caused by allocation failure
mlarsson
parents: 29078
diff changeset
   167
  VM_G1OperationWithAllocRequest::doit_epilogue();
6058
9c9aec6ab47d 6944166: G1: explicit GCs are not always handled correctly
tonyp
parents: 5547
diff changeset
   168
9c9aec6ab47d 6944166: G1: explicit GCs are not always handled correctly
tonyp
parents: 5547
diff changeset
   169
  // If the pause was initiated by a System.gc() and
9c9aec6ab47d 6944166: G1: explicit GCs are not always handled correctly
tonyp
parents: 5547
diff changeset
   170
  // +ExplicitGCInvokesConcurrent, we have to wait here for the cycle
9c9aec6ab47d 6944166: G1: explicit GCs are not always handled correctly
tonyp
parents: 5547
diff changeset
   171
  // that just started (or maybe one that was already in progress) to
9c9aec6ab47d 6944166: G1: explicit GCs are not always handled correctly
tonyp
parents: 5547
diff changeset
   172
  // finish.
31032
8e72621ca186 8072913: [REDO] GCCause should distinguish jcmd GC.run from System.gc()
ysuenaga
parents: 30764
diff changeset
   173
  if (GCCause::is_user_requested_gc(_gc_cause) &&
6058
9c9aec6ab47d 6944166: G1: explicit GCs are not always handled correctly
tonyp
parents: 5547
diff changeset
   174
      _should_initiate_conc_mark) {
9c9aec6ab47d 6944166: G1: explicit GCs are not always handled correctly
tonyp
parents: 5547
diff changeset
   175
    assert(ExplicitGCInvokesConcurrent,
9c9aec6ab47d 6944166: G1: explicit GCs are not always handled correctly
tonyp
parents: 5547
diff changeset
   176
           "the only way to be here is if ExplicitGCInvokesConcurrent is set");
9c9aec6ab47d 6944166: G1: explicit GCs are not always handled correctly
tonyp
parents: 5547
diff changeset
   177
9c9aec6ab47d 6944166: G1: explicit GCs are not always handled correctly
tonyp
parents: 5547
diff changeset
   178
    G1CollectedHeap* g1h = G1CollectedHeap::heap();
9c9aec6ab47d 6944166: G1: explicit GCs are not always handled correctly
tonyp
parents: 5547
diff changeset
   179
12934
f9bc0e664918 7172388: G1: _total_full_collections should not be incremented for concurrent cycles
brutisso
parents: 12628
diff changeset
   180
    // In the doit() method we saved g1h->old_marking_cycles_completed()
f9bc0e664918 7172388: G1: _total_full_collections should not be incremented for concurrent cycles
brutisso
parents: 12628
diff changeset
   181
    // in the _old_marking_cycles_completed_before field. We have to
f9bc0e664918 7172388: G1: _total_full_collections should not be incremented for concurrent cycles
brutisso
parents: 12628
diff changeset
   182
    // wait until we observe that g1h->old_marking_cycles_completed()
6058
9c9aec6ab47d 6944166: G1: explicit GCs are not always handled correctly
tonyp
parents: 5547
diff changeset
   183
    // has increased by at least one. This can happen if a) we started
9c9aec6ab47d 6944166: G1: explicit GCs are not always handled correctly
tonyp
parents: 5547
diff changeset
   184
    // a cycle and it completes, b) a cycle already in progress
9c9aec6ab47d 6944166: G1: explicit GCs are not always handled correctly
tonyp
parents: 5547
diff changeset
   185
    // completes, or c) a Full GC happens.
9c9aec6ab47d 6944166: G1: explicit GCs are not always handled correctly
tonyp
parents: 5547
diff changeset
   186
9c9aec6ab47d 6944166: G1: explicit GCs are not always handled correctly
tonyp
parents: 5547
diff changeset
   187
    // If the condition has already been reached, there's no point in
9c9aec6ab47d 6944166: G1: explicit GCs are not always handled correctly
tonyp
parents: 5547
diff changeset
   188
    // actually taking the lock and doing the wait.
12934
f9bc0e664918 7172388: G1: _total_full_collections should not be incremented for concurrent cycles
brutisso
parents: 12628
diff changeset
   189
    if (g1h->old_marking_cycles_completed() <=
f9bc0e664918 7172388: G1: _total_full_collections should not be incremented for concurrent cycles
brutisso
parents: 12628
diff changeset
   190
                                          _old_marking_cycles_completed_before) {
6058
9c9aec6ab47d 6944166: G1: explicit GCs are not always handled correctly
tonyp
parents: 5547
diff changeset
   191
      // The following is largely copied from CMS
9c9aec6ab47d 6944166: G1: explicit GCs are not always handled correctly
tonyp
parents: 5547
diff changeset
   192
9c9aec6ab47d 6944166: G1: explicit GCs are not always handled correctly
tonyp
parents: 5547
diff changeset
   193
      Thread* thr = Thread::current();
9c9aec6ab47d 6944166: G1: explicit GCs are not always handled correctly
tonyp
parents: 5547
diff changeset
   194
      assert(thr->is_Java_thread(), "invariant");
9c9aec6ab47d 6944166: G1: explicit GCs are not always handled correctly
tonyp
parents: 5547
diff changeset
   195
      JavaThread* jt = (JavaThread*)thr;
9c9aec6ab47d 6944166: G1: explicit GCs are not always handled correctly
tonyp
parents: 5547
diff changeset
   196
      ThreadToNativeFromVM native(jt);
9c9aec6ab47d 6944166: G1: explicit GCs are not always handled correctly
tonyp
parents: 5547
diff changeset
   197
9c9aec6ab47d 6944166: G1: explicit GCs are not always handled correctly
tonyp
parents: 5547
diff changeset
   198
      MutexLockerEx x(FullGCCount_lock, Mutex::_no_safepoint_check_flag);
12934
f9bc0e664918 7172388: G1: _total_full_collections should not be incremented for concurrent cycles
brutisso
parents: 12628
diff changeset
   199
      while (g1h->old_marking_cycles_completed() <=
f9bc0e664918 7172388: G1: _total_full_collections should not be incremented for concurrent cycles
brutisso
parents: 12628
diff changeset
   200
                                          _old_marking_cycles_completed_before) {
6058
9c9aec6ab47d 6944166: G1: explicit GCs are not always handled correctly
tonyp
parents: 5547
diff changeset
   201
        FullGCCount_lock->wait(Mutex::_no_safepoint_check_flag);
9c9aec6ab47d 6944166: G1: explicit GCs are not always handled correctly
tonyp
parents: 5547
diff changeset
   202
      }
9c9aec6ab47d 6944166: G1: explicit GCs are not always handled correctly
tonyp
parents: 5547
diff changeset
   203
    }
9c9aec6ab47d 6944166: G1: explicit GCs are not always handled correctly
tonyp
parents: 5547
diff changeset
   204
  }
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   205
}
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   206
10769
983d377770fd 7099824: G1: we should take the pending list lock before doing the remark pause
johnc
parents: 10523
diff changeset
   207
void VM_CGC_Operation::acquire_pending_list_lock() {
37129
af29e306e50b 8151601: Cleanup locking of the Reference pending list
pliden
parents: 35061
diff changeset
   208
  _pending_list_locker.lock();
10769
983d377770fd 7099824: G1: we should take the pending list lock before doing the remark pause
johnc
parents: 10523
diff changeset
   209
}
983d377770fd 7099824: G1: we should take the pending list lock before doing the remark pause
johnc
parents: 10523
diff changeset
   210
983d377770fd 7099824: G1: we should take the pending list lock before doing the remark pause
johnc
parents: 10523
diff changeset
   211
void VM_CGC_Operation::release_and_notify_pending_list_lock() {
37129
af29e306e50b 8151601: Cleanup locking of the Reference pending list
pliden
parents: 35061
diff changeset
   212
  _pending_list_locker.unlock();
10769
983d377770fd 7099824: G1: we should take the pending list lock before doing the remark pause
johnc
parents: 10523
diff changeset
   213
}
983d377770fd 7099824: G1: we should take the pending list lock before doing the remark pause
johnc
parents: 10523
diff changeset
   214
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   215
void VM_CGC_Operation::doit() {
35061
be6025ebffea 8145092: Use Unified Logging for the GC logging
brutisso
parents: 33107
diff changeset
   216
  GCIdMark gc_id_mark(_gc_id);
be6025ebffea 8145092: Use Unified Logging for the GC logging
brutisso
parents: 33107
diff changeset
   217
  GCTraceCPUTime tcpu;
30147
af9a41999c6e 8076314: Remove the static instance variable SharedHeap:: _sh
brutisso
parents: 29079
diff changeset
   218
  G1CollectedHeap* g1h = G1CollectedHeap::heap();
37137
62fd3fb4b1b1 8151085: Change G1 concurrent timer and tracer measuring time
sangheki
parents: 37129
diff changeset
   219
  GCTraceTime(Info, gc) t(_printGCMessage, g1h->concurrent_mark()->gc_timer_cm(), GCCause::_no_gc, true);
30147
af9a41999c6e 8076314: Remove the static instance variable SharedHeap:: _sh
brutisso
parents: 29079
diff changeset
   220
  IsGCActiveMark x;
af9a41999c6e 8076314: Remove the static instance variable SharedHeap:: _sh
brutisso
parents: 29079
diff changeset
   221
  _cl->do_void();
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   222
}
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   223
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   224
bool VM_CGC_Operation::doit_prologue() {
10769
983d377770fd 7099824: G1: we should take the pending list lock before doing the remark pause
johnc
parents: 10523
diff changeset
   225
  // Note the relative order of the locks must match that in
983d377770fd 7099824: G1: we should take the pending list lock before doing the remark pause
johnc
parents: 10523
diff changeset
   226
  // VM_GC_Operation::doit_prologue() or deadlocks can occur
37129
af29e306e50b 8151601: Cleanup locking of the Reference pending list
pliden
parents: 35061
diff changeset
   227
  if (_needs_pending_list_lock) {
12227
371690c4f281 7147724: G1: hang in SurrogateLockerThread::manipulatePLL
johnc
parents: 11576
diff changeset
   228
    acquire_pending_list_lock();
371690c4f281 7147724: G1: hang in SurrogateLockerThread::manipulatePLL
johnc
parents: 11576
diff changeset
   229
  }
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   230
  Heap_lock->lock();
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   231
  return true;
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   232
}
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   233
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   234
void VM_CGC_Operation::doit_epilogue() {
10769
983d377770fd 7099824: G1: we should take the pending list lock before doing the remark pause
johnc
parents: 10523
diff changeset
   235
  // Note the relative order of the unlocks must match that in
983d377770fd 7099824: G1: we should take the pending list lock before doing the remark pause
johnc
parents: 10523
diff changeset
   236
  // VM_GC_Operation::doit_epilogue()
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   237
  Heap_lock->unlock();
37129
af29e306e50b 8151601: Cleanup locking of the Reference pending list
pliden
parents: 35061
diff changeset
   238
  if (_needs_pending_list_lock) {
12227
371690c4f281 7147724: G1: hang in SurrogateLockerThread::manipulatePLL
johnc
parents: 11576
diff changeset
   239
    release_and_notify_pending_list_lock();
371690c4f281 7147724: G1: hang in SurrogateLockerThread::manipulatePLL
johnc
parents: 11576
diff changeset
   240
  }
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   241
}