author | rkennke |
Wed, 03 Oct 2018 15:22:16 +0200 | |
changeset 52070 | e4d72440d60e |
parent 51332 | c25572739e7c |
child 52925 | 9c18c9d839d3 |
permissions | -rw-r--r-- |
1 | 1 |
/* |
49041
44122f767467
8198286: Direct memory accessors in typeArrayOop.hpp should use Access API
eosterlund
parents:
47998
diff
changeset
|
2 |
* Copyright (c) 2001, 2018, Oracle and/or its affiliates. All rights reserved. |
1 | 3 |
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. |
4 |
* |
|
5 |
* This code is free software; you can redistribute it and/or modify it |
|
6 |
* under the terms of the GNU General Public License version 2 only, as |
|
7 |
* published by the Free Software Foundation. |
|
8 |
* |
|
9 |
* This code is distributed in the hope that it will be useful, but WITHOUT |
|
10 |
* ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or |
|
11 |
* FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License |
|
12 |
* version 2 for more details (a copy is included in the LICENSE file that |
|
13 |
* accompanied this code). |
|
14 |
* |
|
15 |
* You should have received a copy of the GNU General Public License version |
|
16 |
* 2 along with this work; if not, write to the Free Software Foundation, |
|
17 |
* Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA. |
|
18 |
* |
|
5547
f4b087cbb361
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
4738
diff
changeset
|
19 |
* Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA |
f4b087cbb361
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
4738
diff
changeset
|
20 |
* or visit www.oracle.com if you need additional information or have any |
f4b087cbb361
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
4738
diff
changeset
|
21 |
* questions. |
1 | 22 |
* |
23 |
*/ |
|
24 |
||
7397 | 25 |
#include "precompiled.hpp" |
35498
392b50de06c6
8146401: Clean up oop.hpp: add inline directives and fix header files
goetz
parents:
35475
diff
changeset
|
26 |
#include "classfile/javaClasses.inline.hpp" |
7397 | 27 |
#include "classfile/systemDictionary.hpp" |
30764 | 28 |
#include "gc/shared/collectedHeap.hpp" |
29 |
#include "gc/shared/collectedHeap.inline.hpp" |
|
30 |
#include "gc/shared/gcTimer.hpp" |
|
35061 | 31 |
#include "gc/shared/gcTraceTime.inline.hpp" |
30764 | 32 |
#include "gc/shared/referencePolicy.hpp" |
35862
411842d0c882
8146395: Add inline qualifier in oop.hpp and fix inlining in gc files
goetz
parents:
35498
diff
changeset
|
33 |
#include "gc/shared/referenceProcessor.inline.hpp" |
50605
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
34 |
#include "gc/shared/referenceProcessorPhaseTimes.hpp" |
35061 | 35 |
#include "logging/log.hpp" |
49360 | 36 |
#include "memory/allocation.inline.hpp" |
37248 | 37 |
#include "memory/resourceArea.hpp" |
47998
fb0275c320a0
8189871: Refactor GC barriers to use declarative semantics
eosterlund
parents:
47648
diff
changeset
|
38 |
#include "oops/access.inline.hpp" |
7397 | 39 |
#include "oops/oop.inline.hpp" |
40 |
#include "runtime/java.hpp" |
|
1 | 41 |
|
1606
dcf9714addbe
6684579: SoftReference processing can be made more efficient
ysr
parents:
1605
diff
changeset
|
42 |
ReferencePolicy* ReferenceProcessor::_always_clear_soft_ref_policy = NULL; |
dcf9714addbe
6684579: SoftReference processing can be made more efficient
ysr
parents:
1605
diff
changeset
|
43 |
ReferencePolicy* ReferenceProcessor::_default_soft_ref_policy = NULL; |
10683
4b5a5a507864
7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents:
10670
diff
changeset
|
44 |
jlong ReferenceProcessor::_soft_ref_timestamp_clock = 0; |
1606
dcf9714addbe
6684579: SoftReference processing can be made more efficient
ysr
parents:
1605
diff
changeset
|
45 |
|
1 | 46 |
void referenceProcessor_init() { |
47 |
ReferenceProcessor::init_statics(); |
|
48 |
} |
|
49 |
||
50 |
void ReferenceProcessor::init_statics() { |
|
22551 | 51 |
// We need a monotonically non-decreasing time in ms but |
11251
e29da6b5622b
7117303: VM uses non-monotonic time source and complains that it is non-monotonic
johnc
parents:
10747
diff
changeset
|
52 |
// os::javaTimeMillis() does not guarantee monotonicity. |
e29da6b5622b
7117303: VM uses non-monotonic time source and complains that it is non-monotonic
johnc
parents:
10747
diff
changeset
|
53 |
jlong now = os::javaTimeNanos() / NANOSECS_PER_MILLISEC; |
10683
4b5a5a507864
7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents:
10670
diff
changeset
|
54 |
|
4b5a5a507864
7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents:
10670
diff
changeset
|
55 |
// Initialize the soft ref timestamp clock. |
4b5a5a507864
7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents:
10670
diff
changeset
|
56 |
_soft_ref_timestamp_clock = now; |
4b5a5a507864
7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents:
10670
diff
changeset
|
57 |
// Also update the soft ref clock in j.l.r.SoftReference |
4b5a5a507864
7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents:
10670
diff
changeset
|
58 |
java_lang_ref_SoftReference::set_clock(_soft_ref_timestamp_clock); |
1 | 59 |
|
1606
dcf9714addbe
6684579: SoftReference processing can be made more efficient
ysr
parents:
1605
diff
changeset
|
60 |
_always_clear_soft_ref_policy = new AlwaysClearPolicy(); |
43455
96560cffef4d
8166002: Emulate client build on platforms with reduced virtual address space
jcm
parents:
40892
diff
changeset
|
61 |
if (is_server_compilation_mode_vm()) { |
96560cffef4d
8166002: Emulate client build on platforms with reduced virtual address space
jcm
parents:
40892
diff
changeset
|
62 |
_default_soft_ref_policy = new LRUMaxHeapPolicy(); |
96560cffef4d
8166002: Emulate client build on platforms with reduced virtual address space
jcm
parents:
40892
diff
changeset
|
63 |
} else { |
96560cffef4d
8166002: Emulate client build on platforms with reduced virtual address space
jcm
parents:
40892
diff
changeset
|
64 |
_default_soft_ref_policy = new LRUCurrentHeapPolicy(); |
96560cffef4d
8166002: Emulate client build on platforms with reduced virtual address space
jcm
parents:
40892
diff
changeset
|
65 |
} |
1606
dcf9714addbe
6684579: SoftReference processing can be made more efficient
ysr
parents:
1605
diff
changeset
|
66 |
if (_always_clear_soft_ref_policy == NULL || _default_soft_ref_policy == NULL) { |
dcf9714addbe
6684579: SoftReference processing can be made more efficient
ysr
parents:
1605
diff
changeset
|
67 |
vm_exit_during_initialization("Could not allocate reference policy object"); |
dcf9714addbe
6684579: SoftReference processing can be made more efficient
ysr
parents:
1605
diff
changeset
|
68 |
} |
1 | 69 |
guarantee(RefDiscoveryPolicy == ReferenceBasedDiscovery || |
70 |
RefDiscoveryPolicy == ReferentBasedDiscovery, |
|
22775 | 71 |
"Unrecognized RefDiscoveryPolicy"); |
1 | 72 |
} |
73 |
||
28212
647b7d0efb88
8066827: Remove ReferenceProcessor::clean_up_discovered_references()
kbarrett
parents:
26831
diff
changeset
|
74 |
void ReferenceProcessor::enable_discovery(bool check_no_refs) { |
10683
4b5a5a507864
7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents:
10670
diff
changeset
|
75 |
#ifdef ASSERT |
4b5a5a507864
7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents:
10670
diff
changeset
|
76 |
// Verify that we're not currently discovering refs |
28212
647b7d0efb88
8066827: Remove ReferenceProcessor::clean_up_discovered_references()
kbarrett
parents:
26831
diff
changeset
|
77 |
assert(!_discovering_refs, "nested call?"); |
10683
4b5a5a507864
7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents:
10670
diff
changeset
|
78 |
|
4b5a5a507864
7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents:
10670
diff
changeset
|
79 |
if (check_no_refs) { |
4b5a5a507864
7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents:
10670
diff
changeset
|
80 |
// Verify that the discovered lists are empty |
4b5a5a507864
7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents:
10670
diff
changeset
|
81 |
verify_no_references_recorded(); |
4b5a5a507864
7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents:
10670
diff
changeset
|
82 |
} |
4b5a5a507864
7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents:
10670
diff
changeset
|
83 |
#endif // ASSERT |
4b5a5a507864
7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents:
10670
diff
changeset
|
84 |
|
4b5a5a507864
7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents:
10670
diff
changeset
|
85 |
// Someone could have modified the value of the static |
4b5a5a507864
7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents:
10670
diff
changeset
|
86 |
// field in the j.l.r.SoftReference class that holds the |
4b5a5a507864
7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents:
10670
diff
changeset
|
87 |
// soft reference timestamp clock using reflection or |
4b5a5a507864
7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents:
10670
diff
changeset
|
88 |
// Unsafe between GCs. Unconditionally update the static |
4b5a5a507864
7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents:
10670
diff
changeset
|
89 |
// field in ReferenceProcessor here so that we use the new |
4b5a5a507864
7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents:
10670
diff
changeset
|
90 |
// value during reference discovery. |
4b5a5a507864
7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents:
10670
diff
changeset
|
91 |
|
4b5a5a507864
7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents:
10670
diff
changeset
|
92 |
_soft_ref_timestamp_clock = java_lang_ref_SoftReference::clock(); |
4b5a5a507864
7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents:
10670
diff
changeset
|
93 |
_discovering_refs = true; |
4b5a5a507864
7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents:
10670
diff
changeset
|
94 |
} |
4b5a5a507864
7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents:
10670
diff
changeset
|
95 |
|
49964
99e698e94cc7
8201492: Properly implement non-contiguous generations for Reference discovery
tschatzl
parents:
49808
diff
changeset
|
96 |
ReferenceProcessor::ReferenceProcessor(BoolObjectClosure* is_subject_to_discovery, |
8688
493d12ccc6db
6668573: CMS: reference processing crash if ParallelCMSThreads > ParallelGCThreads
ysr
parents:
7420
diff
changeset
|
97 |
bool mt_processing, |
11396
917d8673b5ef
7121618: Change type of number of GC workers to unsigned int.
jmasa
parents:
11251
diff
changeset
|
98 |
uint mt_processing_degree, |
360
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
99 |
bool mt_discovery, |
11396
917d8673b5ef
7121618: Change type of number of GC workers to unsigned int.
jmasa
parents:
11251
diff
changeset
|
100 |
uint mt_discovery_degree, |
8688
493d12ccc6db
6668573: CMS: reference processing crash if ParallelCMSThreads > ParallelGCThreads
ysr
parents:
7420
diff
changeset
|
101 |
bool atomic_discovery, |
50606
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
102 |
BoolObjectClosure* is_alive_non_header, |
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
103 |
bool adjust_no_of_processing_threads) : |
49964
99e698e94cc7
8201492: Properly implement non-contiguous generations for Reference discovery
tschatzl
parents:
49808
diff
changeset
|
104 |
_is_subject_to_discovery(is_subject_to_discovery), |
1 | 105 |
_discovering_refs(false), |
106 |
_enqueuing_is_done(false), |
|
107 |
_processing_is_mt(mt_processing), |
|
50606
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
108 |
_next_id(0), |
51332 | 109 |
_adjust_no_of_processing_threads(adjust_no_of_processing_threads), |
110 |
_is_alive_non_header(is_alive_non_header) |
|
1 | 111 |
{ |
49964
99e698e94cc7
8201492: Properly implement non-contiguous generations for Reference discovery
tschatzl
parents:
49808
diff
changeset
|
112 |
assert(is_subject_to_discovery != NULL, "must be set"); |
99e698e94cc7
8201492: Properly implement non-contiguous generations for Reference discovery
tschatzl
parents:
49808
diff
changeset
|
113 |
|
1 | 114 |
_discovery_is_atomic = atomic_discovery; |
115 |
_discovery_is_mt = mt_discovery; |
|
49967
672ded60a082
8202021: Improve variable naming in ReferenceProcesso
tschatzl
parents:
49964
diff
changeset
|
116 |
_num_queues = MAX2(1U, mt_processing_degree); |
672ded60a082
8202021: Improve variable naming in ReferenceProcesso
tschatzl
parents:
49964
diff
changeset
|
117 |
_max_num_queues = MAX2(_num_queues, mt_discovery_degree); |
10747
dfdb9eb56e49
7095243: Disambiguate ReferenceProcessor::_discoveredSoftRefs
johnc
parents:
10683
diff
changeset
|
118 |
_discovered_refs = NEW_C_HEAP_ARRAY(DiscoveredList, |
49967
672ded60a082
8202021: Improve variable naming in ReferenceProcesso
tschatzl
parents:
49964
diff
changeset
|
119 |
_max_num_queues * number_of_subclasses_of_ref(), mtGC); |
13195 | 120 |
|
10747
dfdb9eb56e49
7095243: Disambiguate ReferenceProcessor::_discoveredSoftRefs
johnc
parents:
10683
diff
changeset
|
121 |
if (_discovered_refs == NULL) { |
1 | 122 |
vm_exit_during_initialization("Could not allocated RefProc Array"); |
123 |
} |
|
10747
dfdb9eb56e49
7095243: Disambiguate ReferenceProcessor::_discoveredSoftRefs
johnc
parents:
10683
diff
changeset
|
124 |
_discoveredSoftRefs = &_discovered_refs[0]; |
49967
672ded60a082
8202021: Improve variable naming in ReferenceProcesso
tschatzl
parents:
49964
diff
changeset
|
125 |
_discoveredWeakRefs = &_discoveredSoftRefs[_max_num_queues]; |
672ded60a082
8202021: Improve variable naming in ReferenceProcesso
tschatzl
parents:
49964
diff
changeset
|
126 |
_discoveredFinalRefs = &_discoveredWeakRefs[_max_num_queues]; |
672ded60a082
8202021: Improve variable naming in ReferenceProcesso
tschatzl
parents:
49964
diff
changeset
|
127 |
_discoveredPhantomRefs = &_discoveredFinalRefs[_max_num_queues]; |
10747
dfdb9eb56e49
7095243: Disambiguate ReferenceProcessor::_discoveredSoftRefs
johnc
parents:
10683
diff
changeset
|
128 |
|
dfdb9eb56e49
7095243: Disambiguate ReferenceProcessor::_discoveredSoftRefs
johnc
parents:
10683
diff
changeset
|
129 |
// Initialize all entries to NULL |
49967
672ded60a082
8202021: Improve variable naming in ReferenceProcesso
tschatzl
parents:
49964
diff
changeset
|
130 |
for (uint i = 0; i < _max_num_queues * number_of_subclasses_of_ref(); i++) { |
50605
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
131 |
_discovered_refs[i].clear(); |
1 | 132 |
} |
10747
dfdb9eb56e49
7095243: Disambiguate ReferenceProcessor::_discoveredSoftRefs
johnc
parents:
10683
diff
changeset
|
133 |
|
8688
493d12ccc6db
6668573: CMS: reference processing crash if ParallelCMSThreads > ParallelGCThreads
ysr
parents:
7420
diff
changeset
|
134 |
setup_policy(false /* default soft ref policy */); |
1 | 135 |
} |
136 |
||
137 |
#ifndef PRODUCT |
|
138 |
void ReferenceProcessor::verify_no_references_recorded() { |
|
139 |
guarantee(!_discovering_refs, "Discovering refs?"); |
|
49967
672ded60a082
8202021: Improve variable naming in ReferenceProcesso
tschatzl
parents:
49964
diff
changeset
|
140 |
for (uint i = 0; i < _max_num_queues * number_of_subclasses_of_ref(); i++) { |
10747
dfdb9eb56e49
7095243: Disambiguate ReferenceProcessor::_discoveredSoftRefs
johnc
parents:
10683
diff
changeset
|
141 |
guarantee(_discovered_refs[i].is_empty(), |
37165
ed7da23aaa16
8149343: assert(rp->num_q() == no_of_gc_workers) failed: sanity
jmasa
parents:
37106
diff
changeset
|
142 |
"Found non-empty discovered list at %u", i); |
1 | 143 |
} |
144 |
} |
|
145 |
#endif |
|
146 |
||
147 |
void ReferenceProcessor::weak_oops_do(OopClosure* f) { |
|
49967
672ded60a082
8202021: Improve variable naming in ReferenceProcesso
tschatzl
parents:
49964
diff
changeset
|
148 |
for (uint i = 0; i < _max_num_queues * number_of_subclasses_of_ref(); i++) { |
360
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
149 |
if (UseCompressedOops) { |
10747
dfdb9eb56e49
7095243: Disambiguate ReferenceProcessor::_discoveredSoftRefs
johnc
parents:
10683
diff
changeset
|
150 |
f->do_oop((narrowOop*)_discovered_refs[i].adr_head()); |
360
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
151 |
} else { |
10747
dfdb9eb56e49
7095243: Disambiguate ReferenceProcessor::_discoveredSoftRefs
johnc
parents:
10683
diff
changeset
|
152 |
f->do_oop((oop*)_discovered_refs[i].adr_head()); |
360
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
153 |
} |
1 | 154 |
} |
155 |
} |
|
156 |
||
360
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
157 |
void ReferenceProcessor::update_soft_ref_master_clock() { |
1 | 158 |
// Update (advance) the soft ref master clock field. This must be done |
159 |
// after processing the soft ref list. |
|
11251
e29da6b5622b
7117303: VM uses non-monotonic time source and complains that it is non-monotonic
johnc
parents:
10747
diff
changeset
|
160 |
|
22551 | 161 |
// We need a monotonically non-decreasing time in ms but |
11251
e29da6b5622b
7117303: VM uses non-monotonic time source and complains that it is non-monotonic
johnc
parents:
10747
diff
changeset
|
162 |
// os::javaTimeMillis() does not guarantee monotonicity. |
e29da6b5622b
7117303: VM uses non-monotonic time source and complains that it is non-monotonic
johnc
parents:
10747
diff
changeset
|
163 |
jlong now = os::javaTimeNanos() / NANOSECS_PER_MILLISEC; |
10683
4b5a5a507864
7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents:
10670
diff
changeset
|
164 |
jlong soft_ref_clock = java_lang_ref_SoftReference::clock(); |
4b5a5a507864
7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents:
10670
diff
changeset
|
165 |
assert(soft_ref_clock == _soft_ref_timestamp_clock, "soft ref clocks out of sync"); |
4b5a5a507864
7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents:
10670
diff
changeset
|
166 |
|
1 | 167 |
NOT_PRODUCT( |
10683
4b5a5a507864
7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents:
10670
diff
changeset
|
168 |
if (now < _soft_ref_timestamp_clock) { |
37073
c39d0903390b
8151605: Change warning() to log_warning(gc) in the GC code
brutisso
parents:
37067
diff
changeset
|
169 |
log_warning(gc)("time warp: " JLONG_FORMAT " to " JLONG_FORMAT, |
c39d0903390b
8151605: Change warning() to log_warning(gc) in the GC code
brutisso
parents:
37067
diff
changeset
|
170 |
_soft_ref_timestamp_clock, now); |
1 | 171 |
} |
172 |
) |
|
11251
e29da6b5622b
7117303: VM uses non-monotonic time source and complains that it is non-monotonic
johnc
parents:
10747
diff
changeset
|
173 |
// The values of now and _soft_ref_timestamp_clock are set using |
e29da6b5622b
7117303: VM uses non-monotonic time source and complains that it is non-monotonic
johnc
parents:
10747
diff
changeset
|
174 |
// javaTimeNanos(), which is guaranteed to be monotonically |
e29da6b5622b
7117303: VM uses non-monotonic time source and complains that it is non-monotonic
johnc
parents:
10747
diff
changeset
|
175 |
// non-decreasing provided the underlying platform provides such |
e29da6b5622b
7117303: VM uses non-monotonic time source and complains that it is non-monotonic
johnc
parents:
10747
diff
changeset
|
176 |
// a time source (and it is bug free). |
22551 | 177 |
// In product mode, however, protect ourselves from non-monotonicity. |
10683
4b5a5a507864
7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents:
10670
diff
changeset
|
178 |
if (now > _soft_ref_timestamp_clock) { |
4b5a5a507864
7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents:
10670
diff
changeset
|
179 |
_soft_ref_timestamp_clock = now; |
1 | 180 |
java_lang_ref_SoftReference::set_clock(now); |
181 |
} |
|
182 |
// Else leave clock stalled at its old value until time progresses |
|
183 |
// past clock value. |
|
184 |
} |
|
185 |
||
46795
623a5e42deb6
8173335: Improve logging for j.l.ref.reference processing
sangheki
parents:
43455
diff
changeset
|
186 |
size_t ReferenceProcessor::total_count(DiscoveredList lists[]) const { |
18025 | 187 |
size_t total = 0; |
49967
672ded60a082
8202021: Improve variable naming in ReferenceProcesso
tschatzl
parents:
49964
diff
changeset
|
188 |
for (uint i = 0; i < _max_num_queues; ++i) { |
18025 | 189 |
total += lists[i].length(); |
190 |
} |
|
191 |
return total; |
|
192 |
} |
|
193 |
||
50605
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
194 |
#ifdef ASSERT |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
195 |
void ReferenceProcessor::verify_total_count_zero(DiscoveredList lists[], const char* type) { |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
196 |
size_t count = total_count(lists); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
197 |
assert(count == 0, "%ss must be empty but has " SIZE_FORMAT " elements", type, count); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
198 |
} |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
199 |
#endif |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
200 |
|
18025 | 201 |
ReferenceProcessorStats ReferenceProcessor::process_discovered_references( |
46795
623a5e42deb6
8173335: Improve logging for j.l.ref.reference processing
sangheki
parents:
43455
diff
changeset
|
202 |
BoolObjectClosure* is_alive, |
623a5e42deb6
8173335: Improve logging for j.l.ref.reference processing
sangheki
parents:
43455
diff
changeset
|
203 |
OopClosure* keep_alive, |
623a5e42deb6
8173335: Improve logging for j.l.ref.reference processing
sangheki
parents:
43455
diff
changeset
|
204 |
VoidClosure* complete_gc, |
623a5e42deb6
8173335: Improve logging for j.l.ref.reference processing
sangheki
parents:
43455
diff
changeset
|
205 |
AbstractRefProcTaskExecutor* task_executor, |
623a5e42deb6
8173335: Improve logging for j.l.ref.reference processing
sangheki
parents:
43455
diff
changeset
|
206 |
ReferenceProcessorPhaseTimes* phase_times) { |
623a5e42deb6
8173335: Improve logging for j.l.ref.reference processing
sangheki
parents:
43455
diff
changeset
|
207 |
|
623a5e42deb6
8173335: Improve logging for j.l.ref.reference processing
sangheki
parents:
43455
diff
changeset
|
208 |
double start_time = os::elapsedTime(); |
1 | 209 |
|
210 |
assert(!enqueuing_is_done(), "If here enqueuing should not be complete"); |
|
211 |
// Stop treating discovered references specially. |
|
212 |
disable_discovery(); |
|
213 |
||
10683
4b5a5a507864
7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents:
10670
diff
changeset
|
214 |
// If discovery was concurrent, someone could have modified |
4b5a5a507864
7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents:
10670
diff
changeset
|
215 |
// the value of the static field in the j.l.r.SoftReference |
4b5a5a507864
7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents:
10670
diff
changeset
|
216 |
// class that holds the soft reference timestamp clock using |
4b5a5a507864
7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents:
10670
diff
changeset
|
217 |
// reflection or Unsafe between when discovery was enabled and |
4b5a5a507864
7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents:
10670
diff
changeset
|
218 |
// now. Unconditionally update the static field in ReferenceProcessor |
4b5a5a507864
7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents:
10670
diff
changeset
|
219 |
// here so that we use the new value during processing of the |
4b5a5a507864
7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents:
10670
diff
changeset
|
220 |
// discovered soft refs. |
4b5a5a507864
7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents:
10670
diff
changeset
|
221 |
|
4b5a5a507864
7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents:
10670
diff
changeset
|
222 |
_soft_ref_timestamp_clock = java_lang_ref_SoftReference::clock(); |
4b5a5a507864
7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents:
10670
diff
changeset
|
223 |
|
46795
623a5e42deb6
8173335: Improve logging for j.l.ref.reference processing
sangheki
parents:
43455
diff
changeset
|
224 |
ReferenceProcessorStats stats(total_count(_discoveredSoftRefs), |
623a5e42deb6
8173335: Improve logging for j.l.ref.reference processing
sangheki
parents:
43455
diff
changeset
|
225 |
total_count(_discoveredWeakRefs), |
623a5e42deb6
8173335: Improve logging for j.l.ref.reference processing
sangheki
parents:
43455
diff
changeset
|
226 |
total_count(_discoveredFinalRefs), |
623a5e42deb6
8173335: Improve logging for j.l.ref.reference processing
sangheki
parents:
43455
diff
changeset
|
227 |
total_count(_discoveredPhantomRefs)); |
33103
116b558af514
8136991: [REDO] Additional number of processed references printed with -XX:+PrintReferenceGC after JDK-8047125
brutisso
parents:
32817
diff
changeset
|
228 |
|
1 | 229 |
{ |
50605
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
230 |
RefProcTotalPhaseTimesTracker tt(RefPhase1, phase_times, this); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
231 |
process_soft_ref_reconsider(is_alive, keep_alive, complete_gc, |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
232 |
task_executor, phase_times); |
1 | 233 |
} |
234 |
||
235 |
update_soft_ref_master_clock(); |
|
236 |
||
237 |
{ |
|
50605
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
238 |
RefProcTotalPhaseTimesTracker tt(RefPhase2, phase_times, this); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
239 |
process_soft_weak_final_refs(is_alive, keep_alive, complete_gc, task_executor, phase_times); |
1 | 240 |
} |
241 |
||
242 |
{ |
|
50605
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
243 |
RefProcTotalPhaseTimesTracker tt(RefPhase3, phase_times, this); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
244 |
process_final_keep_alive(keep_alive, complete_gc, task_executor, phase_times); |
1 | 245 |
} |
246 |
||
247 |
{ |
|
50605
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
248 |
RefProcTotalPhaseTimesTracker tt(RefPhase4, phase_times, this); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
249 |
process_phantom_refs(is_alive, keep_alive, complete_gc, task_executor, phase_times); |
1 | 250 |
} |
251 |
||
47648
226b1fc611b9
8189359: Move native weak oops cleaning out of ReferenceProcessor
stefank
parents:
47216
diff
changeset
|
252 |
if (task_executor != NULL) { |
226b1fc611b9
8189359: Move native weak oops cleaning out of ReferenceProcessor
stefank
parents:
47216
diff
changeset
|
253 |
// Record the work done by the parallel workers. |
226b1fc611b9
8189359: Move native weak oops cleaning out of ReferenceProcessor
stefank
parents:
47216
diff
changeset
|
254 |
task_executor->set_single_threaded_mode(); |
1 | 255 |
} |
18025 | 256 |
|
46795
623a5e42deb6
8173335: Improve logging for j.l.ref.reference processing
sangheki
parents:
43455
diff
changeset
|
257 |
phase_times->set_total_time_ms((os::elapsedTime() - start_time) * 1000); |
623a5e42deb6
8173335: Improve logging for j.l.ref.reference processing
sangheki
parents:
43455
diff
changeset
|
258 |
|
33103
116b558af514
8136991: [REDO] Additional number of processed references printed with -XX:+PrintReferenceGC after JDK-8047125
brutisso
parents:
32817
diff
changeset
|
259 |
return stats; |
1 | 260 |
} |
261 |
||
10670
4ea0e7d2ffbc
6484982: G1: process references during evacuation pauses
johnc
parents:
10526
diff
changeset
|
262 |
void DiscoveredListIterator::load_ptrs(DEBUG_ONLY(bool allow_null_referent)) { |
49967
672ded60a082
8202021: Improve variable naming in ReferenceProcesso
tschatzl
parents:
49964
diff
changeset
|
263 |
_current_discovered_addr = java_lang_ref_Reference::discovered_addr_raw(_current_discovered); |
672ded60a082
8202021: Improve variable naming in ReferenceProcesso
tschatzl
parents:
49964
diff
changeset
|
264 |
oop discovered = java_lang_ref_Reference::discovered(_current_discovered); |
672ded60a082
8202021: Improve variable naming in ReferenceProcesso
tschatzl
parents:
49964
diff
changeset
|
265 |
assert(_current_discovered_addr && oopDesc::is_oop_or_null(discovered), |
33105
294e48b4f704
8080775: Better argument formatting for assert() and friends
david
parents:
33103
diff
changeset
|
266 |
"Expected an oop or NULL for discovered field at " PTR_FORMAT, p2i(discovered)); |
49967
672ded60a082
8202021: Improve variable naming in ReferenceProcesso
tschatzl
parents:
49964
diff
changeset
|
267 |
_next_discovered = discovered; |
672ded60a082
8202021: Improve variable naming in ReferenceProcesso
tschatzl
parents:
49964
diff
changeset
|
268 |
|
672ded60a082
8202021: Improve variable naming in ReferenceProcesso
tschatzl
parents:
49964
diff
changeset
|
269 |
_referent_addr = java_lang_ref_Reference::referent_addr_raw(_current_discovered); |
672ded60a082
8202021: Improve variable naming in ReferenceProcesso
tschatzl
parents:
49964
diff
changeset
|
270 |
_referent = java_lang_ref_Reference::referent(_current_discovered); |
1 | 271 |
assert(Universe::heap()->is_in_reserved_or_null(_referent), |
272 |
"Wrong oop found in java.lang.Reference object"); |
|
273 |
assert(allow_null_referent ? |
|
46968
9119841280f4
8160399: is_oop_or_null involves undefined behavior
coleenp
parents:
46795
diff
changeset
|
274 |
oopDesc::is_oop_or_null(_referent) |
9119841280f4
8160399: is_oop_or_null involves undefined behavior
coleenp
parents:
46795
diff
changeset
|
275 |
: oopDesc::is_oop(_referent), |
33105
294e48b4f704
8080775: Better argument formatting for assert() and friends
david
parents:
33103
diff
changeset
|
276 |
"Expected an oop%s for referent field at " PTR_FORMAT, |
294e48b4f704
8080775: Better argument formatting for assert() and friends
david
parents:
33103
diff
changeset
|
277 |
(allow_null_referent ? " or NULL" : ""), |
294e48b4f704
8080775: Better argument formatting for assert() and friends
david
parents:
33103
diff
changeset
|
278 |
p2i(_referent)); |
1 | 279 |
} |
280 |
||
10670
4ea0e7d2ffbc
6484982: G1: process references during evacuation pauses
johnc
parents:
10526
diff
changeset
|
281 |
void DiscoveredListIterator::remove() { |
49967
672ded60a082
8202021: Improve variable naming in ReferenceProcesso
tschatzl
parents:
49964
diff
changeset
|
282 |
assert(oopDesc::is_oop(_current_discovered), "Dropping a bad reference"); |
672ded60a082
8202021: Improve variable naming in ReferenceProcesso
tschatzl
parents:
49964
diff
changeset
|
283 |
RawAccess<>::oop_store(_current_discovered_addr, oop(NULL)); |
10524
6594ca81279a
7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents:
8688
diff
changeset
|
284 |
|
360
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
285 |
// First _prev_next ref actually points into DiscoveredList (gross). |
10524
6594ca81279a
7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents:
8688
diff
changeset
|
286 |
oop new_next; |
52070 | 287 |
if (oopDesc::equals_raw(_next_discovered, _current_discovered)) { |
10524
6594ca81279a
7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents:
8688
diff
changeset
|
288 |
// At the end of the list, we should make _prev point to itself. |
6594ca81279a
7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents:
8688
diff
changeset
|
289 |
// If _ref is the first ref, then _prev_next will be in the DiscoveredList, |
6594ca81279a
7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents:
8688
diff
changeset
|
290 |
// and _prev will be NULL. |
49967
672ded60a082
8202021: Improve variable naming in ReferenceProcesso
tschatzl
parents:
49964
diff
changeset
|
291 |
new_next = _prev_discovered; |
10524
6594ca81279a
7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents:
8688
diff
changeset
|
292 |
} else { |
49967
672ded60a082
8202021: Improve variable naming in ReferenceProcesso
tschatzl
parents:
49964
diff
changeset
|
293 |
new_next = _next_discovered; |
10524
6594ca81279a
7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents:
8688
diff
changeset
|
294 |
} |
22768
363a44995a71
8031703: Missing post-barrier in ReferenceProcessor
pliden
parents:
22551
diff
changeset
|
295 |
// Remove Reference object from discovered list. Note that G1 does not need a |
363a44995a71
8031703: Missing post-barrier in ReferenceProcessor
pliden
parents:
22551
diff
changeset
|
296 |
// pre-barrier here because we know the Reference has already been found/marked, |
363a44995a71
8031703: Missing post-barrier in ReferenceProcessor
pliden
parents:
22551
diff
changeset
|
297 |
// that's how it ended up in the discovered list in the first place. |
49967
672ded60a082
8202021: Improve variable naming in ReferenceProcesso
tschatzl
parents:
49964
diff
changeset
|
298 |
RawAccess<>::oop_store(_prev_discovered_addr, new_next); |
50605
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
299 |
_removed++; |
1605
6b43d186eb8d
6722113: CMS: Incorrect overflow handling during precleaning of Reference lists
ysr
parents:
1388
diff
changeset
|
300 |
_refs_list.dec_length(1); |
1 | 301 |
} |
302 |
||
10670
4ea0e7d2ffbc
6484982: G1: process references during evacuation pauses
johnc
parents:
10526
diff
changeset
|
303 |
void DiscoveredListIterator::clear_referent() { |
47998
fb0275c320a0
8189871: Refactor GC barriers to use declarative semantics
eosterlund
parents:
47648
diff
changeset
|
304 |
RawAccess<>::oop_store(_referent_addr, oop(NULL)); |
1 | 305 |
} |
306 |
||
50049
9d17c375dc30
8202017: Merge Reference Enqueuing phase with phase 3 of Reference processing
tschatzl
parents:
49967
diff
changeset
|
307 |
void DiscoveredListIterator::enqueue() { |
9d17c375dc30
8202017: Merge Reference Enqueuing phase with phase 3 of Reference processing
tschatzl
parents:
49967
diff
changeset
|
308 |
HeapAccess<AS_NO_KEEPALIVE>::oop_store_at(_current_discovered, |
9d17c375dc30
8202017: Merge Reference Enqueuing phase with phase 3 of Reference processing
tschatzl
parents:
49967
diff
changeset
|
309 |
java_lang_ref_Reference::discovered_offset, |
9d17c375dc30
8202017: Merge Reference Enqueuing phase with phase 3 of Reference processing
tschatzl
parents:
49967
diff
changeset
|
310 |
_next_discovered); |
9d17c375dc30
8202017: Merge Reference Enqueuing phase with phase 3 of Reference processing
tschatzl
parents:
49967
diff
changeset
|
311 |
} |
9d17c375dc30
8202017: Merge Reference Enqueuing phase with phase 3 of Reference processing
tschatzl
parents:
49967
diff
changeset
|
312 |
|
50054
4fa726f796f5
8202781: Fix typo in DiscoveredListIterator::complete_enqeue
tschatzl
parents:
50049
diff
changeset
|
313 |
void DiscoveredListIterator::complete_enqueue() { |
50049
9d17c375dc30
8202017: Merge Reference Enqueuing phase with phase 3 of Reference processing
tschatzl
parents:
49967
diff
changeset
|
314 |
if (_prev_discovered != NULL) { |
9d17c375dc30
8202017: Merge Reference Enqueuing phase with phase 3 of Reference processing
tschatzl
parents:
49967
diff
changeset
|
315 |
// This is the last object. |
9d17c375dc30
8202017: Merge Reference Enqueuing phase with phase 3 of Reference processing
tschatzl
parents:
49967
diff
changeset
|
316 |
// Swap refs_list into pending list and set obj's |
9d17c375dc30
8202017: Merge Reference Enqueuing phase with phase 3 of Reference processing
tschatzl
parents:
49967
diff
changeset
|
317 |
// discovered to what we read from the pending list. |
9d17c375dc30
8202017: Merge Reference Enqueuing phase with phase 3 of Reference processing
tschatzl
parents:
49967
diff
changeset
|
318 |
oop old = Universe::swap_reference_pending_list(_refs_list.head()); |
9d17c375dc30
8202017: Merge Reference Enqueuing phase with phase 3 of Reference processing
tschatzl
parents:
49967
diff
changeset
|
319 |
HeapAccess<AS_NO_KEEPALIVE>::oop_store_at(_prev_discovered, java_lang_ref_Reference::discovered_offset, old); |
9d17c375dc30
8202017: Merge Reference Enqueuing phase with phase 3 of Reference processing
tschatzl
parents:
49967
diff
changeset
|
320 |
} |
9d17c375dc30
8202017: Merge Reference Enqueuing phase with phase 3 of Reference processing
tschatzl
parents:
49967
diff
changeset
|
321 |
} |
9d17c375dc30
8202017: Merge Reference Enqueuing phase with phase 3 of Reference processing
tschatzl
parents:
49967
diff
changeset
|
322 |
|
50605
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
323 |
inline void log_dropped_ref(const DiscoveredListIterator& iter, const char* reason) { |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
324 |
if (log_develop_is_enabled(Trace, gc, ref)) { |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
325 |
ResourceMark rm; |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
326 |
log_develop_trace(gc, ref)("Dropping %s reference " PTR_FORMAT ": %s", |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
327 |
reason, p2i(iter.obj()), |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
328 |
iter.obj()->klass()->internal_name()); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
329 |
} |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
330 |
} |
1 | 331 |
|
50605
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
332 |
inline void log_enqueued_ref(const DiscoveredListIterator& iter, const char* reason) { |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
333 |
if (log_develop_is_enabled(Trace, gc, ref)) { |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
334 |
ResourceMark rm; |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
335 |
log_develop_trace(gc, ref)("Enqueue %s reference (" INTPTR_FORMAT ": %s)", |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
336 |
reason, p2i(iter.obj()), iter.obj()->klass()->internal_name()); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
337 |
} |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
338 |
assert(oopDesc::is_oop(iter.obj(), UseConcMarkSweepGC), "Adding a bad reference"); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
339 |
} |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
340 |
|
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
341 |
size_t ReferenceProcessor::process_soft_ref_reconsider_work(DiscoveredList& refs_list, |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
342 |
ReferencePolicy* policy, |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
343 |
BoolObjectClosure* is_alive, |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
344 |
OopClosure* keep_alive, |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
345 |
VoidClosure* complete_gc) { |
1 | 346 |
assert(policy != NULL, "Must have a non-NULL policy"); |
24845
e8b8894a77df
8043239: G1: Missing post barrier in processing of j.l.ref.Reference objects
brutisso
parents:
24424
diff
changeset
|
347 |
DiscoveredListIterator iter(refs_list, keep_alive, is_alive); |
1 | 348 |
// Decide which softly reachable refs should be kept alive. |
349 |
while (iter.has_next()) { |
|
350 |
iter.load_ptrs(DEBUG_ONLY(!discovery_is_atomic() /* allow_null_referent */)); |
|
351 |
bool referent_is_dead = (iter.referent() != NULL) && !iter.is_referent_alive(); |
|
10683
4b5a5a507864
7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents:
10670
diff
changeset
|
352 |
if (referent_is_dead && |
4b5a5a507864
7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents:
10670
diff
changeset
|
353 |
!policy->should_clear_reference(iter.obj(), _soft_ref_timestamp_clock)) { |
50605
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
354 |
log_dropped_ref(iter, "by policy"); |
1605
6b43d186eb8d
6722113: CMS: Incorrect overflow handling during precleaning of Reference lists
ysr
parents:
1388
diff
changeset
|
355 |
// Remove Reference object from list |
6b43d186eb8d
6722113: CMS: Incorrect overflow handling during precleaning of Reference lists
ysr
parents:
1388
diff
changeset
|
356 |
iter.remove(); |
1 | 357 |
// keep the referent around |
358 |
iter.make_referent_alive(); |
|
1605
6b43d186eb8d
6722113: CMS: Incorrect overflow handling during precleaning of Reference lists
ysr
parents:
1388
diff
changeset
|
359 |
iter.move_to_next(); |
1 | 360 |
} else { |
361 |
iter.next(); |
|
362 |
} |
|
363 |
} |
|
364 |
// Close the reachable set |
|
365 |
complete_gc->do_void(); |
|
50605
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
366 |
|
35061 | 367 |
log_develop_trace(gc, ref)(" Dropped " SIZE_FORMAT " dead Refs out of " SIZE_FORMAT " discovered Refs by policy, from list " INTPTR_FORMAT, |
36198
658f3871f6a2
8150302: Reference processing logging prints the "from list" incorrectly
jmasa
parents:
35862
diff
changeset
|
368 |
iter.removed(), iter.processed(), p2i(&refs_list)); |
50605
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
369 |
return iter.removed(); |
46795
623a5e42deb6
8173335: Improve logging for j.l.ref.reference processing
sangheki
parents:
43455
diff
changeset
|
370 |
} |
1 | 371 |
|
50605
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
372 |
size_t ReferenceProcessor::process_soft_weak_final_refs_work(DiscoveredList& refs_list, |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
373 |
BoolObjectClosure* is_alive, |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
374 |
OopClosure* keep_alive, |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
375 |
bool do_enqueue_and_clear) { |
24845
e8b8894a77df
8043239: G1: Missing post barrier in processing of j.l.ref.Reference objects
brutisso
parents:
24424
diff
changeset
|
376 |
DiscoveredListIterator iter(refs_list, keep_alive, is_alive); |
1 | 377 |
while (iter.has_next()) { |
50277
f84ae8aa5d88
8203028: Simplify reference processing in light of JDK-8175797
kbarrett
parents:
50097
diff
changeset
|
378 |
iter.load_ptrs(DEBUG_ONLY(!discovery_is_atomic() /* allow_null_referent */)); |
f84ae8aa5d88
8203028: Simplify reference processing in light of JDK-8175797
kbarrett
parents:
50097
diff
changeset
|
379 |
if (iter.referent() == NULL) { |
f84ae8aa5d88
8203028: Simplify reference processing in light of JDK-8175797
kbarrett
parents:
50097
diff
changeset
|
380 |
// Reference has been cleared since discovery; only possible if |
f84ae8aa5d88
8203028: Simplify reference processing in light of JDK-8175797
kbarrett
parents:
50097
diff
changeset
|
381 |
// discovery is not atomic (checked by load_ptrs). Remove |
f84ae8aa5d88
8203028: Simplify reference processing in light of JDK-8175797
kbarrett
parents:
50097
diff
changeset
|
382 |
// reference from list. |
f84ae8aa5d88
8203028: Simplify reference processing in light of JDK-8175797
kbarrett
parents:
50097
diff
changeset
|
383 |
log_dropped_ref(iter, "cleared"); |
f84ae8aa5d88
8203028: Simplify reference processing in light of JDK-8175797
kbarrett
parents:
50097
diff
changeset
|
384 |
iter.remove(); |
f84ae8aa5d88
8203028: Simplify reference processing in light of JDK-8175797
kbarrett
parents:
50097
diff
changeset
|
385 |
iter.move_to_next(); |
f84ae8aa5d88
8203028: Simplify reference processing in light of JDK-8175797
kbarrett
parents:
50097
diff
changeset
|
386 |
} else if (iter.is_referent_alive()) { |
1 | 387 |
// The referent is reachable after all. |
50277
f84ae8aa5d88
8203028: Simplify reference processing in light of JDK-8175797
kbarrett
parents:
50097
diff
changeset
|
388 |
// Remove reference from list. |
f84ae8aa5d88
8203028: Simplify reference processing in light of JDK-8175797
kbarrett
parents:
50097
diff
changeset
|
389 |
log_dropped_ref(iter, "reachable"); |
1605
6b43d186eb8d
6722113: CMS: Incorrect overflow handling during precleaning of Reference lists
ysr
parents:
1388
diff
changeset
|
390 |
iter.remove(); |
50277
f84ae8aa5d88
8203028: Simplify reference processing in light of JDK-8175797
kbarrett
parents:
50097
diff
changeset
|
391 |
// Update the referent pointer as necessary. Note that this |
1 | 392 |
// should not entail any recursive marking because the |
393 |
// referent must already have been traversed. |
|
394 |
iter.make_referent_alive(); |
|
1605
6b43d186eb8d
6722113: CMS: Incorrect overflow handling during precleaning of Reference lists
ysr
parents:
1388
diff
changeset
|
395 |
iter.move_to_next(); |
1 | 396 |
} else { |
50605
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
397 |
if (do_enqueue_and_clear) { |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
398 |
iter.clear_referent(); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
399 |
iter.enqueue(); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
400 |
log_enqueued_ref(iter, "cleared"); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
401 |
} |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
402 |
// Keep in discovered list |
1 | 403 |
iter.next(); |
404 |
} |
|
405 |
} |
|
50605
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
406 |
if (do_enqueue_and_clear) { |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
407 |
iter.complete_enqueue(); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
408 |
refs_list.clear(); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
409 |
} |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
410 |
|
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
411 |
log_develop_trace(gc, ref)(" Dropped " SIZE_FORMAT " active Refs out of " SIZE_FORMAT |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
412 |
" Refs in discovered list " INTPTR_FORMAT, |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
413 |
iter.removed(), iter.processed(), p2i(&refs_list)); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
414 |
return iter.removed(); |
1 | 415 |
} |
416 |
||
50605
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
417 |
size_t ReferenceProcessor::process_final_keep_alive_work(DiscoveredList& refs_list, |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
418 |
OopClosure* keep_alive, |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
419 |
VoidClosure* complete_gc) { |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
420 |
DiscoveredListIterator iter(refs_list, keep_alive, NULL); |
1 | 421 |
while (iter.has_next()) { |
422 |
iter.load_ptrs(DEBUG_ONLY(false /* allow_null_referent */)); |
|
50605
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
423 |
// keep the referent and followers around |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
424 |
iter.make_referent_alive(); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
425 |
|
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
426 |
// Self-loop next, to mark the FinalReference not active. |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
427 |
assert(java_lang_ref_Reference::next(iter.obj()) == NULL, "enqueued FinalReference"); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
428 |
java_lang_ref_Reference::set_next_raw(iter.obj(), iter.obj()); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
429 |
|
50049
9d17c375dc30
8202017: Merge Reference Enqueuing phase with phase 3 of Reference processing
tschatzl
parents:
49967
diff
changeset
|
430 |
iter.enqueue(); |
50605
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
431 |
log_enqueued_ref(iter, "Final"); |
1 | 432 |
iter.next(); |
433 |
} |
|
50054
4fa726f796f5
8202781: Fix typo in DiscoveredListIterator::complete_enqeue
tschatzl
parents:
50049
diff
changeset
|
434 |
iter.complete_enqueue(); |
1 | 435 |
// Close the reachable set |
436 |
complete_gc->do_void(); |
|
50605
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
437 |
refs_list.clear(); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
438 |
|
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
439 |
assert(iter.removed() == 0, "This phase does not remove anything."); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
440 |
return iter.removed(); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
441 |
} |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
442 |
|
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
443 |
size_t ReferenceProcessor::process_phantom_refs_work(DiscoveredList& refs_list, |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
444 |
BoolObjectClosure* is_alive, |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
445 |
OopClosure* keep_alive, |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
446 |
VoidClosure* complete_gc) { |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
447 |
DiscoveredListIterator iter(refs_list, keep_alive, is_alive); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
448 |
while (iter.has_next()) { |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
449 |
iter.load_ptrs(DEBUG_ONLY(!discovery_is_atomic() /* allow_null_referent */)); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
450 |
|
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
451 |
oop const referent = iter.referent(); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
452 |
|
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
453 |
if (referent == NULL || iter.is_referent_alive()) { |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
454 |
iter.make_referent_alive(); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
455 |
iter.remove(); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
456 |
iter.move_to_next(); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
457 |
} else { |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
458 |
iter.clear_referent(); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
459 |
iter.enqueue(); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
460 |
log_enqueued_ref(iter, "cleared Phantom"); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
461 |
iter.next(); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
462 |
} |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
463 |
} |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
464 |
iter.complete_enqueue(); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
465 |
// Close the reachable set; needed for collectors which keep_alive_closure do |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
466 |
// not immediately complete their work. |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
467 |
complete_gc->do_void(); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
468 |
refs_list.clear(); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
469 |
|
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
470 |
return iter.removed(); |
1 | 471 |
} |
472 |
||
473 |
void |
|
10524
6594ca81279a
7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents:
8688
diff
changeset
|
474 |
ReferenceProcessor::clear_discovered_references(DiscoveredList& refs_list) { |
6594ca81279a
7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents:
8688
diff
changeset
|
475 |
oop obj = NULL; |
6594ca81279a
7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents:
8688
diff
changeset
|
476 |
oop next = refs_list.head(); |
52070 | 477 |
while (!oopDesc::equals_raw(next, obj)) { |
10524
6594ca81279a
7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents:
8688
diff
changeset
|
478 |
obj = next; |
6594ca81279a
7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents:
8688
diff
changeset
|
479 |
next = java_lang_ref_Reference::discovered(obj); |
360
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
480 |
java_lang_ref_Reference::set_discovered_raw(obj, NULL); |
1 | 481 |
} |
50605
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
482 |
refs_list.clear(); |
1 | 483 |
} |
484 |
||
1374
4c24294029a9
6711316: Open source the Garbage-First garbage collector
ysr
parents:
360
diff
changeset
|
485 |
void ReferenceProcessor::abandon_partial_discovery() { |
4c24294029a9
6711316: Open source the Garbage-First garbage collector
ysr
parents:
360
diff
changeset
|
486 |
// loop over the lists |
49967
672ded60a082
8202021: Improve variable naming in ReferenceProcesso
tschatzl
parents:
49964
diff
changeset
|
487 |
for (uint i = 0; i < _max_num_queues * number_of_subclasses_of_ref(); i++) { |
672ded60a082
8202021: Improve variable naming in ReferenceProcesso
tschatzl
parents:
49964
diff
changeset
|
488 |
if ((i % _max_num_queues) == 0) { |
35061 | 489 |
log_develop_trace(gc, ref)("Abandoning %s discovered list", list_name(i)); |
1374
4c24294029a9
6711316: Open source the Garbage-First garbage collector
ysr
parents:
360
diff
changeset
|
490 |
} |
30766
8afe4435d599
8058265: No callers of ReferenceProcessor::clear_discovered_references
kbarrett
parents:
30764
diff
changeset
|
491 |
clear_discovered_references(_discovered_refs[i]); |
1 | 492 |
} |
493 |
} |
|
494 |
||
46795
623a5e42deb6
8173335: Improve logging for j.l.ref.reference processing
sangheki
parents:
43455
diff
changeset
|
495 |
size_t ReferenceProcessor::total_reference_count(ReferenceType type) const { |
623a5e42deb6
8173335: Improve logging for j.l.ref.reference processing
sangheki
parents:
43455
diff
changeset
|
496 |
DiscoveredList* list = NULL; |
623a5e42deb6
8173335: Improve logging for j.l.ref.reference processing
sangheki
parents:
43455
diff
changeset
|
497 |
|
623a5e42deb6
8173335: Improve logging for j.l.ref.reference processing
sangheki
parents:
43455
diff
changeset
|
498 |
switch (type) { |
623a5e42deb6
8173335: Improve logging for j.l.ref.reference processing
sangheki
parents:
43455
diff
changeset
|
499 |
case REF_SOFT: |
623a5e42deb6
8173335: Improve logging for j.l.ref.reference processing
sangheki
parents:
43455
diff
changeset
|
500 |
list = _discoveredSoftRefs; |
623a5e42deb6
8173335: Improve logging for j.l.ref.reference processing
sangheki
parents:
43455
diff
changeset
|
501 |
break; |
623a5e42deb6
8173335: Improve logging for j.l.ref.reference processing
sangheki
parents:
43455
diff
changeset
|
502 |
case REF_WEAK: |
623a5e42deb6
8173335: Improve logging for j.l.ref.reference processing
sangheki
parents:
43455
diff
changeset
|
503 |
list = _discoveredWeakRefs; |
623a5e42deb6
8173335: Improve logging for j.l.ref.reference processing
sangheki
parents:
43455
diff
changeset
|
504 |
break; |
623a5e42deb6
8173335: Improve logging for j.l.ref.reference processing
sangheki
parents:
43455
diff
changeset
|
505 |
case REF_FINAL: |
623a5e42deb6
8173335: Improve logging for j.l.ref.reference processing
sangheki
parents:
43455
diff
changeset
|
506 |
list = _discoveredFinalRefs; |
623a5e42deb6
8173335: Improve logging for j.l.ref.reference processing
sangheki
parents:
43455
diff
changeset
|
507 |
break; |
623a5e42deb6
8173335: Improve logging for j.l.ref.reference processing
sangheki
parents:
43455
diff
changeset
|
508 |
case REF_PHANTOM: |
623a5e42deb6
8173335: Improve logging for j.l.ref.reference processing
sangheki
parents:
43455
diff
changeset
|
509 |
list = _discoveredPhantomRefs; |
623a5e42deb6
8173335: Improve logging for j.l.ref.reference processing
sangheki
parents:
43455
diff
changeset
|
510 |
break; |
623a5e42deb6
8173335: Improve logging for j.l.ref.reference processing
sangheki
parents:
43455
diff
changeset
|
511 |
case REF_OTHER: |
623a5e42deb6
8173335: Improve logging for j.l.ref.reference processing
sangheki
parents:
43455
diff
changeset
|
512 |
case REF_NONE: |
623a5e42deb6
8173335: Improve logging for j.l.ref.reference processing
sangheki
parents:
43455
diff
changeset
|
513 |
default: |
623a5e42deb6
8173335: Improve logging for j.l.ref.reference processing
sangheki
parents:
43455
diff
changeset
|
514 |
ShouldNotReachHere(); |
623a5e42deb6
8173335: Improve logging for j.l.ref.reference processing
sangheki
parents:
43455
diff
changeset
|
515 |
} |
623a5e42deb6
8173335: Improve logging for j.l.ref.reference processing
sangheki
parents:
43455
diff
changeset
|
516 |
return total_count(list); |
623a5e42deb6
8173335: Improve logging for j.l.ref.reference processing
sangheki
parents:
43455
diff
changeset
|
517 |
} |
623a5e42deb6
8173335: Improve logging for j.l.ref.reference processing
sangheki
parents:
43455
diff
changeset
|
518 |
|
50605
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
519 |
class RefProcPhase1Task : public AbstractRefProcTaskExecutor::ProcessTask { |
1 | 520 |
public: |
46795
623a5e42deb6
8173335: Improve logging for j.l.ref.reference processing
sangheki
parents:
43455
diff
changeset
|
521 |
RefProcPhase1Task(ReferenceProcessor& ref_processor, |
50605
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
522 |
ReferenceProcessorPhaseTimes* phase_times, |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
523 |
ReferencePolicy* policy) |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
524 |
: ProcessTask(ref_processor, true /* marks_oops_alive */, phase_times), |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
525 |
_policy(policy) { } |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
526 |
|
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
527 |
virtual void work(uint worker_id, |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
528 |
BoolObjectClosure& is_alive, |
1 | 529 |
OopClosure& keep_alive, |
530 |
VoidClosure& complete_gc) |
|
531 |
{ |
|
50605
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
532 |
RefProcSubPhasesWorkerTimeTracker tt(ReferenceProcessor::SoftRefSubPhase1, _phase_times, worker_id); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
533 |
size_t const removed = _ref_processor.process_soft_ref_reconsider_work(_ref_processor._discoveredSoftRefs[worker_id], |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
534 |
_policy, |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
535 |
&is_alive, |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
536 |
&keep_alive, |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
537 |
&complete_gc); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
538 |
_phase_times->add_ref_cleared(REF_SOFT, removed); |
1 | 539 |
} |
540 |
private: |
|
541 |
ReferencePolicy* _policy; |
|
542 |
}; |
|
543 |
||
544 |
class RefProcPhase2Task: public AbstractRefProcTaskExecutor::ProcessTask { |
|
50605
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
545 |
void run_phase2(uint worker_id, |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
546 |
DiscoveredList list[], |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
547 |
BoolObjectClosure& is_alive, |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
548 |
OopClosure& keep_alive, |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
549 |
bool do_enqueue_and_clear, |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
550 |
ReferenceType ref_type) { |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
551 |
size_t const removed = _ref_processor.process_soft_weak_final_refs_work(list[worker_id], |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
552 |
&is_alive, |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
553 |
&keep_alive, |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
554 |
do_enqueue_and_clear); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
555 |
_phase_times->add_ref_cleared(ref_type, removed); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
556 |
} |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
557 |
|
1 | 558 |
public: |
50605
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
559 |
RefProcPhase2Task(ReferenceProcessor& ref_processor, |
46795
623a5e42deb6
8173335: Improve logging for j.l.ref.reference processing
sangheki
parents:
43455
diff
changeset
|
560 |
ReferenceProcessorPhaseTimes* phase_times) |
50605
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
561 |
: ProcessTask(ref_processor, false /* marks_oops_alive */, phase_times) { } |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
562 |
|
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
563 |
virtual void work(uint worker_id, |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
564 |
BoolObjectClosure& is_alive, |
1 | 565 |
OopClosure& keep_alive, |
50605
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
566 |
VoidClosure& complete_gc) { |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
567 |
RefProcWorkerTimeTracker t(_phase_times->phase2_worker_time_sec(), worker_id); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
568 |
{ |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
569 |
RefProcSubPhasesWorkerTimeTracker tt(ReferenceProcessor::SoftRefSubPhase2, _phase_times, worker_id); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
570 |
run_phase2(worker_id, _ref_processor._discoveredSoftRefs, is_alive, keep_alive, true /* do_enqueue_and_clear */, REF_SOFT); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
571 |
} |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
572 |
{ |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
573 |
RefProcSubPhasesWorkerTimeTracker tt(ReferenceProcessor::WeakRefSubPhase2, _phase_times, worker_id); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
574 |
run_phase2(worker_id, _ref_processor._discoveredWeakRefs, is_alive, keep_alive, true /* do_enqueue_and_clear */, REF_WEAK); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
575 |
} |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
576 |
{ |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
577 |
RefProcSubPhasesWorkerTimeTracker tt(ReferenceProcessor::FinalRefSubPhase2, _phase_times, worker_id); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
578 |
run_phase2(worker_id, _ref_processor._discoveredFinalRefs, is_alive, keep_alive, false /* do_enqueue_and_clear */, REF_FINAL); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
579 |
} |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
580 |
// Close the reachable set; needed for collectors which keep_alive_closure do |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
581 |
// not immediately complete their work. |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
582 |
complete_gc.do_void(); |
1 | 583 |
} |
584 |
}; |
|
585 |
||
586 |
class RefProcPhase3Task: public AbstractRefProcTaskExecutor::ProcessTask { |
|
587 |
public: |
|
46795
623a5e42deb6
8173335: Improve logging for j.l.ref.reference processing
sangheki
parents:
43455
diff
changeset
|
588 |
RefProcPhase3Task(ReferenceProcessor& ref_processor, |
623a5e42deb6
8173335: Improve logging for j.l.ref.reference processing
sangheki
parents:
43455
diff
changeset
|
589 |
ReferenceProcessorPhaseTimes* phase_times) |
50605
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
590 |
: ProcessTask(ref_processor, true /* marks_oops_alive */, phase_times) { } |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
591 |
|
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
592 |
virtual void work(uint worker_id, |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
593 |
BoolObjectClosure& is_alive, |
1 | 594 |
OopClosure& keep_alive, |
595 |
VoidClosure& complete_gc) |
|
596 |
{ |
|
50605
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
597 |
RefProcSubPhasesWorkerTimeTracker tt(ReferenceProcessor::FinalRefSubPhase3, _phase_times, worker_id); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
598 |
_ref_processor.process_final_keep_alive_work(_ref_processor._discoveredFinalRefs[worker_id], &keep_alive, &complete_gc); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
599 |
} |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
600 |
}; |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
601 |
|
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
602 |
class RefProcPhase4Task: public AbstractRefProcTaskExecutor::ProcessTask { |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
603 |
public: |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
604 |
RefProcPhase4Task(ReferenceProcessor& ref_processor, |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
605 |
ReferenceProcessorPhaseTimes* phase_times) |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
606 |
: ProcessTask(ref_processor, false /* marks_oops_alive */, phase_times) { } |
46795
623a5e42deb6
8173335: Improve logging for j.l.ref.reference processing
sangheki
parents:
43455
diff
changeset
|
607 |
|
50605
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
608 |
virtual void work(uint worker_id, |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
609 |
BoolObjectClosure& is_alive, |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
610 |
OopClosure& keep_alive, |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
611 |
VoidClosure& complete_gc) |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
612 |
{ |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
613 |
RefProcSubPhasesWorkerTimeTracker tt(ReferenceProcessor::PhantomRefSubPhase4, _phase_times, worker_id); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
614 |
size_t const removed = _ref_processor.process_phantom_refs_work(_ref_processor._discoveredPhantomRefs[worker_id], |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
615 |
&is_alive, |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
616 |
&keep_alive, |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
617 |
&complete_gc); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
618 |
_phase_times->add_ref_cleared(REF_PHANTOM, removed); |
1 | 619 |
} |
620 |
}; |
|
621 |
||
50097
ed8a43d83fcc
8201491: G1 support for java.lang.ref.Reference precleaning
tschatzl
parents:
50095
diff
changeset
|
622 |
void ReferenceProcessor::log_reflist(const char* prefix, DiscoveredList list[], uint num_active_queues) { |
ed8a43d83fcc
8201491: G1 support for java.lang.ref.Reference precleaning
tschatzl
parents:
50095
diff
changeset
|
623 |
LogTarget(Trace, gc, ref) lt; |
ed8a43d83fcc
8201491: G1 support for java.lang.ref.Reference precleaning
tschatzl
parents:
50095
diff
changeset
|
624 |
|
ed8a43d83fcc
8201491: G1 support for java.lang.ref.Reference precleaning
tschatzl
parents:
50095
diff
changeset
|
625 |
if (!lt.is_enabled()) { |
ed8a43d83fcc
8201491: G1 support for java.lang.ref.Reference precleaning
tschatzl
parents:
50095
diff
changeset
|
626 |
return; |
ed8a43d83fcc
8201491: G1 support for java.lang.ref.Reference precleaning
tschatzl
parents:
50095
diff
changeset
|
627 |
} |
ed8a43d83fcc
8201491: G1 support for java.lang.ref.Reference precleaning
tschatzl
parents:
50095
diff
changeset
|
628 |
|
ed8a43d83fcc
8201491: G1 support for java.lang.ref.Reference precleaning
tschatzl
parents:
50095
diff
changeset
|
629 |
size_t total = 0; |
ed8a43d83fcc
8201491: G1 support for java.lang.ref.Reference precleaning
tschatzl
parents:
50095
diff
changeset
|
630 |
|
ed8a43d83fcc
8201491: G1 support for java.lang.ref.Reference precleaning
tschatzl
parents:
50095
diff
changeset
|
631 |
LogStream ls(lt); |
ed8a43d83fcc
8201491: G1 support for java.lang.ref.Reference precleaning
tschatzl
parents:
50095
diff
changeset
|
632 |
ls.print("%s", prefix); |
ed8a43d83fcc
8201491: G1 support for java.lang.ref.Reference precleaning
tschatzl
parents:
50095
diff
changeset
|
633 |
for (uint i = 0; i < num_active_queues; i++) { |
ed8a43d83fcc
8201491: G1 support for java.lang.ref.Reference precleaning
tschatzl
parents:
50095
diff
changeset
|
634 |
ls.print(SIZE_FORMAT " ", list[i].length()); |
ed8a43d83fcc
8201491: G1 support for java.lang.ref.Reference precleaning
tschatzl
parents:
50095
diff
changeset
|
635 |
total += list[i].length(); |
ed8a43d83fcc
8201491: G1 support for java.lang.ref.Reference precleaning
tschatzl
parents:
50095
diff
changeset
|
636 |
} |
ed8a43d83fcc
8201491: G1 support for java.lang.ref.Reference precleaning
tschatzl
parents:
50095
diff
changeset
|
637 |
ls.print_cr("(" SIZE_FORMAT ")", total); |
ed8a43d83fcc
8201491: G1 support for java.lang.ref.Reference precleaning
tschatzl
parents:
50095
diff
changeset
|
638 |
} |
ed8a43d83fcc
8201491: G1 support for java.lang.ref.Reference precleaning
tschatzl
parents:
50095
diff
changeset
|
639 |
|
35061 | 640 |
#ifndef PRODUCT |
50097
ed8a43d83fcc
8201491: G1 support for java.lang.ref.Reference precleaning
tschatzl
parents:
50095
diff
changeset
|
641 |
void ReferenceProcessor::log_reflist_counts(DiscoveredList ref_lists[], uint num_active_queues) { |
35061 | 642 |
if (!log_is_enabled(Trace, gc, ref)) { |
643 |
return; |
|
644 |
} |
|
645 |
||
50097
ed8a43d83fcc
8201491: G1 support for java.lang.ref.Reference precleaning
tschatzl
parents:
50095
diff
changeset
|
646 |
log_reflist("", ref_lists, num_active_queues); |
37165
ed7da23aaa16
8149343: assert(rp->num_q() == no_of_gc_workers) failed: sanity
jmasa
parents:
37106
diff
changeset
|
647 |
#ifdef ASSERT |
50097
ed8a43d83fcc
8201491: G1 support for java.lang.ref.Reference precleaning
tschatzl
parents:
50095
diff
changeset
|
648 |
for (uint i = num_active_queues; i < _max_num_queues; i++) { |
37165
ed7da23aaa16
8149343: assert(rp->num_q() == no_of_gc_workers) failed: sanity
jmasa
parents:
37106
diff
changeset
|
649 |
assert(ref_lists[i].length() == 0, SIZE_FORMAT " unexpected References in %u", |
ed7da23aaa16
8149343: assert(rp->num_q() == no_of_gc_workers) failed: sanity
jmasa
parents:
37106
diff
changeset
|
650 |
ref_lists[i].length(), i); |
ed7da23aaa16
8149343: assert(rp->num_q() == no_of_gc_workers) failed: sanity
jmasa
parents:
37106
diff
changeset
|
651 |
} |
ed7da23aaa16
8149343: assert(rp->num_q() == no_of_gc_workers) failed: sanity
jmasa
parents:
37106
diff
changeset
|
652 |
#endif |
35061 | 653 |
} |
654 |
#endif |
|
655 |
||
37165
ed7da23aaa16
8149343: assert(rp->num_q() == no_of_gc_workers) failed: sanity
jmasa
parents:
37106
diff
changeset
|
656 |
void ReferenceProcessor::set_active_mt_degree(uint v) { |
49967
672ded60a082
8202021: Improve variable naming in ReferenceProcesso
tschatzl
parents:
49964
diff
changeset
|
657 |
_num_queues = v; |
37165
ed7da23aaa16
8149343: assert(rp->num_q() == no_of_gc_workers) failed: sanity
jmasa
parents:
37106
diff
changeset
|
658 |
_next_id = 0; |
ed7da23aaa16
8149343: assert(rp->num_q() == no_of_gc_workers) failed: sanity
jmasa
parents:
37106
diff
changeset
|
659 |
} |
ed7da23aaa16
8149343: assert(rp->num_q() == no_of_gc_workers) failed: sanity
jmasa
parents:
37106
diff
changeset
|
660 |
|
50401
bf7eb61349d2
8203319: JDK-8201487 disabled too much queue balancing
kbarrett
parents:
50277
diff
changeset
|
661 |
bool ReferenceProcessor::need_balance_queues(DiscoveredList refs_lists[]) { |
bf7eb61349d2
8203319: JDK-8201487 disabled too much queue balancing
kbarrett
parents:
50277
diff
changeset
|
662 |
assert(_processing_is_mt, "why balance non-mt processing?"); |
bf7eb61349d2
8203319: JDK-8201487 disabled too much queue balancing
kbarrett
parents:
50277
diff
changeset
|
663 |
// _num_queues is the processing degree. Only list entries up to |
bf7eb61349d2
8203319: JDK-8201487 disabled too much queue balancing
kbarrett
parents:
50277
diff
changeset
|
664 |
// _num_queues will be processed, so any non-empty lists beyond |
bf7eb61349d2
8203319: JDK-8201487 disabled too much queue balancing
kbarrett
parents:
50277
diff
changeset
|
665 |
// that must be redistributed to lists in that range. Even if not |
bf7eb61349d2
8203319: JDK-8201487 disabled too much queue balancing
kbarrett
parents:
50277
diff
changeset
|
666 |
// needed for that, balancing may be desirable to eliminate poor |
bf7eb61349d2
8203319: JDK-8201487 disabled too much queue balancing
kbarrett
parents:
50277
diff
changeset
|
667 |
// distribution of references among the lists. |
bf7eb61349d2
8203319: JDK-8201487 disabled too much queue balancing
kbarrett
parents:
50277
diff
changeset
|
668 |
if (ParallelRefProcBalancingEnabled) { |
bf7eb61349d2
8203319: JDK-8201487 disabled too much queue balancing
kbarrett
parents:
50277
diff
changeset
|
669 |
return true; // Configuration says do it. |
bf7eb61349d2
8203319: JDK-8201487 disabled too much queue balancing
kbarrett
parents:
50277
diff
changeset
|
670 |
} else { |
bf7eb61349d2
8203319: JDK-8201487 disabled too much queue balancing
kbarrett
parents:
50277
diff
changeset
|
671 |
// Configuration says don't balance, but if there are non-empty |
bf7eb61349d2
8203319: JDK-8201487 disabled too much queue balancing
kbarrett
parents:
50277
diff
changeset
|
672 |
// lists beyond the processing degree, then must ignore the |
bf7eb61349d2
8203319: JDK-8201487 disabled too much queue balancing
kbarrett
parents:
50277
diff
changeset
|
673 |
// configuration and balance anyway. |
bf7eb61349d2
8203319: JDK-8201487 disabled too much queue balancing
kbarrett
parents:
50277
diff
changeset
|
674 |
for (uint i = _num_queues; i < _max_num_queues; ++i) { |
bf7eb61349d2
8203319: JDK-8201487 disabled too much queue balancing
kbarrett
parents:
50277
diff
changeset
|
675 |
if (!refs_lists[i].is_empty()) { |
bf7eb61349d2
8203319: JDK-8201487 disabled too much queue balancing
kbarrett
parents:
50277
diff
changeset
|
676 |
return true; // Must balance despite configuration. |
bf7eb61349d2
8203319: JDK-8201487 disabled too much queue balancing
kbarrett
parents:
50277
diff
changeset
|
677 |
} |
bf7eb61349d2
8203319: JDK-8201487 disabled too much queue balancing
kbarrett
parents:
50277
diff
changeset
|
678 |
} |
bf7eb61349d2
8203319: JDK-8201487 disabled too much queue balancing
kbarrett
parents:
50277
diff
changeset
|
679 |
return false; // Safe to obey configuration and not balance. |
bf7eb61349d2
8203319: JDK-8201487 disabled too much queue balancing
kbarrett
parents:
50277
diff
changeset
|
680 |
} |
bf7eb61349d2
8203319: JDK-8201487 disabled too much queue balancing
kbarrett
parents:
50277
diff
changeset
|
681 |
} |
bf7eb61349d2
8203319: JDK-8201487 disabled too much queue balancing
kbarrett
parents:
50277
diff
changeset
|
682 |
|
50605
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
683 |
void ReferenceProcessor::maybe_balance_queues(DiscoveredList refs_lists[]) { |
50606
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
684 |
assert(_processing_is_mt, "Should not call this otherwise"); |
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
685 |
if (need_balance_queues(refs_lists)) { |
50605
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
686 |
balance_queues(refs_lists); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
687 |
} |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
688 |
} |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
689 |
|
1 | 690 |
// Balances reference queues. |
6759
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
691 |
// Move entries from all queues[0, 1, ..., _max_num_q-1] to |
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
692 |
// queues[0, 1, ..., _num_q-1] because only the first _num_q |
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
693 |
// corresponding to the active workers will be processed. |
1 | 694 |
void ReferenceProcessor::balance_queues(DiscoveredList ref_lists[]) |
695 |
{ |
|
696 |
// calculate total length |
|
697 |
size_t total_refs = 0; |
|
35061 | 698 |
log_develop_trace(gc, ref)("Balance ref_lists "); |
6759
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
699 |
|
50097
ed8a43d83fcc
8201491: G1 support for java.lang.ref.Reference precleaning
tschatzl
parents:
50095
diff
changeset
|
700 |
log_reflist_counts(ref_lists, _max_num_queues); |
ed8a43d83fcc
8201491: G1 support for java.lang.ref.Reference precleaning
tschatzl
parents:
50095
diff
changeset
|
701 |
|
49967
672ded60a082
8202021: Improve variable naming in ReferenceProcesso
tschatzl
parents:
49964
diff
changeset
|
702 |
for (uint i = 0; i < _max_num_queues; ++i) { |
1 | 703 |
total_refs += ref_lists[i].length(); |
37165
ed7da23aaa16
8149343: assert(rp->num_q() == no_of_gc_workers) failed: sanity
jmasa
parents:
37106
diff
changeset
|
704 |
} |
49967
672ded60a082
8202021: Improve variable naming in ReferenceProcesso
tschatzl
parents:
49964
diff
changeset
|
705 |
size_t avg_refs = total_refs / _num_queues + 1; |
11396
917d8673b5ef
7121618: Change type of number of GC workers to unsigned int.
jmasa
parents:
11251
diff
changeset
|
706 |
uint to_idx = 0; |
49967
672ded60a082
8202021: Improve variable naming in ReferenceProcesso
tschatzl
parents:
49964
diff
changeset
|
707 |
for (uint from_idx = 0; from_idx < _max_num_queues; from_idx++) { |
6759
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
708 |
bool move_all = false; |
49967
672ded60a082
8202021: Improve variable naming in ReferenceProcesso
tschatzl
parents:
49964
diff
changeset
|
709 |
if (from_idx >= _num_queues) { |
6759
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
710 |
move_all = ref_lists[from_idx].length() > 0; |
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
711 |
} |
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
712 |
while ((ref_lists[from_idx].length() > avg_refs) || |
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
713 |
move_all) { |
49967
672ded60a082
8202021: Improve variable naming in ReferenceProcesso
tschatzl
parents:
49964
diff
changeset
|
714 |
assert(to_idx < _num_queues, "Sanity Check!"); |
1 | 715 |
if (ref_lists[to_idx].length() < avg_refs) { |
716 |
// move superfluous refs |
|
6759
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
717 |
size_t refs_to_move; |
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
718 |
// Move all the Ref's if the from queue will not be processed. |
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
719 |
if (move_all) { |
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
720 |
refs_to_move = MIN2(ref_lists[from_idx].length(), |
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
721 |
avg_refs - ref_lists[to_idx].length()); |
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
722 |
} else { |
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
723 |
refs_to_move = MIN2(ref_lists[from_idx].length() - avg_refs, |
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
724 |
avg_refs - ref_lists[to_idx].length()); |
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
725 |
} |
10524
6594ca81279a
7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents:
8688
diff
changeset
|
726 |
|
6594ca81279a
7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents:
8688
diff
changeset
|
727 |
assert(refs_to_move > 0, "otherwise the code below will fail"); |
6594ca81279a
7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents:
8688
diff
changeset
|
728 |
|
1 | 729 |
oop move_head = ref_lists[from_idx].head(); |
730 |
oop move_tail = move_head; |
|
731 |
oop new_head = move_head; |
|
732 |
// find an element to split the list on |
|
733 |
for (size_t j = 0; j < refs_to_move; ++j) { |
|
734 |
move_tail = new_head; |
|
360
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
735 |
new_head = java_lang_ref_Reference::discovered(new_head); |
1 | 736 |
} |
10524
6594ca81279a
7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents:
8688
diff
changeset
|
737 |
|
6594ca81279a
7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents:
8688
diff
changeset
|
738 |
// Add the chain to the to list. |
6594ca81279a
7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents:
8688
diff
changeset
|
739 |
if (ref_lists[to_idx].head() == NULL) { |
6594ca81279a
7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents:
8688
diff
changeset
|
740 |
// to list is empty. Make a loop at the end. |
24845
e8b8894a77df
8043239: G1: Missing post barrier in processing of j.l.ref.Reference objects
brutisso
parents:
24424
diff
changeset
|
741 |
java_lang_ref_Reference::set_discovered_raw(move_tail, move_tail); |
10524
6594ca81279a
7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents:
8688
diff
changeset
|
742 |
} else { |
24845
e8b8894a77df
8043239: G1: Missing post barrier in processing of j.l.ref.Reference objects
brutisso
parents:
24424
diff
changeset
|
743 |
java_lang_ref_Reference::set_discovered_raw(move_tail, ref_lists[to_idx].head()); |
10524
6594ca81279a
7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents:
8688
diff
changeset
|
744 |
} |
1 | 745 |
ref_lists[to_idx].set_head(move_head); |
1605
6b43d186eb8d
6722113: CMS: Incorrect overflow handling during precleaning of Reference lists
ysr
parents:
1388
diff
changeset
|
746 |
ref_lists[to_idx].inc_length(refs_to_move); |
10524
6594ca81279a
7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents:
8688
diff
changeset
|
747 |
|
6594ca81279a
7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents:
8688
diff
changeset
|
748 |
// Remove the chain from the from list. |
52070 | 749 |
if (oopDesc::equals_raw(move_tail, new_head)) { |
10524
6594ca81279a
7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents:
8688
diff
changeset
|
750 |
// We found the end of the from list. |
6594ca81279a
7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents:
8688
diff
changeset
|
751 |
ref_lists[from_idx].set_head(NULL); |
6594ca81279a
7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents:
8688
diff
changeset
|
752 |
} else { |
6594ca81279a
7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents:
8688
diff
changeset
|
753 |
ref_lists[from_idx].set_head(new_head); |
6594ca81279a
7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents:
8688
diff
changeset
|
754 |
} |
1605
6b43d186eb8d
6722113: CMS: Incorrect overflow handling during precleaning of Reference lists
ysr
parents:
1388
diff
changeset
|
755 |
ref_lists[from_idx].dec_length(refs_to_move); |
6759
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
756 |
if (ref_lists[from_idx].length() == 0) { |
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
757 |
break; |
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
758 |
} |
1 | 759 |
} else { |
49967
672ded60a082
8202021: Improve variable naming in ReferenceProcesso
tschatzl
parents:
49964
diff
changeset
|
760 |
to_idx = (to_idx + 1) % _num_queues; |
1 | 761 |
} |
762 |
} |
|
763 |
} |
|
6759
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
764 |
#ifdef ASSERT |
50097
ed8a43d83fcc
8201491: G1 support for java.lang.ref.Reference precleaning
tschatzl
parents:
50095
diff
changeset
|
765 |
log_reflist_counts(ref_lists, _num_queues); |
6759
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
766 |
size_t balanced_total_refs = 0; |
49967
672ded60a082
8202021: Improve variable naming in ReferenceProcesso
tschatzl
parents:
49964
diff
changeset
|
767 |
for (uint i = 0; i < _num_queues; ++i) { |
6759
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
768 |
balanced_total_refs += ref_lists[i].length(); |
37165
ed7da23aaa16
8149343: assert(rp->num_q() == no_of_gc_workers) failed: sanity
jmasa
parents:
37106
diff
changeset
|
769 |
} |
6759
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
770 |
assert(total_refs == balanced_total_refs, "Balancing was incomplete"); |
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
771 |
#endif |
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
772 |
} |
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
773 |
|
50606
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
774 |
bool ReferenceProcessor::is_mt_processing_set_up(AbstractRefProcTaskExecutor* task_executor) const { |
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
775 |
return task_executor != NULL && _processing_is_mt; |
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
776 |
} |
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
777 |
|
50605
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
778 |
void ReferenceProcessor::process_soft_ref_reconsider(BoolObjectClosure* is_alive, |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
779 |
OopClosure* keep_alive, |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
780 |
VoidClosure* complete_gc, |
50606
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
781 |
AbstractRefProcTaskExecutor* task_executor, |
50605
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
782 |
ReferenceProcessorPhaseTimes* phase_times) { |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
783 |
assert(!_processing_is_mt || task_executor != NULL, "Task executor must not be NULL when mt processing is set."); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
784 |
|
50606
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
785 |
size_t const num_soft_refs = total_count(_discoveredSoftRefs); |
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
786 |
phase_times->set_ref_discovered(REF_SOFT, num_soft_refs); |
50605
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
787 |
|
50606
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
788 |
phase_times->set_processing_is_mt(_processing_is_mt); |
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
789 |
|
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
790 |
if (num_soft_refs == 0 || _current_soft_ref_policy == NULL) { |
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
791 |
log_debug(gc, ref)("Skipped phase1 of Reference Processing due to unavailable references"); |
50605
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
792 |
return; |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
793 |
} |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
794 |
|
50606
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
795 |
RefProcMTDegreeAdjuster a(this, RefPhase1, num_soft_refs); |
50605
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
796 |
|
50606
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
797 |
if (_processing_is_mt) { |
50605
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
798 |
RefProcBalanceQueuesTimeTracker tt(RefPhase1, phase_times); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
799 |
maybe_balance_queues(_discoveredSoftRefs); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
800 |
} |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
801 |
|
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
802 |
RefProcPhaseTimeTracker tt(RefPhase1, phase_times); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
803 |
|
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
804 |
log_reflist("Phase1 Soft before", _discoveredSoftRefs, _max_num_queues); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
805 |
if (_processing_is_mt) { |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
806 |
RefProcPhase1Task phase1(*this, phase_times, _current_soft_ref_policy); |
50606
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
807 |
task_executor->execute(phase1, num_queues()); |
50605
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
808 |
} else { |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
809 |
size_t removed = 0; |
46795
623a5e42deb6
8173335: Improve logging for j.l.ref.reference processing
sangheki
parents:
43455
diff
changeset
|
810 |
|
50605
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
811 |
RefProcSubPhasesWorkerTimeTracker tt2(SoftRefSubPhase1, phase_times, 0); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
812 |
for (uint i = 0; i < _max_num_queues; i++) { |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
813 |
removed += process_soft_ref_reconsider_work(_discoveredSoftRefs[i], _current_soft_ref_policy, |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
814 |
is_alive, keep_alive, complete_gc); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
815 |
} |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
816 |
|
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
817 |
phase_times->add_ref_cleared(REF_SOFT, removed); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
818 |
} |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
819 |
log_reflist("Phase1 Soft after", _discoveredSoftRefs, _max_num_queues); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
820 |
} |
46795
623a5e42deb6
8173335: Improve logging for j.l.ref.reference processing
sangheki
parents:
43455
diff
changeset
|
821 |
|
50605
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
822 |
void ReferenceProcessor::process_soft_weak_final_refs(BoolObjectClosure* is_alive, |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
823 |
OopClosure* keep_alive, |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
824 |
VoidClosure* complete_gc, |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
825 |
AbstractRefProcTaskExecutor* task_executor, |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
826 |
ReferenceProcessorPhaseTimes* phase_times) { |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
827 |
assert(!_processing_is_mt || task_executor != NULL, "Task executor must not be NULL when mt processing is set."); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
828 |
|
50606
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
829 |
size_t const num_soft_refs = total_count(_discoveredSoftRefs); |
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
830 |
size_t const num_weak_refs = total_count(_discoveredWeakRefs); |
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
831 |
size_t const num_final_refs = total_count(_discoveredFinalRefs); |
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
832 |
size_t const num_total_refs = num_soft_refs + num_weak_refs + num_final_refs; |
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
833 |
phase_times->set_ref_discovered(REF_WEAK, num_weak_refs); |
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
834 |
phase_times->set_ref_discovered(REF_FINAL, num_final_refs); |
50605
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
835 |
|
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
836 |
phase_times->set_processing_is_mt(_processing_is_mt); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
837 |
|
50606
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
838 |
if (num_total_refs == 0) { |
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
839 |
log_debug(gc, ref)("Skipped phase2 of Reference Processing due to unavailable references"); |
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
840 |
return; |
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
841 |
} |
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
842 |
|
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
843 |
RefProcMTDegreeAdjuster a(this, RefPhase2, num_total_refs); |
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
844 |
|
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
845 |
if (_processing_is_mt) { |
50605
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
846 |
RefProcBalanceQueuesTimeTracker tt(RefPhase2, phase_times); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
847 |
maybe_balance_queues(_discoveredSoftRefs); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
848 |
maybe_balance_queues(_discoveredWeakRefs); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
849 |
maybe_balance_queues(_discoveredFinalRefs); |
1 | 850 |
} |
18025 | 851 |
|
50605
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
852 |
RefProcPhaseTimeTracker tt(RefPhase2, phase_times); |
46795
623a5e42deb6
8173335: Improve logging for j.l.ref.reference processing
sangheki
parents:
43455
diff
changeset
|
853 |
|
50605
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
854 |
log_reflist("Phase2 Soft before", _discoveredSoftRefs, _max_num_queues); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
855 |
log_reflist("Phase2 Weak before", _discoveredWeakRefs, _max_num_queues); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
856 |
log_reflist("Phase2 Final before", _discoveredFinalRefs, _max_num_queues); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
857 |
if (_processing_is_mt) { |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
858 |
RefProcPhase2Task phase2(*this, phase_times); |
50606
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
859 |
task_executor->execute(phase2, num_queues()); |
50605
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
860 |
} else { |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
861 |
RefProcWorkerTimeTracker t(phase_times->phase2_worker_time_sec(), 0); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
862 |
{ |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
863 |
size_t removed = 0; |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
864 |
|
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
865 |
RefProcSubPhasesWorkerTimeTracker tt2(SoftRefSubPhase2, phase_times, 0); |
49967
672ded60a082
8202021: Improve variable naming in ReferenceProcesso
tschatzl
parents:
49964
diff
changeset
|
866 |
for (uint i = 0; i < _max_num_queues; i++) { |
50605
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
867 |
removed += process_soft_weak_final_refs_work(_discoveredSoftRefs[i], is_alive, keep_alive, true /* do_enqueue */); |
1 | 868 |
} |
50605
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
869 |
|
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
870 |
phase_times->add_ref_cleared(REF_SOFT, removed); |
1 | 871 |
} |
50605
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
872 |
{ |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
873 |
size_t removed = 0; |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
874 |
|
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
875 |
RefProcSubPhasesWorkerTimeTracker tt2(WeakRefSubPhase2, phase_times, 0); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
876 |
for (uint i = 0; i < _max_num_queues; i++) { |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
877 |
removed += process_soft_weak_final_refs_work(_discoveredWeakRefs[i], is_alive, keep_alive, true /* do_enqueue */); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
878 |
} |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
879 |
|
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
880 |
phase_times->add_ref_cleared(REF_WEAK, removed); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
881 |
} |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
882 |
{ |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
883 |
size_t removed = 0; |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
884 |
|
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
885 |
RefProcSubPhasesWorkerTimeTracker tt2(FinalRefSubPhase2, phase_times, 0); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
886 |
for (uint i = 0; i < _max_num_queues; i++) { |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
887 |
removed += process_soft_weak_final_refs_work(_discoveredFinalRefs[i], is_alive, keep_alive, false /* do_enqueue */); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
888 |
} |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
889 |
|
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
890 |
phase_times->add_ref_cleared(REF_FINAL, removed); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
891 |
} |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
892 |
complete_gc->do_void(); |
1 | 893 |
} |
50605
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
894 |
verify_total_count_zero(_discoveredSoftRefs, "SoftReference"); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
895 |
verify_total_count_zero(_discoveredWeakRefs, "WeakReference"); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
896 |
log_reflist("Phase2 Final after", _discoveredFinalRefs, _max_num_queues); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
897 |
} |
1 | 898 |
|
50605
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
899 |
void ReferenceProcessor::process_final_keep_alive(OopClosure* keep_alive, |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
900 |
VoidClosure* complete_gc, |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
901 |
AbstractRefProcTaskExecutor* task_executor, |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
902 |
ReferenceProcessorPhaseTimes* phase_times) { |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
903 |
assert(!_processing_is_mt || task_executor != NULL, "Task executor must not be NULL when mt processing is set."); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
904 |
|
50606
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
905 |
size_t const num_final_refs = total_count(_discoveredFinalRefs); |
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
906 |
|
50605
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
907 |
phase_times->set_processing_is_mt(_processing_is_mt); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
908 |
|
50606
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
909 |
if (num_final_refs == 0) { |
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
910 |
log_debug(gc, ref)("Skipped phase3 of Reference Processing due to unavailable references"); |
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
911 |
return; |
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
912 |
} |
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
913 |
|
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
914 |
RefProcMTDegreeAdjuster a(this, RefPhase3, num_final_refs); |
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
915 |
|
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
916 |
if (_processing_is_mt) { |
50605
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
917 |
RefProcBalanceQueuesTimeTracker tt(RefPhase3, phase_times); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
918 |
maybe_balance_queues(_discoveredFinalRefs); |
1 | 919 |
} |
920 |
||
921 |
// Phase 3: |
|
50605
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
922 |
// . Traverse referents of final references and keep them and followers alive. |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
923 |
RefProcPhaseTimeTracker tt(RefPhase3, phase_times); |
46795
623a5e42deb6
8173335: Improve logging for j.l.ref.reference processing
sangheki
parents:
43455
diff
changeset
|
924 |
|
50605
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
925 |
if (_processing_is_mt) { |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
926 |
RefProcPhase3Task phase3(*this, phase_times); |
50606
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
927 |
task_executor->execute(phase3, num_queues()); |
50605
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
928 |
} else { |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
929 |
RefProcSubPhasesWorkerTimeTracker tt2(FinalRefSubPhase3, phase_times, 0); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
930 |
for (uint i = 0; i < _max_num_queues; i++) { |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
931 |
process_final_keep_alive_work(_discoveredFinalRefs[i], keep_alive, complete_gc); |
1 | 932 |
} |
933 |
} |
|
50605
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
934 |
verify_total_count_zero(_discoveredFinalRefs, "FinalReference"); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
935 |
} |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
936 |
|
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
937 |
void ReferenceProcessor::process_phantom_refs(BoolObjectClosure* is_alive, |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
938 |
OopClosure* keep_alive, |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
939 |
VoidClosure* complete_gc, |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
940 |
AbstractRefProcTaskExecutor* task_executor, |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
941 |
ReferenceProcessorPhaseTimes* phase_times) { |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
942 |
assert(!_processing_is_mt || task_executor != NULL, "Task executor must not be NULL when mt processing is set."); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
943 |
|
50606
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
944 |
size_t const num_phantom_refs = total_count(_discoveredPhantomRefs); |
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
945 |
phase_times->set_ref_discovered(REF_PHANTOM, num_phantom_refs); |
50605
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
946 |
|
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
947 |
phase_times->set_processing_is_mt(_processing_is_mt); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
948 |
|
50606
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
949 |
if (num_phantom_refs == 0) { |
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
950 |
log_debug(gc, ref)("Skipped phase4 of Reference Processing due to unavailable references"); |
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
951 |
return; |
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
952 |
} |
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
953 |
|
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
954 |
RefProcMTDegreeAdjuster a(this, RefPhase4, num_phantom_refs); |
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
955 |
|
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
956 |
if (_processing_is_mt) { |
50605
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
957 |
RefProcBalanceQueuesTimeTracker tt(RefPhase4, phase_times); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
958 |
maybe_balance_queues(_discoveredPhantomRefs); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
959 |
} |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
960 |
|
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
961 |
// Phase 4: Walk phantom references appropriately. |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
962 |
RefProcPhaseTimeTracker tt(RefPhase4, phase_times); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
963 |
|
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
964 |
log_reflist("Phase4 Phantom before", _discoveredPhantomRefs, _max_num_queues); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
965 |
if (_processing_is_mt) { |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
966 |
RefProcPhase4Task phase4(*this, phase_times); |
50606
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
967 |
task_executor->execute(phase4, num_queues()); |
50605
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
968 |
} else { |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
969 |
size_t removed = 0; |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
970 |
|
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
971 |
RefProcSubPhasesWorkerTimeTracker tt(PhantomRefSubPhase4, phase_times, 0); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
972 |
for (uint i = 0; i < _max_num_queues; i++) { |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
973 |
removed += process_phantom_refs_work(_discoveredPhantomRefs[i], is_alive, keep_alive, complete_gc); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
974 |
} |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
975 |
|
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
976 |
phase_times->add_ref_cleared(REF_PHANTOM, removed); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
977 |
} |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
978 |
verify_total_count_zero(_discoveredPhantomRefs, "PhantomReference"); |
1 | 979 |
} |
980 |
||
981 |
inline DiscoveredList* ReferenceProcessor::get_discovered_list(ReferenceType rt) { |
|
11396
917d8673b5ef
7121618: Change type of number of GC workers to unsigned int.
jmasa
parents:
11251
diff
changeset
|
982 |
uint id = 0; |
1 | 983 |
// Determine the queue index to use for this object. |
984 |
if (_discovery_is_mt) { |
|
985 |
// During a multi-threaded discovery phase, |
|
986 |
// each thread saves to its "own" list. |
|
987 |
Thread* thr = Thread::current(); |
|
7399
4ecd771fa2d1
6983204: G1: Nightly test nsk/regression/b4958615 failing with +ExplicitGCInvokesConcurrent
johnc
parents:
7397
diff
changeset
|
988 |
id = thr->as_Worker_thread()->id(); |
1 | 989 |
} else { |
990 |
// single-threaded discovery, we save in round-robin |
|
991 |
// fashion to each of the lists. |
|
992 |
if (_processing_is_mt) { |
|
993 |
id = next_id(); |
|
994 |
} |
|
995 |
} |
|
50095 | 996 |
assert(id < _max_num_queues, "Id is out of bounds id %u and max id %u)", id, _max_num_queues); |
1 | 997 |
|
998 |
// Get the discovered queue to which we will add |
|
999 |
DiscoveredList* list = NULL; |
|
1000 |
switch (rt) { |
|
1001 |
case REF_OTHER: |
|
1002 |
// Unknown reference type, no special treatment |
|
1003 |
break; |
|
1004 |
case REF_SOFT: |
|
1005 |
list = &_discoveredSoftRefs[id]; |
|
1006 |
break; |
|
1007 |
case REF_WEAK: |
|
1008 |
list = &_discoveredWeakRefs[id]; |
|
1009 |
break; |
|
1010 |
case REF_FINAL: |
|
1011 |
list = &_discoveredFinalRefs[id]; |
|
1012 |
break; |
|
1013 |
case REF_PHANTOM: |
|
1014 |
list = &_discoveredPhantomRefs[id]; |
|
1015 |
break; |
|
1016 |
case REF_NONE: |
|
13738
d67be49a5beb
7195833: NPG: Rename instanceClassLoaderKlass, instanceRefKlass and instanceMirrorKlass
coleenp
parents:
13728
diff
changeset
|
1017 |
// we should not reach here if we are an InstanceRefKlass |
1 | 1018 |
default: |
32817
acc2744fd84b
8136627: Backout JDK-8133818 Additional number of processed references printed with -XX:+PrintReferenceGC after JDK-8047125
kbarrett
parents:
32623
diff
changeset
|
1019 |
ShouldNotReachHere(); |
1 | 1020 |
} |
35061 | 1021 |
log_develop_trace(gc, ref)("Thread %d gets list " INTPTR_FORMAT, id, p2i(list)); |
1 | 1022 |
return list; |
1023 |
} |
|
1024 |
||
360
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
1025 |
inline void |
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
1026 |
ReferenceProcessor::add_to_discovered_list_mt(DiscoveredList& refs_list, |
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
1027 |
oop obj, |
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
1028 |
HeapWord* discovered_addr) { |
1 | 1029 |
assert(_discovery_is_mt, "!_discovery_is_mt should have been handled by caller"); |
1030 |
// First we must make sure this object is only enqueued once. CAS in a non null |
|
1031 |
// discovered_addr. |
|
1374
4c24294029a9
6711316: Open source the Garbage-First garbage collector
ysr
parents:
360
diff
changeset
|
1032 |
oop current_head = refs_list.head(); |
10524
6594ca81279a
7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents:
8688
diff
changeset
|
1033 |
// The last ref must have its discovered field pointing to itself. |
6594ca81279a
7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents:
8688
diff
changeset
|
1034 |
oop next_discovered = (current_head != NULL) ? current_head : obj; |
1374
4c24294029a9
6711316: Open source the Garbage-First garbage collector
ysr
parents:
360
diff
changeset
|
1035 |
|
50920
d9160a3c97c1
8203848: Missing remembered set entry in j.l.ref.references after JDK-8203028
tschatzl
parents:
50606
diff
changeset
|
1036 |
oop retest = HeapAccess<AS_NO_KEEPALIVE>::oop_atomic_cmpxchg(next_discovered, discovered_addr, oop(NULL)); |
47998
fb0275c320a0
8189871: Refactor GC barriers to use declarative semantics
eosterlund
parents:
47648
diff
changeset
|
1037 |
|
1 | 1038 |
if (retest == NULL) { |
1039 |
// This thread just won the right to enqueue the object. |
|
10526
3e92f211533f
4965777: GC changes to support use of discovered field for pending references
ysr
parents:
10524
diff
changeset
|
1040 |
// We have separate lists for enqueueing, so no synchronization |
1 | 1041 |
// is necessary. |
360
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
1042 |
refs_list.set_head(obj); |
1605
6b43d186eb8d
6722113: CMS: Incorrect overflow handling during precleaning of Reference lists
ysr
parents:
1388
diff
changeset
|
1043 |
refs_list.inc_length(1); |
7399
4ecd771fa2d1
6983204: G1: Nightly test nsk/regression/b4958615 failing with +ExplicitGCInvokesConcurrent
johnc
parents:
7397
diff
changeset
|
1044 |
|
35061 | 1045 |
log_develop_trace(gc, ref)("Discovered reference (mt) (" INTPTR_FORMAT ": %s)", |
1046 |
p2i(obj), obj->klass()->internal_name()); |
|
1 | 1047 |
} else { |
1048 |
// If retest was non NULL, another thread beat us to it: |
|
1049 |
// The reference has already been discovered... |
|
35061 | 1050 |
log_develop_trace(gc, ref)("Already discovered reference (" INTPTR_FORMAT ": %s)", |
1051 |
p2i(obj), obj->klass()->internal_name()); |
|
1 | 1052 |
} |
47998
fb0275c320a0
8189871: Refactor GC barriers to use declarative semantics
eosterlund
parents:
47648
diff
changeset
|
1053 |
} |
1 | 1054 |
|
7420
24071b15dde6
7005259: CMS: BubbleUpRef asserts referent(obj)->is_oop() failed: Enqueued a bad referent
ysr
parents:
7399
diff
changeset
|
1055 |
#ifndef PRODUCT |
24071b15dde6
7005259: CMS: BubbleUpRef asserts referent(obj)->is_oop() failed: Enqueued a bad referent
ysr
parents:
7399
diff
changeset
|
1056 |
// Non-atomic (i.e. concurrent) discovery might allow us |
24071b15dde6
7005259: CMS: BubbleUpRef asserts referent(obj)->is_oop() failed: Enqueued a bad referent
ysr
parents:
7399
diff
changeset
|
1057 |
// to observe j.l.References with NULL referents, being those |
24071b15dde6
7005259: CMS: BubbleUpRef asserts referent(obj)->is_oop() failed: Enqueued a bad referent
ysr
parents:
7399
diff
changeset
|
1058 |
// cleared concurrently by mutators during (or after) discovery. |
24071b15dde6
7005259: CMS: BubbleUpRef asserts referent(obj)->is_oop() failed: Enqueued a bad referent
ysr
parents:
7399
diff
changeset
|
1059 |
void ReferenceProcessor::verify_referent(oop obj) { |
24071b15dde6
7005259: CMS: BubbleUpRef asserts referent(obj)->is_oop() failed: Enqueued a bad referent
ysr
parents:
7399
diff
changeset
|
1060 |
bool da = discovery_is_atomic(); |
24071b15dde6
7005259: CMS: BubbleUpRef asserts referent(obj)->is_oop() failed: Enqueued a bad referent
ysr
parents:
7399
diff
changeset
|
1061 |
oop referent = java_lang_ref_Reference::referent(obj); |
46968
9119841280f4
8160399: is_oop_or_null involves undefined behavior
coleenp
parents:
46795
diff
changeset
|
1062 |
assert(da ? oopDesc::is_oop(referent) : oopDesc::is_oop_or_null(referent), |
33105
294e48b4f704
8080775: Better argument formatting for assert() and friends
david
parents:
33103
diff
changeset
|
1063 |
"Bad referent " INTPTR_FORMAT " found in Reference " |
294e48b4f704
8080775: Better argument formatting for assert() and friends
david
parents:
33103
diff
changeset
|
1064 |
INTPTR_FORMAT " during %satomic discovery ", |
294e48b4f704
8080775: Better argument formatting for assert() and friends
david
parents:
33103
diff
changeset
|
1065 |
p2i(referent), p2i(obj), da ? "" : "non-"); |
7420
24071b15dde6
7005259: CMS: BubbleUpRef asserts referent(obj)->is_oop() failed: Enqueued a bad referent
ysr
parents:
7399
diff
changeset
|
1066 |
} |
24071b15dde6
7005259: CMS: BubbleUpRef asserts referent(obj)->is_oop() failed: Enqueued a bad referent
ysr
parents:
7399
diff
changeset
|
1067 |
#endif |
24071b15dde6
7005259: CMS: BubbleUpRef asserts referent(obj)->is_oop() failed: Enqueued a bad referent
ysr
parents:
7399
diff
changeset
|
1068 |
|
49964
99e698e94cc7
8201492: Properly implement non-contiguous generations for Reference discovery
tschatzl
parents:
49808
diff
changeset
|
1069 |
bool ReferenceProcessor::is_subject_to_discovery(oop const obj) const { |
99e698e94cc7
8201492: Properly implement non-contiguous generations for Reference discovery
tschatzl
parents:
49808
diff
changeset
|
1070 |
return _is_subject_to_discovery->do_object_b(obj); |
99e698e94cc7
8201492: Properly implement non-contiguous generations for Reference discovery
tschatzl
parents:
49808
diff
changeset
|
1071 |
} |
99e698e94cc7
8201492: Properly implement non-contiguous generations for Reference discovery
tschatzl
parents:
49808
diff
changeset
|
1072 |
|
1 | 1073 |
// We mention two of several possible choices here: |
1074 |
// #0: if the reference object is not in the "originating generation" |
|
1075 |
// (or part of the heap being collected, indicated by our "span" |
|
1076 |
// we don't treat it specially (i.e. we scan it as we would |
|
1077 |
// a normal oop, treating its references as strong references). |
|
10526
3e92f211533f
4965777: GC changes to support use of discovered field for pending references
ysr
parents:
10524
diff
changeset
|
1078 |
// This means that references can't be discovered unless their |
1 | 1079 |
// referent is also in the same span. This is the simplest, |
1080 |
// most "local" and most conservative approach, albeit one |
|
1081 |
// that may cause weak references to be enqueued least promptly. |
|
1082 |
// We call this choice the "ReferenceBasedDiscovery" policy. |
|
1083 |
// #1: the reference object may be in any generation (span), but if |
|
1084 |
// the referent is in the generation (span) being currently collected |
|
1085 |
// then we can discover the reference object, provided |
|
1086 |
// the object has not already been discovered by |
|
1087 |
// a different concurrently running collector (as may be the |
|
1088 |
// case, for instance, if the reference object is in CMS and |
|
1089 |
// the referent in DefNewGeneration), and provided the processing |
|
1090 |
// of this reference object by the current collector will |
|
1091 |
// appear atomic to every other collector in the system. |
|
1092 |
// (Thus, for instance, a concurrent collector may not |
|
1093 |
// discover references in other generations even if the |
|
1094 |
// referent is in its own generation). This policy may, |
|
1095 |
// in certain cases, enqueue references somewhat sooner than |
|
1096 |
// might Policy #0 above, but at marginally increased cost |
|
1097 |
// and complexity in processing these references. |
|
1098 |
// We call this choice the "RefeferentBasedDiscovery" policy. |
|
1099 |
bool ReferenceProcessor::discover_reference(oop obj, ReferenceType rt) { |
|
10526
3e92f211533f
4965777: GC changes to support use of discovered field for pending references
ysr
parents:
10524
diff
changeset
|
1100 |
// Make sure we are discovering refs (rather than processing discovered refs). |
1 | 1101 |
if (!_discovering_refs || !RegisterReferences) { |
1102 |
return false; |
|
1103 |
} |
|
50277
f84ae8aa5d88
8203028: Simplify reference processing in light of JDK-8175797
kbarrett
parents:
50097
diff
changeset
|
1104 |
|
f84ae8aa5d88
8203028: Simplify reference processing in light of JDK-8175797
kbarrett
parents:
50097
diff
changeset
|
1105 |
if ((rt == REF_FINAL) && (java_lang_ref_Reference::next(obj) != NULL)) { |
f84ae8aa5d88
8203028: Simplify reference processing in light of JDK-8175797
kbarrett
parents:
50097
diff
changeset
|
1106 |
// Don't rediscover non-active FinalReferences. |
1 | 1107 |
return false; |
1108 |
} |
|
1109 |
||
1110 |
if (RefDiscoveryPolicy == ReferenceBasedDiscovery && |
|
49964
99e698e94cc7
8201492: Properly implement non-contiguous generations for Reference discovery
tschatzl
parents:
49808
diff
changeset
|
1111 |
!is_subject_to_discovery(obj)) { |
1 | 1112 |
// Reference is not in the originating generation; |
1113 |
// don't treat it specially (i.e. we want to scan it as a normal |
|
1114 |
// object with strong references). |
|
1115 |
return false; |
|
1116 |
} |
|
1117 |
||
10526
3e92f211533f
4965777: GC changes to support use of discovered field for pending references
ysr
parents:
10524
diff
changeset
|
1118 |
// We only discover references whose referents are not (yet) |
3e92f211533f
4965777: GC changes to support use of discovered field for pending references
ysr
parents:
10524
diff
changeset
|
1119 |
// known to be strongly reachable. |
1 | 1120 |
if (is_alive_non_header() != NULL) { |
7420
24071b15dde6
7005259: CMS: BubbleUpRef asserts referent(obj)->is_oop() failed: Enqueued a bad referent
ysr
parents:
7399
diff
changeset
|
1121 |
verify_referent(obj); |
24071b15dde6
7005259: CMS: BubbleUpRef asserts referent(obj)->is_oop() failed: Enqueued a bad referent
ysr
parents:
7399
diff
changeset
|
1122 |
if (is_alive_non_header()->do_object_b(java_lang_ref_Reference::referent(obj))) { |
1 | 1123 |
return false; // referent is reachable |
1124 |
} |
|
1125 |
} |
|
1606
dcf9714addbe
6684579: SoftReference processing can be made more efficient
ysr
parents:
1605
diff
changeset
|
1126 |
if (rt == REF_SOFT) { |
dcf9714addbe
6684579: SoftReference processing can be made more efficient
ysr
parents:
1605
diff
changeset
|
1127 |
// For soft refs we can decide now if these are not |
dcf9714addbe
6684579: SoftReference processing can be made more efficient
ysr
parents:
1605
diff
changeset
|
1128 |
// current candidates for clearing, in which case we |
dcf9714addbe
6684579: SoftReference processing can be made more efficient
ysr
parents:
1605
diff
changeset
|
1129 |
// can mark through them now, rather than delaying that |
dcf9714addbe
6684579: SoftReference processing can be made more efficient
ysr
parents:
1605
diff
changeset
|
1130 |
// to the reference-processing phase. Since all current |
dcf9714addbe
6684579: SoftReference processing can be made more efficient
ysr
parents:
1605
diff
changeset
|
1131 |
// time-stamp policies advance the soft-ref clock only |
32623
390a27af5657
8134626: Misc cleanups after generation array removal
jwilhelm
parents:
32607
diff
changeset
|
1132 |
// at a full collection cycle, this is always currently |
1606
dcf9714addbe
6684579: SoftReference processing can be made more efficient
ysr
parents:
1605
diff
changeset
|
1133 |
// accurate. |
10683
4b5a5a507864
7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents:
10670
diff
changeset
|
1134 |
if (!_current_soft_ref_policy->should_clear_reference(obj, _soft_ref_timestamp_clock)) { |
1606
dcf9714addbe
6684579: SoftReference processing can be made more efficient
ysr
parents:
1605
diff
changeset
|
1135 |
return false; |
dcf9714addbe
6684579: SoftReference processing can be made more efficient
ysr
parents:
1605
diff
changeset
|
1136 |
} |
dcf9714addbe
6684579: SoftReference processing can be made more efficient
ysr
parents:
1605
diff
changeset
|
1137 |
} |
1 | 1138 |
|
10670
4ea0e7d2ffbc
6484982: G1: process references during evacuation pauses
johnc
parents:
10526
diff
changeset
|
1139 |
ResourceMark rm; // Needed for tracing. |
4ea0e7d2ffbc
6484982: G1: process references during evacuation pauses
johnc
parents:
10526
diff
changeset
|
1140 |
|
49041
44122f767467
8198286: Direct memory accessors in typeArrayOop.hpp should use Access API
eosterlund
parents:
47998
diff
changeset
|
1141 |
HeapWord* const discovered_addr = java_lang_ref_Reference::discovered_addr_raw(obj); |
1374
4c24294029a9
6711316: Open source the Garbage-First garbage collector
ysr
parents:
360
diff
changeset
|
1142 |
const oop discovered = java_lang_ref_Reference::discovered(obj); |
46968
9119841280f4
8160399: is_oop_or_null involves undefined behavior
coleenp
parents:
46795
diff
changeset
|
1143 |
assert(oopDesc::is_oop_or_null(discovered), "Expected an oop or NULL for discovered field at " PTR_FORMAT, p2i(discovered)); |
360
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
1144 |
if (discovered != NULL) { |
1 | 1145 |
// The reference has already been discovered... |
35061 | 1146 |
log_develop_trace(gc, ref)("Already discovered reference (" INTPTR_FORMAT ": %s)", |
1147 |
p2i(obj), obj->klass()->internal_name()); |
|
1 | 1148 |
if (RefDiscoveryPolicy == ReferentBasedDiscovery) { |
1149 |
// assumes that an object is not processed twice; |
|
1150 |
// if it's been already discovered it must be on another |
|
1151 |
// generation's discovered list; so we won't discover it. |
|
1152 |
return false; |
|
1153 |
} else { |
|
1154 |
assert(RefDiscoveryPolicy == ReferenceBasedDiscovery, |
|
1155 |
"Unrecognized policy"); |
|
1156 |
// Check assumption that an object is not potentially |
|
1157 |
// discovered twice except by concurrent collectors that potentially |
|
1158 |
// trace the same Reference object twice. |
|
7399
4ecd771fa2d1
6983204: G1: Nightly test nsk/regression/b4958615 failing with +ExplicitGCInvokesConcurrent
johnc
parents:
7397
diff
changeset
|
1159 |
assert(UseConcMarkSweepGC || UseG1GC, |
4ecd771fa2d1
6983204: G1: Nightly test nsk/regression/b4958615 failing with +ExplicitGCInvokesConcurrent
johnc
parents:
7397
diff
changeset
|
1160 |
"Only possible with a concurrent marking collector"); |
1 | 1161 |
return true; |
1162 |
} |
|
1163 |
} |
|
1164 |
||
1165 |
if (RefDiscoveryPolicy == ReferentBasedDiscovery) { |
|
7420
24071b15dde6
7005259: CMS: BubbleUpRef asserts referent(obj)->is_oop() failed: Enqueued a bad referent
ysr
parents:
7399
diff
changeset
|
1166 |
verify_referent(obj); |
10526
3e92f211533f
4965777: GC changes to support use of discovered field for pending references
ysr
parents:
10524
diff
changeset
|
1167 |
// Discover if and only if EITHER: |
3e92f211533f
4965777: GC changes to support use of discovered field for pending references
ysr
parents:
10524
diff
changeset
|
1168 |
// .. reference is in our span, OR |
3e92f211533f
4965777: GC changes to support use of discovered field for pending references
ysr
parents:
10524
diff
changeset
|
1169 |
// .. we are an atomic collector and referent is in our span |
49964
99e698e94cc7
8201492: Properly implement non-contiguous generations for Reference discovery
tschatzl
parents:
49808
diff
changeset
|
1170 |
if (is_subject_to_discovery(obj) || |
7420
24071b15dde6
7005259: CMS: BubbleUpRef asserts referent(obj)->is_oop() failed: Enqueued a bad referent
ysr
parents:
7399
diff
changeset
|
1171 |
(discovery_is_atomic() && |
49964
99e698e94cc7
8201492: Properly implement non-contiguous generations for Reference discovery
tschatzl
parents:
49808
diff
changeset
|
1172 |
is_subject_to_discovery(java_lang_ref_Reference::referent(obj)))) { |
1 | 1173 |
} else { |
1174 |
return false; |
|
1175 |
} |
|
1176 |
} else { |
|
1177 |
assert(RefDiscoveryPolicy == ReferenceBasedDiscovery && |
|
49964
99e698e94cc7
8201492: Properly implement non-contiguous generations for Reference discovery
tschatzl
parents:
49808
diff
changeset
|
1178 |
is_subject_to_discovery(obj), "code inconsistency"); |
1 | 1179 |
} |
1180 |
||
1181 |
// Get the right type of discovered queue head. |
|
1182 |
DiscoveredList* list = get_discovered_list(rt); |
|
1183 |
if (list == NULL) { |
|
1184 |
return false; // nothing special needs to be done |
|
1185 |
} |
|
1186 |
||
1187 |
if (_discovery_is_mt) { |
|
1188 |
add_to_discovered_list_mt(*list, obj, discovered_addr); |
|
1189 |
} else { |
|
24845
e8b8894a77df
8043239: G1: Missing post barrier in processing of j.l.ref.Reference objects
brutisso
parents:
24424
diff
changeset
|
1190 |
// We do a raw store here: the field will be visited later when processing |
e8b8894a77df
8043239: G1: Missing post barrier in processing of j.l.ref.Reference objects
brutisso
parents:
24424
diff
changeset
|
1191 |
// the discovered references. |
1374
4c24294029a9
6711316: Open source the Garbage-First garbage collector
ysr
parents:
360
diff
changeset
|
1192 |
oop current_head = list->head(); |
10524
6594ca81279a
7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents:
8688
diff
changeset
|
1193 |
// The last ref must have its discovered field pointing to itself. |
6594ca81279a
7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents:
8688
diff
changeset
|
1194 |
oop next_discovered = (current_head != NULL) ? current_head : obj; |
6594ca81279a
7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents:
8688
diff
changeset
|
1195 |
|
1374
4c24294029a9
6711316: Open source the Garbage-First garbage collector
ysr
parents:
360
diff
changeset
|
1196 |
assert(discovered == NULL, "control point invariant"); |
47998
fb0275c320a0
8189871: Refactor GC barriers to use declarative semantics
eosterlund
parents:
47648
diff
changeset
|
1197 |
RawAccess<>::oop_store(discovered_addr, next_discovered); |
1 | 1198 |
list->set_head(obj); |
1605
6b43d186eb8d
6722113: CMS: Incorrect overflow handling during precleaning of Reference lists
ysr
parents:
1388
diff
changeset
|
1199 |
list->inc_length(1); |
1 | 1200 |
|
35061 | 1201 |
log_develop_trace(gc, ref)("Discovered reference (" INTPTR_FORMAT ": %s)", p2i(obj), obj->klass()->internal_name()); |
1 | 1202 |
} |
46968
9119841280f4
8160399: is_oop_or_null involves undefined behavior
coleenp
parents:
46795
diff
changeset
|
1203 |
assert(oopDesc::is_oop(obj), "Discovered a bad reference"); |
7420
24071b15dde6
7005259: CMS: BubbleUpRef asserts referent(obj)->is_oop() failed: Enqueued a bad referent
ysr
parents:
7399
diff
changeset
|
1204 |
verify_referent(obj); |
1 | 1205 |
return true; |
1206 |
} |
|
1207 |
||
37494
bf6caf8e99cb
8153745: Avoid spawning G1ParPreserveCMReferentsTask when there is no work to be done
sjohanss
parents:
37248
diff
changeset
|
1208 |
bool ReferenceProcessor::has_discovered_references() { |
49967
672ded60a082
8202021: Improve variable naming in ReferenceProcesso
tschatzl
parents:
49964
diff
changeset
|
1209 |
for (uint i = 0; i < _max_num_queues * number_of_subclasses_of_ref(); i++) { |
37494
bf6caf8e99cb
8153745: Avoid spawning G1ParPreserveCMReferentsTask when there is no work to be done
sjohanss
parents:
37248
diff
changeset
|
1210 |
if (!_discovered_refs[i].is_empty()) { |
bf6caf8e99cb
8153745: Avoid spawning G1ParPreserveCMReferentsTask when there is no work to be done
sjohanss
parents:
37248
diff
changeset
|
1211 |
return true; |
bf6caf8e99cb
8153745: Avoid spawning G1ParPreserveCMReferentsTask when there is no work to be done
sjohanss
parents:
37248
diff
changeset
|
1212 |
} |
bf6caf8e99cb
8153745: Avoid spawning G1ParPreserveCMReferentsTask when there is no work to be done
sjohanss
parents:
37248
diff
changeset
|
1213 |
} |
bf6caf8e99cb
8153745: Avoid spawning G1ParPreserveCMReferentsTask when there is no work to be done
sjohanss
parents:
37248
diff
changeset
|
1214 |
return false; |
bf6caf8e99cb
8153745: Avoid spawning G1ParPreserveCMReferentsTask when there is no work to be done
sjohanss
parents:
37248
diff
changeset
|
1215 |
} |
bf6caf8e99cb
8153745: Avoid spawning G1ParPreserveCMReferentsTask when there is no work to be done
sjohanss
parents:
37248
diff
changeset
|
1216 |
|
50097
ed8a43d83fcc
8201491: G1 support for java.lang.ref.Reference precleaning
tschatzl
parents:
50095
diff
changeset
|
1217 |
void ReferenceProcessor::preclean_discovered_references(BoolObjectClosure* is_alive, |
ed8a43d83fcc
8201491: G1 support for java.lang.ref.Reference precleaning
tschatzl
parents:
50095
diff
changeset
|
1218 |
OopClosure* keep_alive, |
ed8a43d83fcc
8201491: G1 support for java.lang.ref.Reference precleaning
tschatzl
parents:
50095
diff
changeset
|
1219 |
VoidClosure* complete_gc, |
ed8a43d83fcc
8201491: G1 support for java.lang.ref.Reference precleaning
tschatzl
parents:
50095
diff
changeset
|
1220 |
YieldClosure* yield, |
ed8a43d83fcc
8201491: G1 support for java.lang.ref.Reference precleaning
tschatzl
parents:
50095
diff
changeset
|
1221 |
GCTimer* gc_timer) { |
ed8a43d83fcc
8201491: G1 support for java.lang.ref.Reference precleaning
tschatzl
parents:
50095
diff
changeset
|
1222 |
// These lists can be handled here in any order and, indeed, concurrently. |
1 | 1223 |
|
1224 |
// Soft references |
|
1225 |
{ |
|
35061 | 1226 |
GCTraceTime(Debug, gc, ref) tm("Preclean SoftReferences", gc_timer); |
50097
ed8a43d83fcc
8201491: G1 support for java.lang.ref.Reference precleaning
tschatzl
parents:
50095
diff
changeset
|
1227 |
log_reflist("SoftRef before: ", _discoveredSoftRefs, _max_num_queues); |
49967
672ded60a082
8202021: Improve variable naming in ReferenceProcesso
tschatzl
parents:
49964
diff
changeset
|
1228 |
for (uint i = 0; i < _max_num_queues; i++) { |
1605
6b43d186eb8d
6722113: CMS: Incorrect overflow handling during precleaning of Reference lists
ysr
parents:
1388
diff
changeset
|
1229 |
if (yield->should_return()) { |
6b43d186eb8d
6722113: CMS: Incorrect overflow handling during precleaning of Reference lists
ysr
parents:
1388
diff
changeset
|
1230 |
return; |
6b43d186eb8d
6722113: CMS: Incorrect overflow handling during precleaning of Reference lists
ysr
parents:
1388
diff
changeset
|
1231 |
} |
50097
ed8a43d83fcc
8201491: G1 support for java.lang.ref.Reference precleaning
tschatzl
parents:
50095
diff
changeset
|
1232 |
if (preclean_discovered_reflist(_discoveredSoftRefs[i], is_alive, |
ed8a43d83fcc
8201491: G1 support for java.lang.ref.Reference precleaning
tschatzl
parents:
50095
diff
changeset
|
1233 |
keep_alive, complete_gc, yield)) { |
ed8a43d83fcc
8201491: G1 support for java.lang.ref.Reference precleaning
tschatzl
parents:
50095
diff
changeset
|
1234 |
log_reflist("SoftRef abort: ", _discoveredSoftRefs, _max_num_queues); |
ed8a43d83fcc
8201491: G1 support for java.lang.ref.Reference precleaning
tschatzl
parents:
50095
diff
changeset
|
1235 |
return; |
ed8a43d83fcc
8201491: G1 support for java.lang.ref.Reference precleaning
tschatzl
parents:
50095
diff
changeset
|
1236 |
} |
1 | 1237 |
} |
50097
ed8a43d83fcc
8201491: G1 support for java.lang.ref.Reference precleaning
tschatzl
parents:
50095
diff
changeset
|
1238 |
log_reflist("SoftRef after: ", _discoveredSoftRefs, _max_num_queues); |
1 | 1239 |
} |
1240 |
||
1241 |
// Weak references |
|
1242 |
{ |
|
35061 | 1243 |
GCTraceTime(Debug, gc, ref) tm("Preclean WeakReferences", gc_timer); |
50097
ed8a43d83fcc
8201491: G1 support for java.lang.ref.Reference precleaning
tschatzl
parents:
50095
diff
changeset
|
1244 |
log_reflist("WeakRef before: ", _discoveredWeakRefs, _max_num_queues); |
49967
672ded60a082
8202021: Improve variable naming in ReferenceProcesso
tschatzl
parents:
49964
diff
changeset
|
1245 |
for (uint i = 0; i < _max_num_queues; i++) { |
1605
6b43d186eb8d
6722113: CMS: Incorrect overflow handling during precleaning of Reference lists
ysr
parents:
1388
diff
changeset
|
1246 |
if (yield->should_return()) { |
6b43d186eb8d
6722113: CMS: Incorrect overflow handling during precleaning of Reference lists
ysr
parents:
1388
diff
changeset
|
1247 |
return; |
6b43d186eb8d
6722113: CMS: Incorrect overflow handling during precleaning of Reference lists
ysr
parents:
1388
diff
changeset
|
1248 |
} |
50097
ed8a43d83fcc
8201491: G1 support for java.lang.ref.Reference precleaning
tschatzl
parents:
50095
diff
changeset
|
1249 |
if (preclean_discovered_reflist(_discoveredWeakRefs[i], is_alive, |
ed8a43d83fcc
8201491: G1 support for java.lang.ref.Reference precleaning
tschatzl
parents:
50095
diff
changeset
|
1250 |
keep_alive, complete_gc, yield)) { |
ed8a43d83fcc
8201491: G1 support for java.lang.ref.Reference precleaning
tschatzl
parents:
50095
diff
changeset
|
1251 |
log_reflist("WeakRef abort: ", _discoveredWeakRefs, _max_num_queues); |
ed8a43d83fcc
8201491: G1 support for java.lang.ref.Reference precleaning
tschatzl
parents:
50095
diff
changeset
|
1252 |
return; |
ed8a43d83fcc
8201491: G1 support for java.lang.ref.Reference precleaning
tschatzl
parents:
50095
diff
changeset
|
1253 |
} |
1 | 1254 |
} |
50097
ed8a43d83fcc
8201491: G1 support for java.lang.ref.Reference precleaning
tschatzl
parents:
50095
diff
changeset
|
1255 |
log_reflist("WeakRef after: ", _discoveredWeakRefs, _max_num_queues); |
1 | 1256 |
} |
1257 |
||
1258 |
// Final references |
|
1259 |
{ |
|
35061 | 1260 |
GCTraceTime(Debug, gc, ref) tm("Preclean FinalReferences", gc_timer); |
50097
ed8a43d83fcc
8201491: G1 support for java.lang.ref.Reference precleaning
tschatzl
parents:
50095
diff
changeset
|
1261 |
log_reflist("FinalRef before: ", _discoveredFinalRefs, _max_num_queues); |
49967
672ded60a082
8202021: Improve variable naming in ReferenceProcesso
tschatzl
parents:
49964
diff
changeset
|
1262 |
for (uint i = 0; i < _max_num_queues; i++) { |
1605
6b43d186eb8d
6722113: CMS: Incorrect overflow handling during precleaning of Reference lists
ysr
parents:
1388
diff
changeset
|
1263 |
if (yield->should_return()) { |
6b43d186eb8d
6722113: CMS: Incorrect overflow handling during precleaning of Reference lists
ysr
parents:
1388
diff
changeset
|
1264 |
return; |
6b43d186eb8d
6722113: CMS: Incorrect overflow handling during precleaning of Reference lists
ysr
parents:
1388
diff
changeset
|
1265 |
} |
50097
ed8a43d83fcc
8201491: G1 support for java.lang.ref.Reference precleaning
tschatzl
parents:
50095
diff
changeset
|
1266 |
if (preclean_discovered_reflist(_discoveredFinalRefs[i], is_alive, |
ed8a43d83fcc
8201491: G1 support for java.lang.ref.Reference precleaning
tschatzl
parents:
50095
diff
changeset
|
1267 |
keep_alive, complete_gc, yield)) { |
ed8a43d83fcc
8201491: G1 support for java.lang.ref.Reference precleaning
tschatzl
parents:
50095
diff
changeset
|
1268 |
log_reflist("FinalRef abort: ", _discoveredFinalRefs, _max_num_queues); |
ed8a43d83fcc
8201491: G1 support for java.lang.ref.Reference precleaning
tschatzl
parents:
50095
diff
changeset
|
1269 |
return; |
ed8a43d83fcc
8201491: G1 support for java.lang.ref.Reference precleaning
tschatzl
parents:
50095
diff
changeset
|
1270 |
} |
1 | 1271 |
} |
50097
ed8a43d83fcc
8201491: G1 support for java.lang.ref.Reference precleaning
tschatzl
parents:
50095
diff
changeset
|
1272 |
log_reflist("FinalRef after: ", _discoveredFinalRefs, _max_num_queues); |
1 | 1273 |
} |
1274 |
||
1275 |
// Phantom references |
|
1276 |
{ |
|
35061 | 1277 |
GCTraceTime(Debug, gc, ref) tm("Preclean PhantomReferences", gc_timer); |
50097
ed8a43d83fcc
8201491: G1 support for java.lang.ref.Reference precleaning
tschatzl
parents:
50095
diff
changeset
|
1278 |
log_reflist("PhantomRef before: ", _discoveredPhantomRefs, _max_num_queues); |
49967
672ded60a082
8202021: Improve variable naming in ReferenceProcesso
tschatzl
parents:
49964
diff
changeset
|
1279 |
for (uint i = 0; i < _max_num_queues; i++) { |
1605
6b43d186eb8d
6722113: CMS: Incorrect overflow handling during precleaning of Reference lists
ysr
parents:
1388
diff
changeset
|
1280 |
if (yield->should_return()) { |
6b43d186eb8d
6722113: CMS: Incorrect overflow handling during precleaning of Reference lists
ysr
parents:
1388
diff
changeset
|
1281 |
return; |
6b43d186eb8d
6722113: CMS: Incorrect overflow handling during precleaning of Reference lists
ysr
parents:
1388
diff
changeset
|
1282 |
} |
50097
ed8a43d83fcc
8201491: G1 support for java.lang.ref.Reference precleaning
tschatzl
parents:
50095
diff
changeset
|
1283 |
if (preclean_discovered_reflist(_discoveredPhantomRefs[i], is_alive, |
ed8a43d83fcc
8201491: G1 support for java.lang.ref.Reference precleaning
tschatzl
parents:
50095
diff
changeset
|
1284 |
keep_alive, complete_gc, yield)) { |
ed8a43d83fcc
8201491: G1 support for java.lang.ref.Reference precleaning
tschatzl
parents:
50095
diff
changeset
|
1285 |
log_reflist("PhantomRef abort: ", _discoveredPhantomRefs, _max_num_queues); |
ed8a43d83fcc
8201491: G1 support for java.lang.ref.Reference precleaning
tschatzl
parents:
50095
diff
changeset
|
1286 |
return; |
ed8a43d83fcc
8201491: G1 support for java.lang.ref.Reference precleaning
tschatzl
parents:
50095
diff
changeset
|
1287 |
} |
1 | 1288 |
} |
50097
ed8a43d83fcc
8201491: G1 support for java.lang.ref.Reference precleaning
tschatzl
parents:
50095
diff
changeset
|
1289 |
log_reflist("PhantomRef after: ", _discoveredPhantomRefs, _max_num_queues); |
1 | 1290 |
} |
1291 |
} |
|
1292 |
||
1293 |
// Walk the given discovered ref list, and remove all reference objects |
|
1294 |
// whose referents are still alive, whose referents are NULL or which |
|
1605
6b43d186eb8d
6722113: CMS: Incorrect overflow handling during precleaning of Reference lists
ysr
parents:
1388
diff
changeset
|
1295 |
// are not active (have a non-NULL next field). NOTE: When we are |
6b43d186eb8d
6722113: CMS: Incorrect overflow handling during precleaning of Reference lists
ysr
parents:
1388
diff
changeset
|
1296 |
// thus precleaning the ref lists (which happens single-threaded today), |
22551 | 1297 |
// we do not disable refs discovery to honor the correct semantics of |
1605
6b43d186eb8d
6722113: CMS: Incorrect overflow handling during precleaning of Reference lists
ysr
parents:
1388
diff
changeset
|
1298 |
// java.lang.Reference. As a result, we need to be careful below |
6b43d186eb8d
6722113: CMS: Incorrect overflow handling during precleaning of Reference lists
ysr
parents:
1388
diff
changeset
|
1299 |
// that ref removal steps interleave safely with ref discovery steps |
6b43d186eb8d
6722113: CMS: Incorrect overflow handling during precleaning of Reference lists
ysr
parents:
1388
diff
changeset
|
1300 |
// (in this thread). |
50097
ed8a43d83fcc
8201491: G1 support for java.lang.ref.Reference precleaning
tschatzl
parents:
50095
diff
changeset
|
1301 |
bool ReferenceProcessor::preclean_discovered_reflist(DiscoveredList& refs_list, |
ed8a43d83fcc
8201491: G1 support for java.lang.ref.Reference precleaning
tschatzl
parents:
50095
diff
changeset
|
1302 |
BoolObjectClosure* is_alive, |
ed8a43d83fcc
8201491: G1 support for java.lang.ref.Reference precleaning
tschatzl
parents:
50095
diff
changeset
|
1303 |
OopClosure* keep_alive, |
ed8a43d83fcc
8201491: G1 support for java.lang.ref.Reference precleaning
tschatzl
parents:
50095
diff
changeset
|
1304 |
VoidClosure* complete_gc, |
ed8a43d83fcc
8201491: G1 support for java.lang.ref.Reference precleaning
tschatzl
parents:
50095
diff
changeset
|
1305 |
YieldClosure* yield) { |
24845
e8b8894a77df
8043239: G1: Missing post barrier in processing of j.l.ref.Reference objects
brutisso
parents:
24424
diff
changeset
|
1306 |
DiscoveredListIterator iter(refs_list, keep_alive, is_alive); |
1 | 1307 |
while (iter.has_next()) { |
50097
ed8a43d83fcc
8201491: G1 support for java.lang.ref.Reference precleaning
tschatzl
parents:
50095
diff
changeset
|
1308 |
if (yield->should_return_fine_grain()) { |
ed8a43d83fcc
8201491: G1 support for java.lang.ref.Reference precleaning
tschatzl
parents:
50095
diff
changeset
|
1309 |
return true; |
ed8a43d83fcc
8201491: G1 support for java.lang.ref.Reference precleaning
tschatzl
parents:
50095
diff
changeset
|
1310 |
} |
1 | 1311 |
iter.load_ptrs(DEBUG_ONLY(true /* allow_null_referent */)); |
50277
f84ae8aa5d88
8203028: Simplify reference processing in light of JDK-8175797
kbarrett
parents:
50097
diff
changeset
|
1312 |
if (iter.referent() == NULL || iter.is_referent_alive()) { |
f84ae8aa5d88
8203028: Simplify reference processing in light of JDK-8175797
kbarrett
parents:
50097
diff
changeset
|
1313 |
// The referent has been cleared, or is alive; we need to trace |
f84ae8aa5d88
8203028: Simplify reference processing in light of JDK-8175797
kbarrett
parents:
50097
diff
changeset
|
1314 |
// and mark its cohort. |
35061 | 1315 |
log_develop_trace(gc, ref)("Precleaning Reference (" INTPTR_FORMAT ": %s)", |
1316 |
p2i(iter.obj()), iter.obj()->klass()->internal_name()); |
|
1 | 1317 |
// Remove Reference object from list |
1318 |
iter.remove(); |
|
1319 |
// Keep alive its cohort. |
|
1320 |
iter.make_referent_alive(); |
|
1605
6b43d186eb8d
6722113: CMS: Incorrect overflow handling during precleaning of Reference lists
ysr
parents:
1388
diff
changeset
|
1321 |
iter.move_to_next(); |
1 | 1322 |
} else { |
1323 |
iter.next(); |
|
1324 |
} |
|
1325 |
} |
|
1326 |
// Close the reachable set |
|
1327 |
complete_gc->do_void(); |
|
1328 |
||
50605
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
1329 |
if (iter.processed() > 0) { |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
1330 |
log_develop_trace(gc, ref)(" Dropped " SIZE_FORMAT " Refs out of " SIZE_FORMAT " Refs in discovered list " INTPTR_FORMAT, |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
1331 |
iter.removed(), iter.processed(), p2i(&refs_list)); |
7f63c74f0974
8202845: Refactor reference processing for improved parallelism
tschatzl
parents:
50401
diff
changeset
|
1332 |
} |
50097
ed8a43d83fcc
8201491: G1 support for java.lang.ref.Reference precleaning
tschatzl
parents:
50095
diff
changeset
|
1333 |
return false; |
1 | 1334 |
} |
1335 |
||
11396
917d8673b5ef
7121618: Change type of number of GC workers to unsigned int.
jmasa
parents:
11251
diff
changeset
|
1336 |
const char* ReferenceProcessor::list_name(uint i) { |
49967
672ded60a082
8202021: Improve variable naming in ReferenceProcesso
tschatzl
parents:
49964
diff
changeset
|
1337 |
assert(i <= _max_num_queues * number_of_subclasses_of_ref(), |
10670
4ea0e7d2ffbc
6484982: G1: process references during evacuation pauses
johnc
parents:
10526
diff
changeset
|
1338 |
"Out of bounds index"); |
4ea0e7d2ffbc
6484982: G1: process references during evacuation pauses
johnc
parents:
10526
diff
changeset
|
1339 |
|
49967
672ded60a082
8202021: Improve variable naming in ReferenceProcesso
tschatzl
parents:
49964
diff
changeset
|
1340 |
int j = i / _max_num_queues; |
1 | 1341 |
switch (j) { |
1342 |
case 0: return "SoftRef"; |
|
1343 |
case 1: return "WeakRef"; |
|
1344 |
case 2: return "FinalRef"; |
|
1345 |
case 3: return "PhantomRef"; |
|
1346 |
} |
|
1347 |
ShouldNotReachHere(); |
|
1348 |
return NULL; |
|
1349 |
} |
|
50606
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
1350 |
|
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
1351 |
uint RefProcMTDegreeAdjuster::ergo_proc_thread_count(size_t ref_count, |
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
1352 |
uint max_threads, |
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
1353 |
RefProcPhases phase) const { |
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
1354 |
assert(0 < max_threads, "must allow at least one thread"); |
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
1355 |
|
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
1356 |
if (use_max_threads(phase) || (ReferencesPerThread == 0)) { |
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
1357 |
return max_threads; |
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
1358 |
} |
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
1359 |
|
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
1360 |
size_t thread_count = 1 + (ref_count / ReferencesPerThread); |
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
1361 |
return (uint)MIN3(thread_count, |
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
1362 |
static_cast<size_t>(max_threads), |
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
1363 |
(size_t)os::active_processor_count()); |
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
1364 |
} |
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
1365 |
|
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
1366 |
bool RefProcMTDegreeAdjuster::use_max_threads(RefProcPhases phase) const { |
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
1367 |
// Even a small number of references in either of those cases could produce large amounts of work. |
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
1368 |
return (phase == ReferenceProcessor::RefPhase1 || phase == ReferenceProcessor::RefPhase3); |
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
1369 |
} |
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
1370 |
|
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
1371 |
RefProcMTDegreeAdjuster::RefProcMTDegreeAdjuster(ReferenceProcessor* rp, |
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
1372 |
RefProcPhases phase, |
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
1373 |
size_t ref_count): |
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
1374 |
_rp(rp), |
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
1375 |
_saved_mt_processing(_rp->processing_is_mt()), |
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
1376 |
_saved_num_queues(_rp->num_queues()) { |
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
1377 |
if (!_rp->processing_is_mt() || !_rp->adjust_no_of_processing_threads() || (ReferencesPerThread == 0)) { |
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
1378 |
return; |
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
1379 |
} |
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
1380 |
|
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
1381 |
uint workers = ergo_proc_thread_count(ref_count, _rp->num_queues(), phase); |
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
1382 |
|
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
1383 |
_rp->set_mt_processing(workers > 1); |
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
1384 |
_rp->set_active_mt_degree(workers); |
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
1385 |
} |
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
1386 |
|
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
1387 |
RefProcMTDegreeAdjuster::~RefProcMTDegreeAdjuster() { |
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
1388 |
// Revert to previous status. |
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
1389 |
_rp->set_mt_processing(_saved_mt_processing); |
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
1390 |
_rp->set_active_mt_degree(_saved_num_queues); |
8f1d5d706bdd
8043575: Dynamically parallelize reference processing work
tschatzl
parents:
50605
diff
changeset
|
1391 |
} |