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