author | tschatzl |
Fri, 08 Feb 2019 12:55:20 +0100 | |
changeset 53703 | 24341625d8f2 |
parent 52897 | 495c05ee2a9a |
child 54465 | c4f16445675a |
permissions | -rw-r--r-- |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
1 |
/* |
53703
24341625d8f2
8217330: Split G1CollectionSetChooser into collection set candidate container and the chooser algorithm
tschatzl
parents:
52897
diff
changeset
|
2 |
* Copyright (c) 2016, 2019, 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" |
53703
24341625d8f2
8217330: Split G1CollectionSetChooser into collection set candidate container and the chooser algorithm
tschatzl
parents:
52897
diff
changeset
|
28 |
#include "gc/g1/g1CollectionSetCandidates.hpp" |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
29 |
#include "gc/g1/g1CollectorState.hpp" |
52897
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
30 |
#include "gc/g1/g1ParScanThreadState.hpp" |
37985
539c597ee0fa
8154154: Separate G1 specific policy code from the CollectorPolicy class hierarchy
mgerdin
parents:
37262
diff
changeset
|
31 |
#include "gc/g1/g1Policy.hpp" |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
32 |
#include "gc/g1/heapRegion.inline.hpp" |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
33 |
#include "gc/g1/heapRegionRemSet.hpp" |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
34 |
#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
|
35 |
#include "logging/logStream.hpp" |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
36 |
#include "utilities/debug.hpp" |
52897
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
37 |
#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
|
38 |
#include "utilities/quickSort.hpp" |
37039
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 |
G1CollectorState* G1CollectionSet::collector_state() { |
49806 | 41 |
return _g1h->collector_state(); |
37039
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 |
G1GCPhaseTimes* G1CollectionSet::phase_times() { |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
45 |
return _policy->phase_times(); |
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) { |
49643
a3453bbd5418
8199742: Clean up state flags in G1CollectorState
tschatzl
parents:
49333
diff
changeset
|
49 |
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
|
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) : |
49806 | 53 |
_g1h(g1h), |
37985
539c597ee0fa
8154154: Separate G1 specific policy code from the CollectorPolicy class hierarchy
mgerdin
parents:
37262
diff
changeset
|
54 |
_policy(policy), |
53703
24341625d8f2
8217330: Split G1CollectionSetChooser into collection set candidate container and the chooser algorithm
tschatzl
parents:
52897
diff
changeset
|
55 |
_candidates(NULL), |
37039
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), |
39698
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
59 |
_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
|
60 |
_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
|
61 |
_collection_set_max_length(0), |
52897
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
62 |
_optional_regions(NULL), |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
63 |
_optional_region_length(0), |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
64 |
_optional_region_max_length(0), |
51332 | 65 |
_bytes_used_before(0), |
66 |
_recorded_rs_lengths(0), |
|
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
67 |
_inc_build_state(Inactive), |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
68 |
_inc_bytes_used_before(0), |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
69 |
_inc_recorded_rs_lengths(0), |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
70 |
_inc_recorded_rs_lengths_diffs(0), |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
71 |
_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
|
72 |
_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
|
73 |
} |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
74 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
75 |
G1CollectionSet::~G1CollectionSet() { |
39698
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
76 |
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
|
77 |
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
|
78 |
} |
52897
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
79 |
free_optional_regions(); |
53703
24341625d8f2
8217330: Split G1CollectionSetChooser into collection set candidate container and the chooser algorithm
tschatzl
parents:
52897
diff
changeset
|
80 |
clear_candidates(); |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
81 |
} |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
82 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
83 |
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
|
84 |
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
|
85 |
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
|
86 |
|
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
87 |
_eden_region_length = eden_cset_region_length; |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
88 |
_survivor_region_length = survivor_cset_region_length; |
38109
6503703df058
8155209: Move setting of young index in cset to G1CollectionSet
mgerdin
parents:
37985
diff
changeset
|
89 |
|
39698
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
90 |
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
|
91 |
"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
|
92 |
|
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
93 |
_old_region_length = 0; |
52897
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
94 |
_optional_region_length = 0; |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
95 |
} |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
96 |
|
39698
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
97 |
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
|
98 |
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
|
99 |
_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
|
100 |
_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
|
101 |
} |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
102 |
|
52897
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
103 |
void G1CollectionSet::initialize_optional(uint max_length) { |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
104 |
assert(_optional_regions == NULL, "Already initialized"); |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
105 |
assert(_optional_region_length == 0, "Already initialized"); |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
106 |
assert(_optional_region_max_length == 0, "Already initialized"); |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
107 |
_optional_region_max_length = max_length; |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
108 |
_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
|
109 |
} |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
110 |
|
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
111 |
void G1CollectionSet::free_optional_regions() { |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
112 |
_optional_region_length = 0; |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
113 |
_optional_region_max_length = 0; |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
114 |
if (_optional_regions != NULL) { |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
115 |
FREE_C_HEAP_ARRAY(HeapRegion*, _optional_regions); |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
116 |
_optional_regions = NULL; |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
117 |
} |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
118 |
} |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
119 |
|
53703
24341625d8f2
8217330: Split G1CollectionSetChooser into collection set candidate container and the chooser algorithm
tschatzl
parents:
52897
diff
changeset
|
120 |
void G1CollectionSet::clear_candidates() { |
24341625d8f2
8217330: Split G1CollectionSetChooser into collection set candidate container and the chooser algorithm
tschatzl
parents:
52897
diff
changeset
|
121 |
delete _candidates; |
24341625d8f2
8217330: Split G1CollectionSetChooser into collection set candidate container and the chooser algorithm
tschatzl
parents:
52897
diff
changeset
|
122 |
_candidates = NULL; |
24341625d8f2
8217330: Split G1CollectionSetChooser into collection set candidate container and the chooser algorithm
tschatzl
parents:
52897
diff
changeset
|
123 |
} |
24341625d8f2
8217330: Split G1CollectionSetChooser into collection set candidate container and the chooser algorithm
tschatzl
parents:
52897
diff
changeset
|
124 |
|
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
125 |
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
|
126 |
_recorded_rs_lengths = rs_lengths; |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
127 |
} |
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 |
// 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
|
130 |
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
|
131 |
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
|
132 |
|
52897
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
133 |
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
|
134 |
"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
|
135 |
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
|
136 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
137 |
assert(!hr->in_collection_set(), "should not already be in the CSet"); |
49806 | 138 |
_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
|
139 |
|
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
140 |
_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
|
141 |
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
|
142 |
|
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
143 |
_bytes_used_before += hr->used(); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
144 |
size_t rs_length = hr->rem_set()->occupied(); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
145 |
_recorded_rs_lengths += rs_length; |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
146 |
_old_region_length += 1; |
52897
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 |
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
|
149 |
} |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
150 |
|
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
151 |
void G1CollectionSet::add_optional_region(HeapRegion* hr) { |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
152 |
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
|
153 |
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
|
154 |
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
|
155 |
|
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
156 |
_g1h->register_optional_region_with_cset(hr); |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
157 |
|
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
158 |
_optional_regions[_optional_region_length] = hr; |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
159 |
uint index = _optional_region_length++; |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
160 |
hr->set_index_in_opt_cset(index); |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
161 |
|
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
162 |
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
|
163 |
} |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
164 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
165 |
// Initialize the per-collection-set information |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
166 |
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
|
167 |
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
|
168 |
assert(_inc_build_state == Inactive, "Precondition"); |
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_bytes_used_before = 0; |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
171 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
172 |
_inc_recorded_rs_lengths = 0; |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
173 |
_inc_recorded_rs_lengths_diffs = 0; |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
174 |
_inc_predicted_elapsed_time_ms = 0.0; |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
175 |
_inc_predicted_elapsed_time_ms_diffs = 0.0; |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
176 |
_inc_build_state = Active; |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
177 |
} |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
178 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
179 |
void G1CollectionSet::finalize_incremental_building() { |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
180 |
assert(_inc_build_state == Active, "Precondition"); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
181 |
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
|
182 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
183 |
// 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
|
184 |
// _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
|
185 |
// 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
|
186 |
// 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
|
187 |
// 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
|
188 |
// the "main" fields. |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
189 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
190 |
if (_inc_recorded_rs_lengths_diffs >= 0) { |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
191 |
_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
|
192 |
} else { |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
193 |
// 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
|
194 |
// 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
|
195 |
// 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
|
196 |
// 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
|
197 |
// could make the calculations somewhat inaccurate. |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
198 |
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
|
199 |
if (_inc_recorded_rs_lengths >= diffs) { |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
200 |
_inc_recorded_rs_lengths -= diffs; |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
201 |
} else { |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
202 |
_inc_recorded_rs_lengths = 0; |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
203 |
} |
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_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
|
206 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
207 |
_inc_recorded_rs_lengths_diffs = 0; |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
208 |
_inc_predicted_elapsed_time_ms_diffs = 0.0; |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
209 |
} |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
210 |
|
39698
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
211 |
void G1CollectionSet::clear() { |
49333
489f1dd40582
8196876: OopStorage::assert_at_safepoint clashes with assert_at_safepoint macros in g1CollectedHeap.hpp
kbarrett
parents:
48969
diff
changeset
|
212 |
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
|
213 |
_collection_set_cur_length = 0; |
52897
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
214 |
_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
|
215 |
} |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
216 |
|
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
217 |
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
|
218 |
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
|
219 |
} |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
220 |
|
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
221 |
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
|
222 |
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
|
223 |
OrderAccess::loadload(); |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
224 |
if (len == 0) { |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
225 |
return; |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
226 |
} |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
227 |
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
|
228 |
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
|
229 |
|
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
230 |
do { |
49806 | 231 |
HeapRegion* r = _g1h->region_at(_collection_set_regions[cur_pos]); |
48969 | 232 |
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
|
233 |
if (result) { |
48969 | 234 |
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
|
235 |
return; |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
236 |
} |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
237 |
cur_pos++; |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
238 |
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
|
239 |
cur_pos = 0; |
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 |
} 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
|
242 |
} |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
243 |
|
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
244 |
void G1CollectionSet::update_young_region_prediction(HeapRegion* hr, |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
245 |
size_t new_rs_length) { |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
246 |
// 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
|
247 |
assert(hr->is_young(), "Precondition"); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
248 |
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
|
249 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
250 |
// 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
|
251 |
// _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
|
252 |
// 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
|
253 |
// refinement thread, potentially concurrently with a mutator thread |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
254 |
// 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
|
255 |
// 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
|
256 |
// 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
|
257 |
// fields at the start of a GC. |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
258 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
259 |
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
|
260 |
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
|
261 |
_inc_recorded_rs_lengths_diffs += rs_lengths_diff; |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
262 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
263 |
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
|
264 |
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
|
265 |
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
|
266 |
_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
|
267 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
268 |
hr->set_recorded_rs_length(new_rs_length); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
269 |
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
|
270 |
} |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
271 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
272 |
void G1CollectionSet::add_young_region_common(HeapRegion* hr) { |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
273 |
assert(hr->is_young(), "invariant"); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
274 |
assert(_inc_build_state == Active, "Precondition"); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
275 |
|
39698
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
276 |
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
|
277 |
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
|
278 |
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
|
279 |
|
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
280 |
_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
|
281 |
// 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
|
282 |
// 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
|
283 |
OrderAccess::storestore(); |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
284 |
_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
|
285 |
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
|
286 |
|
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
287 |
// This routine is used when: |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
288 |
// * 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
|
289 |
// evacuation pause or |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
290 |
// * 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
|
291 |
// when it is retired. |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
292 |
// 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
|
293 |
// 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
|
294 |
// retiring the current allocation region) |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
295 |
// 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
|
296 |
// 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
|
297 |
// 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
|
298 |
// 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
|
299 |
// 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
|
300 |
// by the Young List sampling code. |
49643
a3453bbd5418
8199742: Clean up state flags in G1CollectorState
tschatzl
parents:
49333
diff
changeset
|
301 |
// 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
|
302 |
|
49806 | 303 |
if (!_g1h->collector_state()->in_full_gc()) { |
49643
a3453bbd5418
8199742: Clean up state flags in G1CollectorState
tschatzl
parents:
49333
diff
changeset
|
304 |
size_t rs_length = hr->rem_set()->occupied(); |
a3453bbd5418
8199742: Clean up state flags in G1CollectorState
tschatzl
parents:
49333
diff
changeset
|
305 |
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
|
306 |
|
49643
a3453bbd5418
8199742: Clean up state flags in G1CollectorState
tschatzl
parents:
49333
diff
changeset
|
307 |
// Cache the values we have added to the aggregated information |
a3453bbd5418
8199742: Clean up state flags in G1CollectorState
tschatzl
parents:
49333
diff
changeset
|
308 |
// 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
|
309 |
// the incremental collection set, or it is updated by the |
a3453bbd5418
8199742: Clean up state flags in G1CollectorState
tschatzl
parents:
49333
diff
changeset
|
310 |
// rset sampling code |
a3453bbd5418
8199742: Clean up state flags in G1CollectorState
tschatzl
parents:
49333
diff
changeset
|
311 |
hr->set_recorded_rs_length(rs_length); |
a3453bbd5418
8199742: Clean up state flags in G1CollectorState
tschatzl
parents:
49333
diff
changeset
|
312 |
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
|
313 |
|
49643
a3453bbd5418
8199742: Clean up state flags in G1CollectorState
tschatzl
parents:
49333
diff
changeset
|
314 |
_inc_recorded_rs_lengths += rs_length; |
a3453bbd5418
8199742: Clean up state flags in G1CollectorState
tschatzl
parents:
49333
diff
changeset
|
315 |
_inc_predicted_elapsed_time_ms += region_elapsed_time_ms; |
a3453bbd5418
8199742: Clean up state flags in G1CollectorState
tschatzl
parents:
49333
diff
changeset
|
316 |
_inc_bytes_used_before += hr->used(); |
a3453bbd5418
8199742: Clean up state flags in G1CollectorState
tschatzl
parents:
49333
diff
changeset
|
317 |
} |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
318 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
319 |
assert(!hr->in_collection_set(), "invariant"); |
49806 | 320 |
_g1h->register_young_region_with_cset(hr); |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
321 |
} |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
322 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
323 |
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
|
324 |
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
|
325 |
add_young_region_common(hr); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
326 |
} |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
327 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
328 |
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
|
329 |
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
|
330 |
add_young_region_common(hr); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
331 |
} |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
332 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
333 |
#ifndef PRODUCT |
39698
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
334 |
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
|
335 |
public: |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
336 |
bool _valid; |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
337 |
public: |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
338 |
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
|
339 |
|
48969 | 340 |
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
|
341 |
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
|
342 |
|
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
343 |
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
|
344 |
|
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
345 |
if (group == NULL) { |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
346 |
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
|
347 |
_valid = false; |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
348 |
} |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
349 |
|
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
350 |
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
|
351 |
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
|
352 |
_valid = false; |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
353 |
} |
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 |
return false; |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
356 |
} |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
357 |
|
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
358 |
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
|
359 |
}; |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
360 |
|
39698
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
361 |
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
|
362 |
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
|
363 |
|
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
364 |
G1VerifyYoungAgesClosure cl; |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
365 |
iterate(&cl); |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
366 |
|
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
367 |
if (!cl.valid()) { |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
368 |
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
|
369 |
print(&log); |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
370 |
} |
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 |
return cl.valid(); |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
373 |
} |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
374 |
|
51475
492b366f8e57
8209758: 2 classes with same name G1PrintCollectionSetClosure cause crash when logging is enabled
lmesnik
parents:
51332
diff
changeset
|
375 |
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
|
376 |
outputStream* _st; |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
377 |
public: |
51475
492b366f8e57
8209758: 2 classes with same name G1PrintCollectionSetClosure cause crash when logging is enabled
lmesnik
parents:
51332
diff
changeset
|
378 |
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
|
379 |
|
48969 | 380 |
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
|
381 |
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
|
382 |
_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
|
383 |
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
|
384 |
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
|
385 |
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
|
386 |
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
|
387 |
return false; |
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 |
}; |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
390 |
|
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
391 |
void G1CollectionSet::print(outputStream* st) { |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
392 |
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
|
393 |
|
51475
492b366f8e57
8209758: 2 classes with same name G1PrintCollectionSetClosure cause crash when logging is enabled
lmesnik
parents:
51332
diff
changeset
|
394 |
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
|
395 |
iterate(&cl); |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
396 |
} |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
397 |
#endif // !PRODUCT |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
398 |
|
38183
cb68e4923223
8150721: Don't explicitly manage G1 young regions in YoungList
mgerdin
parents:
38162
diff
changeset
|
399 |
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
|
400 |
double young_start_time_sec = os::elapsedTime(); |
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 |
finalize_incremental_building(); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
403 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
404 |
guarantee(target_pause_time_ms > 0.0, |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
405 |
"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
|
406 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
407 |
size_t pending_cards = _policy->pending_cards(); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
408 |
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
|
409 |
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
|
410 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
411 |
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
|
412 |
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
|
413 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
414 |
// 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
|
415 |
// 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
|
416 |
// [Newly Young Regions ++ Survivors from last pause]. |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
417 |
|
38183
cb68e4923223
8150721: Don't explicitly manage G1 young regions in YoungList
mgerdin
parents:
38162
diff
changeset
|
418 |
uint survivor_region_length = survivors->length(); |
49806 | 419 |
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
|
420 |
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
|
421 |
|
38109
6503703df058
8155209: Move setting of young index in cset to G1CollectionSet
mgerdin
parents:
37985
diff
changeset
|
422 |
verify_young_cset_indices(); |
6503703df058
8155209: Move setting of young index in cset to G1CollectionSet
mgerdin
parents:
37985
diff
changeset
|
423 |
|
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
424 |
// 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
|
425 |
survivors->convert_to_eden(); |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
426 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
427 |
_bytes_used_before = _inc_bytes_used_before; |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
428 |
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
|
429 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
430 |
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
|
431 |
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
|
432 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
433 |
// 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
|
434 |
// collection set's current size |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
435 |
set_recorded_rs_lengths(_inc_recorded_rs_lengths); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
436 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
437 |
double young_end_time_sec = os::elapsedTime(); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
438 |
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
|
439 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
440 |
return time_remaining_ms; |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
441 |
} |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
442 |
|
52897
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
443 |
void G1CollectionSet::add_as_old(HeapRegion* hr) { |
53703
24341625d8f2
8217330: Split G1CollectionSetChooser into collection set candidate container and the chooser algorithm
tschatzl
parents:
52897
diff
changeset
|
444 |
candidates()->pop_front(); // already have region via peek() |
52897
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
445 |
_g1h->old_set_remove(hr); |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
446 |
add_old_region(hr); |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
447 |
} |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
448 |
|
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
449 |
void G1CollectionSet::add_as_optional(HeapRegion* hr) { |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
450 |
assert(_optional_regions != NULL, "Must not be called before array is allocated"); |
53703
24341625d8f2
8217330: Split G1CollectionSetChooser into collection set candidate container and the chooser algorithm
tschatzl
parents:
52897
diff
changeset
|
451 |
candidates()->pop_front(); // already have region via peek() |
52897
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
452 |
_g1h->old_set_remove(hr); |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
453 |
add_optional_region(hr); |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
454 |
} |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
455 |
|
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
456 |
bool G1CollectionSet::optional_is_full() { |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
457 |
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
|
458 |
return _optional_region_length == _optional_region_max_length; |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
459 |
} |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
460 |
|
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
461 |
void G1CollectionSet::clear_optional_region(const HeapRegion* hr) { |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
462 |
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
|
463 |
uint index = hr->index_in_opt_cset(); |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
464 |
_optional_regions[index] = NULL; |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
465 |
} |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
466 |
|
41075
d60e54a9bb67
8165313: Inserting freed regions during Free Collection Set serial phase takes very long on huge heaps
tschatzl
parents:
39698
diff
changeset
|
467 |
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
|
468 |
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 1; |
d60e54a9bb67
8165313: Inserting freed regions during Free Collection Set serial phase takes very long on huge heaps
tschatzl
parents:
39698
diff
changeset
|
470 |
} 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
|
471 |
return 0; |
d60e54a9bb67
8165313: Inserting freed regions during Free Collection Set serial phase takes very long on huge heaps
tschatzl
parents:
39698
diff
changeset
|
472 |
} else { |
d60e54a9bb67
8165313: Inserting freed regions during Free Collection Set serial phase takes very long on huge heaps
tschatzl
parents:
39698
diff
changeset
|
473 |
return -1; |
d60e54a9bb67
8165313: Inserting freed regions during Free Collection Set serial phase takes very long on huge heaps
tschatzl
parents:
39698
diff
changeset
|
474 |
} |
d60e54a9bb67
8165313: Inserting freed regions during Free Collection Set serial phase takes very long on huge heaps
tschatzl
parents:
39698
diff
changeset
|
475 |
} |
d60e54a9bb67
8165313: Inserting freed regions during Free Collection Set serial phase takes very long on huge heaps
tschatzl
parents:
39698
diff
changeset
|
476 |
|
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
477 |
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
|
478 |
double non_young_start_time_sec = os::elapsedTime(); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
479 |
double predicted_old_time_ms = 0.0; |
52897
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
480 |
double predicted_optional_time_ms = 0.0; |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
481 |
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
|
482 |
uint expensive_region_num = 0; |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
483 |
|
49643
a3453bbd5418
8199742: Clean up state flags in G1CollectorState
tschatzl
parents:
49333
diff
changeset
|
484 |
if (collector_state()->in_mixed_phase()) { |
53703
24341625d8f2
8217330: Split G1CollectionSetChooser into collection set candidate container and the chooser algorithm
tschatzl
parents:
52897
diff
changeset
|
485 |
candidates()->verify(); |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
486 |
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
|
487 |
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
|
488 |
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
|
489 |
|
52897
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
490 |
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
|
491 |
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
|
492 |
"time remaining %1.2fms, optional threshold %1.2fms", |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
493 |
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
|
494 |
|
53703
24341625d8f2
8217330: Split G1CollectionSetChooser into collection set candidate container and the chooser algorithm
tschatzl
parents:
52897
diff
changeset
|
495 |
HeapRegion* hr = candidates()->peek_front(); |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
496 |
while (hr != NULL) { |
52897
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
497 |
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
|
498 |
// 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
|
499 |
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
|
500 |
"old %u regions, optional %u regions", |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
501 |
old_region_length(), optional_region_length()); |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
502 |
break; |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
503 |
} |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
504 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
505 |
// 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
|
506 |
// not above G1HeapWastePercent. |
53703
24341625d8f2
8217330: Split G1CollectionSetChooser into collection set candidate container and the chooser algorithm
tschatzl
parents:
52897
diff
changeset
|
507 |
size_t reclaimable_bytes = candidates()->remaining_reclaimable_bytes(); |
47681
149745044e48
8189729: Change _perc suffixes in identifiers to _percent
tschatzl
parents:
47216
diff
changeset
|
508 |
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
|
509 |
double threshold = (double) G1HeapWastePercent; |
47681
149745044e48
8189729: Change _perc suffixes in identifiers to _percent
tschatzl
parents:
47216
diff
changeset
|
510 |
if (reclaimable_percent <= threshold) { |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
511 |
// 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
|
512 |
// 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
|
513 |
// adding old regions to the CSet. |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
514 |
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
|
515 |
"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
|
516 |
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
|
517 |
reclaimable_percent, G1HeapWastePercent); |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
518 |
break; |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
519 |
} |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
520 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
521 |
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
|
522 |
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
|
523 |
// 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
|
524 |
if (old_region_length() < min_old_cset_length) { |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
525 |
predicted_old_time_ms += predicted_time_ms; |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
526 |
add_as_old(hr); |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
527 |
// 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
|
528 |
if (time_remaining_ms == 0.0) { |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
529 |
expensive_region_num++; |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
530 |
} |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
531 |
} else { |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
532 |
// 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
|
533 |
// to the CSet if we reach the minimum. |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
534 |
if (!check_time_remaining) { |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
535 |
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
|
536 |
break; |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
537 |
} |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
538 |
// 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
|
539 |
if (time_remaining_ms > optional_threshold_ms) { |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
540 |
predicted_old_time_ms += predicted_time_ms; |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
541 |
add_as_old(hr); |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
542 |
} else if (time_remaining_ms > 0) { |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
543 |
// Keep adding optional regions until time is up |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
544 |
if (!optional_is_full()) { |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
545 |
predicted_optional_time_ms += predicted_time_ms; |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
546 |
add_as_optional(hr); |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
547 |
} else { |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
548 |
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
|
549 |
break; |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
550 |
} |
52897
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
551 |
} else { |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
552 |
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
|
553 |
break; |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
554 |
} |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
555 |
} |
53703
24341625d8f2
8217330: Split G1CollectionSetChooser into collection set candidate container and the chooser algorithm
tschatzl
parents:
52897
diff
changeset
|
556 |
hr = candidates()->peek_front(); |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
557 |
} |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
558 |
if (hr == NULL) { |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
559 |
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
|
560 |
} |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
561 |
|
53703
24341625d8f2
8217330: Split G1CollectionSetChooser into collection set candidate container and the chooser algorithm
tschatzl
parents:
52897
diff
changeset
|
562 |
candidates()->verify(); |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
563 |
} |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
564 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
565 |
stop_incremental_building(); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
566 |
|
52897
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
567 |
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
|
568 |
"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
|
569 |
old_region_length(), optional_region_length(), |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
570 |
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
|
571 |
if (expensive_region_num > 0) { |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
572 |
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
|
573 |
expensive_region_num); |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
574 |
} |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
575 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
576 |
double non_young_end_time_sec = os::elapsedTime(); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
577 |
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
|
578 |
|
46768 | 579 |
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
|
580 |
} |
38109
6503703df058
8155209: Move setting of young index in cset to G1CollectionSet
mgerdin
parents:
37985
diff
changeset
|
581 |
|
52897
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
582 |
HeapRegion* G1OptionalCSet::region_at(uint index) { |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
583 |
return _cset->optional_region_at(index); |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
584 |
} |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
585 |
|
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
586 |
void G1OptionalCSet::prepare_evacuation(double time_limit) { |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
587 |
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
|
588 |
|
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
589 |
uint prepared_regions = 0; |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
590 |
double prediction_ms = 0; |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
591 |
|
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
592 |
_prepare_failed = true; |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
593 |
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
|
594 |
HeapRegion* hr = region_at(i); |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
595 |
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
|
596 |
if (prediction_ms > time_limit) { |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
597 |
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
|
598 |
return; |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
599 |
} |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
600 |
|
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
601 |
// 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
|
602 |
prepare_to_evacuate_optional_region(hr); |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
603 |
prepared_regions++; |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
604 |
_current_limit++; |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
605 |
_prepare_failed = false; |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
606 |
} |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
607 |
|
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
608 |
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
|
609 |
prepared_regions, prediction_ms); |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
610 |
} |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
611 |
|
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
612 |
bool G1OptionalCSet::prepare_failed() { |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
613 |
return _prepare_failed; |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
614 |
} |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
615 |
|
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
616 |
void G1OptionalCSet::complete_evacuation() { |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
617 |
_evacuation_failed = false; |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
618 |
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
|
619 |
HeapRegion* hr = region_at(i); |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
620 |
_cset->clear_optional_region(hr); |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
621 |
if (hr->evacuation_failed()){ |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
622 |
_evacuation_failed = true; |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
623 |
} |
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 |
_current_index = _current_limit; |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
626 |
} |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
627 |
|
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
628 |
bool G1OptionalCSet::evacuation_failed() { |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
629 |
return _evacuation_failed; |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
630 |
} |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
631 |
|
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
632 |
G1OptionalCSet::~G1OptionalCSet() { |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
633 |
G1CollectedHeap* g1h = G1CollectedHeap::heap(); |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
634 |
while (!is_empty()) { |
53703
24341625d8f2
8217330: Split G1CollectionSetChooser into collection set candidate container and the chooser algorithm
tschatzl
parents:
52897
diff
changeset
|
635 |
// We want to return regions not evacuated to the collection set candidates |
24341625d8f2
8217330: Split G1CollectionSetChooser into collection set candidate container and the chooser algorithm
tschatzl
parents:
52897
diff
changeset
|
636 |
// in reverse order to maintain the old order. |
52897
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
637 |
HeapRegion* hr = _cset->remove_last_optional_region(); |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
638 |
assert(hr != NULL, "Should be valid region left"); |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
639 |
_pset->record_unused_optional_region(hr); |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
640 |
g1h->old_set_add(hr); |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
641 |
g1h->clear_in_cset(hr); |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
642 |
hr->set_index_in_opt_cset(InvalidCSetIndex); |
53703
24341625d8f2
8217330: Split G1CollectionSetChooser into collection set candidate container and the chooser algorithm
tschatzl
parents:
52897
diff
changeset
|
643 |
_cset->candidates()->push_front(hr); |
52897
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 |
_cset->free_optional_regions(); |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
646 |
} |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
647 |
|
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
648 |
uint G1OptionalCSet::size() { |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
649 |
return _cset->optional_region_length() - _current_index; |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
650 |
} |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
651 |
|
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
652 |
bool G1OptionalCSet::is_empty() { |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
653 |
return size() == 0; |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
654 |
} |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
655 |
|
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
656 |
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
|
657 |
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
|
658 |
G1CollectedHeap::heap()->clear_in_cset(hr); |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
659 |
_cset->add_old_region(hr); |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
660 |
} |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
661 |
|
38109
6503703df058
8155209: Move setting of young index in cset to G1CollectionSet
mgerdin
parents:
37985
diff
changeset
|
662 |
#ifdef ASSERT |
39698
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
663 |
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
|
664 |
private: |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
665 |
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
|
666 |
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
|
667 |
public: |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
668 |
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
|
669 |
_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
|
670 |
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
|
671 |
_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
|
672 |
} |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
673 |
} |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
674 |
~G1VerifyYoungCSetIndicesClosure() { |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
675 |
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
|
676 |
} |
6503703df058
8155209: Move setting of young index in cset to G1CollectionSet
mgerdin
parents:
37985
diff
changeset
|
677 |
|
48969 | 678 |
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
|
679 |
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
|
680 |
|
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
681 |
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
|
682 |
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
|
683 |
|
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
684 |
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
|
685 |
"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
|
686 |
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
|
687 |
|
39698
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
688 |
_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
|
689 |
|
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
690 |
return false; |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
691 |
} |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
692 |
}; |
38109
6503703df058
8155209: Move setting of young index in cset to G1CollectionSet
mgerdin
parents:
37985
diff
changeset
|
693 |
|
39698
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
694 |
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
|
695 |
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
|
696 |
|
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
697 |
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
|
698 |
iterate(&cl); |
38109
6503703df058
8155209: Move setting of young index in cset to G1CollectionSet
mgerdin
parents:
37985
diff
changeset
|
699 |
} |
6503703df058
8155209: Move setting of young index in cset to G1CollectionSet
mgerdin
parents:
37985
diff
changeset
|
700 |
#endif |