author | coleenp |
Wed, 24 Jul 2019 10:22:11 -0400 | |
changeset 57511 | 00ae3b739184 |
parent 55655 | 419420eb5230 |
child 58177 | 4932dce35882 |
permissions | -rw-r--r-- |
1 | 1 |
/* |
53904
9c3fe09f69bc
8078725: method adjustments can be done just once for all classes involved into redefinition
coleenp
parents:
52276
diff
changeset
|
2 |
* Copyright (c) 1997, 2019, Oracle and/or its affiliates. All rights reserved. |
1 | 3 |
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. |
4 |
* |
|
5 |
* This code is free software; you can redistribute it and/or modify it |
|
6 |
* under the terms of the GNU General Public License version 2 only, as |
|
7 |
* published by the Free Software Foundation. |
|
8 |
* |
|
9 |
* This code is distributed in the hope that it will be useful, but WITHOUT |
|
10 |
* ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or |
|
11 |
* FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License |
|
12 |
* version 2 for more details (a copy is included in the LICENSE file that |
|
13 |
* accompanied this code). |
|
14 |
* |
|
15 |
* You should have received a copy of the GNU General Public License version |
|
16 |
* 2 along with this work; if not, write to the Free Software Foundation, |
|
17 |
* Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA. |
|
18 |
* |
|
5547
f4b087cbb361
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
5403
diff
changeset
|
19 |
* Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA |
f4b087cbb361
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
5403
diff
changeset
|
20 |
* or visit www.oracle.com if you need additional information or have any |
f4b087cbb361
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
5403
diff
changeset
|
21 |
* questions. |
1 | 22 |
* |
23 |
*/ |
|
24 |
||
7397 | 25 |
#include "precompiled.hpp" |
47765
b7c7428eaab9
8189610: Reconcile jvm.h and all jvm_md.h between java.base and hotspot
coleenp
parents:
47554
diff
changeset
|
26 |
#include "jvm.h" |
51959
db0c3952de52
8209645: Split ClassLoaderData and ClassLoaderDataGraph into separate files
coleenp
parents:
51608
diff
changeset
|
27 |
#include "classfile/classLoaderDataGraph.hpp" |
50036
e0dbf14885b8
8199852: Print more information about class loaders in LinkageErrors.
goetz
parents:
49902
diff
changeset
|
28 |
#include "classfile/javaClasses.hpp" |
7397 | 29 |
#include "classfile/systemDictionary.hpp" |
30 |
#include "classfile/vmSymbols.hpp" |
|
49593
4dd58ecc9912
8200105: Remove cyclic dependency between oop.inline.hpp and collectedHeap.inline.hpp
stefank
parents:
49359
diff
changeset
|
31 |
#include "interpreter/linkResolver.hpp" |
35463
b32e362563bb
8141564: Convert TraceItables and PrintVtables to Unified Logging
rprotacio
parents:
34666
diff
changeset
|
32 |
#include "logging/log.hpp" |
46701
f559541c0daa
8181917: Refactor UL LogStreams to avoid using resource area
stuefe
parents:
46560
diff
changeset
|
33 |
#include "logging/logStream.hpp" |
39714 | 34 |
#include "memory/metaspaceShared.hpp" |
7397 | 35 |
#include "memory/resourceArea.hpp" |
49359
59f6547e151f
8199264: Remove universe.inline.hpp to simplify include dependencies
stefank
parents:
48819
diff
changeset
|
36 |
#include "memory/universe.hpp" |
7397 | 37 |
#include "oops/instanceKlass.hpp" |
38 |
#include "oops/klassVtable.hpp" |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
9172
diff
changeset
|
39 |
#include "oops/method.hpp" |
7397 | 40 |
#include "oops/objArrayOop.hpp" |
41 |
#include "oops/oop.inline.hpp" |
|
42 |
#include "runtime/arguments.hpp" |
|
49902
3661f31c6df4
8202150: [REDO] Split globals.hpp to factor out the Flag class
gziemski
parents:
49860
diff
changeset
|
43 |
#include "runtime/flags/flagSetting.hpp" |
7397 | 44 |
#include "runtime/handles.inline.hpp" |
49594
898ef81cbc0e
8200106: Move NoSafepointVerifier out from gcLocker.hpp
stefank
parents:
49593
diff
changeset
|
45 |
#include "runtime/safepointVerifiers.hpp" |
7397 | 46 |
#include "utilities/copy.hpp" |
1 | 47 |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
9172
diff
changeset
|
48 |
inline InstanceKlass* klassVtable::ik() const { |
46329
53ccc37bda19
8155672: Remove instanceKlassHandles and KlassHandles
coleenp
parents:
41669
diff
changeset
|
49 |
return InstanceKlass::cast(_klass); |
1 | 50 |
} |
51 |
||
39714 | 52 |
bool klassVtable::is_preinitialized_vtable() { |
53 |
return _klass->is_shared() && !MetaspaceShared::remapped_readwrite(); |
|
54 |
} |
|
55 |
||
1 | 56 |
|
57 |
// this function computes the vtable size (including the size needed for miranda |
|
20017
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
58 |
// methods) and the number of miranda methods in this class. |
1 | 59 |
// Note on Miranda methods: Let's say there is a class C that implements |
20017
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
60 |
// interface I, and none of C's superclasses implements I. |
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
61 |
// Let's say there is an abstract method m in I that neither C |
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
62 |
// nor any of its super classes implement (i.e there is no method of any access, |
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
63 |
// with the same name and signature as m), then m is a Miranda method which is |
1 | 64 |
// entered as a public abstract method in C's vtable. From then on it should |
65 |
// treated as any other public method in C for method over-ride purposes. |
|
14385 | 66 |
void klassVtable::compute_vtable_size_and_num_mirandas( |
67 |
int* vtable_length_ret, int* num_new_mirandas, |
|
34666 | 68 |
GrowableArray<Method*>* all_mirandas, const Klass* super, |
40633
c33ad2ff51de
8163808: Fix asserts and logging for old classfile vtable construction
acorn
parents:
39714
diff
changeset
|
69 |
Array<Method*>* methods, AccessFlags class_flags, u2 major_version, |
51329
9c68699bebe5
8208999: Some use of Klass* should be replaced by InstanceKlass*
iklam
parents:
51097
diff
changeset
|
70 |
Handle classloader, Symbol* classname, Array<InstanceKlass*>* local_interfaces, |
14385 | 71 |
TRAPS) { |
35492
c8c0273e6b91
8146690: Make all classes in GC follow the naming convention.
david
parents:
35463
diff
changeset
|
72 |
NoSafepointVerifier nsv; |
1 | 73 |
|
74 |
// set up default result values |
|
14385 | 75 |
int vtable_length = 0; |
1 | 76 |
|
77 |
// start off with super's vtable length |
|
33602 | 78 |
vtable_length = super == NULL ? 0 : super->vtable_length(); |
1 | 79 |
|
80 |
// go thru each method in the methods table to see if it needs a new entry |
|
81 |
int len = methods->length(); |
|
82 |
for (int i = 0; i < len; i++) { |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
9172
diff
changeset
|
83 |
assert(methods->at(i)->is_method(), "must be a Method*"); |
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
9172
diff
changeset
|
84 |
methodHandle mh(THREAD, methods->at(i)); |
1 | 85 |
|
40633
c33ad2ff51de
8163808: Fix asserts and logging for old classfile vtable construction
acorn
parents:
39714
diff
changeset
|
86 |
if (needs_new_vtable_entry(mh, super, classloader, classname, class_flags, major_version, THREAD)) { |
52276
ad9077f044be
8024368: private methods are allocated vtable slots
sspitsyn
parents:
51959
diff
changeset
|
87 |
assert(!methods->at(i)->is_private(), "private methods should not need a vtable entry"); |
1 | 88 |
vtable_length += vtableEntry::size(); // we need a new entry |
89 |
} |
|
90 |
} |
|
91 |
||
14385 | 92 |
GrowableArray<Method*> new_mirandas(20); |
1 | 93 |
// compute the number of mirandas methods that must be added to the end |
48463
474cec233fb2
8154587: Resolution fails for default method named 'clone'
hseigel
parents:
47765
diff
changeset
|
94 |
get_mirandas(&new_mirandas, all_mirandas, super, methods, NULL, local_interfaces, |
474cec233fb2
8154587: Resolution fails for default method named 'clone'
hseigel
parents:
47765
diff
changeset
|
95 |
class_flags.is_interface()); |
14385 | 96 |
*num_new_mirandas = new_mirandas.length(); |
97 |
||
21556
e75cd34a59e0
8027229: ICCE expected for >=2 maximally specific default methods.
acorn
parents:
20391
diff
changeset
|
98 |
// Interfaces do not need interface methods in their vtables |
e75cd34a59e0
8027229: ICCE expected for >=2 maximally specific default methods.
acorn
parents:
20391
diff
changeset
|
99 |
// This includes miranda methods and during later processing, default methods |
e75cd34a59e0
8027229: ICCE expected for >=2 maximally specific default methods.
acorn
parents:
20391
diff
changeset
|
100 |
if (!class_flags.is_interface()) { |
48463
474cec233fb2
8154587: Resolution fails for default method named 'clone'
hseigel
parents:
47765
diff
changeset
|
101 |
vtable_length += *num_new_mirandas * vtableEntry::size(); |
21556
e75cd34a59e0
8027229: ICCE expected for >=2 maximally specific default methods.
acorn
parents:
20391
diff
changeset
|
102 |
} |
1 | 103 |
|
104 |
if (Universe::is_bootstrapping() && vtable_length == 0) { |
|
105 |
// array classes don't have their superclass set correctly during |
|
106 |
// bootstrapping |
|
107 |
vtable_length = Universe::base_vtable_size(); |
|
108 |
} |
|
109 |
||
28373
26fdc99d32f8
6583051: crash when adding non-static methods to java.lang.Object class
coleenp
parents:
25504
diff
changeset
|
110 |
if (super == NULL && vtable_length != Universe::base_vtable_size()) { |
26fdc99d32f8
6583051: crash when adding non-static methods to java.lang.Object class
coleenp
parents:
25504
diff
changeset
|
111 |
if (Universe::is_bootstrapping()) { |
26fdc99d32f8
6583051: crash when adding non-static methods to java.lang.Object class
coleenp
parents:
25504
diff
changeset
|
112 |
// Someone is attempting to override java.lang.Object incorrectly on the |
26fdc99d32f8
6583051: crash when adding non-static methods to java.lang.Object class
coleenp
parents:
25504
diff
changeset
|
113 |
// bootclasspath. The JVM cannot recover from this error including throwing |
26fdc99d32f8
6583051: crash when adding non-static methods to java.lang.Object class
coleenp
parents:
25504
diff
changeset
|
114 |
// an exception |
26fdc99d32f8
6583051: crash when adding non-static methods to java.lang.Object class
coleenp
parents:
25504
diff
changeset
|
115 |
vm_exit_during_initialization("Incompatible definition of java.lang.Object"); |
26fdc99d32f8
6583051: crash when adding non-static methods to java.lang.Object class
coleenp
parents:
25504
diff
changeset
|
116 |
} else { |
26fdc99d32f8
6583051: crash when adding non-static methods to java.lang.Object class
coleenp
parents:
25504
diff
changeset
|
117 |
// Someone is attempting to redefine java.lang.Object incorrectly. The |
26fdc99d32f8
6583051: crash when adding non-static methods to java.lang.Object class
coleenp
parents:
25504
diff
changeset
|
118 |
// only way this should happen is from |
26fdc99d32f8
6583051: crash when adding non-static methods to java.lang.Object class
coleenp
parents:
25504
diff
changeset
|
119 |
// SystemDictionary::resolve_from_stream(), which will detect this later |
26fdc99d32f8
6583051: crash when adding non-static methods to java.lang.Object class
coleenp
parents:
25504
diff
changeset
|
120 |
// and throw a security exception. So don't assert here to let |
26fdc99d32f8
6583051: crash when adding non-static methods to java.lang.Object class
coleenp
parents:
25504
diff
changeset
|
121 |
// the exception occur. |
26fdc99d32f8
6583051: crash when adding non-static methods to java.lang.Object class
coleenp
parents:
25504
diff
changeset
|
122 |
vtable_length = Universe::base_vtable_size(); |
26fdc99d32f8
6583051: crash when adding non-static methods to java.lang.Object class
coleenp
parents:
25504
diff
changeset
|
123 |
} |
1 | 124 |
} |
125 |
assert(vtable_length % vtableEntry::size() == 0, "bad vtable length"); |
|
126 |
assert(vtable_length >= Universe::base_vtable_size(), "vtable too small"); |
|
14385 | 127 |
|
128 |
*vtable_length_ret = vtable_length; |
|
1 | 129 |
} |
130 |
||
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
9172
diff
changeset
|
131 |
int klassVtable::index_of(Method* m, int len) const { |
20017
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
132 |
assert(m->has_vtable_index(), "do not ask this of non-vtable methods"); |
1 | 133 |
return m->vtable_index(); |
134 |
} |
|
135 |
||
20017
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
136 |
// Copy super class's vtable to the first part (prefix) of this class's vtable, |
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
137 |
// and return the number of entries copied. Expects that 'super' is the Java |
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
138 |
// super class (arrays can have "array" super classes that must be skipped). |
46329
53ccc37bda19
8155672: Remove instanceKlassHandles and KlassHandles
coleenp
parents:
41669
diff
changeset
|
139 |
int klassVtable::initialize_from_super(Klass* super) { |
53ccc37bda19
8155672: Remove instanceKlassHandles and KlassHandles
coleenp
parents:
41669
diff
changeset
|
140 |
if (super == NULL) { |
1 | 141 |
return 0; |
39714 | 142 |
} else if (is_preinitialized_vtable()) { |
143 |
// A shared class' vtable is preinitialized at dump time. No need to copy |
|
144 |
// methods from super class for shared class, as that was already done |
|
145 |
// during archiving time. However, if Jvmti has redefined a class, |
|
146 |
// copy super class's vtable in case the super class has changed. |
|
46408
70aab0c2ea8b
8178350: klassVtable and klassItable should be ValueObj
iklam
parents:
46329
diff
changeset
|
147 |
return super->vtable().length(); |
1 | 148 |
} else { |
149 |
// copy methods from superKlass |
|
46408
70aab0c2ea8b
8178350: klassVtable and klassItable should be ValueObj
iklam
parents:
46329
diff
changeset
|
150 |
klassVtable superVtable = super->vtable(); |
70aab0c2ea8b
8178350: klassVtable and klassItable should be ValueObj
iklam
parents:
46329
diff
changeset
|
151 |
assert(superVtable.length() <= _length, "vtable too short"); |
1 | 152 |
#ifdef ASSERT |
46408
70aab0c2ea8b
8178350: klassVtable and klassItable should be ValueObj
iklam
parents:
46329
diff
changeset
|
153 |
superVtable.verify(tty, true); |
1 | 154 |
#endif |
46408
70aab0c2ea8b
8178350: klassVtable and klassItable should be ValueObj
iklam
parents:
46329
diff
changeset
|
155 |
superVtable.copy_vtable_to(table()); |
35901
f5028c67e7cb
8147918: Rename develop_log_is_enabled() to log_develop_is_enabled()
pliden
parents:
35898
diff
changeset
|
156 |
if (log_develop_is_enabled(Trace, vtables)) { |
2264
55d0115a54fe
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
2136
diff
changeset
|
157 |
ResourceMark rm; |
35463
b32e362563bb
8141564: Convert TraceItables and PrintVtables to Unified Logging
rprotacio
parents:
34666
diff
changeset
|
158 |
log_develop_trace(vtables)("copy vtable from %s to %s size %d", |
b32e362563bb
8141564: Convert TraceItables and PrintVtables to Unified Logging
rprotacio
parents:
34666
diff
changeset
|
159 |
super->internal_name(), klass()->internal_name(), |
b32e362563bb
8141564: Convert TraceItables and PrintVtables to Unified Logging
rprotacio
parents:
34666
diff
changeset
|
160 |
_length); |
1 | 161 |
} |
46408
70aab0c2ea8b
8178350: klassVtable and klassItable should be ValueObj
iklam
parents:
46329
diff
changeset
|
162 |
return superVtable.length(); |
1 | 163 |
} |
164 |
} |
|
165 |
||
20017
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
166 |
// |
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
167 |
// Revised lookup semantics introduced 1.3 (Kestrel beta) |
1 | 168 |
void klassVtable::initialize_vtable(bool checkconstraints, TRAPS) { |
169 |
||
170 |
// Note: Arrays can have intermediate array supers. Use java_super to skip them. |
|
51329
9c68699bebe5
8208999: Some use of Klass* should be replaced by InstanceKlass*
iklam
parents:
51097
diff
changeset
|
171 |
InstanceKlass* super = _klass->java_super(); |
1 | 172 |
int nofNewEntries = 0; |
173 |
||
39714 | 174 |
bool is_shared = _klass->is_shared(); |
175 |
||
46329
53ccc37bda19
8155672: Remove instanceKlassHandles and KlassHandles
coleenp
parents:
41669
diff
changeset
|
176 |
if (!_klass->is_array_klass()) { |
1 | 177 |
ResourceMark rm(THREAD); |
35463
b32e362563bb
8141564: Convert TraceItables and PrintVtables to Unified Logging
rprotacio
parents:
34666
diff
changeset
|
178 |
log_develop_debug(vtables)("Initializing: %s", _klass->name()->as_C_string()); |
1 | 179 |
} |
180 |
||
181 |
#ifdef ASSERT |
|
46329
53ccc37bda19
8155672: Remove instanceKlassHandles and KlassHandles
coleenp
parents:
41669
diff
changeset
|
182 |
oop* end_of_obj = (oop*)_klass + _klass->size(); |
1 | 183 |
oop* end_of_vtable = (oop*)&table()[_length]; |
184 |
assert(end_of_vtable <= end_of_obj, "vtable extends beyond end"); |
|
185 |
#endif |
|
186 |
||
187 |
if (Universe::is_bootstrapping()) { |
|
39714 | 188 |
assert(!is_shared, "sanity"); |
1 | 189 |
// just clear everything |
190 |
for (int i = 0; i < _length; i++) table()[i].clear(); |
|
191 |
return; |
|
192 |
} |
|
193 |
||
194 |
int super_vtable_len = initialize_from_super(super); |
|
46329
53ccc37bda19
8155672: Remove instanceKlassHandles and KlassHandles
coleenp
parents:
41669
diff
changeset
|
195 |
if (_klass->is_array_klass()) { |
1 | 196 |
assert(super_vtable_len == _length, "arrays shouldn't introduce new methods"); |
197 |
} else { |
|
33611
9abd65805e19
8139203: Consistent naming for klass type predicates
coleenp
parents:
33602
diff
changeset
|
198 |
assert(_klass->is_instance_klass(), "must be InstanceKlass"); |
1 | 199 |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
9172
diff
changeset
|
200 |
Array<Method*>* methods = ik()->methods(); |
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
9172
diff
changeset
|
201 |
int len = methods->length(); |
1 | 202 |
int initialized = super_vtable_len; |
203 |
||
20017
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
204 |
// Check each of this class's methods against super; |
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
205 |
// if override, replace in copy of super vtable, otherwise append to end |
1 | 206 |
for (int i = 0; i < len; i++) { |
20017
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
207 |
// update_inherited_vtable can stop for gc - ensure using handles |
1 | 208 |
HandleMark hm(THREAD); |
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
9172
diff
changeset
|
209 |
assert(methods->at(i)->is_method(), "must be a Method*"); |
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
9172
diff
changeset
|
210 |
methodHandle mh(THREAD, methods->at(i)); |
1 | 211 |
|
20391
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
212 |
bool needs_new_entry = update_inherited_vtable(ik(), mh, super_vtable_len, -1, checkconstraints, CHECK); |
1 | 213 |
|
214 |
if (needs_new_entry) { |
|
215 |
put_method_at(mh(), initialized); |
|
216 |
mh()->set_vtable_index(initialized); // set primary vtable index |
|
217 |
initialized++; |
|
218 |
} |
|
219 |
} |
|
220 |
||
20391
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
221 |
// update vtable with default_methods |
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
222 |
Array<Method*>* default_methods = ik()->default_methods(); |
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
223 |
if (default_methods != NULL) { |
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
224 |
len = default_methods->length(); |
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
225 |
if (len > 0) { |
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
226 |
Array<int>* def_vtable_indices = NULL; |
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
227 |
if ((def_vtable_indices = ik()->default_vtable_indices()) == NULL) { |
39714 | 228 |
assert(!is_shared, "shared class def_vtable_indices does not exist"); |
20391
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
229 |
def_vtable_indices = ik()->create_new_default_vtable_indices(len, CHECK); |
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
230 |
} else { |
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
231 |
assert(def_vtable_indices->length() == len, "reinit vtable len?"); |
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
232 |
} |
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
233 |
for (int i = 0; i < len; i++) { |
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
234 |
HandleMark hm(THREAD); |
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
235 |
assert(default_methods->at(i)->is_method(), "must be a Method*"); |
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
236 |
methodHandle mh(THREAD, default_methods->at(i)); |
41669
2091069b6851
8081800: AbstractMethodError when evaluating a private method in an interface via debugger
dholmes
parents:
40633
diff
changeset
|
237 |
assert(!mh->is_private(), "private interface method in the default method list"); |
20391
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
238 |
bool needs_new_entry = update_inherited_vtable(ik(), mh, super_vtable_len, i, checkconstraints, CHECK); |
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
239 |
|
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
240 |
// needs new entry |
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
241 |
if (needs_new_entry) { |
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
242 |
put_method_at(mh(), initialized); |
39714 | 243 |
if (is_preinitialized_vtable()) { |
244 |
// At runtime initialize_vtable is rerun for a shared class |
|
245 |
// (loaded by the non-boot loader) as part of link_class_impl(). |
|
246 |
// The dumptime vtable index should be the same as the runtime index. |
|
247 |
assert(def_vtable_indices->at(i) == initialized, |
|
248 |
"dump time vtable index is different from runtime index"); |
|
249 |
} else { |
|
250 |
def_vtable_indices->at_put(i, initialized); //set vtable index |
|
251 |
} |
|
20391
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
252 |
initialized++; |
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
253 |
} |
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
254 |
} |
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
255 |
} |
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
256 |
} |
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
257 |
|
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
258 |
// add miranda methods; it will also return the updated initialized |
21556
e75cd34a59e0
8027229: ICCE expected for >=2 maximally specific default methods.
acorn
parents:
20391
diff
changeset
|
259 |
// Interfaces do not need interface methods in their vtables |
e75cd34a59e0
8027229: ICCE expected for >=2 maximally specific default methods.
acorn
parents:
20391
diff
changeset
|
260 |
// This includes miranda methods and during later processing, default methods |
e75cd34a59e0
8027229: ICCE expected for >=2 maximally specific default methods.
acorn
parents:
20391
diff
changeset
|
261 |
if (!ik()->is_interface()) { |
51096
695dff91a997
8178712: ResourceMark may be missing inside initialize_[vi]table
lfoltan
parents:
50735
diff
changeset
|
262 |
initialized = fill_in_mirandas(initialized, THREAD); |
21556
e75cd34a59e0
8027229: ICCE expected for >=2 maximally specific default methods.
acorn
parents:
20391
diff
changeset
|
263 |
} |
1 | 264 |
|
2264
55d0115a54fe
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
2136
diff
changeset
|
265 |
// In class hierarchies where the accessibility is not increasing (i.e., going from private -> |
20017
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
266 |
// package_private -> public/protected), the vtable might actually be smaller than our initial |
40633
c33ad2ff51de
8163808: Fix asserts and logging for old classfile vtable construction
acorn
parents:
39714
diff
changeset
|
267 |
// calculation, for classfile versions for which we do not do transitive override |
c33ad2ff51de
8163808: Fix asserts and logging for old classfile vtable construction
acorn
parents:
39714
diff
changeset
|
268 |
// calculations. |
c33ad2ff51de
8163808: Fix asserts and logging for old classfile vtable construction
acorn
parents:
39714
diff
changeset
|
269 |
if (ik()->major_version() >= VTABLE_TRANSITIVE_OVERRIDE_VERSION) { |
c33ad2ff51de
8163808: Fix asserts and logging for old classfile vtable construction
acorn
parents:
39714
diff
changeset
|
270 |
assert(initialized == _length, "vtable initialization failed"); |
c33ad2ff51de
8163808: Fix asserts and logging for old classfile vtable construction
acorn
parents:
39714
diff
changeset
|
271 |
} else { |
c33ad2ff51de
8163808: Fix asserts and logging for old classfile vtable construction
acorn
parents:
39714
diff
changeset
|
272 |
assert(initialized <= _length, "vtable initialization failed"); |
c33ad2ff51de
8163808: Fix asserts and logging for old classfile vtable construction
acorn
parents:
39714
diff
changeset
|
273 |
for(;initialized < _length; initialized++) { |
c33ad2ff51de
8163808: Fix asserts and logging for old classfile vtable construction
acorn
parents:
39714
diff
changeset
|
274 |
table()[initialized].clear(); |
c33ad2ff51de
8163808: Fix asserts and logging for old classfile vtable construction
acorn
parents:
39714
diff
changeset
|
275 |
} |
1 | 276 |
} |
277 |
NOT_PRODUCT(verify(tty, true)); |
|
278 |
} |
|
279 |
} |
|
280 |
||
2264
55d0115a54fe
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
2136
diff
changeset
|
281 |
// Called for cases where a method does not override its superclass' vtable entry |
55d0115a54fe
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
2136
diff
changeset
|
282 |
// For bytecodes not produced by javac together it is possible that a method does not override |
55d0115a54fe
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
2136
diff
changeset
|
283 |
// the superclass's method, but might indirectly override a super-super class's vtable entry |
55d0115a54fe
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
2136
diff
changeset
|
284 |
// If none found, return a null superk, else return the superk of the method this does override |
25504
69de8357bc53
8032536: JVM resolves wrong method in some unusual cases
hseigel
parents:
24424
diff
changeset
|
285 |
// For public and protected methods: if they override a superclass, they will |
69de8357bc53
8032536: JVM resolves wrong method in some unusual cases
hseigel
parents:
24424
diff
changeset
|
286 |
// also be overridden themselves appropriately. |
52276
ad9077f044be
8024368: private methods are allocated vtable slots
sspitsyn
parents:
51959
diff
changeset
|
287 |
// Private methods do not override, and are not overridden and are not in the vtable. |
25504
69de8357bc53
8032536: JVM resolves wrong method in some unusual cases
hseigel
parents:
24424
diff
changeset
|
288 |
// Package Private methods are trickier: |
69de8357bc53
8032536: JVM resolves wrong method in some unusual cases
hseigel
parents:
24424
diff
changeset
|
289 |
// e.g. P1.A, pub m |
69de8357bc53
8032536: JVM resolves wrong method in some unusual cases
hseigel
parents:
24424
diff
changeset
|
290 |
// P2.B extends A, package private m |
69de8357bc53
8032536: JVM resolves wrong method in some unusual cases
hseigel
parents:
24424
diff
changeset
|
291 |
// P1.C extends B, public m |
69de8357bc53
8032536: JVM resolves wrong method in some unusual cases
hseigel
parents:
24424
diff
changeset
|
292 |
// P1.C.m needs to override P1.A.m and can not override P2.B.m |
69de8357bc53
8032536: JVM resolves wrong method in some unusual cases
hseigel
parents:
24424
diff
changeset
|
293 |
// Therefore: all package private methods need their own vtable entries for |
69de8357bc53
8032536: JVM resolves wrong method in some unusual cases
hseigel
parents:
24424
diff
changeset
|
294 |
// them to be the root of an inheritance overriding decision |
69de8357bc53
8032536: JVM resolves wrong method in some unusual cases
hseigel
parents:
24424
diff
changeset
|
295 |
// Package private methods may also override other vtable entries |
46727
6e4a84748e2c
8183039: Re-examine methodHandle methods uninlined by 8144256
coleenp
parents:
46701
diff
changeset
|
296 |
InstanceKlass* klassVtable::find_transitive_override(InstanceKlass* initialsuper, const methodHandle& target_method, |
8076
96d498ec7ae1
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
7443
diff
changeset
|
297 |
int vtable_index, Handle target_loader, Symbol* target_classname, Thread * THREAD) { |
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
9172
diff
changeset
|
298 |
InstanceKlass* superk = initialsuper; |
2264
55d0115a54fe
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
2136
diff
changeset
|
299 |
while (superk != NULL && superk->super() != NULL) { |
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
9172
diff
changeset
|
300 |
InstanceKlass* supersuperklass = InstanceKlass::cast(superk->super()); |
46408
70aab0c2ea8b
8178350: klassVtable and klassItable should be ValueObj
iklam
parents:
46329
diff
changeset
|
301 |
klassVtable ssVtable = supersuperklass->vtable(); |
70aab0c2ea8b
8178350: klassVtable and klassItable should be ValueObj
iklam
parents:
46329
diff
changeset
|
302 |
if (vtable_index < ssVtable.length()) { |
70aab0c2ea8b
8178350: klassVtable and klassItable should be ValueObj
iklam
parents:
46329
diff
changeset
|
303 |
Method* super_method = ssVtable.method_at(vtable_index); |
2264
55d0115a54fe
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
2136
diff
changeset
|
304 |
#ifndef PRODUCT |
8076
96d498ec7ae1
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
7443
diff
changeset
|
305 |
Symbol* name= target_method()->name(); |
96d498ec7ae1
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
7443
diff
changeset
|
306 |
Symbol* signature = target_method()->signature(); |
96d498ec7ae1
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
7443
diff
changeset
|
307 |
assert(super_method->name() == name && super_method->signature() == signature, "vtable entry name/sig mismatch"); |
2264
55d0115a54fe
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
2136
diff
changeset
|
308 |
#endif |
55d0115a54fe
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
2136
diff
changeset
|
309 |
if (supersuperklass->is_override(super_method, target_loader, target_classname, THREAD)) { |
35901
f5028c67e7cb
8147918: Rename develop_log_is_enabled() to log_develop_is_enabled()
pliden
parents:
35898
diff
changeset
|
310 |
if (log_develop_is_enabled(Trace, vtables)) { |
2264
55d0115a54fe
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
2136
diff
changeset
|
311 |
ResourceMark rm(THREAD); |
46701
f559541c0daa
8181917: Refactor UL LogStreams to avoid using resource area
stuefe
parents:
46560
diff
changeset
|
312 |
LogTarget(Trace, vtables) lt; |
f559541c0daa
8181917: Refactor UL LogStreams to avoid using resource area
stuefe
parents:
46560
diff
changeset
|
313 |
LogStream ls(lt); |
20391
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
314 |
char* sig = target_method()->name_and_sig_as_C_string(); |
46701
f559541c0daa
8181917: Refactor UL LogStreams to avoid using resource area
stuefe
parents:
46560
diff
changeset
|
315 |
ls.print("transitive overriding superclass %s with %s index %d, original flags: ", |
35463
b32e362563bb
8141564: Convert TraceItables and PrintVtables to Unified Logging
rprotacio
parents:
34666
diff
changeset
|
316 |
supersuperklass->internal_name(), |
40633
c33ad2ff51de
8163808: Fix asserts and logging for old classfile vtable construction
acorn
parents:
39714
diff
changeset
|
317 |
sig, vtable_index); |
46701
f559541c0daa
8181917: Refactor UL LogStreams to avoid using resource area
stuefe
parents:
46560
diff
changeset
|
318 |
super_method->print_linkage_flags(&ls); |
f559541c0daa
8181917: Refactor UL LogStreams to avoid using resource area
stuefe
parents:
46560
diff
changeset
|
319 |
ls.print("overriders flags: "); |
f559541c0daa
8181917: Refactor UL LogStreams to avoid using resource area
stuefe
parents:
46560
diff
changeset
|
320 |
target_method->print_linkage_flags(&ls); |
f559541c0daa
8181917: Refactor UL LogStreams to avoid using resource area
stuefe
parents:
46560
diff
changeset
|
321 |
ls.cr(); |
2264
55d0115a54fe
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
2136
diff
changeset
|
322 |
} |
35463
b32e362563bb
8141564: Convert TraceItables and PrintVtables to Unified Logging
rprotacio
parents:
34666
diff
changeset
|
323 |
|
2264
55d0115a54fe
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
2136
diff
changeset
|
324 |
break; // return found superk |
55d0115a54fe
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
2136
diff
changeset
|
325 |
} |
55d0115a54fe
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
2136
diff
changeset
|
326 |
} else { |
55d0115a54fe
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
2136
diff
changeset
|
327 |
// super class has no vtable entry here, stop transitive search |
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
9172
diff
changeset
|
328 |
superk = (InstanceKlass*)NULL; |
2264
55d0115a54fe
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
2136
diff
changeset
|
329 |
break; |
55d0115a54fe
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
2136
diff
changeset
|
330 |
} |
55d0115a54fe
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
2136
diff
changeset
|
331 |
// if no override found yet, continue to search up |
33602 | 332 |
superk = superk->super() == NULL ? NULL : InstanceKlass::cast(superk->super()); |
2264
55d0115a54fe
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
2136
diff
changeset
|
333 |
} |
1 | 334 |
|
2264
55d0115a54fe
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
2136
diff
changeset
|
335 |
return superk; |
1 | 336 |
} |
337 |
||
46727
6e4a84748e2c
8183039: Re-examine methodHandle methods uninlined by 8144256
coleenp
parents:
46701
diff
changeset
|
338 |
static void log_vtables(int i, bool overrides, const methodHandle& target_method, |
46329
53ccc37bda19
8155672: Remove instanceKlassHandles and KlassHandles
coleenp
parents:
41669
diff
changeset
|
339 |
Klass* target_klass, Method* super_method, |
35463
b32e362563bb
8141564: Convert TraceItables and PrintVtables to Unified Logging
rprotacio
parents:
34666
diff
changeset
|
340 |
Thread* thread) { |
b32e362563bb
8141564: Convert TraceItables and PrintVtables to Unified Logging
rprotacio
parents:
34666
diff
changeset
|
341 |
#ifndef PRODUCT |
35901
f5028c67e7cb
8147918: Rename develop_log_is_enabled() to log_develop_is_enabled()
pliden
parents:
35898
diff
changeset
|
342 |
if (log_develop_is_enabled(Trace, vtables)) { |
35463
b32e362563bb
8141564: Convert TraceItables and PrintVtables to Unified Logging
rprotacio
parents:
34666
diff
changeset
|
343 |
ResourceMark rm(thread); |
46701
f559541c0daa
8181917: Refactor UL LogStreams to avoid using resource area
stuefe
parents:
46560
diff
changeset
|
344 |
LogTarget(Trace, vtables) lt; |
f559541c0daa
8181917: Refactor UL LogStreams to avoid using resource area
stuefe
parents:
46560
diff
changeset
|
345 |
LogStream ls(lt); |
35463
b32e362563bb
8141564: Convert TraceItables and PrintVtables to Unified Logging
rprotacio
parents:
34666
diff
changeset
|
346 |
char* sig = target_method()->name_and_sig_as_C_string(); |
b32e362563bb
8141564: Convert TraceItables and PrintVtables to Unified Logging
rprotacio
parents:
34666
diff
changeset
|
347 |
if (overrides) { |
46701
f559541c0daa
8181917: Refactor UL LogStreams to avoid using resource area
stuefe
parents:
46560
diff
changeset
|
348 |
ls.print("overriding with %s index %d, original flags: ", |
40633
c33ad2ff51de
8163808: Fix asserts and logging for old classfile vtable construction
acorn
parents:
39714
diff
changeset
|
349 |
sig, i); |
35463
b32e362563bb
8141564: Convert TraceItables and PrintVtables to Unified Logging
rprotacio
parents:
34666
diff
changeset
|
350 |
} else { |
46701
f559541c0daa
8181917: Refactor UL LogStreams to avoid using resource area
stuefe
parents:
46560
diff
changeset
|
351 |
ls.print("NOT overriding with %s index %d, original flags: ", |
40633
c33ad2ff51de
8163808: Fix asserts and logging for old classfile vtable construction
acorn
parents:
39714
diff
changeset
|
352 |
sig, i); |
35463
b32e362563bb
8141564: Convert TraceItables and PrintVtables to Unified Logging
rprotacio
parents:
34666
diff
changeset
|
353 |
} |
46701
f559541c0daa
8181917: Refactor UL LogStreams to avoid using resource area
stuefe
parents:
46560
diff
changeset
|
354 |
super_method->print_linkage_flags(&ls); |
f559541c0daa
8181917: Refactor UL LogStreams to avoid using resource area
stuefe
parents:
46560
diff
changeset
|
355 |
ls.print("overriders flags: "); |
f559541c0daa
8181917: Refactor UL LogStreams to avoid using resource area
stuefe
parents:
46560
diff
changeset
|
356 |
target_method->print_linkage_flags(&ls); |
f559541c0daa
8181917: Refactor UL LogStreams to avoid using resource area
stuefe
parents:
46560
diff
changeset
|
357 |
ls.cr(); |
35463
b32e362563bb
8141564: Convert TraceItables and PrintVtables to Unified Logging
rprotacio
parents:
34666
diff
changeset
|
358 |
} |
b32e362563bb
8141564: Convert TraceItables and PrintVtables to Unified Logging
rprotacio
parents:
34666
diff
changeset
|
359 |
#endif |
b32e362563bb
8141564: Convert TraceItables and PrintVtables to Unified Logging
rprotacio
parents:
34666
diff
changeset
|
360 |
} |
b32e362563bb
8141564: Convert TraceItables and PrintVtables to Unified Logging
rprotacio
parents:
34666
diff
changeset
|
361 |
|
1 | 362 |
// Update child's copy of super vtable for overrides |
20017
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
363 |
// OR return true if a new vtable entry is required. |
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
9172
diff
changeset
|
364 |
// Only called for InstanceKlass's, i.e. not for arrays |
1 | 365 |
// If that changed, could not use _klass as handle for klass |
46727
6e4a84748e2c
8183039: Re-examine methodHandle methods uninlined by 8144256
coleenp
parents:
46701
diff
changeset
|
366 |
bool klassVtable::update_inherited_vtable(InstanceKlass* klass, const methodHandle& target_method, |
20391
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
367 |
int super_vtable_len, int default_index, |
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
368 |
bool checkconstraints, TRAPS) { |
51096
695dff91a997
8178712: ResourceMark may be missing inside initialize_[vi]table
lfoltan
parents:
50735
diff
changeset
|
369 |
ResourceMark rm(THREAD); |
1 | 370 |
bool allocate_new = true; |
33611
9abd65805e19
8139203: Consistent naming for klass type predicates
coleenp
parents:
33602
diff
changeset
|
371 |
assert(klass->is_instance_klass(), "must be InstanceKlass"); |
1 | 372 |
|
20391
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
373 |
Array<int>* def_vtable_indices = NULL; |
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
374 |
bool is_default = false; |
41669
2091069b6851
8081800: AbstractMethodError when evaluating a private method in an interface via debugger
dholmes
parents:
40633
diff
changeset
|
375 |
|
2091069b6851
8081800: AbstractMethodError when evaluating a private method in an interface via debugger
dholmes
parents:
40633
diff
changeset
|
376 |
// default methods are non-private concrete methods in superinterfaces which are added |
2091069b6851
8081800: AbstractMethodError when evaluating a private method in an interface via debugger
dholmes
parents:
40633
diff
changeset
|
377 |
// to the vtable with their real method_holder. |
20391
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
378 |
// Since vtable and itable indices share the same storage, don't touch |
41669
2091069b6851
8081800: AbstractMethodError when evaluating a private method in an interface via debugger
dholmes
parents:
40633
diff
changeset
|
379 |
// the default method's real vtable/itable index. |
20391
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
380 |
// default_vtable_indices stores the vtable value relative to this inheritor |
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
381 |
if (default_index >= 0 ) { |
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
382 |
is_default = true; |
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
383 |
def_vtable_indices = klass->default_vtable_indices(); |
41669
2091069b6851
8081800: AbstractMethodError when evaluating a private method in an interface via debugger
dholmes
parents:
40633
diff
changeset
|
384 |
assert(!target_method()->is_private(), "private interface method flagged as default"); |
20391
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
385 |
assert(def_vtable_indices != NULL, "def vtable alloc?"); |
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
386 |
assert(default_index <= def_vtable_indices->length(), "def vtable len?"); |
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
387 |
} else { |
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
388 |
assert(klass == target_method()->method_holder(), "caller resp."); |
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
389 |
// Initialize the method's vtable index to "nonvirtual". |
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
390 |
// If we allocate a vtable entry, we will update it to a non-negative number. |
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
391 |
target_method()->set_vtable_index(Method::nonvirtual_vtable_index); |
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
392 |
} |
1 | 393 |
|
52276
ad9077f044be
8024368: private methods are allocated vtable slots
sspitsyn
parents:
51959
diff
changeset
|
394 |
// Private, static and <init> methods are never in |
ad9077f044be
8024368: private methods are allocated vtable slots
sspitsyn
parents:
51959
diff
changeset
|
395 |
if (target_method()->is_private() || target_method()->is_static() || |
ad9077f044be
8024368: private methods are allocated vtable slots
sspitsyn
parents:
51959
diff
changeset
|
396 |
(target_method()->name()->fast_compare(vmSymbols::object_initializer_name()) == 0)) { |
1 | 397 |
return false; |
398 |
} |
|
399 |
||
20017
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
400 |
if (target_method->is_final_method(klass->access_flags())) { |
1 | 401 |
// a final method never needs a new entry; final methods can be statically |
402 |
// resolved and they have to be present in the vtable only if they override |
|
403 |
// a super's method, in which case they re-use its entry |
|
404 |
allocate_new = false; |
|
20017
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
405 |
} else if (klass->is_interface()) { |
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
406 |
allocate_new = false; // see note below in needs_new_vtable_entry |
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
407 |
// An interface never allocates new vtable slots, only inherits old ones. |
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
408 |
// This method will either be assigned its own itable index later, |
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
409 |
// or be assigned an inherited vtable index in the loop below. |
21556
e75cd34a59e0
8027229: ICCE expected for >=2 maximally specific default methods.
acorn
parents:
20391
diff
changeset
|
410 |
// default methods inherited by classes store their vtable indices |
41669
2091069b6851
8081800: AbstractMethodError when evaluating a private method in an interface via debugger
dholmes
parents:
40633
diff
changeset
|
411 |
// in the inheritor's default_vtable_indices. |
21556
e75cd34a59e0
8027229: ICCE expected for >=2 maximally specific default methods.
acorn
parents:
20391
diff
changeset
|
412 |
// default methods inherited by interfaces may already have a |
41669
2091069b6851
8081800: AbstractMethodError when evaluating a private method in an interface via debugger
dholmes
parents:
40633
diff
changeset
|
413 |
// valid itable index, if so, don't change it. |
2091069b6851
8081800: AbstractMethodError when evaluating a private method in an interface via debugger
dholmes
parents:
40633
diff
changeset
|
414 |
// Overpass methods in an interface will be assigned an itable index later |
2091069b6851
8081800: AbstractMethodError when evaluating a private method in an interface via debugger
dholmes
parents:
40633
diff
changeset
|
415 |
// by an inheriting class. |
52276
ad9077f044be
8024368: private methods are allocated vtable slots
sspitsyn
parents:
51959
diff
changeset
|
416 |
if ((!is_default || !target_method()->has_itable_index())) { |
21556
e75cd34a59e0
8027229: ICCE expected for >=2 maximally specific default methods.
acorn
parents:
20391
diff
changeset
|
417 |
target_method()->set_vtable_index(Method::pending_itable_index); |
e75cd34a59e0
8027229: ICCE expected for >=2 maximally specific default methods.
acorn
parents:
20391
diff
changeset
|
418 |
} |
1 | 419 |
} |
420 |
||
421 |
// we need a new entry if there is no superclass |
|
39714 | 422 |
Klass* super = klass->super(); |
423 |
if (super == NULL) { |
|
1 | 424 |
return allocate_new; |
425 |
} |
|
426 |
||
427 |
// search through the vtable and update overridden entries |
|
428 |
// Since check_signature_loaders acquires SystemDictionary_lock |
|
2264
55d0115a54fe
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
2136
diff
changeset
|
429 |
// which can block for gc, once we are in this loop, use handles |
55d0115a54fe
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
2136
diff
changeset
|
430 |
// For classfiles built with >= jdk7, we now look for transitive overrides |
1 | 431 |
|
8076
96d498ec7ae1
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
7443
diff
changeset
|
432 |
Symbol* name = target_method()->name(); |
96d498ec7ae1
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
7443
diff
changeset
|
433 |
Symbol* signature = target_method()->signature(); |
20391
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
434 |
|
46329
53ccc37bda19
8155672: Remove instanceKlassHandles and KlassHandles
coleenp
parents:
41669
diff
changeset
|
435 |
Klass* target_klass = target_method()->method_holder(); |
20391
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
436 |
if (target_klass == NULL) { |
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
437 |
target_klass = _klass; |
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
438 |
} |
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
439 |
|
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
440 |
Handle target_loader(THREAD, target_klass->class_loader()); |
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
441 |
|
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
442 |
Symbol* target_classname = target_klass->name(); |
1 | 443 |
for(int i = 0; i < super_vtable_len; i++) { |
39714 | 444 |
Method* super_method; |
445 |
if (is_preinitialized_vtable()) { |
|
446 |
// If this is a shared class, the vtable is already in the final state (fully |
|
447 |
// initialized). Need to look at the super's vtable. |
|
46408
70aab0c2ea8b
8178350: klassVtable and klassItable should be ValueObj
iklam
parents:
46329
diff
changeset
|
448 |
klassVtable superVtable = super->vtable(); |
70aab0c2ea8b
8178350: klassVtable and klassItable should be ValueObj
iklam
parents:
46329
diff
changeset
|
449 |
super_method = superVtable.method_at(i); |
39714 | 450 |
} else { |
451 |
super_method = method_at(i); |
|
452 |
} |
|
48463
474cec233fb2
8154587: Resolution fails for default method named 'clone'
hseigel
parents:
47765
diff
changeset
|
453 |
// Check if method name matches. Ignore match if klass is an interface and the |
474cec233fb2
8154587: Resolution fails for default method named 'clone'
hseigel
parents:
47765
diff
changeset
|
454 |
// matching method is a non-public java.lang.Object method. (See JVMS 5.4.3.4) |
474cec233fb2
8154587: Resolution fails for default method named 'clone'
hseigel
parents:
47765
diff
changeset
|
455 |
// This is safe because the method at this slot should never get invoked. |
474cec233fb2
8154587: Resolution fails for default method named 'clone'
hseigel
parents:
47765
diff
changeset
|
456 |
// (TBD: put in a method to throw NoSuchMethodError if this slot is ever used.) |
474cec233fb2
8154587: Resolution fails for default method named 'clone'
hseigel
parents:
47765
diff
changeset
|
457 |
if (super_method->name() == name && super_method->signature() == signature && |
474cec233fb2
8154587: Resolution fails for default method named 'clone'
hseigel
parents:
47765
diff
changeset
|
458 |
(!_klass->is_interface() || |
474cec233fb2
8154587: Resolution fails for default method named 'clone'
hseigel
parents:
47765
diff
changeset
|
459 |
!SystemDictionary::is_nonpublic_Object_method(super_method))) { |
1 | 460 |
|
2264
55d0115a54fe
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
2136
diff
changeset
|
461 |
// get super_klass for method_holder for the found method |
14391
df0a1573d5bd
8000725: NPG: method_holder() and pool_holder() and pool_holder field should be InstanceKlass
coleenp
parents:
14385
diff
changeset
|
462 |
InstanceKlass* super_klass = super_method->method_holder(); |
1 | 463 |
|
35463
b32e362563bb
8141564: Convert TraceItables and PrintVtables to Unified Logging
rprotacio
parents:
34666
diff
changeset
|
464 |
// Whether the method is being overridden |
b32e362563bb
8141564: Convert TraceItables and PrintVtables to Unified Logging
rprotacio
parents:
34666
diff
changeset
|
465 |
bool overrides = false; |
b32e362563bb
8141564: Convert TraceItables and PrintVtables to Unified Logging
rprotacio
parents:
34666
diff
changeset
|
466 |
|
29871 | 467 |
// private methods are also never overridden |
468 |
if (!super_method->is_private() && |
|
469 |
(is_default |
|
20391
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
470 |
|| ((super_klass->is_override(super_method, target_loader, target_classname, THREAD)) |
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
471 |
|| ((klass->major_version() >= VTABLE_TRANSITIVE_OVERRIDE_VERSION) |
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
472 |
&& ((super_klass = find_transitive_override(super_klass, |
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
473 |
target_method, i, target_loader, |
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
474 |
target_classname, THREAD)) |
29871 | 475 |
!= (InstanceKlass*)NULL))))) |
20391
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
476 |
{ |
25504
69de8357bc53
8032536: JVM resolves wrong method in some unusual cases
hseigel
parents:
24424
diff
changeset
|
477 |
// Package private methods always need a new entry to root their own |
69de8357bc53
8032536: JVM resolves wrong method in some unusual cases
hseigel
parents:
24424
diff
changeset
|
478 |
// overriding. They may also override other methods. |
69de8357bc53
8032536: JVM resolves wrong method in some unusual cases
hseigel
parents:
24424
diff
changeset
|
479 |
if (!target_method()->is_package_private()) { |
69de8357bc53
8032536: JVM resolves wrong method in some unusual cases
hseigel
parents:
24424
diff
changeset
|
480 |
allocate_new = false; |
69de8357bc53
8032536: JVM resolves wrong method in some unusual cases
hseigel
parents:
24424
diff
changeset
|
481 |
} |
1 | 482 |
|
47554
bc112140e089
8186092: Unnecessary loader constraints produced when there are multiple defaults
hseigel
parents:
47216
diff
changeset
|
483 |
// Do not check loader constraints for overpass methods because overpass |
bc112140e089
8186092: Unnecessary loader constraints produced when there are multiple defaults
hseigel
parents:
47216
diff
changeset
|
484 |
// methods are created by the jvm to throw exceptions. |
bc112140e089
8186092: Unnecessary loader constraints produced when there are multiple defaults
hseigel
parents:
47216
diff
changeset
|
485 |
if (checkconstraints && !target_method()->is_overpass()) { |
bc112140e089
8186092: Unnecessary loader constraints produced when there are multiple defaults
hseigel
parents:
47216
diff
changeset
|
486 |
// Override vtable entry if passes loader constraint check |
bc112140e089
8186092: Unnecessary loader constraints produced when there are multiple defaults
hseigel
parents:
47216
diff
changeset
|
487 |
// if loader constraint checking requested |
bc112140e089
8186092: Unnecessary loader constraints produced when there are multiple defaults
hseigel
parents:
47216
diff
changeset
|
488 |
// No need to visit his super, since he and his super |
bc112140e089
8186092: Unnecessary loader constraints produced when there are multiple defaults
hseigel
parents:
47216
diff
changeset
|
489 |
// have already made any needed loader constraints. |
bc112140e089
8186092: Unnecessary loader constraints produced when there are multiple defaults
hseigel
parents:
47216
diff
changeset
|
490 |
// Since loader constraints are transitive, it is enough |
bc112140e089
8186092: Unnecessary loader constraints produced when there are multiple defaults
hseigel
parents:
47216
diff
changeset
|
491 |
// to link to the first super, and we get all the others. |
1 | 492 |
Handle super_loader(THREAD, super_klass->class_loader()); |
493 |
||
49658 | 494 |
if (!oopDesc::equals(target_loader(), super_loader())) { |
1 | 495 |
ResourceMark rm(THREAD); |
16591
73047b0eb568
8009731: Confusing error message for loader constraint violation
acorn
parents:
15591
diff
changeset
|
496 |
Symbol* failed_type_symbol = |
2264
55d0115a54fe
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
2136
diff
changeset
|
497 |
SystemDictionary::check_signature_loaders(signature, target_loader, |
1 | 498 |
super_loader, true, |
499 |
CHECK_(false)); |
|
16591
73047b0eb568
8009731: Confusing error message for loader constraint violation
acorn
parents:
15591
diff
changeset
|
500 |
if (failed_type_symbol != NULL) { |
51097
bef02342d179
8205611: Improve the wording of LinkageErrors to include module and class loader information
lfoltan
parents:
51096
diff
changeset
|
501 |
stringStream ss; |
bef02342d179
8205611: Improve the wording of LinkageErrors to include module and class loader information
lfoltan
parents:
51096
diff
changeset
|
502 |
ss.print("loader constraint violation for class %s: when selecting " |
54432
532e88de77eb
8221470: Print methods in exception messages in java-like Syntax.
goetz
parents:
53904
diff
changeset
|
503 |
"overriding method '", klass->external_name()); |
532e88de77eb
8221470: Print methods in exception messages in java-like Syntax.
goetz
parents:
53904
diff
changeset
|
504 |
target_method()->print_external_name(&ss), |
532e88de77eb
8221470: Print methods in exception messages in java-like Syntax.
goetz
parents:
53904
diff
changeset
|
505 |
ss.print("' the class loader %s of the " |
51097
bef02342d179
8205611: Improve the wording of LinkageErrors to include module and class loader information
lfoltan
parents:
51096
diff
changeset
|
506 |
"selected method's type %s, and the class loader %s for its super " |
bef02342d179
8205611: Improve the wording of LinkageErrors to include module and class loader information
lfoltan
parents:
51096
diff
changeset
|
507 |
"type %s have different Class objects for the type %s used in the signature (%s; %s)", |
bef02342d179
8205611: Improve the wording of LinkageErrors to include module and class loader information
lfoltan
parents:
51096
diff
changeset
|
508 |
target_klass->class_loader_data()->loader_name_and_id(), |
bef02342d179
8205611: Improve the wording of LinkageErrors to include module and class loader information
lfoltan
parents:
51096
diff
changeset
|
509 |
target_klass->external_name(), |
bef02342d179
8205611: Improve the wording of LinkageErrors to include module and class loader information
lfoltan
parents:
51096
diff
changeset
|
510 |
super_klass->class_loader_data()->loader_name_and_id(), |
bef02342d179
8205611: Improve the wording of LinkageErrors to include module and class loader information
lfoltan
parents:
51096
diff
changeset
|
511 |
super_klass->external_name(), |
bef02342d179
8205611: Improve the wording of LinkageErrors to include module and class loader information
lfoltan
parents:
51096
diff
changeset
|
512 |
failed_type_symbol->as_klass_external_name(), |
bef02342d179
8205611: Improve the wording of LinkageErrors to include module and class loader information
lfoltan
parents:
51096
diff
changeset
|
513 |
target_klass->class_in_module_of_loader(false, true), |
bef02342d179
8205611: Improve the wording of LinkageErrors to include module and class loader information
lfoltan
parents:
51096
diff
changeset
|
514 |
super_klass->class_in_module_of_loader(false, true)); |
bef02342d179
8205611: Improve the wording of LinkageErrors to include module and class loader information
lfoltan
parents:
51096
diff
changeset
|
515 |
THROW_MSG_(vmSymbols::java_lang_LinkageError(), ss.as_string(), false); |
1 | 516 |
} |
517 |
} |
|
35463
b32e362563bb
8141564: Convert TraceItables and PrintVtables to Unified Logging
rprotacio
parents:
34666
diff
changeset
|
518 |
} |
20391
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
519 |
|
35463
b32e362563bb
8141564: Convert TraceItables and PrintVtables to Unified Logging
rprotacio
parents:
34666
diff
changeset
|
520 |
put_method_at(target_method(), i); |
b32e362563bb
8141564: Convert TraceItables and PrintVtables to Unified Logging
rprotacio
parents:
34666
diff
changeset
|
521 |
overrides = true; |
b32e362563bb
8141564: Convert TraceItables and PrintVtables to Unified Logging
rprotacio
parents:
34666
diff
changeset
|
522 |
if (!is_default) { |
b32e362563bb
8141564: Convert TraceItables and PrintVtables to Unified Logging
rprotacio
parents:
34666
diff
changeset
|
523 |
target_method()->set_vtable_index(i); |
b32e362563bb
8141564: Convert TraceItables and PrintVtables to Unified Logging
rprotacio
parents:
34666
diff
changeset
|
524 |
} else { |
b32e362563bb
8141564: Convert TraceItables and PrintVtables to Unified Logging
rprotacio
parents:
34666
diff
changeset
|
525 |
if (def_vtable_indices != NULL) { |
39714 | 526 |
if (is_preinitialized_vtable()) { |
527 |
// At runtime initialize_vtable is rerun as part of link_class_impl() |
|
528 |
// for a shared class loaded by the non-boot loader. |
|
529 |
// The dumptime vtable index should be the same as the runtime index. |
|
530 |
assert(def_vtable_indices->at(default_index) == i, |
|
531 |
"dump time vtable index is different from runtime index"); |
|
532 |
} else { |
|
533 |
def_vtable_indices->at_put(default_index, i); |
|
534 |
} |
|
35463
b32e362563bb
8141564: Convert TraceItables and PrintVtables to Unified Logging
rprotacio
parents:
34666
diff
changeset
|
535 |
} |
b32e362563bb
8141564: Convert TraceItables and PrintVtables to Unified Logging
rprotacio
parents:
34666
diff
changeset
|
536 |
assert(super_method->is_default_method() || super_method->is_overpass() |
b32e362563bb
8141564: Convert TraceItables and PrintVtables to Unified Logging
rprotacio
parents:
34666
diff
changeset
|
537 |
|| super_method->is_abstract(), "default override error"); |
2264
55d0115a54fe
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
2136
diff
changeset
|
538 |
} |
55d0115a54fe
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
2136
diff
changeset
|
539 |
} else { |
35463
b32e362563bb
8141564: Convert TraceItables and PrintVtables to Unified Logging
rprotacio
parents:
34666
diff
changeset
|
540 |
overrides = false; |
1 | 541 |
} |
35463
b32e362563bb
8141564: Convert TraceItables and PrintVtables to Unified Logging
rprotacio
parents:
34666
diff
changeset
|
542 |
log_vtables(i, overrides, target_method, target_klass, super_method, THREAD); |
1 | 543 |
} |
544 |
} |
|
545 |
return allocate_new; |
|
546 |
} |
|
547 |
||
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
9172
diff
changeset
|
548 |
void klassVtable::put_method_at(Method* m, int index) { |
52276
ad9077f044be
8024368: private methods are allocated vtable slots
sspitsyn
parents:
51959
diff
changeset
|
549 |
assert(!m->is_private(), "private methods should not be in vtable"); |
39714 | 550 |
if (is_preinitialized_vtable()) { |
551 |
// At runtime initialize_vtable is rerun as part of link_class_impl() |
|
552 |
// for shared class loaded by the non-boot loader to obtain the loader |
|
553 |
// constraints based on the runtime classloaders' context. The dumptime |
|
554 |
// method at the vtable index should be the same as the runtime method. |
|
555 |
assert(table()[index].method() == m, |
|
556 |
"archived method is different from the runtime method"); |
|
557 |
} else { |
|
558 |
if (log_develop_is_enabled(Trace, vtables)) { |
|
559 |
ResourceMark rm; |
|
46701
f559541c0daa
8181917: Refactor UL LogStreams to avoid using resource area
stuefe
parents:
46560
diff
changeset
|
560 |
LogTarget(Trace, vtables) lt; |
f559541c0daa
8181917: Refactor UL LogStreams to avoid using resource area
stuefe
parents:
46560
diff
changeset
|
561 |
LogStream ls(lt); |
39714 | 562 |
const char* sig = (m != NULL) ? m->name_and_sig_as_C_string() : "<NULL>"; |
46701
f559541c0daa
8181917: Refactor UL LogStreams to avoid using resource area
stuefe
parents:
46560
diff
changeset
|
563 |
ls.print("adding %s at index %d, flags: ", sig, index); |
39714 | 564 |
if (m != NULL) { |
46701
f559541c0daa
8181917: Refactor UL LogStreams to avoid using resource area
stuefe
parents:
46560
diff
changeset
|
565 |
m->print_linkage_flags(&ls); |
39714 | 566 |
} |
46701
f559541c0daa
8181917: Refactor UL LogStreams to avoid using resource area
stuefe
parents:
46560
diff
changeset
|
567 |
ls.cr(); |
21556
e75cd34a59e0
8027229: ICCE expected for >=2 maximally specific default methods.
acorn
parents:
20391
diff
changeset
|
568 |
} |
39714 | 569 |
table()[index].set(m); |
1 | 570 |
} |
571 |
} |
|
572 |
||
573 |
// Find out if a method "m" with superclass "super", loader "classloader" and |
|
574 |
// name "classname" needs a new vtable entry. Let P be a class package defined |
|
575 |
// by "classloader" and "classname". |
|
576 |
// NOTE: The logic used here is very similar to the one used for computing |
|
577 |
// the vtables indices for a method. We cannot directly use that function because, |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
9172
diff
changeset
|
578 |
// we allocate the InstanceKlass at load time, and that requires that the |
2264
55d0115a54fe
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
2136
diff
changeset
|
579 |
// superclass has been loaded. |
55d0115a54fe
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
2136
diff
changeset
|
580 |
// However, the vtable entries are filled in at link time, and therefore |
55d0115a54fe
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
2136
diff
changeset
|
581 |
// the superclass' vtable may not yet have been filled in. |
46727
6e4a84748e2c
8183039: Re-examine methodHandle methods uninlined by 8144256
coleenp
parents:
46701
diff
changeset
|
582 |
bool klassVtable::needs_new_vtable_entry(const methodHandle& target_method, |
34666 | 583 |
const Klass* super, |
2264
55d0115a54fe
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
2136
diff
changeset
|
584 |
Handle classloader, |
8076
96d498ec7ae1
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
7443
diff
changeset
|
585 |
Symbol* classname, |
2264
55d0115a54fe
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
2136
diff
changeset
|
586 |
AccessFlags class_flags, |
40633
c33ad2ff51de
8163808: Fix asserts and logging for old classfile vtable construction
acorn
parents:
39714
diff
changeset
|
587 |
u2 major_version, |
2264
55d0115a54fe
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
2136
diff
changeset
|
588 |
TRAPS) { |
20017
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
589 |
if (class_flags.is_interface()) { |
25504
69de8357bc53
8032536: JVM resolves wrong method in some unusual cases
hseigel
parents:
24424
diff
changeset
|
590 |
// Interfaces do not use vtables, except for java.lang.Object methods, |
69de8357bc53
8032536: JVM resolves wrong method in some unusual cases
hseigel
parents:
24424
diff
changeset
|
591 |
// so there is no point to assigning |
69de8357bc53
8032536: JVM resolves wrong method in some unusual cases
hseigel
parents:
24424
diff
changeset
|
592 |
// a vtable index to any of their local methods. If we refrain from doing this, |
20017
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
593 |
// we can use Method::_vtable_index to hold the itable index |
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
594 |
return false; |
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
595 |
} |
14385 | 596 |
|
20017
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
597 |
if (target_method->is_final_method(class_flags) || |
1 | 598 |
// a final method never needs a new entry; final methods can be statically |
599 |
// resolved and they have to be present in the vtable only if they override |
|
600 |
// a super's method, in which case they re-use its entry |
|
52276
ad9077f044be
8024368: private methods are allocated vtable slots
sspitsyn
parents:
51959
diff
changeset
|
601 |
(target_method()->is_private()) || |
ad9077f044be
8024368: private methods are allocated vtable slots
sspitsyn
parents:
51959
diff
changeset
|
602 |
// private methods don't need to be in vtable |
2264
55d0115a54fe
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
2136
diff
changeset
|
603 |
(target_method()->is_static()) || |
1 | 604 |
// static methods don't need to be in vtable |
52276
ad9077f044be
8024368: private methods are allocated vtable slots
sspitsyn
parents:
51959
diff
changeset
|
605 |
(target_method()->name()->fast_compare(vmSymbols::object_initializer_name()) == 0) |
1 | 606 |
// <init> is never called dynamically-bound |
607 |
) { |
|
608 |
return false; |
|
609 |
} |
|
610 |
||
20391
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
611 |
// Concrete interface methods do not need new entries, they override |
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
612 |
// abstract method entries using default inheritance rules |
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
613 |
if (target_method()->method_holder() != NULL && |
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
614 |
target_method()->method_holder()->is_interface() && |
41669
2091069b6851
8081800: AbstractMethodError when evaluating a private method in an interface via debugger
dholmes
parents:
40633
diff
changeset
|
615 |
!target_method()->is_abstract()) { |
52276
ad9077f044be
8024368: private methods are allocated vtable slots
sspitsyn
parents:
51959
diff
changeset
|
616 |
assert(target_method()->is_default_method(), |
41669
2091069b6851
8081800: AbstractMethodError when evaluating a private method in an interface via debugger
dholmes
parents:
40633
diff
changeset
|
617 |
"unexpected interface method type"); |
20391
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
618 |
return false; |
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
619 |
} |
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
620 |
|
1 | 621 |
// we need a new entry if there is no superclass |
622 |
if (super == NULL) { |
|
623 |
return true; |
|
624 |
} |
|
625 |
||
25504
69de8357bc53
8032536: JVM resolves wrong method in some unusual cases
hseigel
parents:
24424
diff
changeset
|
626 |
// Package private methods always need a new entry to root their own |
69de8357bc53
8032536: JVM resolves wrong method in some unusual cases
hseigel
parents:
24424
diff
changeset
|
627 |
// overriding. This allows transitive overriding to work. |
69de8357bc53
8032536: JVM resolves wrong method in some unusual cases
hseigel
parents:
24424
diff
changeset
|
628 |
if (target_method()->is_package_private()) { |
69de8357bc53
8032536: JVM resolves wrong method in some unusual cases
hseigel
parents:
24424
diff
changeset
|
629 |
return true; |
69de8357bc53
8032536: JVM resolves wrong method in some unusual cases
hseigel
parents:
24424
diff
changeset
|
630 |
} |
69de8357bc53
8032536: JVM resolves wrong method in some unusual cases
hseigel
parents:
24424
diff
changeset
|
631 |
|
1 | 632 |
// search through the super class hierarchy to see if we need |
633 |
// a new entry |
|
2264
55d0115a54fe
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
2136
diff
changeset
|
634 |
ResourceMark rm; |
8076
96d498ec7ae1
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
7443
diff
changeset
|
635 |
Symbol* name = target_method()->name(); |
96d498ec7ae1
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
7443
diff
changeset
|
636 |
Symbol* signature = target_method()->signature(); |
34666 | 637 |
const Klass* k = super; |
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
9172
diff
changeset
|
638 |
Method* super_method = NULL; |
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
9172
diff
changeset
|
639 |
InstanceKlass *holder = NULL; |
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
9172
diff
changeset
|
640 |
Method* recheck_method = NULL; |
55655
419420eb5230
8226798: JVM crash in klassItable::initialize_itable_for_interface(int, InstanceKlass*, bool, Thread*)
hseigel
parents:
54432
diff
changeset
|
641 |
bool found_pkg_prvt_method = false; |
1 | 642 |
while (k != NULL) { |
643 |
// lookup through the hierarchy for a method with matching name and sign. |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
9172
diff
changeset
|
644 |
super_method = InstanceKlass::cast(k)->lookup_method(name, signature); |
2264
55d0115a54fe
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
2136
diff
changeset
|
645 |
if (super_method == NULL) { |
1 | 646 |
break; // we still have to search for a matching miranda method |
647 |
} |
|
648 |
// get the class holding the matching method |
|
2264
55d0115a54fe
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
2136
diff
changeset
|
649 |
// make sure you use that class for is_override |
14391
df0a1573d5bd
8000725: NPG: method_holder() and pool_holder() and pool_holder field should be InstanceKlass
coleenp
parents:
14385
diff
changeset
|
650 |
InstanceKlass* superk = super_method->method_holder(); |
2264
55d0115a54fe
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
2136
diff
changeset
|
651 |
// we want only instance method matches |
52276
ad9077f044be
8024368: private methods are allocated vtable slots
sspitsyn
parents:
51959
diff
changeset
|
652 |
// ignore private methods found via lookup_method since they do not participate in overriding, |
ad9077f044be
8024368: private methods are allocated vtable slots
sspitsyn
parents:
51959
diff
changeset
|
653 |
// and since we do override around them: e.g. a.m pub/b.m private/c.m pub, |
2264
55d0115a54fe
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
2136
diff
changeset
|
654 |
// ignore private, c.m pub does override a.m pub |
55d0115a54fe
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
2136
diff
changeset
|
655 |
// For classes that were not javac'd together, we also do transitive overriding around |
55d0115a54fe
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
2136
diff
changeset
|
656 |
// methods that have less accessibility |
55d0115a54fe
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
2136
diff
changeset
|
657 |
if ((!super_method->is_static()) && |
55d0115a54fe
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
2136
diff
changeset
|
658 |
(!super_method->is_private())) { |
55d0115a54fe
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
2136
diff
changeset
|
659 |
if (superk->is_override(super_method, classloader, classname, THREAD)) { |
1 | 660 |
return false; |
2264
55d0115a54fe
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
2136
diff
changeset
|
661 |
// else keep looking for transitive overrides |
1 | 662 |
} |
55655
419420eb5230
8226798: JVM crash in klassItable::initialize_itable_for_interface(int, InstanceKlass*, bool, Thread*)
hseigel
parents:
54432
diff
changeset
|
663 |
// If we get here then one of the super classes has a package private method |
419420eb5230
8226798: JVM crash in klassItable::initialize_itable_for_interface(int, InstanceKlass*, bool, Thread*)
hseigel
parents:
54432
diff
changeset
|
664 |
// that will not get overridden because it is in a different package. But, |
419420eb5230
8226798: JVM crash in klassItable::initialize_itable_for_interface(int, InstanceKlass*, bool, Thread*)
hseigel
parents:
54432
diff
changeset
|
665 |
// that package private method does "override" any matching methods in super |
419420eb5230
8226798: JVM crash in klassItable::initialize_itable_for_interface(int, InstanceKlass*, bool, Thread*)
hseigel
parents:
54432
diff
changeset
|
666 |
// interfaces, so there will be no miranda vtable entry created. So, set flag |
419420eb5230
8226798: JVM crash in klassItable::initialize_itable_for_interface(int, InstanceKlass*, bool, Thread*)
hseigel
parents:
54432
diff
changeset
|
667 |
// to TRUE for use below, in case there are no methods in super classes that |
419420eb5230
8226798: JVM crash in klassItable::initialize_itable_for_interface(int, InstanceKlass*, bool, Thread*)
hseigel
parents:
54432
diff
changeset
|
668 |
// this target method overrides. |
419420eb5230
8226798: JVM crash in klassItable::initialize_itable_for_interface(int, InstanceKlass*, bool, Thread*)
hseigel
parents:
54432
diff
changeset
|
669 |
assert(super_method->is_package_private(), "super_method must be package private"); |
419420eb5230
8226798: JVM crash in klassItable::initialize_itable_for_interface(int, InstanceKlass*, bool, Thread*)
hseigel
parents:
54432
diff
changeset
|
670 |
assert(!superk->is_same_class_package(classloader(), classname), |
419420eb5230
8226798: JVM crash in klassItable::initialize_itable_for_interface(int, InstanceKlass*, bool, Thread*)
hseigel
parents:
54432
diff
changeset
|
671 |
"Must be different packages"); |
419420eb5230
8226798: JVM crash in klassItable::initialize_itable_for_interface(int, InstanceKlass*, bool, Thread*)
hseigel
parents:
54432
diff
changeset
|
672 |
found_pkg_prvt_method = true; |
1 | 673 |
} |
674 |
||
40633
c33ad2ff51de
8163808: Fix asserts and logging for old classfile vtable construction
acorn
parents:
39714
diff
changeset
|
675 |
// Start with lookup result and continue to search up, for versions supporting transitive override |
c33ad2ff51de
8163808: Fix asserts and logging for old classfile vtable construction
acorn
parents:
39714
diff
changeset
|
676 |
if (major_version >= VTABLE_TRANSITIVE_OVERRIDE_VERSION) { |
c33ad2ff51de
8163808: Fix asserts and logging for old classfile vtable construction
acorn
parents:
39714
diff
changeset
|
677 |
k = superk->super(); // haven't found an override match yet; continue to look |
c33ad2ff51de
8163808: Fix asserts and logging for old classfile vtable construction
acorn
parents:
39714
diff
changeset
|
678 |
} else { |
c33ad2ff51de
8163808: Fix asserts and logging for old classfile vtable construction
acorn
parents:
39714
diff
changeset
|
679 |
break; |
c33ad2ff51de
8163808: Fix asserts and logging for old classfile vtable construction
acorn
parents:
39714
diff
changeset
|
680 |
} |
1 | 681 |
} |
682 |
||
55655
419420eb5230
8226798: JVM crash in klassItable::initialize_itable_for_interface(int, InstanceKlass*, bool, Thread*)
hseigel
parents:
54432
diff
changeset
|
683 |
// If found_pkg_prvt_method is set, then the ONLY matching method in the |
419420eb5230
8226798: JVM crash in klassItable::initialize_itable_for_interface(int, InstanceKlass*, bool, Thread*)
hseigel
parents:
54432
diff
changeset
|
684 |
// superclasses is package private in another package. That matching method will |
419420eb5230
8226798: JVM crash in klassItable::initialize_itable_for_interface(int, InstanceKlass*, bool, Thread*)
hseigel
parents:
54432
diff
changeset
|
685 |
// prevent a miranda vtable entry from being created. Because the target method can not |
419420eb5230
8226798: JVM crash in klassItable::initialize_itable_for_interface(int, InstanceKlass*, bool, Thread*)
hseigel
parents:
54432
diff
changeset
|
686 |
// override the package private method in another package, then it needs to be the root |
419420eb5230
8226798: JVM crash in klassItable::initialize_itable_for_interface(int, InstanceKlass*, bool, Thread*)
hseigel
parents:
54432
diff
changeset
|
687 |
// for its own vtable entry. |
419420eb5230
8226798: JVM crash in klassItable::initialize_itable_for_interface(int, InstanceKlass*, bool, Thread*)
hseigel
parents:
54432
diff
changeset
|
688 |
if (found_pkg_prvt_method) { |
419420eb5230
8226798: JVM crash in klassItable::initialize_itable_for_interface(int, InstanceKlass*, bool, Thread*)
hseigel
parents:
54432
diff
changeset
|
689 |
return true; |
419420eb5230
8226798: JVM crash in klassItable::initialize_itable_for_interface(int, InstanceKlass*, bool, Thread*)
hseigel
parents:
54432
diff
changeset
|
690 |
} |
419420eb5230
8226798: JVM crash in klassItable::initialize_itable_for_interface(int, InstanceKlass*, bool, Thread*)
hseigel
parents:
54432
diff
changeset
|
691 |
|
1 | 692 |
// if the target method is public or protected it may have a matching |
693 |
// miranda method in the super, whose entry it should re-use. |
|
2264
55d0115a54fe
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
2136
diff
changeset
|
694 |
// Actually, to handle cases that javac would not generate, we need |
55d0115a54fe
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
2136
diff
changeset
|
695 |
// this check for all access permissions. |
34666 | 696 |
const InstanceKlass *sk = InstanceKlass::cast(super); |
2264
55d0115a54fe
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
2136
diff
changeset
|
697 |
if (sk->has_miranda_methods()) { |
28731
f7339cba0a6a
8067480: Crash in klassItable::initialize_itable_for_interface when running vm.runtime.defmeth.StaticMethodsTest.
lfoltan
parents:
28373
diff
changeset
|
698 |
if (sk->lookup_method_in_all_interfaces(name, signature, Klass::find_defaults) != NULL) { |
55655
419420eb5230
8226798: JVM crash in klassItable::initialize_itable_for_interface(int, InstanceKlass*, bool, Thread*)
hseigel
parents:
54432
diff
changeset
|
699 |
return false; // found a matching miranda; we do not need a new entry |
1 | 700 |
} |
701 |
} |
|
702 |
return true; // found no match; we need a new entry |
|
703 |
} |
|
704 |
||
705 |
// Support for miranda methods |
|
706 |
||
707 |
// get the vtable index of a miranda method with matching "name" and "signature" |
|
8076
96d498ec7ae1
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
7443
diff
changeset
|
708 |
int klassVtable::index_of_miranda(Symbol* name, Symbol* signature) { |
1 | 709 |
// search from the bottom, might be faster |
710 |
for (int i = (length() - 1); i >= 0; i--) { |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
9172
diff
changeset
|
711 |
Method* m = table()[i].method(); |
1 | 712 |
if (is_miranda_entry_at(i) && |
713 |
m->name() == name && m->signature() == signature) { |
|
714 |
return i; |
|
715 |
} |
|
716 |
} |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
9172
diff
changeset
|
717 |
return Method::invalid_vtable_index; |
1 | 718 |
} |
719 |
||
20017
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
720 |
// check if an entry at an index is miranda |
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
721 |
// requires that method m at entry be declared ("held") by an interface. |
1 | 722 |
bool klassVtable::is_miranda_entry_at(int i) { |
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
9172
diff
changeset
|
723 |
Method* m = method_at(i); |
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
9172
diff
changeset
|
724 |
Klass* method_holder = m->method_holder(); |
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
9172
diff
changeset
|
725 |
InstanceKlass *mhk = InstanceKlass::cast(method_holder); |
1 | 726 |
|
20284
595a25ab9474
8011311: Private interface methods. Default conflicts:ICCE. no erased_super_default.
acorn
parents:
20017
diff
changeset
|
727 |
// miranda methods are public abstract instance interface methods in a class's vtable |
1 | 728 |
if (mhk->is_interface()) { |
14385 | 729 |
assert(m->is_public(), "should be public"); |
1 | 730 |
assert(ik()->implements_interface(method_holder) , "this class should implement the interface"); |
48463
474cec233fb2
8154587: Resolution fails for default method named 'clone'
hseigel
parents:
47765
diff
changeset
|
731 |
if (is_miranda(m, ik()->methods(), ik()->default_methods(), ik()->super(), klass()->is_interface())) { |
21556
e75cd34a59e0
8027229: ICCE expected for >=2 maximally specific default methods.
acorn
parents:
20391
diff
changeset
|
732 |
return true; |
e75cd34a59e0
8027229: ICCE expected for >=2 maximally specific default methods.
acorn
parents:
20391
diff
changeset
|
733 |
} |
1 | 734 |
} |
735 |
return false; |
|
736 |
} |
|
737 |
||
32189
5264b560ab1b
8087342: Crash in klassItable::initialize_itable_for_interface
acorn
parents:
31019
diff
changeset
|
738 |
// Check if a method is a miranda method, given a class's methods array, |
5264b560ab1b
8087342: Crash in klassItable::initialize_itable_for_interface
acorn
parents:
31019
diff
changeset
|
739 |
// its default_method table and its super class. |
5264b560ab1b
8087342: Crash in klassItable::initialize_itable_for_interface
acorn
parents:
31019
diff
changeset
|
740 |
// "Miranda" means an abstract non-private method that would not be |
5264b560ab1b
8087342: Crash in klassItable::initialize_itable_for_interface
acorn
parents:
31019
diff
changeset
|
741 |
// overridden for the local class. |
5264b560ab1b
8087342: Crash in klassItable::initialize_itable_for_interface
acorn
parents:
31019
diff
changeset
|
742 |
// A "miranda" method should only include non-private interface |
5264b560ab1b
8087342: Crash in klassItable::initialize_itable_for_interface
acorn
parents:
31019
diff
changeset
|
743 |
// instance methods, i.e. not private methods, not static methods, |
5264b560ab1b
8087342: Crash in klassItable::initialize_itable_for_interface
acorn
parents:
31019
diff
changeset
|
744 |
// not default methods (concrete interface methods), not overpass methods. |
5264b560ab1b
8087342: Crash in klassItable::initialize_itable_for_interface
acorn
parents:
31019
diff
changeset
|
745 |
// If a given class already has a local (including overpass) method, a |
5264b560ab1b
8087342: Crash in klassItable::initialize_itable_for_interface
acorn
parents:
31019
diff
changeset
|
746 |
// default method, or any of its superclasses has the same which would have |
5264b560ab1b
8087342: Crash in klassItable::initialize_itable_for_interface
acorn
parents:
31019
diff
changeset
|
747 |
// overridden an abstract method, then this is not a miranda method. |
5264b560ab1b
8087342: Crash in klassItable::initialize_itable_for_interface
acorn
parents:
31019
diff
changeset
|
748 |
// |
5264b560ab1b
8087342: Crash in klassItable::initialize_itable_for_interface
acorn
parents:
31019
diff
changeset
|
749 |
// Miranda methods are checked multiple times. |
5264b560ab1b
8087342: Crash in klassItable::initialize_itable_for_interface
acorn
parents:
31019
diff
changeset
|
750 |
// Pass 1: during class load/class file parsing: before vtable size calculation: |
5264b560ab1b
8087342: Crash in klassItable::initialize_itable_for_interface
acorn
parents:
31019
diff
changeset
|
751 |
// include superinterface abstract and default methods (non-private instance). |
29871 | 752 |
// We include potential default methods to give them space in the vtable. |
32189
5264b560ab1b
8087342: Crash in klassItable::initialize_itable_for_interface
acorn
parents:
31019
diff
changeset
|
753 |
// During the first run, the current instanceKlass has not yet been |
5264b560ab1b
8087342: Crash in klassItable::initialize_itable_for_interface
acorn
parents:
31019
diff
changeset
|
754 |
// created, the superclasses and superinterfaces do have instanceKlasses |
5264b560ab1b
8087342: Crash in klassItable::initialize_itable_for_interface
acorn
parents:
31019
diff
changeset
|
755 |
// but may not have vtables, the default_methods list is empty, no overpasses. |
48463
474cec233fb2
8154587: Resolution fails for default method named 'clone'
hseigel
parents:
47765
diff
changeset
|
756 |
// Default method generation uses the all_mirandas array as the starter set for |
474cec233fb2
8154587: Resolution fails for default method named 'clone'
hseigel
parents:
47765
diff
changeset
|
757 |
// maximally-specific default method calculation. So, for both classes and |
474cec233fb2
8154587: Resolution fails for default method named 'clone'
hseigel
parents:
47765
diff
changeset
|
758 |
// interfaces, it is necessary that the first pass will find all non-private |
474cec233fb2
8154587: Resolution fails for default method named 'clone'
hseigel
parents:
47765
diff
changeset
|
759 |
// interface instance methods, whether or not they are concrete. |
32189
5264b560ab1b
8087342: Crash in klassItable::initialize_itable_for_interface
acorn
parents:
31019
diff
changeset
|
760 |
// |
5264b560ab1b
8087342: Crash in klassItable::initialize_itable_for_interface
acorn
parents:
31019
diff
changeset
|
761 |
// Pass 2: recalculated during vtable initialization: only include abstract methods. |
5264b560ab1b
8087342: Crash in klassItable::initialize_itable_for_interface
acorn
parents:
31019
diff
changeset
|
762 |
// The goal of pass 2 is to walk through the superinterfaces to see if any of |
5264b560ab1b
8087342: Crash in klassItable::initialize_itable_for_interface
acorn
parents:
31019
diff
changeset
|
763 |
// the superinterface methods (which were all abstract pre-default methods) |
5264b560ab1b
8087342: Crash in klassItable::initialize_itable_for_interface
acorn
parents:
31019
diff
changeset
|
764 |
// need to be added to the vtable. |
5264b560ab1b
8087342: Crash in klassItable::initialize_itable_for_interface
acorn
parents:
31019
diff
changeset
|
765 |
// With the addition of default methods, we have three new challenges: |
5264b560ab1b
8087342: Crash in klassItable::initialize_itable_for_interface
acorn
parents:
31019
diff
changeset
|
766 |
// overpasses, static interface methods and private interface methods. |
5264b560ab1b
8087342: Crash in klassItable::initialize_itable_for_interface
acorn
parents:
31019
diff
changeset
|
767 |
// Static and private interface methods do not get added to the vtable and |
5264b560ab1b
8087342: Crash in klassItable::initialize_itable_for_interface
acorn
parents:
31019
diff
changeset
|
768 |
// are not seen by the method resolution process, so we skip those. |
5264b560ab1b
8087342: Crash in klassItable::initialize_itable_for_interface
acorn
parents:
31019
diff
changeset
|
769 |
// Overpass methods are already in the vtable, so vtable lookup will |
5264b560ab1b
8087342: Crash in klassItable::initialize_itable_for_interface
acorn
parents:
31019
diff
changeset
|
770 |
// find them and we don't need to add a miranda method to the end of |
5264b560ab1b
8087342: Crash in klassItable::initialize_itable_for_interface
acorn
parents:
31019
diff
changeset
|
771 |
// the vtable. So we look for overpass methods and if they are found we |
5264b560ab1b
8087342: Crash in klassItable::initialize_itable_for_interface
acorn
parents:
31019
diff
changeset
|
772 |
// return false. Note that we inherit our superclasses vtable, so |
5264b560ab1b
8087342: Crash in klassItable::initialize_itable_for_interface
acorn
parents:
31019
diff
changeset
|
773 |
// the superclass' search also needs to use find_overpass so that if |
5264b560ab1b
8087342: Crash in klassItable::initialize_itable_for_interface
acorn
parents:
31019
diff
changeset
|
774 |
// one is found we return false. |
5264b560ab1b
8087342: Crash in klassItable::initialize_itable_for_interface
acorn
parents:
31019
diff
changeset
|
775 |
// False means - we don't need a miranda method added to the vtable. |
5264b560ab1b
8087342: Crash in klassItable::initialize_itable_for_interface
acorn
parents:
31019
diff
changeset
|
776 |
// |
29871 | 777 |
// During the second run, default_methods is set up, so concrete methods from |
778 |
// superinterfaces with matching names/signatures to default_methods are already |
|
779 |
// in the default_methods list and do not need to be appended to the vtable |
|
32189
5264b560ab1b
8087342: Crash in klassItable::initialize_itable_for_interface
acorn
parents:
31019
diff
changeset
|
780 |
// as mirandas. Abstract methods may already have been handled via |
5264b560ab1b
8087342: Crash in klassItable::initialize_itable_for_interface
acorn
parents:
31019
diff
changeset
|
781 |
// overpasses - either local or superclass overpasses, which may be |
5264b560ab1b
8087342: Crash in klassItable::initialize_itable_for_interface
acorn
parents:
31019
diff
changeset
|
782 |
// in the vtable already. |
5264b560ab1b
8087342: Crash in klassItable::initialize_itable_for_interface
acorn
parents:
31019
diff
changeset
|
783 |
// |
5264b560ab1b
8087342: Crash in klassItable::initialize_itable_for_interface
acorn
parents:
31019
diff
changeset
|
784 |
// Pass 3: They are also checked by link resolution and selection, |
5264b560ab1b
8087342: Crash in klassItable::initialize_itable_for_interface
acorn
parents:
31019
diff
changeset
|
785 |
// for invocation on a method (not interface method) reference that |
5264b560ab1b
8087342: Crash in klassItable::initialize_itable_for_interface
acorn
parents:
31019
diff
changeset
|
786 |
// resolves to a method with an interface as its method_holder. |
5264b560ab1b
8087342: Crash in klassItable::initialize_itable_for_interface
acorn
parents:
31019
diff
changeset
|
787 |
// Used as part of walking from the bottom of the vtable to find |
5264b560ab1b
8087342: Crash in klassItable::initialize_itable_for_interface
acorn
parents:
31019
diff
changeset
|
788 |
// the vtable index for the miranda method. |
5264b560ab1b
8087342: Crash in klassItable::initialize_itable_for_interface
acorn
parents:
31019
diff
changeset
|
789 |
// |
5264b560ab1b
8087342: Crash in klassItable::initialize_itable_for_interface
acorn
parents:
31019
diff
changeset
|
790 |
// Part of the Miranda Rights in the US mean that if you do not have |
5264b560ab1b
8087342: Crash in klassItable::initialize_itable_for_interface
acorn
parents:
31019
diff
changeset
|
791 |
// an attorney one will be appointed for you. |
20391
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
792 |
bool klassVtable::is_miranda(Method* m, Array<Method*>* class_methods, |
48463
474cec233fb2
8154587: Resolution fails for default method named 'clone'
hseigel
parents:
47765
diff
changeset
|
793 |
Array<Method*>* default_methods, const Klass* super, |
474cec233fb2
8154587: Resolution fails for default method named 'clone'
hseigel
parents:
47765
diff
changeset
|
794 |
bool is_interface) { |
21556
e75cd34a59e0
8027229: ICCE expected for >=2 maximally specific default methods.
acorn
parents:
20391
diff
changeset
|
795 |
if (m->is_static() || m->is_private() || m->is_overpass()) { |
17092
8d0bef547de9
8013418: assert(i == total_args_passed) in AdapterHandlerLibrary::get_adapter since 8-b87
bharadwaj
parents:
16591
diff
changeset
|
796 |
return false; |
8d0bef547de9
8013418: assert(i == total_args_passed) in AdapterHandlerLibrary::get_adapter since 8-b87
bharadwaj
parents:
16591
diff
changeset
|
797 |
} |
8076
96d498ec7ae1
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
7443
diff
changeset
|
798 |
Symbol* name = m->name(); |
96d498ec7ae1
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
7443
diff
changeset
|
799 |
Symbol* signature = m->signature(); |
21913
0e2fd7282ac6
8028438: static superclass method masks default methods
acorn
parents:
21770
diff
changeset
|
800 |
|
32189
5264b560ab1b
8087342: Crash in klassItable::initialize_itable_for_interface
acorn
parents:
31019
diff
changeset
|
801 |
// First look in local methods to see if already covered |
5264b560ab1b
8087342: Crash in klassItable::initialize_itable_for_interface
acorn
parents:
31019
diff
changeset
|
802 |
if (InstanceKlass::find_local_method(class_methods, name, signature, |
5264b560ab1b
8087342: Crash in klassItable::initialize_itable_for_interface
acorn
parents:
31019
diff
changeset
|
803 |
Klass::find_overpass, Klass::skip_static, Klass::skip_private) != NULL) |
5264b560ab1b
8087342: Crash in klassItable::initialize_itable_for_interface
acorn
parents:
31019
diff
changeset
|
804 |
{ |
5264b560ab1b
8087342: Crash in klassItable::initialize_itable_for_interface
acorn
parents:
31019
diff
changeset
|
805 |
return false; |
1 | 806 |
} |
7443
39c85d63abf2
6989076: JVM crashes in klassItable::initialize_itable_for_interface
dsamersoff
parents:
7397
diff
changeset
|
807 |
|
32189
5264b560ab1b
8087342: Crash in klassItable::initialize_itable_for_interface
acorn
parents:
31019
diff
changeset
|
808 |
// Check local default methods |
5264b560ab1b
8087342: Crash in klassItable::initialize_itable_for_interface
acorn
parents:
31019
diff
changeset
|
809 |
if ((default_methods != NULL) && |
5264b560ab1b
8087342: Crash in klassItable::initialize_itable_for_interface
acorn
parents:
31019
diff
changeset
|
810 |
(InstanceKlass::find_method(default_methods, name, signature) != NULL)) |
5264b560ab1b
8087342: Crash in klassItable::initialize_itable_for_interface
acorn
parents:
31019
diff
changeset
|
811 |
{ |
5264b560ab1b
8087342: Crash in klassItable::initialize_itable_for_interface
acorn
parents:
31019
diff
changeset
|
812 |
return false; |
5264b560ab1b
8087342: Crash in klassItable::initialize_itable_for_interface
acorn
parents:
31019
diff
changeset
|
813 |
} |
5264b560ab1b
8087342: Crash in klassItable::initialize_itable_for_interface
acorn
parents:
31019
diff
changeset
|
814 |
|
33602 | 815 |
// Iterate on all superclasses, which should be InstanceKlasses. |
32189
5264b560ab1b
8087342: Crash in klassItable::initialize_itable_for_interface
acorn
parents:
31019
diff
changeset
|
816 |
// Note that we explicitly look for overpasses at each level. |
5264b560ab1b
8087342: Crash in klassItable::initialize_itable_for_interface
acorn
parents:
31019
diff
changeset
|
817 |
// Overpasses may or may not exist for supers for pass 1, |
5264b560ab1b
8087342: Crash in klassItable::initialize_itable_for_interface
acorn
parents:
31019
diff
changeset
|
818 |
// they should have been created for pass 2 and later. |
5264b560ab1b
8087342: Crash in klassItable::initialize_itable_for_interface
acorn
parents:
31019
diff
changeset
|
819 |
|
34666 | 820 |
for (const Klass* cursuper = super; cursuper != NULL; cursuper = cursuper->super()) |
32189
5264b560ab1b
8087342: Crash in klassItable::initialize_itable_for_interface
acorn
parents:
31019
diff
changeset
|
821 |
{ |
48463
474cec233fb2
8154587: Resolution fails for default method named 'clone'
hseigel
parents:
47765
diff
changeset
|
822 |
Method* found_mth = InstanceKlass::cast(cursuper)->find_local_method(name, signature, |
474cec233fb2
8154587: Resolution fails for default method named 'clone'
hseigel
parents:
47765
diff
changeset
|
823 |
Klass::find_overpass, Klass::skip_static, Klass::skip_private); |
474cec233fb2
8154587: Resolution fails for default method named 'clone'
hseigel
parents:
47765
diff
changeset
|
824 |
// Ignore non-public methods in java.lang.Object if klass is an interface. |
474cec233fb2
8154587: Resolution fails for default method named 'clone'
hseigel
parents:
47765
diff
changeset
|
825 |
if (found_mth != NULL && (!is_interface || |
474cec233fb2
8154587: Resolution fails for default method named 'clone'
hseigel
parents:
47765
diff
changeset
|
826 |
!SystemDictionary::is_nonpublic_Object_method(found_mth))) { |
32189
5264b560ab1b
8087342: Crash in klassItable::initialize_itable_for_interface
acorn
parents:
31019
diff
changeset
|
827 |
return false; |
5264b560ab1b
8087342: Crash in klassItable::initialize_itable_for_interface
acorn
parents:
31019
diff
changeset
|
828 |
} |
5264b560ab1b
8087342: Crash in klassItable::initialize_itable_for_interface
acorn
parents:
31019
diff
changeset
|
829 |
} |
5264b560ab1b
8087342: Crash in klassItable::initialize_itable_for_interface
acorn
parents:
31019
diff
changeset
|
830 |
|
5264b560ab1b
8087342: Crash in klassItable::initialize_itable_for_interface
acorn
parents:
31019
diff
changeset
|
831 |
return true; |
1 | 832 |
} |
833 |
||
20017
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
834 |
// Scans current_interface_methods for miranda methods that do not |
20391
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
835 |
// already appear in new_mirandas, or default methods, and are also not defined-and-non-private |
20017
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
836 |
// in super (superclass). These mirandas are added to all_mirandas if it is |
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
837 |
// not null; in addition, those that are not duplicates of miranda methods |
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
838 |
// inherited by super from its interfaces are added to new_mirandas. |
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
839 |
// Thus, new_mirandas will be the set of mirandas that this class introduces, |
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
840 |
// all_mirandas will be the set of all mirandas applicable to this class |
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
841 |
// including all defined in superclasses. |
14385 | 842 |
void klassVtable::add_new_mirandas_to_lists( |
843 |
GrowableArray<Method*>* new_mirandas, GrowableArray<Method*>* all_mirandas, |
|
844 |
Array<Method*>* current_interface_methods, Array<Method*>* class_methods, |
|
48463
474cec233fb2
8154587: Resolution fails for default method named 'clone'
hseigel
parents:
47765
diff
changeset
|
845 |
Array<Method*>* default_methods, const Klass* super, bool is_interface) { |
20391
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
846 |
|
1 | 847 |
// iterate thru the current interface's method to see if it a miranda |
848 |
int num_methods = current_interface_methods->length(); |
|
849 |
for (int i = 0; i < num_methods; i++) { |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
9172
diff
changeset
|
850 |
Method* im = current_interface_methods->at(i); |
1 | 851 |
bool is_duplicate = false; |
14385 | 852 |
int num_of_current_mirandas = new_mirandas->length(); |
1 | 853 |
// check for duplicate mirandas in different interfaces we implement |
854 |
for (int j = 0; j < num_of_current_mirandas; j++) { |
|
14385 | 855 |
Method* miranda = new_mirandas->at(j); |
1 | 856 |
if ((im->name() == miranda->name()) && |
857 |
(im->signature() == miranda->signature())) { |
|
858 |
is_duplicate = true; |
|
859 |
break; |
|
860 |
} |
|
861 |
} |
|
862 |
||
863 |
if (!is_duplicate) { // we don't want duplicate miranda entries in the vtable |
|
48463
474cec233fb2
8154587: Resolution fails for default method named 'clone'
hseigel
parents:
47765
diff
changeset
|
864 |
if (is_miranda(im, class_methods, default_methods, super, is_interface)) { // is it a miranda at all? |
34666 | 865 |
const InstanceKlass *sk = InstanceKlass::cast(super); |
1 | 866 |
// check if it is a duplicate of a super's miranda |
28731
f7339cba0a6a
8067480: Crash in klassItable::initialize_itable_for_interface when running vm.runtime.defmeth.StaticMethodsTest.
lfoltan
parents:
28373
diff
changeset
|
867 |
if (sk->lookup_method_in_all_interfaces(im->name(), im->signature(), Klass::find_defaults) == NULL) { |
14385 | 868 |
new_mirandas->append(im); |
869 |
} |
|
870 |
if (all_mirandas != NULL) { |
|
871 |
all_mirandas->append(im); |
|
1 | 872 |
} |
873 |
} |
|
874 |
} |
|
875 |
} |
|
876 |
} |
|
877 |
||
14385 | 878 |
void klassVtable::get_mirandas(GrowableArray<Method*>* new_mirandas, |
879 |
GrowableArray<Method*>* all_mirandas, |
|
34666 | 880 |
const Klass* super, |
881 |
Array<Method*>* class_methods, |
|
20391
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
882 |
Array<Method*>* default_methods, |
51329
9c68699bebe5
8208999: Some use of Klass* should be replaced by InstanceKlass*
iklam
parents:
51097
diff
changeset
|
883 |
Array<InstanceKlass*>* local_interfaces, |
48463
474cec233fb2
8154587: Resolution fails for default method named 'clone'
hseigel
parents:
47765
diff
changeset
|
884 |
bool is_interface) { |
14385 | 885 |
assert((new_mirandas->length() == 0) , "current mirandas must be 0"); |
1 | 886 |
|
887 |
// iterate thru the local interfaces looking for a miranda |
|
888 |
int num_local_ifs = local_interfaces->length(); |
|
889 |
for (int i = 0; i < num_local_ifs; i++) { |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
9172
diff
changeset
|
890 |
InstanceKlass *ik = InstanceKlass::cast(local_interfaces->at(i)); |
14385 | 891 |
add_new_mirandas_to_lists(new_mirandas, all_mirandas, |
20391
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
892 |
ik->methods(), class_methods, |
48463
474cec233fb2
8154587: Resolution fails for default method named 'clone'
hseigel
parents:
47765
diff
changeset
|
893 |
default_methods, super, is_interface); |
1 | 894 |
// iterate thru each local's super interfaces |
51329
9c68699bebe5
8208999: Some use of Klass* should be replaced by InstanceKlass*
iklam
parents:
51097
diff
changeset
|
895 |
Array<InstanceKlass*>* super_ifs = ik->transitive_interfaces(); |
1 | 896 |
int num_super_ifs = super_ifs->length(); |
897 |
for (int j = 0; j < num_super_ifs; j++) { |
|
51329
9c68699bebe5
8208999: Some use of Klass* should be replaced by InstanceKlass*
iklam
parents:
51097
diff
changeset
|
898 |
InstanceKlass *sik = super_ifs->at(j); |
14385 | 899 |
add_new_mirandas_to_lists(new_mirandas, all_mirandas, |
20391
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
900 |
sik->methods(), class_methods, |
48463
474cec233fb2
8154587: Resolution fails for default method named 'clone'
hseigel
parents:
47765
diff
changeset
|
901 |
default_methods, super, is_interface); |
1 | 902 |
} |
903 |
} |
|
904 |
} |
|
905 |
||
20017
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
906 |
// Discover miranda methods ("miranda" = "interface abstract, no binding"), |
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
907 |
// and append them into the vtable starting at index initialized, |
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
908 |
// return the new value of initialized. |
21556
e75cd34a59e0
8027229: ICCE expected for >=2 maximally specific default methods.
acorn
parents:
20391
diff
changeset
|
909 |
// Miranda methods use vtable entries, but do not get assigned a vtable_index |
e75cd34a59e0
8027229: ICCE expected for >=2 maximally specific default methods.
acorn
parents:
20391
diff
changeset
|
910 |
// The vtable_index is discovered by searching from the end of the vtable |
51096
695dff91a997
8178712: ResourceMark may be missing inside initialize_[vi]table
lfoltan
parents:
50735
diff
changeset
|
911 |
int klassVtable::fill_in_mirandas(int initialized, TRAPS) { |
695dff91a997
8178712: ResourceMark may be missing inside initialize_[vi]table
lfoltan
parents:
50735
diff
changeset
|
912 |
ResourceMark rm(THREAD); |
14385 | 913 |
GrowableArray<Method*> mirandas(20); |
914 |
get_mirandas(&mirandas, NULL, ik()->super(), ik()->methods(), |
|
48463
474cec233fb2
8154587: Resolution fails for default method named 'clone'
hseigel
parents:
47765
diff
changeset
|
915 |
ik()->default_methods(), ik()->local_interfaces(), |
474cec233fb2
8154587: Resolution fails for default method named 'clone'
hseigel
parents:
47765
diff
changeset
|
916 |
klass()->is_interface()); |
14385 | 917 |
for (int i = 0; i < mirandas.length(); i++) { |
35901
f5028c67e7cb
8147918: Rename develop_log_is_enabled() to log_develop_is_enabled()
pliden
parents:
35898
diff
changeset
|
918 |
if (log_develop_is_enabled(Trace, vtables)) { |
20391
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
919 |
Method* meth = mirandas.at(i); |
46701
f559541c0daa
8181917: Refactor UL LogStreams to avoid using resource area
stuefe
parents:
46560
diff
changeset
|
920 |
LogTarget(Trace, vtables) lt; |
f559541c0daa
8181917: Refactor UL LogStreams to avoid using resource area
stuefe
parents:
46560
diff
changeset
|
921 |
LogStream ls(lt); |
20391
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
922 |
if (meth != NULL) { |
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
923 |
char* sig = meth->name_and_sig_as_C_string(); |
46701
f559541c0daa
8181917: Refactor UL LogStreams to avoid using resource area
stuefe
parents:
46560
diff
changeset
|
924 |
ls.print("fill in mirandas with %s index %d, flags: ", |
35463
b32e362563bb
8141564: Convert TraceItables and PrintVtables to Unified Logging
rprotacio
parents:
34666
diff
changeset
|
925 |
sig, initialized); |
46701
f559541c0daa
8181917: Refactor UL LogStreams to avoid using resource area
stuefe
parents:
46560
diff
changeset
|
926 |
meth->print_linkage_flags(&ls); |
f559541c0daa
8181917: Refactor UL LogStreams to avoid using resource area
stuefe
parents:
46560
diff
changeset
|
927 |
ls.cr(); |
20391
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
928 |
} |
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
929 |
} |
20017
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
930 |
put_method_at(mirandas.at(i), initialized); |
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
931 |
++initialized; |
1 | 932 |
} |
20017
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
933 |
return initialized; |
1 | 934 |
} |
935 |
||
20017
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
936 |
// Copy this class's vtable to the vtable beginning at start. |
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
937 |
// Used to copy superclass vtable to prefix of subclass's vtable. |
1 | 938 |
void klassVtable::copy_vtable_to(vtableEntry* start) { |
939 |
Copy::disjoint_words((HeapWord*)table(), (HeapWord*)start, _length * vtableEntry::size()); |
|
940 |
} |
|
941 |
||
15591
b8aa0577f137
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
14488
diff
changeset
|
942 |
#if INCLUDE_JVMTI |
20391
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
943 |
bool klassVtable::adjust_default_method(int vtable_index, Method* old_method, Method* new_method) { |
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
944 |
// If old_method is default, find this vtable index in default_vtable_indices |
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
945 |
// and replace that method in the _default_methods list |
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
946 |
bool updated = false; |
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
947 |
|
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
948 |
Array<Method*>* default_methods = ik()->default_methods(); |
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
949 |
if (default_methods != NULL) { |
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
950 |
int len = default_methods->length(); |
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
951 |
for (int idx = 0; idx < len; idx++) { |
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
952 |
if (vtable_index == ik()->default_vtable_indices()->at(idx)) { |
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
953 |
if (default_methods->at(idx) == old_method) { |
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
954 |
default_methods->at_put(idx, new_method); |
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
955 |
updated = true; |
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
956 |
} |
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
957 |
break; |
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
958 |
} |
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
959 |
} |
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
960 |
} |
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
961 |
return updated; |
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
962 |
} |
29316
5287df8a8972
8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents:
28731
diff
changeset
|
963 |
|
5287df8a8972
8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents:
28731
diff
changeset
|
964 |
// search the vtable for uses of either obsolete or EMCP methods |
53904
9c3fe09f69bc
8078725: method adjustments can be done just once for all classes involved into redefinition
coleenp
parents:
52276
diff
changeset
|
965 |
void klassVtable::adjust_method_entries(bool * trace_name_printed) { |
29316
5287df8a8972
8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents:
28731
diff
changeset
|
966 |
int prn_enabled = 0; |
5287df8a8972
8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents:
28731
diff
changeset
|
967 |
for (int index = 0; index < length(); index++) { |
5287df8a8972
8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents:
28731
diff
changeset
|
968 |
Method* old_method = unchecked_method_at(index); |
53904
9c3fe09f69bc
8078725: method adjustments can be done just once for all classes involved into redefinition
coleenp
parents:
52276
diff
changeset
|
969 |
if (old_method == NULL || !old_method->is_old()) { |
29316
5287df8a8972
8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents:
28731
diff
changeset
|
970 |
continue; // skip uninteresting entries |
5287df8a8972
8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents:
28731
diff
changeset
|
971 |
} |
5287df8a8972
8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents:
28731
diff
changeset
|
972 |
assert(!old_method->is_deleted(), "vtable methods may not be deleted"); |
5287df8a8972
8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents:
28731
diff
changeset
|
973 |
|
53904
9c3fe09f69bc
8078725: method adjustments can be done just once for all classes involved into redefinition
coleenp
parents:
52276
diff
changeset
|
974 |
Method* new_method = old_method->get_new_method(); |
9c3fe09f69bc
8078725: method adjustments can be done just once for all classes involved into redefinition
coleenp
parents:
52276
diff
changeset
|
975 |
put_method_at(new_method, index); |
29316
5287df8a8972
8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents:
28731
diff
changeset
|
976 |
|
5287df8a8972
8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents:
28731
diff
changeset
|
977 |
// For default methods, need to update the _default_methods array |
5287df8a8972
8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents:
28731
diff
changeset
|
978 |
// which can only have one method entry for a given signature |
5287df8a8972
8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents:
28731
diff
changeset
|
979 |
bool updated_default = false; |
5287df8a8972
8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents:
28731
diff
changeset
|
980 |
if (old_method->is_default_method()) { |
5287df8a8972
8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents:
28731
diff
changeset
|
981 |
updated_default = adjust_default_method(index, old_method, new_method); |
5287df8a8972
8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents:
28731
diff
changeset
|
982 |
} |
1 | 983 |
|
38259
b495d1cfe673
8153535: Convert TraceRedefineClasses to Unified Logging
rehn
parents:
37242
diff
changeset
|
984 |
if (log_is_enabled(Info, redefine, class, update)) { |
b495d1cfe673
8153535: Convert TraceRedefineClasses to Unified Logging
rehn
parents:
37242
diff
changeset
|
985 |
ResourceMark rm; |
29316
5287df8a8972
8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents:
28731
diff
changeset
|
986 |
if (!(*trace_name_printed)) { |
38259
b495d1cfe673
8153535: Convert TraceRedefineClasses to Unified Logging
rehn
parents:
37242
diff
changeset
|
987 |
log_info(redefine, class, update) |
b495d1cfe673
8153535: Convert TraceRedefineClasses to Unified Logging
rehn
parents:
37242
diff
changeset
|
988 |
("adjust: klassname=%s for methods from name=%s", |
46329
53ccc37bda19
8155672: Remove instanceKlassHandles and KlassHandles
coleenp
parents:
41669
diff
changeset
|
989 |
_klass->external_name(), old_method->method_holder()->external_name()); |
29316
5287df8a8972
8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents:
28731
diff
changeset
|
990 |
*trace_name_printed = true; |
1 | 991 |
} |
38259
b495d1cfe673
8153535: Convert TraceRedefineClasses to Unified Logging
rehn
parents:
37242
diff
changeset
|
992 |
log_debug(redefine, class, update, vtables) |
b495d1cfe673
8153535: Convert TraceRedefineClasses to Unified Logging
rehn
parents:
37242
diff
changeset
|
993 |
("vtable method update: %s(%s), updated default = %s", |
b495d1cfe673
8153535: Convert TraceRedefineClasses to Unified Logging
rehn
parents:
37242
diff
changeset
|
994 |
new_method->name()->as_C_string(), new_method->signature()->as_C_string(), updated_default ? "true" : "false"); |
1 | 995 |
} |
996 |
} |
|
997 |
} |
|
998 |
||
15591
b8aa0577f137
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
14488
diff
changeset
|
999 |
// a vtable should never contain old or obsolete methods |
b8aa0577f137
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
14488
diff
changeset
|
1000 |
bool klassVtable::check_no_old_or_obsolete_entries() { |
b8aa0577f137
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
14488
diff
changeset
|
1001 |
for (int i = 0; i < length(); i++) { |
b8aa0577f137
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
14488
diff
changeset
|
1002 |
Method* m = unchecked_method_at(i); |
b8aa0577f137
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
14488
diff
changeset
|
1003 |
if (m != NULL && |
b8aa0577f137
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
14488
diff
changeset
|
1004 |
(NOT_PRODUCT(!m->is_valid() ||) m->is_old() || m->is_obsolete())) { |
b8aa0577f137
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
14488
diff
changeset
|
1005 |
return false; |
b8aa0577f137
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
14488
diff
changeset
|
1006 |
} |
b8aa0577f137
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
14488
diff
changeset
|
1007 |
} |
b8aa0577f137
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
14488
diff
changeset
|
1008 |
return true; |
b8aa0577f137
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
14488
diff
changeset
|
1009 |
} |
b8aa0577f137
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
14488
diff
changeset
|
1010 |
|
b8aa0577f137
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
14488
diff
changeset
|
1011 |
void klassVtable::dump_vtable() { |
b8aa0577f137
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
14488
diff
changeset
|
1012 |
tty->print_cr("vtable dump --"); |
b8aa0577f137
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
14488
diff
changeset
|
1013 |
for (int i = 0; i < length(); i++) { |
b8aa0577f137
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
14488
diff
changeset
|
1014 |
Method* m = unchecked_method_at(i); |
b8aa0577f137
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
14488
diff
changeset
|
1015 |
if (m != NULL) { |
b8aa0577f137
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
14488
diff
changeset
|
1016 |
tty->print(" (%5d) ", i); |
b8aa0577f137
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
14488
diff
changeset
|
1017 |
m->access_flags().print_on(tty); |
20391
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
1018 |
if (m->is_default_method()) { |
21556
e75cd34a59e0
8027229: ICCE expected for >=2 maximally specific default methods.
acorn
parents:
20391
diff
changeset
|
1019 |
tty->print("default "); |
20391
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
1020 |
} |
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
1021 |
if (m->is_overpass()) { |
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
1022 |
tty->print("overpass"); |
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
1023 |
} |
15591
b8aa0577f137
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
14488
diff
changeset
|
1024 |
tty->print(" -- "); |
b8aa0577f137
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
14488
diff
changeset
|
1025 |
m->print_name(tty); |
b8aa0577f137
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
14488
diff
changeset
|
1026 |
tty->cr(); |
b8aa0577f137
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
14488
diff
changeset
|
1027 |
} |
b8aa0577f137
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
14488
diff
changeset
|
1028 |
} |
b8aa0577f137
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
14488
diff
changeset
|
1029 |
} |
b8aa0577f137
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
14488
diff
changeset
|
1030 |
#endif // INCLUDE_JVMTI |
b8aa0577f137
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
14488
diff
changeset
|
1031 |
|
9172
a4e13ccafc44
7032407: Crash in LinkResolver::runtime_resolve_virtual_method()
coleenp
parents:
8725
diff
changeset
|
1032 |
// CDS/RedefineClasses support - clear vtables so they can be reinitialized |
a4e13ccafc44
7032407: Crash in LinkResolver::runtime_resolve_virtual_method()
coleenp
parents:
8725
diff
changeset
|
1033 |
void klassVtable::clear_vtable() { |
a4e13ccafc44
7032407: Crash in LinkResolver::runtime_resolve_virtual_method()
coleenp
parents:
8725
diff
changeset
|
1034 |
for (int i = 0; i < _length; i++) table()[i].clear(); |
a4e13ccafc44
7032407: Crash in LinkResolver::runtime_resolve_virtual_method()
coleenp
parents:
8725
diff
changeset
|
1035 |
} |
a4e13ccafc44
7032407: Crash in LinkResolver::runtime_resolve_virtual_method()
coleenp
parents:
8725
diff
changeset
|
1036 |
|
a4e13ccafc44
7032407: Crash in LinkResolver::runtime_resolve_virtual_method()
coleenp
parents:
8725
diff
changeset
|
1037 |
bool klassVtable::is_initialized() { |
a4e13ccafc44
7032407: Crash in LinkResolver::runtime_resolve_virtual_method()
coleenp
parents:
8725
diff
changeset
|
1038 |
return _length == 0 || table()[0].method() != NULL; |
a4e13ccafc44
7032407: Crash in LinkResolver::runtime_resolve_virtual_method()
coleenp
parents:
8725
diff
changeset
|
1039 |
} |
a4e13ccafc44
7032407: Crash in LinkResolver::runtime_resolve_virtual_method()
coleenp
parents:
8725
diff
changeset
|
1040 |
|
1 | 1041 |
//----------------------------------------------------------------------------------------- |
1042 |
// Itable code |
|
1043 |
||
1044 |
// Initialize a itableMethodEntry |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
9172
diff
changeset
|
1045 |
void itableMethodEntry::initialize(Method* m) { |
1 | 1046 |
if (m == NULL) return; |
1047 |
||
46413 | 1048 |
#ifdef ASSERT |
48794
ea0d0781c63c
8178351: Simplify MetaspaceShared::is_in_shared_space and MetaspaceObj::is_shared
iklam
parents:
48463
diff
changeset
|
1049 |
if (MetaspaceShared::is_in_shared_metaspace((void*)&_method) && |
39714 | 1050 |
!MetaspaceShared::remapped_readwrite()) { |
1051 |
// At runtime initialize_itable is rerun as part of link_class_impl() |
|
1052 |
// for a shared class loaded by the non-boot loader. |
|
1053 |
// The dumptime itable method entry should be the same as the runtime entry. |
|
1054 |
assert(_method == m, "sanity"); |
|
1055 |
} |
|
46413 | 1056 |
#endif |
1057 |
_method = m; |
|
1 | 1058 |
} |
1059 |
||
46329
53ccc37bda19
8155672: Remove instanceKlassHandles and KlassHandles
coleenp
parents:
41669
diff
changeset
|
1060 |
klassItable::klassItable(InstanceKlass* klass) { |
1 | 1061 |
_klass = klass; |
1062 |
||
1063 |
if (klass->itable_length() > 0) { |
|
1064 |
itableOffsetEntry* offset_entry = (itableOffsetEntry*)klass->start_of_itable(); |
|
1065 |
if (offset_entry != NULL && offset_entry->interface_klass() != NULL) { // Check that itable is initialized |
|
1066 |
// First offset entry points to the first method_entry |
|
46329
53ccc37bda19
8155672: Remove instanceKlassHandles and KlassHandles
coleenp
parents:
41669
diff
changeset
|
1067 |
intptr_t* method_entry = (intptr_t *)(((address)klass) + offset_entry->offset()); |
1 | 1068 |
intptr_t* end = klass->end_of_itable(); |
1069 |
||
46329
53ccc37bda19
8155672: Remove instanceKlassHandles and KlassHandles
coleenp
parents:
41669
diff
changeset
|
1070 |
_table_offset = (intptr_t*)offset_entry - (intptr_t*)klass; |
1 | 1071 |
_size_offset_table = (method_entry - ((intptr_t*)offset_entry)) / itableOffsetEntry::size(); |
1072 |
_size_method_table = (end - method_entry) / itableMethodEntry::size(); |
|
1073 |
assert(_table_offset >= 0 && _size_offset_table >= 0 && _size_method_table >= 0, "wrong computation"); |
|
1074 |
return; |
|
1075 |
} |
|
1076 |
} |
|
1077 |
||
189
4248c8e21063
6664627: Merge changes made only in hotspot 11 forward to jdk 7
dcubed
parents:
1
diff
changeset
|
1078 |
// The length of the itable was either zero, or it has not yet been initialized. |
1 | 1079 |
_table_offset = 0; |
1080 |
_size_offset_table = 0; |
|
1081 |
_size_method_table = 0; |
|
1082 |
} |
|
1083 |
||
1084 |
static int initialize_count = 0; |
|
1085 |
||
1086 |
// Initialization |
|
1087 |
void klassItable::initialize_itable(bool checkconstraints, TRAPS) { |
|
20017
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
1088 |
if (_klass->is_interface()) { |
20391
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
1089 |
// This needs to go after vtable indices are assigned but |
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
1090 |
// before implementors need to know the number of itable indices. |
51329
9c68699bebe5
8208999: Some use of Klass* should be replaced by InstanceKlass*
iklam
parents:
51097
diff
changeset
|
1091 |
assign_itable_indices_for_interface(InstanceKlass::cast(_klass), THREAD); |
20017
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
1092 |
} |
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
1093 |
|
189
4248c8e21063
6664627: Merge changes made only in hotspot 11 forward to jdk 7
dcubed
parents:
1
diff
changeset
|
1094 |
// Cannot be setup doing bootstrapping, interfaces don't have |
4248c8e21063
6664627: Merge changes made only in hotspot 11 forward to jdk 7
dcubed
parents:
1
diff
changeset
|
1095 |
// itables, and klass with only ones entry have empty itables |
4248c8e21063
6664627: Merge changes made only in hotspot 11 forward to jdk 7
dcubed
parents:
1
diff
changeset
|
1096 |
if (Universe::is_bootstrapping() || |
4248c8e21063
6664627: Merge changes made only in hotspot 11 forward to jdk 7
dcubed
parents:
1
diff
changeset
|
1097 |
_klass->is_interface() || |
4248c8e21063
6664627: Merge changes made only in hotspot 11 forward to jdk 7
dcubed
parents:
1
diff
changeset
|
1098 |
_klass->itable_length() == itableOffsetEntry::size()) return; |
1 | 1099 |
|
189
4248c8e21063
6664627: Merge changes made only in hotspot 11 forward to jdk 7
dcubed
parents:
1
diff
changeset
|
1100 |
// There's alway an extra itable entry so we can null-terminate it. |
4248c8e21063
6664627: Merge changes made only in hotspot 11 forward to jdk 7
dcubed
parents:
1
diff
changeset
|
1101 |
guarantee(size_offset_table() >= 1, "too small"); |
4248c8e21063
6664627: Merge changes made only in hotspot 11 forward to jdk 7
dcubed
parents:
1
diff
changeset
|
1102 |
int num_interfaces = size_offset_table() - 1; |
1 | 1103 |
if (num_interfaces > 0) { |
51096
695dff91a997
8178712: ResourceMark may be missing inside initialize_[vi]table
lfoltan
parents:
50735
diff
changeset
|
1104 |
ResourceMark rm(THREAD); |
35463
b32e362563bb
8141564: Convert TraceItables and PrintVtables to Unified Logging
rprotacio
parents:
34666
diff
changeset
|
1105 |
log_develop_debug(itables)("%3d: Initializing itables for %s", ++initialize_count, |
b32e362563bb
8141564: Convert TraceItables and PrintVtables to Unified Logging
rprotacio
parents:
34666
diff
changeset
|
1106 |
_klass->name()->as_C_string()); |
1 | 1107 |
|
1108 |
||
2264
55d0115a54fe
4766230: Hotspot vtable inconsistencies cause core dumps. 6579515. 6582242.
acorn
parents:
2136
diff
changeset
|
1109 |
// Iterate through all interfaces |
1 | 1110 |
int i; |
1111 |
for(i = 0; i < num_interfaces; i++) { |
|
1112 |
itableOffsetEntry* ioe = offset_entry(i); |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
9172
diff
changeset
|
1113 |
HandleMark hm(THREAD); |
46329
53ccc37bda19
8155672: Remove instanceKlassHandles and KlassHandles
coleenp
parents:
41669
diff
changeset
|
1114 |
Klass* interf = ioe->interface_klass(); |
53ccc37bda19
8155672: Remove instanceKlassHandles and KlassHandles
coleenp
parents:
41669
diff
changeset
|
1115 |
assert(interf != NULL && ioe->offset() != 0, "bad offset entry in itable"); |
51329
9c68699bebe5
8208999: Some use of Klass* should be replaced by InstanceKlass*
iklam
parents:
51097
diff
changeset
|
1116 |
initialize_itable_for_interface(ioe->offset(), InstanceKlass::cast(interf), checkconstraints, CHECK); |
1 | 1117 |
} |
1118 |
||
1119 |
} |
|
189
4248c8e21063
6664627: Merge changes made only in hotspot 11 forward to jdk 7
dcubed
parents:
1
diff
changeset
|
1120 |
// Check that the last entry is empty |
4248c8e21063
6664627: Merge changes made only in hotspot 11 forward to jdk 7
dcubed
parents:
1
diff
changeset
|
1121 |
itableOffsetEntry* ioe = offset_entry(size_offset_table() - 1); |
4248c8e21063
6664627: Merge changes made only in hotspot 11 forward to jdk 7
dcubed
parents:
1
diff
changeset
|
1122 |
guarantee(ioe->interface_klass() == NULL && ioe->offset() == 0, "terminator entry missing"); |
1 | 1123 |
} |
1124 |
||
1125 |
||
20017
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
1126 |
inline bool interface_method_needs_itable_index(Method* m) { |
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
1127 |
if (m->is_static()) return false; // e.g., Stream.empty |
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
1128 |
if (m->is_initializer()) return false; // <init> or <clinit> |
50735
2f2af62dfac7
8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents:
50036
diff
changeset
|
1129 |
if (m->is_private()) return false; // uses direct call |
20017
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
1130 |
// If an interface redeclares a method from java.lang.Object, |
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
1131 |
// it should already have a vtable index, don't touch it. |
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
1132 |
// e.g., CharSequence.toString (from initialize_vtable) |
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
1133 |
// if (m->has_vtable_index()) return false; // NO! |
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
1134 |
return true; |
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
1135 |
} |
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
1136 |
|
51329
9c68699bebe5
8208999: Some use of Klass* should be replaced by InstanceKlass*
iklam
parents:
51097
diff
changeset
|
1137 |
int klassItable::assign_itable_indices_for_interface(InstanceKlass* klass, TRAPS) { |
20017
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
1138 |
// an interface does not have an itable, but its methods need to be numbered |
51096
695dff91a997
8178712: ResourceMark may be missing inside initialize_[vi]table
lfoltan
parents:
50735
diff
changeset
|
1139 |
ResourceMark rm(THREAD); |
35463
b32e362563bb
8141564: Convert TraceItables and PrintVtables to Unified Logging
rprotacio
parents:
34666
diff
changeset
|
1140 |
log_develop_debug(itables)("%3d: Initializing itable indices for interface %s", |
b32e362563bb
8141564: Convert TraceItables and PrintVtables to Unified Logging
rprotacio
parents:
34666
diff
changeset
|
1141 |
++initialize_count, klass->name()->as_C_string()); |
51329
9c68699bebe5
8208999: Some use of Klass* should be replaced by InstanceKlass*
iklam
parents:
51097
diff
changeset
|
1142 |
Array<Method*>* methods = klass->methods(); |
20017
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
1143 |
int nof_methods = methods->length(); |
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
1144 |
int ime_num = 0; |
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
1145 |
for (int i = 0; i < nof_methods; i++) { |
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
1146 |
Method* m = methods->at(i); |
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
1147 |
if (interface_method_needs_itable_index(m)) { |
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
1148 |
assert(!m->is_final_method(), "no final interface methods"); |
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
1149 |
// If m is already assigned a vtable index, do not disturb it. |
35901
f5028c67e7cb
8147918: Rename develop_log_is_enabled() to log_develop_is_enabled()
pliden
parents:
35898
diff
changeset
|
1150 |
if (log_develop_is_enabled(Trace, itables)) { |
46701
f559541c0daa
8181917: Refactor UL LogStreams to avoid using resource area
stuefe
parents:
46560
diff
changeset
|
1151 |
LogTarget(Trace, itables) lt; |
f559541c0daa
8181917: Refactor UL LogStreams to avoid using resource area
stuefe
parents:
46560
diff
changeset
|
1152 |
LogStream ls(lt); |
35463
b32e362563bb
8141564: Convert TraceItables and PrintVtables to Unified Logging
rprotacio
parents:
34666
diff
changeset
|
1153 |
assert(m != NULL, "methods can never be null"); |
b32e362563bb
8141564: Convert TraceItables and PrintVtables to Unified Logging
rprotacio
parents:
34666
diff
changeset
|
1154 |
const char* sig = m->name_and_sig_as_C_string(); |
21556
e75cd34a59e0
8027229: ICCE expected for >=2 maximally specific default methods.
acorn
parents:
20391
diff
changeset
|
1155 |
if (m->has_vtable_index()) { |
46701
f559541c0daa
8181917: Refactor UL LogStreams to avoid using resource area
stuefe
parents:
46560
diff
changeset
|
1156 |
ls.print("vtable index %d for method: %s, flags: ", m->vtable_index(), sig); |
21556
e75cd34a59e0
8027229: ICCE expected for >=2 maximally specific default methods.
acorn
parents:
20391
diff
changeset
|
1157 |
} else { |
46701
f559541c0daa
8181917: Refactor UL LogStreams to avoid using resource area
stuefe
parents:
46560
diff
changeset
|
1158 |
ls.print("itable index %d for method: %s, flags: ", ime_num, sig); |
21556
e75cd34a59e0
8027229: ICCE expected for >=2 maximally specific default methods.
acorn
parents:
20391
diff
changeset
|
1159 |
} |
46701
f559541c0daa
8181917: Refactor UL LogStreams to avoid using resource area
stuefe
parents:
46560
diff
changeset
|
1160 |
m->print_linkage_flags(&ls); |
f559541c0daa
8181917: Refactor UL LogStreams to avoid using resource area
stuefe
parents:
46560
diff
changeset
|
1161 |
ls.cr(); |
21556
e75cd34a59e0
8027229: ICCE expected for >=2 maximally specific default methods.
acorn
parents:
20391
diff
changeset
|
1162 |
} |
20017
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
1163 |
if (!m->has_vtable_index()) { |
39714 | 1164 |
// A shared method could have an initialized itable_index that |
1165 |
// is < 0. |
|
1166 |
assert(m->vtable_index() == Method::pending_itable_index || |
|
1167 |
m->is_shared(), |
|
1168 |
"set by initialize_vtable"); |
|
20017
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
1169 |
m->set_itable_index(ime_num); |
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
1170 |
// Progress to next itable entry |
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
1171 |
ime_num++; |
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
1172 |
} |
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
1173 |
} |
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
1174 |
} |
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
1175 |
assert(ime_num == method_count_for_interface(klass), "proper sizing"); |
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
1176 |
return ime_num; |
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
1177 |
} |
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
1178 |
|
51329
9c68699bebe5
8208999: Some use of Klass* should be replaced by InstanceKlass*
iklam
parents:
51097
diff
changeset
|
1179 |
int klassItable::method_count_for_interface(InstanceKlass* interf) { |
20017
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
1180 |
assert(interf->is_interface(), "must be"); |
51329
9c68699bebe5
8208999: Some use of Klass* should be replaced by InstanceKlass*
iklam
parents:
51097
diff
changeset
|
1181 |
Array<Method*>* methods = interf->methods(); |
20017
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
1182 |
int nof_methods = methods->length(); |
30615
4713e7c7b96f
8030680: 292 cleanup from default method code assessment
acorn
parents:
29871
diff
changeset
|
1183 |
int length = 0; |
20017
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
1184 |
while (nof_methods > 0) { |
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
1185 |
Method* m = methods->at(nof_methods-1); |
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
1186 |
if (m->has_itable_index()) { |
30615
4713e7c7b96f
8030680: 292 cleanup from default method code assessment
acorn
parents:
29871
diff
changeset
|
1187 |
length = m->itable_index() + 1; |
4713e7c7b96f
8030680: 292 cleanup from default method code assessment
acorn
parents:
29871
diff
changeset
|
1188 |
break; |
20017
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
1189 |
} |
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
1190 |
nof_methods -= 1; |
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
1191 |
} |
30615
4713e7c7b96f
8030680: 292 cleanup from default method code assessment
acorn
parents:
29871
diff
changeset
|
1192 |
#ifdef ASSERT |
4713e7c7b96f
8030680: 292 cleanup from default method code assessment
acorn
parents:
29871
diff
changeset
|
1193 |
int nof_methods_copy = nof_methods; |
4713e7c7b96f
8030680: 292 cleanup from default method code assessment
acorn
parents:
29871
diff
changeset
|
1194 |
while (nof_methods_copy > 0) { |
4713e7c7b96f
8030680: 292 cleanup from default method code assessment
acorn
parents:
29871
diff
changeset
|
1195 |
Method* mm = methods->at(--nof_methods_copy); |
4713e7c7b96f
8030680: 292 cleanup from default method code assessment
acorn
parents:
29871
diff
changeset
|
1196 |
assert(!mm->has_itable_index() || mm->itable_index() < length, ""); |
4713e7c7b96f
8030680: 292 cleanup from default method code assessment
acorn
parents:
29871
diff
changeset
|
1197 |
} |
4713e7c7b96f
8030680: 292 cleanup from default method code assessment
acorn
parents:
29871
diff
changeset
|
1198 |
#endif //ASSERT |
4713e7c7b96f
8030680: 292 cleanup from default method code assessment
acorn
parents:
29871
diff
changeset
|
1199 |
// return the rightmost itable index, plus one; or 0 if no methods have |
4713e7c7b96f
8030680: 292 cleanup from default method code assessment
acorn
parents:
29871
diff
changeset
|
1200 |
// itable indices |
4713e7c7b96f
8030680: 292 cleanup from default method code assessment
acorn
parents:
29871
diff
changeset
|
1201 |
return length; |
20017
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
1202 |
} |
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
1203 |
|
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
1204 |
|
51329
9c68699bebe5
8208999: Some use of Klass* should be replaced by InstanceKlass*
iklam
parents:
51097
diff
changeset
|
1205 |
void klassItable::initialize_itable_for_interface(int method_table_offset, InstanceKlass* interf, bool checkconstraints, TRAPS) { |
9c68699bebe5
8208999: Some use of Klass* should be replaced by InstanceKlass*
iklam
parents:
51097
diff
changeset
|
1206 |
assert(interf->is_interface(), "must be"); |
9c68699bebe5
8208999: Some use of Klass* should be replaced by InstanceKlass*
iklam
parents:
51097
diff
changeset
|
1207 |
Array<Method*>* methods = interf->methods(); |
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
9172
diff
changeset
|
1208 |
int nof_methods = methods->length(); |
1 | 1209 |
HandleMark hm; |
51329
9c68699bebe5
8208999: Some use of Klass* should be replaced by InstanceKlass*
iklam
parents:
51097
diff
changeset
|
1210 |
Handle interface_loader (THREAD, interf->class_loader()); |
1 | 1211 |
|
46329
53ccc37bda19
8155672: Remove instanceKlassHandles and KlassHandles
coleenp
parents:
41669
diff
changeset
|
1212 |
int ime_count = method_count_for_interface(interf); |
20017
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
1213 |
for (int i = 0; i < nof_methods; i++) { |
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
9172
diff
changeset
|
1214 |
Method* m = methods->at(i); |
20017
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
1215 |
methodHandle target; |
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
1216 |
if (m->has_itable_index()) { |
22232
26acfad336c0
8027804: JCK resolveMethod test fails expecting AbstractMethodError
hseigel
parents:
21913
diff
changeset
|
1217 |
// This search must match the runtime resolution, i.e. selection search for invokeinterface |
50735
2f2af62dfac7
8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents:
50036
diff
changeset
|
1218 |
// to correctly enforce loader constraints for interface method inheritance. |
2f2af62dfac7
8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents:
50036
diff
changeset
|
1219 |
// Private methods are skipped as a private class method can never be the implementation |
2f2af62dfac7
8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents:
50036
diff
changeset
|
1220 |
// of an interface method. |
2f2af62dfac7
8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents:
50036
diff
changeset
|
1221 |
// Invokespecial does not perform selection based on the receiver, so it does not use |
2f2af62dfac7
8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents:
50036
diff
changeset
|
1222 |
// the cached itable. |
2f2af62dfac7
8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents:
50036
diff
changeset
|
1223 |
target = LinkResolver::lookup_instance_method_in_klasses(_klass, m->name(), m->signature(), |
2f2af62dfac7
8010319: Implementation of JEP 181: Nest-Based Access Control
dholmes
parents:
50036
diff
changeset
|
1224 |
Klass::skip_private, CHECK); |
1 | 1225 |
} |
47554
bc112140e089
8186092: Unnecessary loader constraints produced when there are multiple defaults
hseigel
parents:
47216
diff
changeset
|
1226 |
if (target == NULL || !target->is_public() || target->is_abstract() || target->is_overpass()) { |
bc112140e089
8186092: Unnecessary loader constraints produced when there are multiple defaults
hseigel
parents:
47216
diff
changeset
|
1227 |
assert(target == NULL || !target->is_overpass() || target->is_public(), |
bc112140e089
8186092: Unnecessary loader constraints produced when there are multiple defaults
hseigel
parents:
47216
diff
changeset
|
1228 |
"Non-public overpass method!"); |
bc112140e089
8186092: Unnecessary loader constraints produced when there are multiple defaults
hseigel
parents:
47216
diff
changeset
|
1229 |
// Entry does not resolve. Leave it empty for AbstractMethodError or other error. |
bc112140e089
8186092: Unnecessary loader constraints produced when there are multiple defaults
hseigel
parents:
47216
diff
changeset
|
1230 |
if (!(target == NULL) && !target->is_public()) { |
bc112140e089
8186092: Unnecessary loader constraints produced when there are multiple defaults
hseigel
parents:
47216
diff
changeset
|
1231 |
// Stuff an IllegalAccessError throwing method in there instead. |
bc112140e089
8186092: Unnecessary loader constraints produced when there are multiple defaults
hseigel
parents:
47216
diff
changeset
|
1232 |
itableOffsetEntry::method_entry(_klass, method_table_offset)[m->itable_index()]. |
bc112140e089
8186092: Unnecessary loader constraints produced when there are multiple defaults
hseigel
parents:
47216
diff
changeset
|
1233 |
initialize(Universe::throw_illegal_access_error()); |
bc112140e089
8186092: Unnecessary loader constraints produced when there are multiple defaults
hseigel
parents:
47216
diff
changeset
|
1234 |
} |
1 | 1235 |
} else { |
1236 |
// Entry did resolve, check loader constraints before initializing |
|
1237 |
// if checkconstraints requested |
|
1238 |
if (checkconstraints) { |
|
14391
df0a1573d5bd
8000725: NPG: method_holder() and pool_holder() and pool_holder field should be InstanceKlass
coleenp
parents:
14385
diff
changeset
|
1239 |
Handle method_holder_loader (THREAD, target->method_holder()->class_loader()); |
49658 | 1240 |
if (!oopDesc::equals(method_holder_loader(), interface_loader())) { |
1 | 1241 |
ResourceMark rm(THREAD); |
16591
73047b0eb568
8009731: Confusing error message for loader constraint violation
acorn
parents:
15591
diff
changeset
|
1242 |
Symbol* failed_type_symbol = |
20017
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
1243 |
SystemDictionary::check_signature_loaders(m->signature(), |
1 | 1244 |
method_holder_loader, |
1245 |
interface_loader, |
|
1246 |
true, CHECK); |
|
16591
73047b0eb568
8009731: Confusing error message for loader constraint violation
acorn
parents:
15591
diff
changeset
|
1247 |
if (failed_type_symbol != NULL) { |
51097
bef02342d179
8205611: Improve the wording of LinkageErrors to include module and class loader information
lfoltan
parents:
51096
diff
changeset
|
1248 |
stringStream ss; |
bef02342d179
8205611: Improve the wording of LinkageErrors to include module and class loader information
lfoltan
parents:
51096
diff
changeset
|
1249 |
ss.print("loader constraint violation in interface itable" |
54432
532e88de77eb
8221470: Print methods in exception messages in java-like Syntax.
goetz
parents:
53904
diff
changeset
|
1250 |
" initialization for class %s: when selecting method '", |
532e88de77eb
8221470: Print methods in exception messages in java-like Syntax.
goetz
parents:
53904
diff
changeset
|
1251 |
_klass->external_name()); |
532e88de77eb
8221470: Print methods in exception messages in java-like Syntax.
goetz
parents:
53904
diff
changeset
|
1252 |
m->print_external_name(&ss), |
532e88de77eb
8221470: Print methods in exception messages in java-like Syntax.
goetz
parents:
53904
diff
changeset
|
1253 |
ss.print("' the class loader %s for super interface %s, and the class" |
532e88de77eb
8221470: Print methods in exception messages in java-like Syntax.
goetz
parents:
53904
diff
changeset
|
1254 |
" loader %s of the selected method's %s, %s have" |
51097
bef02342d179
8205611: Improve the wording of LinkageErrors to include module and class loader information
lfoltan
parents:
51096
diff
changeset
|
1255 |
" different Class objects for the type %s used in the signature (%s; %s)", |
bef02342d179
8205611: Improve the wording of LinkageErrors to include module and class loader information
lfoltan
parents:
51096
diff
changeset
|
1256 |
interf->class_loader_data()->loader_name_and_id(), |
bef02342d179
8205611: Improve the wording of LinkageErrors to include module and class loader information
lfoltan
parents:
51096
diff
changeset
|
1257 |
interf->external_name(), |
bef02342d179
8205611: Improve the wording of LinkageErrors to include module and class loader information
lfoltan
parents:
51096
diff
changeset
|
1258 |
target()->method_holder()->class_loader_data()->loader_name_and_id(), |
54432
532e88de77eb
8221470: Print methods in exception messages in java-like Syntax.
goetz
parents:
53904
diff
changeset
|
1259 |
target()->method_holder()->external_kind(), |
51097
bef02342d179
8205611: Improve the wording of LinkageErrors to include module and class loader information
lfoltan
parents:
51096
diff
changeset
|
1260 |
target()->method_holder()->external_name(), |
bef02342d179
8205611: Improve the wording of LinkageErrors to include module and class loader information
lfoltan
parents:
51096
diff
changeset
|
1261 |
failed_type_symbol->as_klass_external_name(), |
bef02342d179
8205611: Improve the wording of LinkageErrors to include module and class loader information
lfoltan
parents:
51096
diff
changeset
|
1262 |
interf->class_in_module_of_loader(false, true), |
bef02342d179
8205611: Improve the wording of LinkageErrors to include module and class loader information
lfoltan
parents:
51096
diff
changeset
|
1263 |
target()->method_holder()->class_in_module_of_loader(false, true)); |
bef02342d179
8205611: Improve the wording of LinkageErrors to include module and class loader information
lfoltan
parents:
51096
diff
changeset
|
1264 |
THROW_MSG(vmSymbols::java_lang_LinkageError(), ss.as_string()); |
1 | 1265 |
} |
1266 |
} |
|
1267 |
} |
|
1268 |
||
1269 |
// ime may have moved during GC so recalculate address |
|
20017
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
1270 |
int ime_num = m->itable_index(); |
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
1271 |
assert(ime_num < ime_count, "oob"); |
46329
53ccc37bda19
8155672: Remove instanceKlassHandles and KlassHandles
coleenp
parents:
41669
diff
changeset
|
1272 |
itableOffsetEntry::method_entry(_klass, method_table_offset)[ime_num].initialize(target()); |
35901
f5028c67e7cb
8147918: Rename develop_log_is_enabled() to log_develop_is_enabled()
pliden
parents:
35898
diff
changeset
|
1273 |
if (log_develop_is_enabled(Trace, itables)) { |
20391
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
1274 |
ResourceMark rm(THREAD); |
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
1275 |
if (target() != NULL) { |
46701
f559541c0daa
8181917: Refactor UL LogStreams to avoid using resource area
stuefe
parents:
46560
diff
changeset
|
1276 |
LogTarget(Trace, itables) lt; |
f559541c0daa
8181917: Refactor UL LogStreams to avoid using resource area
stuefe
parents:
46560
diff
changeset
|
1277 |
LogStream ls(lt); |
20391
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
1278 |
char* sig = target()->name_and_sig_as_C_string(); |
46701
f559541c0daa
8181917: Refactor UL LogStreams to avoid using resource area
stuefe
parents:
46560
diff
changeset
|
1279 |
ls.print("interface: %s, ime_num: %d, target: %s, method_holder: %s ", |
46329
53ccc37bda19
8155672: Remove instanceKlassHandles and KlassHandles
coleenp
parents:
41669
diff
changeset
|
1280 |
interf->internal_name(), ime_num, sig, |
35463
b32e362563bb
8141564: Convert TraceItables and PrintVtables to Unified Logging
rprotacio
parents:
34666
diff
changeset
|
1281 |
target()->method_holder()->internal_name()); |
46701
f559541c0daa
8181917: Refactor UL LogStreams to avoid using resource area
stuefe
parents:
46560
diff
changeset
|
1282 |
ls.print("target_method flags: "); |
f559541c0daa
8181917: Refactor UL LogStreams to avoid using resource area
stuefe
parents:
46560
diff
changeset
|
1283 |
target()->print_linkage_flags(&ls); |
f559541c0daa
8181917: Refactor UL LogStreams to avoid using resource area
stuefe
parents:
46560
diff
changeset
|
1284 |
ls.cr(); |
20391
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
1285 |
} |
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
1286 |
} |
1 | 1287 |
} |
1288 |
} |
|
1289 |
} |
|
1290 |
||
15591
b8aa0577f137
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
14488
diff
changeset
|
1291 |
#if INCLUDE_JVMTI |
29316
5287df8a8972
8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents:
28731
diff
changeset
|
1292 |
// search the itable for uses of either obsolete or EMCP methods |
53904
9c3fe09f69bc
8078725: method adjustments can be done just once for all classes involved into redefinition
coleenp
parents:
52276
diff
changeset
|
1293 |
void klassItable::adjust_method_entries(bool * trace_name_printed) { |
1 | 1294 |
|
29316
5287df8a8972
8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents:
28731
diff
changeset
|
1295 |
itableMethodEntry* ime = method_entry(0); |
5287df8a8972
8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents:
28731
diff
changeset
|
1296 |
for (int i = 0; i < _size_method_table; i++, ime++) { |
5287df8a8972
8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents:
28731
diff
changeset
|
1297 |
Method* old_method = ime->method(); |
53904
9c3fe09f69bc
8078725: method adjustments can be done just once for all classes involved into redefinition
coleenp
parents:
52276
diff
changeset
|
1298 |
if (old_method == NULL || !old_method->is_old()) { |
29316
5287df8a8972
8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents:
28731
diff
changeset
|
1299 |
continue; // skip uninteresting entries |
5287df8a8972
8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents:
28731
diff
changeset
|
1300 |
} |
5287df8a8972
8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents:
28731
diff
changeset
|
1301 |
assert(!old_method->is_deleted(), "itable methods may not be deleted"); |
53904
9c3fe09f69bc
8078725: method adjustments can be done just once for all classes involved into redefinition
coleenp
parents:
52276
diff
changeset
|
1302 |
Method* new_method = old_method->get_new_method(); |
29316
5287df8a8972
8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents:
28731
diff
changeset
|
1303 |
ime->initialize(new_method); |
5287df8a8972
8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents:
28731
diff
changeset
|
1304 |
|
38259
b495d1cfe673
8153535: Convert TraceRedefineClasses to Unified Logging
rehn
parents:
37242
diff
changeset
|
1305 |
if (log_is_enabled(Info, redefine, class, update)) { |
b495d1cfe673
8153535: Convert TraceRedefineClasses to Unified Logging
rehn
parents:
37242
diff
changeset
|
1306 |
ResourceMark rm; |
29316
5287df8a8972
8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents:
28731
diff
changeset
|
1307 |
if (!(*trace_name_printed)) { |
38259
b495d1cfe673
8153535: Convert TraceRedefineClasses to Unified Logging
rehn
parents:
37242
diff
changeset
|
1308 |
log_info(redefine, class, update)("adjust: name=%s", old_method->method_holder()->external_name()); |
29316
5287df8a8972
8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents:
28731
diff
changeset
|
1309 |
*trace_name_printed = true; |
1 | 1310 |
} |
38259
b495d1cfe673
8153535: Convert TraceRedefineClasses to Unified Logging
rehn
parents:
37242
diff
changeset
|
1311 |
log_trace(redefine, class, update, itables) |
b495d1cfe673
8153535: Convert TraceRedefineClasses to Unified Logging
rehn
parents:
37242
diff
changeset
|
1312 |
("itable method update: %s(%s)", new_method->name()->as_C_string(), new_method->signature()->as_C_string()); |
1 | 1313 |
} |
1314 |
} |
|
1315 |
} |
|
1316 |
||
15591
b8aa0577f137
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
14488
diff
changeset
|
1317 |
// an itable should never contain old or obsolete methods |
b8aa0577f137
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
14488
diff
changeset
|
1318 |
bool klassItable::check_no_old_or_obsolete_entries() { |
b8aa0577f137
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
14488
diff
changeset
|
1319 |
itableMethodEntry* ime = method_entry(0); |
b8aa0577f137
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
14488
diff
changeset
|
1320 |
for (int i = 0; i < _size_method_table; i++) { |
b8aa0577f137
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
14488
diff
changeset
|
1321 |
Method* m = ime->method(); |
b8aa0577f137
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
14488
diff
changeset
|
1322 |
if (m != NULL && |
b8aa0577f137
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
14488
diff
changeset
|
1323 |
(NOT_PRODUCT(!m->is_valid() ||) m->is_old() || m->is_obsolete())) { |
b8aa0577f137
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
14488
diff
changeset
|
1324 |
return false; |
b8aa0577f137
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
14488
diff
changeset
|
1325 |
} |
b8aa0577f137
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
14488
diff
changeset
|
1326 |
ime++; |
b8aa0577f137
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
14488
diff
changeset
|
1327 |
} |
b8aa0577f137
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
14488
diff
changeset
|
1328 |
return true; |
b8aa0577f137
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
14488
diff
changeset
|
1329 |
} |
b8aa0577f137
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
14488
diff
changeset
|
1330 |
|
b8aa0577f137
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
14488
diff
changeset
|
1331 |
void klassItable::dump_itable() { |
b8aa0577f137
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
14488
diff
changeset
|
1332 |
itableMethodEntry* ime = method_entry(0); |
b8aa0577f137
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
14488
diff
changeset
|
1333 |
tty->print_cr("itable dump --"); |
b8aa0577f137
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
14488
diff
changeset
|
1334 |
for (int i = 0; i < _size_method_table; i++) { |
b8aa0577f137
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
14488
diff
changeset
|
1335 |
Method* m = ime->method(); |
b8aa0577f137
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
14488
diff
changeset
|
1336 |
if (m != NULL) { |
b8aa0577f137
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
14488
diff
changeset
|
1337 |
tty->print(" (%5d) ", i); |
b8aa0577f137
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
14488
diff
changeset
|
1338 |
m->access_flags().print_on(tty); |
20391
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
1339 |
if (m->is_default_method()) { |
21556
e75cd34a59e0
8027229: ICCE expected for >=2 maximally specific default methods.
acorn
parents:
20391
diff
changeset
|
1340 |
tty->print("default "); |
20391
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
1341 |
} |
15591
b8aa0577f137
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
14488
diff
changeset
|
1342 |
tty->print(" -- "); |
b8aa0577f137
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
14488
diff
changeset
|
1343 |
m->print_name(tty); |
b8aa0577f137
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
14488
diff
changeset
|
1344 |
tty->cr(); |
b8aa0577f137
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
14488
diff
changeset
|
1345 |
} |
b8aa0577f137
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
14488
diff
changeset
|
1346 |
ime++; |
b8aa0577f137
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
14488
diff
changeset
|
1347 |
} |
b8aa0577f137
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
14488
diff
changeset
|
1348 |
} |
b8aa0577f137
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
14488
diff
changeset
|
1349 |
#endif // INCLUDE_JVMTI |
b8aa0577f137
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
14488
diff
changeset
|
1350 |
|
1 | 1351 |
// Setup |
1352 |
class InterfaceVisiterClosure : public StackObj { |
|
1353 |
public: |
|
51329
9c68699bebe5
8208999: Some use of Klass* should be replaced by InstanceKlass*
iklam
parents:
51097
diff
changeset
|
1354 |
virtual void doit(InstanceKlass* intf, int method_count) = 0; |
1 | 1355 |
}; |
1356 |
||
20017
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
1357 |
// Visit all interfaces with at least one itable method |
51329
9c68699bebe5
8208999: Some use of Klass* should be replaced by InstanceKlass*
iklam
parents:
51097
diff
changeset
|
1358 |
void visit_all_interfaces(Array<InstanceKlass*>* transitive_intf, InterfaceVisiterClosure *blk) { |
1 | 1359 |
// Handle array argument |
1360 |
for(int i = 0; i < transitive_intf->length(); i++) { |
|
51329
9c68699bebe5
8208999: Some use of Klass* should be replaced by InstanceKlass*
iklam
parents:
51097
diff
changeset
|
1361 |
InstanceKlass* intf = transitive_intf->at(i); |
14488 | 1362 |
assert(intf->is_interface(), "sanity check"); |
1 | 1363 |
|
20017
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
1364 |
// Find no. of itable methods |
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
1365 |
int method_count = 0; |
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
1366 |
// method_count = klassItable::method_count_for_interface(intf); |
51329
9c68699bebe5
8208999: Some use of Klass* should be replaced by InstanceKlass*
iklam
parents:
51097
diff
changeset
|
1367 |
Array<Method*>* methods = intf->methods(); |
20017
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
1368 |
if (methods->length() > 0) { |
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
1369 |
for (int i = methods->length(); --i >= 0; ) { |
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
1370 |
if (interface_method_needs_itable_index(methods->at(i))) { |
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
1371 |
method_count++; |
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
1372 |
} |
1 | 1373 |
} |
1374 |
} |
|
1375 |
||
48557 | 1376 |
// Visit all interfaces which either have any methods or can participate in receiver type check. |
1377 |
// We do not bother to count methods in transitive interfaces, although that would allow us to skip |
|
1378 |
// this step in the rare case of a zero-method interface extending another zero-method interface. |
|
51329
9c68699bebe5
8208999: Some use of Klass* should be replaced by InstanceKlass*
iklam
parents:
51097
diff
changeset
|
1379 |
if (method_count > 0 || intf->transitive_interfaces()->length() > 0) { |
1 | 1380 |
blk->doit(intf, method_count); |
1381 |
} |
|
1382 |
} |
|
1383 |
} |
|
1384 |
||
1385 |
class CountInterfacesClosure : public InterfaceVisiterClosure { |
|
1386 |
private: |
|
1387 |
int _nof_methods; |
|
1388 |
int _nof_interfaces; |
|
1389 |
public: |
|
1390 |
CountInterfacesClosure() { _nof_methods = 0; _nof_interfaces = 0; } |
|
1391 |
||
1392 |
int nof_methods() const { return _nof_methods; } |
|
1393 |
int nof_interfaces() const { return _nof_interfaces; } |
|
1394 |
||
51329
9c68699bebe5
8208999: Some use of Klass* should be replaced by InstanceKlass*
iklam
parents:
51097
diff
changeset
|
1395 |
void doit(InstanceKlass* intf, int method_count) { _nof_methods += method_count; _nof_interfaces++; } |
1 | 1396 |
}; |
1397 |
||
1398 |
class SetupItableClosure : public InterfaceVisiterClosure { |
|
1399 |
private: |
|
1400 |
itableOffsetEntry* _offset_entry; |
|
1401 |
itableMethodEntry* _method_entry; |
|
1402 |
address _klass_begin; |
|
1403 |
public: |
|
1404 |
SetupItableClosure(address klass_begin, itableOffsetEntry* offset_entry, itableMethodEntry* method_entry) { |
|
1405 |
_klass_begin = klass_begin; |
|
1406 |
_offset_entry = offset_entry; |
|
1407 |
_method_entry = method_entry; |
|
1408 |
} |
|
1409 |
||
1410 |
itableMethodEntry* method_entry() const { return _method_entry; } |
|
1411 |
||
51329
9c68699bebe5
8208999: Some use of Klass* should be replaced by InstanceKlass*
iklam
parents:
51097
diff
changeset
|
1412 |
void doit(InstanceKlass* intf, int method_count) { |
1 | 1413 |
int offset = ((address)_method_entry) - _klass_begin; |
1414 |
_offset_entry->initialize(intf, offset); |
|
1415 |
_offset_entry++; |
|
1416 |
_method_entry += method_count; |
|
1417 |
} |
|
1418 |
}; |
|
1419 |
||
51329
9c68699bebe5
8208999: Some use of Klass* should be replaced by InstanceKlass*
iklam
parents:
51097
diff
changeset
|
1420 |
int klassItable::compute_itable_size(Array<InstanceKlass*>* transitive_interfaces) { |
1 | 1421 |
// Count no of interfaces and total number of interface methods |
1422 |
CountInterfacesClosure cic; |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
9172
diff
changeset
|
1423 |
visit_all_interfaces(transitive_interfaces, &cic); |
1 | 1424 |
|
189
4248c8e21063
6664627: Merge changes made only in hotspot 11 forward to jdk 7
dcubed
parents:
1
diff
changeset
|
1425 |
// There's alway an extra itable entry so we can null-terminate it. |
4248c8e21063
6664627: Merge changes made only in hotspot 11 forward to jdk 7
dcubed
parents:
1
diff
changeset
|
1426 |
int itable_size = calc_itable_size(cic.nof_interfaces() + 1, cic.nof_methods()); |
1 | 1427 |
|
1428 |
// Statistics |
|
35898
ddc274f0052f
8145628: hotspot metadata classes shouldn't use HeapWordSize or heap related macros like align_object_size
coleenp
parents:
35492
diff
changeset
|
1429 |
update_stats(itable_size * wordSize); |
1 | 1430 |
|
1431 |
return itable_size; |
|
1432 |
} |
|
1433 |
||
1434 |
||
1435 |
// Fill out offset table and interface klasses into the itable space |
|
46329
53ccc37bda19
8155672: Remove instanceKlassHandles and KlassHandles
coleenp
parents:
41669
diff
changeset
|
1436 |
void klassItable::setup_itable_offset_table(InstanceKlass* klass) { |
1 | 1437 |
if (klass->itable_length() == 0) return; |
1438 |
assert(!klass->is_interface(), "Should have zero length itable"); |
|
1439 |
||
1440 |
// Count no of interfaces and total number of interface methods |
|
1441 |
CountInterfacesClosure cic; |
|
1442 |
visit_all_interfaces(klass->transitive_interfaces(), &cic); |
|
1443 |
int nof_methods = cic.nof_methods(); |
|
1444 |
int nof_interfaces = cic.nof_interfaces(); |
|
1445 |
||
189
4248c8e21063
6664627: Merge changes made only in hotspot 11 forward to jdk 7
dcubed
parents:
1
diff
changeset
|
1446 |
// Add one extra entry so we can null-terminate the table |
4248c8e21063
6664627: Merge changes made only in hotspot 11 forward to jdk 7
dcubed
parents:
1
diff
changeset
|
1447 |
nof_interfaces++; |
1 | 1448 |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
9172
diff
changeset
|
1449 |
assert(compute_itable_size(klass->transitive_interfaces()) == |
1 | 1450 |
calc_itable_size(nof_interfaces, nof_methods), |
1451 |
"mismatch calculation of itable size"); |
|
1452 |
||
1453 |
// Fill-out offset table |
|
1454 |
itableOffsetEntry* ioe = (itableOffsetEntry*)klass->start_of_itable(); |
|
1455 |
itableMethodEntry* ime = (itableMethodEntry*)(ioe + nof_interfaces); |
|
1456 |
intptr_t* end = klass->end_of_itable(); |
|
8725
8c1e3dd5fe1b
7017732: move static fields into Class to prepare for perm gen removal
never
parents:
8653
diff
changeset
|
1457 |
assert((oop*)(ime + nof_methods) <= (oop*)klass->start_of_nonstatic_oop_maps(), "wrong offset calculation (1)"); |
360
21d113ecbf6a
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
189
diff
changeset
|
1458 |
assert((oop*)(end) == (oop*)(ime + nof_methods), "wrong offset calculation (2)"); |
1 | 1459 |
|
1460 |
// Visit all interfaces and initialize itable offset table |
|
46329
53ccc37bda19
8155672: Remove instanceKlassHandles and KlassHandles
coleenp
parents:
41669
diff
changeset
|
1461 |
SetupItableClosure sic((address)klass, ioe, ime); |
1 | 1462 |
visit_all_interfaces(klass->transitive_interfaces(), &sic); |
1463 |
||
1464 |
#ifdef ASSERT |
|
1465 |
ime = sic.method_entry(); |
|
1466 |
oop* v = (oop*) klass->end_of_itable(); |
|
1467 |
assert( (oop*)(ime) == v, "wrong offset calculation (2)"); |
|
1468 |
#endif |
|
1469 |
} |
|
1470 |
||
1471 |
||
20017
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
1472 |
// inverse to itable_index |
51329
9c68699bebe5
8208999: Some use of Klass* should be replaced by InstanceKlass*
iklam
parents:
51097
diff
changeset
|
1473 |
Method* klassItable::method_for_itable_index(InstanceKlass* intf, int itable_index) { |
9c68699bebe5
8208999: Some use of Klass* should be replaced by InstanceKlass*
iklam
parents:
51097
diff
changeset
|
1474 |
assert(intf->is_interface(), "sanity check"); |
20017
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
1475 |
assert(intf->verify_itable_index(itable_index), ""); |
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
9172
diff
changeset
|
1476 |
Array<Method*>* methods = InstanceKlass::cast(intf)->methods(); |
2332
5c7b6f4ce0a1
6814659: separable cleanups and subroutines for 6655638
jrose
parents:
2136
diff
changeset
|
1477 |
|
20017
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
1478 |
if (itable_index < 0 || itable_index >= method_count_for_interface(intf)) |
20391
7b146c5ebb18
8009130: Lambda: Fix access controls, loader constraints.
acorn
parents:
20284
diff
changeset
|
1479 |
return NULL; // help caller defend against bad indices |
20017
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
1480 |
|
2332
5c7b6f4ce0a1
6814659: separable cleanups and subroutines for 6655638
jrose
parents:
2136
diff
changeset
|
1481 |
int index = itable_index; |
20017
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
1482 |
Method* m = methods->at(index); |
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
1483 |
int index2 = -1; |
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
1484 |
while (!m->has_itable_index() || |
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
1485 |
(index2 = m->itable_index()) != itable_index) { |
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
1486 |
assert(index2 < itable_index, "monotonic"); |
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
1487 |
if (++index == methods->length()) |
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
1488 |
return NULL; |
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
1489 |
m = methods->at(index); |
2332
5c7b6f4ce0a1
6814659: separable cleanups and subroutines for 6655638
jrose
parents:
2136
diff
changeset
|
1490 |
} |
20017
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17092
diff
changeset
|
1491 |
assert(m->itable_index() == itable_index, "correct inverse"); |
2332
5c7b6f4ce0a1
6814659: separable cleanups and subroutines for 6655638
jrose
parents:
2136
diff
changeset
|
1492 |
|
5c7b6f4ce0a1
6814659: separable cleanups and subroutines for 6655638
jrose
parents:
2136
diff
changeset
|
1493 |
return m; |
5c7b6f4ce0a1
6814659: separable cleanups and subroutines for 6655638
jrose
parents:
2136
diff
changeset
|
1494 |
} |
5c7b6f4ce0a1
6814659: separable cleanups and subroutines for 6655638
jrose
parents:
2136
diff
changeset
|
1495 |
|
1 | 1496 |
void klassVtable::verify(outputStream* st, bool forced) { |
1497 |
// make sure table is initialized |
|
1498 |
if (!Universe::is_fully_initialized()) return; |
|
1499 |
#ifndef PRODUCT |
|
1500 |
// avoid redundant verifies |
|
1501 |
if (!forced && _verify_count == Universe::verify_count()) return; |
|
1502 |
_verify_count = Universe::verify_count(); |
|
1503 |
#endif |
|
46329
53ccc37bda19
8155672: Remove instanceKlassHandles and KlassHandles
coleenp
parents:
41669
diff
changeset
|
1504 |
oop* end_of_obj = (oop*)_klass + _klass->size(); |
1 | 1505 |
oop* end_of_vtable = (oop *)&table()[_length]; |
1506 |
if (end_of_vtable > end_of_obj) { |
|
51096
695dff91a997
8178712: ResourceMark may be missing inside initialize_[vi]table
lfoltan
parents:
50735
diff
changeset
|
1507 |
ResourceMark rm; |
33105
294e48b4f704
8080775: Better argument formatting for assert() and friends
david
parents:
32189
diff
changeset
|
1508 |
fatal("klass %s: klass object too short (vtable extends beyond end)", |
294e48b4f704
8080775: Better argument formatting for assert() and friends
david
parents:
32189
diff
changeset
|
1509 |
_klass->internal_name()); |
1 | 1510 |
} |
1511 |
||
1512 |
for (int i = 0; i < _length; i++) table()[i].verify(this, st); |
|
1513 |
// verify consistency with superKlass vtable |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
9172
diff
changeset
|
1514 |
Klass* super = _klass->super(); |
1 | 1515 |
if (super != NULL) { |
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
9172
diff
changeset
|
1516 |
InstanceKlass* sk = InstanceKlass::cast(super); |
46408
70aab0c2ea8b
8178350: klassVtable and klassItable should be ValueObj
iklam
parents:
46329
diff
changeset
|
1517 |
klassVtable vt = sk->vtable(); |
70aab0c2ea8b
8178350: klassVtable and klassItable should be ValueObj
iklam
parents:
46329
diff
changeset
|
1518 |
for (int i = 0; i < vt.length(); i++) { |
70aab0c2ea8b
8178350: klassVtable and klassItable should be ValueObj
iklam
parents:
46329
diff
changeset
|
1519 |
verify_against(st, &vt, i); |
1 | 1520 |
} |
1521 |
} |
|
1522 |
} |
|
1523 |
||
1524 |
void klassVtable::verify_against(outputStream* st, klassVtable* vt, int index) { |
|
1525 |
vtableEntry* vte = &vt->table()[index]; |
|
1526 |
if (vte->method()->name() != table()[index].method()->name() || |
|
1527 |
vte->method()->signature() != table()[index].method()->signature()) { |
|
1528 |
fatal("mismatched name/signature of vtable entries"); |
|
1529 |
} |
|
1530 |
} |
|
1531 |
||
1532 |
#ifndef PRODUCT |
|
1533 |
void klassVtable::print() { |
|
1534 |
ResourceMark rm; |
|
1535 |
tty->print("klassVtable for klass %s (length %d):\n", _klass->internal_name(), length()); |
|
1536 |
for (int i = 0; i < length(); i++) { |
|
1537 |
table()[i].print(); |
|
1538 |
tty->cr(); |
|
1539 |
} |
|
1540 |
} |
|
1541 |
#endif |
|
1542 |
||
1543 |
void vtableEntry::verify(klassVtable* vt, outputStream* st) { |
|
1544 |
NOT_PRODUCT(FlagSetting fs(IgnoreLockingAssertions, true)); |
|
46329
53ccc37bda19
8155672: Remove instanceKlassHandles and KlassHandles
coleenp
parents:
41669
diff
changeset
|
1545 |
Klass* vtklass = vt->klass(); |
40633
c33ad2ff51de
8163808: Fix asserts and logging for old classfile vtable construction
acorn
parents:
39714
diff
changeset
|
1546 |
if (vtklass->is_instance_klass() && |
c33ad2ff51de
8163808: Fix asserts and logging for old classfile vtable construction
acorn
parents:
39714
diff
changeset
|
1547 |
(InstanceKlass::cast(vtklass)->major_version() >= klassVtable::VTABLE_TRANSITIVE_OVERRIDE_VERSION)) { |
c33ad2ff51de
8163808: Fix asserts and logging for old classfile vtable construction
acorn
parents:
39714
diff
changeset
|
1548 |
assert(method() != NULL, "must have set method"); |
c33ad2ff51de
8163808: Fix asserts and logging for old classfile vtable construction
acorn
parents:
39714
diff
changeset
|
1549 |
} |
c33ad2ff51de
8163808: Fix asserts and logging for old classfile vtable construction
acorn
parents:
39714
diff
changeset
|
1550 |
if (method() != NULL) { |
c33ad2ff51de
8163808: Fix asserts and logging for old classfile vtable construction
acorn
parents:
39714
diff
changeset
|
1551 |
method()->verify(); |
c33ad2ff51de
8163808: Fix asserts and logging for old classfile vtable construction
acorn
parents:
39714
diff
changeset
|
1552 |
// we sub_type, because it could be a miranda method |
46329
53ccc37bda19
8155672: Remove instanceKlassHandles and KlassHandles
coleenp
parents:
41669
diff
changeset
|
1553 |
if (!vtklass->is_subtype_of(method()->method_holder())) { |
1 | 1554 |
#ifndef PRODUCT |
40633
c33ad2ff51de
8163808: Fix asserts and logging for old classfile vtable construction
acorn
parents:
39714
diff
changeset
|
1555 |
print(); |
1 | 1556 |
#endif |
40633
c33ad2ff51de
8163808: Fix asserts and logging for old classfile vtable construction
acorn
parents:
39714
diff
changeset
|
1557 |
fatal("vtableEntry " PTR_FORMAT ": method is from subclass", p2i(this)); |
c33ad2ff51de
8163808: Fix asserts and logging for old classfile vtable construction
acorn
parents:
39714
diff
changeset
|
1558 |
} |
c33ad2ff51de
8163808: Fix asserts and logging for old classfile vtable construction
acorn
parents:
39714
diff
changeset
|
1559 |
} |
1 | 1560 |
} |
1561 |
||
1562 |
#ifndef PRODUCT |
|
1563 |
||
1564 |
void vtableEntry::print() { |
|
1565 |
ResourceMark rm; |
|
1566 |
tty->print("vtableEntry %s: ", method()->name()->as_C_string()); |
|
1567 |
if (Verbose) { |
|
33148
68fa8b6c4340
8042893: compiler: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents:
33105
diff
changeset
|
1568 |
tty->print("m " PTR_FORMAT " ", p2i(method())); |
1 | 1569 |
} |
1570 |
} |
|
1571 |
||
1572 |
class VtableStats : AllStatic { |
|
1573 |
public: |
|
1574 |
static int no_klasses; // # classes with vtables |
|
1575 |
static int no_array_klasses; // # array classes |
|
1576 |
static int no_instance_klasses; // # instanceKlasses |
|
1577 |
static int sum_of_vtable_len; // total # of vtable entries |
|
1578 |
static int sum_of_array_vtable_len; // total # of vtable entries in array klasses only |
|
1579 |
static int fixed; // total fixed overhead in bytes |
|
1580 |
static int filler; // overhead caused by filler bytes |
|
1581 |
static int entries; // total bytes consumed by vtable entries |
|
1582 |
static int array_entries; // total bytes consumed by array vtable entries |
|
1583 |
||
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
9172
diff
changeset
|
1584 |
static void do_class(Klass* k) { |
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
9172
diff
changeset
|
1585 |
Klass* kl = k; |
46408
70aab0c2ea8b
8178350: klassVtable and klassItable should be ValueObj
iklam
parents:
46329
diff
changeset
|
1586 |
klassVtable vt = kl->vtable(); |
1 | 1587 |
no_klasses++; |
33611
9abd65805e19
8139203: Consistent naming for klass type predicates
coleenp
parents:
33602
diff
changeset
|
1588 |
if (kl->is_instance_klass()) { |
1 | 1589 |
no_instance_klasses++; |
1590 |
kl->array_klasses_do(do_class); |
|
1591 |
} |
|
33611
9abd65805e19
8139203: Consistent naming for klass type predicates
coleenp
parents:
33602
diff
changeset
|
1592 |
if (kl->is_array_klass()) { |
1 | 1593 |
no_array_klasses++; |
46408
70aab0c2ea8b
8178350: klassVtable and klassItable should be ValueObj
iklam
parents:
46329
diff
changeset
|
1594 |
sum_of_array_vtable_len += vt.length(); |
1 | 1595 |
} |
46408
70aab0c2ea8b
8178350: klassVtable and klassItable should be ValueObj
iklam
parents:
46329
diff
changeset
|
1596 |
sum_of_vtable_len += vt.length(); |
1 | 1597 |
} |
1598 |
||
1599 |
static void compute() { |
|
51608 | 1600 |
LockedClassesDo locked_do_class(&do_class); |
1601 |
ClassLoaderDataGraph::classes_do(&locked_do_class); |
|
1 | 1602 |
fixed = no_klasses * oopSize; // vtable length |
1603 |
// filler size is a conservative approximation |
|
13952
e3cf184080bc
8000213: NPG: Should have renamed arrayKlass and typeArrayKlass
coleenp
parents:
13728
diff
changeset
|
1604 |
filler = oopSize * (no_klasses - no_instance_klasses) * (sizeof(InstanceKlass) - sizeof(ArrayKlass) - 1); |
1 | 1605 |
entries = sizeof(vtableEntry) * sum_of_vtable_len; |
1606 |
array_entries = sizeof(vtableEntry) * sum_of_array_vtable_len; |
|
1607 |
} |
|
1608 |
}; |
|
1609 |
||
1610 |
int VtableStats::no_klasses = 0; |
|
1611 |
int VtableStats::no_array_klasses = 0; |
|
1612 |
int VtableStats::no_instance_klasses = 0; |
|
1613 |
int VtableStats::sum_of_vtable_len = 0; |
|
1614 |
int VtableStats::sum_of_array_vtable_len = 0; |
|
1615 |
int VtableStats::fixed = 0; |
|
1616 |
int VtableStats::filler = 0; |
|
1617 |
int VtableStats::entries = 0; |
|
1618 |
int VtableStats::array_entries = 0; |
|
1619 |
||
1620 |
void klassVtable::print_statistics() { |
|
1621 |
ResourceMark rm; |
|
1622 |
HandleMark hm; |
|
1623 |
VtableStats::compute(); |
|
1624 |
tty->print_cr("vtable statistics:"); |
|
1625 |
tty->print_cr("%6d classes (%d instance, %d array)", VtableStats::no_klasses, VtableStats::no_instance_klasses, VtableStats::no_array_klasses); |
|
1626 |
int total = VtableStats::fixed + VtableStats::filler + VtableStats::entries; |
|
1627 |
tty->print_cr("%6d bytes fixed overhead (refs + vtable object header)", VtableStats::fixed); |
|
1628 |
tty->print_cr("%6d bytes filler overhead", VtableStats::filler); |
|
1629 |
tty->print_cr("%6d bytes for vtable entries (%d for arrays)", VtableStats::entries, VtableStats::array_entries); |
|
1630 |
tty->print_cr("%6d bytes total", total); |
|
1631 |
} |
|
1632 |
||
1633 |
int klassItable::_total_classes; // Total no. of classes with itables |
|
1634 |
long klassItable::_total_size; // Total no. of bytes used for itables |
|
1635 |
||
1636 |
void klassItable::print_statistics() { |
|
1637 |
tty->print_cr("itable statistics:"); |
|
1638 |
tty->print_cr("%6d classes with itables", _total_classes); |
|
33148
68fa8b6c4340
8042893: compiler: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents:
33105
diff
changeset
|
1639 |
tty->print_cr("%6lu K uses for itables (average by class: %ld bytes)", _total_size / K, _total_size / _total_classes); |
1 | 1640 |
} |
1641 |
||
1642 |
#endif // PRODUCT |