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