author | david |
Fri, 27 Mar 2015 15:29:19 +0100 | |
changeset 29800 | fa5f7a2bf717 |
parent 29580 | a67a581cfe11 |
child 29807 | c6217428f47f |
permissions | -rw-r--r-- |
1 | 1 |
/* |
29580
a67a581cfe11
8073315: Enable gcc -Wtype-limits and fix upcoming issues.
goetz
parents:
28510
diff
changeset
|
2 |
* Copyright (c) 2001, 2015, 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" |
26 |
#include "classfile/javaClasses.hpp" |
|
27 |
#include "classfile/systemDictionary.hpp" |
|
18025 | 28 |
#include "gc_implementation/shared/gcTimer.hpp" |
29 |
#include "gc_implementation/shared/gcTraceTime.hpp" |
|
7397 | 30 |
#include "gc_interface/collectedHeap.hpp" |
31 |
#include "gc_interface/collectedHeap.inline.hpp" |
|
32 |
#include "memory/referencePolicy.hpp" |
|
33 |
#include "memory/referenceProcessor.hpp" |
|
34 |
#include "oops/oop.inline.hpp" |
|
35 |
#include "runtime/java.hpp" |
|
36 |
#include "runtime/jniHandles.hpp" |
|
1 | 37 |
|
1606
dcf9714addbe
6684579: SoftReference processing can be made more efficient
ysr
parents:
1605
diff
changeset
|
38 |
ReferencePolicy* ReferenceProcessor::_always_clear_soft_ref_policy = NULL; |
dcf9714addbe
6684579: SoftReference processing can be made more efficient
ysr
parents:
1605
diff
changeset
|
39 |
ReferencePolicy* ReferenceProcessor::_default_soft_ref_policy = NULL; |
10526
3e92f211533f
4965777: GC changes to support use of discovered field for pending references
ysr
parents:
10524
diff
changeset
|
40 |
bool ReferenceProcessor::_pending_list_uses_discovered_field = false; |
10683
4b5a5a507864
7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents:
10670
diff
changeset
|
41 |
jlong ReferenceProcessor::_soft_ref_timestamp_clock = 0; |
1606
dcf9714addbe
6684579: SoftReference processing can be made more efficient
ysr
parents:
1605
diff
changeset
|
42 |
|
1 | 43 |
void referenceProcessor_init() { |
44 |
ReferenceProcessor::init_statics(); |
|
45 |
} |
|
46 |
||
47 |
void ReferenceProcessor::init_statics() { |
|
22551 | 48 |
// 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
|
49 |
// 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
|
50 |
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
|
51 |
|
4b5a5a507864
7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents:
10670
diff
changeset
|
52 |
// Initialize the soft ref timestamp clock. |
4b5a5a507864
7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents:
10670
diff
changeset
|
53 |
_soft_ref_timestamp_clock = now; |
4b5a5a507864
7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents:
10670
diff
changeset
|
54 |
// 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
|
55 |
java_lang_ref_SoftReference::set_clock(_soft_ref_timestamp_clock); |
1 | 56 |
|
1606
dcf9714addbe
6684579: SoftReference processing can be made more efficient
ysr
parents:
1605
diff
changeset
|
57 |
_always_clear_soft_ref_policy = new AlwaysClearPolicy(); |
dcf9714addbe
6684579: SoftReference processing can be made more efficient
ysr
parents:
1605
diff
changeset
|
58 |
_default_soft_ref_policy = new COMPILER2_PRESENT(LRUMaxHeapPolicy()) |
dcf9714addbe
6684579: SoftReference processing can be made more efficient
ysr
parents:
1605
diff
changeset
|
59 |
NOT_COMPILER2(LRUCurrentHeapPolicy()); |
dcf9714addbe
6684579: SoftReference processing can be made more efficient
ysr
parents:
1605
diff
changeset
|
60 |
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
|
61 |
vm_exit_during_initialization("Could not allocate reference policy object"); |
dcf9714addbe
6684579: SoftReference processing can be made more efficient
ysr
parents:
1605
diff
changeset
|
62 |
} |
1 | 63 |
guarantee(RefDiscoveryPolicy == ReferenceBasedDiscovery || |
64 |
RefDiscoveryPolicy == ReferentBasedDiscovery, |
|
22775 | 65 |
"Unrecognized RefDiscoveryPolicy"); |
10526
3e92f211533f
4965777: GC changes to support use of discovered field for pending references
ysr
parents:
10524
diff
changeset
|
66 |
_pending_list_uses_discovered_field = JDK_Version::current().pending_list_uses_discovered_field(); |
1 | 67 |
} |
68 |
||
28212
647b7d0efb88
8066827: Remove ReferenceProcessor::clean_up_discovered_references()
kbarrett
parents:
26831
diff
changeset
|
69 |
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
|
70 |
#ifdef ASSERT |
4b5a5a507864
7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents:
10670
diff
changeset
|
71 |
// Verify that we're not currently discovering refs |
28212
647b7d0efb88
8066827: Remove ReferenceProcessor::clean_up_discovered_references()
kbarrett
parents:
26831
diff
changeset
|
72 |
assert(!_discovering_refs, "nested call?"); |
10683
4b5a5a507864
7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents:
10670
diff
changeset
|
73 |
|
4b5a5a507864
7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents:
10670
diff
changeset
|
74 |
if (check_no_refs) { |
4b5a5a507864
7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents:
10670
diff
changeset
|
75 |
// Verify that the discovered lists are empty |
4b5a5a507864
7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents:
10670
diff
changeset
|
76 |
verify_no_references_recorded(); |
4b5a5a507864
7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents:
10670
diff
changeset
|
77 |
} |
4b5a5a507864
7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents:
10670
diff
changeset
|
78 |
#endif // ASSERT |
4b5a5a507864
7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents:
10670
diff
changeset
|
79 |
|
4b5a5a507864
7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents:
10670
diff
changeset
|
80 |
// 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
|
81 |
// 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
|
82 |
// soft reference timestamp clock using reflection or |
4b5a5a507864
7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents:
10670
diff
changeset
|
83 |
// Unsafe between GCs. Unconditionally update the static |
4b5a5a507864
7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents:
10670
diff
changeset
|
84 |
// 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
|
85 |
// value during reference discovery. |
4b5a5a507864
7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents:
10670
diff
changeset
|
86 |
|
4b5a5a507864
7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents:
10670
diff
changeset
|
87 |
_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
|
88 |
_discovering_refs = true; |
4b5a5a507864
7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents:
10670
diff
changeset
|
89 |
} |
4b5a5a507864
7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents:
10670
diff
changeset
|
90 |
|
1 | 91 |
ReferenceProcessor::ReferenceProcessor(MemRegion span, |
8688
493d12ccc6db
6668573: CMS: reference processing crash if ParallelCMSThreads > ParallelGCThreads
ysr
parents:
7420
diff
changeset
|
92 |
bool mt_processing, |
11396
917d8673b5ef
7121618: Change type of number of GC workers to unsigned int.
jmasa
parents:
11251
diff
changeset
|
93 |
uint mt_processing_degree, |
360
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
94 |
bool mt_discovery, |
11396
917d8673b5ef
7121618: Change type of number of GC workers to unsigned int.
jmasa
parents:
11251
diff
changeset
|
95 |
uint mt_discovery_degree, |
8688
493d12ccc6db
6668573: CMS: reference processing crash if ParallelCMSThreads > ParallelGCThreads
ysr
parents:
7420
diff
changeset
|
96 |
bool atomic_discovery, |
24845
e8b8894a77df
8043239: G1: Missing post barrier in processing of j.l.ref.Reference objects
brutisso
parents:
24424
diff
changeset
|
97 |
BoolObjectClosure* is_alive_non_header) : |
1 | 98 |
_discovering_refs(false), |
99 |
_enqueuing_is_done(false), |
|
8688
493d12ccc6db
6668573: CMS: reference processing crash if ParallelCMSThreads > ParallelGCThreads
ysr
parents:
7420
diff
changeset
|
100 |
_is_alive_non_header(is_alive_non_header), |
1 | 101 |
_processing_is_mt(mt_processing), |
102 |
_next_id(0) |
|
103 |
{ |
|
104 |
_span = span; |
|
105 |
_discovery_is_atomic = atomic_discovery; |
|
106 |
_discovery_is_mt = mt_discovery; |
|
11396
917d8673b5ef
7121618: Change type of number of GC workers to unsigned int.
jmasa
parents:
11251
diff
changeset
|
107 |
_num_q = MAX2(1U, mt_processing_degree); |
8688
493d12ccc6db
6668573: CMS: reference processing crash if ParallelCMSThreads > ParallelGCThreads
ysr
parents:
7420
diff
changeset
|
108 |
_max_num_q = MAX2(_num_q, mt_discovery_degree); |
10747
dfdb9eb56e49
7095243: Disambiguate ReferenceProcessor::_discoveredSoftRefs
johnc
parents:
10683
diff
changeset
|
109 |
_discovered_refs = NEW_C_HEAP_ARRAY(DiscoveredList, |
13195 | 110 |
_max_num_q * number_of_subclasses_of_ref(), mtGC); |
111 |
||
10747
dfdb9eb56e49
7095243: Disambiguate ReferenceProcessor::_discoveredSoftRefs
johnc
parents:
10683
diff
changeset
|
112 |
if (_discovered_refs == NULL) { |
1 | 113 |
vm_exit_during_initialization("Could not allocated RefProc Array"); |
114 |
} |
|
10747
dfdb9eb56e49
7095243: Disambiguate ReferenceProcessor::_discoveredSoftRefs
johnc
parents:
10683
diff
changeset
|
115 |
_discoveredSoftRefs = &_discovered_refs[0]; |
6759
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
116 |
_discoveredWeakRefs = &_discoveredSoftRefs[_max_num_q]; |
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
117 |
_discoveredFinalRefs = &_discoveredWeakRefs[_max_num_q]; |
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
118 |
_discoveredPhantomRefs = &_discoveredFinalRefs[_max_num_q]; |
28510 | 119 |
_discoveredCleanerRefs = &_discoveredPhantomRefs[_max_num_q]; |
10747
dfdb9eb56e49
7095243: Disambiguate ReferenceProcessor::_discoveredSoftRefs
johnc
parents:
10683
diff
changeset
|
120 |
|
dfdb9eb56e49
7095243: Disambiguate ReferenceProcessor::_discoveredSoftRefs
johnc
parents:
10683
diff
changeset
|
121 |
// Initialize all entries to NULL |
11396
917d8673b5ef
7121618: Change type of number of GC workers to unsigned int.
jmasa
parents:
11251
diff
changeset
|
122 |
for (uint i = 0; i < _max_num_q * number_of_subclasses_of_ref(); i++) { |
10747
dfdb9eb56e49
7095243: Disambiguate ReferenceProcessor::_discoveredSoftRefs
johnc
parents:
10683
diff
changeset
|
123 |
_discovered_refs[i].set_head(NULL); |
dfdb9eb56e49
7095243: Disambiguate ReferenceProcessor::_discoveredSoftRefs
johnc
parents:
10683
diff
changeset
|
124 |
_discovered_refs[i].set_length(0); |
1 | 125 |
} |
10747
dfdb9eb56e49
7095243: Disambiguate ReferenceProcessor::_discoveredSoftRefs
johnc
parents:
10683
diff
changeset
|
126 |
|
8688
493d12ccc6db
6668573: CMS: reference processing crash if ParallelCMSThreads > ParallelGCThreads
ysr
parents:
7420
diff
changeset
|
127 |
setup_policy(false /* default soft ref policy */); |
1 | 128 |
} |
129 |
||
130 |
#ifndef PRODUCT |
|
131 |
void ReferenceProcessor::verify_no_references_recorded() { |
|
132 |
guarantee(!_discovering_refs, "Discovering refs?"); |
|
11396
917d8673b5ef
7121618: Change type of number of GC workers to unsigned int.
jmasa
parents:
11251
diff
changeset
|
133 |
for (uint i = 0; i < _max_num_q * number_of_subclasses_of_ref(); i++) { |
10747
dfdb9eb56e49
7095243: Disambiguate ReferenceProcessor::_discoveredSoftRefs
johnc
parents:
10683
diff
changeset
|
134 |
guarantee(_discovered_refs[i].is_empty(), |
1 | 135 |
"Found non-empty discovered list"); |
136 |
} |
|
137 |
} |
|
138 |
#endif |
|
139 |
||
140 |
void ReferenceProcessor::weak_oops_do(OopClosure* f) { |
|
11396
917d8673b5ef
7121618: Change type of number of GC workers to unsigned int.
jmasa
parents:
11251
diff
changeset
|
141 |
for (uint i = 0; i < _max_num_q * 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
|
142 |
if (UseCompressedOops) { |
10747
dfdb9eb56e49
7095243: Disambiguate ReferenceProcessor::_discoveredSoftRefs
johnc
parents:
10683
diff
changeset
|
143 |
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
|
144 |
} else { |
10747
dfdb9eb56e49
7095243: Disambiguate ReferenceProcessor::_discoveredSoftRefs
johnc
parents:
10683
diff
changeset
|
145 |
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
|
146 |
} |
1 | 147 |
} |
148 |
} |
|
149 |
||
360
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
150 |
void ReferenceProcessor::update_soft_ref_master_clock() { |
1 | 151 |
// Update (advance) the soft ref master clock field. This must be done |
152 |
// 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
|
153 |
|
22551 | 154 |
// 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
|
155 |
// 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
|
156 |
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
|
157 |
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
|
158 |
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
|
159 |
|
1 | 160 |
NOT_PRODUCT( |
10683
4b5a5a507864
7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents:
10670
diff
changeset
|
161 |
if (now < _soft_ref_timestamp_clock) { |
29800
fa5f7a2bf717
8076073: shared: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents:
29580
diff
changeset
|
162 |
warning("time warp: " JLONG_FORMAT " to " JLONG_FORMAT, |
10683
4b5a5a507864
7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents:
10670
diff
changeset
|
163 |
_soft_ref_timestamp_clock, now); |
1 | 164 |
} |
165 |
) |
|
11251
e29da6b5622b
7117303: VM uses non-monotonic time source and complains that it is non-monotonic
johnc
parents:
10747
diff
changeset
|
166 |
// 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
|
167 |
// 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
|
168 |
// 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
|
169 |
// a time source (and it is bug free). |
22551 | 170 |
// 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
|
171 |
if (now > _soft_ref_timestamp_clock) { |
4b5a5a507864
7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents:
10670
diff
changeset
|
172 |
_soft_ref_timestamp_clock = now; |
1 | 173 |
java_lang_ref_SoftReference::set_clock(now); |
174 |
} |
|
175 |
// Else leave clock stalled at its old value until time progresses |
|
176 |
// past clock value. |
|
177 |
} |
|
178 |
||
18025 | 179 |
size_t ReferenceProcessor::total_count(DiscoveredList lists[]) { |
180 |
size_t total = 0; |
|
181 |
for (uint i = 0; i < _max_num_q; ++i) { |
|
182 |
total += lists[i].length(); |
|
183 |
} |
|
184 |
return total; |
|
185 |
} |
|
186 |
||
187 |
ReferenceProcessorStats ReferenceProcessor::process_discovered_references( |
|
1 | 188 |
BoolObjectClosure* is_alive, |
189 |
OopClosure* keep_alive, |
|
190 |
VoidClosure* complete_gc, |
|
18025 | 191 |
AbstractRefProcTaskExecutor* task_executor, |
25350
6423a57e5451
8043607: Add a GC id as a log decoration similar to PrintGCTimeStamps
brutisso
parents:
24845
diff
changeset
|
192 |
GCTimer* gc_timer, |
6423a57e5451
8043607: Add a GC id as a log decoration similar to PrintGCTimeStamps
brutisso
parents:
24845
diff
changeset
|
193 |
GCId gc_id) { |
1 | 194 |
NOT_PRODUCT(verify_ok_to_handle_reflists()); |
195 |
||
196 |
assert(!enqueuing_is_done(), "If here enqueuing should not be complete"); |
|
197 |
// Stop treating discovered references specially. |
|
198 |
disable_discovery(); |
|
199 |
||
10683
4b5a5a507864
7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents:
10670
diff
changeset
|
200 |
// 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
|
201 |
// 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
|
202 |
// 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
|
203 |
// 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
|
204 |
// 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
|
205 |
// 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
|
206 |
// discovered soft refs. |
4b5a5a507864
7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents:
10670
diff
changeset
|
207 |
|
4b5a5a507864
7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents:
10670
diff
changeset
|
208 |
_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
|
209 |
|
1 | 210 |
bool trace_time = PrintGCDetails && PrintReferenceGC; |
18025 | 211 |
|
1 | 212 |
// Soft references |
18025 | 213 |
size_t soft_count = 0; |
1 | 214 |
{ |
25350
6423a57e5451
8043607: Add a GC id as a log decoration similar to PrintGCTimeStamps
brutisso
parents:
24845
diff
changeset
|
215 |
GCTraceTime tt("SoftReference", trace_time, false, gc_timer, gc_id); |
18025 | 216 |
soft_count = |
217 |
process_discovered_reflist(_discoveredSoftRefs, _current_soft_ref_policy, true, |
|
218 |
is_alive, keep_alive, complete_gc, task_executor); |
|
1 | 219 |
} |
220 |
||
221 |
update_soft_ref_master_clock(); |
|
222 |
||
223 |
// Weak references |
|
18025 | 224 |
size_t weak_count = 0; |
1 | 225 |
{ |
25350
6423a57e5451
8043607: Add a GC id as a log decoration similar to PrintGCTimeStamps
brutisso
parents:
24845
diff
changeset
|
226 |
GCTraceTime tt("WeakReference", trace_time, false, gc_timer, gc_id); |
18025 | 227 |
weak_count = |
228 |
process_discovered_reflist(_discoveredWeakRefs, NULL, true, |
|
229 |
is_alive, keep_alive, complete_gc, task_executor); |
|
1 | 230 |
} |
231 |
||
232 |
// Final references |
|
18025 | 233 |
size_t final_count = 0; |
1 | 234 |
{ |
25350
6423a57e5451
8043607: Add a GC id as a log decoration similar to PrintGCTimeStamps
brutisso
parents:
24845
diff
changeset
|
235 |
GCTraceTime tt("FinalReference", trace_time, false, gc_timer, gc_id); |
18025 | 236 |
final_count = |
237 |
process_discovered_reflist(_discoveredFinalRefs, NULL, false, |
|
238 |
is_alive, keep_alive, complete_gc, task_executor); |
|
1 | 239 |
} |
240 |
||
241 |
// Phantom references |
|
18025 | 242 |
size_t phantom_count = 0; |
1 | 243 |
{ |
25350
6423a57e5451
8043607: Add a GC id as a log decoration similar to PrintGCTimeStamps
brutisso
parents:
24845
diff
changeset
|
244 |
GCTraceTime tt("PhantomReference", trace_time, false, gc_timer, gc_id); |
18025 | 245 |
phantom_count = |
246 |
process_discovered_reflist(_discoveredPhantomRefs, NULL, false, |
|
247 |
is_alive, keep_alive, complete_gc, task_executor); |
|
28510 | 248 |
|
249 |
// Process cleaners, but include them in phantom statistics. We expect |
|
250 |
// Cleaner references to be temporary, and don't want to deal with |
|
251 |
// possible incompatibilities arising from making it more visible. |
|
252 |
phantom_count += |
|
253 |
process_discovered_reflist(_discoveredCleanerRefs, NULL, false, |
|
254 |
is_alive, keep_alive, complete_gc, task_executor); |
|
1 | 255 |
} |
256 |
||
257 |
// Weak global JNI references. It would make more sense (semantically) to |
|
258 |
// traverse these simultaneously with the regular weak references above, but |
|
259 |
// that is not how the JDK1.2 specification is. See #4126360. Native code can |
|
260 |
// thus use JNI weak references to circumvent the phantom references and |
|
261 |
// resurrect a "post-mortem" object. |
|
262 |
{ |
|
25350
6423a57e5451
8043607: Add a GC id as a log decoration similar to PrintGCTimeStamps
brutisso
parents:
24845
diff
changeset
|
263 |
GCTraceTime tt("JNI Weak Reference", trace_time, false, gc_timer, gc_id); |
1 | 264 |
if (task_executor != NULL) { |
265 |
task_executor->set_single_threaded_mode(); |
|
266 |
} |
|
267 |
process_phaseJNI(is_alive, keep_alive, complete_gc); |
|
268 |
} |
|
18025 | 269 |
|
270 |
return ReferenceProcessorStats(soft_count, weak_count, final_count, phantom_count); |
|
1 | 271 |
} |
272 |
||
273 |
#ifndef PRODUCT |
|
274 |
// Calculate the number of jni handles. |
|
360
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
275 |
uint ReferenceProcessor::count_jni_refs() { |
1 | 276 |
class AlwaysAliveClosure: public BoolObjectClosure { |
277 |
public: |
|
360
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
278 |
virtual bool do_object_b(oop obj) { return true; } |
1 | 279 |
}; |
280 |
||
281 |
class CountHandleClosure: public OopClosure { |
|
282 |
private: |
|
283 |
int _count; |
|
284 |
public: |
|
285 |
CountHandleClosure(): _count(0) {} |
|
360
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
286 |
void do_oop(oop* unused) { _count++; } |
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
287 |
void do_oop(narrowOop* unused) { ShouldNotReachHere(); } |
1 | 288 |
int count() { return _count; } |
289 |
}; |
|
290 |
CountHandleClosure global_handle_count; |
|
291 |
AlwaysAliveClosure always_alive; |
|
292 |
JNIHandles::weak_oops_do(&always_alive, &global_handle_count); |
|
293 |
return global_handle_count.count(); |
|
294 |
} |
|
295 |
#endif |
|
296 |
||
297 |
void ReferenceProcessor::process_phaseJNI(BoolObjectClosure* is_alive, |
|
298 |
OopClosure* keep_alive, |
|
299 |
VoidClosure* complete_gc) { |
|
300 |
#ifndef PRODUCT |
|
301 |
if (PrintGCDetails && PrintReferenceGC) { |
|
302 |
unsigned int count = count_jni_refs(); |
|
303 |
gclog_or_tty->print(", %u refs", count); |
|
304 |
} |
|
305 |
#endif |
|
306 |
JNIHandles::weak_oops_do(is_alive, keep_alive); |
|
307 |
complete_gc->do_void(); |
|
308 |
} |
|
309 |
||
360
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
310 |
|
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
311 |
template <class T> |
4493
9204129f065e
6843629: Make current hotspot build part of jdk5 control build
phh
parents:
3690
diff
changeset
|
312 |
bool enqueue_discovered_ref_helper(ReferenceProcessor* ref, |
9204129f065e
6843629: Make current hotspot build part of jdk5 control build
phh
parents:
3690
diff
changeset
|
313 |
AbstractRefProcTaskExecutor* task_executor) { |
360
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
314 |
|
1 | 315 |
// Remember old value of pending references list |
360
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
316 |
T* pending_list_addr = (T*)java_lang_ref_Reference::pending_list_addr(); |
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
317 |
T old_pending_list_value = *pending_list_addr; |
1 | 318 |
|
319 |
// Enqueue references that are not made active again, and |
|
320 |
// clear the decks for the next collection (cycle). |
|
360
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
321 |
ref->enqueue_discovered_reflists((HeapWord*)pending_list_addr, task_executor); |
22496
383a5bdef99d
8029255: G1: Reference processing should not enqueue references on the shared SATB queue
pliden
parents:
20282
diff
changeset
|
322 |
// Do the post-barrier on pending_list_addr missed in |
383a5bdef99d
8029255: G1: Reference processing should not enqueue references on the shared SATB queue
pliden
parents:
20282
diff
changeset
|
323 |
// enqueue_discovered_reflist. |
383a5bdef99d
8029255: G1: Reference processing should not enqueue references on the shared SATB queue
pliden
parents:
20282
diff
changeset
|
324 |
oopDesc::bs()->write_ref_field(pending_list_addr, oopDesc::load_decode_heap_oop(pending_list_addr)); |
1 | 325 |
|
326 |
// Stop treating discovered references specially. |
|
360
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
327 |
ref->disable_discovery(); |
1 | 328 |
|
329 |
// Return true if new pending references were added |
|
330 |
return old_pending_list_value != *pending_list_addr; |
|
331 |
} |
|
332 |
||
360
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
333 |
bool ReferenceProcessor::enqueue_discovered_references(AbstractRefProcTaskExecutor* task_executor) { |
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
334 |
NOT_PRODUCT(verify_ok_to_handle_reflists()); |
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
335 |
if (UseCompressedOops) { |
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
336 |
return enqueue_discovered_ref_helper<narrowOop>(this, task_executor); |
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
337 |
} else { |
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
338 |
return enqueue_discovered_ref_helper<oop>(this, task_executor); |
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
339 |
} |
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
340 |
} |
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
341 |
|
1 | 342 |
void ReferenceProcessor::enqueue_discovered_reflist(DiscoveredList& refs_list, |
360
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
343 |
HeapWord* pending_list_addr) { |
1 | 344 |
// Given a list of refs linked through the "discovered" field |
10526
3e92f211533f
4965777: GC changes to support use of discovered field for pending references
ysr
parents:
10524
diff
changeset
|
345 |
// (java.lang.ref.Reference.discovered), self-loop their "next" field |
3e92f211533f
4965777: GC changes to support use of discovered field for pending references
ysr
parents:
10524
diff
changeset
|
346 |
// thus distinguishing them from active References, then |
3e92f211533f
4965777: GC changes to support use of discovered field for pending references
ysr
parents:
10524
diff
changeset
|
347 |
// prepend them to the pending list. |
24845
e8b8894a77df
8043239: G1: Missing post barrier in processing of j.l.ref.Reference objects
brutisso
parents:
24424
diff
changeset
|
348 |
// |
e8b8894a77df
8043239: G1: Missing post barrier in processing of j.l.ref.Reference objects
brutisso
parents:
24424
diff
changeset
|
349 |
// The Java threads will see the Reference objects linked together through |
e8b8894a77df
8043239: G1: Missing post barrier in processing of j.l.ref.Reference objects
brutisso
parents:
24424
diff
changeset
|
350 |
// the discovered field. Instead of trying to do the write barrier updates |
e8b8894a77df
8043239: G1: Missing post barrier in processing of j.l.ref.Reference objects
brutisso
parents:
24424
diff
changeset
|
351 |
// in all places in the reference processor where we manipulate the discovered |
e8b8894a77df
8043239: G1: Missing post barrier in processing of j.l.ref.Reference objects
brutisso
parents:
24424
diff
changeset
|
352 |
// field we make sure to do the barrier here where we anyway iterate through |
e8b8894a77df
8043239: G1: Missing post barrier in processing of j.l.ref.Reference objects
brutisso
parents:
24424
diff
changeset
|
353 |
// all linked Reference objects. Note that it is important to not dirty any |
e8b8894a77df
8043239: G1: Missing post barrier in processing of j.l.ref.Reference objects
brutisso
parents:
24424
diff
changeset
|
354 |
// cards during reference processing since this will cause card table |
e8b8894a77df
8043239: G1: Missing post barrier in processing of j.l.ref.Reference objects
brutisso
parents:
24424
diff
changeset
|
355 |
// verification to fail for G1. |
e8b8894a77df
8043239: G1: Missing post barrier in processing of j.l.ref.Reference objects
brutisso
parents:
24424
diff
changeset
|
356 |
// |
10526
3e92f211533f
4965777: GC changes to support use of discovered field for pending references
ysr
parents:
10524
diff
changeset
|
357 |
// BKWRD COMPATIBILITY NOTE: For older JDKs (prior to the fix for 4956777), |
3e92f211533f
4965777: GC changes to support use of discovered field for pending references
ysr
parents:
10524
diff
changeset
|
358 |
// the "next" field is used to chain the pending list, not the discovered |
3e92f211533f
4965777: GC changes to support use of discovered field for pending references
ysr
parents:
10524
diff
changeset
|
359 |
// field. |
1 | 360 |
if (TraceReferenceGC && PrintGCDetails) { |
361 |
gclog_or_tty->print_cr("ReferenceProcessor::enqueue_discovered_reflist list " |
|
29800
fa5f7a2bf717
8076073: shared: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents:
29580
diff
changeset
|
362 |
INTPTR_FORMAT, p2i(refs_list.head())); |
1 | 363 |
} |
10524
6594ca81279a
7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents:
8688
diff
changeset
|
364 |
|
6594ca81279a
7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents:
8688
diff
changeset
|
365 |
oop obj = NULL; |
10526
3e92f211533f
4965777: GC changes to support use of discovered field for pending references
ysr
parents:
10524
diff
changeset
|
366 |
oop next_d = refs_list.head(); |
22551 | 367 |
if (pending_list_uses_discovered_field()) { // New behavior |
10526
3e92f211533f
4965777: GC changes to support use of discovered field for pending references
ysr
parents:
10524
diff
changeset
|
368 |
// Walk down the list, self-looping the next field |
3e92f211533f
4965777: GC changes to support use of discovered field for pending references
ysr
parents:
10524
diff
changeset
|
369 |
// so that the References are not considered active. |
3e92f211533f
4965777: GC changes to support use of discovered field for pending references
ysr
parents:
10524
diff
changeset
|
370 |
while (obj != next_d) { |
3e92f211533f
4965777: GC changes to support use of discovered field for pending references
ysr
parents:
10524
diff
changeset
|
371 |
obj = next_d; |
3e92f211533f
4965777: GC changes to support use of discovered field for pending references
ysr
parents:
10524
diff
changeset
|
372 |
assert(obj->is_instanceRef(), "should be reference object"); |
3e92f211533f
4965777: GC changes to support use of discovered field for pending references
ysr
parents:
10524
diff
changeset
|
373 |
next_d = java_lang_ref_Reference::discovered(obj); |
3e92f211533f
4965777: GC changes to support use of discovered field for pending references
ysr
parents:
10524
diff
changeset
|
374 |
if (TraceReferenceGC && PrintGCDetails) { |
3e92f211533f
4965777: GC changes to support use of discovered field for pending references
ysr
parents:
10524
diff
changeset
|
375 |
gclog_or_tty->print_cr(" obj " INTPTR_FORMAT "/next_d " INTPTR_FORMAT, |
29800
fa5f7a2bf717
8076073: shared: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents:
29580
diff
changeset
|
376 |
p2i(obj), p2i(next_d)); |
10526
3e92f211533f
4965777: GC changes to support use of discovered field for pending references
ysr
parents:
10524
diff
changeset
|
377 |
} |
3e92f211533f
4965777: GC changes to support use of discovered field for pending references
ysr
parents:
10524
diff
changeset
|
378 |
assert(java_lang_ref_Reference::next(obj) == NULL, |
3e92f211533f
4965777: GC changes to support use of discovered field for pending references
ysr
parents:
10524
diff
changeset
|
379 |
"Reference not active; should not be discovered"); |
3e92f211533f
4965777: GC changes to support use of discovered field for pending references
ysr
parents:
10524
diff
changeset
|
380 |
// Self-loop next, so as to make Ref not active. |
22496
383a5bdef99d
8029255: G1: Reference processing should not enqueue references on the shared SATB queue
pliden
parents:
20282
diff
changeset
|
381 |
java_lang_ref_Reference::set_next_raw(obj, obj); |
24845
e8b8894a77df
8043239: G1: Missing post barrier in processing of j.l.ref.Reference objects
brutisso
parents:
24424
diff
changeset
|
382 |
if (next_d != obj) { |
e8b8894a77df
8043239: G1: Missing post barrier in processing of j.l.ref.Reference objects
brutisso
parents:
24424
diff
changeset
|
383 |
oopDesc::bs()->write_ref_field(java_lang_ref_Reference::discovered_addr(obj), next_d); |
e8b8894a77df
8043239: G1: Missing post barrier in processing of j.l.ref.Reference objects
brutisso
parents:
24424
diff
changeset
|
384 |
} else { |
e8b8894a77df
8043239: G1: Missing post barrier in processing of j.l.ref.Reference objects
brutisso
parents:
24424
diff
changeset
|
385 |
// This is the last object. |
22551 | 386 |
// Swap refs_list into pending_list_addr and |
10526
3e92f211533f
4965777: GC changes to support use of discovered field for pending references
ysr
parents:
10524
diff
changeset
|
387 |
// set obj's discovered to what we read from pending_list_addr. |
3e92f211533f
4965777: GC changes to support use of discovered field for pending references
ysr
parents:
10524
diff
changeset
|
388 |
oop old = oopDesc::atomic_exchange_oop(refs_list.head(), pending_list_addr); |
24845
e8b8894a77df
8043239: G1: Missing post barrier in processing of j.l.ref.Reference objects
brutisso
parents:
24424
diff
changeset
|
389 |
// Need post-barrier on pending_list_addr. See enqueue_discovered_ref_helper() above. |
22496
383a5bdef99d
8029255: G1: Reference processing should not enqueue references on the shared SATB queue
pliden
parents:
20282
diff
changeset
|
390 |
java_lang_ref_Reference::set_discovered_raw(obj, old); // old may be NULL |
383a5bdef99d
8029255: G1: Reference processing should not enqueue references on the shared SATB queue
pliden
parents:
20282
diff
changeset
|
391 |
oopDesc::bs()->write_ref_field(java_lang_ref_Reference::discovered_addr(obj), old); |
10526
3e92f211533f
4965777: GC changes to support use of discovered field for pending references
ysr
parents:
10524
diff
changeset
|
392 |
} |
1 | 393 |
} |
22551 | 394 |
} else { // Old behavior |
10526
3e92f211533f
4965777: GC changes to support use of discovered field for pending references
ysr
parents:
10524
diff
changeset
|
395 |
// Walk down the list, copying the discovered field into |
3e92f211533f
4965777: GC changes to support use of discovered field for pending references
ysr
parents:
10524
diff
changeset
|
396 |
// the next field and clearing the discovered field. |
3e92f211533f
4965777: GC changes to support use of discovered field for pending references
ysr
parents:
10524
diff
changeset
|
397 |
while (obj != next_d) { |
3e92f211533f
4965777: GC changes to support use of discovered field for pending references
ysr
parents:
10524
diff
changeset
|
398 |
obj = next_d; |
3e92f211533f
4965777: GC changes to support use of discovered field for pending references
ysr
parents:
10524
diff
changeset
|
399 |
assert(obj->is_instanceRef(), "should be reference object"); |
3e92f211533f
4965777: GC changes to support use of discovered field for pending references
ysr
parents:
10524
diff
changeset
|
400 |
next_d = java_lang_ref_Reference::discovered(obj); |
3e92f211533f
4965777: GC changes to support use of discovered field for pending references
ysr
parents:
10524
diff
changeset
|
401 |
if (TraceReferenceGC && PrintGCDetails) { |
3e92f211533f
4965777: GC changes to support use of discovered field for pending references
ysr
parents:
10524
diff
changeset
|
402 |
gclog_or_tty->print_cr(" obj " INTPTR_FORMAT "/next_d " INTPTR_FORMAT, |
29800
fa5f7a2bf717
8076073: shared: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents:
29580
diff
changeset
|
403 |
p2i(obj), p2i(next_d)); |
10526
3e92f211533f
4965777: GC changes to support use of discovered field for pending references
ysr
parents:
10524
diff
changeset
|
404 |
} |
3e92f211533f
4965777: GC changes to support use of discovered field for pending references
ysr
parents:
10524
diff
changeset
|
405 |
assert(java_lang_ref_Reference::next(obj) == NULL, |
3e92f211533f
4965777: GC changes to support use of discovered field for pending references
ysr
parents:
10524
diff
changeset
|
406 |
"The reference should not be enqueued"); |
3e92f211533f
4965777: GC changes to support use of discovered field for pending references
ysr
parents:
10524
diff
changeset
|
407 |
if (next_d == obj) { // obj is last |
22551 | 408 |
// Swap refs_list into pending_list_addr and |
10526
3e92f211533f
4965777: GC changes to support use of discovered field for pending references
ysr
parents:
10524
diff
changeset
|
409 |
// set obj's next to what we read from pending_list_addr. |
3e92f211533f
4965777: GC changes to support use of discovered field for pending references
ysr
parents:
10524
diff
changeset
|
410 |
oop old = oopDesc::atomic_exchange_oop(refs_list.head(), pending_list_addr); |
3e92f211533f
4965777: GC changes to support use of discovered field for pending references
ysr
parents:
10524
diff
changeset
|
411 |
// Need oop_check on pending_list_addr above; |
3e92f211533f
4965777: GC changes to support use of discovered field for pending references
ysr
parents:
10524
diff
changeset
|
412 |
// see special oop-check code at the end of |
3e92f211533f
4965777: GC changes to support use of discovered field for pending references
ysr
parents:
10524
diff
changeset
|
413 |
// enqueue_discovered_reflists() further below. |
3e92f211533f
4965777: GC changes to support use of discovered field for pending references
ysr
parents:
10524
diff
changeset
|
414 |
if (old == NULL) { |
3e92f211533f
4965777: GC changes to support use of discovered field for pending references
ysr
parents:
10524
diff
changeset
|
415 |
// obj should be made to point to itself, since |
3e92f211533f
4965777: GC changes to support use of discovered field for pending references
ysr
parents:
10524
diff
changeset
|
416 |
// pending list was empty. |
3e92f211533f
4965777: GC changes to support use of discovered field for pending references
ysr
parents:
10524
diff
changeset
|
417 |
java_lang_ref_Reference::set_next(obj, obj); |
3e92f211533f
4965777: GC changes to support use of discovered field for pending references
ysr
parents:
10524
diff
changeset
|
418 |
} else { |
3e92f211533f
4965777: GC changes to support use of discovered field for pending references
ysr
parents:
10524
diff
changeset
|
419 |
java_lang_ref_Reference::set_next(obj, old); |
3e92f211533f
4965777: GC changes to support use of discovered field for pending references
ysr
parents:
10524
diff
changeset
|
420 |
} |
1 | 421 |
} else { |
10526
3e92f211533f
4965777: GC changes to support use of discovered field for pending references
ysr
parents:
10524
diff
changeset
|
422 |
java_lang_ref_Reference::set_next(obj, next_d); |
1 | 423 |
} |
10526
3e92f211533f
4965777: GC changes to support use of discovered field for pending references
ysr
parents:
10524
diff
changeset
|
424 |
java_lang_ref_Reference::set_discovered(obj, (oop) NULL); |
1 | 425 |
} |
426 |
} |
|
427 |
} |
|
428 |
||
429 |
// Parallel enqueue task |
|
430 |
class RefProcEnqueueTask: public AbstractRefProcTaskExecutor::EnqueueTask { |
|
431 |
public: |
|
432 |
RefProcEnqueueTask(ReferenceProcessor& ref_processor, |
|
433 |
DiscoveredList discovered_refs[], |
|
360
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
434 |
HeapWord* pending_list_addr, |
1 | 435 |
int n_queues) |
436 |
: EnqueueTask(ref_processor, discovered_refs, |
|
10524
6594ca81279a
7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents:
8688
diff
changeset
|
437 |
pending_list_addr, n_queues) |
1 | 438 |
{ } |
439 |
||
360
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
440 |
virtual void work(unsigned int work_id) { |
8688
493d12ccc6db
6668573: CMS: reference processing crash if ParallelCMSThreads > ParallelGCThreads
ysr
parents:
7420
diff
changeset
|
441 |
assert(work_id < (unsigned int)_ref_processor.max_num_q(), "Index out-of-bounds"); |
1 | 442 |
// Simplest first cut: static partitioning. |
443 |
int index = work_id; |
|
6759
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
444 |
// The increment on "index" must correspond to the maximum number of queues |
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
445 |
// (n_queues) with which that ReferenceProcessor was created. That |
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
446 |
// is because of the "clever" way the discovered references lists were |
8688
493d12ccc6db
6668573: CMS: reference processing crash if ParallelCMSThreads > ParallelGCThreads
ysr
parents:
7420
diff
changeset
|
447 |
// allocated and are indexed into. |
493d12ccc6db
6668573: CMS: reference processing crash if ParallelCMSThreads > ParallelGCThreads
ysr
parents:
7420
diff
changeset
|
448 |
assert(_n_queues == (int) _ref_processor.max_num_q(), "Different number not expected"); |
6759
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
449 |
for (int j = 0; |
10670
4ea0e7d2ffbc
6484982: G1: process references during evacuation pauses
johnc
parents:
10526
diff
changeset
|
450 |
j < ReferenceProcessor::number_of_subclasses_of_ref(); |
6759
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
451 |
j++, index += _n_queues) { |
1 | 452 |
_ref_processor.enqueue_discovered_reflist( |
453 |
_refs_lists[index], _pending_list_addr); |
|
10524
6594ca81279a
7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents:
8688
diff
changeset
|
454 |
_refs_lists[index].set_head(NULL); |
1 | 455 |
_refs_lists[index].set_length(0); |
456 |
} |
|
457 |
} |
|
458 |
}; |
|
459 |
||
460 |
// Enqueue references that are not made active again |
|
360
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
461 |
void ReferenceProcessor::enqueue_discovered_reflists(HeapWord* pending_list_addr, |
1 | 462 |
AbstractRefProcTaskExecutor* task_executor) { |
463 |
if (_processing_is_mt && task_executor != NULL) { |
|
464 |
// Parallel code |
|
10747
dfdb9eb56e49
7095243: Disambiguate ReferenceProcessor::_discoveredSoftRefs
johnc
parents:
10683
diff
changeset
|
465 |
RefProcEnqueueTask tsk(*this, _discovered_refs, |
10524
6594ca81279a
7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents:
8688
diff
changeset
|
466 |
pending_list_addr, _max_num_q); |
1 | 467 |
task_executor->execute(tsk); |
468 |
} else { |
|
469 |
// Serial code: call the parent class's implementation |
|
11396
917d8673b5ef
7121618: Change type of number of GC workers to unsigned int.
jmasa
parents:
11251
diff
changeset
|
470 |
for (uint i = 0; i < _max_num_q * number_of_subclasses_of_ref(); i++) { |
10747
dfdb9eb56e49
7095243: Disambiguate ReferenceProcessor::_discoveredSoftRefs
johnc
parents:
10683
diff
changeset
|
471 |
enqueue_discovered_reflist(_discovered_refs[i], pending_list_addr); |
dfdb9eb56e49
7095243: Disambiguate ReferenceProcessor::_discoveredSoftRefs
johnc
parents:
10683
diff
changeset
|
472 |
_discovered_refs[i].set_head(NULL); |
dfdb9eb56e49
7095243: Disambiguate ReferenceProcessor::_discoveredSoftRefs
johnc
parents:
10683
diff
changeset
|
473 |
_discovered_refs[i].set_length(0); |
1 | 474 |
} |
475 |
} |
|
476 |
} |
|
477 |
||
10670
4ea0e7d2ffbc
6484982: G1: process references during evacuation pauses
johnc
parents:
10526
diff
changeset
|
478 |
void DiscoveredListIterator::load_ptrs(DEBUG_ONLY(bool allow_null_referent)) { |
1 | 479 |
_discovered_addr = java_lang_ref_Reference::discovered_addr(_ref); |
360
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
480 |
oop discovered = java_lang_ref_Reference::discovered(_ref); |
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
481 |
assert(_discovered_addr && discovered->is_oop_or_null(), |
26831
5c67eb0c46bc
8035729: Code using assert(is_oop_or_null) needs better error messages
mlarsson
parents:
25350
diff
changeset
|
482 |
err_msg("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
|
483 |
_next = discovered; |
1 | 484 |
_referent_addr = java_lang_ref_Reference::referent_addr(_ref); |
360
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
485 |
_referent = java_lang_ref_Reference::referent(_ref); |
1 | 486 |
assert(Universe::heap()->is_in_reserved_or_null(_referent), |
487 |
"Wrong oop found in java.lang.Reference object"); |
|
488 |
assert(allow_null_referent ? |
|
489 |
_referent->is_oop_or_null() |
|
490 |
: _referent->is_oop(), |
|
26831
5c67eb0c46bc
8035729: Code using assert(is_oop_or_null) needs better error messages
mlarsson
parents:
25350
diff
changeset
|
491 |
err_msg("Expected an oop%s for referent field at " PTR_FORMAT, |
5c67eb0c46bc
8035729: Code using assert(is_oop_or_null) needs better error messages
mlarsson
parents:
25350
diff
changeset
|
492 |
(allow_null_referent ? " or NULL" : ""), |
5c67eb0c46bc
8035729: Code using assert(is_oop_or_null) needs better error messages
mlarsson
parents:
25350
diff
changeset
|
493 |
p2i(_referent))); |
1 | 494 |
} |
495 |
||
10670
4ea0e7d2ffbc
6484982: G1: process references during evacuation pauses
johnc
parents:
10526
diff
changeset
|
496 |
void DiscoveredListIterator::remove() { |
1 | 497 |
assert(_ref->is_oop(), "Dropping a bad reference"); |
360
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
498 |
oop_store_raw(_discovered_addr, NULL); |
10524
6594ca81279a
7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents:
8688
diff
changeset
|
499 |
|
360
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
500 |
// 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
|
501 |
oop new_next; |
6594ca81279a
7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents:
8688
diff
changeset
|
502 |
if (_next == _ref) { |
6594ca81279a
7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents:
8688
diff
changeset
|
503 |
// 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
|
504 |
// 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
|
505 |
// and _prev will be NULL. |
6594ca81279a
7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents:
8688
diff
changeset
|
506 |
new_next = _prev; |
6594ca81279a
7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents:
8688
diff
changeset
|
507 |
} else { |
6594ca81279a
7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents:
8688
diff
changeset
|
508 |
new_next = _next; |
6594ca81279a
7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents:
8688
diff
changeset
|
509 |
} |
22768
363a44995a71
8031703: Missing post-barrier in ReferenceProcessor
pliden
parents:
22551
diff
changeset
|
510 |
// 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
|
511 |
// 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
|
512 |
// that's how it ended up in the discovered list in the first place. |
363a44995a71
8031703: Missing post-barrier in ReferenceProcessor
pliden
parents:
22551
diff
changeset
|
513 |
oop_store_raw(_prev_next, new_next); |
1 | 514 |
NOT_PRODUCT(_removed++); |
1605
6b43d186eb8d
6722113: CMS: Incorrect overflow handling during precleaning of Reference lists
ysr
parents:
1388
diff
changeset
|
515 |
_refs_list.dec_length(1); |
1 | 516 |
} |
517 |
||
10670
4ea0e7d2ffbc
6484982: G1: process references during evacuation pauses
johnc
parents:
10526
diff
changeset
|
518 |
// Make the Reference object active again. |
4ea0e7d2ffbc
6484982: G1: process references during evacuation pauses
johnc
parents:
10526
diff
changeset
|
519 |
void DiscoveredListIterator::make_active() { |
24845
e8b8894a77df
8043239: G1: Missing post barrier in processing of j.l.ref.Reference objects
brutisso
parents:
24424
diff
changeset
|
520 |
// The pre barrier for G1 is probably just needed for the old |
e8b8894a77df
8043239: G1: Missing post barrier in processing of j.l.ref.Reference objects
brutisso
parents:
24424
diff
changeset
|
521 |
// reference processing behavior. Should we guard this with |
e8b8894a77df
8043239: G1: Missing post barrier in processing of j.l.ref.Reference objects
brutisso
parents:
24424
diff
changeset
|
522 |
// ReferenceProcessor::pending_list_uses_discovered_field() ? |
10670
4ea0e7d2ffbc
6484982: G1: process references during evacuation pauses
johnc
parents:
10526
diff
changeset
|
523 |
if (UseG1GC) { |
4ea0e7d2ffbc
6484982: G1: process references during evacuation pauses
johnc
parents:
10526
diff
changeset
|
524 |
HeapWord* next_addr = java_lang_ref_Reference::next_addr(_ref); |
4ea0e7d2ffbc
6484982: G1: process references during evacuation pauses
johnc
parents:
10526
diff
changeset
|
525 |
if (UseCompressedOops) { |
22496
383a5bdef99d
8029255: G1: Reference processing should not enqueue references on the shared SATB queue
pliden
parents:
20282
diff
changeset
|
526 |
oopDesc::bs()->write_ref_field_pre((narrowOop*)next_addr, NULL); |
10670
4ea0e7d2ffbc
6484982: G1: process references during evacuation pauses
johnc
parents:
10526
diff
changeset
|
527 |
} else { |
22496
383a5bdef99d
8029255: G1: Reference processing should not enqueue references on the shared SATB queue
pliden
parents:
20282
diff
changeset
|
528 |
oopDesc::bs()->write_ref_field_pre((oop*)next_addr, NULL); |
10670
4ea0e7d2ffbc
6484982: G1: process references during evacuation pauses
johnc
parents:
10526
diff
changeset
|
529 |
} |
10524
6594ca81279a
7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents:
8688
diff
changeset
|
530 |
} |
24845
e8b8894a77df
8043239: G1: Missing post barrier in processing of j.l.ref.Reference objects
brutisso
parents:
24424
diff
changeset
|
531 |
java_lang_ref_Reference::set_next_raw(_ref, NULL); |
10670
4ea0e7d2ffbc
6484982: G1: process references during evacuation pauses
johnc
parents:
10526
diff
changeset
|
532 |
} |
4ea0e7d2ffbc
6484982: G1: process references during evacuation pauses
johnc
parents:
10526
diff
changeset
|
533 |
|
4ea0e7d2ffbc
6484982: G1: process references during evacuation pauses
johnc
parents:
10526
diff
changeset
|
534 |
void DiscoveredListIterator::clear_referent() { |
4ea0e7d2ffbc
6484982: G1: process references during evacuation pauses
johnc
parents:
10526
diff
changeset
|
535 |
oop_store_raw(_referent_addr, NULL); |
1 | 536 |
} |
537 |
||
538 |
// NOTE: process_phase*() are largely similar, and at a high level |
|
539 |
// merely iterate over the extant list applying a predicate to |
|
540 |
// each of its elements and possibly removing that element from the |
|
541 |
// list and applying some further closures to that element. |
|
542 |
// We should consider the possibility of replacing these |
|
543 |
// process_phase*() methods by abstracting them into |
|
544 |
// a single general iterator invocation that receives appropriate |
|
545 |
// closures that accomplish this work. |
|
546 |
||
547 |
// (SoftReferences only) Traverse the list and remove any SoftReferences whose |
|
548 |
// referents are not alive, but that should be kept alive for policy reasons. |
|
549 |
// Keep alive the transitive closure of all such referents. |
|
550 |
void |
|
360
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
551 |
ReferenceProcessor::process_phase1(DiscoveredList& refs_list, |
1 | 552 |
ReferencePolicy* policy, |
553 |
BoolObjectClosure* is_alive, |
|
554 |
OopClosure* keep_alive, |
|
555 |
VoidClosure* complete_gc) { |
|
556 |
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
|
557 |
DiscoveredListIterator iter(refs_list, keep_alive, is_alive); |
1 | 558 |
// Decide which softly reachable refs should be kept alive. |
559 |
while (iter.has_next()) { |
|
560 |
iter.load_ptrs(DEBUG_ONLY(!discovery_is_atomic() /* allow_null_referent */)); |
|
561 |
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
|
562 |
if (referent_is_dead && |
4b5a5a507864
7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents:
10670
diff
changeset
|
563 |
!policy->should_clear_reference(iter.obj(), _soft_ref_timestamp_clock)) { |
1 | 564 |
if (TraceReferenceGC) { |
565 |
gclog_or_tty->print_cr("Dropping reference (" INTPTR_FORMAT ": %s" ") by policy", |
|
29800
fa5f7a2bf717
8076073: shared: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents:
29580
diff
changeset
|
566 |
p2i(iter.obj()), iter.obj()->klass()->internal_name()); |
1 | 567 |
} |
1605
6b43d186eb8d
6722113: CMS: Incorrect overflow handling during precleaning of Reference lists
ysr
parents:
1388
diff
changeset
|
568 |
// Remove Reference object from list |
6b43d186eb8d
6722113: CMS: Incorrect overflow handling during precleaning of Reference lists
ysr
parents:
1388
diff
changeset
|
569 |
iter.remove(); |
1 | 570 |
// Make the Reference object active again |
571 |
iter.make_active(); |
|
572 |
// keep the referent around |
|
573 |
iter.make_referent_alive(); |
|
1605
6b43d186eb8d
6722113: CMS: Incorrect overflow handling during precleaning of Reference lists
ysr
parents:
1388
diff
changeset
|
574 |
iter.move_to_next(); |
1 | 575 |
} else { |
576 |
iter.next(); |
|
577 |
} |
|
578 |
} |
|
579 |
// Close the reachable set |
|
580 |
complete_gc->do_void(); |
|
581 |
NOT_PRODUCT( |
|
582 |
if (PrintGCDetails && TraceReferenceGC) { |
|
29800
fa5f7a2bf717
8076073: shared: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents:
29580
diff
changeset
|
583 |
gclog_or_tty->print_cr(" Dropped " SIZE_FORMAT " dead Refs out of " SIZE_FORMAT |
fa5f7a2bf717
8076073: shared: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents:
29580
diff
changeset
|
584 |
" discovered Refs by policy, from list " INTPTR_FORMAT, |
fa5f7a2bf717
8076073: shared: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents:
29580
diff
changeset
|
585 |
iter.removed(), iter.processed(), p2i(refs_list.head())); |
1 | 586 |
} |
587 |
) |
|
588 |
} |
|
589 |
||
590 |
// Traverse the list and remove any Refs that are not active, or |
|
591 |
// whose referents are either alive or NULL. |
|
592 |
void |
|
360
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
593 |
ReferenceProcessor::pp2_work(DiscoveredList& refs_list, |
1 | 594 |
BoolObjectClosure* is_alive, |
360
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
595 |
OopClosure* keep_alive) { |
1 | 596 |
assert(discovery_is_atomic(), "Error"); |
24845
e8b8894a77df
8043239: G1: Missing post barrier in processing of j.l.ref.Reference objects
brutisso
parents:
24424
diff
changeset
|
597 |
DiscoveredListIterator iter(refs_list, keep_alive, is_alive); |
1 | 598 |
while (iter.has_next()) { |
599 |
iter.load_ptrs(DEBUG_ONLY(false /* allow_null_referent */)); |
|
360
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
600 |
DEBUG_ONLY(oop next = java_lang_ref_Reference::next(iter.obj());) |
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
601 |
assert(next == NULL, "Should not discover inactive Reference"); |
1 | 602 |
if (iter.is_referent_alive()) { |
603 |
if (TraceReferenceGC) { |
|
604 |
gclog_or_tty->print_cr("Dropping strongly reachable reference (" INTPTR_FORMAT ": %s)", |
|
29800
fa5f7a2bf717
8076073: shared: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents:
29580
diff
changeset
|
605 |
p2i(iter.obj()), iter.obj()->klass()->internal_name()); |
1 | 606 |
} |
607 |
// The referent is reachable after all. |
|
1605
6b43d186eb8d
6722113: CMS: Incorrect overflow handling during precleaning of Reference lists
ysr
parents:
1388
diff
changeset
|
608 |
// Remove Reference object from list. |
6b43d186eb8d
6722113: CMS: Incorrect overflow handling during precleaning of Reference lists
ysr
parents:
1388
diff
changeset
|
609 |
iter.remove(); |
1 | 610 |
// Update the referent pointer as necessary: Note that this |
611 |
// should not entail any recursive marking because the |
|
612 |
// referent must already have been traversed. |
|
613 |
iter.make_referent_alive(); |
|
1605
6b43d186eb8d
6722113: CMS: Incorrect overflow handling during precleaning of Reference lists
ysr
parents:
1388
diff
changeset
|
614 |
iter.move_to_next(); |
1 | 615 |
} else { |
616 |
iter.next(); |
|
617 |
} |
|
618 |
} |
|
619 |
NOT_PRODUCT( |
|
8688
493d12ccc6db
6668573: CMS: reference processing crash if ParallelCMSThreads > ParallelGCThreads
ysr
parents:
7420
diff
changeset
|
620 |
if (PrintGCDetails && TraceReferenceGC && (iter.processed() > 0)) { |
29800
fa5f7a2bf717
8076073: shared: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents:
29580
diff
changeset
|
621 |
gclog_or_tty->print_cr(" Dropped " SIZE_FORMAT " active Refs out of " SIZE_FORMAT |
fa5f7a2bf717
8076073: shared: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents:
29580
diff
changeset
|
622 |
" Refs in discovered list " INTPTR_FORMAT, |
fa5f7a2bf717
8076073: shared: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents:
29580
diff
changeset
|
623 |
iter.removed(), iter.processed(), p2i(refs_list.head())); |
1 | 624 |
} |
625 |
) |
|
626 |
} |
|
627 |
||
628 |
void |
|
360
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
629 |
ReferenceProcessor::pp2_work_concurrent_discovery(DiscoveredList& refs_list, |
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
630 |
BoolObjectClosure* is_alive, |
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
631 |
OopClosure* keep_alive, |
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
632 |
VoidClosure* complete_gc) { |
1 | 633 |
assert(!discovery_is_atomic(), "Error"); |
24845
e8b8894a77df
8043239: G1: Missing post barrier in processing of j.l.ref.Reference objects
brutisso
parents:
24424
diff
changeset
|
634 |
DiscoveredListIterator iter(refs_list, keep_alive, is_alive); |
1 | 635 |
while (iter.has_next()) { |
636 |
iter.load_ptrs(DEBUG_ONLY(true /* allow_null_referent */)); |
|
360
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
637 |
HeapWord* next_addr = java_lang_ref_Reference::next_addr(iter.obj()); |
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
638 |
oop next = java_lang_ref_Reference::next(iter.obj()); |
1 | 639 |
if ((iter.referent() == NULL || iter.is_referent_alive() || |
360
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
640 |
next != NULL)) { |
26831
5c67eb0c46bc
8035729: Code using assert(is_oop_or_null) needs better error messages
mlarsson
parents:
25350
diff
changeset
|
641 |
assert(next->is_oop_or_null(), err_msg("Expected an oop or NULL for next field at " PTR_FORMAT, p2i(next))); |
1 | 642 |
// Remove Reference object from list |
643 |
iter.remove(); |
|
644 |
// Trace the cohorts |
|
645 |
iter.make_referent_alive(); |
|
360
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
646 |
if (UseCompressedOops) { |
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
647 |
keep_alive->do_oop((narrowOop*)next_addr); |
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
648 |
} else { |
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
649 |
keep_alive->do_oop((oop*)next_addr); |
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
650 |
} |
1605
6b43d186eb8d
6722113: CMS: Incorrect overflow handling during precleaning of Reference lists
ysr
parents:
1388
diff
changeset
|
651 |
iter.move_to_next(); |
1 | 652 |
} else { |
653 |
iter.next(); |
|
654 |
} |
|
655 |
} |
|
656 |
// Now close the newly reachable set |
|
657 |
complete_gc->do_void(); |
|
658 |
NOT_PRODUCT( |
|
8688
493d12ccc6db
6668573: CMS: reference processing crash if ParallelCMSThreads > ParallelGCThreads
ysr
parents:
7420
diff
changeset
|
659 |
if (PrintGCDetails && TraceReferenceGC && (iter.processed() > 0)) { |
29800
fa5f7a2bf717
8076073: shared: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents:
29580
diff
changeset
|
660 |
gclog_or_tty->print_cr(" Dropped " SIZE_FORMAT " active Refs out of " SIZE_FORMAT |
fa5f7a2bf717
8076073: shared: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents:
29580
diff
changeset
|
661 |
" Refs in discovered list " INTPTR_FORMAT, |
fa5f7a2bf717
8076073: shared: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents:
29580
diff
changeset
|
662 |
iter.removed(), iter.processed(), p2i(refs_list.head())); |
1 | 663 |
} |
664 |
) |
|
665 |
} |
|
666 |
||
667 |
// Traverse the list and process the referents, by either |
|
360
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
668 |
// clearing them or keeping them (and their reachable |
1 | 669 |
// closure) alive. |
670 |
void |
|
360
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
671 |
ReferenceProcessor::process_phase3(DiscoveredList& refs_list, |
1 | 672 |
bool clear_referent, |
673 |
BoolObjectClosure* is_alive, |
|
674 |
OopClosure* keep_alive, |
|
675 |
VoidClosure* complete_gc) { |
|
6759
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
676 |
ResourceMark rm; |
24845
e8b8894a77df
8043239: G1: Missing post barrier in processing of j.l.ref.Reference objects
brutisso
parents:
24424
diff
changeset
|
677 |
DiscoveredListIterator iter(refs_list, keep_alive, is_alive); |
1 | 678 |
while (iter.has_next()) { |
679 |
iter.update_discovered(); |
|
680 |
iter.load_ptrs(DEBUG_ONLY(false /* allow_null_referent */)); |
|
681 |
if (clear_referent) { |
|
682 |
// NULL out referent pointer |
|
683 |
iter.clear_referent(); |
|
684 |
} else { |
|
685 |
// keep the referent around |
|
686 |
iter.make_referent_alive(); |
|
687 |
} |
|
688 |
if (TraceReferenceGC) { |
|
689 |
gclog_or_tty->print_cr("Adding %sreference (" INTPTR_FORMAT ": %s) as pending", |
|
690 |
clear_referent ? "cleared " : "", |
|
29800
fa5f7a2bf717
8076073: shared: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents:
29580
diff
changeset
|
691 |
p2i(iter.obj()), iter.obj()->klass()->internal_name()); |
1 | 692 |
} |
693 |
assert(iter.obj()->is_oop(UseConcMarkSweepGC), "Adding a bad reference"); |
|
694 |
iter.next(); |
|
695 |
} |
|
10524
6594ca81279a
7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents:
8688
diff
changeset
|
696 |
// Remember to update the next pointer of the last ref. |
1 | 697 |
iter.update_discovered(); |
698 |
// Close the reachable set |
|
699 |
complete_gc->do_void(); |
|
700 |
} |
|
701 |
||
702 |
void |
|
10524
6594ca81279a
7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents:
8688
diff
changeset
|
703 |
ReferenceProcessor::clear_discovered_references(DiscoveredList& refs_list) { |
6594ca81279a
7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents:
8688
diff
changeset
|
704 |
oop obj = NULL; |
6594ca81279a
7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents:
8688
diff
changeset
|
705 |
oop next = refs_list.head(); |
6594ca81279a
7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents:
8688
diff
changeset
|
706 |
while (next != obj) { |
6594ca81279a
7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents:
8688
diff
changeset
|
707 |
obj = next; |
6594ca81279a
7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents:
8688
diff
changeset
|
708 |
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
|
709 |
java_lang_ref_Reference::set_discovered_raw(obj, NULL); |
1 | 710 |
} |
10524
6594ca81279a
7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents:
8688
diff
changeset
|
711 |
refs_list.set_head(NULL); |
360
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
712 |
refs_list.set_length(0); |
1 | 713 |
} |
714 |
||
10524
6594ca81279a
7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents:
8688
diff
changeset
|
715 |
void |
6594ca81279a
7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents:
8688
diff
changeset
|
716 |
ReferenceProcessor::abandon_partial_discovered_list(DiscoveredList& refs_list) { |
6594ca81279a
7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents:
8688
diff
changeset
|
717 |
clear_discovered_references(refs_list); |
6594ca81279a
7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents:
8688
diff
changeset
|
718 |
} |
6594ca81279a
7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents:
8688
diff
changeset
|
719 |
|
1374
4c24294029a9
6711316: Open source the Garbage-First garbage collector
ysr
parents:
360
diff
changeset
|
720 |
void ReferenceProcessor::abandon_partial_discovery() { |
4c24294029a9
6711316: Open source the Garbage-First garbage collector
ysr
parents:
360
diff
changeset
|
721 |
// loop over the lists |
11396
917d8673b5ef
7121618: Change type of number of GC workers to unsigned int.
jmasa
parents:
11251
diff
changeset
|
722 |
for (uint i = 0; i < _max_num_q * number_of_subclasses_of_ref(); i++) { |
6759
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
723 |
if (TraceReferenceGC && PrintGCDetails && ((i % _max_num_q) == 0)) { |
10670
4ea0e7d2ffbc
6484982: G1: process references during evacuation pauses
johnc
parents:
10526
diff
changeset
|
724 |
gclog_or_tty->print_cr("\nAbandoning %s discovered list", list_name(i)); |
1374
4c24294029a9
6711316: Open source the Garbage-First garbage collector
ysr
parents:
360
diff
changeset
|
725 |
} |
10747
dfdb9eb56e49
7095243: Disambiguate ReferenceProcessor::_discoveredSoftRefs
johnc
parents:
10683
diff
changeset
|
726 |
abandon_partial_discovered_list(_discovered_refs[i]); |
1 | 727 |
} |
728 |
} |
|
729 |
||
730 |
class RefProcPhase1Task: public AbstractRefProcTaskExecutor::ProcessTask { |
|
731 |
public: |
|
732 |
RefProcPhase1Task(ReferenceProcessor& ref_processor, |
|
733 |
DiscoveredList refs_lists[], |
|
734 |
ReferencePolicy* policy, |
|
735 |
bool marks_oops_alive) |
|
736 |
: ProcessTask(ref_processor, refs_lists, marks_oops_alive), |
|
737 |
_policy(policy) |
|
738 |
{ } |
|
739 |
virtual void work(unsigned int i, BoolObjectClosure& is_alive, |
|
740 |
OopClosure& keep_alive, |
|
741 |
VoidClosure& complete_gc) |
|
742 |
{ |
|
6759
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
743 |
Thread* thr = Thread::current(); |
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
744 |
int refs_list_index = ((WorkerThread*)thr)->id(); |
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
745 |
_ref_processor.process_phase1(_refs_lists[refs_list_index], _policy, |
1 | 746 |
&is_alive, &keep_alive, &complete_gc); |
747 |
} |
|
748 |
private: |
|
749 |
ReferencePolicy* _policy; |
|
750 |
}; |
|
751 |
||
752 |
class RefProcPhase2Task: public AbstractRefProcTaskExecutor::ProcessTask { |
|
753 |
public: |
|
754 |
RefProcPhase2Task(ReferenceProcessor& ref_processor, |
|
755 |
DiscoveredList refs_lists[], |
|
756 |
bool marks_oops_alive) |
|
757 |
: ProcessTask(ref_processor, refs_lists, marks_oops_alive) |
|
758 |
{ } |
|
759 |
virtual void work(unsigned int i, BoolObjectClosure& is_alive, |
|
760 |
OopClosure& keep_alive, |
|
761 |
VoidClosure& complete_gc) |
|
762 |
{ |
|
763 |
_ref_processor.process_phase2(_refs_lists[i], |
|
764 |
&is_alive, &keep_alive, &complete_gc); |
|
765 |
} |
|
766 |
}; |
|
767 |
||
768 |
class RefProcPhase3Task: public AbstractRefProcTaskExecutor::ProcessTask { |
|
769 |
public: |
|
770 |
RefProcPhase3Task(ReferenceProcessor& ref_processor, |
|
771 |
DiscoveredList refs_lists[], |
|
772 |
bool clear_referent, |
|
773 |
bool marks_oops_alive) |
|
774 |
: ProcessTask(ref_processor, refs_lists, marks_oops_alive), |
|
775 |
_clear_referent(clear_referent) |
|
776 |
{ } |
|
777 |
virtual void work(unsigned int i, BoolObjectClosure& is_alive, |
|
778 |
OopClosure& keep_alive, |
|
779 |
VoidClosure& complete_gc) |
|
780 |
{ |
|
6759
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
781 |
// Don't use "refs_list_index" calculated in this way because |
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
782 |
// balance_queues() has moved the Ref's into the first n queues. |
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
783 |
// Thread* thr = Thread::current(); |
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
784 |
// int refs_list_index = ((WorkerThread*)thr)->id(); |
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
785 |
// _ref_processor.process_phase3(_refs_lists[refs_list_index], _clear_referent, |
1 | 786 |
_ref_processor.process_phase3(_refs_lists[i], _clear_referent, |
787 |
&is_alive, &keep_alive, &complete_gc); |
|
788 |
} |
|
789 |
private: |
|
790 |
bool _clear_referent; |
|
791 |
}; |
|
792 |
||
793 |
// Balances reference queues. |
|
6759
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
794 |
// 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
|
795 |
// 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
|
796 |
// corresponding to the active workers will be processed. |
1 | 797 |
void ReferenceProcessor::balance_queues(DiscoveredList ref_lists[]) |
798 |
{ |
|
799 |
// calculate total length |
|
800 |
size_t total_refs = 0; |
|
6759
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
801 |
if (TraceReferenceGC && PrintGCDetails) { |
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
802 |
gclog_or_tty->print_cr("\nBalance ref_lists "); |
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
803 |
} |
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
804 |
|
11396
917d8673b5ef
7121618: Change type of number of GC workers to unsigned int.
jmasa
parents:
11251
diff
changeset
|
805 |
for (uint i = 0; i < _max_num_q; ++i) { |
1 | 806 |
total_refs += ref_lists[i].length(); |
6759
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
807 |
if (TraceReferenceGC && PrintGCDetails) { |
29800
fa5f7a2bf717
8076073: shared: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents:
29580
diff
changeset
|
808 |
gclog_or_tty->print(SIZE_FORMAT " ", ref_lists[i].length()); |
6759
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
809 |
} |
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
810 |
} |
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
811 |
if (TraceReferenceGC && PrintGCDetails) { |
29800
fa5f7a2bf717
8076073: shared: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents:
29580
diff
changeset
|
812 |
gclog_or_tty->print_cr(" = " SIZE_FORMAT, total_refs); |
1 | 813 |
} |
814 |
size_t avg_refs = total_refs / _num_q + 1; |
|
11396
917d8673b5ef
7121618: Change type of number of GC workers to unsigned int.
jmasa
parents:
11251
diff
changeset
|
815 |
uint to_idx = 0; |
917d8673b5ef
7121618: Change type of number of GC workers to unsigned int.
jmasa
parents:
11251
diff
changeset
|
816 |
for (uint from_idx = 0; from_idx < _max_num_q; from_idx++) { |
6759
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
817 |
bool move_all = false; |
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
818 |
if (from_idx >= _num_q) { |
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
819 |
move_all = ref_lists[from_idx].length() > 0; |
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
820 |
} |
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
821 |
while ((ref_lists[from_idx].length() > avg_refs) || |
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
822 |
move_all) { |
1 | 823 |
assert(to_idx < _num_q, "Sanity Check!"); |
824 |
if (ref_lists[to_idx].length() < avg_refs) { |
|
825 |
// move superfluous refs |
|
6759
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
826 |
size_t refs_to_move; |
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
827 |
// 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
|
828 |
if (move_all) { |
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
829 |
refs_to_move = MIN2(ref_lists[from_idx].length(), |
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
830 |
avg_refs - ref_lists[to_idx].length()); |
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
831 |
} else { |
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
832 |
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
|
833 |
avg_refs - ref_lists[to_idx].length()); |
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
834 |
} |
10524
6594ca81279a
7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents:
8688
diff
changeset
|
835 |
|
6594ca81279a
7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents:
8688
diff
changeset
|
836 |
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
|
837 |
|
1 | 838 |
oop move_head = ref_lists[from_idx].head(); |
839 |
oop move_tail = move_head; |
|
840 |
oop new_head = move_head; |
|
841 |
// find an element to split the list on |
|
842 |
for (size_t j = 0; j < refs_to_move; ++j) { |
|
843 |
move_tail = new_head; |
|
360
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
844 |
new_head = java_lang_ref_Reference::discovered(new_head); |
1 | 845 |
} |
10524
6594ca81279a
7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents:
8688
diff
changeset
|
846 |
|
6594ca81279a
7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents:
8688
diff
changeset
|
847 |
// Add the chain to the to list. |
6594ca81279a
7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents:
8688
diff
changeset
|
848 |
if (ref_lists[to_idx].head() == NULL) { |
6594ca81279a
7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents:
8688
diff
changeset
|
849 |
// 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
|
850 |
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
|
851 |
} else { |
24845
e8b8894a77df
8043239: G1: Missing post barrier in processing of j.l.ref.Reference objects
brutisso
parents:
24424
diff
changeset
|
852 |
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
|
853 |
} |
1 | 854 |
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
|
855 |
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
|
856 |
|
6594ca81279a
7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents:
8688
diff
changeset
|
857 |
// Remove the chain from the from list. |
6594ca81279a
7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents:
8688
diff
changeset
|
858 |
if (move_tail == new_head) { |
6594ca81279a
7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents:
8688
diff
changeset
|
859 |
// 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
|
860 |
ref_lists[from_idx].set_head(NULL); |
6594ca81279a
7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents:
8688
diff
changeset
|
861 |
} else { |
6594ca81279a
7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents:
8688
diff
changeset
|
862 |
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
|
863 |
} |
1605
6b43d186eb8d
6722113: CMS: Incorrect overflow handling during precleaning of Reference lists
ysr
parents:
1388
diff
changeset
|
864 |
ref_lists[from_idx].dec_length(refs_to_move); |
6759
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
865 |
if (ref_lists[from_idx].length() == 0) { |
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
866 |
break; |
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
867 |
} |
1 | 868 |
} else { |
6759
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
869 |
to_idx = (to_idx + 1) % _num_q; |
1 | 870 |
} |
871 |
} |
|
872 |
} |
|
6759
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
873 |
#ifdef ASSERT |
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
874 |
size_t balanced_total_refs = 0; |
11396
917d8673b5ef
7121618: Change type of number of GC workers to unsigned int.
jmasa
parents:
11251
diff
changeset
|
875 |
for (uint i = 0; i < _max_num_q; ++i) { |
6759
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
876 |
balanced_total_refs += ref_lists[i].length(); |
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
877 |
if (TraceReferenceGC && PrintGCDetails) { |
29800
fa5f7a2bf717
8076073: shared: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents:
29580
diff
changeset
|
878 |
gclog_or_tty->print(SIZE_FORMAT " ", ref_lists[i].length()); |
6759
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
879 |
} |
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
880 |
} |
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
881 |
if (TraceReferenceGC && PrintGCDetails) { |
29800
fa5f7a2bf717
8076073: shared: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents:
29580
diff
changeset
|
882 |
gclog_or_tty->print_cr(" = " SIZE_FORMAT, balanced_total_refs); |
6759
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
883 |
gclog_or_tty->flush(); |
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
884 |
} |
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
885 |
assert(total_refs == balanced_total_refs, "Balancing was incomplete"); |
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
886 |
#endif |
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
887 |
} |
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
888 |
|
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
889 |
void ReferenceProcessor::balance_all_queues() { |
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
890 |
balance_queues(_discoveredSoftRefs); |
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
891 |
balance_queues(_discoveredWeakRefs); |
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
892 |
balance_queues(_discoveredFinalRefs); |
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
893 |
balance_queues(_discoveredPhantomRefs); |
28510 | 894 |
balance_queues(_discoveredCleanerRefs); |
1 | 895 |
} |
896 |
||
18025 | 897 |
size_t |
1 | 898 |
ReferenceProcessor::process_discovered_reflist( |
899 |
DiscoveredList refs_lists[], |
|
900 |
ReferencePolicy* policy, |
|
901 |
bool clear_referent, |
|
902 |
BoolObjectClosure* is_alive, |
|
903 |
OopClosure* keep_alive, |
|
904 |
VoidClosure* complete_gc, |
|
905 |
AbstractRefProcTaskExecutor* task_executor) |
|
906 |
{ |
|
6759
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
907 |
bool mt_processing = task_executor != NULL && _processing_is_mt; |
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
908 |
// If discovery used MT and a dynamic number of GC threads, then |
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
909 |
// the queues must be balanced for correctness if fewer than the |
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
910 |
// maximum number of queues were used. The number of queue used |
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
911 |
// during discovery may be different than the number to be used |
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
912 |
// for processing so don't depend of _num_q < _max_num_q as part |
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
913 |
// of the test. |
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
914 |
bool must_balance = _discovery_is_mt; |
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
915 |
|
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
916 |
if ((mt_processing && ParallelRefProcBalancingEnabled) || |
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
917 |
must_balance) { |
1 | 918 |
balance_queues(refs_lists); |
919 |
} |
|
18025 | 920 |
|
921 |
size_t total_list_count = total_count(refs_lists); |
|
922 |
||
1 | 923 |
if (PrintReferenceGC && PrintGCDetails) { |
29800
fa5f7a2bf717
8076073: shared: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents:
29580
diff
changeset
|
924 |
gclog_or_tty->print(", " SIZE_FORMAT " refs", total_list_count); |
1 | 925 |
} |
926 |
||
927 |
// Phase 1 (soft refs only): |
|
928 |
// . Traverse the list and remove any SoftReferences whose |
|
929 |
// referents are not alive, but that should be kept alive for |
|
930 |
// policy reasons. Keep alive the transitive closure of all |
|
931 |
// such referents. |
|
932 |
if (policy != NULL) { |
|
6759
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
933 |
if (mt_processing) { |
1 | 934 |
RefProcPhase1Task phase1(*this, refs_lists, policy, true /*marks_oops_alive*/); |
935 |
task_executor->execute(phase1); |
|
936 |
} else { |
|
11396
917d8673b5ef
7121618: Change type of number of GC workers to unsigned int.
jmasa
parents:
11251
diff
changeset
|
937 |
for (uint i = 0; i < _max_num_q; i++) { |
1 | 938 |
process_phase1(refs_lists[i], policy, |
939 |
is_alive, keep_alive, complete_gc); |
|
940 |
} |
|
941 |
} |
|
942 |
} else { // policy == NULL |
|
943 |
assert(refs_lists != _discoveredSoftRefs, |
|
944 |
"Policy must be specified for soft references."); |
|
945 |
} |
|
946 |
||
947 |
// Phase 2: |
|
948 |
// . Traverse the list and remove any refs whose referents are alive. |
|
6759
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
949 |
if (mt_processing) { |
1 | 950 |
RefProcPhase2Task phase2(*this, refs_lists, !discovery_is_atomic() /*marks_oops_alive*/); |
951 |
task_executor->execute(phase2); |
|
952 |
} else { |
|
11396
917d8673b5ef
7121618: Change type of number of GC workers to unsigned int.
jmasa
parents:
11251
diff
changeset
|
953 |
for (uint i = 0; i < _max_num_q; i++) { |
1 | 954 |
process_phase2(refs_lists[i], is_alive, keep_alive, complete_gc); |
955 |
} |
|
956 |
} |
|
957 |
||
958 |
// Phase 3: |
|
959 |
// . Traverse the list and process referents as appropriate. |
|
6759
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
960 |
if (mt_processing) { |
1 | 961 |
RefProcPhase3Task phase3(*this, refs_lists, clear_referent, true /*marks_oops_alive*/); |
962 |
task_executor->execute(phase3); |
|
963 |
} else { |
|
11396
917d8673b5ef
7121618: Change type of number of GC workers to unsigned int.
jmasa
parents:
11251
diff
changeset
|
964 |
for (uint i = 0; i < _max_num_q; i++) { |
1 | 965 |
process_phase3(refs_lists[i], clear_referent, |
966 |
is_alive, keep_alive, complete_gc); |
|
967 |
} |
|
968 |
} |
|
18025 | 969 |
|
970 |
return total_list_count; |
|
1 | 971 |
} |
972 |
||
973 |
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
|
974 |
uint id = 0; |
1 | 975 |
// Determine the queue index to use for this object. |
976 |
if (_discovery_is_mt) { |
|
977 |
// During a multi-threaded discovery phase, |
|
978 |
// each thread saves to its "own" list. |
|
979 |
Thread* thr = Thread::current(); |
|
7399
4ecd771fa2d1
6983204: G1: Nightly test nsk/regression/b4958615 failing with +ExplicitGCInvokesConcurrent
johnc
parents:
7397
diff
changeset
|
980 |
id = thr->as_Worker_thread()->id(); |
1 | 981 |
} else { |
982 |
// single-threaded discovery, we save in round-robin |
|
983 |
// fashion to each of the lists. |
|
984 |
if (_processing_is_mt) { |
|
985 |
id = next_id(); |
|
986 |
} |
|
987 |
} |
|
29580
a67a581cfe11
8073315: Enable gcc -Wtype-limits and fix upcoming issues.
goetz
parents:
28510
diff
changeset
|
988 |
assert(id < _max_num_q, "Id is out-of-bounds (call Freud?)"); |
1 | 989 |
|
990 |
// Get the discovered queue to which we will add |
|
991 |
DiscoveredList* list = NULL; |
|
992 |
switch (rt) { |
|
993 |
case REF_OTHER: |
|
994 |
// Unknown reference type, no special treatment |
|
995 |
break; |
|
996 |
case REF_SOFT: |
|
997 |
list = &_discoveredSoftRefs[id]; |
|
998 |
break; |
|
999 |
case REF_WEAK: |
|
1000 |
list = &_discoveredWeakRefs[id]; |
|
1001 |
break; |
|
1002 |
case REF_FINAL: |
|
1003 |
list = &_discoveredFinalRefs[id]; |
|
1004 |
break; |
|
1005 |
case REF_PHANTOM: |
|
1006 |
list = &_discoveredPhantomRefs[id]; |
|
1007 |
break; |
|
28510 | 1008 |
case REF_CLEANER: |
1009 |
list = &_discoveredCleanerRefs[id]; |
|
1010 |
break; |
|
1 | 1011 |
case REF_NONE: |
13738
d67be49a5beb
7195833: NPG: Rename instanceClassLoaderKlass, instanceRefKlass and instanceMirrorKlass
coleenp
parents:
13728
diff
changeset
|
1012 |
// we should not reach here if we are an InstanceRefKlass |
1 | 1013 |
default: |
1014 |
ShouldNotReachHere(); |
|
1015 |
} |
|
6759
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
1016 |
if (TraceReferenceGC && PrintGCDetails) { |
29800
fa5f7a2bf717
8076073: shared: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents:
29580
diff
changeset
|
1017 |
gclog_or_tty->print_cr("Thread %d gets list " INTPTR_FORMAT, id, p2i(list)); |
6759
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
1018 |
} |
1 | 1019 |
return list; |
1020 |
} |
|
1021 |
||
360
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
1022 |
inline void |
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
1023 |
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
|
1024 |
oop obj, |
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
1025 |
HeapWord* discovered_addr) { |
1 | 1026 |
assert(_discovery_is_mt, "!_discovery_is_mt should have been handled by caller"); |
1027 |
// First we must make sure this object is only enqueued once. CAS in a non null |
|
1028 |
// discovered_addr. |
|
1374
4c24294029a9
6711316: Open source the Garbage-First garbage collector
ysr
parents:
360
diff
changeset
|
1029 |
oop current_head = refs_list.head(); |
10524
6594ca81279a
7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents:
8688
diff
changeset
|
1030 |
// 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
|
1031 |
oop next_discovered = (current_head != NULL) ? current_head : obj; |
1374
4c24294029a9
6711316: Open source the Garbage-First garbage collector
ysr
parents:
360
diff
changeset
|
1032 |
|
10524
6594ca81279a
7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents:
8688
diff
changeset
|
1033 |
oop retest = oopDesc::atomic_compare_exchange_oop(next_discovered, discovered_addr, |
360
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
1034 |
NULL); |
1 | 1035 |
if (retest == NULL) { |
1036 |
// 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
|
1037 |
// We have separate lists for enqueueing, so no synchronization |
1 | 1038 |
// is necessary. |
360
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
1039 |
refs_list.set_head(obj); |
1605
6b43d186eb8d
6722113: CMS: Incorrect overflow handling during precleaning of Reference lists
ysr
parents:
1388
diff
changeset
|
1040 |
refs_list.inc_length(1); |
7399
4ecd771fa2d1
6983204: G1: Nightly test nsk/regression/b4958615 failing with +ExplicitGCInvokesConcurrent
johnc
parents:
7397
diff
changeset
|
1041 |
|
4ecd771fa2d1
6983204: G1: Nightly test nsk/regression/b4958615 failing with +ExplicitGCInvokesConcurrent
johnc
parents:
7397
diff
changeset
|
1042 |
if (TraceReferenceGC) { |
10526
3e92f211533f
4965777: GC changes to support use of discovered field for pending references
ysr
parents:
10524
diff
changeset
|
1043 |
gclog_or_tty->print_cr("Discovered reference (mt) (" INTPTR_FORMAT ": %s)", |
29800
fa5f7a2bf717
8076073: shared: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents:
29580
diff
changeset
|
1044 |
p2i(obj), obj->klass()->internal_name()); |
7399
4ecd771fa2d1
6983204: G1: Nightly test nsk/regression/b4958615 failing with +ExplicitGCInvokesConcurrent
johnc
parents:
7397
diff
changeset
|
1045 |
} |
1 | 1046 |
} else { |
1047 |
// If retest was non NULL, another thread beat us to it: |
|
1048 |
// The reference has already been discovered... |
|
1049 |
if (TraceReferenceGC) { |
|
10526
3e92f211533f
4965777: GC changes to support use of discovered field for pending references
ysr
parents:
10524
diff
changeset
|
1050 |
gclog_or_tty->print_cr("Already discovered reference (" INTPTR_FORMAT ": %s)", |
29800
fa5f7a2bf717
8076073: shared: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents:
29580
diff
changeset
|
1051 |
p2i(obj), obj->klass()->internal_name()); |
1 | 1052 |
} |
1053 |
} |
|
1054 |
} |
|
1055 |
||
7420
24071b15dde6
7005259: CMS: BubbleUpRef asserts referent(obj)->is_oop() failed: Enqueued a bad referent
ysr
parents:
7399
diff
changeset
|
1056 |
#ifndef PRODUCT |
24071b15dde6
7005259: CMS: BubbleUpRef asserts referent(obj)->is_oop() failed: Enqueued a bad referent
ysr
parents:
7399
diff
changeset
|
1057 |
// 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
|
1058 |
// 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
|
1059 |
// 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
|
1060 |
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
|
1061 |
bool da = discovery_is_atomic(); |
24071b15dde6
7005259: CMS: BubbleUpRef asserts referent(obj)->is_oop() failed: Enqueued a bad referent
ysr
parents:
7399
diff
changeset
|
1062 |
oop referent = java_lang_ref_Reference::referent(obj); |
24071b15dde6
7005259: CMS: BubbleUpRef asserts referent(obj)->is_oop() failed: Enqueued a bad referent
ysr
parents:
7399
diff
changeset
|
1063 |
assert(da ? referent->is_oop() : referent->is_oop_or_null(), |
24071b15dde6
7005259: CMS: BubbleUpRef asserts referent(obj)->is_oop() failed: Enqueued a bad referent
ysr
parents:
7399
diff
changeset
|
1064 |
err_msg("Bad referent " INTPTR_FORMAT " found in Reference " |
24071b15dde6
7005259: CMS: BubbleUpRef asserts referent(obj)->is_oop() failed: Enqueued a bad referent
ysr
parents:
7399
diff
changeset
|
1065 |
INTPTR_FORMAT " during %satomic discovery ", |
29800
fa5f7a2bf717
8076073: shared: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents:
29580
diff
changeset
|
1066 |
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
|
1067 |
} |
24071b15dde6
7005259: CMS: BubbleUpRef asserts referent(obj)->is_oop() failed: Enqueued a bad referent
ysr
parents:
7399
diff
changeset
|
1068 |
#endif |
24071b15dde6
7005259: CMS: BubbleUpRef asserts referent(obj)->is_oop() failed: Enqueued a bad referent
ysr
parents:
7399
diff
changeset
|
1069 |
|
1 | 1070 |
// We mention two of several possible choices here: |
1071 |
// #0: if the reference object is not in the "originating generation" |
|
1072 |
// (or part of the heap being collected, indicated by our "span" |
|
1073 |
// we don't treat it specially (i.e. we scan it as we would |
|
1074 |
// 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
|
1075 |
// This means that references can't be discovered unless their |
1 | 1076 |
// referent is also in the same span. This is the simplest, |
1077 |
// most "local" and most conservative approach, albeit one |
|
1078 |
// that may cause weak references to be enqueued least promptly. |
|
1079 |
// We call this choice the "ReferenceBasedDiscovery" policy. |
|
1080 |
// #1: the reference object may be in any generation (span), but if |
|
1081 |
// the referent is in the generation (span) being currently collected |
|
1082 |
// then we can discover the reference object, provided |
|
1083 |
// the object has not already been discovered by |
|
1084 |
// a different concurrently running collector (as may be the |
|
1085 |
// case, for instance, if the reference object is in CMS and |
|
1086 |
// the referent in DefNewGeneration), and provided the processing |
|
1087 |
// of this reference object by the current collector will |
|
1088 |
// appear atomic to every other collector in the system. |
|
1089 |
// (Thus, for instance, a concurrent collector may not |
|
1090 |
// discover references in other generations even if the |
|
1091 |
// referent is in its own generation). This policy may, |
|
1092 |
// in certain cases, enqueue references somewhat sooner than |
|
1093 |
// might Policy #0 above, but at marginally increased cost |
|
1094 |
// and complexity in processing these references. |
|
1095 |
// We call this choice the "RefeferentBasedDiscovery" policy. |
|
1096 |
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
|
1097 |
// Make sure we are discovering refs (rather than processing discovered refs). |
1 | 1098 |
if (!_discovering_refs || !RegisterReferences) { |
1099 |
return false; |
|
1100 |
} |
|
10526
3e92f211533f
4965777: GC changes to support use of discovered field for pending references
ysr
parents:
10524
diff
changeset
|
1101 |
// We only discover active references. |
360
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
1102 |
oop next = java_lang_ref_Reference::next(obj); |
10526
3e92f211533f
4965777: GC changes to support use of discovered field for pending references
ysr
parents:
10524
diff
changeset
|
1103 |
if (next != NULL) { // Ref is no longer active |
1 | 1104 |
return false; |
1105 |
} |
|
1106 |
||
1107 |
HeapWord* obj_addr = (HeapWord*)obj; |
|
1108 |
if (RefDiscoveryPolicy == ReferenceBasedDiscovery && |
|
1109 |
!_span.contains(obj_addr)) { |
|
1110 |
// Reference is not in the originating generation; |
|
1111 |
// don't treat it specially (i.e. we want to scan it as a normal |
|
1112 |
// object with strong references). |
|
1113 |
return false; |
|
1114 |
} |
|
1115 |
||
10526
3e92f211533f
4965777: GC changes to support use of discovered field for pending references
ysr
parents:
10524
diff
changeset
|
1116 |
// 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
|
1117 |
// known to be strongly reachable. |
1 | 1118 |
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
|
1119 |
verify_referent(obj); |
24071b15dde6
7005259: CMS: BubbleUpRef asserts referent(obj)->is_oop() failed: Enqueued a bad referent
ysr
parents:
7399
diff
changeset
|
1120 |
if (is_alive_non_header()->do_object_b(java_lang_ref_Reference::referent(obj))) { |
1 | 1121 |
return false; // referent is reachable |
1122 |
} |
|
1123 |
} |
|
1606
dcf9714addbe
6684579: SoftReference processing can be made more efficient
ysr
parents:
1605
diff
changeset
|
1124 |
if (rt == REF_SOFT) { |
dcf9714addbe
6684579: SoftReference processing can be made more efficient
ysr
parents:
1605
diff
changeset
|
1125 |
// 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
|
1126 |
// current candidates for clearing, in which case we |
dcf9714addbe
6684579: SoftReference processing can be made more efficient
ysr
parents:
1605
diff
changeset
|
1127 |
// can mark through them now, rather than delaying that |
dcf9714addbe
6684579: SoftReference processing can be made more efficient
ysr
parents:
1605
diff
changeset
|
1128 |
// to the reference-processing phase. Since all current |
dcf9714addbe
6684579: SoftReference processing can be made more efficient
ysr
parents:
1605
diff
changeset
|
1129 |
// time-stamp policies advance the soft-ref clock only |
dcf9714addbe
6684579: SoftReference processing can be made more efficient
ysr
parents:
1605
diff
changeset
|
1130 |
// at a major collection cycle, this is always currently |
dcf9714addbe
6684579: SoftReference processing can be made more efficient
ysr
parents:
1605
diff
changeset
|
1131 |
// accurate. |
10683
4b5a5a507864
7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents:
10670
diff
changeset
|
1132 |
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
|
1133 |
return false; |
dcf9714addbe
6684579: SoftReference processing can be made more efficient
ysr
parents:
1605
diff
changeset
|
1134 |
} |
dcf9714addbe
6684579: SoftReference processing can be made more efficient
ysr
parents:
1605
diff
changeset
|
1135 |
} |
1 | 1136 |
|
10670
4ea0e7d2ffbc
6484982: G1: process references during evacuation pauses
johnc
parents:
10526
diff
changeset
|
1137 |
ResourceMark rm; // Needed for tracing. |
4ea0e7d2ffbc
6484982: G1: process references during evacuation pauses
johnc
parents:
10526
diff
changeset
|
1138 |
|
1374
4c24294029a9
6711316: Open source the Garbage-First garbage collector
ysr
parents:
360
diff
changeset
|
1139 |
HeapWord* const discovered_addr = java_lang_ref_Reference::discovered_addr(obj); |
4c24294029a9
6711316: Open source the Garbage-First garbage collector
ysr
parents:
360
diff
changeset
|
1140 |
const oop discovered = java_lang_ref_Reference::discovered(obj); |
26831
5c67eb0c46bc
8035729: Code using assert(is_oop_or_null) needs better error messages
mlarsson
parents:
25350
diff
changeset
|
1141 |
assert(discovered->is_oop_or_null(), err_msg("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
|
1142 |
if (discovered != NULL) { |
1 | 1143 |
// The reference has already been discovered... |
1144 |
if (TraceReferenceGC) { |
|
10526
3e92f211533f
4965777: GC changes to support use of discovered field for pending references
ysr
parents:
10524
diff
changeset
|
1145 |
gclog_or_tty->print_cr("Already discovered reference (" INTPTR_FORMAT ": %s)", |
29800
fa5f7a2bf717
8076073: shared: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents:
29580
diff
changeset
|
1146 |
p2i(obj), obj->klass()->internal_name()); |
1 | 1147 |
} |
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 |
1 | 1170 |
if (_span.contains(obj_addr) || |
7420
24071b15dde6
7005259: CMS: BubbleUpRef asserts referent(obj)->is_oop() failed: Enqueued a bad referent
ysr
parents:
7399
diff
changeset
|
1171 |
(discovery_is_atomic() && |
24071b15dde6
7005259: CMS: BubbleUpRef asserts referent(obj)->is_oop() failed: Enqueued a bad referent
ysr
parents:
7399
diff
changeset
|
1172 |
_span.contains(java_lang_ref_Reference::referent(obj)))) { |
1 | 1173 |
// should_enqueue = true; |
1174 |
} else { |
|
1175 |
return false; |
|
1176 |
} |
|
1177 |
} else { |
|
1178 |
assert(RefDiscoveryPolicy == ReferenceBasedDiscovery && |
|
1179 |
_span.contains(obj_addr), "code inconsistency"); |
|
1180 |
} |
|
1181 |
||
1182 |
// Get the right type of discovered queue head. |
|
1183 |
DiscoveredList* list = get_discovered_list(rt); |
|
1184 |
if (list == NULL) { |
|
1185 |
return false; // nothing special needs to be done |
|
1186 |
} |
|
1187 |
||
1188 |
if (_discovery_is_mt) { |
|
1189 |
add_to_discovered_list_mt(*list, obj, discovered_addr); |
|
1190 |
} else { |
|
24845
e8b8894a77df
8043239: G1: Missing post barrier in processing of j.l.ref.Reference objects
brutisso
parents:
24424
diff
changeset
|
1191 |
// 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
|
1192 |
// the discovered references. |
1374
4c24294029a9
6711316: Open source the Garbage-First garbage collector
ysr
parents:
360
diff
changeset
|
1193 |
oop current_head = list->head(); |
10524
6594ca81279a
7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents:
8688
diff
changeset
|
1194 |
// 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
|
1195 |
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
|
1196 |
|
1374
4c24294029a9
6711316: Open source the Garbage-First garbage collector
ysr
parents:
360
diff
changeset
|
1197 |
assert(discovered == NULL, "control point invariant"); |
10524
6594ca81279a
7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents:
8688
diff
changeset
|
1198 |
oop_store_raw(discovered_addr, next_discovered); |
1 | 1199 |
list->set_head(obj); |
1605
6b43d186eb8d
6722113: CMS: Incorrect overflow handling during precleaning of Reference lists
ysr
parents:
1388
diff
changeset
|
1200 |
list->inc_length(1); |
1 | 1201 |
|
7399
4ecd771fa2d1
6983204: G1: Nightly test nsk/regression/b4958615 failing with +ExplicitGCInvokesConcurrent
johnc
parents:
7397
diff
changeset
|
1202 |
if (TraceReferenceGC) { |
10526
3e92f211533f
4965777: GC changes to support use of discovered field for pending references
ysr
parents:
10524
diff
changeset
|
1203 |
gclog_or_tty->print_cr("Discovered reference (" INTPTR_FORMAT ": %s)", |
29800
fa5f7a2bf717
8076073: shared: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents:
29580
diff
changeset
|
1204 |
p2i(obj), obj->klass()->internal_name()); |
1 | 1205 |
} |
1206 |
} |
|
10526
3e92f211533f
4965777: GC changes to support use of discovered field for pending references
ysr
parents:
10524
diff
changeset
|
1207 |
assert(obj->is_oop(), "Discovered a bad reference"); |
7420
24071b15dde6
7005259: CMS: BubbleUpRef asserts referent(obj)->is_oop() failed: Enqueued a bad referent
ysr
parents:
7399
diff
changeset
|
1208 |
verify_referent(obj); |
1 | 1209 |
return true; |
1210 |
} |
|
1211 |
||
1212 |
// Preclean the discovered references by removing those |
|
1213 |
// whose referents are alive, and by marking from those that |
|
1214 |
// are not active. These lists can be handled here |
|
1215 |
// in any order and, indeed, concurrently. |
|
1216 |
void ReferenceProcessor::preclean_discovered_references( |
|
1217 |
BoolObjectClosure* is_alive, |
|
1218 |
OopClosure* keep_alive, |
|
1219 |
VoidClosure* complete_gc, |
|
18025 | 1220 |
YieldClosure* yield, |
25350
6423a57e5451
8043607: Add a GC id as a log decoration similar to PrintGCTimeStamps
brutisso
parents:
24845
diff
changeset
|
1221 |
GCTimer* gc_timer, |
6423a57e5451
8043607: Add a GC id as a log decoration similar to PrintGCTimeStamps
brutisso
parents:
24845
diff
changeset
|
1222 |
GCId gc_id) { |
1 | 1223 |
|
1224 |
NOT_PRODUCT(verify_ok_to_handle_reflists()); |
|
1225 |
||
1226 |
// Soft references |
|
1227 |
{ |
|
18025 | 1228 |
GCTraceTime tt("Preclean SoftReferences", PrintGCDetails && PrintReferenceGC, |
25350
6423a57e5451
8043607: Add a GC id as a log decoration similar to PrintGCTimeStamps
brutisso
parents:
24845
diff
changeset
|
1229 |
false, gc_timer, gc_id); |
11396
917d8673b5ef
7121618: Change type of number of GC workers to unsigned int.
jmasa
parents:
11251
diff
changeset
|
1230 |
for (uint i = 0; i < _max_num_q; i++) { |
1605
6b43d186eb8d
6722113: CMS: Incorrect overflow handling during precleaning of Reference lists
ysr
parents:
1388
diff
changeset
|
1231 |
if (yield->should_return()) { |
6b43d186eb8d
6722113: CMS: Incorrect overflow handling during precleaning of Reference lists
ysr
parents:
1388
diff
changeset
|
1232 |
return; |
6b43d186eb8d
6722113: CMS: Incorrect overflow handling during precleaning of Reference lists
ysr
parents:
1388
diff
changeset
|
1233 |
} |
1 | 1234 |
preclean_discovered_reflist(_discoveredSoftRefs[i], is_alive, |
1235 |
keep_alive, complete_gc, yield); |
|
1236 |
} |
|
1237 |
} |
|
1238 |
||
1239 |
// Weak references |
|
1240 |
{ |
|
18025 | 1241 |
GCTraceTime tt("Preclean WeakReferences", PrintGCDetails && PrintReferenceGC, |
25350
6423a57e5451
8043607: Add a GC id as a log decoration similar to PrintGCTimeStamps
brutisso
parents:
24845
diff
changeset
|
1242 |
false, gc_timer, gc_id); |
11396
917d8673b5ef
7121618: Change type of number of GC workers to unsigned int.
jmasa
parents:
11251
diff
changeset
|
1243 |
for (uint i = 0; i < _max_num_q; i++) { |
1605
6b43d186eb8d
6722113: CMS: Incorrect overflow handling during precleaning of Reference lists
ysr
parents:
1388
diff
changeset
|
1244 |
if (yield->should_return()) { |
6b43d186eb8d
6722113: CMS: Incorrect overflow handling during precleaning of Reference lists
ysr
parents:
1388
diff
changeset
|
1245 |
return; |
6b43d186eb8d
6722113: CMS: Incorrect overflow handling during precleaning of Reference lists
ysr
parents:
1388
diff
changeset
|
1246 |
} |
1 | 1247 |
preclean_discovered_reflist(_discoveredWeakRefs[i], is_alive, |
1248 |
keep_alive, complete_gc, yield); |
|
1249 |
} |
|
1250 |
} |
|
1251 |
||
1252 |
// Final references |
|
1253 |
{ |
|
18025 | 1254 |
GCTraceTime tt("Preclean FinalReferences", PrintGCDetails && PrintReferenceGC, |
25350
6423a57e5451
8043607: Add a GC id as a log decoration similar to PrintGCTimeStamps
brutisso
parents:
24845
diff
changeset
|
1255 |
false, gc_timer, gc_id); |
11396
917d8673b5ef
7121618: Change type of number of GC workers to unsigned int.
jmasa
parents:
11251
diff
changeset
|
1256 |
for (uint i = 0; i < _max_num_q; i++) { |
1605
6b43d186eb8d
6722113: CMS: Incorrect overflow handling during precleaning of Reference lists
ysr
parents:
1388
diff
changeset
|
1257 |
if (yield->should_return()) { |
6b43d186eb8d
6722113: CMS: Incorrect overflow handling during precleaning of Reference lists
ysr
parents:
1388
diff
changeset
|
1258 |
return; |
6b43d186eb8d
6722113: CMS: Incorrect overflow handling during precleaning of Reference lists
ysr
parents:
1388
diff
changeset
|
1259 |
} |
1 | 1260 |
preclean_discovered_reflist(_discoveredFinalRefs[i], is_alive, |
1261 |
keep_alive, complete_gc, yield); |
|
1262 |
} |
|
1263 |
} |
|
1264 |
||
1265 |
// Phantom references |
|
1266 |
{ |
|
18025 | 1267 |
GCTraceTime tt("Preclean PhantomReferences", PrintGCDetails && PrintReferenceGC, |
25350
6423a57e5451
8043607: Add a GC id as a log decoration similar to PrintGCTimeStamps
brutisso
parents:
24845
diff
changeset
|
1268 |
false, gc_timer, gc_id); |
11396
917d8673b5ef
7121618: Change type of number of GC workers to unsigned int.
jmasa
parents:
11251
diff
changeset
|
1269 |
for (uint i = 0; i < _max_num_q; i++) { |
1605
6b43d186eb8d
6722113: CMS: Incorrect overflow handling during precleaning of Reference lists
ysr
parents:
1388
diff
changeset
|
1270 |
if (yield->should_return()) { |
6b43d186eb8d
6722113: CMS: Incorrect overflow handling during precleaning of Reference lists
ysr
parents:
1388
diff
changeset
|
1271 |
return; |
6b43d186eb8d
6722113: CMS: Incorrect overflow handling during precleaning of Reference lists
ysr
parents:
1388
diff
changeset
|
1272 |
} |
1 | 1273 |
preclean_discovered_reflist(_discoveredPhantomRefs[i], is_alive, |
1274 |
keep_alive, complete_gc, yield); |
|
1275 |
} |
|
28510 | 1276 |
|
1277 |
// Cleaner references. Included in timing for phantom references. We |
|
1278 |
// expect Cleaner references to be temporary, and don't want to deal with |
|
1279 |
// possible incompatibilities arising from making it more visible. |
|
1280 |
for (uint i = 0; i < _max_num_q; i++) { |
|
1281 |
if (yield->should_return()) { |
|
1282 |
return; |
|
1283 |
} |
|
1284 |
preclean_discovered_reflist(_discoveredCleanerRefs[i], is_alive, |
|
1285 |
keep_alive, complete_gc, yield); |
|
1286 |
} |
|
1 | 1287 |
} |
1288 |
} |
|
1289 |
||
1290 |
// Walk the given discovered ref list, and remove all reference objects |
|
1291 |
// 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
|
1292 |
// 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
|
1293 |
// thus precleaning the ref lists (which happens single-threaded today), |
22551 | 1294 |
// 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
|
1295 |
// 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
|
1296 |
// 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
|
1297 |
// (in this thread). |
360
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
1298 |
void |
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
1299 |
ReferenceProcessor::preclean_discovered_reflist(DiscoveredList& refs_list, |
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
1300 |
BoolObjectClosure* is_alive, |
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
1301 |
OopClosure* keep_alive, |
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
1302 |
VoidClosure* complete_gc, |
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
1303 |
YieldClosure* yield) { |
24845
e8b8894a77df
8043239: G1: Missing post barrier in processing of j.l.ref.Reference objects
brutisso
parents:
24424
diff
changeset
|
1304 |
DiscoveredListIterator iter(refs_list, keep_alive, is_alive); |
1 | 1305 |
while (iter.has_next()) { |
1306 |
iter.load_ptrs(DEBUG_ONLY(true /* allow_null_referent */)); |
|
360
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
1307 |
oop obj = iter.obj(); |
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
1308 |
oop next = java_lang_ref_Reference::next(obj); |
1 | 1309 |
if (iter.referent() == NULL || iter.is_referent_alive() || |
360
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
1310 |
next != NULL) { |
1 | 1311 |
// The referent has been cleared, or is alive, or the Reference is not |
1312 |
// active; we need to trace and mark its cohort. |
|
1313 |
if (TraceReferenceGC) { |
|
1314 |
gclog_or_tty->print_cr("Precleaning Reference (" INTPTR_FORMAT ": %s)", |
|
29800
fa5f7a2bf717
8076073: shared: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents:
29580
diff
changeset
|
1315 |
p2i(iter.obj()), iter.obj()->klass()->internal_name()); |
1 | 1316 |
} |
1317 |
// Remove Reference object from list |
|
1318 |
iter.remove(); |
|
1319 |
// Keep alive its cohort. |
|
1320 |
iter.make_referent_alive(); |
|
360
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
1321 |
if (UseCompressedOops) { |
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
1322 |
narrowOop* next_addr = (narrowOop*)java_lang_ref_Reference::next_addr(obj); |
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
1323 |
keep_alive->do_oop(next_addr); |
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
1324 |
} else { |
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
1325 |
oop* next_addr = (oop*)java_lang_ref_Reference::next_addr(obj); |
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
1326 |
keep_alive->do_oop(next_addr); |
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
178
diff
changeset
|
1327 |
} |
1605
6b43d186eb8d
6722113: CMS: Incorrect overflow handling during precleaning of Reference lists
ysr
parents:
1388
diff
changeset
|
1328 |
iter.move_to_next(); |
1 | 1329 |
} else { |
1330 |
iter.next(); |
|
1331 |
} |
|
1332 |
} |
|
1333 |
// Close the reachable set |
|
1334 |
complete_gc->do_void(); |
|
1335 |
||
1336 |
NOT_PRODUCT( |
|
8688
493d12ccc6db
6668573: CMS: reference processing crash if ParallelCMSThreads > ParallelGCThreads
ysr
parents:
7420
diff
changeset
|
1337 |
if (PrintGCDetails && PrintReferenceGC && (iter.processed() > 0)) { |
29800
fa5f7a2bf717
8076073: shared: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents:
29580
diff
changeset
|
1338 |
gclog_or_tty->print_cr(" Dropped " SIZE_FORMAT " Refs out of " SIZE_FORMAT |
fa5f7a2bf717
8076073: shared: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents:
29580
diff
changeset
|
1339 |
" Refs in discovered list " INTPTR_FORMAT, |
fa5f7a2bf717
8076073: shared: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents:
29580
diff
changeset
|
1340 |
iter.removed(), iter.processed(), p2i(refs_list.head())); |
1 | 1341 |
} |
1342 |
) |
|
1343 |
} |
|
1344 |
||
11396
917d8673b5ef
7121618: Change type of number of GC workers to unsigned int.
jmasa
parents:
11251
diff
changeset
|
1345 |
const char* ReferenceProcessor::list_name(uint i) { |
29580
a67a581cfe11
8073315: Enable gcc -Wtype-limits and fix upcoming issues.
goetz
parents:
28510
diff
changeset
|
1346 |
assert(i <= _max_num_q * number_of_subclasses_of_ref(), |
10670
4ea0e7d2ffbc
6484982: G1: process references during evacuation pauses
johnc
parents:
10526
diff
changeset
|
1347 |
"Out of bounds index"); |
4ea0e7d2ffbc
6484982: G1: process references during evacuation pauses
johnc
parents:
10526
diff
changeset
|
1348 |
|
6759
67b1a69ef5aa
6984287: Regularize how GC parallel workers are specified.
jmasa
parents:
5547
diff
changeset
|
1349 |
int j = i / _max_num_q; |
1 | 1350 |
switch (j) { |
1351 |
case 0: return "SoftRef"; |
|
1352 |
case 1: return "WeakRef"; |
|
1353 |
case 2: return "FinalRef"; |
|
1354 |
case 3: return "PhantomRef"; |
|
28510 | 1355 |
case 4: return "CleanerRef"; |
1 | 1356 |
} |
1357 |
ShouldNotReachHere(); |
|
1358 |
return NULL; |
|
1359 |
} |
|
1360 |
||
1361 |
#ifndef PRODUCT |
|
1362 |
void ReferenceProcessor::verify_ok_to_handle_reflists() { |
|
1363 |
// empty for now |
|
1364 |
} |
|
1365 |
#endif |
|
1366 |
||
1367 |
#ifndef PRODUCT |
|
1368 |
void ReferenceProcessor::clear_discovered_references() { |
|
1369 |
guarantee(!_discovering_refs, "Discovering refs?"); |
|
11396
917d8673b5ef
7121618: Change type of number of GC workers to unsigned int.
jmasa
parents:
11251
diff
changeset
|
1370 |
for (uint i = 0; i < _max_num_q * number_of_subclasses_of_ref(); i++) { |
10747
dfdb9eb56e49
7095243: Disambiguate ReferenceProcessor::_discoveredSoftRefs
johnc
parents:
10683
diff
changeset
|
1371 |
clear_discovered_references(_discovered_refs[i]); |
1 | 1372 |
} |
1373 |
} |
|
10524
6594ca81279a
7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents:
8688
diff
changeset
|
1374 |
|
1 | 1375 |
#endif // PRODUCT |