author | kbarrett |
Tue, 30 Aug 2016 23:48:16 -0400 | |
changeset 40892 | 330a02d935ad |
parent 37413 | 2f71679d06dd |
child 46282 | a001553763bc |
permissions | -rw-r--r-- |
1374 | 1 |
/* |
35862
411842d0c882
8146395: Add inline qualifier in oop.hpp and fix inlining in gc files
goetz
parents:
35069
diff
changeset
|
2 |
* Copyright (c) 2001, 2016, Oracle and/or its affiliates. All rights reserved. |
1374 | 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:
4902
diff
changeset
|
19 |
* Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA |
f4b087cbb361
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
4902
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:
4902
diff
changeset
|
21 |
* questions. |
1374 | 22 |
* |
23 |
*/ |
|
24 |
||
30764 | 25 |
#ifndef SHARE_VM_GC_G1_G1OOPCLOSURES_INLINE_HPP |
26 |
#define SHARE_VM_GC_G1_G1OOPCLOSURES_INLINE_HPP |
|
7397 | 27 |
|
30764 | 28 |
#include "gc/g1/g1CollectedHeap.hpp" |
35943
e726308008c0
8148973: Rename g1/concurrentMark.{hpp,cpp,inline.hpp} to g1/g1ConcurrentMark.{hpp,cpp,inline.hpp}
ehelin
parents:
35862
diff
changeset
|
29 |
#include "gc/g1/g1ConcurrentMark.inline.hpp" |
30764 | 30 |
#include "gc/g1/g1OopClosures.hpp" |
31 |
#include "gc/g1/g1ParScanThreadState.inline.hpp" |
|
32 |
#include "gc/g1/g1RemSet.hpp" |
|
33 |
#include "gc/g1/g1RemSet.inline.hpp" |
|
32185
49a57ff2c3cb
8073052: Rename and clean up the allocation manager hierarchy in g1Allocator.?pp
tschatzl
parents:
31592
diff
changeset
|
34 |
#include "gc/g1/heapRegion.inline.hpp" |
30764 | 35 |
#include "gc/g1/heapRegionRemSet.hpp" |
25492
d27050bdfb04
8049421: G1 Class Unloading after completing a concurrent mark cycle
stefank
parents:
25491
diff
changeset
|
36 |
#include "memory/iterator.inline.hpp" |
24487
71ff0bd674eb
8042737: Introduce umbrella header prefetch.inline.hpp
goetz
parents:
24424
diff
changeset
|
37 |
#include "runtime/prefetch.inline.hpp" |
7397 | 38 |
|
1374 | 39 |
/* |
40 |
* This really ought to be an inline function, but apparently the C++ |
|
41 |
* compiler sometimes sees fit to ignore inline declarations. Sigh. |
|
42 |
*/ |
|
43 |
||
11584
e1df4d08a1f4
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
11202
diff
changeset
|
44 |
template <class T> |
35068
b0d5016aecc7
8144584: Change FilterIntoCSClosure to inherit OopClosure
sjohanss
parents:
35067
diff
changeset
|
45 |
inline void FilterIntoCSClosure::do_oop_work(T* p) { |
3262
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
1374
diff
changeset
|
46 |
T heap_oop = oopDesc::load_heap_oop(p); |
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
1374
diff
changeset
|
47 |
if (!oopDesc::is_null(heap_oop) && |
25889
221296ac4359
8027959: Early reclamation of large objects in G1
tschatzl
parents:
25492
diff
changeset
|
48 |
_g1->is_in_cset_or_humongous(oopDesc::decode_heap_oop_not_null(heap_oop))) { |
1374 | 49 |
_oc->do_oop(p); |
50 |
} |
|
51 |
} |
|
52 |
||
11584
e1df4d08a1f4
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
11202
diff
changeset
|
53 |
template <class T> |
e1df4d08a1f4
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
11202
diff
changeset
|
54 |
inline void FilterOutOfRegionClosure::do_oop_nv(T* p) { |
3262
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
1374
diff
changeset
|
55 |
T heap_oop = oopDesc::load_heap_oop(p); |
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
1374
diff
changeset
|
56 |
if (!oopDesc::is_null(heap_oop)) { |
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
1374
diff
changeset
|
57 |
HeapWord* obj_hw = (HeapWord*)oopDesc::decode_heap_oop_not_null(heap_oop); |
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
1374
diff
changeset
|
58 |
if (obj_hw < _r_bottom || obj_hw >= _r_end) { |
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
1374
diff
changeset
|
59 |
_oc->do_oop(p); |
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
1374
diff
changeset
|
60 |
} |
1374 | 61 |
} |
62 |
} |
|
63 |
||
3262
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
1374
diff
changeset
|
64 |
// This closure is applied to the fields of the objects that have just been copied. |
11584
e1df4d08a1f4
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
11202
diff
changeset
|
65 |
template <class T> |
e1df4d08a1f4
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
11202
diff
changeset
|
66 |
inline void G1ParScanClosure::do_oop_nv(T* p) { |
3262
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
1374
diff
changeset
|
67 |
T heap_oop = oopDesc::load_heap_oop(p); |
1374 | 68 |
|
3262
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
1374
diff
changeset
|
69 |
if (!oopDesc::is_null(heap_oop)) { |
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
1374
diff
changeset
|
70 |
oop obj = oopDesc::decode_heap_oop_not_null(heap_oop); |
28213
b0bf57cd1e9d
8060025: Object copy time regressions after JDK-8031323 and JDK-8057536
tschatzl
parents:
25889
diff
changeset
|
71 |
const InCSetState state = _g1->in_cset_state(obj); |
b0bf57cd1e9d
8060025: Object copy time regressions after JDK-8031323 and JDK-8057536
tschatzl
parents:
25889
diff
changeset
|
72 |
if (state.is_in_cset()) { |
3262
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
1374
diff
changeset
|
73 |
// We're not going to even bother checking whether the object is |
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
1374
diff
changeset
|
74 |
// already forwarded or not, as this usually causes an immediate |
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
1374
diff
changeset
|
75 |
// stall. We'll try to prefetch the object (for write, given that |
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
1374
diff
changeset
|
76 |
// we might need to install the forwarding reference) and we'll |
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
1374
diff
changeset
|
77 |
// get back to it when pop it from the queue |
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
1374
diff
changeset
|
78 |
Prefetch::write(obj->mark_addr(), 0); |
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
1374
diff
changeset
|
79 |
Prefetch::read(obj->mark_addr(), (HeapWordSize*2)); |
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
1374
diff
changeset
|
80 |
|
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
1374
diff
changeset
|
81 |
// slightly paranoid test; I'm trying to catch potential |
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
1374
diff
changeset
|
82 |
// problems before we go into push_on_queue to know where the |
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
1374
diff
changeset
|
83 |
// problem is coming from |
11202
6be05533238b
7114095: G1: assert(obj == oopDesc::load_decode_heap_oop(p)) failed: p should still be pointing to obj
johnc
parents:
9994
diff
changeset
|
84 |
assert((obj == oopDesc::load_decode_heap_oop(p)) || |
6be05533238b
7114095: G1: assert(obj == oopDesc::load_decode_heap_oop(p)) failed: p should still be pointing to obj
johnc
parents:
9994
diff
changeset
|
85 |
(obj->is_forwarded() && |
6be05533238b
7114095: G1: assert(obj == oopDesc::load_decode_heap_oop(p)) failed: p should still be pointing to obj
johnc
parents:
9994
diff
changeset
|
86 |
obj->forwardee() == oopDesc::load_decode_heap_oop(p)), |
6be05533238b
7114095: G1: assert(obj == oopDesc::load_decode_heap_oop(p)) failed: p should still be pointing to obj
johnc
parents:
9994
diff
changeset
|
87 |
"p should still be pointing to obj or to its forwardee"); |
6be05533238b
7114095: G1: assert(obj == oopDesc::load_decode_heap_oop(p)) failed: p should still be pointing to obj
johnc
parents:
9994
diff
changeset
|
88 |
|
3262
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
1374
diff
changeset
|
89 |
_par_scan_state->push_on_queue(p); |
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
1374
diff
changeset
|
90 |
} else { |
28213
b0bf57cd1e9d
8060025: Object copy time regressions after JDK-8031323 and JDK-8057536
tschatzl
parents:
25889
diff
changeset
|
91 |
if (state.is_humongous()) { |
25889
221296ac4359
8027959: Early reclamation of large objects in G1
tschatzl
parents:
25492
diff
changeset
|
92 |
_g1->set_humongous_is_live(obj); |
35051
002d874a4284
8144714: Add extension point to G1 evacuation closures
mgerdin
parents:
34616
diff
changeset
|
93 |
} else if (state.is_ext()) { |
002d874a4284
8144714: Add extension point to G1 evacuation closures
mgerdin
parents:
34616
diff
changeset
|
94 |
_par_scan_state->do_oop_ext(p); |
25889
221296ac4359
8027959: Early reclamation of large objects in G1
tschatzl
parents:
25492
diff
changeset
|
95 |
} |
34616
421a6405ca30
8144067: Pass obj directly to G1ParScanThreadState::update_rs
ehelin
parents:
34245
diff
changeset
|
96 |
_par_scan_state->update_rs(_from, p, obj); |
3262
30d1c247fc25
6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents:
1374
diff
changeset
|
97 |
} |
1374 | 98 |
} |
99 |
} |
|
4902
991aaddb5165
6923991: G1: improve scalability of RSet scanning
iveresov
parents:
3262
diff
changeset
|
100 |
|
11584
e1df4d08a1f4
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
11202
diff
changeset
|
101 |
template <class T> |
e1df4d08a1f4
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
11202
diff
changeset
|
102 |
inline void G1ParPushHeapRSClosure::do_oop_nv(T* p) { |
4902
991aaddb5165
6923991: G1: improve scalability of RSet scanning
iveresov
parents:
3262
diff
changeset
|
103 |
T heap_oop = oopDesc::load_heap_oop(p); |
991aaddb5165
6923991: G1: improve scalability of RSet scanning
iveresov
parents:
3262
diff
changeset
|
104 |
|
991aaddb5165
6923991: G1: improve scalability of RSet scanning
iveresov
parents:
3262
diff
changeset
|
105 |
if (!oopDesc::is_null(heap_oop)) { |
991aaddb5165
6923991: G1: improve scalability of RSet scanning
iveresov
parents:
3262
diff
changeset
|
106 |
oop obj = oopDesc::decode_heap_oop_not_null(heap_oop); |
35051
002d874a4284
8144714: Add extension point to G1 evacuation closures
mgerdin
parents:
34616
diff
changeset
|
107 |
const InCSetState state = _g1->in_cset_state(obj); |
002d874a4284
8144714: Add extension point to G1 evacuation closures
mgerdin
parents:
34616
diff
changeset
|
108 |
if (state.is_in_cset_or_humongous()) { |
4902
991aaddb5165
6923991: G1: improve scalability of RSet scanning
iveresov
parents:
3262
diff
changeset
|
109 |
Prefetch::write(obj->mark_addr(), 0); |
991aaddb5165
6923991: G1: improve scalability of RSet scanning
iveresov
parents:
3262
diff
changeset
|
110 |
Prefetch::read(obj->mark_addr(), (HeapWordSize*2)); |
6247 | 111 |
|
112 |
// Place on the references queue |
|
4902
991aaddb5165
6923991: G1: improve scalability of RSet scanning
iveresov
parents:
3262
diff
changeset
|
113 |
_par_scan_state->push_on_queue(p); |
35051
002d874a4284
8144714: Add extension point to G1 evacuation closures
mgerdin
parents:
34616
diff
changeset
|
114 |
} else if (state.is_ext()) { |
002d874a4284
8144714: Add extension point to G1 evacuation closures
mgerdin
parents:
34616
diff
changeset
|
115 |
_par_scan_state->do_oop_ext(p); |
25889
221296ac4359
8027959: Early reclamation of large objects in G1
tschatzl
parents:
25492
diff
changeset
|
116 |
} else { |
221296ac4359
8027959: Early reclamation of large objects in G1
tschatzl
parents:
25492
diff
changeset
|
117 |
assert(!_g1->obj_in_cs(obj), "checking"); |
4902
991aaddb5165
6923991: G1: improve scalability of RSet scanning
iveresov
parents:
3262
diff
changeset
|
118 |
} |
991aaddb5165
6923991: G1: improve scalability of RSet scanning
iveresov
parents:
3262
diff
changeset
|
119 |
} |
991aaddb5165
6923991: G1: improve scalability of RSet scanning
iveresov
parents:
3262
diff
changeset
|
120 |
} |
6247 | 121 |
|
11584
e1df4d08a1f4
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
11202
diff
changeset
|
122 |
template <class T> |
e1df4d08a1f4
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
11202
diff
changeset
|
123 |
inline void G1CMOopClosure::do_oop_nv(T* p) { |
9994 | 124 |
oop obj = oopDesc::load_decode_heap_oop(p); |
125 |
_task->deal_with_reference(obj); |
|
126 |
} |
|
7397 | 127 |
|
11584
e1df4d08a1f4
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
11202
diff
changeset
|
128 |
template <class T> |
e1df4d08a1f4
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
11202
diff
changeset
|
129 |
inline void G1RootRegionScanClosure::do_oop_nv(T* p) { |
e1df4d08a1f4
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
11202
diff
changeset
|
130 |
T heap_oop = oopDesc::load_heap_oop(p); |
e1df4d08a1f4
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
11202
diff
changeset
|
131 |
if (!oopDesc::is_null(heap_oop)) { |
e1df4d08a1f4
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
11202
diff
changeset
|
132 |
oop obj = oopDesc::decode_heap_oop_not_null(heap_oop); |
e1df4d08a1f4
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
11202
diff
changeset
|
133 |
HeapRegion* hr = _g1h->heap_region_containing((HeapWord*) obj); |
37413
2f71679d06dd
8077144: Concurrent mark initialization takes too long
tschatzl
parents:
37258
diff
changeset
|
134 |
_cm->grayRoot(obj, hr); |
11584
e1df4d08a1f4
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
11202
diff
changeset
|
135 |
} |
e1df4d08a1f4
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
11202
diff
changeset
|
136 |
} |
e1df4d08a1f4
7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents:
11202
diff
changeset
|
137 |
|
11586
ccc217c177ee
7133038: G1: Some small profile based optimizations
johnc
parents:
11584
diff
changeset
|
138 |
template <class T> |
35069
24d8a63c9486
8144701: Change three G1 remembererd set closures to be OopClosures
sjohanss
parents:
35068
diff
changeset
|
139 |
inline void G1Mux2Closure::do_oop_work(T* p) { |
11586
ccc217c177ee
7133038: G1: Some small profile based optimizations
johnc
parents:
11584
diff
changeset
|
140 |
// Apply first closure; then apply the second. |
ccc217c177ee
7133038: G1: Some small profile based optimizations
johnc
parents:
11584
diff
changeset
|
141 |
_c1->do_oop(p); |
ccc217c177ee
7133038: G1: Some small profile based optimizations
johnc
parents:
11584
diff
changeset
|
142 |
_c2->do_oop(p); |
ccc217c177ee
7133038: G1: Some small profile based optimizations
johnc
parents:
11584
diff
changeset
|
143 |
} |
35862
411842d0c882
8146395: Add inline qualifier in oop.hpp and fix inlining in gc files
goetz
parents:
35069
diff
changeset
|
144 |
void G1Mux2Closure::do_oop(oop* p) { do_oop_work(p); } |
411842d0c882
8146395: Add inline qualifier in oop.hpp and fix inlining in gc files
goetz
parents:
35069
diff
changeset
|
145 |
void G1Mux2Closure::do_oop(narrowOop* p) { do_oop_work(p); } |
11586
ccc217c177ee
7133038: G1: Some small profile based optimizations
johnc
parents:
11584
diff
changeset
|
146 |
|
ccc217c177ee
7133038: G1: Some small profile based optimizations
johnc
parents:
11584
diff
changeset
|
147 |
template <class T> |
35069
24d8a63c9486
8144701: Change three G1 remembererd set closures to be OopClosures
sjohanss
parents:
35068
diff
changeset
|
148 |
inline void G1TriggerClosure::do_oop_work(T* p) { |
11586
ccc217c177ee
7133038: G1: Some small profile based optimizations
johnc
parents:
11584
diff
changeset
|
149 |
// Record that this closure was actually applied (triggered). |
ccc217c177ee
7133038: G1: Some small profile based optimizations
johnc
parents:
11584
diff
changeset
|
150 |
_triggered = true; |
ccc217c177ee
7133038: G1: Some small profile based optimizations
johnc
parents:
11584
diff
changeset
|
151 |
} |
35862
411842d0c882
8146395: Add inline qualifier in oop.hpp and fix inlining in gc files
goetz
parents:
35069
diff
changeset
|
152 |
void G1TriggerClosure::do_oop(oop* p) { do_oop_work(p); } |
411842d0c882
8146395: Add inline qualifier in oop.hpp and fix inlining in gc files
goetz
parents:
35069
diff
changeset
|
153 |
void G1TriggerClosure::do_oop(narrowOop* p) { do_oop_work(p); } |
11586
ccc217c177ee
7133038: G1: Some small profile based optimizations
johnc
parents:
11584
diff
changeset
|
154 |
|
ccc217c177ee
7133038: G1: Some small profile based optimizations
johnc
parents:
11584
diff
changeset
|
155 |
template <class T> |
35069
24d8a63c9486
8144701: Change three G1 remembererd set closures to be OopClosures
sjohanss
parents:
35068
diff
changeset
|
156 |
inline void G1InvokeIfNotTriggeredClosure::do_oop_work(T* p) { |
11586
ccc217c177ee
7133038: G1: Some small profile based optimizations
johnc
parents:
11584
diff
changeset
|
157 |
if (!_trigger_cl->triggered()) { |
ccc217c177ee
7133038: G1: Some small profile based optimizations
johnc
parents:
11584
diff
changeset
|
158 |
_oop_cl->do_oop(p); |
ccc217c177ee
7133038: G1: Some small profile based optimizations
johnc
parents:
11584
diff
changeset
|
159 |
} |
ccc217c177ee
7133038: G1: Some small profile based optimizations
johnc
parents:
11584
diff
changeset
|
160 |
} |
35862
411842d0c882
8146395: Add inline qualifier in oop.hpp and fix inlining in gc files
goetz
parents:
35069
diff
changeset
|
161 |
void G1InvokeIfNotTriggeredClosure::do_oop(oop* p) { do_oop_work(p); } |
411842d0c882
8146395: Add inline qualifier in oop.hpp and fix inlining in gc files
goetz
parents:
35069
diff
changeset
|
162 |
void G1InvokeIfNotTriggeredClosure::do_oop(narrowOop* p) { do_oop_work(p); } |
11586
ccc217c177ee
7133038: G1: Some small profile based optimizations
johnc
parents:
11584
diff
changeset
|
163 |
|
ccc217c177ee
7133038: G1: Some small profile based optimizations
johnc
parents:
11584
diff
changeset
|
164 |
template <class T> |
35062
f3e0dee91918
8144486: Change G1UpdateRSOrPushRefOopClosure to inherit OopClosure
sjohanss
parents:
35051
diff
changeset
|
165 |
inline void G1UpdateRSOrPushRefOopClosure::do_oop_work(T* p) { |
11586
ccc217c177ee
7133038: G1: Some small profile based optimizations
johnc
parents:
11584
diff
changeset
|
166 |
oop obj = oopDesc::load_decode_heap_oop(p); |
24106
dae9277bdf2a
8040722: G1: Clean up usages of heap_region_containing
brutisso
parents:
22905
diff
changeset
|
167 |
if (obj == NULL) { |
dae9277bdf2a
8040722: G1: Clean up usages of heap_region_containing
brutisso
parents:
22905
diff
changeset
|
168 |
return; |
dae9277bdf2a
8040722: G1: Clean up usages of heap_region_containing
brutisso
parents:
22905
diff
changeset
|
169 |
} |
dae9277bdf2a
8040722: G1: Clean up usages of heap_region_containing
brutisso
parents:
22905
diff
changeset
|
170 |
|
11586
ccc217c177ee
7133038: G1: Some small profile based optimizations
johnc
parents:
11584
diff
changeset
|
171 |
#ifdef ASSERT |
ccc217c177ee
7133038: G1: Some small profile based optimizations
johnc
parents:
11584
diff
changeset
|
172 |
// can't do because of races |
ccc217c177ee
7133038: G1: Some small profile based optimizations
johnc
parents:
11584
diff
changeset
|
173 |
// assert(obj == NULL || obj->is_oop(), "expected an oop"); |
36572
bdbf53032b6a
8149973: Optimize object alignment check in debug builds.
ddmitriev
parents:
35943
diff
changeset
|
174 |
assert(check_obj_alignment(obj), "not oop aligned"); |
30173
13cf7580b000
8077413: Avoid use of Universe::heap() inside collectors
pliden
parents:
28213
diff
changeset
|
175 |
assert(_g1->is_in_reserved(obj), "must be in heap"); |
11586
ccc217c177ee
7133038: G1: Some small profile based optimizations
johnc
parents:
11584
diff
changeset
|
176 |
#endif // ASSERT |
ccc217c177ee
7133038: G1: Some small profile based optimizations
johnc
parents:
11584
diff
changeset
|
177 |
|
ccc217c177ee
7133038: G1: Some small profile based optimizations
johnc
parents:
11584
diff
changeset
|
178 |
assert(_from != NULL, "from region must be non-NULL"); |
37258
d5fb0ee5e41a
8151499: gc/g1/g1OopClosures.inline.hpp: assert(_from->is_in_reserved(p)) failed: p is not in from
tschatzl
parents:
36572
diff
changeset
|
179 |
assert(_from->is_in_reserved(p) || |
d5fb0ee5e41a
8151499: gc/g1/g1OopClosures.inline.hpp: assert(_from->is_in_reserved(p)) failed: p is not in from
tschatzl
parents:
36572
diff
changeset
|
180 |
(_from->is_humongous() && |
d5fb0ee5e41a
8151499: gc/g1/g1OopClosures.inline.hpp: assert(_from->is_in_reserved(p)) failed: p is not in from
tschatzl
parents:
36572
diff
changeset
|
181 |
_g1->heap_region_containing(p)->is_humongous() && |
d5fb0ee5e41a
8151499: gc/g1/g1OopClosures.inline.hpp: assert(_from->is_in_reserved(p)) failed: p is not in from
tschatzl
parents:
36572
diff
changeset
|
182 |
_from->humongous_start_region() == _g1->heap_region_containing(p)->humongous_start_region()), |
d5fb0ee5e41a
8151499: gc/g1/g1OopClosures.inline.hpp: assert(_from->is_in_reserved(p)) failed: p is not in from
tschatzl
parents:
36572
diff
changeset
|
183 |
"p " PTR_FORMAT " is not in the same region %u or part of the correct humongous object starting at region %u.", |
d5fb0ee5e41a
8151499: gc/g1/g1OopClosures.inline.hpp: assert(_from->is_in_reserved(p)) failed: p is not in from
tschatzl
parents:
36572
diff
changeset
|
184 |
p2i(p), _from->hrm_index(), _from->humongous_start_region()->hrm_index()); |
11586
ccc217c177ee
7133038: G1: Some small profile based optimizations
johnc
parents:
11584
diff
changeset
|
185 |
|
ccc217c177ee
7133038: G1: Some small profile based optimizations
johnc
parents:
11584
diff
changeset
|
186 |
HeapRegion* to = _g1->heap_region_containing(obj); |
24106
dae9277bdf2a
8040722: G1: Clean up usages of heap_region_containing
brutisso
parents:
22905
diff
changeset
|
187 |
if (_from == to) { |
dae9277bdf2a
8040722: G1: Clean up usages of heap_region_containing
brutisso
parents:
22905
diff
changeset
|
188 |
// Normally this closure should only be called with cross-region references. |
dae9277bdf2a
8040722: G1: Clean up usages of heap_region_containing
brutisso
parents:
22905
diff
changeset
|
189 |
// But since Java threads are manipulating the references concurrently and we |
dae9277bdf2a
8040722: G1: Clean up usages of heap_region_containing
brutisso
parents:
22905
diff
changeset
|
190 |
// reload the values things may have changed. |
37258
d5fb0ee5e41a
8151499: gc/g1/g1OopClosures.inline.hpp: assert(_from->is_in_reserved(p)) failed: p is not in from
tschatzl
parents:
36572
diff
changeset
|
191 |
// Also this check lets slip through references from a humongous continues region |
d5fb0ee5e41a
8151499: gc/g1/g1OopClosures.inline.hpp: assert(_from->is_in_reserved(p)) failed: p is not in from
tschatzl
parents:
36572
diff
changeset
|
192 |
// to its humongous start region, as they are in different regions, and adds a |
d5fb0ee5e41a
8151499: gc/g1/g1OopClosures.inline.hpp: assert(_from->is_in_reserved(p)) failed: p is not in from
tschatzl
parents:
36572
diff
changeset
|
193 |
// remembered set entry. This is benign (apart from memory usage), as we never |
d5fb0ee5e41a
8151499: gc/g1/g1OopClosures.inline.hpp: assert(_from->is_in_reserved(p)) failed: p is not in from
tschatzl
parents:
36572
diff
changeset
|
194 |
// try to either evacuate or eager reclaim these kind of regions. |
24106
dae9277bdf2a
8040722: G1: Clean up usages of heap_region_containing
brutisso
parents:
22905
diff
changeset
|
195 |
return; |
dae9277bdf2a
8040722: G1: Clean up usages of heap_region_containing
brutisso
parents:
22905
diff
changeset
|
196 |
} |
11586
ccc217c177ee
7133038: G1: Some small profile based optimizations
johnc
parents:
11584
diff
changeset
|
197 |
|
24106
dae9277bdf2a
8040722: G1: Clean up usages of heap_region_containing
brutisso
parents:
22905
diff
changeset
|
198 |
// The _record_refs_into_cset flag is true during the RSet |
dae9277bdf2a
8040722: G1: Clean up usages of heap_region_containing
brutisso
parents:
22905
diff
changeset
|
199 |
// updating part of an evacuation pause. It is false at all |
dae9277bdf2a
8040722: G1: Clean up usages of heap_region_containing
brutisso
parents:
22905
diff
changeset
|
200 |
// other times: |
dae9277bdf2a
8040722: G1: Clean up usages of heap_region_containing
brutisso
parents:
22905
diff
changeset
|
201 |
// * rebuilding the remembered sets after a full GC |
dae9277bdf2a
8040722: G1: Clean up usages of heap_region_containing
brutisso
parents:
22905
diff
changeset
|
202 |
// * during concurrent refinement. |
dae9277bdf2a
8040722: G1: Clean up usages of heap_region_containing
brutisso
parents:
22905
diff
changeset
|
203 |
// * updating the remembered sets of regions in the collection |
dae9277bdf2a
8040722: G1: Clean up usages of heap_region_containing
brutisso
parents:
22905
diff
changeset
|
204 |
// set in the event of an evacuation failure (when deferred |
dae9277bdf2a
8040722: G1: Clean up usages of heap_region_containing
brutisso
parents:
22905
diff
changeset
|
205 |
// updates are enabled). |
11586
ccc217c177ee
7133038: G1: Some small profile based optimizations
johnc
parents:
11584
diff
changeset
|
206 |
|
24106
dae9277bdf2a
8040722: G1: Clean up usages of heap_region_containing
brutisso
parents:
22905
diff
changeset
|
207 |
if (_record_refs_into_cset && to->in_collection_set()) { |
dae9277bdf2a
8040722: G1: Clean up usages of heap_region_containing
brutisso
parents:
22905
diff
changeset
|
208 |
// We are recording references that point into the collection |
dae9277bdf2a
8040722: G1: Clean up usages of heap_region_containing
brutisso
parents:
22905
diff
changeset
|
209 |
// set and this particular reference does exactly that... |
dae9277bdf2a
8040722: G1: Clean up usages of heap_region_containing
brutisso
parents:
22905
diff
changeset
|
210 |
// If the referenced object has already been forwarded |
dae9277bdf2a
8040722: G1: Clean up usages of heap_region_containing
brutisso
parents:
22905
diff
changeset
|
211 |
// to itself, we are handling an evacuation failure and |
dae9277bdf2a
8040722: G1: Clean up usages of heap_region_containing
brutisso
parents:
22905
diff
changeset
|
212 |
// we have already visited/tried to copy this object |
dae9277bdf2a
8040722: G1: Clean up usages of heap_region_containing
brutisso
parents:
22905
diff
changeset
|
213 |
// there is no need to retry. |
dae9277bdf2a
8040722: G1: Clean up usages of heap_region_containing
brutisso
parents:
22905
diff
changeset
|
214 |
if (!self_forwarded(obj)) { |
dae9277bdf2a
8040722: G1: Clean up usages of heap_region_containing
brutisso
parents:
22905
diff
changeset
|
215 |
assert(_push_ref_cl != NULL, "should not be null"); |
dae9277bdf2a
8040722: G1: Clean up usages of heap_region_containing
brutisso
parents:
22905
diff
changeset
|
216 |
// Push the reference in the refs queue of the G1ParScanThreadState |
dae9277bdf2a
8040722: G1: Clean up usages of heap_region_containing
brutisso
parents:
22905
diff
changeset
|
217 |
// instance for this worker thread. |
dae9277bdf2a
8040722: G1: Clean up usages of heap_region_containing
brutisso
parents:
22905
diff
changeset
|
218 |
_push_ref_cl->do_oop(p); |
11586
ccc217c177ee
7133038: G1: Some small profile based optimizations
johnc
parents:
11584
diff
changeset
|
219 |
} |
13333
f51af0093d29
7173712: G1: Duplicated code in G1UpdateRSOrPushRefOopClosure::do_oop_nv()
johnc
parents:
11586
diff
changeset
|
220 |
|
24106
dae9277bdf2a
8040722: G1: Clean up usages of heap_region_containing
brutisso
parents:
22905
diff
changeset
|
221 |
// Deferred updates to the CSet are either discarded (in the normal case), |
dae9277bdf2a
8040722: G1: Clean up usages of heap_region_containing
brutisso
parents:
22905
diff
changeset
|
222 |
// or processed (if an evacuation failure occurs) at the end |
dae9277bdf2a
8040722: G1: Clean up usages of heap_region_containing
brutisso
parents:
22905
diff
changeset
|
223 |
// of the collection. |
dae9277bdf2a
8040722: G1: Clean up usages of heap_region_containing
brutisso
parents:
22905
diff
changeset
|
224 |
// See G1RemSet::cleanup_after_oops_into_collection_set_do(). |
dae9277bdf2a
8040722: G1: Clean up usages of heap_region_containing
brutisso
parents:
22905
diff
changeset
|
225 |
} else { |
13333
f51af0093d29
7173712: G1: Duplicated code in G1UpdateRSOrPushRefOopClosure::do_oop_nv()
johnc
parents:
11586
diff
changeset
|
226 |
// We either don't care about pushing references that point into the |
f51af0093d29
7173712: G1: Duplicated code in G1UpdateRSOrPushRefOopClosure::do_oop_nv()
johnc
parents:
11586
diff
changeset
|
227 |
// collection set (i.e. we're not during an evacuation pause) _or_ |
f51af0093d29
7173712: G1: Duplicated code in G1UpdateRSOrPushRefOopClosure::do_oop_nv()
johnc
parents:
11586
diff
changeset
|
228 |
// the reference doesn't point into the collection set. Either way |
f51af0093d29
7173712: G1: Duplicated code in G1UpdateRSOrPushRefOopClosure::do_oop_nv()
johnc
parents:
11586
diff
changeset
|
229 |
// we add the reference directly to the RSet of the region containing |
f51af0093d29
7173712: G1: Duplicated code in G1UpdateRSOrPushRefOopClosure::do_oop_nv()
johnc
parents:
11586
diff
changeset
|
230 |
// the referenced object. |
f51af0093d29
7173712: G1: Duplicated code in G1UpdateRSOrPushRefOopClosure::do_oop_nv()
johnc
parents:
11586
diff
changeset
|
231 |
assert(to->rem_set() != NULL, "Need per-region 'into' remsets."); |
f51af0093d29
7173712: G1: Duplicated code in G1UpdateRSOrPushRefOopClosure::do_oop_nv()
johnc
parents:
11586
diff
changeset
|
232 |
to->rem_set()->add_reference(p, _worker_i); |
11586
ccc217c177ee
7133038: G1: Some small profile based optimizations
johnc
parents:
11584
diff
changeset
|
233 |
} |
ccc217c177ee
7133038: G1: Some small profile based optimizations
johnc
parents:
11584
diff
changeset
|
234 |
} |
35862
411842d0c882
8146395: Add inline qualifier in oop.hpp and fix inlining in gc files
goetz
parents:
35069
diff
changeset
|
235 |
void G1UpdateRSOrPushRefOopClosure::do_oop(oop* p) { do_oop_work(p); } |
411842d0c882
8146395: Add inline qualifier in oop.hpp and fix inlining in gc files
goetz
parents:
35069
diff
changeset
|
236 |
void G1UpdateRSOrPushRefOopClosure::do_oop(narrowOop* p) { do_oop_work(p); } |
11586
ccc217c177ee
7133038: G1: Some small profile based optimizations
johnc
parents:
11584
diff
changeset
|
237 |
|
33213
b937f634f56e
8138762: Refactor setup of evacuation closures in G1
mgerdin
parents:
32185
diff
changeset
|
238 |
template <class T> |
b937f634f56e
8138762: Refactor setup of evacuation closures in G1
mgerdin
parents:
32185
diff
changeset
|
239 |
void G1ParCopyHelper::do_klass_barrier(T* p, oop new_obj) { |
33786
ac8da6513351
8139867: Change how startsHumongous and continuesHumongous regions work in G1.
david
parents:
33740
diff
changeset
|
240 |
if (_g1->heap_region_containing(new_obj)->is_young()) { |
33213
b937f634f56e
8138762: Refactor setup of evacuation closures in G1
mgerdin
parents:
32185
diff
changeset
|
241 |
_scanned_klass->record_modified_oops(); |
b937f634f56e
8138762: Refactor setup of evacuation closures in G1
mgerdin
parents:
32185
diff
changeset
|
242 |
} |
b937f634f56e
8138762: Refactor setup of evacuation closures in G1
mgerdin
parents:
32185
diff
changeset
|
243 |
} |
b937f634f56e
8138762: Refactor setup of evacuation closures in G1
mgerdin
parents:
32185
diff
changeset
|
244 |
|
b937f634f56e
8138762: Refactor setup of evacuation closures in G1
mgerdin
parents:
32185
diff
changeset
|
245 |
void G1ParCopyHelper::mark_object(oop obj) { |
b937f634f56e
8138762: Refactor setup of evacuation closures in G1
mgerdin
parents:
32185
diff
changeset
|
246 |
assert(!_g1->heap_region_containing(obj)->in_collection_set(), "should not mark objects in the CSet"); |
b937f634f56e
8138762: Refactor setup of evacuation closures in G1
mgerdin
parents:
32185
diff
changeset
|
247 |
|
b937f634f56e
8138762: Refactor setup of evacuation closures in G1
mgerdin
parents:
32185
diff
changeset
|
248 |
// We know that the object is not moving so it's safe to read its size. |
37413
2f71679d06dd
8077144: Concurrent mark initialization takes too long
tschatzl
parents:
37258
diff
changeset
|
249 |
_cm->grayRoot(obj); |
33213
b937f634f56e
8138762: Refactor setup of evacuation closures in G1
mgerdin
parents:
32185
diff
changeset
|
250 |
} |
b937f634f56e
8138762: Refactor setup of evacuation closures in G1
mgerdin
parents:
32185
diff
changeset
|
251 |
|
b937f634f56e
8138762: Refactor setup of evacuation closures in G1
mgerdin
parents:
32185
diff
changeset
|
252 |
void G1ParCopyHelper::mark_forwarded_object(oop from_obj, oop to_obj) { |
b937f634f56e
8138762: Refactor setup of evacuation closures in G1
mgerdin
parents:
32185
diff
changeset
|
253 |
assert(from_obj->is_forwarded(), "from obj should be forwarded"); |
b937f634f56e
8138762: Refactor setup of evacuation closures in G1
mgerdin
parents:
32185
diff
changeset
|
254 |
assert(from_obj->forwardee() == to_obj, "to obj should be the forwardee"); |
b937f634f56e
8138762: Refactor setup of evacuation closures in G1
mgerdin
parents:
32185
diff
changeset
|
255 |
assert(from_obj != to_obj, "should not be self-forwarded"); |
b937f634f56e
8138762: Refactor setup of evacuation closures in G1
mgerdin
parents:
32185
diff
changeset
|
256 |
|
b937f634f56e
8138762: Refactor setup of evacuation closures in G1
mgerdin
parents:
32185
diff
changeset
|
257 |
assert(_g1->heap_region_containing(from_obj)->in_collection_set(), "from obj should be in the CSet"); |
b937f634f56e
8138762: Refactor setup of evacuation closures in G1
mgerdin
parents:
32185
diff
changeset
|
258 |
assert(!_g1->heap_region_containing(to_obj)->in_collection_set(), "should not mark objects in the CSet"); |
b937f634f56e
8138762: Refactor setup of evacuation closures in G1
mgerdin
parents:
32185
diff
changeset
|
259 |
|
b937f634f56e
8138762: Refactor setup of evacuation closures in G1
mgerdin
parents:
32185
diff
changeset
|
260 |
// The object might be in the process of being copied by another |
b937f634f56e
8138762: Refactor setup of evacuation closures in G1
mgerdin
parents:
32185
diff
changeset
|
261 |
// worker so we cannot trust that its to-space image is |
b937f634f56e
8138762: Refactor setup of evacuation closures in G1
mgerdin
parents:
32185
diff
changeset
|
262 |
// well-formed. So we have to read its size from its from-space |
b937f634f56e
8138762: Refactor setup of evacuation closures in G1
mgerdin
parents:
32185
diff
changeset
|
263 |
// image which we know should not be changing. |
37413
2f71679d06dd
8077144: Concurrent mark initialization takes too long
tschatzl
parents:
37258
diff
changeset
|
264 |
_cm->grayRoot(to_obj); |
33213
b937f634f56e
8138762: Refactor setup of evacuation closures in G1
mgerdin
parents:
32185
diff
changeset
|
265 |
} |
b937f634f56e
8138762: Refactor setup of evacuation closures in G1
mgerdin
parents:
32185
diff
changeset
|
266 |
|
35051
002d874a4284
8144714: Add extension point to G1 evacuation closures
mgerdin
parents:
34616
diff
changeset
|
267 |
template <G1Barrier barrier, G1Mark do_mark_object, bool use_ext> |
33213
b937f634f56e
8138762: Refactor setup of evacuation closures in G1
mgerdin
parents:
32185
diff
changeset
|
268 |
template <class T> |
35067
0fa08ea22241
8144505: Change G1ParCopyHelper to inherit OopClosure
sjohanss
parents:
35062
diff
changeset
|
269 |
void G1ParCopyClosure<barrier, do_mark_object, use_ext>::do_oop_work(T* p) { |
33213
b937f634f56e
8138762: Refactor setup of evacuation closures in G1
mgerdin
parents:
32185
diff
changeset
|
270 |
T heap_oop = oopDesc::load_heap_oop(p); |
b937f634f56e
8138762: Refactor setup of evacuation closures in G1
mgerdin
parents:
32185
diff
changeset
|
271 |
|
b937f634f56e
8138762: Refactor setup of evacuation closures in G1
mgerdin
parents:
32185
diff
changeset
|
272 |
if (oopDesc::is_null(heap_oop)) { |
b937f634f56e
8138762: Refactor setup of evacuation closures in G1
mgerdin
parents:
32185
diff
changeset
|
273 |
return; |
b937f634f56e
8138762: Refactor setup of evacuation closures in G1
mgerdin
parents:
32185
diff
changeset
|
274 |
} |
b937f634f56e
8138762: Refactor setup of evacuation closures in G1
mgerdin
parents:
32185
diff
changeset
|
275 |
|
b937f634f56e
8138762: Refactor setup of evacuation closures in G1
mgerdin
parents:
32185
diff
changeset
|
276 |
oop obj = oopDesc::decode_heap_oop_not_null(heap_oop); |
b937f634f56e
8138762: Refactor setup of evacuation closures in G1
mgerdin
parents:
32185
diff
changeset
|
277 |
|
b937f634f56e
8138762: Refactor setup of evacuation closures in G1
mgerdin
parents:
32185
diff
changeset
|
278 |
assert(_worker_id == _par_scan_state->worker_id(), "sanity"); |
b937f634f56e
8138762: Refactor setup of evacuation closures in G1
mgerdin
parents:
32185
diff
changeset
|
279 |
|
b937f634f56e
8138762: Refactor setup of evacuation closures in G1
mgerdin
parents:
32185
diff
changeset
|
280 |
const InCSetState state = _g1->in_cset_state(obj); |
b937f634f56e
8138762: Refactor setup of evacuation closures in G1
mgerdin
parents:
32185
diff
changeset
|
281 |
if (state.is_in_cset()) { |
b937f634f56e
8138762: Refactor setup of evacuation closures in G1
mgerdin
parents:
32185
diff
changeset
|
282 |
oop forwardee; |
b937f634f56e
8138762: Refactor setup of evacuation closures in G1
mgerdin
parents:
32185
diff
changeset
|
283 |
markOop m = obj->mark(); |
b937f634f56e
8138762: Refactor setup of evacuation closures in G1
mgerdin
parents:
32185
diff
changeset
|
284 |
if (m->is_marked()) { |
b937f634f56e
8138762: Refactor setup of evacuation closures in G1
mgerdin
parents:
32185
diff
changeset
|
285 |
forwardee = (oop) m->decode_pointer(); |
b937f634f56e
8138762: Refactor setup of evacuation closures in G1
mgerdin
parents:
32185
diff
changeset
|
286 |
} else { |
b937f634f56e
8138762: Refactor setup of evacuation closures in G1
mgerdin
parents:
32185
diff
changeset
|
287 |
forwardee = _par_scan_state->copy_to_survivor_space(state, obj, m); |
b937f634f56e
8138762: Refactor setup of evacuation closures in G1
mgerdin
parents:
32185
diff
changeset
|
288 |
} |
b937f634f56e
8138762: Refactor setup of evacuation closures in G1
mgerdin
parents:
32185
diff
changeset
|
289 |
assert(forwardee != NULL, "forwardee should not be NULL"); |
b937f634f56e
8138762: Refactor setup of evacuation closures in G1
mgerdin
parents:
32185
diff
changeset
|
290 |
oopDesc::encode_store_heap_oop(p, forwardee); |
b937f634f56e
8138762: Refactor setup of evacuation closures in G1
mgerdin
parents:
32185
diff
changeset
|
291 |
if (do_mark_object != G1MarkNone && forwardee != obj) { |
b937f634f56e
8138762: Refactor setup of evacuation closures in G1
mgerdin
parents:
32185
diff
changeset
|
292 |
// If the object is self-forwarded we don't need to explicitly |
b937f634f56e
8138762: Refactor setup of evacuation closures in G1
mgerdin
parents:
32185
diff
changeset
|
293 |
// mark it, the evacuation failure protocol will do so. |
b937f634f56e
8138762: Refactor setup of evacuation closures in G1
mgerdin
parents:
32185
diff
changeset
|
294 |
mark_forwarded_object(obj, forwardee); |
b937f634f56e
8138762: Refactor setup of evacuation closures in G1
mgerdin
parents:
32185
diff
changeset
|
295 |
} |
b937f634f56e
8138762: Refactor setup of evacuation closures in G1
mgerdin
parents:
32185
diff
changeset
|
296 |
|
b937f634f56e
8138762: Refactor setup of evacuation closures in G1
mgerdin
parents:
32185
diff
changeset
|
297 |
if (barrier == G1BarrierKlass) { |
b937f634f56e
8138762: Refactor setup of evacuation closures in G1
mgerdin
parents:
32185
diff
changeset
|
298 |
do_klass_barrier(p, forwardee); |
b937f634f56e
8138762: Refactor setup of evacuation closures in G1
mgerdin
parents:
32185
diff
changeset
|
299 |
} |
b937f634f56e
8138762: Refactor setup of evacuation closures in G1
mgerdin
parents:
32185
diff
changeset
|
300 |
} else { |
b937f634f56e
8138762: Refactor setup of evacuation closures in G1
mgerdin
parents:
32185
diff
changeset
|
301 |
if (state.is_humongous()) { |
b937f634f56e
8138762: Refactor setup of evacuation closures in G1
mgerdin
parents:
32185
diff
changeset
|
302 |
_g1->set_humongous_is_live(obj); |
b937f634f56e
8138762: Refactor setup of evacuation closures in G1
mgerdin
parents:
32185
diff
changeset
|
303 |
} |
35051
002d874a4284
8144714: Add extension point to G1 evacuation closures
mgerdin
parents:
34616
diff
changeset
|
304 |
|
002d874a4284
8144714: Add extension point to G1 evacuation closures
mgerdin
parents:
34616
diff
changeset
|
305 |
if (use_ext && state.is_ext()) { |
002d874a4284
8144714: Add extension point to G1 evacuation closures
mgerdin
parents:
34616
diff
changeset
|
306 |
_par_scan_state->do_oop_ext(p); |
002d874a4284
8144714: Add extension point to G1 evacuation closures
mgerdin
parents:
34616
diff
changeset
|
307 |
} |
33213
b937f634f56e
8138762: Refactor setup of evacuation closures in G1
mgerdin
parents:
32185
diff
changeset
|
308 |
// The object is not in collection set. If we're a root scanning |
b937f634f56e
8138762: Refactor setup of evacuation closures in G1
mgerdin
parents:
32185
diff
changeset
|
309 |
// closure during an initial mark pause then attempt to mark the object. |
b937f634f56e
8138762: Refactor setup of evacuation closures in G1
mgerdin
parents:
32185
diff
changeset
|
310 |
if (do_mark_object == G1MarkFromRoot) { |
b937f634f56e
8138762: Refactor setup of evacuation closures in G1
mgerdin
parents:
32185
diff
changeset
|
311 |
mark_object(obj); |
b937f634f56e
8138762: Refactor setup of evacuation closures in G1
mgerdin
parents:
32185
diff
changeset
|
312 |
} |
b937f634f56e
8138762: Refactor setup of evacuation closures in G1
mgerdin
parents:
32185
diff
changeset
|
313 |
} |
b937f634f56e
8138762: Refactor setup of evacuation closures in G1
mgerdin
parents:
32185
diff
changeset
|
314 |
} |
b937f634f56e
8138762: Refactor setup of evacuation closures in G1
mgerdin
parents:
32185
diff
changeset
|
315 |
|
30764 | 316 |
#endif // SHARE_VM_GC_G1_G1OOPCLOSURES_INLINE_HPP |