Mercurial > hg > truffle
annotate src/share/vm/oops/cpCache.cpp @ 8074:b9c5e46bf915
8008188: Add regression test for 8005875
Summary: Add regression test for crash seen in 8005875. Test is run with G1 and PGCT=0 and issues "jcmd <pid> Thread.print" against itself. Without the fix for 8005875 the test will crash.
Reviewed-by: brutisso
author | johnc |
---|---|
date | Wed, 20 Feb 2013 12:52:13 -0800 |
parents | 461a3adac4d1 |
children | 3efdfd6ddbf2 |
rev | line source |
---|---|
0 | 1 /* |
8021
8d9fc28831cc
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
6985
diff
changeset
|
2 * Copyright (c) 1998, 2013, Oracle and/or its affiliates. All rights reserved. |
0 | 3 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. |
4 * | |
5 * This code is free software; you can redistribute it and/or modify it | |
6 * under the terms of the GNU General Public License version 2 only, as | |
7 * published by the Free Software Foundation. | |
8 * | |
9 * This code is distributed in the hope that it will be useful, but WITHOUT | |
10 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or | |
11 * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License | |
12 * version 2 for more details (a copy is included in the LICENSE file that | |
13 * accompanied this code). | |
14 * | |
15 * You should have received a copy of the GNU General Public License version | |
16 * 2 along with this work; if not, write to the Free Software Foundation, | |
17 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA. | |
18 * | |
1552
c18cbe5936b8
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
1507
diff
changeset
|
19 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA |
c18cbe5936b8
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
1507
diff
changeset
|
20 * or visit www.oracle.com if you need additional information or have any |
c18cbe5936b8
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
1507
diff
changeset
|
21 * questions. |
0 | 22 * |
23 */ | |
24 | |
1972 | 25 #include "precompiled.hpp" |
26 #include "gc_implementation/shared/markSweep.inline.hpp" | |
27 #include "interpreter/interpreter.hpp" | |
28 #include "interpreter/rewriter.hpp" | |
29 #include "memory/universe.inline.hpp" | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
30 #include "oops/cpCache.hpp" |
1972 | 31 #include "oops/objArrayOop.hpp" |
32 #include "oops/oop.inline.hpp" | |
33 #include "prims/jvmtiRedefineClassesTrace.hpp" | |
6266
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
34 #include "prims/methodHandles.hpp" |
1972 | 35 #include "runtime/handles.inline.hpp" |
8001
db9981fd3124
8005915: Unify SERIALGC and INCLUDE_ALTERNATE_GCS
jprovino
parents:
6985
diff
changeset
|
36 #include "utilities/macros.hpp" |
db9981fd3124
8005915: Unify SERIALGC and INCLUDE_ALTERNATE_GCS
jprovino
parents:
6985
diff
changeset
|
37 #if INCLUDE_ALL_GCS |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
38 # include "gc_implementation/parallelScavenge/psPromotionManager.hpp" |
8001
db9981fd3124
8005915: Unify SERIALGC and INCLUDE_ALTERNATE_GCS
jprovino
parents:
6985
diff
changeset
|
39 #endif // INCLUDE_ALL_GCS |
0 | 40 |
41 | |
42 // Implememtation of ConstantPoolCacheEntry | |
43 | |
1059
389049f3f393
6858164: invokedynamic code needs some cleanup (post-6655638)
jrose
parents:
844
diff
changeset
|
44 void ConstantPoolCacheEntry::initialize_entry(int index) { |
726
be93aad57795
6655646: dynamic languages need dynamically linked call sites
jrose
parents:
196
diff
changeset
|
45 assert(0 < index && index < 0x10000, "sanity check"); |
0 | 46 _indices = index; |
726
be93aad57795
6655646: dynamic languages need dynamically linked call sites
jrose
parents:
196
diff
changeset
|
47 assert(constant_pool_index() == index, ""); |
0 | 48 } |
49 | |
6266
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
50 int ConstantPoolCacheEntry::make_flags(TosState state, |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
51 int option_bits, |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
52 int field_index_or_method_params) { |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
53 assert(state < number_of_states, "Invalid state in make_flags"); |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
54 int f = ((int)state << tos_state_shift) | option_bits | field_index_or_method_params; |
0 | 55 // Preserve existing flag bit values |
6266
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
56 // The low bits are a field offset, or else the method parameter size. |
0 | 57 #ifdef ASSERT |
6266
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
58 TosState old_state = flag_state(); |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
59 assert(old_state == (TosState)0 || old_state == state, |
0 | 60 "inconsistent cpCache flags state"); |
61 #endif | |
62 return (_flags | f) ; | |
63 } | |
64 | |
65 void ConstantPoolCacheEntry::set_bytecode_1(Bytecodes::Code code) { | |
66 #ifdef ASSERT | |
67 // Read once. | |
68 volatile Bytecodes::Code c = bytecode_1(); | |
69 assert(c == 0 || c == code || code == 0, "update must be consistent"); | |
70 #endif | |
71 // Need to flush pending stores here before bytecode is written. | |
6266
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
72 OrderAccess::release_store_ptr(&_indices, _indices | ((u_char)code << bytecode_1_shift)); |
0 | 73 } |
74 | |
75 void ConstantPoolCacheEntry::set_bytecode_2(Bytecodes::Code code) { | |
76 #ifdef ASSERT | |
77 // Read once. | |
78 volatile Bytecodes::Code c = bytecode_2(); | |
79 assert(c == 0 || c == code || code == 0, "update must be consistent"); | |
80 #endif | |
81 // Need to flush pending stores here before bytecode is written. | |
6266
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
82 OrderAccess::release_store_ptr(&_indices, _indices | ((u_char)code << bytecode_2_shift)); |
0 | 83 } |
84 | |
6266
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
85 // Sets f1, ordering with previous writes. |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
86 void ConstantPoolCacheEntry::release_set_f1(Metadata* f1) { |
6266
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
87 assert(f1 != NULL, ""); |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
88 OrderAccess::release_store_ptr((HeapWord*) &_f1, f1); |
6266
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
89 } |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
90 |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
91 // Sets flags, but only if the value was previously zero. |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
92 bool ConstantPoolCacheEntry::init_flags_atomic(intptr_t flags) { |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
93 intptr_t result = Atomic::cmpxchg_ptr(flags, &_flags, 0); |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
94 return (result == 0); |
2246
fbf3184da15d
7012339: JSR 292 crash in G1SATBCardTableModRefBS::write_ref_field_pre_work()
twisti
parents:
1972
diff
changeset
|
95 } |
1903 | 96 |
0 | 97 // Note that concurrent update of both bytecodes can leave one of them |
98 // reset to zero. This is harmless; the interpreter will simply re-resolve | |
99 // the damaged entry. More seriously, the memory synchronization is needed | |
100 // to flush other fields (f1, f2) completely to memory before the bytecodes | |
101 // are updated, lest other processors see a non-zero bytecode but zero f1/f2. | |
102 void ConstantPoolCacheEntry::set_field(Bytecodes::Code get_code, | |
103 Bytecodes::Code put_code, | |
104 KlassHandle field_holder, | |
3938 | 105 int field_index, |
0 | 106 int field_offset, |
107 TosState field_type, | |
108 bool is_final, | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
109 bool is_volatile, |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
110 Klass* root_klass) { |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
111 set_f1(field_holder()); |
0 | 112 set_f2(field_offset); |
6266
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
113 assert((field_index & field_index_mask) == field_index, |
0 | 114 "field index does not fit in low flag bits"); |
6266
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
115 set_field_flags(field_type, |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
116 ((is_volatile ? 1 : 0) << is_volatile_shift) | |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
117 ((is_final ? 1 : 0) << is_final_shift), |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
118 field_index); |
0 | 119 set_bytecode_1(get_code); |
120 set_bytecode_2(put_code); | |
121 NOT_PRODUCT(verify(tty)); | |
122 } | |
123 | |
6266
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
124 void ConstantPoolCacheEntry::set_parameter_size(int value) { |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
125 // This routine is called only in corner cases where the CPCE is not yet initialized. |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
126 // See AbstractInterpreter::deopt_continue_after_entry. |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
127 assert(_flags == 0 || parameter_size() == 0 || parameter_size() == value, |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
128 err_msg("size must not change: parameter_size=%d, value=%d", parameter_size(), value)); |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
129 // Setting the parameter size by itself is only safe if the |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
130 // current value of _flags is 0, otherwise another thread may have |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
131 // updated it and we don't want to overwrite that value. Don't |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
132 // bother trying to update it once it's nonzero but always make |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
133 // sure that the final parameter size agrees with what was passed. |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
134 if (_flags == 0) { |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
135 Atomic::cmpxchg_ptr((value & parameter_size_mask), &_flags, 0); |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
136 } |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
137 guarantee(parameter_size() == value, |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
138 err_msg("size must not change: parameter_size=%d, value=%d", parameter_size(), value)); |
0 | 139 } |
140 | |
141 void ConstantPoolCacheEntry::set_method(Bytecodes::Code invoke_code, | |
142 methodHandle method, | |
143 int vtable_index) { | |
144 assert(method->interpreter_entry() != NULL, "should have been set at this point"); | |
145 assert(!method->is_obsolete(), "attempt to write obsolete method to cpCache"); | |
146 | |
147 int byte_no = -1; | |
6266
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
148 bool change_to_virtual = false; |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
149 |
0 | 150 switch (invoke_code) { |
6266
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
151 case Bytecodes::_invokeinterface: |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
152 // We get here from InterpreterRuntime::resolve_invoke when an invokeinterface |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
153 // instruction somehow links to a non-interface method (in Object). |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
154 // In that case, the method has no itable index and must be invoked as a virtual. |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
155 // Set a flag to keep track of this corner case. |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
156 change_to_virtual = true; |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
157 |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
158 // ...and fall through as if we were handling invokevirtual: |
0 | 159 case Bytecodes::_invokevirtual: |
6266
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
160 { |
0 | 161 if (method->can_be_statically_bound()) { |
6266
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
162 // set_f2_as_vfinal_method checks if is_vfinal flag is true. |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
163 set_method_flags(as_TosState(method->result_type()), |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
164 ( 1 << is_vfinal_shift) | |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
165 ((method->is_final_method() ? 1 : 0) << is_final_shift) | |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
166 ((change_to_virtual ? 1 : 0) << is_forced_virtual_shift), |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
167 method()->size_of_parameters()); |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
168 set_f2_as_vfinal_method(method()); |
0 | 169 } else { |
170 assert(vtable_index >= 0, "valid index"); | |
6266
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
171 assert(!method->is_final_method(), "sanity"); |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
172 set_method_flags(as_TosState(method->result_type()), |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
173 ((change_to_virtual ? 1 : 0) << is_forced_virtual_shift), |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
174 method()->size_of_parameters()); |
0 | 175 set_f2(vtable_index); |
176 } | |
177 byte_no = 2; | |
178 break; | |
1660
083fde3b838e
6964498: JSR 292 invokedynamic sites need local bootstrap methods
jrose
parents:
1552
diff
changeset
|
179 } |
083fde3b838e
6964498: JSR 292 invokedynamic sites need local bootstrap methods
jrose
parents:
1552
diff
changeset
|
180 |
0 | 181 case Bytecodes::_invokespecial: |
182 case Bytecodes::_invokestatic: | |
6266
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
183 // Note: Read and preserve the value of the is_vfinal flag on any |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
184 // invokevirtual bytecode shared with this constant pool cache entry. |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
185 // It is cheap and safe to consult is_vfinal() at all times. |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
186 // Once is_vfinal is set, it must stay that way, lest we get a dangling oop. |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
187 set_method_flags(as_TosState(method->result_type()), |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
188 ((is_vfinal() ? 1 : 0) << is_vfinal_shift) | |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
189 ((method->is_final_method() ? 1 : 0) << is_final_shift), |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
190 method()->size_of_parameters()); |
0 | 191 set_f1(method()); |
192 byte_no = 1; | |
193 break; | |
194 default: | |
195 ShouldNotReachHere(); | |
196 break; | |
197 } | |
198 | |
199 // Note: byte_no also appears in TemplateTable::resolve. | |
200 if (byte_no == 1) { | |
6266
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
201 assert(invoke_code != Bytecodes::_invokevirtual && |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
202 invoke_code != Bytecodes::_invokeinterface, ""); |
0 | 203 set_bytecode_1(invoke_code); |
204 } else if (byte_no == 2) { | |
205 if (change_to_virtual) { | |
6266
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
206 assert(invoke_code == Bytecodes::_invokeinterface, ""); |
0 | 207 // NOTE: THIS IS A HACK - BE VERY CAREFUL!!! |
208 // | |
209 // Workaround for the case where we encounter an invokeinterface, but we | |
210 // should really have an _invokevirtual since the resolved method is a | |
211 // virtual method in java.lang.Object. This is a corner case in the spec | |
212 // but is presumably legal. javac does not generate this code. | |
213 // | |
214 // We set bytecode_1() to _invokeinterface, because that is the | |
215 // bytecode # used by the interpreter to see if it is resolved. | |
216 // We set bytecode_2() to _invokevirtual. | |
217 // See also interpreterRuntime.cpp. (8/25/2000) | |
218 // Only set resolved for the invokeinterface case if method is public. | |
219 // Otherwise, the method needs to be reresolved with caller for each | |
220 // interface call. | |
221 if (method->is_public()) set_bytecode_1(invoke_code); | |
222 } else { | |
6266
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
223 assert(invoke_code == Bytecodes::_invokevirtual, ""); |
0 | 224 } |
6266
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
225 // set up for invokevirtual, even if linking for invokeinterface also: |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
226 set_bytecode_2(Bytecodes::_invokevirtual); |
0 | 227 } else { |
228 ShouldNotReachHere(); | |
229 } | |
230 NOT_PRODUCT(verify(tty)); | |
231 } | |
232 | |
233 | |
234 void ConstantPoolCacheEntry::set_interface_call(methodHandle method, int index) { | |
6940
18fb7da42534
8000725: NPG: method_holder() and pool_holder() and pool_holder field should be InstanceKlass
coleenp
parents:
6822
diff
changeset
|
235 InstanceKlass* interf = method->method_holder(); |
18fb7da42534
8000725: NPG: method_holder() and pool_holder() and pool_holder field should be InstanceKlass
coleenp
parents:
6822
diff
changeset
|
236 assert(interf->is_interface(), "must be an interface"); |
6266
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
237 assert(!method->is_final_method(), "interfaces do not have final methods; cannot link to one here"); |
0 | 238 set_f1(interf); |
239 set_f2(index); | |
6266
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
240 set_method_flags(as_TosState(method->result_type()), |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
241 0, // no option bits |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
242 method()->size_of_parameters()); |
0 | 243 set_bytecode_1(Bytecodes::_invokeinterface); |
244 } | |
245 | |
246 | |
6942
64672b22ef05
8001658: No need to pass resolved_references as argument to ConstantPoolCacheEntry::set_method_handle_common
twisti
parents:
6822
diff
changeset
|
247 void ConstantPoolCacheEntry::set_method_handle(constantPoolHandle cpool, const CallInfo &call_info) { |
64672b22ef05
8001658: No need to pass resolved_references as argument to ConstantPoolCacheEntry::set_method_handle_common
twisti
parents:
6822
diff
changeset
|
248 set_method_handle_common(cpool, Bytecodes::_invokehandle, call_info); |
1660
083fde3b838e
6964498: JSR 292 invokedynamic sites need local bootstrap methods
jrose
parents:
1552
diff
changeset
|
249 } |
083fde3b838e
6964498: JSR 292 invokedynamic sites need local bootstrap methods
jrose
parents:
1552
diff
changeset
|
250 |
6942
64672b22ef05
8001658: No need to pass resolved_references as argument to ConstantPoolCacheEntry::set_method_handle_common
twisti
parents:
6822
diff
changeset
|
251 void ConstantPoolCacheEntry::set_dynamic_call(constantPoolHandle cpool, const CallInfo &call_info) { |
64672b22ef05
8001658: No need to pass resolved_references as argument to ConstantPoolCacheEntry::set_method_handle_common
twisti
parents:
6822
diff
changeset
|
252 set_method_handle_common(cpool, Bytecodes::_invokedynamic, call_info); |
1660
083fde3b838e
6964498: JSR 292 invokedynamic sites need local bootstrap methods
jrose
parents:
1552
diff
changeset
|
253 } |
083fde3b838e
6964498: JSR 292 invokedynamic sites need local bootstrap methods
jrose
parents:
1552
diff
changeset
|
254 |
6737
4bfe8b33cf66
7196242: vm/mlvm/indy/stress/java/loopsAndThreads crashed
twisti
parents:
6725
diff
changeset
|
255 void ConstantPoolCacheEntry::set_method_handle_common(constantPoolHandle cpool, |
4bfe8b33cf66
7196242: vm/mlvm/indy/stress/java/loopsAndThreads crashed
twisti
parents:
6725
diff
changeset
|
256 Bytecodes::Code invoke_code, |
6942
64672b22ef05
8001658: No need to pass resolved_references as argument to ConstantPoolCacheEntry::set_method_handle_common
twisti
parents:
6822
diff
changeset
|
257 const CallInfo &call_info) { |
6266
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
258 // NOTE: This CPCE can be the subject of data races. |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
259 // There are three words to update: flags, refs[f2], f1 (in that order). |
6266
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
260 // Writers must store all other values before f1. |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
261 // Readers must test f1 first for non-null before reading other fields. |
6737
4bfe8b33cf66
7196242: vm/mlvm/indy/stress/java/loopsAndThreads crashed
twisti
parents:
6725
diff
changeset
|
262 // Competing writers must acquire exclusive access via a lock. |
4bfe8b33cf66
7196242: vm/mlvm/indy/stress/java/loopsAndThreads crashed
twisti
parents:
6725
diff
changeset
|
263 // A losing writer waits on the lock until the winner writes f1 and leaves |
4bfe8b33cf66
7196242: vm/mlvm/indy/stress/java/loopsAndThreads crashed
twisti
parents:
6725
diff
changeset
|
264 // the lock, so that when the losing writer returns, he can use the linked |
4bfe8b33cf66
7196242: vm/mlvm/indy/stress/java/loopsAndThreads crashed
twisti
parents:
6725
diff
changeset
|
265 // cache entry. |
4bfe8b33cf66
7196242: vm/mlvm/indy/stress/java/loopsAndThreads crashed
twisti
parents:
6725
diff
changeset
|
266 |
4bfe8b33cf66
7196242: vm/mlvm/indy/stress/java/loopsAndThreads crashed
twisti
parents:
6725
diff
changeset
|
267 MonitorLockerEx ml(cpool->lock()); |
4bfe8b33cf66
7196242: vm/mlvm/indy/stress/java/loopsAndThreads crashed
twisti
parents:
6725
diff
changeset
|
268 if (!is_f1_null()) { |
4bfe8b33cf66
7196242: vm/mlvm/indy/stress/java/loopsAndThreads crashed
twisti
parents:
6725
diff
changeset
|
269 return; |
4bfe8b33cf66
7196242: vm/mlvm/indy/stress/java/loopsAndThreads crashed
twisti
parents:
6725
diff
changeset
|
270 } |
6266
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
271 |
6942
64672b22ef05
8001658: No need to pass resolved_references as argument to ConstantPoolCacheEntry::set_method_handle_common
twisti
parents:
6822
diff
changeset
|
272 const methodHandle adapter = call_info.resolved_method(); |
64672b22ef05
8001658: No need to pass resolved_references as argument to ConstantPoolCacheEntry::set_method_handle_common
twisti
parents:
6822
diff
changeset
|
273 const Handle appendix = call_info.resolved_appendix(); |
64672b22ef05
8001658: No need to pass resolved_references as argument to ConstantPoolCacheEntry::set_method_handle_common
twisti
parents:
6822
diff
changeset
|
274 const Handle method_type = call_info.resolved_method_type(); |
6822
f6b0eb4e44cf
7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents:
6737
diff
changeset
|
275 const bool has_appendix = appendix.not_null(); |
f6b0eb4e44cf
7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents:
6737
diff
changeset
|
276 const bool has_method_type = method_type.not_null(); |
6266
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
277 |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
278 // Write the flags. |
6737
4bfe8b33cf66
7196242: vm/mlvm/indy/stress/java/loopsAndThreads crashed
twisti
parents:
6725
diff
changeset
|
279 set_method_flags(as_TosState(adapter->result_type()), |
6822
f6b0eb4e44cf
7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents:
6737
diff
changeset
|
280 ((has_appendix ? 1 : 0) << has_appendix_shift ) | |
f6b0eb4e44cf
7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents:
6737
diff
changeset
|
281 ((has_method_type ? 1 : 0) << has_method_type_shift) | |
f6b0eb4e44cf
7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents:
6737
diff
changeset
|
282 ( 1 << is_final_shift ), |
6266
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
283 adapter->size_of_parameters()); |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
284 |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
285 if (TraceInvokeDynamic) { |
6822
f6b0eb4e44cf
7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents:
6737
diff
changeset
|
286 tty->print_cr("set_method_handle bc=%d appendix="PTR_FORMAT"%s method_type="PTR_FORMAT"%s method="PTR_FORMAT" ", |
6266
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
287 invoke_code, |
6822
f6b0eb4e44cf
7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents:
6737
diff
changeset
|
288 (intptr_t)appendix(), (has_appendix ? "" : " (unused)"), |
f6b0eb4e44cf
7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents:
6737
diff
changeset
|
289 (intptr_t)method_type(), (has_method_type ? "" : " (unused)"), |
6266
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
290 (intptr_t)adapter()); |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
291 adapter->print(); |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
292 if (has_appendix) appendix()->print(); |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
293 } |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
294 |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
295 // Method handle invokes and invokedynamic sites use both cp cache words. |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
296 // refs[f2], if not null, contains a value passed as a trailing argument to the adapter. |
6266
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
297 // In the general case, this could be the call site's MethodType, |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
298 // for use with java.lang.Invokers.checkExactType, or else a CallSite object. |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
299 // f1 contains the adapter method which manages the actual call. |
6266
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
300 // In the general case, this is a compiled LambdaForm. |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
301 // (The Java code is free to optimize these calls by binding other |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
302 // sorts of methods and appendices to call sites.) |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
303 // JVM-level linking is via f1, as if for invokespecial, and signatures are erased. |
6266
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
304 // The appendix argument (if any) is added to the signature, and is counted in the parameter_size bits. |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
305 // Even with the appendix, the method will never take more than 255 parameter slots. |
6266
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
306 // |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
307 // This means that given a call site like (List)mh.invoke("foo"), |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
308 // the f1 method has signature '(Ljl/Object;Ljl/invoke/MethodType;)Ljl/Object;', |
6266
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
309 // not '(Ljava/lang/String;)Ljava/util/List;'. |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
310 // The fact that String and List are involved is encoded in the MethodType in refs[f2]. |
6266
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
311 // This allows us to create fewer method oops, while keeping type safety. |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
312 // |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
313 |
6942
64672b22ef05
8001658: No need to pass resolved_references as argument to ConstantPoolCacheEntry::set_method_handle_common
twisti
parents:
6822
diff
changeset
|
314 objArrayHandle resolved_references = cpool->resolved_references(); |
6822
f6b0eb4e44cf
7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents:
6737
diff
changeset
|
315 // Store appendix, if any. |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
316 if (has_appendix) { |
6822
f6b0eb4e44cf
7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents:
6737
diff
changeset
|
317 const int appendix_index = f2_as_index() + _indy_resolved_references_appendix_offset; |
f6b0eb4e44cf
7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents:
6737
diff
changeset
|
318 assert(appendix_index >= 0 && appendix_index < resolved_references->length(), "oob"); |
f6b0eb4e44cf
7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents:
6737
diff
changeset
|
319 assert(resolved_references->obj_at(appendix_index) == NULL, "init just once"); |
f6b0eb4e44cf
7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents:
6737
diff
changeset
|
320 resolved_references->obj_at_put(appendix_index, appendix()); |
f6b0eb4e44cf
7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents:
6737
diff
changeset
|
321 } |
f6b0eb4e44cf
7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents:
6737
diff
changeset
|
322 |
f6b0eb4e44cf
7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents:
6737
diff
changeset
|
323 // Store MethodType, if any. |
f6b0eb4e44cf
7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents:
6737
diff
changeset
|
324 if (has_method_type) { |
f6b0eb4e44cf
7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents:
6737
diff
changeset
|
325 const int method_type_index = f2_as_index() + _indy_resolved_references_method_type_offset; |
f6b0eb4e44cf
7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents:
6737
diff
changeset
|
326 assert(method_type_index >= 0 && method_type_index < resolved_references->length(), "oob"); |
f6b0eb4e44cf
7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents:
6737
diff
changeset
|
327 assert(resolved_references->obj_at(method_type_index) == NULL, "init just once"); |
f6b0eb4e44cf
7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents:
6737
diff
changeset
|
328 resolved_references->obj_at_put(method_type_index, method_type()); |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
329 } |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
330 |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
331 release_set_f1(adapter()); // This must be the last one to set (see NOTE above)! |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
332 |
6822
f6b0eb4e44cf
7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents:
6737
diff
changeset
|
333 // The interpreter assembly code does not check byte_2, |
f6b0eb4e44cf
7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents:
6737
diff
changeset
|
334 // but it is used by is_resolved, method_if_resolved, etc. |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
335 set_bytecode_1(invoke_code); |
6266
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
336 NOT_PRODUCT(verify(tty)); |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
337 if (TraceInvokeDynamic) { |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
338 this->print(tty, 0); |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
339 } |
726
be93aad57795
6655646: dynamic languages need dynamically linked call sites
jrose
parents:
196
diff
changeset
|
340 } |
be93aad57795
6655646: dynamic languages need dynamically linked call sites
jrose
parents:
196
diff
changeset
|
341 |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
342 Method* ConstantPoolCacheEntry::method_if_resolved(constantPoolHandle cpool) { |
3785
ddd894528dbc
7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents:
3279
diff
changeset
|
343 // Decode the action of set_method and set_interface_call |
6266
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
344 Bytecodes::Code invoke_code = bytecode_1(); |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
345 if (invoke_code != (Bytecodes::Code)0) { |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
346 Metadata* f1 = (Metadata*)_f1; |
3785
ddd894528dbc
7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents:
3279
diff
changeset
|
347 if (f1 != NULL) { |
ddd894528dbc
7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents:
3279
diff
changeset
|
348 switch (invoke_code) { |
ddd894528dbc
7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents:
3279
diff
changeset
|
349 case Bytecodes::_invokeinterface: |
ddd894528dbc
7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents:
3279
diff
changeset
|
350 assert(f1->is_klass(), ""); |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
351 return klassItable::method_for_itable_index((Klass*)f1, f2_as_index()); |
3785
ddd894528dbc
7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents:
3279
diff
changeset
|
352 case Bytecodes::_invokestatic: |
ddd894528dbc
7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents:
3279
diff
changeset
|
353 case Bytecodes::_invokespecial: |
6266
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
354 assert(!has_appendix(), ""); |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
355 case Bytecodes::_invokehandle: |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
356 case Bytecodes::_invokedynamic: |
3785
ddd894528dbc
7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents:
3279
diff
changeset
|
357 assert(f1->is_method(), ""); |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
358 return (Method*)f1; |
3785
ddd894528dbc
7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents:
3279
diff
changeset
|
359 } |
ddd894528dbc
7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents:
3279
diff
changeset
|
360 } |
ddd894528dbc
7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents:
3279
diff
changeset
|
361 } |
6266
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
362 invoke_code = bytecode_2(); |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
363 if (invoke_code != (Bytecodes::Code)0) { |
3785
ddd894528dbc
7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents:
3279
diff
changeset
|
364 switch (invoke_code) { |
ddd894528dbc
7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents:
3279
diff
changeset
|
365 case Bytecodes::_invokevirtual: |
ddd894528dbc
7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents:
3279
diff
changeset
|
366 if (is_vfinal()) { |
ddd894528dbc
7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents:
3279
diff
changeset
|
367 // invokevirtual |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
368 Method* m = f2_as_vfinal_method(); |
3785
ddd894528dbc
7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents:
3279
diff
changeset
|
369 assert(m->is_method(), ""); |
ddd894528dbc
7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents:
3279
diff
changeset
|
370 return m; |
ddd894528dbc
7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents:
3279
diff
changeset
|
371 } else { |
ddd894528dbc
7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents:
3279
diff
changeset
|
372 int holder_index = cpool->uncached_klass_ref_index_at(constant_pool_index()); |
ddd894528dbc
7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents:
3279
diff
changeset
|
373 if (cpool->tag_at(holder_index).is_klass()) { |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
374 Klass* klass = cpool->resolved_klass_at(holder_index); |
6983 | 375 if (!klass->oop_is_instance()) |
3785
ddd894528dbc
7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents:
3279
diff
changeset
|
376 klass = SystemDictionary::Object_klass(); |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
377 return InstanceKlass::cast(klass)->method_at_vtable(f2_as_index()); |
3785
ddd894528dbc
7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents:
3279
diff
changeset
|
378 } |
ddd894528dbc
7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents:
3279
diff
changeset
|
379 } |
6266
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
380 break; |
3785
ddd894528dbc
7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents:
3279
diff
changeset
|
381 } |
ddd894528dbc
7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents:
3279
diff
changeset
|
382 } |
ddd894528dbc
7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents:
3279
diff
changeset
|
383 return NULL; |
ddd894528dbc
7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents:
3279
diff
changeset
|
384 } |
ddd894528dbc
7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents:
3279
diff
changeset
|
385 |
ddd894528dbc
7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents:
3279
diff
changeset
|
386 |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
387 oop ConstantPoolCacheEntry::appendix_if_resolved(constantPoolHandle cpool) { |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
388 if (is_f1_null() || !has_appendix()) |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
389 return NULL; |
6822
f6b0eb4e44cf
7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents:
6737
diff
changeset
|
390 const int ref_index = f2_as_index() + _indy_resolved_references_appendix_offset; |
f6b0eb4e44cf
7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents:
6737
diff
changeset
|
391 objArrayOop resolved_references = cpool->resolved_references(); |
f6b0eb4e44cf
7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents:
6737
diff
changeset
|
392 return resolved_references->obj_at(ref_index); |
f6b0eb4e44cf
7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents:
6737
diff
changeset
|
393 } |
f6b0eb4e44cf
7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents:
6737
diff
changeset
|
394 |
f6b0eb4e44cf
7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents:
6737
diff
changeset
|
395 |
f6b0eb4e44cf
7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents:
6737
diff
changeset
|
396 oop ConstantPoolCacheEntry::method_type_if_resolved(constantPoolHandle cpool) { |
f6b0eb4e44cf
7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents:
6737
diff
changeset
|
397 if (is_f1_null() || !has_method_type()) |
f6b0eb4e44cf
7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents:
6737
diff
changeset
|
398 return NULL; |
f6b0eb4e44cf
7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents:
6737
diff
changeset
|
399 const int ref_index = f2_as_index() + _indy_resolved_references_method_type_offset; |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
400 objArrayOop resolved_references = cpool->resolved_references(); |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
401 return resolved_references->obj_at(ref_index); |
0 | 402 } |
403 | |
404 | |
8021
8d9fc28831cc
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
6985
diff
changeset
|
405 #if INCLUDE_JVMTI |
0 | 406 // RedefineClasses() API support: |
8021
8d9fc28831cc
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
6985
diff
changeset
|
407 // If this ConstantPoolCacheEntry refers to old_method then update it |
0 | 408 // to refer to new_method. |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
409 bool ConstantPoolCacheEntry::adjust_method_entry(Method* old_method, |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
410 Method* new_method, bool * trace_name_printed) { |
0 | 411 |
412 if (is_vfinal()) { | |
6266
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
413 // virtual and final so _f2 contains method ptr instead of vtable index |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
414 if (f2_as_vfinal_method() == old_method) { |
0 | 415 // match old_method so need an update |
6266
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
416 // NOTE: can't use set_f2_as_vfinal_method as it asserts on different values |
0 | 417 _f2 = (intptr_t)new_method; |
418 if (RC_TRACE_IN_RANGE(0x00100000, 0x00400000)) { | |
419 if (!(*trace_name_printed)) { | |
420 // RC_TRACE_MESG macro has an embedded ResourceMark | |
421 RC_TRACE_MESG(("adjust: name=%s", | |
6940
18fb7da42534
8000725: NPG: method_holder() and pool_holder() and pool_holder field should be InstanceKlass
coleenp
parents:
6822
diff
changeset
|
422 old_method->method_holder()->external_name())); |
0 | 423 *trace_name_printed = true; |
424 } | |
425 // RC_TRACE macro has an embedded ResourceMark | |
426 RC_TRACE(0x00400000, ("cpc vf-entry update: %s(%s)", | |
427 new_method->name()->as_C_string(), | |
428 new_method->signature()->as_C_string())); | |
429 } | |
430 | |
431 return true; | |
432 } | |
433 | |
434 // f1() is not used with virtual entries so bail out | |
435 return false; | |
436 } | |
437 | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
438 if (_f1 == NULL) { |
0 | 439 // NULL f1() means this is a virtual entry so bail out |
440 // We are assuming that the vtable index does not need change. | |
441 return false; | |
442 } | |
443 | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
444 if (_f1 == old_method) { |
0 | 445 _f1 = new_method; |
446 if (RC_TRACE_IN_RANGE(0x00100000, 0x00400000)) { | |
447 if (!(*trace_name_printed)) { | |
448 // RC_TRACE_MESG macro has an embedded ResourceMark | |
449 RC_TRACE_MESG(("adjust: name=%s", | |
6940
18fb7da42534
8000725: NPG: method_holder() and pool_holder() and pool_holder field should be InstanceKlass
coleenp
parents:
6822
diff
changeset
|
450 old_method->method_holder()->external_name())); |
0 | 451 *trace_name_printed = true; |
452 } | |
453 // RC_TRACE macro has an embedded ResourceMark | |
454 RC_TRACE(0x00400000, ("cpc entry update: %s(%s)", | |
455 new_method->name()->as_C_string(), | |
456 new_method->signature()->as_C_string())); | |
457 } | |
458 | |
459 return true; | |
460 } | |
461 | |
462 return false; | |
463 } | |
464 | |
8021
8d9fc28831cc
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
6985
diff
changeset
|
465 // a constant pool cache entry should never contain old or obsolete methods |
8d9fc28831cc
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
6985
diff
changeset
|
466 bool ConstantPoolCacheEntry::check_no_old_or_obsolete_entries() { |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
467 if (is_vfinal()) { |
8021
8d9fc28831cc
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
6985
diff
changeset
|
468 // virtual and final so _f2 contains method ptr instead of vtable index |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
469 Metadata* f2 = (Metadata*)_f2; |
8021
8d9fc28831cc
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
6985
diff
changeset
|
470 // Return false if _f2 refers to an old or an obsolete method. |
8d9fc28831cc
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
6985
diff
changeset
|
471 // _f2 == NULL || !_f2->is_method() are just as unexpected here. |
8d9fc28831cc
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
6985
diff
changeset
|
472 return (f2 != NULL NOT_PRODUCT(&& f2->is_valid()) && f2->is_method() && |
8d9fc28831cc
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
6985
diff
changeset
|
473 !((Method*)f2)->is_old() && !((Method*)f2)->is_obsolete()); |
8d9fc28831cc
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
6985
diff
changeset
|
474 } else if (_f1 == NULL || |
8d9fc28831cc
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
6985
diff
changeset
|
475 (NOT_PRODUCT(_f1->is_valid() &&) !_f1->is_method())) { |
8d9fc28831cc
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
6985
diff
changeset
|
476 // _f1 == NULL || !_f1->is_method() are OK here |
8d9fc28831cc
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
6985
diff
changeset
|
477 return true; |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
478 } |
8021
8d9fc28831cc
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
6985
diff
changeset
|
479 // return false if _f1 refers to an old or an obsolete method |
8d9fc28831cc
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
6985
diff
changeset
|
480 return (NOT_PRODUCT(_f1->is_valid() &&) _f1->is_method() && |
8d9fc28831cc
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
6985
diff
changeset
|
481 !((Method*)_f1)->is_old() && !((Method*)_f1)->is_obsolete()); |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
482 } |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
483 |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
484 bool ConstantPoolCacheEntry::is_interesting_method_entry(Klass* k) { |
0 | 485 if (!is_method_entry()) { |
486 // not a method entry so not interesting by default | |
487 return false; | |
488 } | |
489 | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
490 Method* m = NULL; |
0 | 491 if (is_vfinal()) { |
492 // virtual and final so _f2 contains method ptr instead of vtable index | |
6266
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
493 m = f2_as_vfinal_method(); |
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
494 } else if (is_f1_null()) { |
0 | 495 // NULL _f1 means this is a virtual entry so also not interesting |
496 return false; | |
497 } else { | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
498 if (!(_f1->is_method())) { |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
499 // _f1 can also contain a Klass* for an interface |
0 | 500 return false; |
501 } | |
6266
1d7922586cf6
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
5898
diff
changeset
|
502 m = f1_as_method(); |
0 | 503 } |
504 | |
505 assert(m != NULL && m->is_method(), "sanity check"); | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
506 if (m == NULL || !m->is_method() || (k != NULL && m->method_holder() != k)) { |
0 | 507 // robustness for above sanity checks or method is not in |
508 // the interesting class | |
509 return false; | |
510 } | |
511 | |
512 // the method is in the interesting class so the entry is interesting | |
513 return true; | |
514 } | |
8021
8d9fc28831cc
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
6985
diff
changeset
|
515 #endif // INCLUDE_JVMTI |
0 | 516 |
517 void ConstantPoolCacheEntry::print(outputStream* st, int index) const { | |
518 // print separator | |
5898
f096e1b74d85
7148126: ConstantPoolCacheEntry::print prints to wrong stream
dholmes
parents:
3938
diff
changeset
|
519 if (index == 0) st->print_cr(" -------------"); |
0 | 520 // print entry |
5898
f096e1b74d85
7148126: ConstantPoolCacheEntry::print prints to wrong stream
dholmes
parents:
3938
diff
changeset
|
521 st->print("%3d ("PTR_FORMAT") ", index, (intptr_t)this); |
8021
8d9fc28831cc
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
6985
diff
changeset
|
522 st->print_cr("[%02x|%02x|%5d]", bytecode_2(), bytecode_1(), |
8d9fc28831cc
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
6985
diff
changeset
|
523 constant_pool_index()); |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
524 st->print_cr(" [ "PTR_FORMAT"]", (intptr_t)_f1); |
5898
f096e1b74d85
7148126: ConstantPoolCacheEntry::print prints to wrong stream
dholmes
parents:
3938
diff
changeset
|
525 st->print_cr(" [ "PTR_FORMAT"]", (intptr_t)_f2); |
f096e1b74d85
7148126: ConstantPoolCacheEntry::print prints to wrong stream
dholmes
parents:
3938
diff
changeset
|
526 st->print_cr(" [ "PTR_FORMAT"]", (intptr_t)_flags); |
f096e1b74d85
7148126: ConstantPoolCacheEntry::print prints to wrong stream
dholmes
parents:
3938
diff
changeset
|
527 st->print_cr(" -------------"); |
0 | 528 } |
529 | |
530 void ConstantPoolCacheEntry::verify(outputStream* st) const { | |
531 // not implemented yet | |
532 } | |
533 | |
534 // Implementation of ConstantPoolCache | |
535 | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
536 ConstantPoolCache* ConstantPoolCache::allocate(ClassLoaderData* loader_data, int length, TRAPS) { |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
537 int size = ConstantPoolCache::size(length); |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
538 |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
539 return new (loader_data, size, false, THREAD) ConstantPoolCache(length); |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
540 } |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
541 |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
542 void ConstantPoolCache::initialize(intArray& inverse_index_map, intArray& invokedynamic_references_map) { |
0 | 543 assert(inverse_index_map.length() == length(), "inverse index map must have same length as cache"); |
1059
389049f3f393
6858164: invokedynamic code needs some cleanup (post-6655638)
jrose
parents:
844
diff
changeset
|
544 for (int i = 0; i < length(); i++) { |
389049f3f393
6858164: invokedynamic code needs some cleanup (post-6655638)
jrose
parents:
844
diff
changeset
|
545 ConstantPoolCacheEntry* e = entry_at(i); |
389049f3f393
6858164: invokedynamic code needs some cleanup (post-6655638)
jrose
parents:
844
diff
changeset
|
546 int original_index = inverse_index_map[i]; |
6822
f6b0eb4e44cf
7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents:
6737
diff
changeset
|
547 e->initialize_entry(original_index); |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
548 assert(entry_at(i) == e, "sanity"); |
6822
f6b0eb4e44cf
7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents:
6737
diff
changeset
|
549 } |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
550 for (int ref = 0; ref < invokedynamic_references_map.length(); ref++) { |
6822
f6b0eb4e44cf
7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents:
6737
diff
changeset
|
551 const int cpci = invokedynamic_references_map[ref]; |
f6b0eb4e44cf
7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents:
6737
diff
changeset
|
552 if (cpci >= 0) { |
f6b0eb4e44cf
7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents:
6737
diff
changeset
|
553 #ifdef ASSERT |
f6b0eb4e44cf
7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents:
6737
diff
changeset
|
554 // invokedynamic and invokehandle have more entries; check if they |
f6b0eb4e44cf
7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents:
6737
diff
changeset
|
555 // all point to the same constant pool cache entry. |
f6b0eb4e44cf
7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents:
6737
diff
changeset
|
556 for (int entry = 1; entry < ConstantPoolCacheEntry::_indy_resolved_references_entries; entry++) { |
f6b0eb4e44cf
7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents:
6737
diff
changeset
|
557 const int cpci_next = invokedynamic_references_map[ref + entry]; |
f6b0eb4e44cf
7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents:
6737
diff
changeset
|
558 assert(cpci == cpci_next, err_msg_res("%d == %d", cpci, cpci_next)); |
f6b0eb4e44cf
7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents:
6737
diff
changeset
|
559 } |
f6b0eb4e44cf
7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents:
6737
diff
changeset
|
560 #endif |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
561 entry_at(cpci)->initialize_resolved_reference_index(ref); |
6822
f6b0eb4e44cf
7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents:
6737
diff
changeset
|
562 ref += ConstantPoolCacheEntry::_indy_resolved_references_entries - 1; // skip extra entries |
f6b0eb4e44cf
7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents:
6737
diff
changeset
|
563 } |
1059
389049f3f393
6858164: invokedynamic code needs some cleanup (post-6655638)
jrose
parents:
844
diff
changeset
|
564 } |
0 | 565 } |
566 | |
8021
8d9fc28831cc
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
6985
diff
changeset
|
567 #if INCLUDE_JVMTI |
0 | 568 // RedefineClasses() API support: |
8021
8d9fc28831cc
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
6985
diff
changeset
|
569 // If any entry of this ConstantPoolCache points to any of |
0 | 570 // old_methods, replace it with the corresponding new_method. |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
571 void ConstantPoolCache::adjust_method_entries(Method** old_methods, Method** new_methods, |
0 | 572 int methods_length, bool * trace_name_printed) { |
573 | |
574 if (methods_length == 0) { | |
575 // nothing to do if there are no methods | |
576 return; | |
577 } | |
578 | |
579 // get shorthand for the interesting class | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
580 Klass* old_holder = old_methods[0]->method_holder(); |
0 | 581 |
582 for (int i = 0; i < length(); i++) { | |
583 if (!entry_at(i)->is_interesting_method_entry(old_holder)) { | |
584 // skip uninteresting methods | |
585 continue; | |
586 } | |
587 | |
8021
8d9fc28831cc
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
6985
diff
changeset
|
588 // The ConstantPoolCache contains entries for several different |
0 | 589 // things, but we only care about methods. In fact, we only care |
590 // about methods in the same class as the one that contains the | |
591 // old_methods. At this point, we have an interesting entry. | |
592 | |
593 for (int j = 0; j < methods_length; j++) { | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
594 Method* old_method = old_methods[j]; |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
595 Method* new_method = new_methods[j]; |
0 | 596 |
597 if (entry_at(i)->adjust_method_entry(old_method, new_method, | |
598 trace_name_printed)) { | |
599 // current old_method matched this entry and we updated it so | |
600 // break out and get to the next interesting entry if there one | |
601 break; | |
602 } | |
603 } | |
604 } | |
605 } | |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
606 |
8021
8d9fc28831cc
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
6985
diff
changeset
|
607 // the constant pool cache should never contain old or obsolete methods |
8d9fc28831cc
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
6985
diff
changeset
|
608 bool ConstantPoolCache::check_no_old_or_obsolete_entries() { |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
609 for (int i = 1; i < length(); i++) { |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
610 if (entry_at(i)->is_interesting_method_entry(NULL) && |
8021
8d9fc28831cc
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
6985
diff
changeset
|
611 !entry_at(i)->check_no_old_or_obsolete_entries()) { |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
612 return false; |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
613 } |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
614 } |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
615 return true; |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
616 } |
8021
8d9fc28831cc
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
6985
diff
changeset
|
617 |
8d9fc28831cc
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
6985
diff
changeset
|
618 void ConstantPoolCache::dump_cache() { |
8d9fc28831cc
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
6985
diff
changeset
|
619 for (int i = 1; i < length(); i++) { |
8d9fc28831cc
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
6985
diff
changeset
|
620 if (entry_at(i)->is_interesting_method_entry(NULL)) { |
8d9fc28831cc
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
6985
diff
changeset
|
621 entry_at(i)->print(tty, i); |
8d9fc28831cc
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
6985
diff
changeset
|
622 } |
8d9fc28831cc
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
6985
diff
changeset
|
623 } |
8d9fc28831cc
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
6985
diff
changeset
|
624 } |
8d9fc28831cc
7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents:
6985
diff
changeset
|
625 #endif // INCLUDE_JVMTI |
6725
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
626 |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
627 |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
628 // Printing |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
629 |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
630 void ConstantPoolCache::print_on(outputStream* st) const { |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
631 assert(is_constantPoolCache(), "obj must be constant pool cache"); |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
632 st->print_cr(internal_name()); |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
633 // print constant pool cache entries |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
634 for (int i = 0; i < length(); i++) entry_at(i)->print(st, i); |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
635 } |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
636 |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
637 void ConstantPoolCache::print_value_on(outputStream* st) const { |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
638 assert(is_constantPoolCache(), "obj must be constant pool cache"); |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
639 st->print("cache [%d]", length()); |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
640 print_address_on(st); |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
641 st->print(" for "); |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
642 constant_pool()->print_value_on(st); |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
643 } |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
644 |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
645 |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
646 // Verification |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
647 |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
648 void ConstantPoolCache::verify_on(outputStream* st) { |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
649 guarantee(is_constantPoolCache(), "obj must be constant pool cache"); |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
650 // print constant pool cache entries |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
651 for (int i = 0; i < length(); i++) entry_at(i)->verify(st); |
da91efe96a93
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
6266
diff
changeset
|
652 } |