Mercurial > hg > truffle
annotate src/share/vm/oops/klassVtable.cpp @ 2226:c5a923563727
6912621: iCMS: Error: assert(_markBitMap.isMarked(addr + 1),"Missing Printezis bit?")
Summary: Fix block_size_if_printezis_bits() so it does not expect the bits, only uses them when available. Fix block_size_no_stall() so it does not stall when the bits are missing such cases, letting the caller deal with zero size returns. Constant pool cache oops do not need to be unparsable or conc_unsafe after their klass pointer is installed. Some cosmetic clean-ups and some assertion checking for conc-usafety which, in the presence of class file redefinition, has no a-priori time boundedness, so all GCs must be able to safely deal with putatively conc-unsafe objects in a stop-world pause.
Reviewed-by: jmasa, johnc
author | ysr |
---|---|
date | Mon, 07 Feb 2011 22:19:57 -0800 |
parents | 3582bf76420e |
children | e5383553fd4e |
rev | line source |
---|---|
0 | 1 /* |
1972 | 2 * Copyright (c) 1997, 2010, 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:
1490
diff
changeset
|
19 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA |
c18cbe5936b8
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
1490
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:
1490
diff
changeset
|
21 * questions. |
0 | 22 * |
23 */ | |
24 | |
1972 | 25 #include "precompiled.hpp" |
26 #include "classfile/systemDictionary.hpp" | |
27 #include "classfile/vmSymbols.hpp" | |
28 #include "gc_implementation/shared/markSweep.inline.hpp" | |
29 #include "memory/gcLocker.hpp" | |
30 #include "memory/resourceArea.hpp" | |
31 #include "memory/universe.inline.hpp" | |
32 #include "oops/instanceKlass.hpp" | |
33 #include "oops/klassOop.hpp" | |
34 #include "oops/klassVtable.hpp" | |
35 #include "oops/methodOop.hpp" | |
36 #include "oops/objArrayOop.hpp" | |
37 #include "oops/oop.inline.hpp" | |
38 #include "prims/jvmtiRedefineClassesTrace.hpp" | |
39 #include "runtime/arguments.hpp" | |
40 #include "runtime/handles.inline.hpp" | |
41 #include "utilities/copy.hpp" | |
0 | 42 |
43 inline instanceKlass* klassVtable::ik() const { | |
44 Klass* k = _klass()->klass_part(); | |
45 assert(k->oop_is_instance(), "not an instanceKlass"); | |
46 return (instanceKlass*)k; | |
47 } | |
48 | |
49 | |
50 // this function computes the vtable size (including the size needed for miranda | |
51 // methods) and the number of miranda methods in this class | |
52 // Note on Miranda methods: Let's say there is a class C that implements | |
53 // interface I. Let's say there is a method m in I that neither C nor any | |
54 // of its super classes implement (i.e there is no method of any access, with | |
55 // the same name and signature as m), then m is a Miranda method which is | |
56 // entered as a public abstract method in C's vtable. From then on it should | |
57 // treated as any other public method in C for method over-ride purposes. | |
58 void klassVtable::compute_vtable_size_and_num_mirandas(int &vtable_length, | |
59 int &num_miranda_methods, | |
60 klassOop super, | |
61 objArrayOop methods, | |
62 AccessFlags class_flags, | |
652
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
63 Handle classloader, |
2177
3582bf76420e
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
2018
diff
changeset
|
64 Symbol* classname, |
652
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
65 objArrayOop local_interfaces, |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
66 TRAPS |
0 | 67 ) { |
68 | |
69 No_Safepoint_Verifier nsv; | |
70 | |
71 // set up default result values | |
72 vtable_length = 0; | |
73 num_miranda_methods = 0; | |
74 | |
75 // start off with super's vtable length | |
76 instanceKlass* sk = (instanceKlass*)super->klass_part(); | |
77 vtable_length = super == NULL ? 0 : sk->vtable_length(); | |
78 | |
79 // go thru each method in the methods table to see if it needs a new entry | |
80 int len = methods->length(); | |
81 for (int i = 0; i < len; i++) { | |
82 assert(methods->obj_at(i)->is_method(), "must be a methodOop"); | |
652
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
83 methodHandle mh(THREAD, methodOop(methods->obj_at(i))); |
0 | 84 |
652
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
85 if (needs_new_vtable_entry(mh, super, classloader, classname, class_flags, THREAD)) { |
0 | 86 vtable_length += vtableEntry::size(); // we need a new entry |
87 } | |
88 } | |
89 | |
90 // compute the number of mirandas methods that must be added to the end | |
91 num_miranda_methods = get_num_mirandas(super, methods, local_interfaces); | |
92 vtable_length += (num_miranda_methods * vtableEntry::size()); | |
93 | |
94 if (Universe::is_bootstrapping() && vtable_length == 0) { | |
95 // array classes don't have their superclass set correctly during | |
96 // bootstrapping | |
97 vtable_length = Universe::base_vtable_size(); | |
98 } | |
99 | |
100 if (super == NULL && !Universe::is_bootstrapping() && | |
101 vtable_length != Universe::base_vtable_size()) { | |
102 // Someone is attempting to redefine java.lang.Object incorrectly. The | |
103 // only way this should happen is from | |
104 // SystemDictionary::resolve_from_stream(), which will detect this later | |
105 // and throw a security exception. So don't assert here to let | |
106 // the exception occur. | |
107 vtable_length = Universe::base_vtable_size(); | |
108 } | |
109 assert(super != NULL || vtable_length == Universe::base_vtable_size(), | |
110 "bad vtable size for class Object"); | |
111 assert(vtable_length % vtableEntry::size() == 0, "bad vtable length"); | |
112 assert(vtable_length >= Universe::base_vtable_size(), "vtable too small"); | |
113 } | |
114 | |
115 int klassVtable::index_of(methodOop m, int len) const { | |
116 assert(m->vtable_index() >= 0, "do not ask this of non-vtable methods"); | |
117 return m->vtable_index(); | |
118 } | |
119 | |
120 int klassVtable::initialize_from_super(KlassHandle super) { | |
121 if (super.is_null()) { | |
122 return 0; | |
123 } else { | |
124 // copy methods from superKlass | |
125 // can't inherit from array class, so must be instanceKlass | |
126 assert(super->oop_is_instance(), "must be instance klass"); | |
127 instanceKlass* sk = (instanceKlass*)super()->klass_part(); | |
128 klassVtable* superVtable = sk->vtable(); | |
129 assert(superVtable->length() <= _length, "vtable too short"); | |
130 #ifdef ASSERT | |
131 superVtable->verify(tty, true); | |
132 #endif | |
133 superVtable->copy_vtable_to(table()); | |
134 #ifndef PRODUCT | |
135 if (PrintVtables && Verbose) { | |
652
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
136 ResourceMark rm; |
0 | 137 tty->print_cr("copy vtable from %s to %s size %d", sk->internal_name(), klass()->internal_name(), _length); |
138 } | |
139 #endif | |
140 return superVtable->length(); | |
141 } | |
142 } | |
143 | |
144 // Revised lookup semantics introduced 1.3 (Kestral beta) | |
145 void klassVtable::initialize_vtable(bool checkconstraints, TRAPS) { | |
146 | |
147 // Note: Arrays can have intermediate array supers. Use java_super to skip them. | |
148 KlassHandle super (THREAD, klass()->java_super()); | |
149 int nofNewEntries = 0; | |
150 | |
151 | |
152 if (PrintVtables && !klass()->oop_is_array()) { | |
153 ResourceMark rm(THREAD); | |
154 tty->print_cr("Initializing: %s", _klass->name()->as_C_string()); | |
155 } | |
156 | |
157 #ifdef ASSERT | |
158 oop* end_of_obj = (oop*)_klass() + _klass()->size(); | |
159 oop* end_of_vtable = (oop*)&table()[_length]; | |
160 assert(end_of_vtable <= end_of_obj, "vtable extends beyond end"); | |
161 #endif | |
162 | |
163 if (Universe::is_bootstrapping()) { | |
164 // just clear everything | |
165 for (int i = 0; i < _length; i++) table()[i].clear(); | |
166 return; | |
167 } | |
168 | |
169 int super_vtable_len = initialize_from_super(super); | |
170 if (klass()->oop_is_array()) { | |
171 assert(super_vtable_len == _length, "arrays shouldn't introduce new methods"); | |
172 } else { | |
173 assert(_klass->oop_is_instance(), "must be instanceKlass"); | |
174 | |
175 objArrayHandle methods(THREAD, ik()->methods()); | |
176 int len = methods()->length(); | |
177 int initialized = super_vtable_len; | |
178 | |
652
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
179 // update_inherited_vtable can stop for gc - ensure using handles |
0 | 180 for (int i = 0; i < len; i++) { |
181 HandleMark hm(THREAD); | |
182 assert(methods()->obj_at(i)->is_method(), "must be a methodOop"); | |
183 methodHandle mh(THREAD, (methodOop)methods()->obj_at(i)); | |
184 | |
652
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
185 bool needs_new_entry = update_inherited_vtable(ik(), mh, super_vtable_len, checkconstraints, CHECK); |
0 | 186 |
187 if (needs_new_entry) { | |
188 put_method_at(mh(), initialized); | |
189 mh()->set_vtable_index(initialized); // set primary vtable index | |
190 initialized++; | |
191 } | |
192 } | |
193 | |
194 // add miranda methods; it will also update the value of initialized | |
195 fill_in_mirandas(initialized); | |
196 | |
652
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
197 // In class hierarchies where the accessibility is not increasing (i.e., going from private -> |
0 | 198 // package_private -> publicprotected), the vtable might actually be smaller than our initial |
199 // calculation. | |
200 assert(initialized <= _length, "vtable initialization failed"); | |
201 for(;initialized < _length; initialized++) { | |
202 put_method_at(NULL, initialized); | |
203 } | |
204 NOT_PRODUCT(verify(tty, true)); | |
205 } | |
206 } | |
207 | |
652
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
208 // Called for cases where a method does not override its superclass' vtable entry |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
209 // For bytecodes not produced by javac together it is possible that a method does not override |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
210 // the superclass's method, but might indirectly override a super-super class's vtable entry |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
211 // If none found, return a null superk, else return the superk of the method this does override |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
212 instanceKlass* klassVtable::find_transitive_override(instanceKlass* initialsuper, methodHandle target_method, |
2177
3582bf76420e
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
2018
diff
changeset
|
213 int vtable_index, Handle target_loader, Symbol* target_classname, Thread * THREAD) { |
652
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
214 instanceKlass* superk = initialsuper; |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
215 while (superk != NULL && superk->super() != NULL) { |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
216 instanceKlass* supersuperklass = instanceKlass::cast(superk->super()); |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
217 klassVtable* ssVtable = supersuperklass->vtable(); |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
218 if (vtable_index < ssVtable->length()) { |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
219 methodOop super_method = ssVtable->method_at(vtable_index); |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
220 #ifndef PRODUCT |
2177
3582bf76420e
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
2018
diff
changeset
|
221 Symbol* name= target_method()->name(); |
3582bf76420e
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
2018
diff
changeset
|
222 Symbol* signature = target_method()->signature(); |
3582bf76420e
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
2018
diff
changeset
|
223 assert(super_method->name() == name && super_method->signature() == signature, "vtable entry name/sig mismatch"); |
652
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
224 #endif |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
225 if (supersuperklass->is_override(super_method, target_loader, target_classname, THREAD)) { |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
226 #ifndef PRODUCT |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
227 if (PrintVtables && Verbose) { |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
228 ResourceMark rm(THREAD); |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
229 tty->print("transitive overriding superclass %s with %s::%s index %d, original flags: ", |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
230 supersuperklass->internal_name(), |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
231 _klass->internal_name(), (target_method() != NULL) ? |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
232 target_method()->name()->as_C_string() : "<NULL>", vtable_index); |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
233 super_method->access_flags().print_on(tty); |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
234 tty->print("overriders flags: "); |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
235 target_method->access_flags().print_on(tty); |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
236 tty->cr(); |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
237 } |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
238 #endif /*PRODUCT*/ |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
239 break; // return found superk |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
240 } |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
241 } else { |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
242 // super class has no vtable entry here, stop transitive search |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
243 superk = (instanceKlass*)NULL; |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
244 break; |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
245 } |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
246 // if no override found yet, continue to search up |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
247 superk = instanceKlass::cast(superk->super()); |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
248 } |
0 | 249 |
652
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
250 return superk; |
0 | 251 } |
252 | |
253 | |
254 // Update child's copy of super vtable for overrides | |
255 // OR return true if a new vtable entry is required | |
256 // Only called for instanceKlass's, i.e. not for arrays | |
257 // If that changed, could not use _klass as handle for klass | |
652
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
258 bool klassVtable::update_inherited_vtable(instanceKlass* klass, methodHandle target_method, int super_vtable_len, |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
259 bool checkconstraints, TRAPS) { |
0 | 260 ResourceMark rm; |
261 bool allocate_new = true; | |
262 assert(klass->oop_is_instance(), "must be instanceKlass"); | |
263 | |
264 // Initialize the method's vtable index to "nonvirtual". | |
265 // If we allocate a vtable entry, we will update it to a non-negative number. | |
266 target_method()->set_vtable_index(methodOopDesc::nonvirtual_vtable_index); | |
267 | |
268 // Static and <init> methods are never in | |
269 if (target_method()->is_static() || target_method()->name() == vmSymbols::object_initializer_name()) { | |
270 return false; | |
271 } | |
272 | |
273 if (klass->is_final() || target_method()->is_final()) { | |
274 // a final method never needs a new entry; final methods can be statically | |
275 // resolved and they have to be present in the vtable only if they override | |
276 // a super's method, in which case they re-use its entry | |
277 allocate_new = false; | |
278 } | |
279 | |
280 // we need a new entry if there is no superclass | |
281 if (klass->super() == NULL) { | |
282 return allocate_new; | |
283 } | |
284 | |
285 // private methods always have a new entry in the vtable | |
652
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
286 // specification interpretation since classic has |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
287 // private methods not overriding |
0 | 288 if (target_method()->is_private()) { |
289 return allocate_new; | |
290 } | |
291 | |
292 // search through the vtable and update overridden entries | |
293 // Since check_signature_loaders acquires SystemDictionary_lock | |
652
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
294 // which can block for gc, once we are in this loop, use handles |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
295 // For classfiles built with >= jdk7, we now look for transitive overrides |
0 | 296 |
2177
3582bf76420e
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
2018
diff
changeset
|
297 Symbol* name = target_method()->name(); |
3582bf76420e
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
2018
diff
changeset
|
298 Symbol* signature = target_method()->signature(); |
652
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
299 Handle target_loader(THREAD, _klass->class_loader()); |
2177
3582bf76420e
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
2018
diff
changeset
|
300 Symbol* target_classname = _klass->name(); |
0 | 301 for(int i = 0; i < super_vtable_len; i++) { |
652
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
302 methodOop super_method = method_at(i); |
0 | 303 // Check if method name matches |
2177
3582bf76420e
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
2018
diff
changeset
|
304 if (super_method->name() == name && super_method->signature() == signature) { |
0 | 305 |
652
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
306 // get super_klass for method_holder for the found method |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
307 instanceKlass* super_klass = instanceKlass::cast(super_method->method_holder()); |
0 | 308 |
652
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
309 if ((super_klass->is_override(super_method, target_loader, target_classname, THREAD)) || |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
310 ((klass->major_version() >= VTABLE_TRANSITIVE_OVERRIDE_VERSION) |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
311 && ((super_klass = find_transitive_override(super_klass, target_method, i, target_loader, |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
312 target_classname, THREAD)) != (instanceKlass*)NULL))) { |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
313 // overriding, so no new entry |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
314 allocate_new = false; |
0 | 315 |
316 if (checkconstraints) { | |
317 // Override vtable entry if passes loader constraint check | |
318 // if loader constraint checking requested | |
319 // No need to visit his super, since he and his super | |
320 // have already made any needed loader constraints. | |
321 // Since loader constraints are transitive, it is enough | |
322 // to link to the first super, and we get all the others. | |
323 Handle super_loader(THREAD, super_klass->class_loader()); | |
324 | |
652
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
325 if (target_loader() != super_loader()) { |
0 | 326 ResourceMark rm(THREAD); |
327 char* failed_type_name = | |
652
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
328 SystemDictionary::check_signature_loaders(signature, target_loader, |
0 | 329 super_loader, true, |
330 CHECK_(false)); | |
331 if (failed_type_name != NULL) { | |
332 const char* msg = "loader constraint violation: when resolving " | |
333 "overridden method \"%s\" the class loader (instance" | |
334 " of %s) of the current class, %s, and its superclass loader " | |
335 "(instance of %s), have different Class objects for the type " | |
336 "%s used in the signature"; | |
337 char* sig = target_method()->name_and_sig_as_C_string(); | |
652
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
338 const char* loader1 = SystemDictionary::loader_name(target_loader()); |
0 | 339 char* current = _klass->name()->as_C_string(); |
340 const char* loader2 = SystemDictionary::loader_name(super_loader()); | |
341 size_t buflen = strlen(msg) + strlen(sig) + strlen(loader1) + | |
342 strlen(current) + strlen(loader2) + strlen(failed_type_name); | |
343 char* buf = NEW_RESOURCE_ARRAY_IN_THREAD(THREAD, char, buflen); | |
344 jio_snprintf(buf, buflen, msg, sig, loader1, current, loader2, | |
345 failed_type_name); | |
346 THROW_MSG_(vmSymbols::java_lang_LinkageError(), buf, false); | |
347 } | |
348 } | |
652
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
349 } |
0 | 350 |
652
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
351 put_method_at(target_method(), i); |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
352 target_method()->set_vtable_index(i); |
0 | 353 #ifndef PRODUCT |
652
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
354 if (PrintVtables && Verbose) { |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
355 tty->print("overriding with %s::%s index %d, original flags: ", |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
356 _klass->internal_name(), (target_method() != NULL) ? |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
357 target_method()->name()->as_C_string() : "<NULL>", i); |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
358 super_method->access_flags().print_on(tty); |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
359 tty->print("overriders flags: "); |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
360 target_method->access_flags().print_on(tty); |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
361 tty->cr(); |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
362 } |
0 | 363 #endif /*PRODUCT*/ |
652
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
364 } else { |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
365 // allocate_new = true; default. We might override one entry, |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
366 // but not override another. Once we override one, not need new |
0 | 367 #ifndef PRODUCT |
652
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
368 if (PrintVtables && Verbose) { |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
369 tty->print("NOT overriding with %s::%s index %d, original flags: ", |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
370 _klass->internal_name(), (target_method() != NULL) ? |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
371 target_method()->name()->as_C_string() : "<NULL>", i); |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
372 super_method->access_flags().print_on(tty); |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
373 tty->print("overriders flags: "); |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
374 target_method->access_flags().print_on(tty); |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
375 tty->cr(); |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
376 } |
0 | 377 #endif /*PRODUCT*/ |
378 } | |
379 } | |
380 } | |
381 return allocate_new; | |
382 } | |
383 | |
384 void klassVtable::put_method_at(methodOop m, int index) { | |
385 assert(m->is_oop_or_null(), "Not an oop or null"); | |
386 #ifndef PRODUCT | |
387 if (PrintVtables && Verbose) { | |
652
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
388 ResourceMark rm; |
0 | 389 tty->print_cr("adding %s::%s at index %d", _klass->internal_name(), |
390 (m != NULL) ? m->name()->as_C_string() : "<NULL>", index); | |
391 } | |
392 assert(unchecked_method_at(index)->is_oop_or_null(), "Not an oop or null"); | |
393 #endif | |
394 table()[index].set(m); | |
395 } | |
396 | |
397 // Find out if a method "m" with superclass "super", loader "classloader" and | |
398 // name "classname" needs a new vtable entry. Let P be a class package defined | |
399 // by "classloader" and "classname". | |
400 // NOTE: The logic used here is very similar to the one used for computing | |
401 // the vtables indices for a method. We cannot directly use that function because, | |
652
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
402 // we allocate the instanceKlass at load time, and that requires that the |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
403 // superclass has been loaded. |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
404 // However, the vtable entries are filled in at link time, and therefore |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
405 // the superclass' vtable may not yet have been filled in. |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
406 bool klassVtable::needs_new_vtable_entry(methodHandle target_method, |
0 | 407 klassOop super, |
652
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
408 Handle classloader, |
2177
3582bf76420e
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
2018
diff
changeset
|
409 Symbol* classname, |
652
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
410 AccessFlags class_flags, |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
411 TRAPS) { |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
412 if ((class_flags.is_final() || target_method()->is_final()) || |
0 | 413 // a final method never needs a new entry; final methods can be statically |
414 // resolved and they have to be present in the vtable only if they override | |
415 // a super's method, in which case they re-use its entry | |
652
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
416 (target_method()->is_static()) || |
0 | 417 // static methods don't need to be in vtable |
652
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
418 (target_method()->name() == vmSymbols::object_initializer_name()) |
0 | 419 // <init> is never called dynamically-bound |
420 ) { | |
421 return false; | |
422 } | |
423 | |
424 // we need a new entry if there is no superclass | |
425 if (super == NULL) { | |
426 return true; | |
427 } | |
428 | |
429 // private methods always have a new entry in the vtable | |
652
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
430 // specification interpretation since classic has |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
431 // private methods not overriding |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
432 if (target_method()->is_private()) { |
0 | 433 return true; |
434 } | |
435 | |
436 // search through the super class hierarchy to see if we need | |
437 // a new entry | |
652
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
438 ResourceMark rm; |
2177
3582bf76420e
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
2018
diff
changeset
|
439 Symbol* name = target_method()->name(); |
3582bf76420e
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
2018
diff
changeset
|
440 Symbol* signature = target_method()->signature(); |
0 | 441 klassOop k = super; |
652
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
442 methodOop super_method = NULL; |
0 | 443 instanceKlass *holder = NULL; |
652
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
444 methodOop recheck_method = NULL; |
0 | 445 while (k != NULL) { |
446 // lookup through the hierarchy for a method with matching name and sign. | |
652
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
447 super_method = instanceKlass::cast(k)->lookup_method(name, signature); |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
448 if (super_method == NULL) { |
0 | 449 break; // we still have to search for a matching miranda method |
450 } | |
451 // get the class holding the matching method | |
652
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
452 // make sure you use that class for is_override |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
453 instanceKlass* superk = instanceKlass::cast(super_method->method_holder()); |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
454 // we want only instance method matches |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
455 // pretend private methods are not in the super vtable |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
456 // since we do override around them: e.g. a.m pub/b.m private/c.m pub, |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
457 // ignore private, c.m pub does override a.m pub |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
458 // For classes that were not javac'd together, we also do transitive overriding around |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
459 // methods that have less accessibility |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
460 if ((!super_method->is_static()) && |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
461 (!super_method->is_private())) { |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
462 if (superk->is_override(super_method, classloader, classname, THREAD)) { |
0 | 463 return false; |
652
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
464 // else keep looking for transitive overrides |
0 | 465 } |
466 } | |
467 | |
652
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
468 // Start with lookup result and continue to search up |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
469 k = superk->super(); // haven't found an override match yet; continue to look |
0 | 470 } |
471 | |
472 // if the target method is public or protected it may have a matching | |
473 // miranda method in the super, whose entry it should re-use. | |
652
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
474 // Actually, to handle cases that javac would not generate, we need |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
475 // this check for all access permissions. |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
476 instanceKlass *sk = instanceKlass::cast(super); |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
477 if (sk->has_miranda_methods()) { |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
478 if (sk->lookup_method_in_all_interfaces(name, signature) != NULL) { |
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
479 return false; // found a matching miranda; we do not need a new entry |
0 | 480 } |
481 } | |
482 return true; // found no match; we need a new entry | |
483 } | |
484 | |
485 // Support for miranda methods | |
486 | |
487 // get the vtable index of a miranda method with matching "name" and "signature" | |
2177
3582bf76420e
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
2018
diff
changeset
|
488 int klassVtable::index_of_miranda(Symbol* name, Symbol* signature) { |
0 | 489 // search from the bottom, might be faster |
490 for (int i = (length() - 1); i >= 0; i--) { | |
491 methodOop m = table()[i].method(); | |
492 if (is_miranda_entry_at(i) && | |
493 m->name() == name && m->signature() == signature) { | |
494 return i; | |
495 } | |
496 } | |
497 return methodOopDesc::invalid_vtable_index; | |
498 } | |
499 | |
500 // check if an entry is miranda | |
501 bool klassVtable::is_miranda_entry_at(int i) { | |
502 methodOop m = method_at(i); | |
503 klassOop method_holder = m->method_holder(); | |
504 instanceKlass *mhk = instanceKlass::cast(method_holder); | |
505 | |
506 // miranda methods are interface methods in a class's vtable | |
507 if (mhk->is_interface()) { | |
508 assert(m->is_public() && m->is_abstract(), "should be public and abstract"); | |
509 assert(ik()->implements_interface(method_holder) , "this class should implement the interface"); | |
510 assert(is_miranda(m, ik()->methods(), ik()->super()), "should be a miranda_method"); | |
511 return true; | |
512 } | |
513 return false; | |
514 } | |
515 | |
516 // check if a method is a miranda method, given a class's methods table and it's super | |
517 // the caller must make sure that the method belongs to an interface implemented by the class | |
518 bool klassVtable::is_miranda(methodOop m, objArrayOop class_methods, klassOop super) { | |
2177
3582bf76420e
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
2018
diff
changeset
|
519 Symbol* name = m->name(); |
3582bf76420e
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
2018
diff
changeset
|
520 Symbol* signature = m->signature(); |
0 | 521 if (instanceKlass::find_method(class_methods, name, signature) == NULL) { |
2018
642e54d1850a
6989076: JVM crashes in klassItable::initialize_itable_for_interface
dsamersoff
parents:
1972
diff
changeset
|
522 // did not find it in the method table of the current class |
0 | 523 if (super == NULL) { |
524 // super doesn't exist | |
525 return true; | |
2018
642e54d1850a
6989076: JVM crashes in klassItable::initialize_itable_for_interface
dsamersoff
parents:
1972
diff
changeset
|
526 } |
642e54d1850a
6989076: JVM crashes in klassItable::initialize_itable_for_interface
dsamersoff
parents:
1972
diff
changeset
|
527 |
642e54d1850a
6989076: JVM crashes in klassItable::initialize_itable_for_interface
dsamersoff
parents:
1972
diff
changeset
|
528 methodOop mo = instanceKlass::cast(super)->lookup_method(name, signature); |
642e54d1850a
6989076: JVM crashes in klassItable::initialize_itable_for_interface
dsamersoff
parents:
1972
diff
changeset
|
529 if (mo == NULL || mo->access_flags().is_private() ) { |
642e54d1850a
6989076: JVM crashes in klassItable::initialize_itable_for_interface
dsamersoff
parents:
1972
diff
changeset
|
530 // super class hierarchy does not implement it or protection is different |
642e54d1850a
6989076: JVM crashes in klassItable::initialize_itable_for_interface
dsamersoff
parents:
1972
diff
changeset
|
531 return true; |
0 | 532 } |
533 } | |
2018
642e54d1850a
6989076: JVM crashes in klassItable::initialize_itable_for_interface
dsamersoff
parents:
1972
diff
changeset
|
534 |
0 | 535 return false; |
536 } | |
537 | |
538 void klassVtable::add_new_mirandas_to_list(GrowableArray<methodOop>* list_of_current_mirandas, | |
539 objArrayOop current_interface_methods, | |
540 objArrayOop class_methods, | |
541 klassOop super) { | |
542 // iterate thru the current interface's method to see if it a miranda | |
543 int num_methods = current_interface_methods->length(); | |
544 for (int i = 0; i < num_methods; i++) { | |
545 methodOop im = methodOop(current_interface_methods->obj_at(i)); | |
546 bool is_duplicate = false; | |
547 int num_of_current_mirandas = list_of_current_mirandas->length(); | |
548 // check for duplicate mirandas in different interfaces we implement | |
549 for (int j = 0; j < num_of_current_mirandas; j++) { | |
550 methodOop miranda = list_of_current_mirandas->at(j); | |
551 if ((im->name() == miranda->name()) && | |
552 (im->signature() == miranda->signature())) { | |
553 is_duplicate = true; | |
554 break; | |
555 } | |
556 } | |
557 | |
558 if (!is_duplicate) { // we don't want duplicate miranda entries in the vtable | |
559 if (is_miranda(im, class_methods, super)) { // is it a miranda at all? | |
560 instanceKlass *sk = instanceKlass::cast(super); | |
561 // check if it is a duplicate of a super's miranda | |
562 if (sk->lookup_method_in_all_interfaces(im->name(), im->signature()) == NULL) { | |
563 list_of_current_mirandas->append(im); | |
564 } | |
565 } | |
566 } | |
567 } | |
568 } | |
569 | |
570 void klassVtable::get_mirandas(GrowableArray<methodOop>* mirandas, | |
571 klassOop super, objArrayOop class_methods, | |
572 objArrayOop local_interfaces) { | |
573 assert((mirandas->length() == 0) , "current mirandas must be 0"); | |
574 | |
575 // iterate thru the local interfaces looking for a miranda | |
576 int num_local_ifs = local_interfaces->length(); | |
577 for (int i = 0; i < num_local_ifs; i++) { | |
578 instanceKlass *ik = instanceKlass::cast(klassOop(local_interfaces->obj_at(i))); | |
579 add_new_mirandas_to_list(mirandas, ik->methods(), class_methods, super); | |
580 // iterate thru each local's super interfaces | |
581 objArrayOop super_ifs = ik->transitive_interfaces(); | |
582 int num_super_ifs = super_ifs->length(); | |
583 for (int j = 0; j < num_super_ifs; j++) { | |
584 instanceKlass *sik = instanceKlass::cast(klassOop(super_ifs->obj_at(j))); | |
585 add_new_mirandas_to_list(mirandas, sik->methods(), class_methods, super); | |
586 } | |
587 } | |
588 } | |
589 | |
590 // get number of mirandas | |
591 int klassVtable::get_num_mirandas(klassOop super, objArrayOop class_methods, objArrayOop local_interfaces) { | |
592 ResourceMark rm; | |
593 GrowableArray<methodOop>* mirandas = new GrowableArray<methodOop>(20); | |
594 get_mirandas(mirandas, super, class_methods, local_interfaces); | |
595 return mirandas->length(); | |
596 } | |
597 | |
598 // fill in mirandas | |
599 void klassVtable::fill_in_mirandas(int& initialized) { | |
600 ResourceMark rm; | |
601 GrowableArray<methodOop>* mirandas = new GrowableArray<methodOop>(20); | |
602 instanceKlass *this_ik = ik(); | |
603 get_mirandas(mirandas, this_ik->super(), this_ik->methods(), this_ik->local_interfaces()); | |
604 int num_mirandas = mirandas->length(); | |
605 for (int i = 0; i < num_mirandas; i++) { | |
606 put_method_at(mirandas->at(i), initialized); | |
607 initialized++; | |
608 } | |
609 } | |
610 | |
611 void klassVtable::copy_vtable_to(vtableEntry* start) { | |
612 Copy::disjoint_words((HeapWord*)table(), (HeapWord*)start, _length * vtableEntry::size()); | |
613 } | |
614 | |
615 void klassVtable::adjust_method_entries(methodOop* old_methods, methodOop* new_methods, | |
616 int methods_length, bool * trace_name_printed) { | |
617 // search the vtable for uses of either obsolete or EMCP methods | |
618 for (int j = 0; j < methods_length; j++) { | |
619 methodOop old_method = old_methods[j]; | |
620 methodOop new_method = new_methods[j]; | |
621 | |
622 // In the vast majority of cases we could get the vtable index | |
623 // by using: old_method->vtable_index() | |
624 // However, there are rare cases, eg. sun.awt.X11.XDecoratedPeer.getX() | |
625 // in sun.awt.X11.XFramePeer where methods occur more than once in the | |
626 // vtable, so, alas, we must do an exhaustive search. | |
627 for (int index = 0; index < length(); index++) { | |
628 if (unchecked_method_at(index) == old_method) { | |
629 put_method_at(new_method, index); | |
630 | |
631 if (RC_TRACE_IN_RANGE(0x00100000, 0x00400000)) { | |
632 if (!(*trace_name_printed)) { | |
633 // RC_TRACE_MESG macro has an embedded ResourceMark | |
634 RC_TRACE_MESG(("adjust: name=%s", | |
635 Klass::cast(old_method->method_holder())->external_name())); | |
636 *trace_name_printed = true; | |
637 } | |
638 // RC_TRACE macro has an embedded ResourceMark | |
639 RC_TRACE(0x00100000, ("vtable method update: %s(%s)", | |
640 new_method->name()->as_C_string(), | |
641 new_method->signature()->as_C_string())); | |
642 } | |
643 } | |
644 } | |
645 } | |
646 } | |
647 | |
648 | |
649 // Garbage collection | |
650 void klassVtable::oop_follow_contents() { | |
651 int len = length(); | |
652 for (int i = 0; i < len; i++) { | |
653 MarkSweep::mark_and_push(adr_method_at(i)); | |
654 } | |
655 } | |
656 | |
657 #ifndef SERIALGC | |
658 void klassVtable::oop_follow_contents(ParCompactionManager* cm) { | |
659 int len = length(); | |
660 for (int i = 0; i < len; i++) { | |
661 PSParallelCompact::mark_and_push(cm, adr_method_at(i)); | |
662 } | |
663 } | |
664 #endif // SERIALGC | |
665 | |
666 void klassVtable::oop_adjust_pointers() { | |
667 int len = length(); | |
668 for (int i = 0; i < len; i++) { | |
669 MarkSweep::adjust_pointer(adr_method_at(i)); | |
670 } | |
671 } | |
672 | |
673 #ifndef SERIALGC | |
674 void klassVtable::oop_update_pointers(ParCompactionManager* cm) { | |
675 const int n = length(); | |
676 for (int i = 0; i < n; i++) { | |
677 PSParallelCompact::adjust_pointer(adr_method_at(i)); | |
678 } | |
679 } | |
680 | |
681 void klassVtable::oop_update_pointers(ParCompactionManager* cm, | |
682 HeapWord* beg_addr, HeapWord* end_addr) { | |
683 const int n = length(); | |
684 const int entry_size = vtableEntry::size(); | |
685 | |
686 int beg_idx = 0; | |
687 HeapWord* const method_0 = (HeapWord*)adr_method_at(0); | |
688 if (beg_addr > method_0) { | |
689 // it's safe to use cast, as we have guarantees on vtable size to be sane | |
690 beg_idx = int((pointer_delta(beg_addr, method_0) + entry_size - 1) / entry_size); | |
691 } | |
692 | |
693 oop* const beg_oop = adr_method_at(beg_idx); | |
694 oop* const end_oop = MIN2((oop*)end_addr, adr_method_at(n)); | |
695 for (oop* cur_oop = beg_oop; cur_oop < end_oop; cur_oop += entry_size) { | |
696 PSParallelCompact::adjust_pointer(cur_oop); | |
697 } | |
698 } | |
699 #endif // SERIALGC | |
700 | |
701 // Iterators | |
702 void klassVtable::oop_oop_iterate(OopClosure* blk) { | |
703 int len = length(); | |
704 for (int i = 0; i < len; i++) { | |
705 blk->do_oop(adr_method_at(i)); | |
706 } | |
707 } | |
708 | |
709 void klassVtable::oop_oop_iterate_m(OopClosure* blk, MemRegion mr) { | |
710 int len = length(); | |
711 int i; | |
712 for (i = 0; i < len; i++) { | |
713 if ((HeapWord*)adr_method_at(i) >= mr.start()) break; | |
714 } | |
715 for (; i < len; i++) { | |
716 oop* adr = adr_method_at(i); | |
717 if ((HeapWord*)adr < mr.end()) blk->do_oop(adr); | |
718 } | |
719 } | |
720 | |
721 //----------------------------------------------------------------------------------------- | |
722 // Itable code | |
723 | |
724 // Initialize a itableMethodEntry | |
725 void itableMethodEntry::initialize(methodOop m) { | |
726 if (m == NULL) return; | |
727 | |
728 _method = m; | |
729 } | |
730 | |
731 klassItable::klassItable(instanceKlassHandle klass) { | |
732 _klass = klass; | |
733 | |
734 if (klass->itable_length() > 0) { | |
735 itableOffsetEntry* offset_entry = (itableOffsetEntry*)klass->start_of_itable(); | |
736 if (offset_entry != NULL && offset_entry->interface_klass() != NULL) { // Check that itable is initialized | |
737 // First offset entry points to the first method_entry | |
738 intptr_t* method_entry = (intptr_t *)(((address)klass->as_klassOop()) + offset_entry->offset()); | |
739 intptr_t* end = klass->end_of_itable(); | |
740 | |
741 _table_offset = (intptr_t*)offset_entry - (intptr_t*)klass->as_klassOop(); | |
742 _size_offset_table = (method_entry - ((intptr_t*)offset_entry)) / itableOffsetEntry::size(); | |
743 _size_method_table = (end - method_entry) / itableMethodEntry::size(); | |
744 assert(_table_offset >= 0 && _size_offset_table >= 0 && _size_method_table >= 0, "wrong computation"); | |
745 return; | |
746 } | |
747 } | |
748 | |
16
f8236e79048a
6664627: Merge changes made only in hotspot 11 forward to jdk 7
dcubed
parents:
0
diff
changeset
|
749 // The length of the itable was either zero, or it has not yet been initialized. |
0 | 750 _table_offset = 0; |
751 _size_offset_table = 0; | |
752 _size_method_table = 0; | |
753 } | |
754 | |
755 // Garbage Collection | |
756 | |
757 void klassItable::oop_follow_contents() { | |
758 // offset table | |
759 itableOffsetEntry* ioe = offset_entry(0); | |
760 for(int i = 0; i < _size_offset_table; i++) { | |
761 MarkSweep::mark_and_push((oop*)&ioe->_interface); | |
762 ioe++; | |
763 } | |
764 | |
765 // method table | |
766 itableMethodEntry* ime = method_entry(0); | |
767 for(int j = 0; j < _size_method_table; j++) { | |
768 MarkSweep::mark_and_push((oop*)&ime->_method); | |
769 ime++; | |
770 } | |
771 } | |
772 | |
773 #ifndef SERIALGC | |
774 void klassItable::oop_follow_contents(ParCompactionManager* cm) { | |
775 // offset table | |
776 itableOffsetEntry* ioe = offset_entry(0); | |
777 for(int i = 0; i < _size_offset_table; i++) { | |
778 PSParallelCompact::mark_and_push(cm, (oop*)&ioe->_interface); | |
779 ioe++; | |
780 } | |
781 | |
782 // method table | |
783 itableMethodEntry* ime = method_entry(0); | |
784 for(int j = 0; j < _size_method_table; j++) { | |
785 PSParallelCompact::mark_and_push(cm, (oop*)&ime->_method); | |
786 ime++; | |
787 } | |
788 } | |
789 #endif // SERIALGC | |
790 | |
791 void klassItable::oop_adjust_pointers() { | |
792 // offset table | |
793 itableOffsetEntry* ioe = offset_entry(0); | |
794 for(int i = 0; i < _size_offset_table; i++) { | |
795 MarkSweep::adjust_pointer((oop*)&ioe->_interface); | |
796 ioe++; | |
797 } | |
798 | |
799 // method table | |
800 itableMethodEntry* ime = method_entry(0); | |
801 for(int j = 0; j < _size_method_table; j++) { | |
802 MarkSweep::adjust_pointer((oop*)&ime->_method); | |
803 ime++; | |
804 } | |
805 } | |
806 | |
807 #ifndef SERIALGC | |
808 void klassItable::oop_update_pointers(ParCompactionManager* cm) { | |
809 // offset table | |
810 itableOffsetEntry* ioe = offset_entry(0); | |
811 for(int i = 0; i < _size_offset_table; i++) { | |
812 PSParallelCompact::adjust_pointer((oop*)&ioe->_interface); | |
813 ioe++; | |
814 } | |
815 | |
816 // method table | |
817 itableMethodEntry* ime = method_entry(0); | |
818 for(int j = 0; j < _size_method_table; j++) { | |
819 PSParallelCompact::adjust_pointer((oop*)&ime->_method); | |
820 ime++; | |
821 } | |
822 } | |
823 | |
824 void klassItable::oop_update_pointers(ParCompactionManager* cm, | |
825 HeapWord* beg_addr, HeapWord* end_addr) { | |
826 // offset table | |
827 itableOffsetEntry* ioe = offset_entry(0); | |
828 for(int i = 0; i < _size_offset_table; i++) { | |
829 oop* p = (oop*)&ioe->_interface; | |
830 PSParallelCompact::adjust_pointer(p, beg_addr, end_addr); | |
831 ioe++; | |
832 } | |
833 | |
834 // method table | |
835 itableMethodEntry* ime = method_entry(0); | |
836 for(int j = 0; j < _size_method_table; j++) { | |
837 oop* p = (oop*)&ime->_method; | |
838 PSParallelCompact::adjust_pointer(p, beg_addr, end_addr); | |
839 ime++; | |
840 } | |
841 } | |
842 #endif // SERIALGC | |
843 | |
844 // Iterators | |
845 void klassItable::oop_oop_iterate(OopClosure* blk) { | |
846 // offset table | |
847 itableOffsetEntry* ioe = offset_entry(0); | |
848 for(int i = 0; i < _size_offset_table; i++) { | |
849 blk->do_oop((oop*)&ioe->_interface); | |
850 ioe++; | |
851 } | |
852 | |
853 // method table | |
854 itableMethodEntry* ime = method_entry(0); | |
855 for(int j = 0; j < _size_method_table; j++) { | |
856 blk->do_oop((oop*)&ime->_method); | |
857 ime++; | |
858 } | |
859 } | |
860 | |
861 void klassItable::oop_oop_iterate_m(OopClosure* blk, MemRegion mr) { | |
862 // offset table | |
863 itableOffsetEntry* ioe = offset_entry(0); | |
864 for(int i = 0; i < _size_offset_table; i++) { | |
865 oop* adr = (oop*)&ioe->_interface; | |
866 if (mr.contains(adr)) blk->do_oop(adr); | |
867 ioe++; | |
868 } | |
869 | |
870 // method table | |
871 itableMethodEntry* ime = method_entry(0); | |
872 for(int j = 0; j < _size_method_table; j++) { | |
873 oop* adr = (oop*)&ime->_method; | |
874 if (mr.contains(adr)) blk->do_oop(adr); | |
875 ime++; | |
876 } | |
877 } | |
878 | |
879 | |
880 static int initialize_count = 0; | |
881 | |
882 // Initialization | |
883 void klassItable::initialize_itable(bool checkconstraints, TRAPS) { | |
16
f8236e79048a
6664627: Merge changes made only in hotspot 11 forward to jdk 7
dcubed
parents:
0
diff
changeset
|
884 // Cannot be setup doing bootstrapping, interfaces don't have |
f8236e79048a
6664627: Merge changes made only in hotspot 11 forward to jdk 7
dcubed
parents:
0
diff
changeset
|
885 // itables, and klass with only ones entry have empty itables |
f8236e79048a
6664627: Merge changes made only in hotspot 11 forward to jdk 7
dcubed
parents:
0
diff
changeset
|
886 if (Universe::is_bootstrapping() || |
f8236e79048a
6664627: Merge changes made only in hotspot 11 forward to jdk 7
dcubed
parents:
0
diff
changeset
|
887 _klass->is_interface() || |
f8236e79048a
6664627: Merge changes made only in hotspot 11 forward to jdk 7
dcubed
parents:
0
diff
changeset
|
888 _klass->itable_length() == itableOffsetEntry::size()) return; |
0 | 889 |
16
f8236e79048a
6664627: Merge changes made only in hotspot 11 forward to jdk 7
dcubed
parents:
0
diff
changeset
|
890 // There's alway an extra itable entry so we can null-terminate it. |
f8236e79048a
6664627: Merge changes made only in hotspot 11 forward to jdk 7
dcubed
parents:
0
diff
changeset
|
891 guarantee(size_offset_table() >= 1, "too small"); |
f8236e79048a
6664627: Merge changes made only in hotspot 11 forward to jdk 7
dcubed
parents:
0
diff
changeset
|
892 int num_interfaces = size_offset_table() - 1; |
0 | 893 if (num_interfaces > 0) { |
16
f8236e79048a
6664627: Merge changes made only in hotspot 11 forward to jdk 7
dcubed
parents:
0
diff
changeset
|
894 if (TraceItables) tty->print_cr("%3d: Initializing itables for %s", ++initialize_count, |
f8236e79048a
6664627: Merge changes made only in hotspot 11 forward to jdk 7
dcubed
parents:
0
diff
changeset
|
895 _klass->name()->as_C_string()); |
0 | 896 |
897 | |
652
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
898 // Iterate through all interfaces |
0 | 899 int i; |
900 for(i = 0; i < num_interfaces; i++) { | |
901 itableOffsetEntry* ioe = offset_entry(i); | |
902 KlassHandle interf_h (THREAD, ioe->interface_klass()); | |
903 assert(interf_h() != NULL && ioe->offset() != 0, "bad offset entry in itable"); | |
904 initialize_itable_for_interface(ioe->offset(), interf_h, checkconstraints, CHECK); | |
905 } | |
906 | |
907 } | |
16
f8236e79048a
6664627: Merge changes made only in hotspot 11 forward to jdk 7
dcubed
parents:
0
diff
changeset
|
908 // Check that the last entry is empty |
f8236e79048a
6664627: Merge changes made only in hotspot 11 forward to jdk 7
dcubed
parents:
0
diff
changeset
|
909 itableOffsetEntry* ioe = offset_entry(size_offset_table() - 1); |
f8236e79048a
6664627: Merge changes made only in hotspot 11 forward to jdk 7
dcubed
parents:
0
diff
changeset
|
910 guarantee(ioe->interface_klass() == NULL && ioe->offset() == 0, "terminator entry missing"); |
0 | 911 } |
912 | |
913 | |
914 void klassItable::initialize_itable_for_interface(int method_table_offset, KlassHandle interf_h, bool checkconstraints, TRAPS) { | |
915 objArrayHandle methods(THREAD, instanceKlass::cast(interf_h())->methods()); | |
916 int nof_methods = methods()->length(); | |
917 HandleMark hm; | |
918 KlassHandle klass = _klass; | |
1489
cff162798819
6888953: some calls to function-like macros are missing semicolons
jcoomes
parents:
676
diff
changeset
|
919 assert(nof_methods > 0, "at least one method must exist for interface to be in vtable"); |
0 | 920 Handle interface_loader (THREAD, instanceKlass::cast(interf_h())->class_loader()); |
921 int ime_num = 0; | |
922 | |
923 // Skip first methodOop if it is a class initializer | |
924 int i = ((methodOop)methods()->obj_at(0))->name() != vmSymbols::class_initializer_name() ? 0 : 1; | |
925 | |
926 // m, method_name, method_signature, klass reset each loop so they | |
927 // don't need preserving across check_signature_loaders call | |
928 // methods needs a handle in case of gc from check_signature_loaders | |
929 for(; i < nof_methods; i++) { | |
930 methodOop m = (methodOop)methods()->obj_at(i); | |
2177
3582bf76420e
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
2018
diff
changeset
|
931 Symbol* method_name = m->name(); |
3582bf76420e
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
2018
diff
changeset
|
932 Symbol* method_signature = m->signature(); |
0 | 933 |
934 // This is same code as in Linkresolver::lookup_instance_method_in_klasses | |
935 methodOop target = klass->uncached_lookup_method(method_name, method_signature); | |
936 while (target != NULL && target->is_static()) { | |
937 // continue with recursive lookup through the superclass | |
938 klassOop super = Klass::cast(target->method_holder())->super(); | |
939 target = (super == NULL) ? methodOop(NULL) : Klass::cast(super)->uncached_lookup_method(method_name, method_signature); | |
940 } | |
941 if (target == NULL || !target->is_public() || target->is_abstract()) { | |
942 // Entry do not resolve. Leave it empty | |
943 } else { | |
944 // Entry did resolve, check loader constraints before initializing | |
945 // if checkconstraints requested | |
946 methodHandle target_h (THREAD, target); // preserve across gc | |
947 if (checkconstraints) { | |
948 Handle method_holder_loader (THREAD, instanceKlass::cast(target->method_holder())->class_loader()); | |
949 if (method_holder_loader() != interface_loader()) { | |
950 ResourceMark rm(THREAD); | |
951 char* failed_type_name = | |
952 SystemDictionary::check_signature_loaders(method_signature, | |
953 method_holder_loader, | |
954 interface_loader, | |
955 true, CHECK); | |
956 if (failed_type_name != NULL) { | |
957 const char* msg = "loader constraint violation in interface " | |
958 "itable initialization: when resolving method \"%s\" the class" | |
959 " loader (instance of %s) of the current class, %s, " | |
960 "and the class loader (instance of %s) for interface " | |
961 "%s have different Class objects for the type %s " | |
962 "used in the signature"; | |
963 char* sig = target_h()->name_and_sig_as_C_string(); | |
964 const char* loader1 = SystemDictionary::loader_name(method_holder_loader()); | |
965 char* current = klass->name()->as_C_string(); | |
966 const char* loader2 = SystemDictionary::loader_name(interface_loader()); | |
967 char* iface = instanceKlass::cast(interf_h())->name()->as_C_string(); | |
968 size_t buflen = strlen(msg) + strlen(sig) + strlen(loader1) + | |
969 strlen(current) + strlen(loader2) + strlen(iface) + | |
970 strlen(failed_type_name); | |
971 char* buf = NEW_RESOURCE_ARRAY_IN_THREAD(THREAD, char, buflen); | |
972 jio_snprintf(buf, buflen, msg, sig, loader1, current, loader2, | |
973 iface, failed_type_name); | |
974 THROW_MSG(vmSymbols::java_lang_LinkageError(), buf); | |
975 } | |
976 } | |
977 } | |
978 | |
979 // ime may have moved during GC so recalculate address | |
980 itableOffsetEntry::method_entry(_klass(), method_table_offset)[ime_num].initialize(target_h()); | |
981 } | |
982 // Progress to next entry | |
983 ime_num++; | |
984 } | |
985 } | |
986 | |
16
f8236e79048a
6664627: Merge changes made only in hotspot 11 forward to jdk 7
dcubed
parents:
0
diff
changeset
|
987 // Update entry for specific methodOop |
0 | 988 void klassItable::initialize_with_method(methodOop m) { |
989 itableMethodEntry* ime = method_entry(0); | |
990 for(int i = 0; i < _size_method_table; i++) { | |
991 if (ime->method() == m) { | |
992 ime->initialize(m); | |
993 } | |
994 ime++; | |
995 } | |
996 } | |
997 | |
998 void klassItable::adjust_method_entries(methodOop* old_methods, methodOop* new_methods, | |
999 int methods_length, bool * trace_name_printed) { | |
1000 // search the itable for uses of either obsolete or EMCP methods | |
1001 for (int j = 0; j < methods_length; j++) { | |
1002 methodOop old_method = old_methods[j]; | |
1003 methodOop new_method = new_methods[j]; | |
1004 itableMethodEntry* ime = method_entry(0); | |
1005 | |
610
70998f2e05ef
6805864: 4/3 Problem with jvmti->redefineClasses: some methods don't get redefined
dcubed
parents:
196
diff
changeset
|
1006 // The itable can describe more than one interface and the same |
70998f2e05ef
6805864: 4/3 Problem with jvmti->redefineClasses: some methods don't get redefined
dcubed
parents:
196
diff
changeset
|
1007 // method signature can be specified by more than one interface. |
70998f2e05ef
6805864: 4/3 Problem with jvmti->redefineClasses: some methods don't get redefined
dcubed
parents:
196
diff
changeset
|
1008 // This means we have to do an exhaustive search to find all the |
70998f2e05ef
6805864: 4/3 Problem with jvmti->redefineClasses: some methods don't get redefined
dcubed
parents:
196
diff
changeset
|
1009 // old_method references. |
0 | 1010 for (int i = 0; i < _size_method_table; i++) { |
1011 if (ime->method() == old_method) { | |
1012 ime->initialize(new_method); | |
1013 | |
1014 if (RC_TRACE_IN_RANGE(0x00100000, 0x00400000)) { | |
1015 if (!(*trace_name_printed)) { | |
1016 // RC_TRACE_MESG macro has an embedded ResourceMark | |
1017 RC_TRACE_MESG(("adjust: name=%s", | |
1018 Klass::cast(old_method->method_holder())->external_name())); | |
1019 *trace_name_printed = true; | |
1020 } | |
1021 // RC_TRACE macro has an embedded ResourceMark | |
1022 RC_TRACE(0x00200000, ("itable method update: %s(%s)", | |
1023 new_method->name()->as_C_string(), | |
1024 new_method->signature()->as_C_string())); | |
1025 } | |
652
4aaa9f5e02a8
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
610
diff
changeset
|
1026 break; |
0 | 1027 } |
1028 ime++; | |
1029 } | |
1030 } | |
1031 } | |
1032 | |
1033 | |
1034 // Setup | |
1035 class InterfaceVisiterClosure : public StackObj { | |
1036 public: | |
1037 virtual void doit(klassOop intf, int method_count) = 0; | |
1038 }; | |
1039 | |
1040 // Visit all interfaces with at-least one method (excluding <clinit>) | |
1041 void visit_all_interfaces(objArrayOop transitive_intf, InterfaceVisiterClosure *blk) { | |
1042 // Handle array argument | |
1043 for(int i = 0; i < transitive_intf->length(); i++) { | |
1044 klassOop intf = (klassOop)transitive_intf->obj_at(i); | |
1045 assert(Klass::cast(intf)->is_interface(), "sanity check"); | |
1046 | |
1047 // Find no. of methods excluding a <clinit> | |
1048 int method_count = instanceKlass::cast(intf)->methods()->length(); | |
1049 if (method_count > 0) { | |
1050 methodOop m = (methodOop)instanceKlass::cast(intf)->methods()->obj_at(0); | |
1051 assert(m != NULL && m->is_method(), "sanity check"); | |
1052 if (m->name() == vmSymbols::object_initializer_name()) { | |
1053 method_count--; | |
1054 } | |
1055 } | |
1056 | |
1057 // Only count interfaces with at least one method | |
1058 if (method_count > 0) { | |
1059 blk->doit(intf, method_count); | |
1060 } | |
1061 } | |
1062 } | |
1063 | |
1064 class CountInterfacesClosure : public InterfaceVisiterClosure { | |
1065 private: | |
1066 int _nof_methods; | |
1067 int _nof_interfaces; | |
1068 public: | |
1069 CountInterfacesClosure() { _nof_methods = 0; _nof_interfaces = 0; } | |
1070 | |
1071 int nof_methods() const { return _nof_methods; } | |
1072 int nof_interfaces() const { return _nof_interfaces; } | |
1073 | |
1074 void doit(klassOop intf, int method_count) { _nof_methods += method_count; _nof_interfaces++; } | |
1075 }; | |
1076 | |
1077 class SetupItableClosure : public InterfaceVisiterClosure { | |
1078 private: | |
1079 itableOffsetEntry* _offset_entry; | |
1080 itableMethodEntry* _method_entry; | |
1081 address _klass_begin; | |
1082 public: | |
1083 SetupItableClosure(address klass_begin, itableOffsetEntry* offset_entry, itableMethodEntry* method_entry) { | |
1084 _klass_begin = klass_begin; | |
1085 _offset_entry = offset_entry; | |
1086 _method_entry = method_entry; | |
1087 } | |
1088 | |
1089 itableMethodEntry* method_entry() const { return _method_entry; } | |
1090 | |
1091 void doit(klassOop intf, int method_count) { | |
1092 int offset = ((address)_method_entry) - _klass_begin; | |
1093 _offset_entry->initialize(intf, offset); | |
1094 _offset_entry++; | |
1095 _method_entry += method_count; | |
1096 } | |
1097 }; | |
1098 | |
1099 int klassItable::compute_itable_size(objArrayHandle transitive_interfaces) { | |
1100 // Count no of interfaces and total number of interface methods | |
1101 CountInterfacesClosure cic; | |
1102 visit_all_interfaces(transitive_interfaces(), &cic); | |
1103 | |
16
f8236e79048a
6664627: Merge changes made only in hotspot 11 forward to jdk 7
dcubed
parents:
0
diff
changeset
|
1104 // There's alway an extra itable entry so we can null-terminate it. |
f8236e79048a
6664627: Merge changes made only in hotspot 11 forward to jdk 7
dcubed
parents:
0
diff
changeset
|
1105 int itable_size = calc_itable_size(cic.nof_interfaces() + 1, cic.nof_methods()); |
0 | 1106 |
1107 // Statistics | |
1108 update_stats(itable_size * HeapWordSize); | |
1109 | |
1110 return itable_size; | |
1111 } | |
1112 | |
1113 | |
1114 // Fill out offset table and interface klasses into the itable space | |
1115 void klassItable::setup_itable_offset_table(instanceKlassHandle klass) { | |
1116 if (klass->itable_length() == 0) return; | |
1117 assert(!klass->is_interface(), "Should have zero length itable"); | |
1118 | |
1119 // Count no of interfaces and total number of interface methods | |
1120 CountInterfacesClosure cic; | |
1121 visit_all_interfaces(klass->transitive_interfaces(), &cic); | |
1122 int nof_methods = cic.nof_methods(); | |
1123 int nof_interfaces = cic.nof_interfaces(); | |
1124 | |
16
f8236e79048a
6664627: Merge changes made only in hotspot 11 forward to jdk 7
dcubed
parents:
0
diff
changeset
|
1125 // Add one extra entry so we can null-terminate the table |
f8236e79048a
6664627: Merge changes made only in hotspot 11 forward to jdk 7
dcubed
parents:
0
diff
changeset
|
1126 nof_interfaces++; |
0 | 1127 |
1128 assert(compute_itable_size(objArrayHandle(klass->transitive_interfaces())) == | |
1129 calc_itable_size(nof_interfaces, nof_methods), | |
1130 "mismatch calculation of itable size"); | |
1131 | |
1132 // Fill-out offset table | |
1133 itableOffsetEntry* ioe = (itableOffsetEntry*)klass->start_of_itable(); | |
1134 itableMethodEntry* ime = (itableMethodEntry*)(ioe + nof_interfaces); | |
1135 intptr_t* end = klass->end_of_itable(); | |
113
ba764ed4b6f2
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
16
diff
changeset
|
1136 assert((oop*)(ime + nof_methods) <= (oop*)klass->start_of_static_fields(), "wrong offset calculation (1)"); |
ba764ed4b6f2
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
16
diff
changeset
|
1137 assert((oop*)(end) == (oop*)(ime + nof_methods), "wrong offset calculation (2)"); |
0 | 1138 |
1139 // Visit all interfaces and initialize itable offset table | |
1140 SetupItableClosure sic((address)klass->as_klassOop(), ioe, ime); | |
1141 visit_all_interfaces(klass->transitive_interfaces(), &sic); | |
1142 | |
1143 #ifdef ASSERT | |
1144 ime = sic.method_entry(); | |
1145 oop* v = (oop*) klass->end_of_itable(); | |
1146 assert( (oop*)(ime) == v, "wrong offset calculation (2)"); | |
1147 #endif | |
1148 } | |
1149 | |
1150 | |
1151 // m must be a method in an interface | |
1152 int klassItable::compute_itable_index(methodOop m) { | |
1153 klassOop intf = m->method_holder(); | |
1154 assert(instanceKlass::cast(intf)->is_interface(), "sanity check"); | |
1155 objArrayOop methods = instanceKlass::cast(intf)->methods(); | |
1156 int index = 0; | |
1157 while(methods->obj_at(index) != m) { | |
1158 index++; | |
1159 assert(index < methods->length(), "should find index for resolve_invoke"); | |
1160 } | |
1161 // Adjust for <clinit>, which is left out of table if first method | |
1162 if (methods->length() > 0 && ((methodOop)methods->obj_at(0))->name() == vmSymbols::class_initializer_name()) { | |
1163 index--; | |
1164 } | |
1165 return index; | |
1166 } | |
1167 | |
665
c89f86385056
6814659: separable cleanups and subroutines for 6655638
jrose
parents:
610
diff
changeset
|
1168 |
c89f86385056
6814659: separable cleanups and subroutines for 6655638
jrose
parents:
610
diff
changeset
|
1169 // inverse to compute_itable_index |
c89f86385056
6814659: separable cleanups and subroutines for 6655638
jrose
parents:
610
diff
changeset
|
1170 methodOop klassItable::method_for_itable_index(klassOop intf, int itable_index) { |
c89f86385056
6814659: separable cleanups and subroutines for 6655638
jrose
parents:
610
diff
changeset
|
1171 assert(instanceKlass::cast(intf)->is_interface(), "sanity check"); |
c89f86385056
6814659: separable cleanups and subroutines for 6655638
jrose
parents:
610
diff
changeset
|
1172 objArrayOop methods = instanceKlass::cast(intf)->methods(); |
c89f86385056
6814659: separable cleanups and subroutines for 6655638
jrose
parents:
610
diff
changeset
|
1173 |
c89f86385056
6814659: separable cleanups and subroutines for 6655638
jrose
parents:
610
diff
changeset
|
1174 int index = itable_index; |
c89f86385056
6814659: separable cleanups and subroutines for 6655638
jrose
parents:
610
diff
changeset
|
1175 // Adjust for <clinit>, which is left out of table if first method |
c89f86385056
6814659: separable cleanups and subroutines for 6655638
jrose
parents:
610
diff
changeset
|
1176 if (methods->length() > 0 && ((methodOop)methods->obj_at(0))->name() == vmSymbols::class_initializer_name()) { |
c89f86385056
6814659: separable cleanups and subroutines for 6655638
jrose
parents:
610
diff
changeset
|
1177 index++; |
c89f86385056
6814659: separable cleanups and subroutines for 6655638
jrose
parents:
610
diff
changeset
|
1178 } |
c89f86385056
6814659: separable cleanups and subroutines for 6655638
jrose
parents:
610
diff
changeset
|
1179 |
c89f86385056
6814659: separable cleanups and subroutines for 6655638
jrose
parents:
610
diff
changeset
|
1180 if (itable_index < 0 || index >= methods->length()) |
c89f86385056
6814659: separable cleanups and subroutines for 6655638
jrose
parents:
610
diff
changeset
|
1181 return NULL; // help caller defend against bad indexes |
c89f86385056
6814659: separable cleanups and subroutines for 6655638
jrose
parents:
610
diff
changeset
|
1182 |
c89f86385056
6814659: separable cleanups and subroutines for 6655638
jrose
parents:
610
diff
changeset
|
1183 methodOop m = (methodOop)methods->obj_at(index); |
c89f86385056
6814659: separable cleanups and subroutines for 6655638
jrose
parents:
610
diff
changeset
|
1184 assert(compute_itable_index(m) == itable_index, "correct inverse"); |
c89f86385056
6814659: separable cleanups and subroutines for 6655638
jrose
parents:
610
diff
changeset
|
1185 |
c89f86385056
6814659: separable cleanups and subroutines for 6655638
jrose
parents:
610
diff
changeset
|
1186 return m; |
c89f86385056
6814659: separable cleanups and subroutines for 6655638
jrose
parents:
610
diff
changeset
|
1187 } |
c89f86385056
6814659: separable cleanups and subroutines for 6655638
jrose
parents:
610
diff
changeset
|
1188 |
0 | 1189 void klassVtable::verify(outputStream* st, bool forced) { |
1190 // make sure table is initialized | |
1191 if (!Universe::is_fully_initialized()) return; | |
1192 #ifndef PRODUCT | |
1193 // avoid redundant verifies | |
1194 if (!forced && _verify_count == Universe::verify_count()) return; | |
1195 _verify_count = Universe::verify_count(); | |
1196 #endif | |
1197 oop* end_of_obj = (oop*)_klass() + _klass()->size(); | |
1198 oop* end_of_vtable = (oop *)&table()[_length]; | |
1199 if (end_of_vtable > end_of_obj) { | |
1490
f03d0a26bf83
6888954: argument formatting for assert() and friends
jcoomes
parents:
1489
diff
changeset
|
1200 fatal(err_msg("klass %s: klass object too short (vtable extends beyond " |
f03d0a26bf83
6888954: argument formatting for assert() and friends
jcoomes
parents:
1489
diff
changeset
|
1201 "end)", _klass->internal_name())); |
0 | 1202 } |
1203 | |
1204 for (int i = 0; i < _length; i++) table()[i].verify(this, st); | |
1205 // verify consistency with superKlass vtable | |
1206 klassOop super = _klass->super(); | |
1207 if (super != NULL) { | |
1208 instanceKlass* sk = instanceKlass::cast(super); | |
1209 klassVtable* vt = sk->vtable(); | |
1210 for (int i = 0; i < vt->length(); i++) { | |
1211 verify_against(st, vt, i); | |
1212 } | |
1213 } | |
1214 } | |
1215 | |
1216 void klassVtable::verify_against(outputStream* st, klassVtable* vt, int index) { | |
1217 vtableEntry* vte = &vt->table()[index]; | |
1218 if (vte->method()->name() != table()[index].method()->name() || | |
1219 vte->method()->signature() != table()[index].method()->signature()) { | |
1220 fatal("mismatched name/signature of vtable entries"); | |
1221 } | |
1222 } | |
1223 | |
1224 #ifndef PRODUCT | |
1225 void klassVtable::print() { | |
1226 ResourceMark rm; | |
1227 tty->print("klassVtable for klass %s (length %d):\n", _klass->internal_name(), length()); | |
1228 for (int i = 0; i < length(); i++) { | |
1229 table()[i].print(); | |
1230 tty->cr(); | |
1231 } | |
1232 } | |
1233 #endif | |
1234 | |
1235 void vtableEntry::verify(klassVtable* vt, outputStream* st) { | |
1236 NOT_PRODUCT(FlagSetting fs(IgnoreLockingAssertions, true)); | |
1237 assert(method() != NULL, "must have set method"); | |
1238 method()->verify(); | |
1239 // we sub_type, because it could be a miranda method | |
1240 if (!vt->klass()->is_subtype_of(method()->method_holder())) { | |
1241 #ifndef PRODUCT | |
1242 print(); | |
1243 #endif | |
1490
f03d0a26bf83
6888954: argument formatting for assert() and friends
jcoomes
parents:
1489
diff
changeset
|
1244 fatal(err_msg("vtableEntry " PTR_FORMAT ": method is from subclass", this)); |
0 | 1245 } |
1246 } | |
1247 | |
1248 #ifndef PRODUCT | |
1249 | |
1250 void vtableEntry::print() { | |
1251 ResourceMark rm; | |
1252 tty->print("vtableEntry %s: ", method()->name()->as_C_string()); | |
1253 if (Verbose) { | |
1254 tty->print("m %#lx ", (address)method()); | |
1255 } | |
1256 } | |
1257 | |
1258 class VtableStats : AllStatic { | |
1259 public: | |
1260 static int no_klasses; // # classes with vtables | |
1261 static int no_array_klasses; // # array classes | |
1262 static int no_instance_klasses; // # instanceKlasses | |
1263 static int sum_of_vtable_len; // total # of vtable entries | |
1264 static int sum_of_array_vtable_len; // total # of vtable entries in array klasses only | |
1265 static int fixed; // total fixed overhead in bytes | |
1266 static int filler; // overhead caused by filler bytes | |
1267 static int entries; // total bytes consumed by vtable entries | |
1268 static int array_entries; // total bytes consumed by array vtable entries | |
1269 | |
1270 static void do_class(klassOop k) { | |
1271 Klass* kl = k->klass_part(); | |
1272 klassVtable* vt = kl->vtable(); | |
1273 if (vt == NULL) return; | |
1274 no_klasses++; | |
1275 if (kl->oop_is_instance()) { | |
1276 no_instance_klasses++; | |
1277 kl->array_klasses_do(do_class); | |
1278 } | |
1279 if (kl->oop_is_array()) { | |
1280 no_array_klasses++; | |
1281 sum_of_array_vtable_len += vt->length(); | |
1282 } | |
1283 sum_of_vtable_len += vt->length(); | |
1284 } | |
1285 | |
1286 static void compute() { | |
1287 SystemDictionary::classes_do(do_class); | |
1288 fixed = no_klasses * oopSize; // vtable length | |
1289 // filler size is a conservative approximation | |
1290 filler = oopSize * (no_klasses - no_instance_klasses) * (sizeof(instanceKlass) - sizeof(arrayKlass) - 1); | |
1291 entries = sizeof(vtableEntry) * sum_of_vtable_len; | |
1292 array_entries = sizeof(vtableEntry) * sum_of_array_vtable_len; | |
1293 } | |
1294 }; | |
1295 | |
1296 int VtableStats::no_klasses = 0; | |
1297 int VtableStats::no_array_klasses = 0; | |
1298 int VtableStats::no_instance_klasses = 0; | |
1299 int VtableStats::sum_of_vtable_len = 0; | |
1300 int VtableStats::sum_of_array_vtable_len = 0; | |
1301 int VtableStats::fixed = 0; | |
1302 int VtableStats::filler = 0; | |
1303 int VtableStats::entries = 0; | |
1304 int VtableStats::array_entries = 0; | |
1305 | |
1306 void klassVtable::print_statistics() { | |
1307 ResourceMark rm; | |
1308 HandleMark hm; | |
1309 VtableStats::compute(); | |
1310 tty->print_cr("vtable statistics:"); | |
1311 tty->print_cr("%6d classes (%d instance, %d array)", VtableStats::no_klasses, VtableStats::no_instance_klasses, VtableStats::no_array_klasses); | |
1312 int total = VtableStats::fixed + VtableStats::filler + VtableStats::entries; | |
1313 tty->print_cr("%6d bytes fixed overhead (refs + vtable object header)", VtableStats::fixed); | |
1314 tty->print_cr("%6d bytes filler overhead", VtableStats::filler); | |
1315 tty->print_cr("%6d bytes for vtable entries (%d for arrays)", VtableStats::entries, VtableStats::array_entries); | |
1316 tty->print_cr("%6d bytes total", total); | |
1317 } | |
1318 | |
1319 bool klassVtable::check_no_old_entries() { | |
1320 // Check that there really is no entry | |
1321 for (int i = 0; i < length(); i++) { | |
1322 methodOop m = unchecked_method_at(i); | |
1323 if (m != NULL) { | |
1324 if (m->is_old()) { | |
1325 return false; | |
1326 } | |
1327 } | |
1328 } | |
1329 return true; | |
1330 } | |
1331 | |
1332 void klassVtable::dump_vtable() { | |
1333 tty->print_cr("vtable dump --"); | |
1334 for (int i = 0; i < length(); i++) { | |
1335 methodOop m = unchecked_method_at(i); | |
1336 if (m != NULL) { | |
1337 tty->print(" (%5d) ", i); | |
1338 m->access_flags().print_on(tty); | |
1339 tty->print(" -- "); | |
1340 m->print_name(tty); | |
1341 tty->cr(); | |
1342 } | |
1343 } | |
1344 } | |
1345 | |
1346 int klassItable::_total_classes; // Total no. of classes with itables | |
1347 long klassItable::_total_size; // Total no. of bytes used for itables | |
1348 | |
1349 void klassItable::print_statistics() { | |
1350 tty->print_cr("itable statistics:"); | |
1351 tty->print_cr("%6d classes with itables", _total_classes); | |
1352 tty->print_cr("%6d K uses for itables (average by class: %d bytes)", _total_size / K, _total_size / _total_classes); | |
1353 } | |
1354 | |
1355 #endif // PRODUCT |