Mercurial > hg > truffle
annotate src/share/vm/code/dependencies.cpp @ 6844:9a9b6e05ffb4
8000232: NPG: SIGSEGV in Dependencies::DepStream::check_klass_dependency on solaris-x64
Summary: Move decoding into Dependencies::DepStream::argument, so no caller could see encoded context value (NULL) anymore.
Reviewed-by: twisti, kvn
author | vlivanov |
---|---|
date | Fri, 05 Oct 2012 19:29:29 -0700 |
parents | 75f33eecc1b3 |
children | 4735d2c84362 |
rev | line source |
---|---|
0 | 1 /* |
5998
49036505ab5f
7154670: The instanceKlass _implementors[] and _nof_implementors are not needed for non-interface klass.
jiangli
parents:
4772
diff
changeset
|
2 * Copyright (c) 2005, 2012, Oracle and/or its affiliates. All rights reserved. |
0 | 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:
1206
diff
changeset
|
19 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA |
c18cbe5936b8
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
1206
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:
1206
diff
changeset
|
21 * questions. |
0 | 22 * |
23 */ | |
24 | |
1972 | 25 #include "precompiled.hpp" |
26 #include "ci/ciArrayKlass.hpp" | |
27 #include "ci/ciEnv.hpp" | |
28 #include "ci/ciKlass.hpp" | |
29 #include "ci/ciMethod.hpp" | |
30 #include "code/dependencies.hpp" | |
31 #include "compiler/compileLog.hpp" | |
32 #include "oops/oop.inline.hpp" | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
33 #include "runtime/handles.hpp" |
1972 | 34 #include "runtime/handles.inline.hpp" |
35 #include "utilities/copy.hpp" | |
0 | 36 |
37 | |
38 #ifdef ASSERT | |
39 static bool must_be_in_vm() { | |
40 Thread* thread = Thread::current(); | |
41 if (thread->is_Java_thread()) | |
42 return ((JavaThread*)thread)->thread_state() == _thread_in_vm; | |
43 else | |
44 return true; //something like this: thread->is_VM_thread(); | |
45 } | |
46 #endif //ASSERT | |
47 | |
48 void Dependencies::initialize(ciEnv* env) { | |
49 Arena* arena = env->arena(); | |
50 _oop_recorder = env->oop_recorder(); | |
51 _log = env->log(); | |
52 _dep_seen = new(arena) GrowableArray<int>(arena, 500, 0, 0); | |
53 DEBUG_ONLY(_deps[end_marker] = NULL); | |
54 for (int i = (int)FIRST_TYPE; i < (int)TYPE_LIMIT; i++) { | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
55 _deps[i] = new(arena) GrowableArray<ciBaseObject*>(arena, 10, 0, 0); |
0 | 56 } |
57 _content_bytes = NULL; | |
58 _size_in_bytes = (size_t)-1; | |
59 | |
60 assert(TYPE_LIMIT <= (1<<LG2_TYPE_LIMIT), "sanity"); | |
61 } | |
62 | |
63 void Dependencies::assert_evol_method(ciMethod* m) { | |
64 assert_common_1(evol_method, m); | |
65 } | |
66 | |
67 void Dependencies::assert_leaf_type(ciKlass* ctxk) { | |
68 if (ctxk->is_array_klass()) { | |
69 // As a special case, support this assertion on an array type, | |
70 // which reduces to an assertion on its element type. | |
71 // Note that this cannot be done with assertions that | |
72 // relate to concreteness or abstractness. | |
73 ciType* elemt = ctxk->as_array_klass()->base_element_type(); | |
74 if (!elemt->is_instance_klass()) return; // Ex: int[][] | |
75 ctxk = elemt->as_instance_klass(); | |
76 //if (ctxk->is_final()) return; // Ex: String[][] | |
77 } | |
78 check_ctxk(ctxk); | |
79 assert_common_1(leaf_type, ctxk); | |
80 } | |
81 | |
82 void Dependencies::assert_abstract_with_unique_concrete_subtype(ciKlass* ctxk, ciKlass* conck) { | |
83 check_ctxk_abstract(ctxk); | |
84 assert_common_2(abstract_with_unique_concrete_subtype, ctxk, conck); | |
85 } | |
86 | |
87 void Dependencies::assert_abstract_with_no_concrete_subtype(ciKlass* ctxk) { | |
88 check_ctxk_abstract(ctxk); | |
89 assert_common_1(abstract_with_no_concrete_subtype, ctxk); | |
90 } | |
91 | |
92 void Dependencies::assert_concrete_with_no_concrete_subtype(ciKlass* ctxk) { | |
93 check_ctxk_concrete(ctxk); | |
94 assert_common_1(concrete_with_no_concrete_subtype, ctxk); | |
95 } | |
96 | |
97 void Dependencies::assert_unique_concrete_method(ciKlass* ctxk, ciMethod* uniqm) { | |
98 check_ctxk(ctxk); | |
99 assert_common_2(unique_concrete_method, ctxk, uniqm); | |
100 } | |
101 | |
102 void Dependencies::assert_abstract_with_exclusive_concrete_subtypes(ciKlass* ctxk, ciKlass* k1, ciKlass* k2) { | |
103 check_ctxk(ctxk); | |
104 assert_common_3(abstract_with_exclusive_concrete_subtypes_2, ctxk, k1, k2); | |
105 } | |
106 | |
107 void Dependencies::assert_exclusive_concrete_methods(ciKlass* ctxk, ciMethod* m1, ciMethod* m2) { | |
108 check_ctxk(ctxk); | |
109 assert_common_3(exclusive_concrete_methods_2, ctxk, m1, m2); | |
110 } | |
111 | |
112 void Dependencies::assert_has_no_finalizable_subclasses(ciKlass* ctxk) { | |
113 check_ctxk(ctxk); | |
114 assert_common_1(no_finalizable_subclasses, ctxk); | |
115 } | |
116 | |
3894
b27c72d69fd1
7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents:
3852
diff
changeset
|
117 void Dependencies::assert_call_site_target_value(ciCallSite* call_site, ciMethodHandle* method_handle) { |
b27c72d69fd1
7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents:
3852
diff
changeset
|
118 check_ctxk(call_site->klass()); |
b27c72d69fd1
7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents:
3852
diff
changeset
|
119 assert_common_2(call_site_target_value, call_site, method_handle); |
3852
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
120 } |
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
121 |
0 | 122 // Helper function. If we are adding a new dep. under ctxk2, |
123 // try to find an old dep. under a broader* ctxk1. If there is | |
124 // | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
125 bool Dependencies::maybe_merge_ctxk(GrowableArray<ciBaseObject*>* deps, |
0 | 126 int ctxk_i, ciKlass* ctxk2) { |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
127 ciKlass* ctxk1 = deps->at(ctxk_i)->as_metadata()->as_klass(); |
0 | 128 if (ctxk2->is_subtype_of(ctxk1)) { |
129 return true; // success, and no need to change | |
130 } else if (ctxk1->is_subtype_of(ctxk2)) { | |
131 // new context class fully subsumes previous one | |
132 deps->at_put(ctxk_i, ctxk2); | |
133 return true; | |
134 } else { | |
135 return false; | |
136 } | |
137 } | |
138 | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
139 void Dependencies::assert_common_1(DepType dept, ciBaseObject* x) { |
0 | 140 assert(dep_args(dept) == 1, "sanity"); |
141 log_dependency(dept, x); | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
142 GrowableArray<ciBaseObject*>* deps = _deps[dept]; |
0 | 143 |
144 // see if the same (or a similar) dep is already recorded | |
145 if (note_dep_seen(dept, x)) { | |
146 assert(deps->find(x) >= 0, "sanity"); | |
147 } else { | |
148 deps->append(x); | |
149 } | |
150 } | |
151 | |
3894
b27c72d69fd1
7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents:
3852
diff
changeset
|
152 void Dependencies::assert_common_2(DepType dept, |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
153 ciBaseObject* x0, ciBaseObject* x1) { |
0 | 154 assert(dep_args(dept) == 2, "sanity"); |
3894
b27c72d69fd1
7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents:
3852
diff
changeset
|
155 log_dependency(dept, x0, x1); |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
156 GrowableArray<ciBaseObject*>* deps = _deps[dept]; |
0 | 157 |
158 // see if the same (or a similar) dep is already recorded | |
3894
b27c72d69fd1
7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents:
3852
diff
changeset
|
159 bool has_ctxk = has_explicit_context_arg(dept); |
b27c72d69fd1
7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents:
3852
diff
changeset
|
160 if (has_ctxk) { |
b27c72d69fd1
7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents:
3852
diff
changeset
|
161 assert(dep_context_arg(dept) == 0, "sanity"); |
b27c72d69fd1
7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents:
3852
diff
changeset
|
162 if (note_dep_seen(dept, x1)) { |
b27c72d69fd1
7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents:
3852
diff
changeset
|
163 // look in this bucket for redundant assertions |
b27c72d69fd1
7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents:
3852
diff
changeset
|
164 const int stride = 2; |
b27c72d69fd1
7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents:
3852
diff
changeset
|
165 for (int i = deps->length(); (i -= stride) >= 0; ) { |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
166 ciBaseObject* y1 = deps->at(i+1); |
3894
b27c72d69fd1
7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents:
3852
diff
changeset
|
167 if (x1 == y1) { // same subject; check the context |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
168 if (maybe_merge_ctxk(deps, i+0, x0->as_metadata()->as_klass())) { |
3894
b27c72d69fd1
7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents:
3852
diff
changeset
|
169 return; |
b27c72d69fd1
7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents:
3852
diff
changeset
|
170 } |
b27c72d69fd1
7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents:
3852
diff
changeset
|
171 } |
b27c72d69fd1
7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents:
3852
diff
changeset
|
172 } |
b27c72d69fd1
7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents:
3852
diff
changeset
|
173 } |
b27c72d69fd1
7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents:
3852
diff
changeset
|
174 } else { |
b27c72d69fd1
7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents:
3852
diff
changeset
|
175 assert(dep_implicit_context_arg(dept) == 0, "sanity"); |
b27c72d69fd1
7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents:
3852
diff
changeset
|
176 if (note_dep_seen(dept, x0) && note_dep_seen(dept, x1)) { |
b27c72d69fd1
7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents:
3852
diff
changeset
|
177 // look in this bucket for redundant assertions |
b27c72d69fd1
7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents:
3852
diff
changeset
|
178 const int stride = 2; |
b27c72d69fd1
7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents:
3852
diff
changeset
|
179 for (int i = deps->length(); (i -= stride) >= 0; ) { |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
180 ciBaseObject* y0 = deps->at(i+0); |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
181 ciBaseObject* y1 = deps->at(i+1); |
3894
b27c72d69fd1
7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents:
3852
diff
changeset
|
182 if (x0 == y0 && x1 == y1) { |
0 | 183 return; |
184 } | |
185 } | |
186 } | |
187 } | |
188 | |
189 // append the assertion in the correct bucket: | |
3894
b27c72d69fd1
7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents:
3852
diff
changeset
|
190 deps->append(x0); |
b27c72d69fd1
7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents:
3852
diff
changeset
|
191 deps->append(x1); |
0 | 192 } |
193 | |
3894
b27c72d69fd1
7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents:
3852
diff
changeset
|
194 void Dependencies::assert_common_3(DepType dept, |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
195 ciKlass* ctxk, ciBaseObject* x, ciBaseObject* x2) { |
0 | 196 assert(dep_context_arg(dept) == 0, "sanity"); |
197 assert(dep_args(dept) == 3, "sanity"); | |
198 log_dependency(dept, ctxk, x, x2); | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
199 GrowableArray<ciBaseObject*>* deps = _deps[dept]; |
0 | 200 |
201 // try to normalize an unordered pair: | |
202 bool swap = false; | |
203 switch (dept) { | |
204 case abstract_with_exclusive_concrete_subtypes_2: | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
205 swap = (x->ident() > x2->ident() && x->as_metadata()->as_klass() != ctxk); |
0 | 206 break; |
207 case exclusive_concrete_methods_2: | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
208 swap = (x->ident() > x2->ident() && x->as_metadata()->as_method()->holder() != ctxk); |
0 | 209 break; |
210 } | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
211 if (swap) { ciBaseObject* t = x; x = x2; x2 = t; } |
0 | 212 |
213 // see if the same (or a similar) dep is already recorded | |
214 if (note_dep_seen(dept, x) && note_dep_seen(dept, x2)) { | |
215 // look in this bucket for redundant assertions | |
216 const int stride = 3; | |
217 for (int i = deps->length(); (i -= stride) >= 0; ) { | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
218 ciBaseObject* y = deps->at(i+1); |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
219 ciBaseObject* y2 = deps->at(i+2); |
0 | 220 if (x == y && x2 == y2) { // same subjects; check the context |
221 if (maybe_merge_ctxk(deps, i+0, ctxk)) { | |
222 return; | |
223 } | |
224 } | |
225 } | |
226 } | |
227 // append the assertion in the correct bucket: | |
228 deps->append(ctxk); | |
229 deps->append(x); | |
230 deps->append(x2); | |
231 } | |
232 | |
233 /// Support for encoding dependencies into an nmethod: | |
234 | |
235 void Dependencies::copy_to(nmethod* nm) { | |
236 address beg = nm->dependencies_begin(); | |
237 address end = nm->dependencies_end(); | |
238 guarantee(end - beg >= (ptrdiff_t) size_in_bytes(), "bad sizing"); | |
239 Copy::disjoint_words((HeapWord*) content_bytes(), | |
240 (HeapWord*) beg, | |
241 size_in_bytes() / sizeof(HeapWord)); | |
242 assert(size_in_bytes() % sizeof(HeapWord) == 0, "copy by words"); | |
243 } | |
244 | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
245 static int sort_dep(ciBaseObject** p1, ciBaseObject** p2, int narg) { |
0 | 246 for (int i = 0; i < narg; i++) { |
247 int diff = p1[i]->ident() - p2[i]->ident(); | |
248 if (diff != 0) return diff; | |
249 } | |
250 return 0; | |
251 } | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
252 static int sort_dep_arg_1(ciBaseObject** p1, ciBaseObject** p2) |
0 | 253 { return sort_dep(p1, p2, 1); } |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
254 static int sort_dep_arg_2(ciBaseObject** p1, ciBaseObject** p2) |
0 | 255 { return sort_dep(p1, p2, 2); } |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
256 static int sort_dep_arg_3(ciBaseObject** p1, ciBaseObject** p2) |
0 | 257 { return sort_dep(p1, p2, 3); } |
258 | |
259 void Dependencies::sort_all_deps() { | |
260 for (int deptv = (int)FIRST_TYPE; deptv < (int)TYPE_LIMIT; deptv++) { | |
261 DepType dept = (DepType)deptv; | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
262 GrowableArray<ciBaseObject*>* deps = _deps[dept]; |
0 | 263 if (deps->length() <= 1) continue; |
264 switch (dep_args(dept)) { | |
265 case 1: deps->sort(sort_dep_arg_1, 1); break; | |
266 case 2: deps->sort(sort_dep_arg_2, 2); break; | |
267 case 3: deps->sort(sort_dep_arg_3, 3); break; | |
268 default: ShouldNotReachHere(); | |
269 } | |
270 } | |
271 } | |
272 | |
273 size_t Dependencies::estimate_size_in_bytes() { | |
274 size_t est_size = 100; | |
275 for (int deptv = (int)FIRST_TYPE; deptv < (int)TYPE_LIMIT; deptv++) { | |
276 DepType dept = (DepType)deptv; | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
277 GrowableArray<ciBaseObject*>* deps = _deps[dept]; |
0 | 278 est_size += deps->length()*2; // tags and argument(s) |
279 } | |
280 return est_size; | |
281 } | |
282 | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
283 ciKlass* Dependencies::ctxk_encoded_as_null(DepType dept, ciBaseObject* x) { |
0 | 284 switch (dept) { |
285 case abstract_with_exclusive_concrete_subtypes_2: | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
286 return x->as_metadata()->as_klass(); |
0 | 287 case unique_concrete_method: |
288 case exclusive_concrete_methods_2: | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
289 return x->as_metadata()->as_method()->holder(); |
0 | 290 } |
291 return NULL; // let NULL be NULL | |
292 } | |
293 | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
294 Klass* Dependencies::ctxk_encoded_as_null(DepType dept, Metadata* x) { |
0 | 295 assert(must_be_in_vm(), "raw oops here"); |
296 switch (dept) { | |
297 case abstract_with_exclusive_concrete_subtypes_2: | |
298 assert(x->is_klass(), "sanity"); | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
299 return (Klass*) x; |
0 | 300 case unique_concrete_method: |
301 case exclusive_concrete_methods_2: | |
302 assert(x->is_method(), "sanity"); | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
303 return ((Method*)x)->method_holder(); |
0 | 304 } |
305 return NULL; // let NULL be NULL | |
306 } | |
307 | |
308 void Dependencies::encode_content_bytes() { | |
309 sort_all_deps(); | |
310 | |
311 // cast is safe, no deps can overflow INT_MAX | |
312 CompressedWriteStream bytes((int)estimate_size_in_bytes()); | |
313 | |
314 for (int deptv = (int)FIRST_TYPE; deptv < (int)TYPE_LIMIT; deptv++) { | |
315 DepType dept = (DepType)deptv; | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
316 GrowableArray<ciBaseObject*>* deps = _deps[dept]; |
0 | 317 if (deps->length() == 0) continue; |
318 int stride = dep_args(dept); | |
319 int ctxkj = dep_context_arg(dept); // -1 if no context arg | |
320 assert(stride > 0, "sanity"); | |
321 for (int i = 0; i < deps->length(); i += stride) { | |
322 jbyte code_byte = (jbyte)dept; | |
323 int skipj = -1; | |
324 if (ctxkj >= 0 && ctxkj+1 < stride) { | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
325 ciKlass* ctxk = deps->at(i+ctxkj+0)->as_metadata()->as_klass(); |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
326 ciBaseObject* x = deps->at(i+ctxkj+1); // following argument |
0 | 327 if (ctxk == ctxk_encoded_as_null(dept, x)) { |
328 skipj = ctxkj; // we win: maybe one less oop to keep track of | |
329 code_byte |= default_context_type_bit; | |
330 } | |
331 } | |
332 bytes.write_byte(code_byte); | |
333 for (int j = 0; j < stride; j++) { | |
334 if (j == skipj) continue; | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
335 ciBaseObject* v = deps->at(i+j); |
6844
9a9b6e05ffb4
8000232: NPG: SIGSEGV in Dependencies::DepStream::check_klass_dependency on solaris-x64
vlivanov
parents:
6740
diff
changeset
|
336 int idx; |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
337 if (v->is_object()) { |
6844
9a9b6e05ffb4
8000232: NPG: SIGSEGV in Dependencies::DepStream::check_klass_dependency on solaris-x64
vlivanov
parents:
6740
diff
changeset
|
338 idx = _oop_recorder->find_index(v->as_object()->constant_encoding()); |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
339 } else { |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
340 ciMetadata* meta = v->as_metadata(); |
6844
9a9b6e05ffb4
8000232: NPG: SIGSEGV in Dependencies::DepStream::check_klass_dependency on solaris-x64
vlivanov
parents:
6740
diff
changeset
|
341 idx = _oop_recorder->find_index(meta->constant_encoding()); |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
342 } |
6844
9a9b6e05ffb4
8000232: NPG: SIGSEGV in Dependencies::DepStream::check_klass_dependency on solaris-x64
vlivanov
parents:
6740
diff
changeset
|
343 bytes.write_int(idx); |
0 | 344 } |
345 } | |
346 } | |
347 | |
348 // write a sentinel byte to mark the end | |
349 bytes.write_byte(end_marker); | |
350 | |
351 // round it out to a word boundary | |
352 while (bytes.position() % sizeof(HeapWord) != 0) { | |
353 bytes.write_byte(end_marker); | |
354 } | |
355 | |
356 // check whether the dept byte encoding really works | |
357 assert((jbyte)default_context_type_bit != 0, "byte overflow"); | |
358 | |
359 _content_bytes = bytes.buffer(); | |
360 _size_in_bytes = bytes.position(); | |
361 } | |
362 | |
363 | |
364 const char* Dependencies::_dep_name[TYPE_LIMIT] = { | |
365 "end_marker", | |
366 "evol_method", | |
367 "leaf_type", | |
368 "abstract_with_unique_concrete_subtype", | |
369 "abstract_with_no_concrete_subtype", | |
370 "concrete_with_no_concrete_subtype", | |
371 "unique_concrete_method", | |
372 "abstract_with_exclusive_concrete_subtypes_2", | |
373 "exclusive_concrete_methods_2", | |
3852
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
374 "no_finalizable_subclasses", |
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
375 "call_site_target_value" |
0 | 376 }; |
377 | |
378 int Dependencies::_dep_args[TYPE_LIMIT] = { | |
379 -1,// end_marker | |
380 1, // evol_method m | |
381 1, // leaf_type ctxk | |
382 2, // abstract_with_unique_concrete_subtype ctxk, k | |
383 1, // abstract_with_no_concrete_subtype ctxk | |
384 1, // concrete_with_no_concrete_subtype ctxk | |
385 2, // unique_concrete_method ctxk, m | |
386 3, // unique_concrete_subtypes_2 ctxk, k1, k2 | |
387 3, // unique_concrete_methods_2 ctxk, m1, m2 | |
3852
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
388 1, // no_finalizable_subclasses ctxk |
3894
b27c72d69fd1
7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents:
3852
diff
changeset
|
389 2 // call_site_target_value call_site, method_handle |
0 | 390 }; |
391 | |
392 const char* Dependencies::dep_name(Dependencies::DepType dept) { | |
393 if (!dept_in_mask(dept, all_types)) return "?bad-dep?"; | |
394 return _dep_name[dept]; | |
395 } | |
396 | |
397 int Dependencies::dep_args(Dependencies::DepType dept) { | |
398 if (!dept_in_mask(dept, all_types)) return -1; | |
399 return _dep_args[dept]; | |
400 } | |
401 | |
3852
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
402 void Dependencies::check_valid_dependency_type(DepType dept) { |
3894
b27c72d69fd1
7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents:
3852
diff
changeset
|
403 guarantee(FIRST_TYPE <= dept && dept < TYPE_LIMIT, err_msg("invalid dependency type: %d", (int) dept)); |
3852
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
404 } |
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
405 |
0 | 406 // for the sake of the compiler log, print out current dependencies: |
407 void Dependencies::log_all_dependencies() { | |
408 if (log() == NULL) return; | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
409 ciBaseObject* args[max_arg_count]; |
0 | 410 for (int deptv = (int)FIRST_TYPE; deptv < (int)TYPE_LIMIT; deptv++) { |
411 DepType dept = (DepType)deptv; | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
412 GrowableArray<ciBaseObject*>* deps = _deps[dept]; |
0 | 413 if (deps->length() == 0) continue; |
414 int stride = dep_args(dept); | |
415 for (int i = 0; i < deps->length(); i += stride) { | |
416 for (int j = 0; j < stride; j++) { | |
417 // flush out the identities before printing | |
418 args[j] = deps->at(i+j); | |
419 } | |
420 write_dependency_to(log(), dept, stride, args); | |
421 } | |
422 } | |
423 } | |
424 | |
425 void Dependencies::write_dependency_to(CompileLog* log, | |
426 DepType dept, | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
427 int nargs, DepArgument args[], |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
428 Klass* witness) { |
0 | 429 if (log == NULL) { |
430 return; | |
431 } | |
432 ciEnv* env = ciEnv::current(); | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
433 ciBaseObject* ciargs[max_arg_count]; |
0 | 434 assert(nargs <= max_arg_count, "oob"); |
435 for (int j = 0; j < nargs; j++) { | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
436 if (args[j].is_oop()) { |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
437 ciargs[j] = env->get_object(args[j].oop_value()); |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
438 } else { |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
439 ciargs[j] = env->get_metadata(args[j].metadata_value()); |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
440 } |
0 | 441 } |
442 Dependencies::write_dependency_to(log, dept, nargs, ciargs, witness); | |
443 } | |
444 | |
445 void Dependencies::write_dependency_to(CompileLog* log, | |
446 DepType dept, | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
447 int nargs, ciBaseObject* args[], |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
448 Klass* witness) { |
0 | 449 if (log == NULL) return; |
450 assert(nargs <= max_arg_count, "oob"); | |
451 int argids[max_arg_count]; | |
452 int ctxkj = dep_context_arg(dept); // -1 if no context arg | |
453 int j; | |
454 for (j = 0; j < nargs; j++) { | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
455 if (args[j]->is_object()) { |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
456 argids[j] = log->identify(args[j]->as_object()); |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
457 } else { |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
458 argids[j] = log->identify(args[j]->as_metadata()); |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
459 } |
0 | 460 } |
461 if (witness != NULL) { | |
462 log->begin_elem("dependency_failed"); | |
463 } else { | |
464 log->begin_elem("dependency"); | |
465 } | |
466 log->print(" type='%s'", dep_name(dept)); | |
467 if (ctxkj >= 0) { | |
468 log->print(" ctxk='%d'", argids[ctxkj]); | |
469 } | |
470 // write remaining arguments, if any. | |
471 for (j = 0; j < nargs; j++) { | |
472 if (j == ctxkj) continue; // already logged | |
473 if (j == 1) { | |
474 log->print( " x='%d'", argids[j]); | |
475 } else { | |
476 log->print(" x%d='%d'", j, argids[j]); | |
477 } | |
478 } | |
479 if (witness != NULL) { | |
480 log->object("witness", witness); | |
481 log->stamp(); | |
482 } | |
483 log->end_elem(); | |
484 } | |
485 | |
486 void Dependencies::write_dependency_to(xmlStream* xtty, | |
487 DepType dept, | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
488 int nargs, DepArgument args[], |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
489 Klass* witness) { |
0 | 490 if (xtty == NULL) return; |
491 ttyLocker ttyl; | |
492 int ctxkj = dep_context_arg(dept); // -1 if no context arg | |
493 if (witness != NULL) { | |
494 xtty->begin_elem("dependency_failed"); | |
495 } else { | |
496 xtty->begin_elem("dependency"); | |
497 } | |
498 xtty->print(" type='%s'", dep_name(dept)); | |
499 if (ctxkj >= 0) { | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
500 xtty->object("ctxk", args[ctxkj].metadata_value()); |
0 | 501 } |
502 // write remaining arguments, if any. | |
503 for (int j = 0; j < nargs; j++) { | |
504 if (j == ctxkj) continue; // already logged | |
505 if (j == 1) { | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
506 if (args[j].is_oop()) { |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
507 xtty->object("x", args[j].oop_value()); |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
508 } else { |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
509 xtty->object("x", args[j].metadata_value()); |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
510 } |
0 | 511 } else { |
512 char xn[10]; sprintf(xn, "x%d", j); | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
513 if (args[j].is_oop()) { |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
514 xtty->object(xn, args[j].oop_value()); |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
515 } else { |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
516 xtty->object(xn, args[j].metadata_value()); |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
517 } |
0 | 518 } |
519 } | |
520 if (witness != NULL) { | |
521 xtty->object("witness", witness); | |
522 xtty->stamp(); | |
523 } | |
524 xtty->end_elem(); | |
525 } | |
526 | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
527 void Dependencies::print_dependency(DepType dept, int nargs, DepArgument args[], |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
528 Klass* witness) { |
0 | 529 ResourceMark rm; |
530 ttyLocker ttyl; // keep the following output all in one block | |
531 tty->print_cr("%s of type %s", | |
532 (witness == NULL)? "Dependency": "Failed dependency", | |
533 dep_name(dept)); | |
534 // print arguments | |
535 int ctxkj = dep_context_arg(dept); // -1 if no context arg | |
536 for (int j = 0; j < nargs; j++) { | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
537 DepArgument arg = args[j]; |
0 | 538 bool put_star = false; |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
539 if (arg.is_null()) continue; |
0 | 540 const char* what; |
541 if (j == ctxkj) { | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
542 assert(arg.is_metadata(), "must be"); |
0 | 543 what = "context"; |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
544 put_star = !Dependencies::is_concrete_klass((Klass*)arg.metadata_value()); |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
545 } else if (arg.is_method()) { |
0 | 546 what = "method "; |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
547 put_star = !Dependencies::is_concrete_method((Method*)arg.metadata_value()); |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
548 } else if (arg.is_klass()) { |
0 | 549 what = "class "; |
550 } else { | |
551 what = "object "; | |
552 } | |
553 tty->print(" %s = %s", what, (put_star? "*": "")); | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
554 if (arg.is_klass()) |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
555 tty->print("%s", Klass::cast((Klass*)arg.metadata_value())->external_name()); |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
556 else if (arg.is_method()) |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
557 ((Method*)arg.metadata_value())->print_value(); |
0 | 558 else |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
559 ShouldNotReachHere(); // Provide impl for this type. |
0 | 560 tty->cr(); |
561 } | |
562 if (witness != NULL) { | |
563 bool put_star = !Dependencies::is_concrete_klass(witness); | |
564 tty->print_cr(" witness = %s%s", | |
565 (put_star? "*": ""), | |
566 Klass::cast(witness)->external_name()); | |
567 } | |
568 } | |
569 | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
570 void Dependencies::DepStream::log_dependency(Klass* witness) { |
0 | 571 if (_deps == NULL && xtty == NULL) return; // fast cutout for runtime |
572 int nargs = argument_count(); | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
573 DepArgument args[max_arg_count]; |
0 | 574 for (int j = 0; j < nargs; j++) { |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
575 if (type() == call_site_target_value) { |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
576 args[j] = argument_oop(j); |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
577 } else { |
6844
9a9b6e05ffb4
8000232: NPG: SIGSEGV in Dependencies::DepStream::check_klass_dependency on solaris-x64
vlivanov
parents:
6740
diff
changeset
|
578 args[j] = argument(j); |
9a9b6e05ffb4
8000232: NPG: SIGSEGV in Dependencies::DepStream::check_klass_dependency on solaris-x64
vlivanov
parents:
6740
diff
changeset
|
579 } |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
580 } |
0 | 581 if (_deps != NULL && _deps->log() != NULL) { |
582 Dependencies::write_dependency_to(_deps->log(), | |
583 type(), nargs, args, witness); | |
584 } else { | |
585 Dependencies::write_dependency_to(xtty, | |
586 type(), nargs, args, witness); | |
587 } | |
588 } | |
589 | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
590 void Dependencies::DepStream::print_dependency(Klass* witness, bool verbose) { |
0 | 591 int nargs = argument_count(); |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
592 DepArgument args[max_arg_count]; |
0 | 593 for (int j = 0; j < nargs; j++) { |
594 args[j] = argument(j); | |
595 } | |
596 Dependencies::print_dependency(type(), nargs, args, witness); | |
597 if (verbose) { | |
598 if (_code != NULL) { | |
599 tty->print(" code: "); | |
600 _code->print_value_on(tty); | |
601 tty->cr(); | |
602 } | |
603 } | |
604 } | |
605 | |
606 | |
607 /// Dependency stream support (decodes dependencies from an nmethod): | |
608 | |
609 #ifdef ASSERT | |
610 void Dependencies::DepStream::initial_asserts(size_t byte_limit) { | |
611 assert(must_be_in_vm(), "raw oops here"); | |
612 _byte_limit = byte_limit; | |
613 _type = (DepType)(end_marker-1); // defeat "already at end" assert | |
614 assert((_code!=NULL) + (_deps!=NULL) == 1, "one or t'other"); | |
615 } | |
616 #endif //ASSERT | |
617 | |
618 bool Dependencies::DepStream::next() { | |
619 assert(_type != end_marker, "already at end"); | |
620 if (_bytes.position() == 0 && _code != NULL | |
621 && _code->dependencies_size() == 0) { | |
622 // Method has no dependencies at all. | |
623 return false; | |
624 } | |
625 int code_byte = (_bytes.read_byte() & 0xFF); | |
626 if (code_byte == end_marker) { | |
627 DEBUG_ONLY(_type = end_marker); | |
628 return false; | |
629 } else { | |
630 int ctxk_bit = (code_byte & Dependencies::default_context_type_bit); | |
631 code_byte -= ctxk_bit; | |
632 DepType dept = (DepType)code_byte; | |
633 _type = dept; | |
3894
b27c72d69fd1
7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents:
3852
diff
changeset
|
634 Dependencies::check_valid_dependency_type(dept); |
0 | 635 int stride = _dep_args[dept]; |
636 assert(stride == dep_args(dept), "sanity"); | |
637 int skipj = -1; | |
638 if (ctxk_bit != 0) { | |
639 skipj = 0; // currently the only context argument is at zero | |
640 assert(skipj == dep_context_arg(dept), "zero arg always ctxk"); | |
641 } | |
642 for (int j = 0; j < stride; j++) { | |
643 _xi[j] = (j == skipj)? 0: _bytes.read_int(); | |
644 } | |
645 DEBUG_ONLY(_xi[stride] = -1); // help detect overruns | |
646 return true; | |
647 } | |
648 } | |
649 | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
650 inline Metadata* Dependencies::DepStream::recorded_metadata_at(int i) { |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
651 Metadata* o = NULL; |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
652 if (_code != NULL) { |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
653 o = _code->metadata_at(i); |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
654 } else { |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
655 o = _deps->oop_recorder()->metadata_at(i); |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
656 } |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
657 assert(o == NULL || o->is_metadata(), |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
658 err_msg("Should be perm " PTR_FORMAT, o)); |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
659 return o; |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
660 } |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
661 |
0 | 662 inline oop Dependencies::DepStream::recorded_oop_at(int i) { |
663 return (_code != NULL) | |
664 ? _code->oop_at(i) | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
665 : JNIHandles::resolve(_deps->oop_recorder()->oop_at(i)); |
0 | 666 } |
667 | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
668 Metadata* Dependencies::DepStream::argument(int i) { |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
669 Metadata* result = recorded_metadata_at(argument_index(i)); |
6844
9a9b6e05ffb4
8000232: NPG: SIGSEGV in Dependencies::DepStream::check_klass_dependency on solaris-x64
vlivanov
parents:
6740
diff
changeset
|
670 |
9a9b6e05ffb4
8000232: NPG: SIGSEGV in Dependencies::DepStream::check_klass_dependency on solaris-x64
vlivanov
parents:
6740
diff
changeset
|
671 if (result == NULL) { // Explicit context argument can be compressed |
9a9b6e05ffb4
8000232: NPG: SIGSEGV in Dependencies::DepStream::check_klass_dependency on solaris-x64
vlivanov
parents:
6740
diff
changeset
|
672 int ctxkj = dep_context_arg(type()); // -1 if no explicit context arg |
9a9b6e05ffb4
8000232: NPG: SIGSEGV in Dependencies::DepStream::check_klass_dependency on solaris-x64
vlivanov
parents:
6740
diff
changeset
|
673 if (ctxkj >= 0 && i == ctxkj && ctxkj+1 < argument_count()) { |
9a9b6e05ffb4
8000232: NPG: SIGSEGV in Dependencies::DepStream::check_klass_dependency on solaris-x64
vlivanov
parents:
6740
diff
changeset
|
674 result = ctxk_encoded_as_null(type(), argument(ctxkj+1)); |
9a9b6e05ffb4
8000232: NPG: SIGSEGV in Dependencies::DepStream::check_klass_dependency on solaris-x64
vlivanov
parents:
6740
diff
changeset
|
675 } |
9a9b6e05ffb4
8000232: NPG: SIGSEGV in Dependencies::DepStream::check_klass_dependency on solaris-x64
vlivanov
parents:
6740
diff
changeset
|
676 } |
9a9b6e05ffb4
8000232: NPG: SIGSEGV in Dependencies::DepStream::check_klass_dependency on solaris-x64
vlivanov
parents:
6740
diff
changeset
|
677 |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
678 assert(result == NULL || result->is_klass() || result->is_method(), "must be"); |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
679 return result; |
0 | 680 } |
681 | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
682 oop Dependencies::DepStream::argument_oop(int i) { |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
683 oop result = recorded_oop_at(argument_index(i)); |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
684 assert(result == NULL || result->is_oop(), "must be"); |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
685 return result; |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
686 } |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
687 |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
688 Klass* Dependencies::DepStream::context_type() { |
0 | 689 assert(must_be_in_vm(), "raw oops here"); |
3894
b27c72d69fd1
7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents:
3852
diff
changeset
|
690 |
b27c72d69fd1
7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents:
3852
diff
changeset
|
691 // Most dependencies have an explicit context type argument. |
b27c72d69fd1
7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents:
3852
diff
changeset
|
692 { |
6844
9a9b6e05ffb4
8000232: NPG: SIGSEGV in Dependencies::DepStream::check_klass_dependency on solaris-x64
vlivanov
parents:
6740
diff
changeset
|
693 int ctxkj = dep_context_arg(type()); // -1 if no explicit context arg |
3894
b27c72d69fd1
7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents:
3852
diff
changeset
|
694 if (ctxkj >= 0) { |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
695 Metadata* k = argument(ctxkj); |
6844
9a9b6e05ffb4
8000232: NPG: SIGSEGV in Dependencies::DepStream::check_klass_dependency on solaris-x64
vlivanov
parents:
6740
diff
changeset
|
696 assert(k != NULL && k->is_klass(), "type check"); |
9a9b6e05ffb4
8000232: NPG: SIGSEGV in Dependencies::DepStream::check_klass_dependency on solaris-x64
vlivanov
parents:
6740
diff
changeset
|
697 return (Klass*)k; |
3894
b27c72d69fd1
7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents:
3852
diff
changeset
|
698 } |
b27c72d69fd1
7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents:
3852
diff
changeset
|
699 } |
b27c72d69fd1
7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents:
3852
diff
changeset
|
700 |
b27c72d69fd1
7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents:
3852
diff
changeset
|
701 // Some dependencies are using the klass of the first object |
b27c72d69fd1
7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents:
3852
diff
changeset
|
702 // argument as implicit context type (e.g. call_site_target_value). |
b27c72d69fd1
7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents:
3852
diff
changeset
|
703 { |
6844
9a9b6e05ffb4
8000232: NPG: SIGSEGV in Dependencies::DepStream::check_klass_dependency on solaris-x64
vlivanov
parents:
6740
diff
changeset
|
704 int ctxkj = dep_implicit_context_arg(type()); |
3894
b27c72d69fd1
7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents:
3852
diff
changeset
|
705 if (ctxkj >= 0) { |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
706 Klass* k = argument_oop(ctxkj)->klass(); |
6844
9a9b6e05ffb4
8000232: NPG: SIGSEGV in Dependencies::DepStream::check_klass_dependency on solaris-x64
vlivanov
parents:
6740
diff
changeset
|
707 assert(k != NULL && k->is_klass(), "type check"); |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
708 return (Klass*) k; |
0 | 709 } |
710 } | |
3894
b27c72d69fd1
7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents:
3852
diff
changeset
|
711 |
b27c72d69fd1
7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents:
3852
diff
changeset
|
712 // And some dependencies don't have a context type at all, |
b27c72d69fd1
7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents:
3852
diff
changeset
|
713 // e.g. evol_method. |
b27c72d69fd1
7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents:
3852
diff
changeset
|
714 return NULL; |
0 | 715 } |
716 | |
717 /// Checking dependencies: | |
718 | |
719 // This hierarchy walker inspects subtypes of a given type, | |
720 // trying to find a "bad" class which breaks a dependency. | |
721 // Such a class is called a "witness" to the broken dependency. | |
722 // While searching around, we ignore "participants", which | |
723 // are already known to the dependency. | |
724 class ClassHierarchyWalker { | |
725 public: | |
726 enum { PARTICIPANT_LIMIT = 3 }; | |
727 | |
728 private: | |
729 // optional method descriptor to check for: | |
2177
3582bf76420e
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
1972
diff
changeset
|
730 Symbol* _name; |
3582bf76420e
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
1972
diff
changeset
|
731 Symbol* _signature; |
0 | 732 |
733 // special classes which are not allowed to be witnesses: | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
734 Klass* _participants[PARTICIPANT_LIMIT+1]; |
0 | 735 int _num_participants; |
736 | |
737 // cache of method lookups | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
738 Method* _found_methods[PARTICIPANT_LIMIT+1]; |
0 | 739 |
740 // if non-zero, tells how many witnesses to convert to participants | |
741 int _record_witnesses; | |
742 | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
743 void initialize(Klass* participant) { |
0 | 744 _record_witnesses = 0; |
745 _participants[0] = participant; | |
746 _found_methods[0] = NULL; | |
747 _num_participants = 0; | |
748 if (participant != NULL) { | |
749 // Terminating NULL. | |
750 _participants[1] = NULL; | |
751 _found_methods[1] = NULL; | |
752 _num_participants = 1; | |
753 } | |
754 } | |
755 | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
756 void initialize_from_method(Method* m) { |
0 | 757 assert(m != NULL && m->is_method(), "sanity"); |
758 _name = m->name(); | |
759 _signature = m->signature(); | |
760 } | |
761 | |
762 public: | |
763 // The walker is initialized to recognize certain methods and/or types | |
764 // as friendly participants. | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
765 ClassHierarchyWalker(Klass* participant, Method* m) { |
0 | 766 initialize_from_method(m); |
767 initialize(participant); | |
768 } | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
769 ClassHierarchyWalker(Method* m) { |
0 | 770 initialize_from_method(m); |
771 initialize(NULL); | |
772 } | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
773 ClassHierarchyWalker(Klass* participant = NULL) { |
0 | 774 _name = NULL; |
775 _signature = NULL; | |
776 initialize(participant); | |
777 } | |
778 | |
779 // This is common code for two searches: One for concrete subtypes, | |
780 // the other for concrete method implementations and overrides. | |
781 bool doing_subtype_search() { | |
782 return _name == NULL; | |
783 } | |
784 | |
785 int num_participants() { return _num_participants; } | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
786 Klass* participant(int n) { |
0 | 787 assert((uint)n <= (uint)_num_participants, "oob"); |
788 return _participants[n]; | |
789 } | |
790 | |
791 // Note: If n==num_participants, returns NULL. | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
792 Method* found_method(int n) { |
0 | 793 assert((uint)n <= (uint)_num_participants, "oob"); |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
794 Method* fm = _found_methods[n]; |
0 | 795 assert(n == _num_participants || fm != NULL, "proper usage"); |
796 assert(fm == NULL || fm->method_holder() == _participants[n], "sanity"); | |
797 return fm; | |
798 } | |
799 | |
800 #ifdef ASSERT | |
801 // Assert that m is inherited into ctxk, without intervening overrides. | |
802 // (May return true even if this is not true, in corner cases where we punt.) | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
803 bool check_method_context(Klass* ctxk, Method* m) { |
0 | 804 if (m->method_holder() == ctxk) |
805 return true; // Quick win. | |
806 if (m->is_private()) | |
807 return false; // Quick lose. Should not happen. | |
808 if (!(m->is_public() || m->is_protected())) | |
809 // The override story is complex when packages get involved. | |
810 return true; // Must punt the assertion to true. | |
811 Klass* k = Klass::cast(ctxk); | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
812 Method* lm = k->lookup_method(m->name(), m->signature()); |
0 | 813 if (lm == NULL && k->oop_is_instance()) { |
814 // It might be an abstract interface method, devoid of mirandas. | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
815 lm = ((InstanceKlass*)k)->lookup_method_in_all_interfaces(m->name(), |
0 | 816 m->signature()); |
817 } | |
818 if (lm == m) | |
819 // Method m is inherited into ctxk. | |
820 return true; | |
821 if (lm != NULL) { | |
4060 | 822 if (!(lm->is_public() || lm->is_protected())) { |
0 | 823 // Method is [package-]private, so the override story is complex. |
824 return true; // Must punt the assertion to true. | |
4060 | 825 } |
826 if (lm->is_static()) { | |
827 // Static methods don't override non-static so punt | |
828 return true; | |
829 } | |
0 | 830 if ( !Dependencies::is_concrete_method(lm) |
831 && !Dependencies::is_concrete_method(m) | |
832 && Klass::cast(lm->method_holder())->is_subtype_of(m->method_holder())) | |
833 // Method m is overridden by lm, but both are non-concrete. | |
834 return true; | |
835 } | |
836 ResourceMark rm; | |
837 tty->print_cr("Dependency method not found in the associated context:"); | |
838 tty->print_cr(" context = %s", Klass::cast(ctxk)->external_name()); | |
839 tty->print( " method = "); m->print_short_name(tty); tty->cr(); | |
840 if (lm != NULL) { | |
841 tty->print( " found = "); lm->print_short_name(tty); tty->cr(); | |
842 } | |
843 return false; | |
844 } | |
845 #endif | |
846 | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
847 void add_participant(Klass* participant) { |
0 | 848 assert(_num_participants + _record_witnesses < PARTICIPANT_LIMIT, "oob"); |
849 int np = _num_participants++; | |
850 _participants[np] = participant; | |
851 _participants[np+1] = NULL; | |
852 _found_methods[np+1] = NULL; | |
853 } | |
854 | |
855 void record_witnesses(int add) { | |
856 if (add > PARTICIPANT_LIMIT) add = PARTICIPANT_LIMIT; | |
857 assert(_num_participants + add < PARTICIPANT_LIMIT, "oob"); | |
858 _record_witnesses = add; | |
859 } | |
860 | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
861 bool is_witness(Klass* k) { |
0 | 862 if (doing_subtype_search()) { |
863 return Dependencies::is_concrete_klass(k); | |
864 } else { | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
865 Method* m = InstanceKlass::cast(k)->find_method(_name, _signature); |
0 | 866 if (m == NULL || !Dependencies::is_concrete_method(m)) return false; |
867 _found_methods[_num_participants] = m; | |
868 // Note: If add_participant(k) is called, | |
869 // the method m will already be memoized for it. | |
870 return true; | |
871 } | |
872 } | |
873 | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
874 bool is_participant(Klass* k) { |
0 | 875 if (k == _participants[0]) { |
876 return true; | |
877 } else if (_num_participants <= 1) { | |
878 return false; | |
879 } else { | |
880 return in_list(k, &_participants[1]); | |
881 } | |
882 } | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
883 bool ignore_witness(Klass* witness) { |
0 | 884 if (_record_witnesses == 0) { |
885 return false; | |
886 } else { | |
887 --_record_witnesses; | |
888 add_participant(witness); | |
889 return true; | |
890 } | |
891 } | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
892 static bool in_list(Klass* x, Klass** list) { |
0 | 893 for (int i = 0; ; i++) { |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
894 Klass* y = list[i]; |
0 | 895 if (y == NULL) break; |
896 if (y == x) return true; | |
897 } | |
898 return false; // not in list | |
899 } | |
900 | |
901 private: | |
902 // the actual search method: | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
903 Klass* find_witness_anywhere(Klass* context_type, |
0 | 904 bool participants_hide_witnesses, |
905 bool top_level_call = true); | |
906 // the spot-checking version: | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
907 Klass* find_witness_in(KlassDepChange& changes, |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
908 Klass* context_type, |
0 | 909 bool participants_hide_witnesses); |
910 public: | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
911 Klass* find_witness_subtype(Klass* context_type, KlassDepChange* changes = NULL) { |
0 | 912 assert(doing_subtype_search(), "must set up a subtype search"); |
913 // When looking for unexpected concrete types, | |
914 // do not look beneath expected ones. | |
915 const bool participants_hide_witnesses = true; | |
916 // CX > CC > C' is OK, even if C' is new. | |
917 // CX > { CC, C' } is not OK if C' is new, and C' is the witness. | |
918 if (changes != NULL) { | |
919 return find_witness_in(*changes, context_type, participants_hide_witnesses); | |
920 } else { | |
921 return find_witness_anywhere(context_type, participants_hide_witnesses); | |
922 } | |
923 } | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
924 Klass* find_witness_definer(Klass* context_type, KlassDepChange* changes = NULL) { |
0 | 925 assert(!doing_subtype_search(), "must set up a method definer search"); |
926 // When looking for unexpected concrete methods, | |
927 // look beneath expected ones, to see if there are overrides. | |
928 const bool participants_hide_witnesses = true; | |
929 // CX.m > CC.m > C'.m is not OK, if C'.m is new, and C' is the witness. | |
930 if (changes != NULL) { | |
931 return find_witness_in(*changes, context_type, !participants_hide_witnesses); | |
932 } else { | |
933 return find_witness_anywhere(context_type, !participants_hide_witnesses); | |
934 } | |
935 } | |
936 }; | |
937 | |
938 #ifndef PRODUCT | |
939 static int deps_find_witness_calls = 0; | |
940 static int deps_find_witness_steps = 0; | |
941 static int deps_find_witness_recursions = 0; | |
942 static int deps_find_witness_singles = 0; | |
943 static int deps_find_witness_print = 0; // set to -1 to force a final print | |
944 static bool count_find_witness_calls() { | |
945 if (TraceDependencies || LogCompilation) { | |
946 int pcount = deps_find_witness_print + 1; | |
947 bool final_stats = (pcount == 0); | |
948 bool initial_call = (pcount == 1); | |
949 bool occasional_print = ((pcount & ((1<<10) - 1)) == 0); | |
950 if (pcount < 0) pcount = 1; // crude overflow protection | |
951 deps_find_witness_print = pcount; | |
952 if (VerifyDependencies && initial_call) { | |
953 tty->print_cr("Warning: TraceDependencies results may be inflated by VerifyDependencies"); | |
954 } | |
955 if (occasional_print || final_stats) { | |
956 // Every now and then dump a little info about dependency searching. | |
957 if (xtty != NULL) { | |
1206
87684f1a88b5
6614597: Performance variability in jvm2008 xml.validation
kvn
parents:
1123
diff
changeset
|
958 ttyLocker ttyl; |
87684f1a88b5
6614597: Performance variability in jvm2008 xml.validation
kvn
parents:
1123
diff
changeset
|
959 xtty->elem("deps_find_witness calls='%d' steps='%d' recursions='%d' singles='%d'", |
0 | 960 deps_find_witness_calls, |
961 deps_find_witness_steps, | |
962 deps_find_witness_recursions, | |
963 deps_find_witness_singles); | |
964 } | |
965 if (final_stats || (TraceDependencies && WizardMode)) { | |
1206
87684f1a88b5
6614597: Performance variability in jvm2008 xml.validation
kvn
parents:
1123
diff
changeset
|
966 ttyLocker ttyl; |
0 | 967 tty->print_cr("Dependency check (find_witness) " |
968 "calls=%d, steps=%d (avg=%.1f), recursions=%d, singles=%d", | |
969 deps_find_witness_calls, | |
970 deps_find_witness_steps, | |
971 (double)deps_find_witness_steps / deps_find_witness_calls, | |
972 deps_find_witness_recursions, | |
973 deps_find_witness_singles); | |
974 } | |
975 } | |
976 return true; | |
977 } | |
978 return false; | |
979 } | |
980 #else | |
981 #define count_find_witness_calls() (0) | |
982 #endif //PRODUCT | |
983 | |
984 | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
985 Klass* ClassHierarchyWalker::find_witness_in(KlassDepChange& changes, |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
986 Klass* context_type, |
0 | 987 bool participants_hide_witnesses) { |
988 assert(changes.involves_context(context_type), "irrelevant dependency"); | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
989 Klass* new_type = changes.new_type(); |
0 | 990 |
991 count_find_witness_calls(); | |
992 NOT_PRODUCT(deps_find_witness_singles++); | |
993 | |
994 // Current thread must be in VM (not native mode, as in CI): | |
995 assert(must_be_in_vm(), "raw oops here"); | |
996 // Must not move the class hierarchy during this check: | |
997 assert_locked_or_safepoint(Compile_lock); | |
998 | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
999 int nof_impls = InstanceKlass::cast(context_type)->nof_implementors(); |
30 | 1000 if (nof_impls > 1) { |
1001 // Avoid this case: *I.m > { A.m, C }; B.m > C | |
1002 // %%% Until this is fixed more systematically, bail out. | |
1003 // See corresponding comment in find_witness_anywhere. | |
1004 return context_type; | |
1005 } | |
1006 | |
0 | 1007 assert(!is_participant(new_type), "only old classes are participants"); |
1008 if (participants_hide_witnesses) { | |
1009 // If the new type is a subtype of a participant, we are done. | |
1010 for (int i = 0; i < num_participants(); i++) { | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
1011 Klass* part = participant(i); |
0 | 1012 if (part == NULL) continue; |
1013 assert(changes.involves_context(part) == Klass::cast(new_type)->is_subtype_of(part), | |
1014 "correct marking of participants, b/c new_type is unique"); | |
1015 if (changes.involves_context(part)) { | |
1016 // new guy is protected from this check by previous participant | |
1017 return NULL; | |
1018 } | |
1019 } | |
1020 } | |
1021 | |
1022 if (is_witness(new_type) && | |
1023 !ignore_witness(new_type)) { | |
1024 return new_type; | |
1025 } | |
1026 | |
1027 return NULL; | |
1028 } | |
1029 | |
1030 | |
1031 // Walk hierarchy under a context type, looking for unexpected types. | |
1032 // Do not report participant types, and recursively walk beneath | |
1033 // them only if participants_hide_witnesses is false. | |
1034 // If top_level_call is false, skip testing the context type, | |
1035 // because the caller has already considered it. | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
1036 Klass* ClassHierarchyWalker::find_witness_anywhere(Klass* context_type, |
0 | 1037 bool participants_hide_witnesses, |
1038 bool top_level_call) { | |
1039 // Current thread must be in VM (not native mode, as in CI): | |
1040 assert(must_be_in_vm(), "raw oops here"); | |
1041 // Must not move the class hierarchy during this check: | |
1042 assert_locked_or_safepoint(Compile_lock); | |
1043 | |
1044 bool do_counts = count_find_witness_calls(); | |
1045 | |
1046 // Check the root of the sub-hierarchy first. | |
1047 if (top_level_call) { | |
1048 if (do_counts) { | |
1049 NOT_PRODUCT(deps_find_witness_calls++); | |
1050 NOT_PRODUCT(deps_find_witness_steps++); | |
1051 } | |
1052 if (is_participant(context_type)) { | |
1053 if (participants_hide_witnesses) return NULL; | |
1054 // else fall through to search loop... | |
1055 } else if (is_witness(context_type) && !ignore_witness(context_type)) { | |
1056 // The context is an abstract class or interface, to start with. | |
1057 return context_type; | |
1058 } | |
1059 } | |
1060 | |
1061 // Now we must check each implementor and each subclass. | |
1062 // Use a short worklist to avoid blowing the stack. | |
1063 // Each worklist entry is a *chain* of subklass siblings to process. | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
1064 const int CHAINMAX = 100; // >= 1 + InstanceKlass::implementors_limit |
0 | 1065 Klass* chains[CHAINMAX]; |
1066 int chaini = 0; // index into worklist | |
1067 Klass* chain; // scratch variable | |
1068 #define ADD_SUBCLASS_CHAIN(k) { \ | |
1069 assert(chaini < CHAINMAX, "oob"); \ | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
1070 chain = InstanceKlass::cast(k)->subklass(); \ |
0 | 1071 if (chain != NULL) chains[chaini++] = chain; } |
1072 | |
1073 // Look for non-abstract subclasses. | |
1074 // (Note: Interfaces do not have subclasses.) | |
1075 ADD_SUBCLASS_CHAIN(context_type); | |
1076 | |
1077 // If it is an interface, search its direct implementors. | |
1078 // (Their subclasses are additional indirect implementors. | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
1079 // See InstanceKlass::add_implementor.) |
0 | 1080 // (Note: nof_implementors is always zero for non-interfaces.) |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
1081 int nof_impls = InstanceKlass::cast(context_type)->nof_implementors(); |
0 | 1082 if (nof_impls > 1) { |
1083 // Avoid this case: *I.m > { A.m, C }; B.m > C | |
1084 // Here, I.m has 2 concrete implementations, but m appears unique | |
1085 // as A.m, because the search misses B.m when checking C. | |
1086 // The inherited method B.m was getting missed by the walker | |
1087 // when interface 'I' was the starting point. | |
1088 // %%% Until this is fixed more systematically, bail out. | |
1089 // (Old CHA had the same limitation.) | |
1090 return context_type; | |
1091 } | |
5998
49036505ab5f
7154670: The instanceKlass _implementors[] and _nof_implementors are not needed for non-interface klass.
jiangli
parents:
4772
diff
changeset
|
1092 if (nof_impls > 0) { |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
1093 Klass* impl = InstanceKlass::cast(context_type)->implementor(); |
5998
49036505ab5f
7154670: The instanceKlass _implementors[] and _nof_implementors are not needed for non-interface klass.
jiangli
parents:
4772
diff
changeset
|
1094 assert(impl != NULL, "just checking"); |
49036505ab5f
7154670: The instanceKlass _implementors[] and _nof_implementors are not needed for non-interface klass.
jiangli
parents:
4772
diff
changeset
|
1095 // If impl is the same as the context_type, then more than one |
49036505ab5f
7154670: The instanceKlass _implementors[] and _nof_implementors are not needed for non-interface klass.
jiangli
parents:
4772
diff
changeset
|
1096 // implementor has seen. No exact info in this case. |
49036505ab5f
7154670: The instanceKlass _implementors[] and _nof_implementors are not needed for non-interface klass.
jiangli
parents:
4772
diff
changeset
|
1097 if (impl == context_type) { |
0 | 1098 return context_type; // report an inexact witness to this sad affair |
1099 } | |
1100 if (do_counts) | |
1101 { NOT_PRODUCT(deps_find_witness_steps++); } | |
1102 if (is_participant(impl)) { | |
5998
49036505ab5f
7154670: The instanceKlass _implementors[] and _nof_implementors are not needed for non-interface klass.
jiangli
parents:
4772
diff
changeset
|
1103 if (!participants_hide_witnesses) { |
49036505ab5f
7154670: The instanceKlass _implementors[] and _nof_implementors are not needed for non-interface klass.
jiangli
parents:
4772
diff
changeset
|
1104 ADD_SUBCLASS_CHAIN(impl); |
49036505ab5f
7154670: The instanceKlass _implementors[] and _nof_implementors are not needed for non-interface klass.
jiangli
parents:
4772
diff
changeset
|
1105 } |
0 | 1106 } else if (is_witness(impl) && !ignore_witness(impl)) { |
1107 return impl; | |
5998
49036505ab5f
7154670: The instanceKlass _implementors[] and _nof_implementors are not needed for non-interface klass.
jiangli
parents:
4772
diff
changeset
|
1108 } else { |
49036505ab5f
7154670: The instanceKlass _implementors[] and _nof_implementors are not needed for non-interface klass.
jiangli
parents:
4772
diff
changeset
|
1109 ADD_SUBCLASS_CHAIN(impl); |
0 | 1110 } |
1111 } | |
1112 | |
1113 // Recursively process each non-trivial sibling chain. | |
1114 while (chaini > 0) { | |
1115 Klass* chain = chains[--chaini]; | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
1116 for (Klass* sub = chain; sub != NULL; sub = sub->next_sibling()) { |
0 | 1117 if (do_counts) { NOT_PRODUCT(deps_find_witness_steps++); } |
1118 if (is_participant(sub)) { | |
1119 if (participants_hide_witnesses) continue; | |
1120 // else fall through to process this guy's subclasses | |
1121 } else if (is_witness(sub) && !ignore_witness(sub)) { | |
1122 return sub; | |
1123 } | |
1124 if (chaini < (VerifyDependencies? 2: CHAINMAX)) { | |
1125 // Fast path. (Partially disabled if VerifyDependencies.) | |
1126 ADD_SUBCLASS_CHAIN(sub); | |
1127 } else { | |
1128 // Worklist overflow. Do a recursive call. Should be rare. | |
1129 // The recursive call will have its own worklist, of course. | |
1130 // (Note that sub has already been tested, so that there is | |
1131 // no need for the recursive call to re-test. That's handy, | |
1132 // since the recursive call sees sub as the context_type.) | |
1133 if (do_counts) { NOT_PRODUCT(deps_find_witness_recursions++); } | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
1134 Klass* witness = find_witness_anywhere(sub, |
0 | 1135 participants_hide_witnesses, |
1136 /*top_level_call=*/ false); | |
1137 if (witness != NULL) return witness; | |
1138 } | |
1139 } | |
1140 } | |
1141 | |
1142 // No witness found. The dependency remains unbroken. | |
1143 return NULL; | |
1144 #undef ADD_SUBCLASS_CHAIN | |
1145 } | |
1146 | |
1147 | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
1148 bool Dependencies::is_concrete_klass(Klass* k) { |
0 | 1149 if (Klass::cast(k)->is_abstract()) return false; |
1150 // %%% We could treat classes which are concrete but | |
1151 // have not yet been instantiated as virtually abstract. | |
1152 // This would require a deoptimization barrier on first instantiation. | |
1153 //if (k->is_not_instantiated()) return false; | |
1154 return true; | |
1155 } | |
1156 | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
1157 bool Dependencies::is_concrete_method(Method* m) { |
4060 | 1158 // Statics are irrelevant to virtual call sites. |
1159 if (m->is_static()) return false; | |
1160 | |
1161 // We could also return false if m does not yet appear to be | |
1162 // executed, if the VM version supports this distinction also. | |
0 | 1163 return !m->is_abstract(); |
1164 } | |
1165 | |
1166 | |
1167 Klass* Dependencies::find_finalizable_subclass(Klass* k) { | |
1168 if (k->is_interface()) return NULL; | |
1169 if (k->has_finalizer()) return k; | |
1170 k = k->subklass(); | |
1171 while (k != NULL) { | |
1172 Klass* result = find_finalizable_subclass(k); | |
1173 if (result != NULL) return result; | |
1174 k = k->next_sibling(); | |
1175 } | |
1176 return NULL; | |
1177 } | |
1178 | |
1179 | |
1180 bool Dependencies::is_concrete_klass(ciInstanceKlass* k) { | |
1181 if (k->is_abstract()) return false; | |
4060 | 1182 // We could also return false if k does not yet appear to be |
0 | 1183 // instantiated, if the VM version supports this distinction also. |
1184 //if (k->is_not_instantiated()) return false; | |
1185 return true; | |
1186 } | |
1187 | |
1188 bool Dependencies::is_concrete_method(ciMethod* m) { | |
1189 // Statics are irrelevant to virtual call sites. | |
1190 if (m->is_static()) return false; | |
1191 | |
4060 | 1192 // We could also return false if m does not yet appear to be |
0 | 1193 // executed, if the VM version supports this distinction also. |
1194 return !m->is_abstract(); | |
1195 } | |
1196 | |
1197 | |
1198 bool Dependencies::has_finalizable_subclass(ciInstanceKlass* k) { | |
1199 return k->has_finalizable_subclass(); | |
1200 } | |
1201 | |
1202 | |
1203 // Any use of the contents (bytecodes) of a method must be | |
1204 // marked by an "evol_method" dependency, if those contents | |
1205 // can change. (Note: A method is always dependent on itself.) | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
1206 Klass* Dependencies::check_evol_method(Method* m) { |
0 | 1207 assert(must_be_in_vm(), "raw oops here"); |
1208 // Did somebody do a JVMTI RedefineClasses while our backs were turned? | |
1209 // Or is there a now a breakpoint? | |
1210 // (Assumes compiled code cannot handle bkpts; change if UseFastBreakpoints.) | |
1211 if (m->is_old() | |
1212 || m->number_of_breakpoints() > 0) { | |
1213 return m->method_holder(); | |
1214 } else { | |
1215 return NULL; | |
1216 } | |
1217 } | |
1218 | |
1219 // This is a strong assertion: It is that the given type | |
1220 // has no subtypes whatever. It is most useful for | |
1221 // optimizing checks on reflected types or on array types. | |
1222 // (Checks on types which are derived from real instances | |
1223 // can be optimized more strongly than this, because we | |
1224 // know that the checked type comes from a concrete type, | |
1225 // and therefore we can disregard abstract types.) | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
1226 Klass* Dependencies::check_leaf_type(Klass* ctxk) { |
0 | 1227 assert(must_be_in_vm(), "raw oops here"); |
1228 assert_locked_or_safepoint(Compile_lock); | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
1229 InstanceKlass* ctx = InstanceKlass::cast(ctxk); |
0 | 1230 Klass* sub = ctx->subklass(); |
1231 if (sub != NULL) { | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
1232 return sub; |
0 | 1233 } else if (ctx->nof_implementors() != 0) { |
1234 // if it is an interface, it must be unimplemented | |
1235 // (if it is not an interface, nof_implementors is always zero) | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
1236 Klass* impl = ctx->implementor(); |
5998
49036505ab5f
7154670: The instanceKlass _implementors[] and _nof_implementors are not needed for non-interface klass.
jiangli
parents:
4772
diff
changeset
|
1237 assert(impl != NULL, "must be set"); |
49036505ab5f
7154670: The instanceKlass _implementors[] and _nof_implementors are not needed for non-interface klass.
jiangli
parents:
4772
diff
changeset
|
1238 return impl; |
0 | 1239 } else { |
1240 return NULL; | |
1241 } | |
1242 } | |
1243 | |
1244 // Test the assertion that conck is the only concrete subtype* of ctxk. | |
1245 // The type conck itself is allowed to have have further concrete subtypes. | |
1246 // This allows the compiler to narrow occurrences of ctxk by conck, | |
1247 // when dealing with the types of actual instances. | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
1248 Klass* Dependencies::check_abstract_with_unique_concrete_subtype(Klass* ctxk, |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
1249 Klass* conck, |
3852
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1250 KlassDepChange* changes) { |
0 | 1251 ClassHierarchyWalker wf(conck); |
1252 return wf.find_witness_subtype(ctxk, changes); | |
1253 } | |
1254 | |
1255 // If a non-concrete class has no concrete subtypes, it is not (yet) | |
1256 // instantiatable. This can allow the compiler to make some paths go | |
1257 // dead, if they are gated by a test of the type. | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
1258 Klass* Dependencies::check_abstract_with_no_concrete_subtype(Klass* ctxk, |
3852
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1259 KlassDepChange* changes) { |
0 | 1260 // Find any concrete subtype, with no participants: |
1261 ClassHierarchyWalker wf; | |
1262 return wf.find_witness_subtype(ctxk, changes); | |
1263 } | |
1264 | |
1265 | |
1266 // If a concrete class has no concrete subtypes, it can always be | |
1267 // exactly typed. This allows the use of a cheaper type test. | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
1268 Klass* Dependencies::check_concrete_with_no_concrete_subtype(Klass* ctxk, |
3852
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1269 KlassDepChange* changes) { |
0 | 1270 // Find any concrete subtype, with only the ctxk as participant: |
1271 ClassHierarchyWalker wf(ctxk); | |
1272 return wf.find_witness_subtype(ctxk, changes); | |
1273 } | |
1274 | |
1275 | |
1276 // Find the unique concrete proper subtype of ctxk, or NULL if there | |
1277 // is more than one concrete proper subtype. If there are no concrete | |
1278 // proper subtypes, return ctxk itself, whether it is concrete or not. | |
1279 // The returned subtype is allowed to have have further concrete subtypes. | |
1280 // That is, return CC1 for CX > CC1 > CC2, but NULL for CX > { CC1, CC2 }. | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
1281 Klass* Dependencies::find_unique_concrete_subtype(Klass* ctxk) { |
0 | 1282 ClassHierarchyWalker wf(ctxk); // Ignore ctxk when walking. |
1283 wf.record_witnesses(1); // Record one other witness when walking. | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
1284 Klass* wit = wf.find_witness_subtype(ctxk); |
0 | 1285 if (wit != NULL) return NULL; // Too many witnesses. |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
1286 Klass* conck = wf.participant(0); |
0 | 1287 if (conck == NULL) { |
1288 #ifndef PRODUCT | |
1289 // Make sure the dependency mechanism will pass this discovery: | |
1290 if (VerifyDependencies) { | |
1291 // Turn off dependency tracing while actually testing deps. | |
1292 FlagSetting fs(TraceDependencies, false); | |
1293 if (!Dependencies::is_concrete_klass(ctxk)) { | |
1294 guarantee(NULL == | |
1295 (void *)check_abstract_with_no_concrete_subtype(ctxk), | |
1296 "verify dep."); | |
1297 } else { | |
1298 guarantee(NULL == | |
1299 (void *)check_concrete_with_no_concrete_subtype(ctxk), | |
1300 "verify dep."); | |
1301 } | |
1302 } | |
1303 #endif //PRODUCT | |
1304 return ctxk; // Return ctxk as a flag for "no subtypes". | |
1305 } else { | |
1306 #ifndef PRODUCT | |
1307 // Make sure the dependency mechanism will pass this discovery: | |
1308 if (VerifyDependencies) { | |
1309 // Turn off dependency tracing while actually testing deps. | |
1310 FlagSetting fs(TraceDependencies, false); | |
1311 if (!Dependencies::is_concrete_klass(ctxk)) { | |
1312 guarantee(NULL == (void *) | |
1313 check_abstract_with_unique_concrete_subtype(ctxk, conck), | |
1314 "verify dep."); | |
1315 } | |
1316 } | |
1317 #endif //PRODUCT | |
1318 return conck; | |
1319 } | |
1320 } | |
1321 | |
1322 // Test the assertion that the k[12] are the only concrete subtypes of ctxk, | |
1323 // except possibly for further subtypes of k[12] themselves. | |
1324 // The context type must be abstract. The types k1 and k2 are themselves | |
1325 // allowed to have further concrete subtypes. | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
1326 Klass* Dependencies::check_abstract_with_exclusive_concrete_subtypes( |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
1327 Klass* ctxk, |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
1328 Klass* k1, |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
1329 Klass* k2, |
3852
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1330 KlassDepChange* changes) { |
0 | 1331 ClassHierarchyWalker wf; |
1332 wf.add_participant(k1); | |
1333 wf.add_participant(k2); | |
1334 return wf.find_witness_subtype(ctxk, changes); | |
1335 } | |
1336 | |
1337 // Search ctxk for concrete implementations. If there are klen or fewer, | |
1338 // pack them into the given array and return the number. | |
1339 // Otherwise, return -1, meaning the given array would overflow. | |
1340 // (Note that a return of 0 means there are exactly no concrete subtypes.) | |
1341 // In this search, if ctxk is concrete, it will be reported alone. | |
1342 // For any type CC reported, no proper subtypes of CC will be reported. | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
1343 int Dependencies::find_exclusive_concrete_subtypes(Klass* ctxk, |
0 | 1344 int klen, |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
1345 Klass* karray[]) { |
0 | 1346 ClassHierarchyWalker wf; |
1347 wf.record_witnesses(klen); | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
1348 Klass* wit = wf.find_witness_subtype(ctxk); |
0 | 1349 if (wit != NULL) return -1; // Too many witnesses. |
1350 int num = wf.num_participants(); | |
1351 assert(num <= klen, "oob"); | |
1352 // Pack the result array with the good news. | |
1353 for (int i = 0; i < num; i++) | |
1354 karray[i] = wf.participant(i); | |
1355 #ifndef PRODUCT | |
1356 // Make sure the dependency mechanism will pass this discovery: | |
1357 if (VerifyDependencies) { | |
1358 // Turn off dependency tracing while actually testing deps. | |
1359 FlagSetting fs(TraceDependencies, false); | |
1360 switch (Dependencies::is_concrete_klass(ctxk)? -1: num) { | |
1361 case -1: // ctxk was itself concrete | |
1362 guarantee(num == 1 && karray[0] == ctxk, "verify dep."); | |
1363 break; | |
1364 case 0: | |
1365 guarantee(NULL == (void *)check_abstract_with_no_concrete_subtype(ctxk), | |
1366 "verify dep."); | |
1367 break; | |
1368 case 1: | |
1369 guarantee(NULL == (void *) | |
1370 check_abstract_with_unique_concrete_subtype(ctxk, karray[0]), | |
1371 "verify dep."); | |
1372 break; | |
1373 case 2: | |
1374 guarantee(NULL == (void *) | |
1375 check_abstract_with_exclusive_concrete_subtypes(ctxk, | |
1376 karray[0], | |
1377 karray[1]), | |
1378 "verify dep."); | |
1379 break; | |
1380 default: | |
1381 ShouldNotReachHere(); // klen > 2 yet supported | |
1382 } | |
1383 } | |
1384 #endif //PRODUCT | |
1385 return num; | |
1386 } | |
1387 | |
1388 // If a class (or interface) has a unique concrete method uniqm, return NULL. | |
1389 // Otherwise, return a class that contains an interfering method. | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
1390 Klass* Dependencies::check_unique_concrete_method(Klass* ctxk, Method* uniqm, |
3852
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1391 KlassDepChange* changes) { |
0 | 1392 // Here is a missing optimization: If uniqm->is_final(), |
1393 // we don't really need to search beneath it for overrides. | |
1394 // This is probably not important, since we don't use dependencies | |
1395 // to track final methods. (They can't be "definalized".) | |
1396 ClassHierarchyWalker wf(uniqm->method_holder(), uniqm); | |
1397 return wf.find_witness_definer(ctxk, changes); | |
1398 } | |
1399 | |
1400 // Find the set of all non-abstract methods under ctxk that match m. | |
1401 // (The method m must be defined or inherited in ctxk.) | |
1402 // Include m itself in the set, unless it is abstract. | |
1403 // If this set has exactly one element, return that element. | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
1404 Method* Dependencies::find_unique_concrete_method(Klass* ctxk, Method* m) { |
0 | 1405 ClassHierarchyWalker wf(m); |
1406 assert(wf.check_method_context(ctxk, m), "proper context"); | |
1407 wf.record_witnesses(1); | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
1408 Klass* wit = wf.find_witness_definer(ctxk); |
0 | 1409 if (wit != NULL) return NULL; // Too many witnesses. |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
1410 Method* fm = wf.found_method(0); // Will be NULL if num_parts == 0. |
0 | 1411 if (Dependencies::is_concrete_method(m)) { |
1412 if (fm == NULL) { | |
1413 // It turns out that m was always the only implementation. | |
1414 fm = m; | |
1415 } else if (fm != m) { | |
1416 // Two conflicting implementations after all. | |
1417 // (This can happen if m is inherited into ctxk and fm overrides it.) | |
1418 return NULL; | |
1419 } | |
1420 } | |
1421 #ifndef PRODUCT | |
1422 // Make sure the dependency mechanism will pass this discovery: | |
1423 if (VerifyDependencies && fm != NULL) { | |
1424 guarantee(NULL == (void *)check_unique_concrete_method(ctxk, fm), | |
1425 "verify dep."); | |
1426 } | |
1427 #endif //PRODUCT | |
1428 return fm; | |
1429 } | |
1430 | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
1431 Klass* Dependencies::check_exclusive_concrete_methods(Klass* ctxk, |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
1432 Method* m1, |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
1433 Method* m2, |
3852
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1434 KlassDepChange* changes) { |
0 | 1435 ClassHierarchyWalker wf(m1); |
1436 wf.add_participant(m1->method_holder()); | |
1437 wf.add_participant(m2->method_holder()); | |
1438 return wf.find_witness_definer(ctxk, changes); | |
1439 } | |
1440 | |
1441 // Find the set of all non-abstract methods under ctxk that match m[0]. | |
1442 // (The method m[0] must be defined or inherited in ctxk.) | |
1443 // Include m itself in the set, unless it is abstract. | |
1444 // Fill the given array m[0..(mlen-1)] with this set, and return the length. | |
1445 // (The length may be zero if no concrete methods are found anywhere.) | |
1446 // If there are too many concrete methods to fit in marray, return -1. | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
1447 int Dependencies::find_exclusive_concrete_methods(Klass* ctxk, |
0 | 1448 int mlen, |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
1449 Method* marray[]) { |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
1450 Method* m0 = marray[0]; |
0 | 1451 ClassHierarchyWalker wf(m0); |
1452 assert(wf.check_method_context(ctxk, m0), "proper context"); | |
1453 wf.record_witnesses(mlen); | |
1454 bool participants_hide_witnesses = true; | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
1455 Klass* wit = wf.find_witness_definer(ctxk); |
0 | 1456 if (wit != NULL) return -1; // Too many witnesses. |
1457 int num = wf.num_participants(); | |
1458 assert(num <= mlen, "oob"); | |
1459 // Keep track of whether m is also part of the result set. | |
1460 int mfill = 0; | |
1461 assert(marray[mfill] == m0, "sanity"); | |
1462 if (Dependencies::is_concrete_method(m0)) | |
1463 mfill++; // keep m0 as marray[0], the first result | |
1464 for (int i = 0; i < num; i++) { | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
1465 Method* fm = wf.found_method(i); |
0 | 1466 if (fm == m0) continue; // Already put this guy in the list. |
1467 if (mfill == mlen) { | |
1468 return -1; // Oops. Too many methods after all! | |
1469 } | |
1470 marray[mfill++] = fm; | |
1471 } | |
1472 #ifndef PRODUCT | |
1473 // Make sure the dependency mechanism will pass this discovery: | |
1474 if (VerifyDependencies) { | |
1475 // Turn off dependency tracing while actually testing deps. | |
1476 FlagSetting fs(TraceDependencies, false); | |
1477 switch (mfill) { | |
1478 case 1: | |
1479 guarantee(NULL == (void *)check_unique_concrete_method(ctxk, marray[0]), | |
1480 "verify dep."); | |
1481 break; | |
1482 case 2: | |
1483 guarantee(NULL == (void *) | |
1484 check_exclusive_concrete_methods(ctxk, marray[0], marray[1]), | |
1485 "verify dep."); | |
1486 break; | |
1487 default: | |
1488 ShouldNotReachHere(); // mlen > 2 yet supported | |
1489 } | |
1490 } | |
1491 #endif //PRODUCT | |
1492 return mfill; | |
1493 } | |
1494 | |
1495 | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
1496 Klass* Dependencies::check_has_no_finalizable_subclasses(Klass* ctxk, KlassDepChange* changes) { |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
1497 Klass* search_at = ctxk; |
0 | 1498 if (changes != NULL) |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
1499 search_at = changes->new_type(); // just look at the new bit |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
1500 return find_finalizable_subclass(search_at); |
0 | 1501 } |
1502 | |
1503 | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
1504 Klass* Dependencies::check_call_site_target_value(oop call_site, oop method_handle, CallSiteDepChange* changes) { |
3852
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1505 assert(call_site ->is_a(SystemDictionary::CallSite_klass()), "sanity"); |
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1506 assert(method_handle->is_a(SystemDictionary::MethodHandle_klass()), "sanity"); |
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1507 if (changes == NULL) { |
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1508 // Validate all CallSites |
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1509 if (java_lang_invoke_CallSite::target(call_site) != method_handle) |
3894
b27c72d69fd1
7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents:
3852
diff
changeset
|
1510 return call_site->klass(); // assertion failed |
3852
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1511 } else { |
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1512 // Validate the given CallSite |
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1513 if (call_site == changes->call_site() && java_lang_invoke_CallSite::target(call_site) != changes->method_handle()) { |
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1514 assert(method_handle != changes->method_handle(), "must be"); |
3894
b27c72d69fd1
7083184: JSR 292: don't store context class argument with call site dependencies
twisti
parents:
3852
diff
changeset
|
1515 return call_site->klass(); // assertion failed |
3852
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1516 } |
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1517 } |
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1518 return NULL; // assertion still valid |
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1519 } |
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1520 |
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1521 |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
1522 void Dependencies::DepStream::trace_and_log_witness(Klass* witness) { |
3852
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1523 if (witness != NULL) { |
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1524 if (TraceDependencies) { |
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1525 print_dependency(witness, /*verbose=*/ true); |
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1526 } |
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1527 // The following is a no-op unless logging is enabled: |
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1528 log_dependency(witness); |
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1529 } |
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1530 } |
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1531 |
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1532 |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
1533 Klass* Dependencies::DepStream::check_klass_dependency(KlassDepChange* changes) { |
0 | 1534 assert_locked_or_safepoint(Compile_lock); |
3852
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1535 Dependencies::check_valid_dependency_type(type()); |
0 | 1536 |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
1537 Klass* witness = NULL; |
0 | 1538 switch (type()) { |
1539 case evol_method: | |
1540 witness = check_evol_method(method_argument(0)); | |
1541 break; | |
1542 case leaf_type: | |
1543 witness = check_leaf_type(context_type()); | |
1544 break; | |
1545 case abstract_with_unique_concrete_subtype: | |
3852
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1546 witness = check_abstract_with_unique_concrete_subtype(context_type(), type_argument(1), changes); |
0 | 1547 break; |
1548 case abstract_with_no_concrete_subtype: | |
3852
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1549 witness = check_abstract_with_no_concrete_subtype(context_type(), changes); |
0 | 1550 break; |
1551 case concrete_with_no_concrete_subtype: | |
3852
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1552 witness = check_concrete_with_no_concrete_subtype(context_type(), changes); |
0 | 1553 break; |
1554 case unique_concrete_method: | |
3852
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1555 witness = check_unique_concrete_method(context_type(), method_argument(1), changes); |
0 | 1556 break; |
1557 case abstract_with_exclusive_concrete_subtypes_2: | |
3852
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1558 witness = check_abstract_with_exclusive_concrete_subtypes(context_type(), type_argument(1), type_argument(2), changes); |
0 | 1559 break; |
1560 case exclusive_concrete_methods_2: | |
3852
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1561 witness = check_exclusive_concrete_methods(context_type(), method_argument(1), method_argument(2), changes); |
0 | 1562 break; |
1563 case no_finalizable_subclasses: | |
3852
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1564 witness = check_has_no_finalizable_subclasses(context_type(), changes); |
0 | 1565 break; |
3852
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1566 default: |
0 | 1567 witness = NULL; |
1568 break; | |
1569 } | |
3852
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1570 trace_and_log_witness(witness); |
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1571 return witness; |
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1572 } |
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1573 |
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1574 |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
1575 Klass* Dependencies::DepStream::check_call_site_dependency(CallSiteDepChange* changes) { |
3852
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1576 assert_locked_or_safepoint(Compile_lock); |
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1577 Dependencies::check_valid_dependency_type(type()); |
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1578 |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
1579 Klass* witness = NULL; |
3852
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1580 switch (type()) { |
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1581 case call_site_target_value: |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
1582 witness = check_call_site_target_value(argument_oop(0), argument_oop(1), changes); |
3852
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1583 break; |
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1584 default: |
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1585 witness = NULL; |
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1586 break; |
0 | 1587 } |
3852
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1588 trace_and_log_witness(witness); |
0 | 1589 return witness; |
1590 } | |
1591 | |
1592 | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
1593 Klass* Dependencies::DepStream::spot_check_dependency_at(DepChange& changes) { |
3852
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1594 // Handle klass dependency |
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1595 if (changes.is_klass_change() && changes.as_klass_change()->involves_context(context_type())) |
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1596 return check_klass_dependency(changes.as_klass_change()); |
0 | 1597 |
3852
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1598 // Handle CallSite dependency |
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1599 if (changes.is_call_site_change()) |
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1600 return check_call_site_dependency(changes.as_call_site_change()); |
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1601 |
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1602 // irrelevant dependency; skip it |
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1603 return NULL; |
0 | 1604 } |
1605 | |
1606 | |
3852
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1607 void DepChange::print() { |
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1608 int nsup = 0, nint = 0; |
0 | 1609 for (ContextStream str(*this); str.next(); ) { |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
1610 Klass* k = str.klass(); |
3852
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1611 switch (str.change_type()) { |
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1612 case Change_new_type: |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
1613 tty->print_cr(" dependee = %s", InstanceKlass::cast(k)->external_name()); |
3852
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1614 break; |
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1615 case Change_new_sub: |
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1616 if (!WizardMode) { |
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1617 ++nsup; |
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1618 } else { |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
1619 tty->print_cr(" context super = %s", InstanceKlass::cast(k)->external_name()); |
3852
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1620 } |
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1621 break; |
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1622 case Change_new_impl: |
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1623 if (!WizardMode) { |
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1624 ++nint; |
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1625 } else { |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
1626 tty->print_cr(" context interface = %s", InstanceKlass::cast(k)->external_name()); |
3852
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1627 } |
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1628 break; |
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1629 } |
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1630 } |
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1631 if (nsup + nint != 0) { |
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1632 tty->print_cr(" context supers = %d, interfaces = %d", nsup, nint); |
0 | 1633 } |
1634 } | |
1635 | |
3852
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1636 void DepChange::ContextStream::start() { |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
1637 Klass* new_type = _changes.is_klass_change() ? _changes.as_klass_change()->new_type() : (Klass*) NULL; |
3852
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1638 _change_type = (new_type == NULL ? NO_CHANGE : Start_Klass); |
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1639 _klass = new_type; |
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1640 _ti_base = NULL; |
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1641 _ti_index = 0; |
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1642 _ti_limit = 0; |
0 | 1643 } |
1644 | |
1645 bool DepChange::ContextStream::next() { | |
1646 switch (_change_type) { | |
1647 case Start_Klass: // initial state; _klass is the new type | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
1648 _ti_base = InstanceKlass::cast(_klass)->transitive_interfaces(); |
0 | 1649 _ti_index = 0; |
1650 _change_type = Change_new_type; | |
1651 return true; | |
1652 case Change_new_type: | |
1653 // fall through: | |
1654 _change_type = Change_new_sub; | |
1655 case Change_new_sub: | |
54
d4a0f561287a
6598190: JPRT tests fail when run with -XX:+CheckUnhandledOops
sbohne
parents:
0
diff
changeset
|
1656 // 6598190: brackets workaround Sun Studio C++ compiler bug 6629277 |
d4a0f561287a
6598190: JPRT tests fail when run with -XX:+CheckUnhandledOops
sbohne
parents:
0
diff
changeset
|
1657 { |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
1658 _klass = InstanceKlass::cast(_klass)->super(); |
54
d4a0f561287a
6598190: JPRT tests fail when run with -XX:+CheckUnhandledOops
sbohne
parents:
0
diff
changeset
|
1659 if (_klass != NULL) { |
d4a0f561287a
6598190: JPRT tests fail when run with -XX:+CheckUnhandledOops
sbohne
parents:
0
diff
changeset
|
1660 return true; |
d4a0f561287a
6598190: JPRT tests fail when run with -XX:+CheckUnhandledOops
sbohne
parents:
0
diff
changeset
|
1661 } |
0 | 1662 } |
1663 // else set up _ti_limit and fall through: | |
1664 _ti_limit = (_ti_base == NULL) ? 0 : _ti_base->length(); | |
1665 _change_type = Change_new_impl; | |
1666 case Change_new_impl: | |
1667 if (_ti_index < _ti_limit) { | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
1668 _klass = _ti_base->at(_ti_index++); |
0 | 1669 return true; |
1670 } | |
1671 // fall through: | |
1672 _change_type = NO_CHANGE; // iterator is exhausted | |
1673 case NO_CHANGE: | |
1674 break; | |
1675 default: | |
1676 ShouldNotReachHere(); | |
1677 } | |
1678 return false; | |
1679 } | |
1680 | |
3852
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1681 void KlassDepChange::initialize() { |
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1682 // entire transaction must be under this lock: |
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1683 assert_lock_strong(Compile_lock); |
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1684 |
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1685 // Mark all dependee and all its superclasses |
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1686 // Mark transitive interfaces |
0 | 1687 for (ContextStream str(*this); str.next(); ) { |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
1688 Klass* d = str.klass(); |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
1689 assert(!InstanceKlass::cast(d)->is_marked_dependent(), "checking"); |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
1690 InstanceKlass::cast(d)->set_is_marked_dependent(true); |
0 | 1691 } |
3852
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1692 } |
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1693 |
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1694 KlassDepChange::~KlassDepChange() { |
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1695 // Unmark all dependee and all its superclasses |
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1696 // Unmark transitive interfaces |
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1697 for (ContextStream str(*this); str.next(); ) { |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
1698 Klass* d = str.klass(); |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
1699 InstanceKlass::cast(d)->set_is_marked_dependent(false); |
0 | 1700 } |
1701 } | |
1702 | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
1703 bool KlassDepChange::involves_context(Klass* k) { |
3852
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1704 if (k == NULL || !Klass::cast(k)->oop_is_instance()) { |
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1705 return false; |
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1706 } |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
5998
diff
changeset
|
1707 InstanceKlass* ik = InstanceKlass::cast(k); |
3852
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1708 bool is_contained = ik->is_marked_dependent(); |
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1709 assert(is_contained == Klass::cast(new_type())->is_subtype_of(k), |
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1710 "correct marking of potential context types"); |
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1711 return is_contained; |
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1712 } |
fdb992d83a87
7071653: JSR 292: call site change notification should be pushed not pulled
twisti
parents:
2426
diff
changeset
|
1713 |
0 | 1714 #ifndef PRODUCT |
1715 void Dependencies::print_statistics() { | |
1716 if (deps_find_witness_print != 0) { | |
1717 // Call one final time, to flush out the data. | |
1718 deps_find_witness_print = -1; | |
1719 count_find_witness_calls(); | |
1720 } | |
1721 } | |
1722 #endif |