author | kbarrett |
Tue, 30 Aug 2016 23:48:16 -0400 | |
changeset 40892 | 330a02d935ad |
parent 39698 | 4016de4e596b |
child 41075 | d60e54a9bb67 |
permissions | -rw-r--r-- |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
1 |
/* |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
2 |
* Copyright (c) 2016, Oracle and/or its affiliates. All rights reserved. |
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" |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
26 |
#include "gc/g1/g1CollectedHeap.hpp" |
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" |
37985
539c597ee0fa
8154154: Separate G1 specific policy code from the CollectorPolicy class hierarchy
mgerdin
parents:
37262
diff
changeset
|
29 |
#include "gc/g1/g1Policy.hpp" |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
30 |
#include "gc/g1/heapRegion.inline.hpp" |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
31 |
#include "gc/g1/heapRegionRemSet.hpp" |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
32 |
#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
|
33 |
#include "logging/logStream.hpp" |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
34 |
#include "utilities/debug.hpp" |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
35 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
36 |
G1CollectorState* G1CollectionSet::collector_state() { |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
37 |
return _g1->collector_state(); |
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 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
40 |
G1GCPhaseTimes* G1CollectionSet::phase_times() { |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
41 |
return _policy->phase_times(); |
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 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
44 |
CollectionSetChooser* G1CollectionSet::cset_chooser() { |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
45 |
return _cset_chooser; |
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 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
48 |
double G1CollectionSet::predict_region_elapsed_time_ms(HeapRegion* hr) { |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
49 |
return _policy->predict_region_elapsed_time_ms(hr, collector_state()->gcs_are_young()); |
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 |
|
37985
539c597ee0fa
8154154: Separate G1 specific policy code from the CollectorPolicy class hierarchy
mgerdin
parents:
37262
diff
changeset
|
52 |
G1CollectionSet::G1CollectionSet(G1CollectedHeap* g1h, G1Policy* policy) : |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
53 |
_g1(g1h), |
37985
539c597ee0fa
8154154: Separate G1 specific policy code from the CollectorPolicy class hierarchy
mgerdin
parents:
37262
diff
changeset
|
54 |
_policy(policy), |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
55 |
_cset_chooser(new CollectionSetChooser()), |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
56 |
_eden_region_length(0), |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
57 |
_survivor_region_length(0), |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
58 |
_old_region_length(0), |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
59 |
_bytes_used_before(0), |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
60 |
_recorded_rs_lengths(0), |
39698
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
61 |
_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
|
62 |
_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
|
63 |
_collection_set_max_length(0), |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
64 |
// Incremental CSet attributes |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
65 |
_inc_build_state(Inactive), |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
66 |
_inc_bytes_used_before(0), |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
67 |
_inc_recorded_rs_lengths(0), |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
68 |
_inc_recorded_rs_lengths_diffs(0), |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
69 |
_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
|
70 |
_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
|
71 |
} |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
72 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
73 |
G1CollectionSet::~G1CollectionSet() { |
39698
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
74 |
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
|
75 |
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
|
76 |
} |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
77 |
delete _cset_chooser; |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
78 |
} |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
79 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
80 |
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
|
81 |
uint survivor_cset_region_length) { |
39698
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
82 |
assert_at_safepoint(true); |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
83 |
|
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
84 |
_eden_region_length = eden_cset_region_length; |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
85 |
_survivor_region_length = survivor_cset_region_length; |
38109
6503703df058
8155209: Move setting of young index in cset to G1CollectionSet
mgerdin
parents:
37985
diff
changeset
|
86 |
|
39698
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
87 |
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
|
88 |
"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
|
89 |
|
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
90 |
_old_region_length = 0; |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
91 |
} |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
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 |
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
|
94 |
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
|
95 |
_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
|
96 |
_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
|
97 |
} |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
98 |
|
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
99 |
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
|
100 |
_recorded_rs_lengths = rs_lengths; |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
101 |
} |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
102 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
103 |
// 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
|
104 |
void G1CollectionSet::add_old_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
|
105 |
assert_at_safepoint(true); |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
106 |
|
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
107 |
assert(_inc_build_state == Active, "Precondition"); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
108 |
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
|
109 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
110 |
assert(!hr->in_collection_set(), "should not already be in the CSet"); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
111 |
_g1->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
|
112 |
|
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
113 |
_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
|
114 |
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
|
115 |
|
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
116 |
_bytes_used_before += hr->used(); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
117 |
size_t rs_length = hr->rem_set()->occupied(); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
118 |
_recorded_rs_lengths += rs_length; |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
119 |
_old_region_length += 1; |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
120 |
} |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
121 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
122 |
// Initialize the per-collection-set information |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
123 |
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
|
124 |
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
|
125 |
assert(_inc_build_state == Inactive, "Precondition"); |
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 |
_inc_bytes_used_before = 0; |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
128 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
129 |
_inc_recorded_rs_lengths = 0; |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
130 |
_inc_recorded_rs_lengths_diffs = 0; |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
131 |
_inc_predicted_elapsed_time_ms = 0.0; |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
132 |
_inc_predicted_elapsed_time_ms_diffs = 0.0; |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
133 |
_inc_build_state = Active; |
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 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
136 |
void G1CollectionSet::finalize_incremental_building() { |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
137 |
assert(_inc_build_state == Active, "Precondition"); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
138 |
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
|
139 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
140 |
// 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
|
141 |
// _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
|
142 |
// 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
|
143 |
// 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
|
144 |
// 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
|
145 |
// the "main" fields. |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
146 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
147 |
if (_inc_recorded_rs_lengths_diffs >= 0) { |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
148 |
_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
|
149 |
} else { |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
150 |
// 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
|
151 |
// 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
|
152 |
// 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
|
153 |
// 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
|
154 |
// could make the calculations somewhat inaccurate. |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
155 |
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
|
156 |
if (_inc_recorded_rs_lengths >= diffs) { |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
157 |
_inc_recorded_rs_lengths -= diffs; |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
158 |
} else { |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
159 |
_inc_recorded_rs_lengths = 0; |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
160 |
} |
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 |
_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
|
163 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
164 |
_inc_recorded_rs_lengths_diffs = 0; |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
165 |
_inc_predicted_elapsed_time_ms_diffs = 0.0; |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
166 |
} |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
167 |
|
39698
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
168 |
void G1CollectionSet::clear() { |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
169 |
assert_at_safepoint(true); |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
170 |
_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
|
171 |
} |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
172 |
|
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
173 |
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
|
174 |
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
|
175 |
} |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
176 |
|
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
177 |
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
|
178 |
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
|
179 |
OrderAccess::loadload(); |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
180 |
if (len == 0) { |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
181 |
return; |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
182 |
} |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
183 |
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
|
184 |
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
|
185 |
|
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
186 |
do { |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
187 |
HeapRegion* r = G1CollectedHeap::heap()->region_at(_collection_set_regions[cur_pos]); |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
188 |
bool result = cl->doHeapRegion(r); |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
189 |
if (result) { |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
190 |
cl->incomplete(); |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
191 |
return; |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
192 |
} |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
193 |
cur_pos++; |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
194 |
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
|
195 |
cur_pos = 0; |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
196 |
} |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
197 |
} 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
|
198 |
} |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
199 |
|
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
200 |
void G1CollectionSet::update_young_region_prediction(HeapRegion* hr, |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
201 |
size_t new_rs_length) { |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
202 |
// 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
|
203 |
assert(hr->is_young(), "Precondition"); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
204 |
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
|
205 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
206 |
// 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
|
207 |
// _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
|
208 |
// 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
|
209 |
// refinement thread, potentially concurrently with a mutator thread |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
210 |
// 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
|
211 |
// 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
|
212 |
// 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
|
213 |
// fields at the start of a GC. |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
214 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
215 |
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
|
216 |
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
|
217 |
_inc_recorded_rs_lengths_diffs += rs_lengths_diff; |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
218 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
219 |
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
|
220 |
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
|
221 |
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
|
222 |
_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
|
223 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
224 |
hr->set_recorded_rs_length(new_rs_length); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
225 |
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
|
226 |
} |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
227 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
228 |
void G1CollectionSet::add_young_region_common(HeapRegion* hr) { |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
229 |
assert(hr->is_young(), "invariant"); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
230 |
assert(_inc_build_state == Active, "Precondition"); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
231 |
|
39698
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
232 |
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
|
233 |
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
|
234 |
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
|
235 |
|
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
236 |
_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
|
237 |
// 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
|
238 |
// 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
|
239 |
OrderAccess::storestore(); |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
240 |
_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
|
241 |
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
|
242 |
|
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
243 |
// This routine is used when: |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
244 |
// * 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
|
245 |
// evacuation pause or |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
246 |
// * 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
|
247 |
// when it is retired. |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
248 |
// 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
|
249 |
// 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
|
250 |
// retiring the current allocation region) |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
251 |
// 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
|
252 |
// 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
|
253 |
// 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
|
254 |
// 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
|
255 |
// 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
|
256 |
// by the Young List sampling code. |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
257 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
258 |
size_t rs_length = hr->rem_set()->occupied(); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
259 |
double 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
|
260 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
261 |
// Cache the values we have added to the aggregated information |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
262 |
// in the heap region in case we have to remove this region from |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
263 |
// the incremental collection set, or it is updated by the |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
264 |
// rset sampling code |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
265 |
hr->set_recorded_rs_length(rs_length); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
266 |
hr->set_predicted_elapsed_time_ms(region_elapsed_time_ms); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
267 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
268 |
size_t used_bytes = hr->used(); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
269 |
_inc_recorded_rs_lengths += rs_length; |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
270 |
_inc_predicted_elapsed_time_ms += region_elapsed_time_ms; |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
271 |
_inc_bytes_used_before += used_bytes; |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
272 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
273 |
assert(!hr->in_collection_set(), "invariant"); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
274 |
_g1->register_young_region_with_cset(hr); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
275 |
} |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
276 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
277 |
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
|
278 |
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
|
279 |
add_young_region_common(hr); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
280 |
} |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
281 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
282 |
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
|
283 |
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
|
284 |
add_young_region_common(hr); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
285 |
} |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
286 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
287 |
#ifndef PRODUCT |
39698
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
288 |
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
|
289 |
public: |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
290 |
bool _valid; |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
291 |
public: |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
292 |
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
|
293 |
|
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
294 |
virtual bool doHeapRegion(HeapRegion* r) { |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
295 |
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
|
296 |
|
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
297 |
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
|
298 |
|
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
299 |
if (group == NULL) { |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
300 |
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
|
301 |
_valid = false; |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
302 |
} |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
303 |
|
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
304 |
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
|
305 |
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
|
306 |
_valid = false; |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
307 |
} |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
308 |
|
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
309 |
return false; |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
310 |
} |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
311 |
|
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
312 |
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
|
313 |
}; |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
314 |
|
39698
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
315 |
bool G1CollectionSet::verify_young_ages() { |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
316 |
assert_at_safepoint(true); |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
317 |
|
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
318 |
G1VerifyYoungAgesClosure cl; |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
319 |
iterate(&cl); |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
320 |
|
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
321 |
if (!cl.valid()) { |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
322 |
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
|
323 |
print(&log); |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
324 |
} |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
325 |
|
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
326 |
return cl.valid(); |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
327 |
} |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
328 |
|
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
329 |
class G1PrintCollectionSetClosure : public HeapRegionClosure { |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
330 |
outputStream* _st; |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
331 |
public: |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
332 |
G1PrintCollectionSetClosure(outputStream* st) : HeapRegionClosure(), _st(st) { } |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
333 |
|
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
334 |
virtual bool doHeapRegion(HeapRegion* r) { |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
335 |
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
|
336 |
_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
|
337 |
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
|
338 |
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
|
339 |
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
|
340 |
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
|
341 |
return false; |
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 |
}; |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
344 |
|
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
345 |
void G1CollectionSet::print(outputStream* st) { |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
346 |
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
|
347 |
|
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
348 |
G1PrintCollectionSetClosure cl(st); |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
349 |
iterate(&cl); |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
350 |
} |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
351 |
#endif // !PRODUCT |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
352 |
|
38183
cb68e4923223
8150721: Don't explicitly manage G1 young regions in YoungList
mgerdin
parents:
38162
diff
changeset
|
353 |
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
|
354 |
double young_start_time_sec = os::elapsedTime(); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
355 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
356 |
finalize_incremental_building(); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
357 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
358 |
guarantee(target_pause_time_ms > 0.0, |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
359 |
"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
|
360 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
361 |
size_t pending_cards = _policy->pending_cards(); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
362 |
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
|
363 |
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
|
364 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
365 |
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
|
366 |
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
|
367 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
368 |
collector_state()->set_last_gc_was_young(collector_state()->gcs_are_young()); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
369 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
370 |
// 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
|
371 |
// 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
|
372 |
// [Newly Young Regions ++ Survivors from last pause]. |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
373 |
|
38183
cb68e4923223
8150721: Don't explicitly manage G1 young regions in YoungList
mgerdin
parents:
38162
diff
changeset
|
374 |
uint survivor_region_length = survivors->length(); |
cb68e4923223
8150721: Don't explicitly manage G1 young regions in YoungList
mgerdin
parents:
38162
diff
changeset
|
375 |
uint eden_region_length = _g1->eden_regions_count(); |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
376 |
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
|
377 |
|
38109
6503703df058
8155209: Move setting of young index in cset to G1CollectionSet
mgerdin
parents:
37985
diff
changeset
|
378 |
verify_young_cset_indices(); |
6503703df058
8155209: Move setting of young index in cset to G1CollectionSet
mgerdin
parents:
37985
diff
changeset
|
379 |
|
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
380 |
// 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
|
381 |
survivors->convert_to_eden(); |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
382 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
383 |
_bytes_used_before = _inc_bytes_used_before; |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
384 |
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
|
385 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
386 |
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
|
387 |
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
|
388 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
389 |
// 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
|
390 |
// collection set's current size |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
391 |
set_recorded_rs_lengths(_inc_recorded_rs_lengths); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
392 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
393 |
double young_end_time_sec = os::elapsedTime(); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
394 |
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
|
395 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
396 |
return time_remaining_ms; |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
397 |
} |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
398 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
399 |
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
|
400 |
double non_young_start_time_sec = os::elapsedTime(); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
401 |
double predicted_old_time_ms = 0.0; |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
402 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
403 |
if (!collector_state()->gcs_are_young()) { |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
404 |
cset_chooser()->verify(); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
405 |
const uint min_old_cset_length = _policy->calc_min_old_cset_length(); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
406 |
const uint max_old_cset_length = _policy->calc_max_old_cset_length(); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
407 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
408 |
uint expensive_region_num = 0; |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
409 |
bool check_time_remaining = _policy->adaptive_young_list_length(); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
410 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
411 |
HeapRegion* hr = cset_chooser()->peek(); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
412 |
while (hr != NULL) { |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
413 |
if (old_region_length() >= max_old_cset_length) { |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
414 |
// Added maximum number of old regions to the CSet. |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
415 |
log_debug(gc, ergo, cset)("Finish adding old regions to CSet (old CSet region num reached max). old %u regions, max %u regions", |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
416 |
old_region_length(), max_old_cset_length); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
417 |
break; |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
418 |
} |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
419 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
420 |
// 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
|
421 |
// not above G1HeapWastePercent. |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
422 |
size_t reclaimable_bytes = cset_chooser()->remaining_reclaimable_bytes(); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
423 |
double reclaimable_perc = _policy->reclaimable_bytes_perc(reclaimable_bytes); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
424 |
double threshold = (double) G1HeapWastePercent; |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
425 |
if (reclaimable_perc <= threshold) { |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
426 |
// 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
|
427 |
// 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
|
428 |
// adding old regions to the CSet. |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
429 |
log_debug(gc, ergo, cset)("Finish adding old regions to CSet (reclaimable percentage not over threshold). " |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
430 |
"old %u regions, max %u regions, reclaimable: " SIZE_FORMAT "B (%1.2f%%) threshold: " UINTX_FORMAT "%%", |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
431 |
old_region_length(), max_old_cset_length, reclaimable_bytes, reclaimable_perc, G1HeapWastePercent); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
432 |
break; |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
433 |
} |
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 predicted_time_ms = predict_region_elapsed_time_ms(hr); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
436 |
if (check_time_remaining) { |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
437 |
if (predicted_time_ms > time_remaining_ms) { |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
438 |
// Too expensive for the current CSet. |
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 |
if (old_region_length() >= min_old_cset_length) { |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
441 |
// We have added the minimum number of old regions to the CSet, |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
442 |
// we are done with this CSet. |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
443 |
log_debug(gc, ergo, cset)("Finish adding old regions to CSet (predicted time is too high). " |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
444 |
"predicted time: %1.2fms, remaining time: %1.2fms old %u regions, min %u regions", |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
445 |
predicted_time_ms, time_remaining_ms, old_region_length(), min_old_cset_length); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
446 |
break; |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
447 |
} |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
448 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
449 |
// We'll add it anyway given that we haven't reached the |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
450 |
// minimum number of old regions. |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
451 |
expensive_region_num += 1; |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
452 |
} |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
453 |
} else { |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
454 |
if (old_region_length() >= min_old_cset_length) { |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
455 |
// In the non-auto-tuning case, we'll finish adding regions |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
456 |
// to the CSet if we reach the minimum. |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
457 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
458 |
log_debug(gc, ergo, cset)("Finish adding old regions to CSet (old CSet region num reached min). old %u regions, min %u regions", |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
459 |
old_region_length(), min_old_cset_length); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
460 |
break; |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
461 |
} |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
462 |
} |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
463 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
464 |
// We will add this region to the CSet. |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
465 |
time_remaining_ms = MAX2(time_remaining_ms - predicted_time_ms, 0.0); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
466 |
predicted_old_time_ms += predicted_time_ms; |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
467 |
cset_chooser()->pop(); // already have region via peek() |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
468 |
_g1->old_set_remove(hr); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
469 |
add_old_region(hr); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
470 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
471 |
hr = cset_chooser()->peek(); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
472 |
} |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
473 |
if (hr == NULL) { |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
474 |
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
|
475 |
} |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
476 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
477 |
if (expensive_region_num > 0) { |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
478 |
// We print the information once here at the end, predicated on |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
479 |
// whether we added any apparently expensive regions or not, to |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
480 |
// avoid generating output per region. |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
481 |
log_debug(gc, ergo, cset)("Added expensive regions to CSet (old CSet region num not reached min)." |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
482 |
"old: %u regions, expensive: %u regions, min: %u regions, remaining time: %1.2fms", |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
483 |
old_region_length(), expensive_region_num, min_old_cset_length, time_remaining_ms); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
484 |
} |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
485 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
486 |
cset_chooser()->verify(); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
487 |
} |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
488 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
489 |
stop_incremental_building(); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
490 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
491 |
log_debug(gc, ergo, cset)("Finish choosing CSet. old: %u regions, predicted old region time: %1.2fms, time remaining: %1.2f", |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
492 |
old_region_length(), predicted_old_time_ms, time_remaining_ms); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
493 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
494 |
double non_young_end_time_sec = os::elapsedTime(); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
495 |
phase_times()->record_non_young_cset_choice_time_ms((non_young_end_time_sec - non_young_start_time_sec) * 1000.0); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
496 |
} |
38109
6503703df058
8155209: Move setting of young index in cset to G1CollectionSet
mgerdin
parents:
37985
diff
changeset
|
497 |
|
6503703df058
8155209: Move setting of young index in cset to G1CollectionSet
mgerdin
parents:
37985
diff
changeset
|
498 |
#ifdef ASSERT |
39698
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
499 |
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
|
500 |
private: |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
501 |
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
|
502 |
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
|
503 |
public: |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
504 |
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
|
505 |
_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
|
506 |
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
|
507 |
_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
|
508 |
} |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
509 |
} |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
510 |
~G1VerifyYoungCSetIndicesClosure() { |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
511 |
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
|
512 |
} |
6503703df058
8155209: Move setting of young index in cset to G1CollectionSet
mgerdin
parents:
37985
diff
changeset
|
513 |
|
39698
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
514 |
virtual bool doHeapRegion(HeapRegion* r) { |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
515 |
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
|
516 |
|
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
517 |
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
|
518 |
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
|
519 |
|
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
520 |
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
|
521 |
"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
|
522 |
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
|
523 |
|
39698
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
524 |
_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
|
525 |
|
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
526 |
return false; |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
527 |
} |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
528 |
}; |
38109
6503703df058
8155209: Move setting of young index in cset to G1CollectionSet
mgerdin
parents:
37985
diff
changeset
|
529 |
|
39698
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
530 |
void G1CollectionSet::verify_young_cset_indices() const { |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
531 |
assert_at_safepoint(true); |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
532 |
|
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
533 |
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
|
534 |
iterate(&cl); |
38109
6503703df058
8155209: Move setting of young index in cset to G1CollectionSet
mgerdin
parents:
37985
diff
changeset
|
535 |
} |
6503703df058
8155209: Move setting of young index in cset to G1CollectionSet
mgerdin
parents:
37985
diff
changeset
|
536 |
#endif |