author | sjohanss |
Wed, 15 Nov 2017 17:05:28 +0100 | |
changeset 47897 | eb3696e0c6d8 |
parent 47885 | 5caa1d5f74c1 |
child 49592 | 77fb0be7d19f |
permissions | -rw-r--r-- |
1 | 1 |
/* |
46415
7d5a286cdf89
8138737: Remove oop_ms_adjust_pointers and use oop_iterate instead
sjohanss
parents:
35498
diff
changeset
|
2 |
* Copyright (c) 1997, 2017, 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:
5080
diff
changeset
|
19 |
* Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA |
f4b087cbb361
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
5080
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:
5080
diff
changeset
|
21 |
* questions. |
1 | 22 |
* |
23 |
*/ |
|
24 |
||
7397 | 25 |
#include "precompiled.hpp" |
26 |
#include "compiler/compileBroker.hpp" |
|
30764 | 27 |
#include "gc/serial/markSweep.inline.hpp" |
28 |
#include "gc/shared/collectedHeap.inline.hpp" |
|
29 |
#include "gc/shared/gcTimer.hpp" |
|
30 |
#include "gc/shared/gcTrace.hpp" |
|
32606
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
31 |
#include "gc/shared/specialized_oop_closures.hpp" |
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
32 |
#include "memory/iterator.inline.hpp" |
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
33 |
#include "oops/instanceClassLoaderKlass.inline.hpp" |
30150
d9c940aa42ef
8075955: Replace the macro based implementation of oop_oop_iterate with a template based solution
stefank
parents:
29800
diff
changeset
|
34 |
#include "oops/instanceKlass.inline.hpp" |
d9c940aa42ef
8075955: Replace the macro based implementation of oop_oop_iterate with a template based solution
stefank
parents:
29800
diff
changeset
|
35 |
#include "oops/instanceMirrorKlass.inline.hpp" |
32606
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
36 |
#include "oops/instanceRefKlass.inline.hpp" |
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13195
diff
changeset
|
37 |
#include "oops/methodData.hpp" |
7397 | 38 |
#include "oops/objArrayKlass.inline.hpp" |
39 |
#include "oops/oop.inline.hpp" |
|
35498
392b50de06c6
8146401: Clean up oop.hpp: add inline directives and fix header files
goetz
parents:
35061
diff
changeset
|
40 |
#include "oops/typeArrayOop.inline.hpp" |
32606
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
41 |
#include "utilities/macros.hpp" |
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
42 |
#include "utilities/stack.inline.hpp" |
1 | 43 |
|
17392
2f967c0e4246
6843347: Boundary values in some public GC options cause crashes
tschatzl
parents:
17105
diff
changeset
|
44 |
uint MarkSweep::_total_invocations = 0; |
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13195
diff
changeset
|
45 |
|
13195 | 46 |
Stack<oop, mtGC> MarkSweep::_marking_stack; |
47 |
Stack<ObjArrayTask, mtGC> MarkSweep::_objarray_stack; |
|
1 | 48 |
|
13195 | 49 |
Stack<oop, mtGC> MarkSweep::_preserved_oop_stack; |
50 |
Stack<markOop, mtGC> MarkSweep::_preserved_mark_stack; |
|
1 | 51 |
size_t MarkSweep::_preserved_count = 0; |
52 |
size_t MarkSweep::_preserved_count_max = 0; |
|
53 |
PreservedMark* MarkSweep::_preserved_marks = NULL; |
|
54 |
ReferenceProcessor* MarkSweep::_ref_processor = NULL; |
|
18025 | 55 |
STWGCTimer* MarkSweep::_gc_timer = NULL; |
56 |
SerialOldTracer* MarkSweep::_gc_tracer = NULL; |
|
1 | 57 |
|
360
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
1
diff
changeset
|
58 |
MarkSweep::FollowRootClosure MarkSweep::follow_root_closure; |
1 | 59 |
|
32606
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
60 |
MarkAndPushClosure MarkSweep::mark_and_push_closure; |
25492
d27050bdfb04
8049421: G1 Class Unloading after completing a concurrent mark cycle
stefank
parents:
25491
diff
changeset
|
61 |
CLDToOopClosure MarkSweep::follow_cld_closure(&mark_and_push_closure); |
d27050bdfb04
8049421: G1 Class Unloading after completing a concurrent mark cycle
stefank
parents:
25491
diff
changeset
|
62 |
CLDToOopClosure MarkSweep::adjust_cld_closure(&adjust_pointer_closure); |
1 | 63 |
|
32606
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
64 |
inline void MarkSweep::mark_object(oop obj) { |
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
65 |
// some marks may contain information we need to preserve so we store them away |
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
66 |
// and overwrite the mark. We'll restore it at the end of markSweep. |
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
67 |
markOop mark = obj->mark(); |
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
68 |
obj->set_mark(markOopDesc::prototype()->set_marked()); |
1 | 69 |
|
32606
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
70 |
if (mark->must_be_preserved(obj)) { |
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
71 |
preserve_mark(obj, mark); |
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
72 |
} |
30150
d9c940aa42ef
8075955: Replace the macro based implementation of oop_oop_iterate with a template based solution
stefank
parents:
29800
diff
changeset
|
73 |
} |
d9c940aa42ef
8075955: Replace the macro based implementation of oop_oop_iterate with a template based solution
stefank
parents:
29800
diff
changeset
|
74 |
|
32606
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
75 |
template <class T> inline void MarkSweep::mark_and_push(T* p) { |
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
76 |
T heap_oop = oopDesc::load_heap_oop(p); |
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
77 |
if (!oopDesc::is_null(heap_oop)) { |
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
78 |
oop obj = oopDesc::decode_heap_oop_not_null(heap_oop); |
47897
eb3696e0c6d8
8189734: Cleanup MarkSweep when G1 dependency is gone
sjohanss
parents:
47885
diff
changeset
|
79 |
if (!obj->mark()->is_marked()) { |
32606
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
80 |
mark_object(obj); |
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
81 |
_marking_stack.push(obj); |
30150
d9c940aa42ef
8075955: Replace the macro based implementation of oop_oop_iterate with a template based solution
stefank
parents:
29800
diff
changeset
|
82 |
} |
d9c940aa42ef
8075955: Replace the macro based implementation of oop_oop_iterate with a template based solution
stefank
parents:
29800
diff
changeset
|
83 |
} |
d9c940aa42ef
8075955: Replace the macro based implementation of oop_oop_iterate with a template based solution
stefank
parents:
29800
diff
changeset
|
84 |
} |
d9c940aa42ef
8075955: Replace the macro based implementation of oop_oop_iterate with a template based solution
stefank
parents:
29800
diff
changeset
|
85 |
|
32606
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
86 |
inline void MarkSweep::follow_klass(Klass* klass) { |
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
87 |
oop op = klass->klass_holder(); |
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
88 |
MarkSweep::mark_and_push(&op); |
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
89 |
} |
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
90 |
|
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
91 |
inline void MarkSweep::follow_cld(ClassLoaderData* cld) { |
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
92 |
MarkSweep::follow_cld_closure.do_cld(cld); |
30150
d9c940aa42ef
8075955: Replace the macro based implementation of oop_oop_iterate with a template based solution
stefank
parents:
29800
diff
changeset
|
93 |
} |
d9c940aa42ef
8075955: Replace the macro based implementation of oop_oop_iterate with a template based solution
stefank
parents:
29800
diff
changeset
|
94 |
|
32606
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
95 |
template <typename T> |
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
96 |
inline void MarkAndPushClosure::do_oop_nv(T* p) { MarkSweep::mark_and_push(p); } |
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
97 |
void MarkAndPushClosure::do_oop(oop* p) { do_oop_nv(p); } |
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
98 |
void MarkAndPushClosure::do_oop(narrowOop* p) { do_oop_nv(p); } |
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
99 |
inline bool MarkAndPushClosure::do_metadata_nv() { return true; } |
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
100 |
bool MarkAndPushClosure::do_metadata() { return do_metadata_nv(); } |
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
101 |
inline void MarkAndPushClosure::do_klass_nv(Klass* k) { MarkSweep::follow_klass(k); } |
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
102 |
void MarkAndPushClosure::do_klass(Klass* k) { do_klass_nv(k); } |
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
103 |
inline void MarkAndPushClosure::do_cld_nv(ClassLoaderData* cld) { MarkSweep::follow_cld(cld); } |
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
104 |
void MarkAndPushClosure::do_cld(ClassLoaderData* cld) { do_cld_nv(cld); } |
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
105 |
|
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
106 |
template <class T> inline void MarkSweep::KeepAliveClosure::do_oop_work(T* p) { |
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
107 |
mark_and_push(p); |
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
108 |
} |
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
109 |
|
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
110 |
void MarkSweep::push_objarray(oop obj, size_t index) { |
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
111 |
ObjArrayTask task(obj, index); |
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
112 |
assert(task.is_valid(), "bad ObjArrayTask"); |
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
113 |
_objarray_stack.push(task); |
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
114 |
} |
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
115 |
|
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
116 |
inline void MarkSweep::follow_array(objArrayOop array) { |
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
117 |
MarkSweep::follow_klass(array->klass()); |
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
118 |
// Don't push empty arrays to avoid unnecessary work. |
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
119 |
if (array->length() > 0) { |
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
120 |
MarkSweep::push_objarray(array, 0); |
30150
d9c940aa42ef
8075955: Replace the macro based implementation of oop_oop_iterate with a template based solution
stefank
parents:
29800
diff
changeset
|
121 |
} |
d9c940aa42ef
8075955: Replace the macro based implementation of oop_oop_iterate with a template based solution
stefank
parents:
29800
diff
changeset
|
122 |
} |
d9c940aa42ef
8075955: Replace the macro based implementation of oop_oop_iterate with a template based solution
stefank
parents:
29800
diff
changeset
|
123 |
|
32606
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
124 |
inline void MarkSweep::follow_object(oop obj) { |
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
125 |
assert(obj->is_gc_marked(), "should be marked"); |
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
126 |
if (obj->is_objArray()) { |
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
127 |
// Handle object arrays explicitly to allow them to |
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
128 |
// be split into chunks if needed. |
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
129 |
MarkSweep::follow_array((objArrayOop)obj); |
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
130 |
} else { |
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
131 |
obj->oop_iterate(&mark_and_push_closure); |
30150
d9c940aa42ef
8075955: Replace the macro based implementation of oop_oop_iterate with a template based solution
stefank
parents:
29800
diff
changeset
|
132 |
} |
d9c940aa42ef
8075955: Replace the macro based implementation of oop_oop_iterate with a template based solution
stefank
parents:
29800
diff
changeset
|
133 |
} |
d9c940aa42ef
8075955: Replace the macro based implementation of oop_oop_iterate with a template based solution
stefank
parents:
29800
diff
changeset
|
134 |
|
32606
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
135 |
void MarkSweep::follow_array_chunk(objArrayOop array, int index) { |
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
136 |
const int len = array->length(); |
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
137 |
const int beg_index = index; |
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
138 |
assert(beg_index < len || len == 0, "index too large"); |
30150
d9c940aa42ef
8075955: Replace the macro based implementation of oop_oop_iterate with a template based solution
stefank
parents:
29800
diff
changeset
|
139 |
|
32606
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
140 |
const int stride = MIN2(len - beg_index, (int) ObjArrayMarkingStride); |
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
141 |
const int end_index = beg_index + stride; |
30150
d9c940aa42ef
8075955: Replace the macro based implementation of oop_oop_iterate with a template based solution
stefank
parents:
29800
diff
changeset
|
142 |
|
32606
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
143 |
array->oop_iterate_range(&mark_and_push_closure, beg_index, end_index); |
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
144 |
|
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
145 |
if (end_index < len) { |
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
146 |
MarkSweep::push_objarray(array, end_index); // Push the continuation. |
30150
d9c940aa42ef
8075955: Replace the macro based implementation of oop_oop_iterate with a template based solution
stefank
parents:
29800
diff
changeset
|
147 |
} |
29792
8c6fa07f0869
8075957: Reduce calls to the GC specific object visitors in oopDesc
stefank
parents:
29701
diff
changeset
|
148 |
} |
8c6fa07f0869
8075957: Reduce calls to the GC specific object visitors in oopDesc
stefank
parents:
29701
diff
changeset
|
149 |
|
1 | 150 |
void MarkSweep::follow_stack() { |
5076
8b74a4b60b31
4396719: Mark Sweep stack overflow on deeply nested Object arrays
jcoomes
parents:
3919
diff
changeset
|
151 |
do { |
6762
f8d1b560700e
6423256: GC stacks should use a better data structure
jcoomes
parents:
5547
diff
changeset
|
152 |
while (!_marking_stack.is_empty()) { |
f8d1b560700e
6423256: GC stacks should use a better data structure
jcoomes
parents:
5547
diff
changeset
|
153 |
oop obj = _marking_stack.pop(); |
5076
8b74a4b60b31
4396719: Mark Sweep stack overflow on deeply nested Object arrays
jcoomes
parents:
3919
diff
changeset
|
154 |
assert (obj->is_gc_marked(), "p must be marked"); |
29792
8c6fa07f0869
8075957: Reduce calls to the GC specific object visitors in oopDesc
stefank
parents:
29701
diff
changeset
|
155 |
follow_object(obj); |
5076
8b74a4b60b31
4396719: Mark Sweep stack overflow on deeply nested Object arrays
jcoomes
parents:
3919
diff
changeset
|
156 |
} |
5080
eff0cc882603
6935839: excessive marking stack growth during full gcs
jcoomes
parents:
5076
diff
changeset
|
157 |
// Process ObjArrays one at a time to avoid marking stack bloat. |
6762
f8d1b560700e
6423256: GC stacks should use a better data structure
jcoomes
parents:
5547
diff
changeset
|
158 |
if (!_objarray_stack.is_empty()) { |
f8d1b560700e
6423256: GC stacks should use a better data structure
jcoomes
parents:
5547
diff
changeset
|
159 |
ObjArrayTask task = _objarray_stack.pop(); |
32606
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
160 |
follow_array_chunk(objArrayOop(task.obj()), task.index()); |
5076
8b74a4b60b31
4396719: Mark Sweep stack overflow on deeply nested Object arrays
jcoomes
parents:
3919
diff
changeset
|
161 |
} |
6762
f8d1b560700e
6423256: GC stacks should use a better data structure
jcoomes
parents:
5547
diff
changeset
|
162 |
} while (!_marking_stack.is_empty() || !_objarray_stack.is_empty()); |
1 | 163 |
} |
164 |
||
165 |
MarkSweep::FollowStackClosure MarkSweep::follow_stack_closure; |
|
166 |
||
360
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
1
diff
changeset
|
167 |
void MarkSweep::FollowStackClosure::do_void() { follow_stack(); } |
1 | 168 |
|
32606
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
169 |
template <class T> inline void MarkSweep::follow_root(T* p) { |
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
170 |
assert(!Universe::heap()->is_in_reserved(p), |
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
171 |
"roots shouldn't be things within the heap"); |
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
172 |
T heap_oop = oopDesc::load_heap_oop(p); |
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
173 |
if (!oopDesc::is_null(heap_oop)) { |
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
174 |
oop obj = oopDesc::decode_heap_oop_not_null(heap_oop); |
47897
eb3696e0c6d8
8189734: Cleanup MarkSweep when G1 dependency is gone
sjohanss
parents:
47885
diff
changeset
|
175 |
if (!obj->mark()->is_marked()) { |
32606
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
176 |
mark_object(obj); |
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
177 |
follow_object(obj); |
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
178 |
} |
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
179 |
} |
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
180 |
follow_stack(); |
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
181 |
} |
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
182 |
|
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
183 |
void MarkSweep::FollowRootClosure::do_oop(oop* p) { follow_root(p); } |
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
184 |
void MarkSweep::FollowRootClosure::do_oop(narrowOop* p) { follow_root(p); } |
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
185 |
|
29701
f638fc81a3d0
8075803: Fix GC includes and forward declarations
stefank
parents:
25492
diff
changeset
|
186 |
void PreservedMark::adjust_pointer() { |
f638fc81a3d0
8075803: Fix GC includes and forward declarations
stefank
parents:
25492
diff
changeset
|
187 |
MarkSweep::adjust_pointer(&_obj); |
f638fc81a3d0
8075803: Fix GC includes and forward declarations
stefank
parents:
25492
diff
changeset
|
188 |
} |
f638fc81a3d0
8075803: Fix GC includes and forward declarations
stefank
parents:
25492
diff
changeset
|
189 |
|
f638fc81a3d0
8075803: Fix GC includes and forward declarations
stefank
parents:
25492
diff
changeset
|
190 |
void PreservedMark::restore() { |
f638fc81a3d0
8075803: Fix GC includes and forward declarations
stefank
parents:
25492
diff
changeset
|
191 |
_obj->set_mark(_mark); |
f638fc81a3d0
8075803: Fix GC includes and forward declarations
stefank
parents:
25492
diff
changeset
|
192 |
} |
f638fc81a3d0
8075803: Fix GC includes and forward declarations
stefank
parents:
25492
diff
changeset
|
193 |
|
6762
f8d1b560700e
6423256: GC stacks should use a better data structure
jcoomes
parents:
5547
diff
changeset
|
194 |
// We preserve the mark which should be replaced at the end and the location |
f8d1b560700e
6423256: GC stacks should use a better data structure
jcoomes
parents:
5547
diff
changeset
|
195 |
// that it will go. Note that the object that this markOop belongs to isn't |
f8d1b560700e
6423256: GC stacks should use a better data structure
jcoomes
parents:
5547
diff
changeset
|
196 |
// currently at that address but it will be after phase4 |
1 | 197 |
void MarkSweep::preserve_mark(oop obj, markOop mark) { |
6762
f8d1b560700e
6423256: GC stacks should use a better data structure
jcoomes
parents:
5547
diff
changeset
|
198 |
// We try to store preserved marks in the to space of the new generation since |
f8d1b560700e
6423256: GC stacks should use a better data structure
jcoomes
parents:
5547
diff
changeset
|
199 |
// this is storage which should be available. Most of the time this should be |
f8d1b560700e
6423256: GC stacks should use a better data structure
jcoomes
parents:
5547
diff
changeset
|
200 |
// sufficient space for the marks we need to preserve but if it isn't we fall |
f8d1b560700e
6423256: GC stacks should use a better data structure
jcoomes
parents:
5547
diff
changeset
|
201 |
// back to using Stacks to keep track of the overflow. |
1 | 202 |
if (_preserved_count < _preserved_count_max) { |
203 |
_preserved_marks[_preserved_count++].init(obj, mark); |
|
204 |
} else { |
|
6762
f8d1b560700e
6423256: GC stacks should use a better data structure
jcoomes
parents:
5547
diff
changeset
|
205 |
_preserved_mark_stack.push(mark); |
f8d1b560700e
6423256: GC stacks should use a better data structure
jcoomes
parents:
5547
diff
changeset
|
206 |
_preserved_oop_stack.push(obj); |
1 | 207 |
} |
208 |
} |
|
209 |
||
32606
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
210 |
void MarkSweep::set_ref_processor(ReferenceProcessor* rp) { |
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
211 |
_ref_processor = rp; |
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
212 |
mark_and_push_closure.set_ref_processor(_ref_processor); |
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
213 |
} |
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
214 |
|
46415
7d5a286cdf89
8138737: Remove oop_ms_adjust_pointers and use oop_iterate instead
sjohanss
parents:
35498
diff
changeset
|
215 |
AdjustPointerClosure MarkSweep::adjust_pointer_closure; |
1 | 216 |
|
30150
d9c940aa42ef
8075955: Replace the macro based implementation of oop_oop_iterate with a template based solution
stefank
parents:
29800
diff
changeset
|
217 |
template <typename T> |
46415
7d5a286cdf89
8138737: Remove oop_ms_adjust_pointers and use oop_iterate instead
sjohanss
parents:
35498
diff
changeset
|
218 |
void AdjustPointerClosure::do_oop_nv(T* p) { MarkSweep::adjust_pointer(p); } |
7d5a286cdf89
8138737: Remove oop_ms_adjust_pointers and use oop_iterate instead
sjohanss
parents:
35498
diff
changeset
|
219 |
void AdjustPointerClosure::do_oop(oop* p) { do_oop_nv(p); } |
7d5a286cdf89
8138737: Remove oop_ms_adjust_pointers and use oop_iterate instead
sjohanss
parents:
35498
diff
changeset
|
220 |
void AdjustPointerClosure::do_oop(narrowOop* p) { do_oop_nv(p); } |
360
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
1
diff
changeset
|
221 |
|
1 | 222 |
void MarkSweep::adjust_marks() { |
6762
f8d1b560700e
6423256: GC stacks should use a better data structure
jcoomes
parents:
5547
diff
changeset
|
223 |
assert( _preserved_oop_stack.size() == _preserved_mark_stack.size(), |
1 | 224 |
"inconsistent preserved oop stacks"); |
225 |
||
226 |
// adjust the oops we saved earlier |
|
227 |
for (size_t i = 0; i < _preserved_count; i++) { |
|
228 |
_preserved_marks[i].adjust_pointer(); |
|
229 |
} |
|
230 |
||
231 |
// deal with the overflow stack |
|
13195 | 232 |
StackIterator<oop, mtGC> iter(_preserved_oop_stack); |
6762
f8d1b560700e
6423256: GC stacks should use a better data structure
jcoomes
parents:
5547
diff
changeset
|
233 |
while (!iter.is_empty()) { |
f8d1b560700e
6423256: GC stacks should use a better data structure
jcoomes
parents:
5547
diff
changeset
|
234 |
oop* p = iter.next_addr(); |
f8d1b560700e
6423256: GC stacks should use a better data structure
jcoomes
parents:
5547
diff
changeset
|
235 |
adjust_pointer(p); |
1 | 236 |
} |
237 |
} |
|
238 |
||
239 |
void MarkSweep::restore_marks() { |
|
6762
f8d1b560700e
6423256: GC stacks should use a better data structure
jcoomes
parents:
5547
diff
changeset
|
240 |
assert(_preserved_oop_stack.size() == _preserved_mark_stack.size(), |
1 | 241 |
"inconsistent preserved oop stacks"); |
35061 | 242 |
log_trace(gc)("Restoring " SIZE_FORMAT " marks", _preserved_count + _preserved_oop_stack.size()); |
1 | 243 |
|
244 |
// restore the marks we saved earlier |
|
245 |
for (size_t i = 0; i < _preserved_count; i++) { |
|
246 |
_preserved_marks[i].restore(); |
|
247 |
} |
|
248 |
||
249 |
// deal with the overflow |
|
6762
f8d1b560700e
6423256: GC stacks should use a better data structure
jcoomes
parents:
5547
diff
changeset
|
250 |
while (!_preserved_oop_stack.is_empty()) { |
f8d1b560700e
6423256: GC stacks should use a better data structure
jcoomes
parents:
5547
diff
changeset
|
251 |
oop obj = _preserved_oop_stack.pop(); |
f8d1b560700e
6423256: GC stacks should use a better data structure
jcoomes
parents:
5547
diff
changeset
|
252 |
markOop mark = _preserved_mark_stack.pop(); |
f8d1b560700e
6423256: GC stacks should use a better data structure
jcoomes
parents:
5547
diff
changeset
|
253 |
obj->set_mark(mark); |
1 | 254 |
} |
255 |
} |
|
256 |
||
360
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
1
diff
changeset
|
257 |
MarkSweep::IsAliveClosure MarkSweep::is_alive; |
1 | 258 |
|
47897
eb3696e0c6d8
8189734: Cleanup MarkSweep when G1 dependency is gone
sjohanss
parents:
47885
diff
changeset
|
259 |
bool MarkSweep::IsAliveClosure::do_object_b(oop p) { return p->is_gc_marked(); } |
1 | 260 |
|
261 |
MarkSweep::KeepAliveClosure MarkSweep::keep_alive; |
|
262 |
||
360
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
1
diff
changeset
|
263 |
void MarkSweep::KeepAliveClosure::do_oop(oop* p) { MarkSweep::KeepAliveClosure::do_oop_work(p); } |
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
1
diff
changeset
|
264 |
void MarkSweep::KeepAliveClosure::do_oop(narrowOop* p) { MarkSweep::KeepAliveClosure::do_oop_work(p); } |
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
1
diff
changeset
|
265 |
|
18025 | 266 |
void marksweep_init() { |
267 |
MarkSweep::_gc_timer = new (ResourceObj::C_HEAP, mtGC) STWGCTimer(); |
|
268 |
MarkSweep::_gc_tracer = new (ResourceObj::C_HEAP, mtGC) SerialOldTracer(); |
|
269 |
} |
|
1 | 270 |
|
32606
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
271 |
// Generate MS specialized oop_oop_iterate functions. |
fdaa30d06ada
8129417: Oop iteration clean-up to remove oop_ms_follow_contents
sjohanss
parents:
31346
diff
changeset
|
272 |
SPECIALIZED_OOP_OOP_ITERATE_CLOSURES_MS(ALL_KLASS_OOP_OOP_ITERATE_DEFN) |