src/hotspot/share/gc/g1/g1ConcurrentMarkThread.cpp
author ehelin
Wed, 04 Apr 2018 10:12:02 +0200
changeset 49667 2fef34f04314
parent 49644 src/hotspot/share/gc/g1/concurrentMarkThread.cpp@50a01910e00a
child 49668 8263950638ed
permissions -rw-r--r--
8200626: Restore history for g1ConcurrentMarkThread.* Reviewed-by: shade, sjohanss
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
46384
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
     1
/*
49031
e4a0cc16b050 8198369: Clean up GCId and GCIdMark
pliden
parents: 47756
diff changeset
     2
 * Copyright (c) 2001, 2018, 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"
36084
9a3bf78e9a76 8148992: VM can hang on exit if root region scanning is initiated but not executed
brutisso
parents: 35943
diff changeset
    26
#include "classfile/classLoaderData.hpp"
37134
9dd3975ca940 8151711: Move G1 number sequences out of the G1 collector policy
mgerdin
parents: 37129
diff changeset
    27
#include "gc/g1/g1Analytics.hpp"
30764
fec48bf5a827 8079792: GC directory structure cleanup
pliden
parents: 30613
diff changeset
    28
#include "gc/g1/g1CollectedHeap.inline.hpp"
37414
2672ba9af0dc 8151386: Extract card live data out of G1ConcurrentMark
tschatzl
parents: 37413
diff changeset
    29
#include "gc/g1/g1ConcurrentMark.inline.hpp"
49667
2fef34f04314 8200626: Restore history for g1ConcurrentMarkThread.*
ehelin
parents: 49644
diff changeset
    30
#include "gc/g1/g1ConcurrentMarkThread.inline.hpp"
30764
fec48bf5a827 8079792: GC directory structure cleanup
pliden
parents: 30613
diff changeset
    31
#include "gc/g1/g1MMUTracker.hpp"
37985
539c597ee0fa 8154154: Separate G1 specific policy code from the CollectorPolicy class hierarchy
mgerdin
parents: 37414
diff changeset
    32
#include "gc/g1/g1Policy.hpp"
49607
acffe6ff3ae7 8180415: Rebuild remembered sets during the concurrent cycle
tschatzl
parents: 49361
diff changeset
    33
#include "gc/g1/g1RemSet.hpp"
30764
fec48bf5a827 8079792: GC directory structure cleanup
pliden
parents: 30613
diff changeset
    34
#include "gc/g1/vm_operations_g1.hpp"
46384
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
    35
#include "gc/shared/concurrentGCPhaseManager.hpp"
33107
77bf0d2069a3 8134953: Make the GC ID available in a central place
brutisso
parents: 33106
diff changeset
    36
#include "gc/shared/gcId.hpp"
30764
fec48bf5a827 8079792: GC directory structure cleanup
pliden
parents: 30613
diff changeset
    37
#include "gc/shared/gcTrace.hpp"
35061
be6025ebffea 8145092: Use Unified Logging for the GC logging
brutisso
parents: 33608
diff changeset
    38
#include "gc/shared/gcTraceTime.inline.hpp"
47647
64dba69fc528 8189276: Make SuspendibleThreadSet and related code available to other GCs
rkennke
parents: 47216
diff changeset
    39
#include "gc/shared/suspendibleThreadSet.hpp"
35061
be6025ebffea 8145092: Use Unified Logging for the GC logging
brutisso
parents: 33608
diff changeset
    40
#include "logging/log.hpp"
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6766
diff changeset
    41
#include "memory/resourceArea.hpp"
49361
1956d0ec092a 8199319: Remove handles.inline.hpp include from reflectionUtils.hpp
stefank
parents: 49031
diff changeset
    42
#include "runtime/handles.inline.hpp"
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6766
diff changeset
    43
#include "runtime/vmThread.hpp"
46384
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
    44
#include "utilities/debug.hpp"
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    45
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    46
// ======= Concurrent Mark Thread ========
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    47
46384
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
    48
// Check order in EXPAND_CURRENT_PHASES
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
    49
STATIC_ASSERT(ConcurrentGCPhaseManager::UNCONSTRAINED_PHASE <
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
    50
              ConcurrentGCPhaseManager::IDLE_PHASE);
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
    51
49632
64f9ebc85e67 8197573: Remove concurrent cleanup and secondary free list handling
tschatzl
parents: 49607
diff changeset
    52
#define EXPAND_CONCURRENT_PHASES(expander)                                 \
64f9ebc85e67 8197573: Remove concurrent cleanup and secondary free list handling
tschatzl
parents: 49607
diff changeset
    53
  expander(ANY, = ConcurrentGCPhaseManager::UNCONSTRAINED_PHASE, NULL)     \
64f9ebc85e67 8197573: Remove concurrent cleanup and secondary free list handling
tschatzl
parents: 49607
diff changeset
    54
  expander(IDLE, = ConcurrentGCPhaseManager::IDLE_PHASE, NULL)             \
64f9ebc85e67 8197573: Remove concurrent cleanup and secondary free list handling
tschatzl
parents: 49607
diff changeset
    55
  expander(CONCURRENT_CYCLE,, "Concurrent Cycle")                          \
64f9ebc85e67 8197573: Remove concurrent cleanup and secondary free list handling
tschatzl
parents: 49607
diff changeset
    56
  expander(CLEAR_CLAIMED_MARKS,, "Concurrent Clear Claimed Marks")         \
64f9ebc85e67 8197573: Remove concurrent cleanup and secondary free list handling
tschatzl
parents: 49607
diff changeset
    57
  expander(SCAN_ROOT_REGIONS,, "Concurrent Scan Root Regions")             \
64f9ebc85e67 8197573: Remove concurrent cleanup and secondary free list handling
tschatzl
parents: 49607
diff changeset
    58
  expander(CONCURRENT_MARK,, "Concurrent Mark")                            \
64f9ebc85e67 8197573: Remove concurrent cleanup and secondary free list handling
tschatzl
parents: 49607
diff changeset
    59
  expander(MARK_FROM_ROOTS,, "Concurrent Mark From Roots")                 \
64f9ebc85e67 8197573: Remove concurrent cleanup and secondary free list handling
tschatzl
parents: 49607
diff changeset
    60
  expander(BEFORE_REMARK,, NULL)                                           \
64f9ebc85e67 8197573: Remove concurrent cleanup and secondary free list handling
tschatzl
parents: 49607
diff changeset
    61
  expander(REMARK,, NULL)                                                  \
49607
acffe6ff3ae7 8180415: Rebuild remembered sets during the concurrent cycle
tschatzl
parents: 49361
diff changeset
    62
  expander(REBUILD_REMEMBERED_SETS,, "Concurrent Rebuild Remembered Sets") \
49632
64f9ebc85e67 8197573: Remove concurrent cleanup and secondary free list handling
tschatzl
parents: 49607
diff changeset
    63
  expander(CLEANUP_FOR_NEXT_MARK,, "Concurrent Cleanup for Next Mark")     \
46384
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
    64
  /* */
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
    65
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
    66
class G1ConcurrentPhase : public AllStatic {
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
    67
public:
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
    68
  enum {
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
    69
#define CONCURRENT_PHASE_ENUM(tag, value, ignore_title) tag value,
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
    70
    EXPAND_CONCURRENT_PHASES(CONCURRENT_PHASE_ENUM)
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
    71
#undef CONCURRENT_PHASE_ENUM
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
    72
    PHASE_ID_LIMIT
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
    73
  };
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
    74
};
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
    75
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    76
// 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
    77
49667
2fef34f04314 8200626: Restore history for g1ConcurrentMarkThread.*
ehelin
parents: 49644
diff changeset
    78
G1ConcurrentMarkThread::G1ConcurrentMarkThread(G1ConcurrentMark* cm) :
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    79
  ConcurrentGCThread(),
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    80
  _cm(cm),
31597
f63dde8cc8f7 8129626: G1: set_in_progress() and clear_started() needs a barrier on non-TSO platforms
brutisso
parents: 31397
diff changeset
    81
  _state(Idle),
46384
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
    82
  _phase_manager_stack(),
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    83
  _vtime_accum(0.0),
11583
83a7383de44c 6484965: G1: piggy-back liveness accounting phase on marking
johnc
parents: 11175
diff changeset
    84
  _vtime_mark_accum(0.0) {
24331
c0bc7e5653fb 6885993: Named Thread: introduce print() and print_on(outputStream* st) methods
zgu
parents: 24094
diff changeset
    85
29464
02c245ad3ec9 8073545: Use shorter and more descriptive names for GC worker threads
david
parents: 27885
diff changeset
    86
  set_name("G1 Main Marker");
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    87
  create_and_start();
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    88
}
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    89
49644
50a01910e00a 8151171: Bring g1ConcurrentMark files up to current coding conventions
tschatzl
parents: 49632
diff changeset
    90
class CMRemark : public VoidClosure {
35943
e726308008c0 8148973: Rename g1/concurrentMark.{hpp,cpp,inline.hpp} to g1/g1ConcurrentMark.{hpp,cpp,inline.hpp}
ehelin
parents: 35204
diff changeset
    91
  G1ConcurrentMark* _cm;
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    92
public:
49644
50a01910e00a 8151171: Bring g1ConcurrentMark files up to current coding conventions
tschatzl
parents: 49632
diff changeset
    93
  CMRemark(G1ConcurrentMark* cm) : _cm(cm) {}
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    94
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    95
  void do_void(){
49644
50a01910e00a 8151171: Bring g1ConcurrentMark files up to current coding conventions
tschatzl
parents: 49632
diff changeset
    96
    _cm->remark();
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    97
  }
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    98
};
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    99
49644
50a01910e00a 8151171: Bring g1ConcurrentMark files up to current coding conventions
tschatzl
parents: 49632
diff changeset
   100
class CMCleanup : public VoidClosure {
35943
e726308008c0 8148973: Rename g1/concurrentMark.{hpp,cpp,inline.hpp} to g1/g1ConcurrentMark.{hpp,cpp,inline.hpp}
ehelin
parents: 35204
diff changeset
   101
  G1ConcurrentMark* _cm;
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   102
public:
49644
50a01910e00a 8151171: Bring g1ConcurrentMark files up to current coding conventions
tschatzl
parents: 49632
diff changeset
   103
  CMCleanup(G1ConcurrentMark* cm) : _cm(cm) {}
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   104
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   105
  void do_void(){
49644
50a01910e00a 8151171: Bring g1ConcurrentMark files up to current coding conventions
tschatzl
parents: 49632
diff changeset
   106
    _cm->cleanup();
50a01910e00a 8151171: Bring g1ConcurrentMark files up to current coding conventions
tschatzl
parents: 49632
diff changeset
   107
  }
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   108
};
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   109
49667
2fef34f04314 8200626: Restore history for g1ConcurrentMarkThread.*
ehelin
parents: 49644
diff changeset
   110
double G1ConcurrentMarkThread::mmu_sleep_time(G1Policy* g1_policy, bool remark) {
47756
55714c3d544c 8185278: TestGreyReclaimedHumongousObjects.java fails guarantee(index != trim_index(_head_index + 1)) failed: should not go past head
sangheki
parents: 47678
diff changeset
   111
  // There are 3 reasons to use SuspendibleThreadSetJoiner.
55714c3d544c 8185278: TestGreyReclaimedHumongousObjects.java fails guarantee(index != trim_index(_head_index + 1)) failed: should not go past head
sangheki
parents: 47678
diff changeset
   112
  // 1. To avoid concurrency problem.
55714c3d544c 8185278: TestGreyReclaimedHumongousObjects.java fails guarantee(index != trim_index(_head_index + 1)) failed: should not go past head
sangheki
parents: 47678
diff changeset
   113
  //    - G1MMUTracker::add_pause(), when_sec() and its variation(when_ms() etc..) can be called
55714c3d544c 8185278: TestGreyReclaimedHumongousObjects.java fails guarantee(index != trim_index(_head_index + 1)) failed: should not go past head
sangheki
parents: 47678
diff changeset
   114
  //      concurrently from ConcurrentMarkThread and VMThread.
55714c3d544c 8185278: TestGreyReclaimedHumongousObjects.java fails guarantee(index != trim_index(_head_index + 1)) failed: should not go past head
sangheki
parents: 47678
diff changeset
   115
  // 2. If currently a gc is running, but it has not yet updated the MMU,
55714c3d544c 8185278: TestGreyReclaimedHumongousObjects.java fails guarantee(index != trim_index(_head_index + 1)) failed: should not go past head
sangheki
parents: 47678
diff changeset
   116
  //    we will not forget to consider that pause in the MMU calculation.
55714c3d544c 8185278: TestGreyReclaimedHumongousObjects.java fails guarantee(index != trim_index(_head_index + 1)) failed: should not go past head
sangheki
parents: 47678
diff changeset
   117
  // 3. If currently a gc is running, ConcurrentMarkThread will wait it to be finished.
55714c3d544c 8185278: TestGreyReclaimedHumongousObjects.java fails guarantee(index != trim_index(_head_index + 1)) failed: should not go past head
sangheki
parents: 47678
diff changeset
   118
  //    And then sleep for predicted amount of time by delay_to_keep_mmu().
55714c3d544c 8185278: TestGreyReclaimedHumongousObjects.java fails guarantee(index != trim_index(_head_index + 1)) failed: should not go past head
sangheki
parents: 47678
diff changeset
   119
  SuspendibleThreadSetJoiner sts_join;
55714c3d544c 8185278: TestGreyReclaimedHumongousObjects.java fails guarantee(index != trim_index(_head_index + 1)) failed: should not go past head
sangheki
parents: 47678
diff changeset
   120
55714c3d544c 8185278: TestGreyReclaimedHumongousObjects.java fails guarantee(index != trim_index(_head_index + 1)) failed: should not go past head
sangheki
parents: 47678
diff changeset
   121
  const G1Analytics* analytics = g1_policy->analytics();
55714c3d544c 8185278: TestGreyReclaimedHumongousObjects.java fails guarantee(index != trim_index(_head_index + 1)) failed: should not go past head
sangheki
parents: 47678
diff changeset
   122
  double now = os::elapsedTime();
55714c3d544c 8185278: TestGreyReclaimedHumongousObjects.java fails guarantee(index != trim_index(_head_index + 1)) failed: should not go past head
sangheki
parents: 47678
diff changeset
   123
  double prediction_ms = remark ? analytics->predict_remark_time_ms()
55714c3d544c 8185278: TestGreyReclaimedHumongousObjects.java fails guarantee(index != trim_index(_head_index + 1)) failed: should not go past head
sangheki
parents: 47678
diff changeset
   124
                                : analytics->predict_cleanup_time_ms();
55714c3d544c 8185278: TestGreyReclaimedHumongousObjects.java fails guarantee(index != trim_index(_head_index + 1)) failed: should not go past head
sangheki
parents: 47678
diff changeset
   125
  G1MMUTracker *mmu_tracker = g1_policy->mmu_tracker();
55714c3d544c 8185278: TestGreyReclaimedHumongousObjects.java fails guarantee(index != trim_index(_head_index + 1)) failed: should not go past head
sangheki
parents: 47678
diff changeset
   126
  return mmu_tracker->when_ms(now, prediction_ms);
55714c3d544c 8185278: TestGreyReclaimedHumongousObjects.java fails guarantee(index != trim_index(_head_index + 1)) failed: should not go past head
sangheki
parents: 47678
diff changeset
   127
}
55714c3d544c 8185278: TestGreyReclaimedHumongousObjects.java fails guarantee(index != trim_index(_head_index + 1)) failed: should not go past head
sangheki
parents: 47678
diff changeset
   128
49667
2fef34f04314 8200626: Restore history for g1ConcurrentMarkThread.*
ehelin
parents: 49644
diff changeset
   129
void G1ConcurrentMarkThread::delay_to_keep_mmu(G1Policy* g1_policy, bool remark) {
33608
7afc768e4d62 8138920: Refactor the sampling thread from ConcurrentG1RefineThread
drwhite
parents: 33152
diff changeset
   130
  if (g1_policy->adaptive_young_list_length()) {
47756
55714c3d544c 8185278: TestGreyReclaimedHumongousObjects.java fails guarantee(index != trim_index(_head_index + 1)) failed: should not go past head
sangheki
parents: 47678
diff changeset
   131
    jlong sleep_time_ms = mmu_sleep_time(g1_policy, remark);
49644
50a01910e00a 8151171: Bring g1ConcurrentMark files up to current coding conventions
tschatzl
parents: 49632
diff changeset
   132
    if (!_cm->has_aborted() && sleep_time_ms > 0) {
47756
55714c3d544c 8185278: TestGreyReclaimedHumongousObjects.java fails guarantee(index != trim_index(_head_index + 1)) failed: should not go past head
sangheki
parents: 47678
diff changeset
   133
      os::sleep(this, sleep_time_ms, false);
55714c3d544c 8185278: TestGreyReclaimedHumongousObjects.java fails guarantee(index != trim_index(_head_index + 1)) failed: should not go past head
sangheki
parents: 47678
diff changeset
   134
    }
33608
7afc768e4d62 8138920: Refactor the sampling thread from ConcurrentG1RefineThread
drwhite
parents: 33152
diff changeset
   135
  }
7afc768e4d62 8138920: Refactor the sampling thread from ConcurrentG1RefineThread
drwhite
parents: 33152
diff changeset
   136
}
35204
78a0fd90a70f 8068394: Trace event for concurrent GC phases
sangheki
parents: 35061
diff changeset
   137
37123
bf4f064a10a8 8151614: Improve logging in concurrent mark code
tschatzl
parents: 37122
diff changeset
   138
class G1ConcPhaseTimer : public GCTraceConcTimeImpl<LogLevel::Info, LOG_TAGS(gc, marking)> {
35943
e726308008c0 8148973: Rename g1/concurrentMark.{hpp,cpp,inline.hpp} to g1/g1ConcurrentMark.{hpp,cpp,inline.hpp}
ehelin
parents: 35204
diff changeset
   139
  G1ConcurrentMark* _cm;
35204
78a0fd90a70f 8068394: Trace event for concurrent GC phases
sangheki
parents: 35061
diff changeset
   140
78a0fd90a70f 8068394: Trace event for concurrent GC phases
sangheki
parents: 35061
diff changeset
   141
 public:
37123
bf4f064a10a8 8151614: Improve logging in concurrent mark code
tschatzl
parents: 37122
diff changeset
   142
  G1ConcPhaseTimer(G1ConcurrentMark* cm, const char* title) :
46384
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   143
    GCTraceConcTimeImpl<LogLevel::Info,  LogTag::_gc, LogTag::_marking>(title),
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   144
    _cm(cm)
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   145
  {
37137
62fd3fb4b1b1 8151085: Change G1 concurrent timer and tracer measuring time
sangheki
parents: 37134
diff changeset
   146
    _cm->gc_timer_cm()->register_gc_concurrent_start(title);
35204
78a0fd90a70f 8068394: Trace event for concurrent GC phases
sangheki
parents: 35061
diff changeset
   147
  }
78a0fd90a70f 8068394: Trace event for concurrent GC phases
sangheki
parents: 35061
diff changeset
   148
37123
bf4f064a10a8 8151614: Improve logging in concurrent mark code
tschatzl
parents: 37122
diff changeset
   149
  ~G1ConcPhaseTimer() {
37137
62fd3fb4b1b1 8151085: Change G1 concurrent timer and tracer measuring time
sangheki
parents: 37134
diff changeset
   150
    _cm->gc_timer_cm()->register_gc_concurrent_end();
35204
78a0fd90a70f 8068394: Trace event for concurrent GC phases
sangheki
parents: 35061
diff changeset
   151
  }
78a0fd90a70f 8068394: Trace event for concurrent GC phases
sangheki
parents: 35061
diff changeset
   152
};
78a0fd90a70f 8068394: Trace event for concurrent GC phases
sangheki
parents: 35061
diff changeset
   153
46384
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   154
static const char* const concurrent_phase_names[] = {
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   155
#define CONCURRENT_PHASE_NAME(tag, ignore_value, ignore_title) XSTR(tag),
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   156
  EXPAND_CONCURRENT_PHASES(CONCURRENT_PHASE_NAME)
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   157
#undef CONCURRENT_PHASE_NAME
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   158
  NULL                          // terminator
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   159
};
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   160
// Verify dense enum assumption.  +1 for terminator.
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   161
STATIC_ASSERT(G1ConcurrentPhase::PHASE_ID_LIMIT + 1 ==
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   162
              ARRAY_SIZE(concurrent_phase_names));
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   163
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   164
// Returns the phase number for name, or a negative value if unknown.
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   165
static int lookup_concurrent_phase(const char* name) {
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   166
  const char* const* names = concurrent_phase_names;
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   167
  for (uint i = 0; names[i] != NULL; ++i) {
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   168
    if (strcmp(name, names[i]) == 0) {
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   169
      return static_cast<int>(i);
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   170
    }
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   171
  }
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   172
  return -1;
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   173
}
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   174
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   175
// The phase must be valid and must have a title.
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   176
static const char* lookup_concurrent_phase_title(int phase) {
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   177
  static const char* const titles[] = {
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   178
#define CONCURRENT_PHASE_TITLE(ignore_tag, ignore_value, title) title,
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   179
    EXPAND_CONCURRENT_PHASES(CONCURRENT_PHASE_TITLE)
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   180
#undef CONCURRENT_PHASE_TITLE
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   181
  };
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   182
  // Verify dense enum assumption.
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   183
  STATIC_ASSERT(G1ConcurrentPhase::PHASE_ID_LIMIT == ARRAY_SIZE(titles));
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   184
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   185
  assert(0 <= phase, "precondition");
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   186
  assert((uint)phase < ARRAY_SIZE(titles), "precondition");
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   187
  const char* title = titles[phase];
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   188
  assert(title != NULL, "precondition");
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   189
  return title;
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   190
}
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   191
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   192
class G1ConcPhaseManager : public StackObj {
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   193
  G1ConcurrentMark* _cm;
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   194
  ConcurrentGCPhaseManager _manager;
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   195
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   196
public:
49667
2fef34f04314 8200626: Restore history for g1ConcurrentMarkThread.*
ehelin
parents: 49644
diff changeset
   197
  G1ConcPhaseManager(int phase, G1ConcurrentMarkThread* thread) :
46384
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   198
    _cm(thread->cm()),
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   199
    _manager(phase, thread->phase_manager_stack())
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   200
  { }
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   201
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   202
  ~G1ConcPhaseManager() {
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   203
    // Deactivate the manager if marking aborted, to avoid blocking on
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   204
    // phase exit when the phase has been requested.
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   205
    if (_cm->has_aborted()) {
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   206
      _manager.deactivate();
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   207
    }
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   208
  }
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   209
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   210
  void set_phase(int phase, bool force) {
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   211
    _manager.set_phase(phase, force);
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   212
  }
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   213
};
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   214
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   215
// Combine phase management and timing into one convenient utility.
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   216
class G1ConcPhase : public StackObj {
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   217
  G1ConcPhaseTimer _timer;
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   218
  G1ConcPhaseManager _manager;
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   219
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   220
public:
49667
2fef34f04314 8200626: Restore history for g1ConcurrentMarkThread.*
ehelin
parents: 49644
diff changeset
   221
  G1ConcPhase(int phase, G1ConcurrentMarkThread* thread) :
46384
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   222
    _timer(thread->cm(), lookup_concurrent_phase_title(phase)),
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   223
    _manager(phase, thread)
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   224
  { }
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   225
};
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   226
49667
2fef34f04314 8200626: Restore history for g1ConcurrentMarkThread.*
ehelin
parents: 49644
diff changeset
   227
const char* const* G1ConcurrentMarkThread::concurrent_phases() const {
46384
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   228
  return concurrent_phase_names;
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   229
}
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   230
49667
2fef34f04314 8200626: Restore history for g1ConcurrentMarkThread.*
ehelin
parents: 49644
diff changeset
   231
bool G1ConcurrentMarkThread::request_concurrent_phase(const char* phase_name) {
46384
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   232
  int phase = lookup_concurrent_phase(phase_name);
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   233
  if (phase < 0) return false;
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   234
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   235
  while (!ConcurrentGCPhaseManager::wait_for_phase(phase,
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   236
                                                   phase_manager_stack())) {
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   237
    assert(phase != G1ConcurrentPhase::ANY, "Wait for ANY phase must succeed");
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   238
    if ((phase != G1ConcurrentPhase::IDLE) && !during_cycle()) {
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   239
      // If idle and the goal is !idle, start a collection.
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   240
      G1CollectedHeap::heap()->collect(GCCause::_wb_conc_mark);
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   241
    }
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   242
  }
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   243
  return true;
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   244
}
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   245
49667
2fef34f04314 8200626: Restore history for g1ConcurrentMarkThread.*
ehelin
parents: 49644
diff changeset
   246
void G1ConcurrentMarkThread::run_service() {
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   247
  _vtime_start = os::elapsedVTime();
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   248
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents: 7455
diff changeset
   249
  G1CollectedHeap* g1h = G1CollectedHeap::heap();
37985
539c597ee0fa 8154154: Separate G1 specific policy code from the CollectorPolicy class hierarchy
mgerdin
parents: 37414
diff changeset
   250
  G1Policy* g1_policy = g1h->g1_policy();
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   251
46384
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   252
  G1ConcPhaseManager cpmanager(G1ConcurrentPhase::IDLE, this);
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   253
37081
7656f5356a5d 8140257: Add support for "gc service threads" to ConcurrentGCThread
drwhite
parents: 36084
diff changeset
   254
  while (!should_terminate()) {
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   255
    // wait until started is set.
49644
50a01910e00a 8151171: Bring g1ConcurrentMark files up to current coding conventions
tschatzl
parents: 49632
diff changeset
   256
    sleep_before_next_cycle();
37081
7656f5356a5d 8140257: Add support for "gc service threads" to ConcurrentGCThread
drwhite
parents: 36084
diff changeset
   257
    if (should_terminate()) {
24093
095cc0a63ed9 8037112: gc/g1/TestHumongousAllocInitialMark.java caused SIGSEGV
pliden
parents: 18025
diff changeset
   258
      break;
095cc0a63ed9 8037112: gc/g1/TestHumongousAllocInitialMark.java caused SIGSEGV
pliden
parents: 18025
diff changeset
   259
    }
095cc0a63ed9 8037112: gc/g1/TestHumongousAllocInitialMark.java caused SIGSEGV
pliden
parents: 18025
diff changeset
   260
46384
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   261
    cpmanager.set_phase(G1ConcurrentPhase::CONCURRENT_CYCLE, false /* force */);
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   262
37137
62fd3fb4b1b1 8151085: Change G1 concurrent timer and tracer measuring time
sangheki
parents: 37134
diff changeset
   263
    GCIdMark gc_id_mark;
62fd3fb4b1b1 8151085: Change G1 concurrent timer and tracer measuring time
sangheki
parents: 37134
diff changeset
   264
49644
50a01910e00a 8151171: Bring g1ConcurrentMark files up to current coding conventions
tschatzl
parents: 49632
diff changeset
   265
    _cm->concurrent_cycle_start();
37137
62fd3fb4b1b1 8151085: Change G1 concurrent timer and tracer measuring time
sangheki
parents: 37134
diff changeset
   266
37123
bf4f064a10a8 8151614: Improve logging in concurrent mark code
tschatzl
parents: 37122
diff changeset
   267
    GCTraceConcTime(Info, gc) tt("Concurrent Cycle");
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   268
    {
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   269
      ResourceMark rm;
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   270
      HandleMark   hm;
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   271
      double cycle_start = os::elapsedVTime();
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   272
36084
9a3bf78e9a76 8148992: VM can hang on exit if root region scanning is initiated but not executed
brutisso
parents: 35943
diff changeset
   273
      {
46384
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   274
        G1ConcPhase p(G1ConcurrentPhase::CLEAR_CLAIMED_MARKS, this);
36084
9a3bf78e9a76 8148992: VM can hang on exit if root region scanning is initiated but not executed
brutisso
parents: 35943
diff changeset
   275
        ClassLoaderDataGraph::clear_claimed_marks();
9a3bf78e9a76 8148992: VM can hang on exit if root region scanning is initiated but not executed
brutisso
parents: 35943
diff changeset
   276
      }
9a3bf78e9a76 8148992: VM can hang on exit if root region scanning is initiated but not executed
brutisso
parents: 35943
diff changeset
   277
11584
e1df4d08a1f4 7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents: 11583
diff changeset
   278
      // 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
   279
      // 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
   280
      // 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
   281
      // 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
   282
      // 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
   283
      // 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
   284
      // correctness issue.
e1df4d08a1f4 7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents: 11583
diff changeset
   285
36084
9a3bf78e9a76 8148992: VM can hang on exit if root region scanning is initiated but not executed
brutisso
parents: 35943
diff changeset
   286
      {
46384
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   287
        G1ConcPhase p(G1ConcurrentPhase::SCAN_ROOT_REGIONS, this);
37123
bf4f064a10a8 8151614: Improve logging in concurrent mark code
tschatzl
parents: 37122
diff changeset
   288
        _cm->scan_root_regions();
11584
e1df4d08a1f4 7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents: 11583
diff changeset
   289
      }
e1df4d08a1f4 7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents: 11583
diff changeset
   290
46384
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   291
      // It would be nice to use the G1ConcPhase class here but
35061
be6025ebffea 8145092: Use Unified Logging for the GC logging
brutisso
parents: 33608
diff changeset
   292
      // the "end" logging is inside the loop and not at the end of
46384
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   293
      // a scope. Also, the timer doesn't support nesting.
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   294
      // Mimicking the same log output instead.
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   295
      {
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   296
        G1ConcPhaseManager mark_manager(G1ConcurrentPhase::CONCURRENT_MARK, this);
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   297
        jlong mark_start = os::elapsed_counter();
49644
50a01910e00a 8151171: Bring g1ConcurrentMark files up to current coding conventions
tschatzl
parents: 49632
diff changeset
   298
        const char* cm_title = lookup_concurrent_phase_title(G1ConcurrentPhase::CONCURRENT_MARK);
46384
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   299
        log_info(gc, marking)("%s (%.3fs)",
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   300
                              cm_title,
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   301
                              TimeHelper::counter_to_seconds(mark_start));
49644
50a01910e00a 8151171: Bring g1ConcurrentMark files up to current coding conventions
tschatzl
parents: 49632
diff changeset
   302
        for (uint iter = 1; !_cm->has_aborted(); ++iter) {
46384
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   303
          // Concurrent marking.
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   304
          {
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   305
            G1ConcPhase p(G1ConcurrentPhase::MARK_FROM_ROOTS, this);
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   306
            _cm->mark_from_roots();
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   307
          }
49644
50a01910e00a 8151171: Bring g1ConcurrentMark files up to current coding conventions
tschatzl
parents: 49632
diff changeset
   308
          if (_cm->has_aborted()) {
50a01910e00a 8151171: Bring g1ConcurrentMark files up to current coding conventions
tschatzl
parents: 49632
diff changeset
   309
            break;
50a01910e00a 8151171: Bring g1ConcurrentMark files up to current coding conventions
tschatzl
parents: 49632
diff changeset
   310
          }
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   311
46384
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   312
          // Provide a control point after mark_from_roots.
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   313
          {
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   314
            G1ConcPhaseManager p(G1ConcurrentPhase::BEFORE_REMARK, this);
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   315
          }
49644
50a01910e00a 8151171: Bring g1ConcurrentMark files up to current coding conventions
tschatzl
parents: 49632
diff changeset
   316
          if (_cm->has_aborted()) {
50a01910e00a 8151171: Bring g1ConcurrentMark files up to current coding conventions
tschatzl
parents: 49632
diff changeset
   317
            break;
50a01910e00a 8151171: Bring g1ConcurrentMark files up to current coding conventions
tschatzl
parents: 49632
diff changeset
   318
          }
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   319
46384
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   320
          // Delay remark pause for MMU.
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   321
          double mark_end_time = os::elapsedVTime();
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   322
          jlong mark_end = os::elapsed_counter();
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   323
          _vtime_mark_accum += (mark_end_time - cycle_start);
33608
7afc768e4d62 8138920: Refactor the sampling thread from ConcurrentG1RefineThread
drwhite
parents: 33152
diff changeset
   324
          delay_to_keep_mmu(g1_policy, true /* remark */);
49644
50a01910e00a 8151171: Bring g1ConcurrentMark files up to current coding conventions
tschatzl
parents: 49632
diff changeset
   325
          if (_cm->has_aborted()) {
50a01910e00a 8151171: Bring g1ConcurrentMark files up to current coding conventions
tschatzl
parents: 49632
diff changeset
   326
            break;
50a01910e00a 8151171: Bring g1ConcurrentMark files up to current coding conventions
tschatzl
parents: 49632
diff changeset
   327
          }
46384
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   328
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   329
          // Pause Remark.
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   330
          log_info(gc, marking)("%s (%.3fs, %.3fs) %.3fms",
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   331
                                cm_title,
37123
bf4f064a10a8 8151614: Improve logging in concurrent mark code
tschatzl
parents: 37122
diff changeset
   332
                                TimeHelper::counter_to_seconds(mark_start),
bf4f064a10a8 8151614: Improve logging in concurrent mark code
tschatzl
parents: 37122
diff changeset
   333
                                TimeHelper::counter_to_seconds(mark_end),
bf4f064a10a8 8151614: Improve logging in concurrent mark code
tschatzl
parents: 37122
diff changeset
   334
                                TimeHelper::counter_to_millis(mark_end - mark_start));
46384
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   335
          mark_manager.set_phase(G1ConcurrentPhase::REMARK, false);
49644
50a01910e00a 8151171: Bring g1ConcurrentMark files up to current coding conventions
tschatzl
parents: 49632
diff changeset
   336
          CMRemark cl(_cm);
50a01910e00a 8151171: Bring g1ConcurrentMark files up to current coding conventions
tschatzl
parents: 49632
diff changeset
   337
          VM_CGC_Operation op(&cl, "Pause Remark");
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   338
          VMThread::execute(&op);
49644
50a01910e00a 8151171: Bring g1ConcurrentMark files up to current coding conventions
tschatzl
parents: 49632
diff changeset
   339
          if (_cm->has_aborted()) {
46384
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   340
            break;
49644
50a01910e00a 8151171: Bring g1ConcurrentMark files up to current coding conventions
tschatzl
parents: 49632
diff changeset
   341
          } else if (!_cm->restart_for_overflow()) {
46384
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   342
            break;              // Exit loop if no restart requested.
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   343
          } else {
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   344
            // Loop to restart for overflow.
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   345
            mark_manager.set_phase(G1ConcurrentPhase::CONCURRENT_MARK, false);
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   346
            log_info(gc, marking)("%s Restart for Mark Stack Overflow (iteration #%u)",
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   347
                                  cm_title, iter);
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   348
          }
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   349
        }
42570
5a4f3339b5d9 8169423: Infinite loop in G1's ConcurrentMarkThread
kbarrett
parents: 40892
diff changeset
   350
      }
10769
983d377770fd 7099824: G1: we should take the pending list lock before doing the remark pause
johnc
parents: 10745
diff changeset
   351
49644
50a01910e00a 8151171: Bring g1ConcurrentMark files up to current coding conventions
tschatzl
parents: 49632
diff changeset
   352
      if (!_cm->has_aborted()) {
49607
acffe6ff3ae7 8180415: Rebuild remembered sets during the concurrent cycle
tschatzl
parents: 49361
diff changeset
   353
        G1ConcPhase p(G1ConcurrentPhase::REBUILD_REMEMBERED_SETS, this);
49644
50a01910e00a 8151171: Bring g1ConcurrentMark files up to current coding conventions
tschatzl
parents: 49632
diff changeset
   354
        _cm->rebuild_rem_set_concurrently();
37413
2f71679d06dd 8077144: Concurrent mark initialization takes too long
tschatzl
parents: 37137
diff changeset
   355
      }
2f71679d06dd 8077144: Concurrent mark initialization takes too long
tschatzl
parents: 37137
diff changeset
   356
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   357
      double end_time = os::elapsedVTime();
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   358
      // Update the total virtual time before doing this, since it will try
49632
64f9ebc85e67 8197573: Remove concurrent cleanup and secondary free list handling
tschatzl
parents: 49607
diff changeset
   359
      // to measure it to get the vtime for this marking.
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   360
      _vtime_accum = (end_time - _vtime_start);
11583
83a7383de44c 6484965: G1: piggy-back liveness accounting phase on marking
johnc
parents: 11175
diff changeset
   361
49644
50a01910e00a 8151171: Bring g1ConcurrentMark files up to current coding conventions
tschatzl
parents: 49632
diff changeset
   362
      if (!_cm->has_aborted()) {
33608
7afc768e4d62 8138920: Refactor the sampling thread from ConcurrentG1RefineThread
drwhite
parents: 33152
diff changeset
   363
        delay_to_keep_mmu(g1_policy, false /* cleanup */);
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   364
      }
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   365
49644
50a01910e00a 8151171: Bring g1ConcurrentMark files up to current coding conventions
tschatzl
parents: 49632
diff changeset
   366
      if (!_cm->has_aborted()) {
50a01910e00a 8151171: Bring g1ConcurrentMark files up to current coding conventions
tschatzl
parents: 49632
diff changeset
   367
        CMCleanup cl_cl(_cm);
49632
64f9ebc85e67 8197573: Remove concurrent cleanup and secondary free list handling
tschatzl
parents: 49607
diff changeset
   368
        VM_CGC_Operation op(&cl_cl, "Pause Cleanup");
64f9ebc85e67 8197573: Remove concurrent cleanup and secondary free list handling
tschatzl
parents: 49607
diff changeset
   369
        VMThread::execute(&op);
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   370
      }
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   371
10745
6b33ec421509 7098085: G1: partially-young GCs not initiated under certain circumstances
tonyp
parents: 10280
diff changeset
   372
      // 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
   373
      // 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
   374
      // 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
   375
      // bitmap then, as it has been done during mark abort.
49644
50a01910e00a 8151171: Bring g1ConcurrentMark files up to current coding conventions
tschatzl
parents: 49632
diff changeset
   376
      if (!_cm->has_aborted()) {
46384
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   377
        G1ConcPhase p(G1ConcurrentPhase::CLEANUP_FOR_NEXT_MARK, this);
37122
5318124fe3a8 8151126: Clean up duplicate code for clearing the mark bitmaps
tschatzl
parents: 37081
diff changeset
   378
        _cm->cleanup_for_next_mark();
25728
7f8a76e3baa8 8048085: Aborting marking just before remark results in useless additional clearing of the next mark bitmap
tschatzl
parents: 25350
diff changeset
   379
      } else {
47678
c84eeb55c55e 8184667: Clean up G1ConcurrentMark files
tschatzl
parents: 47647
diff changeset
   380
        assert(!G1VerifyBitmaps || _cm->next_mark_bitmap_is_clear(), "Next mark bitmap must be clear");
24094
5dbf1f44de18 8039147: Cleanup SuspendibleThreadSet
pliden
parents: 24093
diff changeset
   381
      }
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   382
    }
6058
9c9aec6ab47d 6944166: G1: explicit GCs are not always handled correctly
tonyp
parents: 5547
diff changeset
   383
9c9aec6ab47d 6944166: G1: explicit GCs are not always handled correctly
tonyp
parents: 5547
diff changeset
   384
    // Update the number of full collections that have been
9c9aec6ab47d 6944166: G1: explicit GCs are not always handled correctly
tonyp
parents: 5547
diff changeset
   385
    // 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
   386
    // 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
   387
    // called System.gc() with +ExplicitGCInvokesConcurrent).
24094
5dbf1f44de18 8039147: Cleanup SuspendibleThreadSet
pliden
parents: 24093
diff changeset
   388
    {
30613
a7815bb05ae2 8079579: Add SuspendibleThreadSetLeaver and make SuspendibleThreadSet::joint()/leave() private
pliden
parents: 29464
diff changeset
   389
      SuspendibleThreadSetJoiner sts_join;
24094
5dbf1f44de18 8039147: Cleanup SuspendibleThreadSet
pliden
parents: 24093
diff changeset
   390
      g1h->increment_old_marking_cycles_completed(true /* concurrent */);
37137
62fd3fb4b1b1 8151085: Change G1 concurrent timer and tracer measuring time
sangheki
parents: 37134
diff changeset
   391
49644
50a01910e00a 8151171: Bring g1ConcurrentMark files up to current coding conventions
tschatzl
parents: 49632
diff changeset
   392
      _cm->concurrent_cycle_end();
24094
5dbf1f44de18 8039147: Cleanup SuspendibleThreadSet
pliden
parents: 24093
diff changeset
   393
    }
46384
dacebddcdea0 8169517: WhiteBox should provide concurrent GC phase control
kbarrett
parents: 42570
diff changeset
   394
49644
50a01910e00a 8151171: Bring g1ConcurrentMark files up to current coding conventions
tschatzl
parents: 49632
diff changeset
   395
    cpmanager.set_phase(G1ConcurrentPhase::IDLE, _cm->has_aborted() /* force */);
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   396
  }
37120
7f2e805b6b77 8151336: Root region scanning should be cancelled and disabled when the ConcurrentMarkThread::run_service() exits
brutisso
parents: 37081
diff changeset
   397
  _cm->root_regions()->cancel_scan();
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   398
}
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   399
49667
2fef34f04314 8200626: Restore history for g1ConcurrentMarkThread.*
ehelin
parents: 49644
diff changeset
   400
void G1ConcurrentMarkThread::stop_service() {
33608
7afc768e4d62 8138920: Refactor the sampling thread from ConcurrentG1RefineThread
drwhite
parents: 33152
diff changeset
   401
  MutexLockerEx ml(CGC_lock, Mutex::_no_safepoint_check_flag);
7afc768e4d62 8138920: Refactor the sampling thread from ConcurrentG1RefineThread
drwhite
parents: 33152
diff changeset
   402
  CGC_lock->notify_all();
7afc768e4d62 8138920: Refactor the sampling thread from ConcurrentG1RefineThread
drwhite
parents: 33152
diff changeset
   403
}
7afc768e4d62 8138920: Refactor the sampling thread from ConcurrentG1RefineThread
drwhite
parents: 33152
diff changeset
   404
49667
2fef34f04314 8200626: Restore history for g1ConcurrentMarkThread.*
ehelin
parents: 49644
diff changeset
   405
2fef34f04314 8200626: Restore history for g1ConcurrentMarkThread.*
ehelin
parents: 49644
diff changeset
   406
void G1ConcurrentMarkThread::sleep_before_next_cycle() {
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   407
  // 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
   408
  // below while the world is otherwise stopped.
6766
839211600ad0 6983311: G1: LoopTest hangs when run with -XX:+ExplicitInvokesConcurrent
johnc
parents: 6058
diff changeset
   409
  assert(!in_progress(), "should have been cleared");
839211600ad0 6983311: G1: LoopTest hangs when run with -XX:+ExplicitInvokesConcurrent
johnc
parents: 6058
diff changeset
   410
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   411
  MutexLockerEx x(CGC_lock, Mutex::_no_safepoint_check_flag);
37081
7656f5356a5d 8140257: Add support for "gc service threads" to ConcurrentGCThread
drwhite
parents: 36084
diff changeset
   412
  while (!started() && !should_terminate()) {
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   413
    CGC_lock->wait(Mutex::_no_safepoint_check_flag);
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   414
  }
24093
095cc0a63ed9 8037112: gc/g1/TestHumongousAllocInitialMark.java caused SIGSEGV
pliden
parents: 18025
diff changeset
   415
095cc0a63ed9 8037112: gc/g1/TestHumongousAllocInitialMark.java caused SIGSEGV
pliden
parents: 18025
diff changeset
   416
  if (started()) {
095cc0a63ed9 8037112: gc/g1/TestHumongousAllocInitialMark.java caused SIGSEGV
pliden
parents: 18025
diff changeset
   417
    set_in_progress();
095cc0a63ed9 8037112: gc/g1/TestHumongousAllocInitialMark.java caused SIGSEGV
pliden
parents: 18025
diff changeset
   418
  }
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   419
}