Mercurial > hg > graal-jvmci-8
annotate src/share/vm/gc_implementation/g1/g1OopClosures.inline.hpp @ 20417:d35872270666
8057658: Enable G1 FullGC extensions
Summary: Refactored the G1 FullGC code to enable it to be extended.
Reviewed-by: mgerdin, brutisso
author | sjohanss |
---|---|
date | Tue, 09 Sep 2014 00:05:25 +0200 |
parents | eec72fa4b108 |
children | 7848fc12602b c132be0fb74d |
rev | line source |
---|---|
342 | 1 /* |
17937
78bbf4d43a14
8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents:
17805
diff
changeset
|
2 * Copyright (c) 2001, 2014, Oracle and/or its affiliates. All rights reserved. |
342 | 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 * | |
1552
c18cbe5936b8
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
1261
diff
changeset
|
19 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA |
c18cbe5936b8
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
1261
diff
changeset
|
20 * or visit www.oracle.com if you need additional information or have any |
c18cbe5936b8
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
1261
diff
changeset
|
21 * questions. |
342 | 22 * |
23 */ | |
24 | |
1972 | 25 #ifndef SHARE_VM_GC_IMPLEMENTATION_G1_G1OOPCLOSURES_INLINE_HPP |
26 #define SHARE_VM_GC_IMPLEMENTATION_G1_G1OOPCLOSURES_INLINE_HPP | |
27 | |
3771 | 28 #include "gc_implementation/g1/concurrentMark.inline.hpp" |
1972 | 29 #include "gc_implementation/g1/g1CollectedHeap.hpp" |
30 #include "gc_implementation/g1/g1OopClosures.hpp" | |
20223
b0c374311c4e
8035400: Move G1ParScanThreadState into its own files
tschatzl
parents:
20198
diff
changeset
|
31 #include "gc_implementation/g1/g1ParScanThreadState.inline.hpp" |
1972 | 32 #include "gc_implementation/g1/g1RemSet.hpp" |
14391 | 33 #include "gc_implementation/g1/g1RemSet.inline.hpp" |
6251
594dff5e3c2e
7173712: G1: Duplicated code in G1UpdateRSOrPushRefOopClosure::do_oop_nv()
johnc
parents:
4839
diff
changeset
|
34 #include "gc_implementation/g1/heapRegionRemSet.hpp" |
20278
2c6ef90f030a
8049421: G1 Class Unloading after completing a concurrent mark cycle
stefank
parents:
20223
diff
changeset
|
35 #include "memory/iterator.inline.hpp" |
20198
c49dcaf78a65
8042737: Introduce umbrella header prefetch.inline.hpp
goetz
parents:
17937
diff
changeset
|
36 #include "runtime/prefetch.inline.hpp" |
1972 | 37 |
342 | 38 /* |
39 * This really ought to be an inline function, but apparently the C++ | |
40 * compiler sometimes sees fit to ignore inline declarations. Sigh. | |
41 */ | |
42 | |
4837
eff609af17d7
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
4126
diff
changeset
|
43 template <class T> |
eff609af17d7
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
4126
diff
changeset
|
44 inline void FilterIntoCSClosure::do_oop_nv(T* p) { |
845
df6caf649ff7
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
342
diff
changeset
|
45 T heap_oop = oopDesc::load_heap_oop(p); |
df6caf649ff7
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
342
diff
changeset
|
46 if (!oopDesc::is_null(heap_oop) && |
20296
a3953c777565
8027959: Early reclamation of large objects in G1
tschatzl
parents:
20278
diff
changeset
|
47 _g1->is_in_cset_or_humongous(oopDesc::decode_heap_oop_not_null(heap_oop))) { |
342 | 48 _oc->do_oop(p); |
49 } | |
50 } | |
51 | |
4837
eff609af17d7
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
4126
diff
changeset
|
52 template <class T> |
eff609af17d7
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
4126
diff
changeset
|
53 inline void FilterOutOfRegionClosure::do_oop_nv(T* p) { |
845
df6caf649ff7
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
342
diff
changeset
|
54 T heap_oop = oopDesc::load_heap_oop(p); |
df6caf649ff7
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
342
diff
changeset
|
55 if (!oopDesc::is_null(heap_oop)) { |
df6caf649ff7
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
342
diff
changeset
|
56 HeapWord* obj_hw = (HeapWord*)oopDesc::decode_heap_oop_not_null(heap_oop); |
df6caf649ff7
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
342
diff
changeset
|
57 if (obj_hw < _r_bottom || obj_hw >= _r_end) { |
df6caf649ff7
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
342
diff
changeset
|
58 _oc->do_oop(p); |
df6caf649ff7
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
342
diff
changeset
|
59 } |
342 | 60 } |
61 } | |
62 | |
845
df6caf649ff7
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
342
diff
changeset
|
63 // This closure is applied to the fields of the objects that have just been copied. |
4837
eff609af17d7
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
4126
diff
changeset
|
64 template <class T> |
eff609af17d7
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
4126
diff
changeset
|
65 inline void G1ParScanClosure::do_oop_nv(T* p) { |
845
df6caf649ff7
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
342
diff
changeset
|
66 T heap_oop = oopDesc::load_heap_oop(p); |
342 | 67 |
845
df6caf649ff7
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
342
diff
changeset
|
68 if (!oopDesc::is_null(heap_oop)) { |
df6caf649ff7
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
342
diff
changeset
|
69 oop obj = oopDesc::decode_heap_oop_not_null(heap_oop); |
20296
a3953c777565
8027959: Early reclamation of large objects in G1
tschatzl
parents:
20278
diff
changeset
|
70 G1CollectedHeap::in_cset_state_t state = _g1->in_cset_state(obj); |
a3953c777565
8027959: Early reclamation of large objects in G1
tschatzl
parents:
20278
diff
changeset
|
71 if (state == G1CollectedHeap::InCSet) { |
845
df6caf649ff7
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
342
diff
changeset
|
72 // We're not going to even bother checking whether the object is |
df6caf649ff7
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
342
diff
changeset
|
73 // already forwarded or not, as this usually causes an immediate |
df6caf649ff7
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
342
diff
changeset
|
74 // stall. We'll try to prefetch the object (for write, given that |
df6caf649ff7
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
342
diff
changeset
|
75 // we might need to install the forwarding reference) and we'll |
df6caf649ff7
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
342
diff
changeset
|
76 // get back to it when pop it from the queue |
df6caf649ff7
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
342
diff
changeset
|
77 Prefetch::write(obj->mark_addr(), 0); |
df6caf649ff7
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
342
diff
changeset
|
78 Prefetch::read(obj->mark_addr(), (HeapWordSize*2)); |
df6caf649ff7
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
342
diff
changeset
|
79 |
df6caf649ff7
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
342
diff
changeset
|
80 // slightly paranoid test; I'm trying to catch potential |
df6caf649ff7
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
342
diff
changeset
|
81 // problems before we go into push_on_queue to know where the |
df6caf649ff7
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
342
diff
changeset
|
82 // problem is coming from |
4126
4406629aa157
7114095: G1: assert(obj == oopDesc::load_decode_heap_oop(p)) failed: p should still be pointing to obj
johnc
parents:
3771
diff
changeset
|
83 assert((obj == oopDesc::load_decode_heap_oop(p)) || |
4406629aa157
7114095: G1: assert(obj == oopDesc::load_decode_heap_oop(p)) failed: p should still be pointing to obj
johnc
parents:
3771
diff
changeset
|
84 (obj->is_forwarded() && |
4406629aa157
7114095: G1: assert(obj == oopDesc::load_decode_heap_oop(p)) failed: p should still be pointing to obj
johnc
parents:
3771
diff
changeset
|
85 obj->forwardee() == oopDesc::load_decode_heap_oop(p)), |
4406629aa157
7114095: G1: assert(obj == oopDesc::load_decode_heap_oop(p)) failed: p should still be pointing to obj
johnc
parents:
3771
diff
changeset
|
86 "p should still be pointing to obj or to its forwardee"); |
4406629aa157
7114095: G1: assert(obj == oopDesc::load_decode_heap_oop(p)) failed: p should still be pointing to obj
johnc
parents:
3771
diff
changeset
|
87 |
845
df6caf649ff7
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
342
diff
changeset
|
88 _par_scan_state->push_on_queue(p); |
df6caf649ff7
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
342
diff
changeset
|
89 } else { |
20296
a3953c777565
8027959: Early reclamation of large objects in G1
tschatzl
parents:
20278
diff
changeset
|
90 if (state == G1CollectedHeap::IsHumongous) { |
a3953c777565
8027959: Early reclamation of large objects in G1
tschatzl
parents:
20278
diff
changeset
|
91 _g1->set_humongous_is_live(obj); |
a3953c777565
8027959: Early reclamation of large objects in G1
tschatzl
parents:
20278
diff
changeset
|
92 } |
17687
86b64209f715
8027559: Decrease code size and templatizing in G1ParCopyClosure::do_oop_work
tschatzl
parents:
6862
diff
changeset
|
93 _par_scan_state->update_rs(_from, p, _worker_id); |
845
df6caf649ff7
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
342
diff
changeset
|
94 } |
342 | 95 } |
96 } | |
1261 | 97 |
4837
eff609af17d7
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
4126
diff
changeset
|
98 template <class T> |
eff609af17d7
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
4126
diff
changeset
|
99 inline void G1ParPushHeapRSClosure::do_oop_nv(T* p) { |
1261 | 100 T heap_oop = oopDesc::load_heap_oop(p); |
101 | |
102 if (!oopDesc::is_null(heap_oop)) { | |
103 oop obj = oopDesc::decode_heap_oop_not_null(heap_oop); | |
20296
a3953c777565
8027959: Early reclamation of large objects in G1
tschatzl
parents:
20278
diff
changeset
|
104 if (_g1->is_in_cset_or_humongous(obj)) { |
1261 | 105 Prefetch::write(obj->mark_addr(), 0); |
106 Prefetch::read(obj->mark_addr(), (HeapWordSize*2)); | |
1705 | 107 |
108 // Place on the references queue | |
1261 | 109 _par_scan_state->push_on_queue(p); |
20296
a3953c777565
8027959: Early reclamation of large objects in G1
tschatzl
parents:
20278
diff
changeset
|
110 } else { |
a3953c777565
8027959: Early reclamation of large objects in G1
tschatzl
parents:
20278
diff
changeset
|
111 assert(!_g1->obj_in_cs(obj), "checking"); |
1261 | 112 } |
113 } | |
114 } | |
1705 | 115 |
4837
eff609af17d7
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
4126
diff
changeset
|
116 template <class T> |
eff609af17d7
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
4126
diff
changeset
|
117 inline void G1CMOopClosure::do_oop_nv(T* p) { |
3771 | 118 oop obj = oopDesc::load_decode_heap_oop(p); |
119 if (_cm->verbose_high()) { | |
6862
8a5ea0a9ccc4
7127708: G1: change task num types from int to uint in concurrent mark
johnc
parents:
6254
diff
changeset
|
120 gclog_or_tty->print_cr("[%u] we're looking at location " |
3771 | 121 "*"PTR_FORMAT" = "PTR_FORMAT, |
17937
78bbf4d43a14
8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents:
17805
diff
changeset
|
122 _task->worker_id(), p2i(p), p2i((void*) obj)); |
3771 | 123 } |
124 _task->deal_with_reference(obj); | |
125 } | |
1972 | 126 |
4837
eff609af17d7
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
4126
diff
changeset
|
127 template <class T> |
eff609af17d7
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
4126
diff
changeset
|
128 inline void G1RootRegionScanClosure::do_oop_nv(T* p) { |
eff609af17d7
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
4126
diff
changeset
|
129 T heap_oop = oopDesc::load_heap_oop(p); |
eff609af17d7
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
4126
diff
changeset
|
130 if (!oopDesc::is_null(heap_oop)) { |
eff609af17d7
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
4126
diff
changeset
|
131 oop obj = oopDesc::decode_heap_oop_not_null(heap_oop); |
eff609af17d7
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
4126
diff
changeset
|
132 HeapRegion* hr = _g1h->heap_region_containing((HeapWord*) obj); |
20335
eec72fa4b108
8040722: G1: Clean up usages of heap_region_containing
brutisso
parents:
20296
diff
changeset
|
133 _cm->grayRoot(obj, obj->size(), _worker_id, hr); |
4837
eff609af17d7
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
4126
diff
changeset
|
134 } |
eff609af17d7
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
4126
diff
changeset
|
135 } |
eff609af17d7
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
4126
diff
changeset
|
136 |
4839 | 137 template <class T> |
138 inline void G1Mux2Closure::do_oop_nv(T* p) { | |
139 // Apply first closure; then apply the second. | |
140 _c1->do_oop(p); | |
141 _c2->do_oop(p); | |
142 } | |
143 | |
144 template <class T> | |
145 inline void G1TriggerClosure::do_oop_nv(T* p) { | |
146 // Record that this closure was actually applied (triggered). | |
147 _triggered = true; | |
148 } | |
149 | |
150 template <class T> | |
151 inline void G1InvokeIfNotTriggeredClosure::do_oop_nv(T* p) { | |
152 if (!_trigger_cl->triggered()) { | |
153 _oop_cl->do_oop(p); | |
154 } | |
155 } | |
156 | |
157 template <class T> | |
158 inline void G1UpdateRSOrPushRefOopClosure::do_oop_nv(T* p) { | |
159 oop obj = oopDesc::load_decode_heap_oop(p); | |
20335
eec72fa4b108
8040722: G1: Clean up usages of heap_region_containing
brutisso
parents:
20296
diff
changeset
|
160 if (obj == NULL) { |
eec72fa4b108
8040722: G1: Clean up usages of heap_region_containing
brutisso
parents:
20296
diff
changeset
|
161 return; |
eec72fa4b108
8040722: G1: Clean up usages of heap_region_containing
brutisso
parents:
20296
diff
changeset
|
162 } |
4839 | 163 #ifdef ASSERT |
164 // can't do because of races | |
165 // assert(obj == NULL || obj->is_oop(), "expected an oop"); | |
166 | |
167 // Do the safe subset of is_oop | |
168 #ifdef CHECK_UNHANDLED_OOPS | |
20335
eec72fa4b108
8040722: G1: Clean up usages of heap_region_containing
brutisso
parents:
20296
diff
changeset
|
169 oopDesc* o = obj.obj(); |
4839 | 170 #else |
20335
eec72fa4b108
8040722: G1: Clean up usages of heap_region_containing
brutisso
parents:
20296
diff
changeset
|
171 oopDesc* o = obj; |
4839 | 172 #endif // CHECK_UNHANDLED_OOPS |
20335
eec72fa4b108
8040722: G1: Clean up usages of heap_region_containing
brutisso
parents:
20296
diff
changeset
|
173 assert((intptr_t)o % MinObjAlignmentInBytes == 0, "not oop aligned"); |
eec72fa4b108
8040722: G1: Clean up usages of heap_region_containing
brutisso
parents:
20296
diff
changeset
|
174 assert(Universe::heap()->is_in_reserved(obj), "must be in heap"); |
4839 | 175 #endif // ASSERT |
176 | |
177 assert(_from != NULL, "from region must be non-NULL"); | |
6251
594dff5e3c2e
7173712: G1: Duplicated code in G1UpdateRSOrPushRefOopClosure::do_oop_nv()
johnc
parents:
4839
diff
changeset
|
178 assert(_from->is_in_reserved(p), "p is not in from"); |
4839 | 179 |
180 HeapRegion* to = _g1->heap_region_containing(obj); | |
20335
eec72fa4b108
8040722: G1: Clean up usages of heap_region_containing
brutisso
parents:
20296
diff
changeset
|
181 if (_from == to) { |
eec72fa4b108
8040722: G1: Clean up usages of heap_region_containing
brutisso
parents:
20296
diff
changeset
|
182 // Normally this closure should only be called with cross-region references. |
eec72fa4b108
8040722: G1: Clean up usages of heap_region_containing
brutisso
parents:
20296
diff
changeset
|
183 // But since Java threads are manipulating the references concurrently and we |
eec72fa4b108
8040722: G1: Clean up usages of heap_region_containing
brutisso
parents:
20296
diff
changeset
|
184 // reload the values things may have changed. |
eec72fa4b108
8040722: G1: Clean up usages of heap_region_containing
brutisso
parents:
20296
diff
changeset
|
185 return; |
eec72fa4b108
8040722: G1: Clean up usages of heap_region_containing
brutisso
parents:
20296
diff
changeset
|
186 } |
eec72fa4b108
8040722: G1: Clean up usages of heap_region_containing
brutisso
parents:
20296
diff
changeset
|
187 // The _record_refs_into_cset flag is true during the RSet |
eec72fa4b108
8040722: G1: Clean up usages of heap_region_containing
brutisso
parents:
20296
diff
changeset
|
188 // updating part of an evacuation pause. It is false at all |
eec72fa4b108
8040722: G1: Clean up usages of heap_region_containing
brutisso
parents:
20296
diff
changeset
|
189 // other times: |
eec72fa4b108
8040722: G1: Clean up usages of heap_region_containing
brutisso
parents:
20296
diff
changeset
|
190 // * rebuilding the remembered sets after a full GC |
eec72fa4b108
8040722: G1: Clean up usages of heap_region_containing
brutisso
parents:
20296
diff
changeset
|
191 // * during concurrent refinement. |
eec72fa4b108
8040722: G1: Clean up usages of heap_region_containing
brutisso
parents:
20296
diff
changeset
|
192 // * updating the remembered sets of regions in the collection |
eec72fa4b108
8040722: G1: Clean up usages of heap_region_containing
brutisso
parents:
20296
diff
changeset
|
193 // set in the event of an evacuation failure (when deferred |
eec72fa4b108
8040722: G1: Clean up usages of heap_region_containing
brutisso
parents:
20296
diff
changeset
|
194 // updates are enabled). |
4839 | 195 |
20335
eec72fa4b108
8040722: G1: Clean up usages of heap_region_containing
brutisso
parents:
20296
diff
changeset
|
196 if (_record_refs_into_cset && to->in_collection_set()) { |
eec72fa4b108
8040722: G1: Clean up usages of heap_region_containing
brutisso
parents:
20296
diff
changeset
|
197 // We are recording references that point into the collection |
eec72fa4b108
8040722: G1: Clean up usages of heap_region_containing
brutisso
parents:
20296
diff
changeset
|
198 // set and this particular reference does exactly that... |
eec72fa4b108
8040722: G1: Clean up usages of heap_region_containing
brutisso
parents:
20296
diff
changeset
|
199 // If the referenced object has already been forwarded |
eec72fa4b108
8040722: G1: Clean up usages of heap_region_containing
brutisso
parents:
20296
diff
changeset
|
200 // to itself, we are handling an evacuation failure and |
eec72fa4b108
8040722: G1: Clean up usages of heap_region_containing
brutisso
parents:
20296
diff
changeset
|
201 // we have already visited/tried to copy this object |
eec72fa4b108
8040722: G1: Clean up usages of heap_region_containing
brutisso
parents:
20296
diff
changeset
|
202 // there is no need to retry. |
eec72fa4b108
8040722: G1: Clean up usages of heap_region_containing
brutisso
parents:
20296
diff
changeset
|
203 if (!self_forwarded(obj)) { |
eec72fa4b108
8040722: G1: Clean up usages of heap_region_containing
brutisso
parents:
20296
diff
changeset
|
204 assert(_push_ref_cl != NULL, "should not be null"); |
eec72fa4b108
8040722: G1: Clean up usages of heap_region_containing
brutisso
parents:
20296
diff
changeset
|
205 // Push the reference in the refs queue of the G1ParScanThreadState |
eec72fa4b108
8040722: G1: Clean up usages of heap_region_containing
brutisso
parents:
20296
diff
changeset
|
206 // instance for this worker thread. |
eec72fa4b108
8040722: G1: Clean up usages of heap_region_containing
brutisso
parents:
20296
diff
changeset
|
207 _push_ref_cl->do_oop(p); |
eec72fa4b108
8040722: G1: Clean up usages of heap_region_containing
brutisso
parents:
20296
diff
changeset
|
208 } |
4839 | 209 |
20335
eec72fa4b108
8040722: G1: Clean up usages of heap_region_containing
brutisso
parents:
20296
diff
changeset
|
210 // Deferred updates to the CSet are either discarded (in the normal case), |
eec72fa4b108
8040722: G1: Clean up usages of heap_region_containing
brutisso
parents:
20296
diff
changeset
|
211 // or processed (if an evacuation failure occurs) at the end |
eec72fa4b108
8040722: G1: Clean up usages of heap_region_containing
brutisso
parents:
20296
diff
changeset
|
212 // of the collection. |
eec72fa4b108
8040722: G1: Clean up usages of heap_region_containing
brutisso
parents:
20296
diff
changeset
|
213 // See G1RemSet::cleanup_after_oops_into_collection_set_do(). |
eec72fa4b108
8040722: G1: Clean up usages of heap_region_containing
brutisso
parents:
20296
diff
changeset
|
214 } else { |
6251
594dff5e3c2e
7173712: G1: Duplicated code in G1UpdateRSOrPushRefOopClosure::do_oop_nv()
johnc
parents:
4839
diff
changeset
|
215 // We either don't care about pushing references that point into the |
594dff5e3c2e
7173712: G1: Duplicated code in G1UpdateRSOrPushRefOopClosure::do_oop_nv()
johnc
parents:
4839
diff
changeset
|
216 // collection set (i.e. we're not during an evacuation pause) _or_ |
594dff5e3c2e
7173712: G1: Duplicated code in G1UpdateRSOrPushRefOopClosure::do_oop_nv()
johnc
parents:
4839
diff
changeset
|
217 // the reference doesn't point into the collection set. Either way |
594dff5e3c2e
7173712: G1: Duplicated code in G1UpdateRSOrPushRefOopClosure::do_oop_nv()
johnc
parents:
4839
diff
changeset
|
218 // we add the reference directly to the RSet of the region containing |
594dff5e3c2e
7173712: G1: Duplicated code in G1UpdateRSOrPushRefOopClosure::do_oop_nv()
johnc
parents:
4839
diff
changeset
|
219 // the referenced object. |
594dff5e3c2e
7173712: G1: Duplicated code in G1UpdateRSOrPushRefOopClosure::do_oop_nv()
johnc
parents:
4839
diff
changeset
|
220 assert(to->rem_set() != NULL, "Need per-region 'into' remsets."); |
594dff5e3c2e
7173712: G1: Duplicated code in G1UpdateRSOrPushRefOopClosure::do_oop_nv()
johnc
parents:
4839
diff
changeset
|
221 to->rem_set()->add_reference(p, _worker_i); |
4839 | 222 } |
223 } | |
224 | |
1972 | 225 #endif // SHARE_VM_GC_IMPLEMENTATION_G1_G1OOPCLOSURES_INLINE_HPP |