author | sjohanss |
Fri, 07 Dec 2018 13:54:45 +0100 | |
changeset 52897 | 495c05ee2a9a |
parent 51475 | 492b366f8e57 |
child 53703 | 24341625d8f2 |
permissions | -rw-r--r-- |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
1 |
/* |
49333
489f1dd40582
8196876: OopStorage::assert_at_safepoint clashes with assert_at_safepoint macros in g1CollectedHeap.hpp
kbarrett
parents:
48969
diff
changeset
|
2 |
* Copyright (c) 2016, 2018, Oracle and/or its affiliates. All rights reserved. |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
3 |
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
4 |
* |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
5 |
* This code is free software; you can redistribute it and/or modify it |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
6 |
* under the terms of the GNU General Public License version 2 only, as |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
7 |
* published by the Free Software Foundation. |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
8 |
* |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
9 |
* This code is distributed in the hope that it will be useful, but WITHOUT |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
10 |
* ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
11 |
* FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
12 |
* version 2 for more details (a copy is included in the LICENSE file that |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
13 |
* accompanied this code). |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
14 |
* |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
15 |
* You should have received a copy of the GNU General Public License version |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
16 |
* 2 along with this work; if not, write to the Free Software Foundation, |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
17 |
* Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA. |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
18 |
* |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
19 |
* Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
20 |
* or visit www.oracle.com if you need additional information or have any |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
21 |
* questions. |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
22 |
* |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
23 |
*/ |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
24 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
25 |
#include "precompiled.hpp" |
52897
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
26 |
#include "gc/g1/g1CollectedHeap.inline.hpp" |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
27 |
#include "gc/g1/g1CollectionSet.hpp" |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
28 |
#include "gc/g1/g1CollectorState.hpp" |
52897
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
29 |
#include "gc/g1/g1ParScanThreadState.hpp" |
37985
539c597ee0fa
8154154: Separate G1 specific policy code from the CollectorPolicy class hierarchy
mgerdin
parents:
37262
diff
changeset
|
30 |
#include "gc/g1/g1Policy.hpp" |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
31 |
#include "gc/g1/heapRegion.inline.hpp" |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
32 |
#include "gc/g1/heapRegionRemSet.hpp" |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
33 |
#include "gc/g1/heapRegionSet.hpp" |
39698
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
34 |
#include "logging/logStream.hpp" |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
35 |
#include "utilities/debug.hpp" |
52897
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
36 |
#include "utilities/globalDefinitions.hpp" |
41075
d60e54a9bb67
8165313: Inserting freed regions during Free Collection Set serial phase takes very long on huge heaps
tschatzl
parents:
39698
diff
changeset
|
37 |
#include "utilities/quickSort.hpp" |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
38 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
39 |
G1CollectorState* G1CollectionSet::collector_state() { |
49806 | 40 |
return _g1h->collector_state(); |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
41 |
} |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
42 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
43 |
G1GCPhaseTimes* G1CollectionSet::phase_times() { |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
44 |
return _policy->phase_times(); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
45 |
} |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
46 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
47 |
CollectionSetChooser* G1CollectionSet::cset_chooser() { |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
48 |
return _cset_chooser; |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
49 |
} |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
50 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
51 |
double G1CollectionSet::predict_region_elapsed_time_ms(HeapRegion* hr) { |
49643
a3453bbd5418
8199742: Clean up state flags in G1CollectorState
tschatzl
parents:
49333
diff
changeset
|
52 |
return _policy->predict_region_elapsed_time_ms(hr, collector_state()->in_young_only_phase()); |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
53 |
} |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
54 |
|
37985
539c597ee0fa
8154154: Separate G1 specific policy code from the CollectorPolicy class hierarchy
mgerdin
parents:
37262
diff
changeset
|
55 |
G1CollectionSet::G1CollectionSet(G1CollectedHeap* g1h, G1Policy* policy) : |
49806 | 56 |
_g1h(g1h), |
37985
539c597ee0fa
8154154: Separate G1 specific policy code from the CollectorPolicy class hierarchy
mgerdin
parents:
37262
diff
changeset
|
57 |
_policy(policy), |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
58 |
_cset_chooser(new CollectionSetChooser()), |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
59 |
_eden_region_length(0), |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
60 |
_survivor_region_length(0), |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
61 |
_old_region_length(0), |
39698
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
62 |
_collection_set_regions(NULL), |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
63 |
_collection_set_cur_length(0), |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
64 |
_collection_set_max_length(0), |
52897
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
65 |
_optional_regions(NULL), |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
66 |
_optional_region_length(0), |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
67 |
_optional_region_max_length(0), |
51332 | 68 |
_bytes_used_before(0), |
69 |
_recorded_rs_lengths(0), |
|
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
70 |
_inc_build_state(Inactive), |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
71 |
_inc_bytes_used_before(0), |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
72 |
_inc_recorded_rs_lengths(0), |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
73 |
_inc_recorded_rs_lengths_diffs(0), |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
74 |
_inc_predicted_elapsed_time_ms(0.0), |
39698
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
75 |
_inc_predicted_elapsed_time_ms_diffs(0.0) { |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
76 |
} |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
77 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
78 |
G1CollectionSet::~G1CollectionSet() { |
39698
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
79 |
if (_collection_set_regions != NULL) { |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
80 |
FREE_C_HEAP_ARRAY(uint, _collection_set_regions); |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
81 |
} |
52897
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
82 |
free_optional_regions(); |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
83 |
delete _cset_chooser; |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
84 |
} |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
85 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
86 |
void G1CollectionSet::init_region_lengths(uint eden_cset_region_length, |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
87 |
uint survivor_cset_region_length) { |
49333
489f1dd40582
8196876: OopStorage::assert_at_safepoint clashes with assert_at_safepoint macros in g1CollectedHeap.hpp
kbarrett
parents:
48969
diff
changeset
|
88 |
assert_at_safepoint_on_vm_thread(); |
39698
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
89 |
|
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
90 |
_eden_region_length = eden_cset_region_length; |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
91 |
_survivor_region_length = survivor_cset_region_length; |
38109
6503703df058
8155209: Move setting of young index in cset to G1CollectionSet
mgerdin
parents:
37985
diff
changeset
|
92 |
|
39698
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
93 |
assert((size_t) young_region_length() == _collection_set_cur_length, |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
94 |
"Young region length %u should match collection set length " SIZE_FORMAT, young_region_length(), _collection_set_cur_length); |
38109
6503703df058
8155209: Move setting of young index in cset to G1CollectionSet
mgerdin
parents:
37985
diff
changeset
|
95 |
|
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
96 |
_old_region_length = 0; |
52897
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
97 |
_optional_region_length = 0; |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
98 |
} |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
99 |
|
39698
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
100 |
void G1CollectionSet::initialize(uint max_region_length) { |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
101 |
guarantee(_collection_set_regions == NULL, "Must only initialize once."); |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
102 |
_collection_set_max_length = max_region_length; |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
103 |
_collection_set_regions = NEW_C_HEAP_ARRAY(uint, max_region_length, mtGC); |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
104 |
} |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
105 |
|
52897
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
106 |
void G1CollectionSet::initialize_optional(uint max_length) { |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
107 |
assert(_optional_regions == NULL, "Already initialized"); |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
108 |
assert(_optional_region_length == 0, "Already initialized"); |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
109 |
assert(_optional_region_max_length == 0, "Already initialized"); |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
110 |
_optional_region_max_length = max_length; |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
111 |
_optional_regions = NEW_C_HEAP_ARRAY(HeapRegion*, _optional_region_max_length, mtGC); |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
112 |
} |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
113 |
|
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
114 |
void G1CollectionSet::free_optional_regions() { |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
115 |
_optional_region_length = 0; |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
116 |
_optional_region_max_length = 0; |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
117 |
if (_optional_regions != NULL) { |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
118 |
FREE_C_HEAP_ARRAY(HeapRegion*, _optional_regions); |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
119 |
_optional_regions = NULL; |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
120 |
} |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
121 |
} |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
122 |
|
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
123 |
void G1CollectionSet::set_recorded_rs_lengths(size_t rs_lengths) { |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
124 |
_recorded_rs_lengths = rs_lengths; |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
125 |
} |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
126 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
127 |
// Add the heap region at the head of the non-incremental collection set |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
128 |
void G1CollectionSet::add_old_region(HeapRegion* hr) { |
49333
489f1dd40582
8196876: OopStorage::assert_at_safepoint clashes with assert_at_safepoint macros in g1CollectedHeap.hpp
kbarrett
parents:
48969
diff
changeset
|
129 |
assert_at_safepoint_on_vm_thread(); |
39698
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
130 |
|
52897
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
131 |
assert(_inc_build_state == Active || hr->index_in_opt_cset() != G1OptionalCSet::InvalidCSetIndex, |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
132 |
"Precondition, actively building cset or adding optional later on"); |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
133 |
assert(hr->is_old(), "the region should be old"); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
134 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
135 |
assert(!hr->in_collection_set(), "should not already be in the CSet"); |
49806 | 136 |
_g1h->register_old_region_with_cset(hr); |
39698
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
137 |
|
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
138 |
_collection_set_regions[_collection_set_cur_length++] = hr->hrm_index(); |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
139 |
assert(_collection_set_cur_length <= _collection_set_max_length, "Collection set now larger than maximum size."); |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
140 |
|
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
141 |
_bytes_used_before += hr->used(); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
142 |
size_t rs_length = hr->rem_set()->occupied(); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
143 |
_recorded_rs_lengths += rs_length; |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
144 |
_old_region_length += 1; |
52897
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
145 |
|
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
146 |
log_trace(gc, cset)("Added old region %d to collection set", hr->hrm_index()); |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
147 |
} |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
148 |
|
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
149 |
void G1CollectionSet::add_optional_region(HeapRegion* hr) { |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
150 |
assert(!optional_is_full(), "Precondition, must have room left for this region"); |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
151 |
assert(hr->is_old(), "the region should be old"); |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
152 |
assert(!hr->in_collection_set(), "should not already be in the CSet"); |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
153 |
|
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
154 |
_g1h->register_optional_region_with_cset(hr); |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
155 |
|
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
156 |
_optional_regions[_optional_region_length] = hr; |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
157 |
uint index = _optional_region_length++; |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
158 |
hr->set_index_in_opt_cset(index); |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
159 |
|
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
160 |
log_trace(gc, cset)("Added region %d to optional collection set (%u)", hr->hrm_index(), _optional_region_length); |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
161 |
} |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
162 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
163 |
// Initialize the per-collection-set information |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
164 |
void G1CollectionSet::start_incremental_building() { |
39698
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
165 |
assert(_collection_set_cur_length == 0, "Collection set must be empty before starting a new collection set."); |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
166 |
assert(_inc_build_state == Inactive, "Precondition"); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
167 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
168 |
_inc_bytes_used_before = 0; |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
169 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
170 |
_inc_recorded_rs_lengths = 0; |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
171 |
_inc_recorded_rs_lengths_diffs = 0; |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
172 |
_inc_predicted_elapsed_time_ms = 0.0; |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
173 |
_inc_predicted_elapsed_time_ms_diffs = 0.0; |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
174 |
_inc_build_state = Active; |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
175 |
} |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
176 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
177 |
void G1CollectionSet::finalize_incremental_building() { |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
178 |
assert(_inc_build_state == Active, "Precondition"); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
179 |
assert(SafepointSynchronize::is_at_safepoint(), "should be at a safepoint"); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
180 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
181 |
// The two "main" fields, _inc_recorded_rs_lengths and |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
182 |
// _inc_predicted_elapsed_time_ms, are updated by the thread |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
183 |
// that adds a new region to the CSet. Further updates by the |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
184 |
// concurrent refinement thread that samples the young RSet lengths |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
185 |
// are accumulated in the *_diffs fields. Here we add the diffs to |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
186 |
// the "main" fields. |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
187 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
188 |
if (_inc_recorded_rs_lengths_diffs >= 0) { |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
189 |
_inc_recorded_rs_lengths += _inc_recorded_rs_lengths_diffs; |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
190 |
} else { |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
191 |
// This is defensive. The diff should in theory be always positive |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
192 |
// as RSets can only grow between GCs. However, given that we |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
193 |
// sample their size concurrently with other threads updating them |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
194 |
// it's possible that we might get the wrong size back, which |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
195 |
// could make the calculations somewhat inaccurate. |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
196 |
size_t diffs = (size_t) (-_inc_recorded_rs_lengths_diffs); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
197 |
if (_inc_recorded_rs_lengths >= diffs) { |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
198 |
_inc_recorded_rs_lengths -= diffs; |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
199 |
} else { |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
200 |
_inc_recorded_rs_lengths = 0; |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
201 |
} |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
202 |
} |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
203 |
_inc_predicted_elapsed_time_ms += _inc_predicted_elapsed_time_ms_diffs; |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
204 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
205 |
_inc_recorded_rs_lengths_diffs = 0; |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
206 |
_inc_predicted_elapsed_time_ms_diffs = 0.0; |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
207 |
} |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
208 |
|
39698
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
209 |
void G1CollectionSet::clear() { |
49333
489f1dd40582
8196876: OopStorage::assert_at_safepoint clashes with assert_at_safepoint macros in g1CollectedHeap.hpp
kbarrett
parents:
48969
diff
changeset
|
210 |
assert_at_safepoint_on_vm_thread(); |
39698
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
211 |
_collection_set_cur_length = 0; |
52897
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
212 |
_optional_region_length = 0; |
39698
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
213 |
} |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
214 |
|
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
215 |
void G1CollectionSet::iterate(HeapRegionClosure* cl) const { |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
216 |
iterate_from(cl, 0, 1); |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
217 |
} |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
218 |
|
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
219 |
void G1CollectionSet::iterate_from(HeapRegionClosure* cl, uint worker_id, uint total_workers) const { |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
220 |
size_t len = _collection_set_cur_length; |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
221 |
OrderAccess::loadload(); |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
222 |
if (len == 0) { |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
223 |
return; |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
224 |
} |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
225 |
size_t start_pos = (worker_id * len) / total_workers; |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
226 |
size_t cur_pos = start_pos; |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
227 |
|
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
228 |
do { |
49806 | 229 |
HeapRegion* r = _g1h->region_at(_collection_set_regions[cur_pos]); |
48969 | 230 |
bool result = cl->do_heap_region(r); |
39698
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
231 |
if (result) { |
48969 | 232 |
cl->set_incomplete(); |
39698
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
233 |
return; |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
234 |
} |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
235 |
cur_pos++; |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
236 |
if (cur_pos == len) { |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
237 |
cur_pos = 0; |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
238 |
} |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
239 |
} while (cur_pos != start_pos); |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
240 |
} |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
241 |
|
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
242 |
void G1CollectionSet::update_young_region_prediction(HeapRegion* hr, |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
243 |
size_t new_rs_length) { |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
244 |
// Update the CSet information that is dependent on the new RS length |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
245 |
assert(hr->is_young(), "Precondition"); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
246 |
assert(!SafepointSynchronize::is_at_safepoint(), "should not be at a safepoint"); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
247 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
248 |
// We could have updated _inc_recorded_rs_lengths and |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
249 |
// _inc_predicted_elapsed_time_ms directly but we'd need to do |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
250 |
// that atomically, as this code is executed by a concurrent |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
251 |
// refinement thread, potentially concurrently with a mutator thread |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
252 |
// allocating a new region and also updating the same fields. To |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
253 |
// avoid the atomic operations we accumulate these updates on two |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
254 |
// separate fields (*_diffs) and we'll just add them to the "main" |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
255 |
// fields at the start of a GC. |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
256 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
257 |
ssize_t old_rs_length = (ssize_t) hr->recorded_rs_length(); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
258 |
ssize_t rs_lengths_diff = (ssize_t) new_rs_length - old_rs_length; |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
259 |
_inc_recorded_rs_lengths_diffs += rs_lengths_diff; |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
260 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
261 |
double old_elapsed_time_ms = hr->predicted_elapsed_time_ms(); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
262 |
double new_region_elapsed_time_ms = predict_region_elapsed_time_ms(hr); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
263 |
double elapsed_ms_diff = new_region_elapsed_time_ms - old_elapsed_time_ms; |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
264 |
_inc_predicted_elapsed_time_ms_diffs += elapsed_ms_diff; |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
265 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
266 |
hr->set_recorded_rs_length(new_rs_length); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
267 |
hr->set_predicted_elapsed_time_ms(new_region_elapsed_time_ms); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
268 |
} |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
269 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
270 |
void G1CollectionSet::add_young_region_common(HeapRegion* hr) { |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
271 |
assert(hr->is_young(), "invariant"); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
272 |
assert(_inc_build_state == Active, "Precondition"); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
273 |
|
39698
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
274 |
size_t collection_set_length = _collection_set_cur_length; |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
275 |
assert(collection_set_length <= INT_MAX, "Collection set is too large with %d entries", (int)collection_set_length); |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
276 |
hr->set_young_index_in_cset((int)collection_set_length); |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
277 |
|
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
278 |
_collection_set_regions[collection_set_length] = hr->hrm_index(); |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
279 |
// Concurrent readers must observe the store of the value in the array before an |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
280 |
// update to the length field. |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
281 |
OrderAccess::storestore(); |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
282 |
_collection_set_cur_length++; |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
283 |
assert(_collection_set_cur_length <= _collection_set_max_length, "Collection set larger than maximum allowed."); |
38109
6503703df058
8155209: Move setting of young index in cset to G1CollectionSet
mgerdin
parents:
37985
diff
changeset
|
284 |
|
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
285 |
// This routine is used when: |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
286 |
// * adding survivor regions to the incremental cset at the end of an |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
287 |
// evacuation pause or |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
288 |
// * adding the current allocation region to the incremental cset |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
289 |
// when it is retired. |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
290 |
// Therefore this routine may be called at a safepoint by the |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
291 |
// VM thread, or in-between safepoints by mutator threads (when |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
292 |
// retiring the current allocation region) |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
293 |
// We need to clear and set the cached recorded/cached collection set |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
294 |
// information in the heap region here (before the region gets added |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
295 |
// to the collection set). An individual heap region's cached values |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
296 |
// are calculated, aggregated with the policy collection set info, |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
297 |
// and cached in the heap region here (initially) and (subsequently) |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
298 |
// by the Young List sampling code. |
49643
a3453bbd5418
8199742: Clean up state flags in G1CollectorState
tschatzl
parents:
49333
diff
changeset
|
299 |
// Ignore calls to this due to retirement during full gc. |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
300 |
|
49806 | 301 |
if (!_g1h->collector_state()->in_full_gc()) { |
49643
a3453bbd5418
8199742: Clean up state flags in G1CollectorState
tschatzl
parents:
49333
diff
changeset
|
302 |
size_t rs_length = hr->rem_set()->occupied(); |
a3453bbd5418
8199742: Clean up state flags in G1CollectorState
tschatzl
parents:
49333
diff
changeset
|
303 |
double region_elapsed_time_ms = predict_region_elapsed_time_ms(hr); |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
304 |
|
49643
a3453bbd5418
8199742: Clean up state flags in G1CollectorState
tschatzl
parents:
49333
diff
changeset
|
305 |
// Cache the values we have added to the aggregated information |
a3453bbd5418
8199742: Clean up state flags in G1CollectorState
tschatzl
parents:
49333
diff
changeset
|
306 |
// in the heap region in case we have to remove this region from |
a3453bbd5418
8199742: Clean up state flags in G1CollectorState
tschatzl
parents:
49333
diff
changeset
|
307 |
// the incremental collection set, or it is updated by the |
a3453bbd5418
8199742: Clean up state flags in G1CollectorState
tschatzl
parents:
49333
diff
changeset
|
308 |
// rset sampling code |
a3453bbd5418
8199742: Clean up state flags in G1CollectorState
tschatzl
parents:
49333
diff
changeset
|
309 |
hr->set_recorded_rs_length(rs_length); |
a3453bbd5418
8199742: Clean up state flags in G1CollectorState
tschatzl
parents:
49333
diff
changeset
|
310 |
hr->set_predicted_elapsed_time_ms(region_elapsed_time_ms); |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
311 |
|
49643
a3453bbd5418
8199742: Clean up state flags in G1CollectorState
tschatzl
parents:
49333
diff
changeset
|
312 |
_inc_recorded_rs_lengths += rs_length; |
a3453bbd5418
8199742: Clean up state flags in G1CollectorState
tschatzl
parents:
49333
diff
changeset
|
313 |
_inc_predicted_elapsed_time_ms += region_elapsed_time_ms; |
a3453bbd5418
8199742: Clean up state flags in G1CollectorState
tschatzl
parents:
49333
diff
changeset
|
314 |
_inc_bytes_used_before += hr->used(); |
a3453bbd5418
8199742: Clean up state flags in G1CollectorState
tschatzl
parents:
49333
diff
changeset
|
315 |
} |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
316 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
317 |
assert(!hr->in_collection_set(), "invariant"); |
49806 | 318 |
_g1h->register_young_region_with_cset(hr); |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
319 |
} |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
320 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
321 |
void G1CollectionSet::add_survivor_regions(HeapRegion* hr) { |
39698
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
322 |
assert(hr->is_survivor(), "Must only add survivor regions, but is %s", hr->get_type_str()); |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
323 |
add_young_region_common(hr); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
324 |
} |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
325 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
326 |
void G1CollectionSet::add_eden_region(HeapRegion* hr) { |
39698
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
327 |
assert(hr->is_eden(), "Must only add eden regions, but is %s", hr->get_type_str()); |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
328 |
add_young_region_common(hr); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
329 |
} |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
330 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
331 |
#ifndef PRODUCT |
39698
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
332 |
class G1VerifyYoungAgesClosure : public HeapRegionClosure { |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
333 |
public: |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
334 |
bool _valid; |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
335 |
public: |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
336 |
G1VerifyYoungAgesClosure() : HeapRegionClosure(), _valid(true) { } |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
337 |
|
48969 | 338 |
virtual bool do_heap_region(HeapRegion* r) { |
39698
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
339 |
guarantee(r->is_young(), "Region must be young but is %s", r->get_type_str()); |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
340 |
|
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
341 |
SurvRateGroup* group = r->surv_rate_group(); |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
342 |
|
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
343 |
if (group == NULL) { |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
344 |
log_error(gc, verify)("## encountered NULL surv_rate_group in young region"); |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
345 |
_valid = false; |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
346 |
} |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
347 |
|
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
348 |
if (r->age_in_surv_rate_group() < 0) { |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
349 |
log_error(gc, verify)("## encountered negative age in young region"); |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
350 |
_valid = false; |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
351 |
} |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
352 |
|
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
353 |
return false; |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
354 |
} |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
355 |
|
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
356 |
bool valid() const { return _valid; } |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
357 |
}; |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
358 |
|
39698
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
359 |
bool G1CollectionSet::verify_young_ages() { |
49333
489f1dd40582
8196876: OopStorage::assert_at_safepoint clashes with assert_at_safepoint macros in g1CollectedHeap.hpp
kbarrett
parents:
48969
diff
changeset
|
360 |
assert_at_safepoint_on_vm_thread(); |
39698
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
361 |
|
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
362 |
G1VerifyYoungAgesClosure cl; |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
363 |
iterate(&cl); |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
364 |
|
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
365 |
if (!cl.valid()) { |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
366 |
LogStreamHandle(Error, gc, verify) log; |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
367 |
print(&log); |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
368 |
} |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
369 |
|
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
370 |
return cl.valid(); |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
371 |
} |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
372 |
|
51475
492b366f8e57
8209758: 2 classes with same name G1PrintCollectionSetClosure cause crash when logging is enabled
lmesnik
parents:
51332
diff
changeset
|
373 |
class G1PrintCollectionSetDetailClosure : public HeapRegionClosure { |
39698
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
374 |
outputStream* _st; |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
375 |
public: |
51475
492b366f8e57
8209758: 2 classes with same name G1PrintCollectionSetClosure cause crash when logging is enabled
lmesnik
parents:
51332
diff
changeset
|
376 |
G1PrintCollectionSetDetailClosure(outputStream* st) : HeapRegionClosure(), _st(st) { } |
39698
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
377 |
|
48969 | 378 |
virtual bool do_heap_region(HeapRegion* r) { |
39698
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
379 |
assert(r->in_collection_set(), "Region %u should be in collection set", r->hrm_index()); |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
380 |
_st->print_cr(" " HR_FORMAT ", P: " PTR_FORMAT "N: " PTR_FORMAT ", age: %4d", |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
381 |
HR_FORMAT_PARAMS(r), |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
382 |
p2i(r->prev_top_at_mark_start()), |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
383 |
p2i(r->next_top_at_mark_start()), |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
384 |
r->age_in_surv_rate_group_cond()); |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
385 |
return false; |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
386 |
} |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
387 |
}; |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
388 |
|
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
389 |
void G1CollectionSet::print(outputStream* st) { |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
390 |
st->print_cr("\nCollection_set:"); |
39698
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
391 |
|
51475
492b366f8e57
8209758: 2 classes with same name G1PrintCollectionSetClosure cause crash when logging is enabled
lmesnik
parents:
51332
diff
changeset
|
392 |
G1PrintCollectionSetDetailClosure cl(st); |
39698
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
393 |
iterate(&cl); |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
394 |
} |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
395 |
#endif // !PRODUCT |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
396 |
|
38183
cb68e4923223
8150721: Don't explicitly manage G1 young regions in YoungList
mgerdin
parents:
38162
diff
changeset
|
397 |
double G1CollectionSet::finalize_young_part(double target_pause_time_ms, G1SurvivorRegions* survivors) { |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
398 |
double young_start_time_sec = os::elapsedTime(); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
399 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
400 |
finalize_incremental_building(); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
401 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
402 |
guarantee(target_pause_time_ms > 0.0, |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
403 |
"target_pause_time_ms = %1.6lf should be positive", target_pause_time_ms); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
404 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
405 |
size_t pending_cards = _policy->pending_cards(); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
406 |
double base_time_ms = _policy->predict_base_elapsed_time_ms(pending_cards); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
407 |
double time_remaining_ms = MAX2(target_pause_time_ms - base_time_ms, 0.0); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
408 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
409 |
log_trace(gc, ergo, cset)("Start choosing CSet. pending cards: " SIZE_FORMAT " predicted base time: %1.2fms remaining time: %1.2fms target pause time: %1.2fms", |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
410 |
pending_cards, base_time_ms, time_remaining_ms, target_pause_time_ms); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
411 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
412 |
// The young list is laid with the survivor regions from the previous |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
413 |
// pause are appended to the RHS of the young list, i.e. |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
414 |
// [Newly Young Regions ++ Survivors from last pause]. |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
415 |
|
38183
cb68e4923223
8150721: Don't explicitly manage G1 young regions in YoungList
mgerdin
parents:
38162
diff
changeset
|
416 |
uint survivor_region_length = survivors->length(); |
49806 | 417 |
uint eden_region_length = _g1h->eden_regions_count(); |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
418 |
init_region_lengths(eden_region_length, survivor_region_length); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
419 |
|
38109
6503703df058
8155209: Move setting of young index in cset to G1CollectionSet
mgerdin
parents:
37985
diff
changeset
|
420 |
verify_young_cset_indices(); |
6503703df058
8155209: Move setting of young index in cset to G1CollectionSet
mgerdin
parents:
37985
diff
changeset
|
421 |
|
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
422 |
// Clear the fields that point to the survivor list - they are all young now. |
38183
cb68e4923223
8150721: Don't explicitly manage G1 young regions in YoungList
mgerdin
parents:
38162
diff
changeset
|
423 |
survivors->convert_to_eden(); |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
424 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
425 |
_bytes_used_before = _inc_bytes_used_before; |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
426 |
time_remaining_ms = MAX2(time_remaining_ms - _inc_predicted_elapsed_time_ms, 0.0); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
427 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
428 |
log_trace(gc, ergo, cset)("Add young regions to CSet. eden: %u regions, survivors: %u regions, predicted young region time: %1.2fms, target pause time: %1.2fms", |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
429 |
eden_region_length, survivor_region_length, _inc_predicted_elapsed_time_ms, target_pause_time_ms); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
430 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
431 |
// The number of recorded young regions is the incremental |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
432 |
// collection set's current size |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
433 |
set_recorded_rs_lengths(_inc_recorded_rs_lengths); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
434 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
435 |
double young_end_time_sec = os::elapsedTime(); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
436 |
phase_times()->record_young_cset_choice_time_ms((young_end_time_sec - young_start_time_sec) * 1000.0); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
437 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
438 |
return time_remaining_ms; |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
439 |
} |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
440 |
|
52897
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
441 |
void G1CollectionSet::add_as_old(HeapRegion* hr) { |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
442 |
cset_chooser()->pop(); // already have region via peek() |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
443 |
_g1h->old_set_remove(hr); |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
444 |
add_old_region(hr); |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
445 |
} |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
446 |
|
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
447 |
void G1CollectionSet::add_as_optional(HeapRegion* hr) { |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
448 |
assert(_optional_regions != NULL, "Must not be called before array is allocated"); |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
449 |
cset_chooser()->pop(); // already have region via peek() |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
450 |
_g1h->old_set_remove(hr); |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
451 |
add_optional_region(hr); |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
452 |
} |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
453 |
|
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
454 |
bool G1CollectionSet::optional_is_full() { |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
455 |
assert(_optional_region_length <= _optional_region_max_length, "Invariant"); |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
456 |
return _optional_region_length == _optional_region_max_length; |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
457 |
} |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
458 |
|
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
459 |
void G1CollectionSet::clear_optional_region(const HeapRegion* hr) { |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
460 |
assert(_optional_regions != NULL, "Must not be called before array is allocated"); |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
461 |
uint index = hr->index_in_opt_cset(); |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
462 |
_optional_regions[index] = NULL; |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
463 |
} |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
464 |
|
41075
d60e54a9bb67
8165313: Inserting freed regions during Free Collection Set serial phase takes very long on huge heaps
tschatzl
parents:
39698
diff
changeset
|
465 |
static int compare_region_idx(const uint a, const uint b) { |
d60e54a9bb67
8165313: Inserting freed regions during Free Collection Set serial phase takes very long on huge heaps
tschatzl
parents:
39698
diff
changeset
|
466 |
if (a > b) { |
d60e54a9bb67
8165313: Inserting freed regions during Free Collection Set serial phase takes very long on huge heaps
tschatzl
parents:
39698
diff
changeset
|
467 |
return 1; |
d60e54a9bb67
8165313: Inserting freed regions during Free Collection Set serial phase takes very long on huge heaps
tschatzl
parents:
39698
diff
changeset
|
468 |
} else if (a == b) { |
d60e54a9bb67
8165313: Inserting freed regions during Free Collection Set serial phase takes very long on huge heaps
tschatzl
parents:
39698
diff
changeset
|
469 |
return 0; |
d60e54a9bb67
8165313: Inserting freed regions during Free Collection Set serial phase takes very long on huge heaps
tschatzl
parents:
39698
diff
changeset
|
470 |
} else { |
d60e54a9bb67
8165313: Inserting freed regions during Free Collection Set serial phase takes very long on huge heaps
tschatzl
parents:
39698
diff
changeset
|
471 |
return -1; |
d60e54a9bb67
8165313: Inserting freed regions during Free Collection Set serial phase takes very long on huge heaps
tschatzl
parents:
39698
diff
changeset
|
472 |
} |
d60e54a9bb67
8165313: Inserting freed regions during Free Collection Set serial phase takes very long on huge heaps
tschatzl
parents:
39698
diff
changeset
|
473 |
} |
d60e54a9bb67
8165313: Inserting freed regions during Free Collection Set serial phase takes very long on huge heaps
tschatzl
parents:
39698
diff
changeset
|
474 |
|
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
475 |
void G1CollectionSet::finalize_old_part(double time_remaining_ms) { |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
476 |
double non_young_start_time_sec = os::elapsedTime(); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
477 |
double predicted_old_time_ms = 0.0; |
52897
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
478 |
double predicted_optional_time_ms = 0.0; |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
479 |
double optional_threshold_ms = time_remaining_ms * _policy->optional_prediction_fraction(); |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
480 |
uint expensive_region_num = 0; |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
481 |
|
49643
a3453bbd5418
8199742: Clean up state flags in G1CollectorState
tschatzl
parents:
49333
diff
changeset
|
482 |
if (collector_state()->in_mixed_phase()) { |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
483 |
cset_chooser()->verify(); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
484 |
const uint min_old_cset_length = _policy->calc_min_old_cset_length(); |
52897
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
485 |
const uint max_old_cset_length = MAX2(min_old_cset_length, _policy->calc_max_old_cset_length()); |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
486 |
bool check_time_remaining = _policy->adaptive_young_list_length(); |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
487 |
|
52897
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
488 |
initialize_optional(max_old_cset_length - min_old_cset_length); |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
489 |
log_debug(gc, ergo, cset)("Start adding old regions for mixed gc. min %u regions, max %u regions, " |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
490 |
"time remaining %1.2fms, optional threshold %1.2fms", |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
491 |
min_old_cset_length, max_old_cset_length, time_remaining_ms, optional_threshold_ms); |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
492 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
493 |
HeapRegion* hr = cset_chooser()->peek(); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
494 |
while (hr != NULL) { |
52897
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
495 |
if (old_region_length() + optional_region_length() >= max_old_cset_length) { |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
496 |
// Added maximum number of old regions to the CSet. |
52897
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
497 |
log_debug(gc, ergo, cset)("Finish adding old regions to CSet (old CSet region num reached max). " |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
498 |
"old %u regions, optional %u regions", |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
499 |
old_region_length(), optional_region_length()); |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
500 |
break; |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
501 |
} |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
502 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
503 |
// Stop adding regions if the remaining reclaimable space is |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
504 |
// not above G1HeapWastePercent. |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
505 |
size_t reclaimable_bytes = cset_chooser()->remaining_reclaimable_bytes(); |
47681
149745044e48
8189729: Change _perc suffixes in identifiers to _percent
tschatzl
parents:
47216
diff
changeset
|
506 |
double reclaimable_percent = _policy->reclaimable_bytes_percent(reclaimable_bytes); |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
507 |
double threshold = (double) G1HeapWastePercent; |
47681
149745044e48
8189729: Change _perc suffixes in identifiers to _percent
tschatzl
parents:
47216
diff
changeset
|
508 |
if (reclaimable_percent <= threshold) { |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
509 |
// We've added enough old regions that the amount of uncollected |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
510 |
// reclaimable space is at or below the waste threshold. Stop |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
511 |
// adding old regions to the CSet. |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
512 |
log_debug(gc, ergo, cset)("Finish adding old regions to CSet (reclaimable percentage not over threshold). " |
52897
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
513 |
"reclaimable: " SIZE_FORMAT "%s (%1.2f%%) threshold: " UINTX_FORMAT "%%", |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
514 |
byte_size_in_proper_unit(reclaimable_bytes), proper_unit_for_byte_size(reclaimable_bytes), |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
515 |
reclaimable_percent, G1HeapWastePercent); |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
516 |
break; |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
517 |
} |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
518 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
519 |
double predicted_time_ms = predict_region_elapsed_time_ms(hr); |
52897
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
520 |
time_remaining_ms = MAX2(time_remaining_ms - predicted_time_ms, 0.0); |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
521 |
// Add regions to old set until we reach minimum amount |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
522 |
if (old_region_length() < min_old_cset_length) { |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
523 |
predicted_old_time_ms += predicted_time_ms; |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
524 |
add_as_old(hr); |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
525 |
// Record the number of regions added when no time remaining |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
526 |
if (time_remaining_ms == 0.0) { |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
527 |
expensive_region_num++; |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
528 |
} |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
529 |
} else { |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
530 |
// In the non-auto-tuning case, we'll finish adding regions |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
531 |
// to the CSet if we reach the minimum. |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
532 |
if (!check_time_remaining) { |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
533 |
log_debug(gc, ergo, cset)("Finish adding old regions to CSet (old CSet region num reached min)."); |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
534 |
break; |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
535 |
} |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
536 |
// Keep adding regions to old set until we reach optional threshold |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
537 |
if (time_remaining_ms > optional_threshold_ms) { |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
538 |
predicted_old_time_ms += predicted_time_ms; |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
539 |
add_as_old(hr); |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
540 |
} else if (time_remaining_ms > 0) { |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
541 |
// Keep adding optional regions until time is up |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
542 |
if (!optional_is_full()) { |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
543 |
predicted_optional_time_ms += predicted_time_ms; |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
544 |
add_as_optional(hr); |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
545 |
} else { |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
546 |
log_debug(gc, ergo, cset)("Finish adding old regions to CSet (optional set full)."); |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
547 |
break; |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
548 |
} |
52897
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
549 |
} else { |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
550 |
log_debug(gc, ergo, cset)("Finish adding old regions to CSet (predicted time is too high)."); |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
551 |
break; |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
552 |
} |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
553 |
} |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
554 |
hr = cset_chooser()->peek(); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
555 |
} |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
556 |
if (hr == NULL) { |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
557 |
log_debug(gc, ergo, cset)("Finish adding old regions to CSet (candidate old regions not available)"); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
558 |
} |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
559 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
560 |
cset_chooser()->verify(); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
561 |
} |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
562 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
563 |
stop_incremental_building(); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
564 |
|
52897
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
565 |
log_debug(gc, ergo, cset)("Finish choosing CSet regions old: %u, optional: %u, " |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
566 |
"predicted old time: %1.2fms, predicted optional time: %1.2fms, time remaining: %1.2f", |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
567 |
old_region_length(), optional_region_length(), |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
568 |
predicted_old_time_ms, predicted_optional_time_ms, time_remaining_ms); |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
569 |
if (expensive_region_num > 0) { |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
570 |
log_debug(gc, ergo, cset)("CSet contains %u old regions that were added although the predicted time was too high.", |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
571 |
expensive_region_num); |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
572 |
} |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
573 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
574 |
double non_young_end_time_sec = os::elapsedTime(); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
575 |
phase_times()->record_non_young_cset_choice_time_ms((non_young_end_time_sec - non_young_start_time_sec) * 1000.0); |
41075
d60e54a9bb67
8165313: Inserting freed regions during Free Collection Set serial phase takes very long on huge heaps
tschatzl
parents:
39698
diff
changeset
|
576 |
|
46768 | 577 |
QuickSort::sort(_collection_set_regions, _collection_set_cur_length, compare_region_idx, true); |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
578 |
} |
38109
6503703df058
8155209: Move setting of young index in cset to G1CollectionSet
mgerdin
parents:
37985
diff
changeset
|
579 |
|
52897
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
580 |
HeapRegion* G1OptionalCSet::region_at(uint index) { |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
581 |
return _cset->optional_region_at(index); |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
582 |
} |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
583 |
|
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
584 |
void G1OptionalCSet::prepare_evacuation(double time_limit) { |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
585 |
assert(_current_index == _current_limit, "Before prepare no regions should be ready for evac"); |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
586 |
|
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
587 |
uint prepared_regions = 0; |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
588 |
double prediction_ms = 0; |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
589 |
|
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
590 |
_prepare_failed = true; |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
591 |
for (uint i = _current_index; i < _cset->optional_region_length(); i++) { |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
592 |
HeapRegion* hr = region_at(i); |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
593 |
prediction_ms += _cset->predict_region_elapsed_time_ms(hr); |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
594 |
if (prediction_ms > time_limit) { |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
595 |
log_debug(gc, cset)("Prepared %u regions for optional evacuation. Predicted time: %.3fms", prepared_regions, prediction_ms); |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
596 |
return; |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
597 |
} |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
598 |
|
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
599 |
// This region will be included in the next optional evacuation. |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
600 |
prepare_to_evacuate_optional_region(hr); |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
601 |
prepared_regions++; |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
602 |
_current_limit++; |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
603 |
_prepare_failed = false; |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
604 |
} |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
605 |
|
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
606 |
log_debug(gc, cset)("Prepared all %u regions for optional evacuation. Predicted time: %.3fms", |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
607 |
prepared_regions, prediction_ms); |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
608 |
} |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
609 |
|
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
610 |
bool G1OptionalCSet::prepare_failed() { |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
611 |
return _prepare_failed; |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
612 |
} |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
613 |
|
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
614 |
void G1OptionalCSet::complete_evacuation() { |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
615 |
_evacuation_failed = false; |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
616 |
for (uint i = _current_index; i < _current_limit; i++) { |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
617 |
HeapRegion* hr = region_at(i); |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
618 |
_cset->clear_optional_region(hr); |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
619 |
if (hr->evacuation_failed()){ |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
620 |
_evacuation_failed = true; |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
621 |
} |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
622 |
} |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
623 |
_current_index = _current_limit; |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
624 |
} |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
625 |
|
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
626 |
bool G1OptionalCSet::evacuation_failed() { |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
627 |
return _evacuation_failed; |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
628 |
} |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
629 |
|
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
630 |
G1OptionalCSet::~G1OptionalCSet() { |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
631 |
G1CollectedHeap* g1h = G1CollectedHeap::heap(); |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
632 |
while (!is_empty()) { |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
633 |
// We want to return regions not evacuated to the |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
634 |
// chooser in reverse order to maintain the old order. |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
635 |
HeapRegion* hr = _cset->remove_last_optional_region(); |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
636 |
assert(hr != NULL, "Should be valid region left"); |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
637 |
_pset->record_unused_optional_region(hr); |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
638 |
g1h->old_set_add(hr); |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
639 |
g1h->clear_in_cset(hr); |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
640 |
hr->set_index_in_opt_cset(InvalidCSetIndex); |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
641 |
_cset->cset_chooser()->push(hr); |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
642 |
} |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
643 |
_cset->free_optional_regions(); |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
644 |
} |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
645 |
|
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
646 |
uint G1OptionalCSet::size() { |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
647 |
return _cset->optional_region_length() - _current_index; |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
648 |
} |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
649 |
|
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
650 |
bool G1OptionalCSet::is_empty() { |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
651 |
return size() == 0; |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
652 |
} |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
653 |
|
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
654 |
void G1OptionalCSet::prepare_to_evacuate_optional_region(HeapRegion* hr) { |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
655 |
log_trace(gc, cset)("Adding region %u for optional evacuation", hr->hrm_index()); |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
656 |
G1CollectedHeap::heap()->clear_in_cset(hr); |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
657 |
_cset->add_old_region(hr); |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
658 |
} |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
659 |
|
38109
6503703df058
8155209: Move setting of young index in cset to G1CollectionSet
mgerdin
parents:
37985
diff
changeset
|
660 |
#ifdef ASSERT |
39698
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
661 |
class G1VerifyYoungCSetIndicesClosure : public HeapRegionClosure { |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
662 |
private: |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
663 |
size_t _young_length; |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
664 |
int* _heap_region_indices; |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
665 |
public: |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
666 |
G1VerifyYoungCSetIndicesClosure(size_t young_length) : HeapRegionClosure(), _young_length(young_length) { |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
667 |
_heap_region_indices = NEW_C_HEAP_ARRAY(int, young_length, mtGC); |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
668 |
for (size_t i = 0; i < young_length; i++) { |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
669 |
_heap_region_indices[i] = -1; |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
670 |
} |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
671 |
} |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
672 |
~G1VerifyYoungCSetIndicesClosure() { |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
673 |
FREE_C_HEAP_ARRAY(int, _heap_region_indices); |
38109
6503703df058
8155209: Move setting of young index in cset to G1CollectionSet
mgerdin
parents:
37985
diff
changeset
|
674 |
} |
6503703df058
8155209: Move setting of young index in cset to G1CollectionSet
mgerdin
parents:
37985
diff
changeset
|
675 |
|
48969 | 676 |
virtual bool do_heap_region(HeapRegion* r) { |
39698
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
677 |
const int idx = r->young_index_in_cset(); |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
678 |
|
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
679 |
assert(idx > -1, "Young index must be set for all regions in the incremental collection set but is not for region %u.", r->hrm_index()); |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
680 |
assert((size_t)idx < _young_length, "Young cset index too large for region %u", r->hrm_index()); |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
681 |
|
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
682 |
assert(_heap_region_indices[idx] == -1, |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
683 |
"Index %d used by multiple regions, first use by region %u, second by region %u", |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
684 |
idx, _heap_region_indices[idx], r->hrm_index()); |
38109
6503703df058
8155209: Move setting of young index in cset to G1CollectionSet
mgerdin
parents:
37985
diff
changeset
|
685 |
|
39698
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
686 |
_heap_region_indices[idx] = r->hrm_index(); |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
687 |
|
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
688 |
return false; |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
689 |
} |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
690 |
}; |
38109
6503703df058
8155209: Move setting of young index in cset to G1CollectionSet
mgerdin
parents:
37985
diff
changeset
|
691 |
|
39698
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
692 |
void G1CollectionSet::verify_young_cset_indices() const { |
49333
489f1dd40582
8196876: OopStorage::assert_at_safepoint clashes with assert_at_safepoint macros in g1CollectedHeap.hpp
kbarrett
parents:
48969
diff
changeset
|
693 |
assert_at_safepoint_on_vm_thread(); |
39698
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
694 |
|
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
695 |
G1VerifyYoungCSetIndicesClosure cl(_collection_set_cur_length); |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
696 |
iterate(&cl); |
38109
6503703df058
8155209: Move setting of young index in cset to G1CollectionSet
mgerdin
parents:
37985
diff
changeset
|
697 |
} |
6503703df058
8155209: Move setting of young index in cset to G1CollectionSet
mgerdin
parents:
37985
diff
changeset
|
698 |
#endif |