author | tschatzl |
Fri, 29 Nov 2019 10:20:17 +0100 | |
changeset 59321 | 5775e4825e58 |
parent 59319 | 9ee940f1de90 |
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" |
59319
9ee940f1de90
8227739: Merge cost predictions for scanning cards and log buffer entries
tschatzl
parents:
59290
diff
changeset
|
30 |
#include "gc/g1/g1HotCardCache.hpp" |
52897
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
31 |
#include "gc/g1/g1ParScanThreadState.hpp" |
37985
539c597ee0fa
8154154: Separate G1 specific policy code from the CollectorPolicy class hierarchy
mgerdin
parents:
37262
diff
changeset
|
32 |
#include "gc/g1/g1Policy.hpp" |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
33 |
#include "gc/g1/heapRegion.inline.hpp" |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
34 |
#include "gc/g1/heapRegionRemSet.hpp" |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
35 |
#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
|
36 |
#include "logging/logStream.hpp" |
59290
97d13893ec3c
8234748: Clean up atomic and orderAccess includes
stefank
parents:
58508
diff
changeset
|
37 |
#include "runtime/orderAccess.hpp" |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
38 |
#include "utilities/debug.hpp" |
52897
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
39 |
#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
|
40 |
#include "utilities/quickSort.hpp" |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
41 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
42 |
G1CollectorState* G1CollectionSet::collector_state() { |
49806 | 43 |
return _g1h->collector_state(); |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
44 |
} |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
45 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
46 |
G1GCPhaseTimes* G1CollectionSet::phase_times() { |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
47 |
return _policy->phase_times(); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
48 |
} |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
49 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
50 |
double G1CollectionSet::predict_region_elapsed_time_ms(HeapRegion* hr) { |
49643
a3453bbd5418
8199742: Clean up state flags in G1CollectorState
tschatzl
parents:
49333
diff
changeset
|
51 |
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
|
52 |
} |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
53 |
|
37985
539c597ee0fa
8154154: Separate G1 specific policy code from the CollectorPolicy class hierarchy
mgerdin
parents:
37262
diff
changeset
|
54 |
G1CollectionSet::G1CollectionSet(G1CollectedHeap* g1h, G1Policy* policy) : |
49806 | 55 |
_g1h(g1h), |
37985
539c597ee0fa
8154154: Separate G1 specific policy code from the CollectorPolicy class hierarchy
mgerdin
parents:
37262
diff
changeset
|
56 |
_policy(policy), |
53703
24341625d8f2
8217330: Split G1CollectionSetChooser into collection set candidate container and the chooser algorithm
tschatzl
parents:
52897
diff
changeset
|
57 |
_candidates(NULL), |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
58 |
_eden_region_length(0), |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
59 |
_survivor_region_length(0), |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
60 |
_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
|
61 |
_collection_set_regions(NULL), |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
62 |
_collection_set_cur_length(0), |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
63 |
_collection_set_max_length(0), |
54465
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
64 |
_num_optional_regions(0), |
51332 | 65 |
_bytes_used_before(0), |
57663
bf8e76d86d05
8228503: Rename "rs_lengths" to "rs_length" in ergo code
tschatzl
parents:
55510
diff
changeset
|
66 |
_recorded_rs_length(0), |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
67 |
_inc_build_state(Inactive), |
54465
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
68 |
_inc_part_start(0), |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
69 |
_inc_bytes_used_before(0), |
57663
bf8e76d86d05
8228503: Rename "rs_lengths" to "rs_length" in ergo code
tschatzl
parents:
55510
diff
changeset
|
70 |
_inc_recorded_rs_length(0), |
bf8e76d86d05
8228503: Rename "rs_lengths" to "rs_length" in ergo code
tschatzl
parents:
55510
diff
changeset
|
71 |
_inc_recorded_rs_length_diff(0), |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
72 |
_inc_predicted_elapsed_time_ms(0.0), |
57663
bf8e76d86d05
8228503: Rename "rs_lengths" to "rs_length" in ergo code
tschatzl
parents:
55510
diff
changeset
|
73 |
_inc_predicted_elapsed_time_ms_diff(0.0) { |
39698
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
74 |
} |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
75 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
76 |
G1CollectionSet::~G1CollectionSet() { |
58084
cddef3bde924
8230398: Remove NULL checks before FREE_C_HEAP_ARRAY
lkorinth
parents:
57802
diff
changeset
|
77 |
FREE_C_HEAP_ARRAY(uint, _collection_set_regions); |
52897
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
78 |
free_optional_regions(); |
53703
24341625d8f2
8217330: Split G1CollectionSetChooser into collection set candidate container and the chooser algorithm
tschatzl
parents:
52897
diff
changeset
|
79 |
clear_candidates(); |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
80 |
} |
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 |
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
|
83 |
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
|
84 |
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
|
85 |
|
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
86 |
_eden_region_length = eden_cset_region_length; |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
87 |
_survivor_region_length = survivor_cset_region_length; |
38109
6503703df058
8155209: Move setting of young index in cset to G1CollectionSet
mgerdin
parents:
37985
diff
changeset
|
88 |
|
39698
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
89 |
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
|
90 |
"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
|
91 |
|
54465
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
92 |
_old_region_length = 0; |
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
93 |
free_optional_regions(); |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
94 |
} |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
95 |
|
39698
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
96 |
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
|
97 |
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
|
98 |
_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
|
99 |
_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
|
100 |
} |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
101 |
|
52897
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
102 |
void G1CollectionSet::free_optional_regions() { |
54465
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
103 |
_num_optional_regions = 0; |
52897
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
104 |
} |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
105 |
|
53703
24341625d8f2
8217330: Split G1CollectionSetChooser into collection set candidate container and the chooser algorithm
tschatzl
parents:
52897
diff
changeset
|
106 |
void G1CollectionSet::clear_candidates() { |
24341625d8f2
8217330: Split G1CollectionSetChooser into collection set candidate container and the chooser algorithm
tschatzl
parents:
52897
diff
changeset
|
107 |
delete _candidates; |
24341625d8f2
8217330: Split G1CollectionSetChooser into collection set candidate container and the chooser algorithm
tschatzl
parents:
52897
diff
changeset
|
108 |
_candidates = NULL; |
24341625d8f2
8217330: Split G1CollectionSetChooser into collection set candidate container and the chooser algorithm
tschatzl
parents:
52897
diff
changeset
|
109 |
} |
24341625d8f2
8217330: Split G1CollectionSetChooser into collection set candidate container and the chooser algorithm
tschatzl
parents:
52897
diff
changeset
|
110 |
|
57663
bf8e76d86d05
8228503: Rename "rs_lengths" to "rs_length" in ergo code
tschatzl
parents:
55510
diff
changeset
|
111 |
void G1CollectionSet::set_recorded_rs_length(size_t rs_length) { |
bf8e76d86d05
8228503: Rename "rs_lengths" to "rs_length" in ergo code
tschatzl
parents:
55510
diff
changeset
|
112 |
_recorded_rs_length = rs_length; |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
113 |
} |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
114 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
115 |
// 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
|
116 |
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
|
117 |
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
|
118 |
|
54465
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
119 |
assert(_inc_build_state == Active, |
52897
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
120 |
"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
|
121 |
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
|
122 |
|
54465
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
123 |
assert(!hr->in_collection_set(), "should not already be in the collection set"); |
54843
25c329958c70
8200545: Improve filter for enqueued deferred cards
tschatzl
parents:
54465
diff
changeset
|
124 |
_g1h->register_old_region_with_region_attr(hr); |
39698
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
125 |
|
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
126 |
_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
|
127 |
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
|
128 |
|
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
129 |
_bytes_used_before += hr->used(); |
57663
bf8e76d86d05
8228503: Rename "rs_lengths" to "rs_length" in ergo code
tschatzl
parents:
55510
diff
changeset
|
130 |
_recorded_rs_length += hr->rem_set()->occupied(); |
54465
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
131 |
_old_region_length++; |
52897
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
132 |
|
54465
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
133 |
_g1h->old_set_remove(hr); |
52897
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
134 |
} |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
135 |
|
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
136 |
void G1CollectionSet::add_optional_region(HeapRegion* hr) { |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
137 |
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
|
138 |
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
|
139 |
|
54843
25c329958c70
8200545: Improve filter for enqueued deferred cards
tschatzl
parents:
54465
diff
changeset
|
140 |
_g1h->register_optional_region_with_region_attr(hr); |
52897
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
141 |
|
54465
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
142 |
hr->set_index_in_opt_cset(_num_optional_regions++); |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
143 |
} |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
144 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
145 |
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
|
146 |
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
|
147 |
assert(_inc_build_state == Inactive, "Precondition"); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
148 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
149 |
_inc_bytes_used_before = 0; |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
150 |
|
57663
bf8e76d86d05
8228503: Rename "rs_lengths" to "rs_length" in ergo code
tschatzl
parents:
55510
diff
changeset
|
151 |
_inc_recorded_rs_length = 0; |
bf8e76d86d05
8228503: Rename "rs_lengths" to "rs_length" in ergo code
tschatzl
parents:
55510
diff
changeset
|
152 |
_inc_recorded_rs_length_diff = 0; |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
153 |
_inc_predicted_elapsed_time_ms = 0.0; |
57663
bf8e76d86d05
8228503: Rename "rs_lengths" to "rs_length" in ergo code
tschatzl
parents:
55510
diff
changeset
|
154 |
_inc_predicted_elapsed_time_ms_diff = 0.0; |
54465
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
155 |
|
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
156 |
update_incremental_marker(); |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
157 |
} |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
158 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
159 |
void G1CollectionSet::finalize_incremental_building() { |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
160 |
assert(_inc_build_state == Active, "Precondition"); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
161 |
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
|
162 |
|
57663
bf8e76d86d05
8228503: Rename "rs_lengths" to "rs_length" in ergo code
tschatzl
parents:
55510
diff
changeset
|
163 |
// The two "main" fields, _inc_recorded_rs_length and |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
164 |
// _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
|
165 |
// 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
|
166 |
// concurrent refinement thread that samples the young RSet lengths |
57663
bf8e76d86d05
8228503: Rename "rs_lengths" to "rs_length" in ergo code
tschatzl
parents:
55510
diff
changeset
|
167 |
// are accumulated in the *_diff fields. Here we add the diffs to |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
168 |
// the "main" fields. |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
169 |
|
57663
bf8e76d86d05
8228503: Rename "rs_lengths" to "rs_length" in ergo code
tschatzl
parents:
55510
diff
changeset
|
170 |
if (_inc_recorded_rs_length_diff >= 0) { |
bf8e76d86d05
8228503: Rename "rs_lengths" to "rs_length" in ergo code
tschatzl
parents:
55510
diff
changeset
|
171 |
_inc_recorded_rs_length += _inc_recorded_rs_length_diff; |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
172 |
} else { |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
173 |
// 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
|
174 |
// 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
|
175 |
// 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
|
176 |
// 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
|
177 |
// could make the calculations somewhat inaccurate. |
57663
bf8e76d86d05
8228503: Rename "rs_lengths" to "rs_length" in ergo code
tschatzl
parents:
55510
diff
changeset
|
178 |
size_t diffs = (size_t) (-_inc_recorded_rs_length_diff); |
bf8e76d86d05
8228503: Rename "rs_lengths" to "rs_length" in ergo code
tschatzl
parents:
55510
diff
changeset
|
179 |
if (_inc_recorded_rs_length >= diffs) { |
bf8e76d86d05
8228503: Rename "rs_lengths" to "rs_length" in ergo code
tschatzl
parents:
55510
diff
changeset
|
180 |
_inc_recorded_rs_length -= diffs; |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
181 |
} else { |
57663
bf8e76d86d05
8228503: Rename "rs_lengths" to "rs_length" in ergo code
tschatzl
parents:
55510
diff
changeset
|
182 |
_inc_recorded_rs_length = 0; |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
183 |
} |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
184 |
} |
57663
bf8e76d86d05
8228503: Rename "rs_lengths" to "rs_length" in ergo code
tschatzl
parents:
55510
diff
changeset
|
185 |
_inc_predicted_elapsed_time_ms += _inc_predicted_elapsed_time_ms_diff; |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
186 |
|
57663
bf8e76d86d05
8228503: Rename "rs_lengths" to "rs_length" in ergo code
tschatzl
parents:
55510
diff
changeset
|
187 |
_inc_recorded_rs_length_diff = 0; |
bf8e76d86d05
8228503: Rename "rs_lengths" to "rs_length" in ergo code
tschatzl
parents:
55510
diff
changeset
|
188 |
_inc_predicted_elapsed_time_ms_diff = 0.0; |
37039
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 |
|
39698
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
191 |
void G1CollectionSet::clear() { |
49333
489f1dd40582
8196876: OopStorage::assert_at_safepoint clashes with assert_at_safepoint macros in g1CollectedHeap.hpp
kbarrett
parents:
48969
diff
changeset
|
192 |
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
|
193 |
_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
|
194 |
} |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
195 |
|
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
196 |
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
|
197 |
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
|
198 |
OrderAccess::loadload(); |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
199 |
|
54465
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
200 |
for (uint i = 0; i < len; i++) { |
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
201 |
HeapRegion* r = _g1h->region_at(_collection_set_regions[i]); |
48969 | 202 |
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
|
203 |
if (result) { |
48969 | 204 |
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
|
205 |
return; |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
206 |
} |
54465
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
207 |
} |
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
208 |
} |
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
209 |
|
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
210 |
void G1CollectionSet::iterate_optional(HeapRegionClosure* cl) const { |
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
211 |
assert_at_safepoint(); |
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
212 |
|
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
213 |
for (uint i = 0; i < _num_optional_regions; i++) { |
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
214 |
HeapRegion* r = _candidates->at(i); |
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
215 |
bool result = cl->do_heap_region(r); |
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
216 |
guarantee(!result, "Must not cancel iteration"); |
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
217 |
} |
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
218 |
} |
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
219 |
|
55510
3e31a8beaae4
8213108: Improve work distribution during remembered set scan
tschatzl
parents:
55377
diff
changeset
|
220 |
void G1CollectionSet::iterate_incremental_part_from(HeapRegionClosure* cl, |
3e31a8beaae4
8213108: Improve work distribution during remembered set scan
tschatzl
parents:
55377
diff
changeset
|
221 |
HeapRegionClaimer* hr_claimer, |
3e31a8beaae4
8213108: Improve work distribution during remembered set scan
tschatzl
parents:
55377
diff
changeset
|
222 |
uint worker_id, |
3e31a8beaae4
8213108: Improve work distribution during remembered set scan
tschatzl
parents:
55377
diff
changeset
|
223 |
uint total_workers) const { |
54465
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
224 |
assert_at_safepoint(); |
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
225 |
|
55510
3e31a8beaae4
8213108: Improve work distribution during remembered set scan
tschatzl
parents:
55377
diff
changeset
|
226 |
size_t len = increment_length(); |
54465
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
227 |
if (len == 0) { |
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
228 |
return; |
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
229 |
} |
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
230 |
|
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
231 |
size_t start_pos = (worker_id * len) / total_workers; |
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
232 |
size_t cur_pos = start_pos; |
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
233 |
|
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
234 |
do { |
55510
3e31a8beaae4
8213108: Improve work distribution during remembered set scan
tschatzl
parents:
55377
diff
changeset
|
235 |
uint region_idx = _collection_set_regions[cur_pos + _inc_part_start]; |
3e31a8beaae4
8213108: Improve work distribution during remembered set scan
tschatzl
parents:
55377
diff
changeset
|
236 |
if (hr_claimer == NULL || hr_claimer->claim_region(region_idx)) { |
3e31a8beaae4
8213108: Improve work distribution during remembered set scan
tschatzl
parents:
55377
diff
changeset
|
237 |
HeapRegion* r = _g1h->region_at(region_idx); |
3e31a8beaae4
8213108: Improve work distribution during remembered set scan
tschatzl
parents:
55377
diff
changeset
|
238 |
bool result = cl->do_heap_region(r); |
3e31a8beaae4
8213108: Improve work distribution during remembered set scan
tschatzl
parents:
55377
diff
changeset
|
239 |
guarantee(!result, "Must not cancel iteration"); |
3e31a8beaae4
8213108: Improve work distribution during remembered set scan
tschatzl
parents:
55377
diff
changeset
|
240 |
} |
54465
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
241 |
|
39698
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
242 |
cur_pos++; |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
243 |
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
|
244 |
cur_pos = 0; |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
245 |
} |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
246 |
} 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
|
247 |
} |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
248 |
|
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
249 |
void G1CollectionSet::update_young_region_prediction(HeapRegion* hr, |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
250 |
size_t new_rs_length) { |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
251 |
// 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
|
252 |
assert(hr->is_young(), "Precondition"); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
253 |
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
|
254 |
|
57663
bf8e76d86d05
8228503: Rename "rs_lengths" to "rs_length" in ergo code
tschatzl
parents:
55510
diff
changeset
|
255 |
// We could have updated _inc_recorded_rs_length and |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
256 |
// _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
|
257 |
// 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
|
258 |
// refinement thread, potentially concurrently with a mutator thread |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
259 |
// 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
|
260 |
// avoid the atomic operations we accumulate these updates on two |
57663
bf8e76d86d05
8228503: Rename "rs_lengths" to "rs_length" in ergo code
tschatzl
parents:
55510
diff
changeset
|
261 |
// separate fields (*_diff) and we'll just add them to the "main" |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
262 |
// fields at the start of a GC. |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
263 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
264 |
ssize_t old_rs_length = (ssize_t) hr->recorded_rs_length(); |
57663
bf8e76d86d05
8228503: Rename "rs_lengths" to "rs_length" in ergo code
tschatzl
parents:
55510
diff
changeset
|
265 |
ssize_t rs_length_diff = (ssize_t) new_rs_length - old_rs_length; |
bf8e76d86d05
8228503: Rename "rs_lengths" to "rs_length" in ergo code
tschatzl
parents:
55510
diff
changeset
|
266 |
_inc_recorded_rs_length_diff += rs_length_diff; |
37039
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 |
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
|
269 |
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
|
270 |
double elapsed_ms_diff = new_region_elapsed_time_ms - old_elapsed_time_ms; |
57663
bf8e76d86d05
8228503: Rename "rs_lengths" to "rs_length" in ergo code
tschatzl
parents:
55510
diff
changeset
|
271 |
_inc_predicted_elapsed_time_ms_diff += elapsed_ms_diff; |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
272 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
273 |
hr->set_recorded_rs_length(new_rs_length); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
274 |
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
|
275 |
} |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
276 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
277 |
void G1CollectionSet::add_young_region_common(HeapRegion* hr) { |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
278 |
assert(hr->is_young(), "invariant"); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
279 |
assert(_inc_build_state == Active, "Precondition"); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
280 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
281 |
// This routine is used when: |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
282 |
// * 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
|
283 |
// evacuation pause or |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
284 |
// * 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
|
285 |
// when it is retired. |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
286 |
// 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
|
287 |
// 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
|
288 |
// retiring the current allocation region) |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
289 |
// 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
|
290 |
// 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
|
291 |
// 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
|
292 |
// 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
|
293 |
// 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
|
294 |
// by the Young List sampling code. |
49643
a3453bbd5418
8199742: Clean up state flags in G1CollectorState
tschatzl
parents:
49333
diff
changeset
|
295 |
// 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
|
296 |
|
49806 | 297 |
if (!_g1h->collector_state()->in_full_gc()) { |
49643
a3453bbd5418
8199742: Clean up state flags in G1CollectorState
tschatzl
parents:
49333
diff
changeset
|
298 |
size_t rs_length = hr->rem_set()->occupied(); |
a3453bbd5418
8199742: Clean up state flags in G1CollectorState
tschatzl
parents:
49333
diff
changeset
|
299 |
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
|
300 |
|
49643
a3453bbd5418
8199742: Clean up state flags in G1CollectorState
tschatzl
parents:
49333
diff
changeset
|
301 |
// Cache the values we have added to the aggregated information |
a3453bbd5418
8199742: Clean up state flags in G1CollectorState
tschatzl
parents:
49333
diff
changeset
|
302 |
// 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
|
303 |
// the incremental collection set, or it is updated by the |
a3453bbd5418
8199742: Clean up state flags in G1CollectorState
tschatzl
parents:
49333
diff
changeset
|
304 |
// rset sampling code |
a3453bbd5418
8199742: Clean up state flags in G1CollectorState
tschatzl
parents:
49333
diff
changeset
|
305 |
hr->set_recorded_rs_length(rs_length); |
a3453bbd5418
8199742: Clean up state flags in G1CollectorState
tschatzl
parents:
49333
diff
changeset
|
306 |
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
|
307 |
|
57663
bf8e76d86d05
8228503: Rename "rs_lengths" to "rs_length" in ergo code
tschatzl
parents:
55510
diff
changeset
|
308 |
_inc_recorded_rs_length += rs_length; |
49643
a3453bbd5418
8199742: Clean up state flags in G1CollectorState
tschatzl
parents:
49333
diff
changeset
|
309 |
_inc_predicted_elapsed_time_ms += region_elapsed_time_ms; |
a3453bbd5418
8199742: Clean up state flags in G1CollectorState
tschatzl
parents:
49333
diff
changeset
|
310 |
_inc_bytes_used_before += hr->used(); |
a3453bbd5418
8199742: Clean up state flags in G1CollectorState
tschatzl
parents:
49333
diff
changeset
|
311 |
} |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
312 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
313 |
assert(!hr->in_collection_set(), "invariant"); |
54843
25c329958c70
8200545: Improve filter for enqueued deferred cards
tschatzl
parents:
54465
diff
changeset
|
314 |
_g1h->register_young_region_with_region_attr(hr); |
59321
5775e4825e58
8233998: New young regions registered too early in collection set
tschatzl
parents:
59319
diff
changeset
|
315 |
|
5775e4825e58
8233998: New young regions registered too early in collection set
tschatzl
parents:
59319
diff
changeset
|
316 |
size_t collection_set_length = _collection_set_cur_length; |
5775e4825e58
8233998: New young regions registered too early in collection set
tschatzl
parents:
59319
diff
changeset
|
317 |
// We use UINT_MAX as "invalid" marker in verification. |
5775e4825e58
8233998: New young regions registered too early in collection set
tschatzl
parents:
59319
diff
changeset
|
318 |
assert(collection_set_length < (UINT_MAX - 1), |
5775e4825e58
8233998: New young regions registered too early in collection set
tschatzl
parents:
59319
diff
changeset
|
319 |
"Collection set is too large with " SIZE_FORMAT " entries", collection_set_length); |
5775e4825e58
8233998: New young regions registered too early in collection set
tschatzl
parents:
59319
diff
changeset
|
320 |
hr->set_young_index_in_cset((uint)collection_set_length + 1); |
5775e4825e58
8233998: New young regions registered too early in collection set
tschatzl
parents:
59319
diff
changeset
|
321 |
|
5775e4825e58
8233998: New young regions registered too early in collection set
tschatzl
parents:
59319
diff
changeset
|
322 |
_collection_set_regions[collection_set_length] = hr->hrm_index(); |
5775e4825e58
8233998: New young regions registered too early in collection set
tschatzl
parents:
59319
diff
changeset
|
323 |
// Concurrent readers must observe the store of the value in the array before an |
5775e4825e58
8233998: New young regions registered too early in collection set
tschatzl
parents:
59319
diff
changeset
|
324 |
// update to the length field. |
5775e4825e58
8233998: New young regions registered too early in collection set
tschatzl
parents:
59319
diff
changeset
|
325 |
OrderAccess::storestore(); |
5775e4825e58
8233998: New young regions registered too early in collection set
tschatzl
parents:
59319
diff
changeset
|
326 |
_collection_set_cur_length++; |
5775e4825e58
8233998: New young regions registered too early in collection set
tschatzl
parents:
59319
diff
changeset
|
327 |
assert(_collection_set_cur_length <= _collection_set_max_length, "Collection set larger than maximum allowed."); |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
328 |
} |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
329 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
330 |
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
|
331 |
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
|
332 |
add_young_region_common(hr); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
333 |
} |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
334 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
335 |
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
|
336 |
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
|
337 |
add_young_region_common(hr); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
338 |
} |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
339 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
340 |
#ifndef PRODUCT |
39698
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
341 |
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
|
342 |
public: |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
343 |
bool _valid; |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
344 |
public: |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
345 |
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
|
346 |
|
48969 | 347 |
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
|
348 |
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
|
349 |
|
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
350 |
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
|
351 |
|
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
352 |
if (group == NULL) { |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
353 |
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
|
354 |
_valid = false; |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
355 |
} |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
356 |
|
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
357 |
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
|
358 |
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
|
359 |
_valid = false; |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
360 |
} |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
361 |
|
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
362 |
return false; |
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 |
|
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
365 |
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
|
366 |
}; |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
367 |
|
39698
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
368 |
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
|
369 |
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
|
370 |
|
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
371 |
G1VerifyYoungAgesClosure cl; |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
372 |
iterate(&cl); |
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 |
if (!cl.valid()) { |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
375 |
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
|
376 |
print(&log); |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
377 |
} |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
378 |
|
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
379 |
return cl.valid(); |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
380 |
} |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
381 |
|
51475
492b366f8e57
8209758: 2 classes with same name G1PrintCollectionSetClosure cause crash when logging is enabled
lmesnik
parents:
51332
diff
changeset
|
382 |
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
|
383 |
outputStream* _st; |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
384 |
public: |
51475
492b366f8e57
8209758: 2 classes with same name G1PrintCollectionSetClosure cause crash when logging is enabled
lmesnik
parents:
51332
diff
changeset
|
385 |
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
|
386 |
|
48969 | 387 |
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
|
388 |
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
|
389 |
_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
|
390 |
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
|
391 |
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
|
392 |
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
|
393 |
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
|
394 |
return false; |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
395 |
} |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
396 |
}; |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
397 |
|
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
398 |
void G1CollectionSet::print(outputStream* st) { |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
399 |
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
|
400 |
|
51475
492b366f8e57
8209758: 2 classes with same name G1PrintCollectionSetClosure cause crash when logging is enabled
lmesnik
parents:
51332
diff
changeset
|
401 |
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
|
402 |
iterate(&cl); |
37039
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 |
#endif // !PRODUCT |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
405 |
|
38183
cb68e4923223
8150721: Don't explicitly manage G1 young regions in YoungList
mgerdin
parents:
38162
diff
changeset
|
406 |
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
|
407 |
double young_start_time_sec = os::elapsedTime(); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
408 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
409 |
finalize_incremental_building(); |
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 |
guarantee(target_pause_time_ms > 0.0, |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
412 |
"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
|
413 |
|
59319
9ee940f1de90
8227739: Merge cost predictions for scanning cards and log buffer entries
tschatzl
parents:
59290
diff
changeset
|
414 |
size_t pending_cards = _policy->pending_cards_at_gc_start() + _g1h->hot_card_cache()->num_entries(); |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
415 |
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
|
416 |
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
|
417 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
418 |
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
|
419 |
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
|
420 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
421 |
// 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
|
422 |
// 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
|
423 |
// [Newly Young Regions ++ Survivors from last pause]. |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
424 |
|
38183
cb68e4923223
8150721: Don't explicitly manage G1 young regions in YoungList
mgerdin
parents:
38162
diff
changeset
|
425 |
uint survivor_region_length = survivors->length(); |
49806 | 426 |
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
|
427 |
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
|
428 |
|
38109
6503703df058
8155209: Move setting of young index in cset to G1CollectionSet
mgerdin
parents:
37985
diff
changeset
|
429 |
verify_young_cset_indices(); |
6503703df058
8155209: Move setting of young index in cset to G1CollectionSet
mgerdin
parents:
37985
diff
changeset
|
430 |
|
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
431 |
// 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
|
432 |
survivors->convert_to_eden(); |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
433 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
434 |
_bytes_used_before = _inc_bytes_used_before; |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
435 |
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
|
436 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
437 |
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
|
438 |
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
|
439 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
440 |
// 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
|
441 |
// collection set's current size |
57663
bf8e76d86d05
8228503: Rename "rs_lengths" to "rs_length" in ergo code
tschatzl
parents:
55510
diff
changeset
|
442 |
set_recorded_rs_length(_inc_recorded_rs_length); |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
443 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
444 |
double young_end_time_sec = os::elapsedTime(); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
445 |
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
|
446 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
447 |
return time_remaining_ms; |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
448 |
} |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
449 |
|
41075
d60e54a9bb67
8165313: Inserting freed regions during Free Collection Set serial phase takes very long on huge heaps
tschatzl
parents:
39698
diff
changeset
|
450 |
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
|
451 |
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
|
452 |
return 1; |
d60e54a9bb67
8165313: Inserting freed regions during Free Collection Set serial phase takes very long on huge heaps
tschatzl
parents:
39698
diff
changeset
|
453 |
} 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
|
454 |
return 0; |
d60e54a9bb67
8165313: Inserting freed regions during Free Collection Set serial phase takes very long on huge heaps
tschatzl
parents:
39698
diff
changeset
|
455 |
} else { |
d60e54a9bb67
8165313: Inserting freed regions during Free Collection Set serial phase takes very long on huge heaps
tschatzl
parents:
39698
diff
changeset
|
456 |
return -1; |
d60e54a9bb67
8165313: Inserting freed regions during Free Collection Set serial phase takes very long on huge heaps
tschatzl
parents:
39698
diff
changeset
|
457 |
} |
d60e54a9bb67
8165313: Inserting freed regions during Free Collection Set serial phase takes very long on huge heaps
tschatzl
parents:
39698
diff
changeset
|
458 |
} |
d60e54a9bb67
8165313: Inserting freed regions during Free Collection Set serial phase takes very long on huge heaps
tschatzl
parents:
39698
diff
changeset
|
459 |
|
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
460 |
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
|
461 |
double non_young_start_time_sec = os::elapsedTime(); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
462 |
|
49643
a3453bbd5418
8199742: Clean up state flags in G1CollectorState
tschatzl
parents:
49333
diff
changeset
|
463 |
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
|
464 |
candidates()->verify(); |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
465 |
|
54465
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
466 |
uint num_initial_old_regions; |
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
467 |
uint num_optional_old_regions; |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
468 |
|
54465
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
469 |
_policy->calculate_old_collection_set_regions(candidates(), |
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
470 |
time_remaining_ms, |
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
471 |
num_initial_old_regions, |
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
472 |
num_optional_old_regions); |
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
473 |
|
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
474 |
// Prepare initial old regions. |
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
475 |
move_candidates_to_collection_set(num_initial_old_regions); |
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
476 |
|
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
477 |
// Prepare optional old regions for evacuation. |
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
478 |
uint candidate_idx = candidates()->cur_idx(); |
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
479 |
for (uint i = 0; i < num_optional_old_regions; i++) { |
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
480 |
add_optional_region(candidates()->at(candidate_idx + i)); |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
481 |
} |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
482 |
|
53703
24341625d8f2
8217330: Split G1CollectionSetChooser into collection set candidate container and the chooser algorithm
tschatzl
parents:
52897
diff
changeset
|
483 |
candidates()->verify(); |
37039
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
484 |
} |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
485 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
486 |
stop_incremental_building(); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
487 |
|
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
488 |
double non_young_end_time_sec = os::elapsedTime(); |
79f62b89a7a6
8151178: Move the collection set out of the G1 collector policy
mgerdin
parents:
diff
changeset
|
489 |
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
|
490 |
|
46768 | 491 |
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
|
492 |
} |
38109
6503703df058
8155209: Move setting of young index in cset to G1CollectionSet
mgerdin
parents:
37985
diff
changeset
|
493 |
|
54465
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
494 |
void G1CollectionSet::move_candidates_to_collection_set(uint num_old_candidate_regions) { |
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
495 |
if (num_old_candidate_regions == 0) { |
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
496 |
return; |
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
497 |
} |
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
498 |
uint candidate_idx = candidates()->cur_idx(); |
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
499 |
for (uint i = 0; i < num_old_candidate_regions; i++) { |
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
500 |
HeapRegion* r = candidates()->at(candidate_idx + i); |
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
501 |
// This potentially optional candidate region is going to be an actual collection |
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
502 |
// set region. Clear cset marker. |
54843
25c329958c70
8200545: Improve filter for enqueued deferred cards
tschatzl
parents:
54465
diff
changeset
|
503 |
_g1h->clear_region_attr(r); |
54465
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
504 |
add_old_region(r); |
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
505 |
} |
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
506 |
candidates()->remove(num_old_candidate_regions); |
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
507 |
|
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
508 |
candidates()->verify(); |
52897
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
509 |
} |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
510 |
|
54465
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
511 |
void G1CollectionSet::finalize_initial_collection_set(double target_pause_time_ms, G1SurvivorRegions* survivor) { |
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
512 |
double time_remaining_ms = finalize_young_part(target_pause_time_ms, survivor); |
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
513 |
finalize_old_part(time_remaining_ms); |
52897
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
514 |
} |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
515 |
|
54465
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
516 |
bool G1CollectionSet::finalize_optional_for_evacuation(double remaining_pause_time) { |
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
517 |
update_incremental_marker(); |
52897
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
518 |
|
54465
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
519 |
uint num_selected_regions; |
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
520 |
_policy->calculate_optional_collection_set_regions(candidates(), |
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
521 |
_num_optional_regions, |
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
522 |
remaining_pause_time, |
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
523 |
num_selected_regions); |
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
524 |
|
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
525 |
move_candidates_to_collection_set(num_selected_regions); |
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
526 |
|
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
527 |
_num_optional_regions -= num_selected_regions; |
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
528 |
|
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
529 |
stop_incremental_building(); |
55377
f48d3bec75ba
8225418: G1: region attribute remembered set state disagrees with actual remembered set state
tschatzl
parents:
54843
diff
changeset
|
530 |
|
f48d3bec75ba
8225418: G1: region attribute remembered set state disagrees with actual remembered set state
tschatzl
parents:
54843
diff
changeset
|
531 |
_g1h->verify_region_attr_remset_update(); |
f48d3bec75ba
8225418: G1: region attribute remembered set state disagrees with actual remembered set state
tschatzl
parents:
54843
diff
changeset
|
532 |
|
54465
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
533 |
return num_selected_regions > 0; |
52897
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
534 |
} |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
535 |
|
54465
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
536 |
void G1CollectionSet::abandon_optional_collection_set(G1ParScanThreadStateSet* pss) { |
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
537 |
for (uint i = 0; i < _num_optional_regions; i++) { |
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
538 |
HeapRegion* r = candidates()->at(candidates()->cur_idx() + i); |
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
539 |
pss->record_unused_optional_region(r); |
55377
f48d3bec75ba
8225418: G1: region attribute remembered set state disagrees with actual remembered set state
tschatzl
parents:
54843
diff
changeset
|
540 |
// Clear collection set marker and make sure that the remembered set information |
f48d3bec75ba
8225418: G1: region attribute remembered set state disagrees with actual remembered set state
tschatzl
parents:
54843
diff
changeset
|
541 |
// is correct as we still need it later. |
54843
25c329958c70
8200545: Improve filter for enqueued deferred cards
tschatzl
parents:
54465
diff
changeset
|
542 |
_g1h->clear_region_attr(r); |
55377
f48d3bec75ba
8225418: G1: region attribute remembered set state disagrees with actual remembered set state
tschatzl
parents:
54843
diff
changeset
|
543 |
_g1h->register_region_with_region_attr(r); |
54465
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
544 |
r->clear_index_in_opt_cset(); |
52897
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
545 |
} |
54465
c4f16445675a
8218668: Clean up evacuation of optional collection set
tschatzl
parents:
53703
diff
changeset
|
546 |
free_optional_regions(); |
55377
f48d3bec75ba
8225418: G1: region attribute remembered set state disagrees with actual remembered set state
tschatzl
parents:
54843
diff
changeset
|
547 |
|
f48d3bec75ba
8225418: G1: region attribute remembered set state disagrees with actual remembered set state
tschatzl
parents:
54843
diff
changeset
|
548 |
_g1h->verify_region_attr_remset_update(); |
52897
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
549 |
} |
495c05ee2a9a
8213890: Implementation of JEP 344: Abortable Mixed Collections for G1
sjohanss
parents:
51475
diff
changeset
|
550 |
|
38109
6503703df058
8155209: Move setting of young index in cset to G1CollectionSet
mgerdin
parents:
37985
diff
changeset
|
551 |
#ifdef ASSERT |
39698
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
552 |
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
|
553 |
private: |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
554 |
size_t _young_length; |
57802 | 555 |
uint* _heap_region_indices; |
39698
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
556 |
public: |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
557 |
G1VerifyYoungCSetIndicesClosure(size_t young_length) : HeapRegionClosure(), _young_length(young_length) { |
57802 | 558 |
_heap_region_indices = NEW_C_HEAP_ARRAY(uint, young_length + 1, mtGC); |
559 |
for (size_t i = 0; i < young_length + 1; i++) { |
|
560 |
_heap_region_indices[i] = UINT_MAX; |
|
39698
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
561 |
} |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
562 |
} |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
563 |
~G1VerifyYoungCSetIndicesClosure() { |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
564 |
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
|
565 |
} |
6503703df058
8155209: Move setting of young index in cset to G1CollectionSet
mgerdin
parents:
37985
diff
changeset
|
566 |
|
48969 | 567 |
virtual bool do_heap_region(HeapRegion* r) { |
57802 | 568 |
const uint idx = r->young_index_in_cset(); |
39698
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
569 |
|
57802 | 570 |
assert(idx > 0, "Young index must be set for all regions in the incremental collection set but is not for region %u.", r->hrm_index()); |
571 |
assert(idx <= _young_length, "Young cset index %u too large for region %u", idx, r->hrm_index()); |
|
39698
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
572 |
|
57802 | 573 |
assert(_heap_region_indices[idx] == UINT_MAX, |
39698
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
574 |
"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
|
575 |
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
|
576 |
|
39698
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
577 |
_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
|
578 |
|
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
579 |
return false; |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
580 |
} |
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
581 |
}; |
38109
6503703df058
8155209: Move setting of young index in cset to G1CollectionSet
mgerdin
parents:
37985
diff
changeset
|
582 |
|
39698
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
583 |
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
|
584 |
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
|
585 |
|
4016de4e596b
8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents:
38183
diff
changeset
|
586 |
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
|
587 |
iterate(&cl); |
38109
6503703df058
8155209: Move setting of young index in cset to G1CollectionSet
mgerdin
parents:
37985
diff
changeset
|
588 |
} |
6503703df058
8155209: Move setting of young index in cset to G1CollectionSet
mgerdin
parents:
37985
diff
changeset
|
589 |
#endif |