annotate src/share/vm/oops/cpCache.cpp @ 24170:0b85ccd62409 jdk8u131-b01

8168699: Validate special case invocations Reviewed-by: kevinw, vlivanov
author coleenp
date Tue, 13 Dec 2016 14:37:04 -0500
parents 157895117ad5
children 719853999215
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
rev   line source
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1 /*
24170
0b85ccd62409 8168699: Validate special case invocations
coleenp
parents: 22991
diff changeset
2 * Copyright (c) 1998, 2016, Oracle and/or its affiliates. All rights reserved.
0
a61af66fc99e Initial load
duke
parents:
diff changeset
3 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
a61af66fc99e Initial load
duke
parents:
diff changeset
4 *
a61af66fc99e Initial load
duke
parents:
diff changeset
5 * This code is free software; you can redistribute it and/or modify it
a61af66fc99e Initial load
duke
parents:
diff changeset
6 * under the terms of the GNU General Public License version 2 only, as
a61af66fc99e Initial load
duke
parents:
diff changeset
7 * published by the Free Software Foundation.
a61af66fc99e Initial load
duke
parents:
diff changeset
8 *
a61af66fc99e Initial load
duke
parents:
diff changeset
9 * This code is distributed in the hope that it will be useful, but WITHOUT
a61af66fc99e Initial load
duke
parents:
diff changeset
10 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
a61af66fc99e Initial load
duke
parents:
diff changeset
11 * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License
a61af66fc99e Initial load
duke
parents:
diff changeset
12 * version 2 for more details (a copy is included in the LICENSE file that
a61af66fc99e Initial load
duke
parents:
diff changeset
13 * accompanied this code).
a61af66fc99e Initial load
duke
parents:
diff changeset
14 *
a61af66fc99e Initial load
duke
parents:
diff changeset
15 * You should have received a copy of the GNU General Public License version
a61af66fc99e Initial load
duke
parents:
diff changeset
16 * 2 along with this work; if not, write to the Free Software Foundation,
a61af66fc99e Initial load
duke
parents:
diff changeset
17 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
a61af66fc99e Initial load
duke
parents:
diff changeset
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
a61af66fc99e Initial load
duke
parents:
diff changeset
22 *
a61af66fc99e Initial load
duke
parents:
diff changeset
23 */
a61af66fc99e Initial load
duke
parents:
diff changeset
24
1972
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1903
diff changeset
25 #include "precompiled.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1903
diff changeset
26 #include "gc_implementation/shared/markSweep.inline.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1903
diff changeset
27 #include "interpreter/interpreter.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1903
diff changeset
28 #include "interpreter/rewriter.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1903
diff changeset
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
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1903
diff changeset
31 #include "oops/objArrayOop.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1903
diff changeset
32 #include "oops/oop.inline.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1903
diff changeset
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
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1903
diff changeset
35 #include "runtime/handles.inline.hpp"
20197
ce8f6bb717c9 8042195: Introduce umbrella header orderAccess.inline.hpp.
goetz
parents: 17977
diff changeset
36 #include "runtime/orderAccess.inline.hpp"
8001
db9981fd3124 8005915: Unify SERIALGC and INCLUDE_ALTERNATE_GCS
jprovino
parents: 6985
diff changeset
37 #include "utilities/macros.hpp"
db9981fd3124 8005915: Unify SERIALGC and INCLUDE_ALTERNATE_GCS
jprovino
parents: 6985
diff changeset
38 #if INCLUDE_ALL_GCS
6725
da91efe96a93 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 6266
diff changeset
39 # include "gc_implementation/parallelScavenge/psPromotionManager.hpp"
8001
db9981fd3124 8005915: Unify SERIALGC and INCLUDE_ALTERNATE_GCS
jprovino
parents: 6985
diff changeset
40 #endif // INCLUDE_ALL_GCS
0
a61af66fc99e Initial load
duke
parents:
diff changeset
41
17937
78bbf4d43a14 8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents: 14443
diff changeset
42 PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC
0
a61af66fc99e Initial load
duke
parents:
diff changeset
43
17937
78bbf4d43a14 8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents: 14443
diff changeset
44 // Implementation of ConstantPoolCacheEntry
0
a61af66fc99e Initial load
duke
parents:
diff changeset
45
1059
389049f3f393 6858164: invokedynamic code needs some cleanup (post-6655638)
jrose
parents: 844
diff changeset
46 void ConstantPoolCacheEntry::initialize_entry(int index) {
726
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 196
diff changeset
47 assert(0 < index && index < 0x10000, "sanity check");
0
a61af66fc99e Initial load
duke
parents:
diff changeset
48 _indices = index;
8712
3efdfd6ddbf2 8003553: NPG: metaspace objects should be zeroed in constructors
coleenp
parents: 8025
diff changeset
49 _f1 = NULL;
3efdfd6ddbf2 8003553: NPG: metaspace objects should be zeroed in constructors
coleenp
parents: 8025
diff changeset
50 _f2 = _flags = 0;
726
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 196
diff changeset
51 assert(constant_pool_index() == index, "");
0
a61af66fc99e Initial load
duke
parents:
diff changeset
52 }
a61af66fc99e Initial load
duke
parents:
diff changeset
53
6266
1d7922586cf6 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 5898
diff changeset
54 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
55 int option_bits,
1d7922586cf6 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 5898
diff changeset
56 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
57 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
58 int f = ((int)state << tos_state_shift) | option_bits | field_index_or_method_params;
0
a61af66fc99e Initial load
duke
parents:
diff changeset
59 // 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
60 // The low bits are a field offset, or else the method parameter size.
0
a61af66fc99e Initial load
duke
parents:
diff changeset
61 #ifdef ASSERT
6266
1d7922586cf6 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 5898
diff changeset
62 TosState old_state = flag_state();
1d7922586cf6 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 5898
diff changeset
63 assert(old_state == (TosState)0 || old_state == state,
0
a61af66fc99e Initial load
duke
parents:
diff changeset
64 "inconsistent cpCache flags state");
a61af66fc99e Initial load
duke
parents:
diff changeset
65 #endif
a61af66fc99e Initial load
duke
parents:
diff changeset
66 return (_flags | f) ;
a61af66fc99e Initial load
duke
parents:
diff changeset
67 }
a61af66fc99e Initial load
duke
parents:
diff changeset
68
a61af66fc99e Initial load
duke
parents:
diff changeset
69 void ConstantPoolCacheEntry::set_bytecode_1(Bytecodes::Code code) {
a61af66fc99e Initial load
duke
parents:
diff changeset
70 #ifdef ASSERT
a61af66fc99e Initial load
duke
parents:
diff changeset
71 // Read once.
a61af66fc99e Initial load
duke
parents:
diff changeset
72 volatile Bytecodes::Code c = bytecode_1();
a61af66fc99e Initial load
duke
parents:
diff changeset
73 assert(c == 0 || c == code || code == 0, "update must be consistent");
a61af66fc99e Initial load
duke
parents:
diff changeset
74 #endif
a61af66fc99e Initial load
duke
parents:
diff changeset
75 // 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
76 OrderAccess::release_store_ptr(&_indices, _indices | ((u_char)code << bytecode_1_shift));
0
a61af66fc99e Initial load
duke
parents:
diff changeset
77 }
a61af66fc99e Initial load
duke
parents:
diff changeset
78
a61af66fc99e Initial load
duke
parents:
diff changeset
79 void ConstantPoolCacheEntry::set_bytecode_2(Bytecodes::Code code) {
a61af66fc99e Initial load
duke
parents:
diff changeset
80 #ifdef ASSERT
a61af66fc99e Initial load
duke
parents:
diff changeset
81 // Read once.
a61af66fc99e Initial load
duke
parents:
diff changeset
82 volatile Bytecodes::Code c = bytecode_2();
a61af66fc99e Initial load
duke
parents:
diff changeset
83 assert(c == 0 || c == code || code == 0, "update must be consistent");
a61af66fc99e Initial load
duke
parents:
diff changeset
84 #endif
a61af66fc99e Initial load
duke
parents:
diff changeset
85 // 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
86 OrderAccess::release_store_ptr(&_indices, _indices | ((u_char)code << bytecode_2_shift));
0
a61af66fc99e Initial load
duke
parents:
diff changeset
87 }
a61af66fc99e Initial load
duke
parents:
diff changeset
88
6266
1d7922586cf6 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 5898
diff changeset
89 // Sets f1, ordering with previous writes.
6725
da91efe96a93 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 6266
diff changeset
90 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
91 assert(f1 != NULL, "");
6725
da91efe96a93 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 6266
diff changeset
92 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
93 }
1d7922586cf6 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 5898
diff changeset
94
1d7922586cf6 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 5898
diff changeset
95 // 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
96 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
97 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
98 return (result == 0);
2246
fbf3184da15d 7012339: JSR 292 crash in G1SATBCardTableModRefBS::write_ref_field_pre_work()
twisti
parents: 1972
diff changeset
99 }
1903
87d6a4d1ecbc 6990192: VM crashes in ciTypeFlow::get_block_for()
twisti
parents: 1662
diff changeset
100
0
a61af66fc99e Initial load
duke
parents:
diff changeset
101 // Note that concurrent update of both bytecodes can leave one of them
a61af66fc99e Initial load
duke
parents:
diff changeset
102 // reset to zero. This is harmless; the interpreter will simply re-resolve
a61af66fc99e Initial load
duke
parents:
diff changeset
103 // the damaged entry. More seriously, the memory synchronization is needed
a61af66fc99e Initial load
duke
parents:
diff changeset
104 // to flush other fields (f1, f2) completely to memory before the bytecodes
a61af66fc99e Initial load
duke
parents:
diff changeset
105 // are updated, lest other processors see a non-zero bytecode but zero f1/f2.
a61af66fc99e Initial load
duke
parents:
diff changeset
106 void ConstantPoolCacheEntry::set_field(Bytecodes::Code get_code,
a61af66fc99e Initial load
duke
parents:
diff changeset
107 Bytecodes::Code put_code,
a61af66fc99e Initial load
duke
parents:
diff changeset
108 KlassHandle field_holder,
3938
e6b1331a51d2 7086585: make Java field injection more flexible
never
parents: 3785
diff changeset
109 int field_index,
0
a61af66fc99e Initial load
duke
parents:
diff changeset
110 int field_offset,
a61af66fc99e Initial load
duke
parents:
diff changeset
111 TosState field_type,
a61af66fc99e Initial load
duke
parents:
diff changeset
112 bool is_final,
6725
da91efe96a93 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 6266
diff changeset
113 bool is_volatile,
da91efe96a93 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 6266
diff changeset
114 Klass* root_klass) {
da91efe96a93 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 6266
diff changeset
115 set_f1(field_holder());
0
a61af66fc99e Initial load
duke
parents:
diff changeset
116 set_f2(field_offset);
6266
1d7922586cf6 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 5898
diff changeset
117 assert((field_index & field_index_mask) == field_index,
0
a61af66fc99e Initial load
duke
parents:
diff changeset
118 "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
119 set_field_flags(field_type,
1d7922586cf6 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 5898
diff changeset
120 ((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
121 ((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
122 field_index);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
123 set_bytecode_1(get_code);
a61af66fc99e Initial load
duke
parents:
diff changeset
124 set_bytecode_2(put_code);
a61af66fc99e Initial load
duke
parents:
diff changeset
125 NOT_PRODUCT(verify(tty));
a61af66fc99e Initial load
duke
parents:
diff changeset
126 }
a61af66fc99e Initial load
duke
parents:
diff changeset
127
6266
1d7922586cf6 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 5898
diff changeset
128 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
129 // 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
130 // 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
131 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
132 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
133 // 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
134 // 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
135 // 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
136 // 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
137 // 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
138 if (_flags == 0) {
1d7922586cf6 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 5898
diff changeset
139 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
140 }
1d7922586cf6 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 5898
diff changeset
141 guarantee(parameter_size() == value,
1d7922586cf6 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 5898
diff changeset
142 err_msg("size must not change: parameter_size=%d, value=%d", parameter_size(), value));
0
a61af66fc99e Initial load
duke
parents:
diff changeset
143 }
a61af66fc99e Initial load
duke
parents:
diff changeset
144
12264
b2e698d2276c 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 10376
diff changeset
145 void ConstantPoolCacheEntry::set_direct_or_vtable_call(Bytecodes::Code invoke_code,
b2e698d2276c 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 10376
diff changeset
146 methodHandle method,
24170
0b85ccd62409 8168699: Validate special case invocations
coleenp
parents: 22991
diff changeset
147 int vtable_index,
0b85ccd62409 8168699: Validate special case invocations
coleenp
parents: 22991
diff changeset
148 bool sender_is_interface) {
12264
b2e698d2276c 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 10376
diff changeset
149 bool is_vtable_call = (vtable_index >= 0); // FIXME: split this method on this boolean
0
a61af66fc99e Initial load
duke
parents:
diff changeset
150 assert(method->interpreter_entry() != NULL, "should have been set at this point");
a61af66fc99e Initial load
duke
parents:
diff changeset
151 assert(!method->is_obsolete(), "attempt to write obsolete method to cpCache");
a61af66fc99e Initial load
duke
parents:
diff changeset
152
a61af66fc99e Initial load
duke
parents:
diff changeset
153 int byte_no = -1;
6266
1d7922586cf6 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 5898
diff changeset
154 bool change_to_virtual = false;
1d7922586cf6 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 5898
diff changeset
155
0
a61af66fc99e Initial load
duke
parents:
diff changeset
156 switch (invoke_code) {
6266
1d7922586cf6 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 5898
diff changeset
157 case Bytecodes::_invokeinterface:
1d7922586cf6 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 5898
diff changeset
158 // 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
159 // 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
160 // 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
161 // 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
162 change_to_virtual = true;
1d7922586cf6 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 5898
diff changeset
163
1d7922586cf6 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 5898
diff changeset
164 // ...and fall through as if we were handling invokevirtual:
0
a61af66fc99e Initial load
duke
parents:
diff changeset
165 case Bytecodes::_invokevirtual:
6266
1d7922586cf6 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 5898
diff changeset
166 {
12264
b2e698d2276c 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 10376
diff changeset
167 if (!is_vtable_call) {
b2e698d2276c 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 10376
diff changeset
168 assert(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
169 // 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
170 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
171 ( 1 << is_vfinal_shift) |
1d7922586cf6 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 5898
diff changeset
172 ((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
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());
1d7922586cf6 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 5898
diff changeset
175 set_f2_as_vfinal_method(method());
0
a61af66fc99e Initial load
duke
parents:
diff changeset
176 } else {
12264
b2e698d2276c 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 10376
diff changeset
177 assert(!method->can_be_statically_bound(), "");
0
a61af66fc99e Initial load
duke
parents:
diff changeset
178 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
179 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
180 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
181 ((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
182 method()->size_of_parameters());
0
a61af66fc99e Initial load
duke
parents:
diff changeset
183 set_f2(vtable_index);
a61af66fc99e Initial load
duke
parents:
diff changeset
184 }
a61af66fc99e Initial load
duke
parents:
diff changeset
185 byte_no = 2;
a61af66fc99e Initial load
duke
parents:
diff changeset
186 break;
1660
083fde3b838e 6964498: JSR 292 invokedynamic sites need local bootstrap methods
jrose
parents: 1552
diff changeset
187 }
083fde3b838e 6964498: JSR 292 invokedynamic sites need local bootstrap methods
jrose
parents: 1552
diff changeset
188
0
a61af66fc99e Initial load
duke
parents:
diff changeset
189 case Bytecodes::_invokespecial:
a61af66fc99e Initial load
duke
parents:
diff changeset
190 case Bytecodes::_invokestatic:
12264
b2e698d2276c 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 10376
diff changeset
191 assert(!is_vtable_call, "");
6266
1d7922586cf6 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 5898
diff changeset
192 // 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
193 // 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
194 // 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
195 // 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
196 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
197 ((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
198 ((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
199 method()->size_of_parameters());
0
a61af66fc99e Initial load
duke
parents:
diff changeset
200 set_f1(method());
a61af66fc99e Initial load
duke
parents:
diff changeset
201 byte_no = 1;
a61af66fc99e Initial load
duke
parents:
diff changeset
202 break;
a61af66fc99e Initial load
duke
parents:
diff changeset
203 default:
a61af66fc99e Initial load
duke
parents:
diff changeset
204 ShouldNotReachHere();
a61af66fc99e Initial load
duke
parents:
diff changeset
205 break;
a61af66fc99e Initial load
duke
parents:
diff changeset
206 }
a61af66fc99e Initial load
duke
parents:
diff changeset
207
a61af66fc99e Initial load
duke
parents:
diff changeset
208 // Note: byte_no also appears in TemplateTable::resolve.
a61af66fc99e Initial load
duke
parents:
diff changeset
209 if (byte_no == 1) {
6266
1d7922586cf6 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 5898
diff changeset
210 assert(invoke_code != Bytecodes::_invokevirtual &&
1d7922586cf6 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 5898
diff changeset
211 invoke_code != Bytecodes::_invokeinterface, "");
24170
0b85ccd62409 8168699: Validate special case invocations
coleenp
parents: 22991
diff changeset
212 // Don't mark invokespecial to method as resolved if sender is an interface. The receiver
0b85ccd62409 8168699: Validate special case invocations
coleenp
parents: 22991
diff changeset
213 // has to be checked that it is a subclass of the current class every time this bytecode
0b85ccd62409 8168699: Validate special case invocations
coleenp
parents: 22991
diff changeset
214 // is executed.
0b85ccd62409 8168699: Validate special case invocations
coleenp
parents: 22991
diff changeset
215 if (invoke_code != Bytecodes::_invokespecial || !sender_is_interface ||
0b85ccd62409 8168699: Validate special case invocations
coleenp
parents: 22991
diff changeset
216 method->name() == vmSymbols::object_initializer_name()) {
0
a61af66fc99e Initial load
duke
parents:
diff changeset
217 set_bytecode_1(invoke_code);
24170
0b85ccd62409 8168699: Validate special case invocations
coleenp
parents: 22991
diff changeset
218 }
0
a61af66fc99e Initial load
duke
parents:
diff changeset
219 } else if (byte_no == 2) {
a61af66fc99e Initial load
duke
parents:
diff changeset
220 if (change_to_virtual) {
6266
1d7922586cf6 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 5898
diff changeset
221 assert(invoke_code == Bytecodes::_invokeinterface, "");
0
a61af66fc99e Initial load
duke
parents:
diff changeset
222 // NOTE: THIS IS A HACK - BE VERY CAREFUL!!!
a61af66fc99e Initial load
duke
parents:
diff changeset
223 //
a61af66fc99e Initial load
duke
parents:
diff changeset
224 // Workaround for the case where we encounter an invokeinterface, but we
a61af66fc99e Initial load
duke
parents:
diff changeset
225 // should really have an _invokevirtual since the resolved method is a
a61af66fc99e Initial load
duke
parents:
diff changeset
226 // virtual method in java.lang.Object. This is a corner case in the spec
a61af66fc99e Initial load
duke
parents:
diff changeset
227 // but is presumably legal. javac does not generate this code.
a61af66fc99e Initial load
duke
parents:
diff changeset
228 //
a61af66fc99e Initial load
duke
parents:
diff changeset
229 // We set bytecode_1() to _invokeinterface, because that is the
a61af66fc99e Initial load
duke
parents:
diff changeset
230 // bytecode # used by the interpreter to see if it is resolved.
a61af66fc99e Initial load
duke
parents:
diff changeset
231 // We set bytecode_2() to _invokevirtual.
a61af66fc99e Initial load
duke
parents:
diff changeset
232 // See also interpreterRuntime.cpp. (8/25/2000)
a61af66fc99e Initial load
duke
parents:
diff changeset
233 // Only set resolved for the invokeinterface case if method is public.
a61af66fc99e Initial load
duke
parents:
diff changeset
234 // Otherwise, the method needs to be reresolved with caller for each
a61af66fc99e Initial load
duke
parents:
diff changeset
235 // interface call.
a61af66fc99e Initial load
duke
parents:
diff changeset
236 if (method->is_public()) set_bytecode_1(invoke_code);
a61af66fc99e Initial load
duke
parents:
diff changeset
237 } else {
6266
1d7922586cf6 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 5898
diff changeset
238 assert(invoke_code == Bytecodes::_invokevirtual, "");
0
a61af66fc99e Initial load
duke
parents:
diff changeset
239 }
6266
1d7922586cf6 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 5898
diff changeset
240 // 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
241 set_bytecode_2(Bytecodes::_invokevirtual);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
242 } else {
a61af66fc99e Initial load
duke
parents:
diff changeset
243 ShouldNotReachHere();
a61af66fc99e Initial load
duke
parents:
diff changeset
244 }
a61af66fc99e Initial load
duke
parents:
diff changeset
245 NOT_PRODUCT(verify(tty));
a61af66fc99e Initial load
duke
parents:
diff changeset
246 }
a61af66fc99e Initial load
duke
parents:
diff changeset
247
24170
0b85ccd62409 8168699: Validate special case invocations
coleenp
parents: 22991
diff changeset
248 void ConstantPoolCacheEntry::set_direct_call(Bytecodes::Code invoke_code, methodHandle method,
0b85ccd62409 8168699: Validate special case invocations
coleenp
parents: 22991
diff changeset
249 bool sender_is_interface) {
12264
b2e698d2276c 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 10376
diff changeset
250 int index = Method::nonvirtual_vtable_index;
b2e698d2276c 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 10376
diff changeset
251 // index < 0; FIXME: inline and customize set_direct_or_vtable_call
24170
0b85ccd62409 8168699: Validate special case invocations
coleenp
parents: 22991
diff changeset
252 set_direct_or_vtable_call(invoke_code, method, index, sender_is_interface);
12264
b2e698d2276c 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 10376
diff changeset
253 }
0
a61af66fc99e Initial load
duke
parents:
diff changeset
254
12264
b2e698d2276c 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 10376
diff changeset
255 void ConstantPoolCacheEntry::set_vtable_call(Bytecodes::Code invoke_code, methodHandle method, int index) {
b2e698d2276c 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 10376
diff changeset
256 // either the method is a miranda or its holder should accept the given index
b2e698d2276c 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 10376
diff changeset
257 assert(method->method_holder()->is_interface() || method->method_holder()->verify_vtable_index(index), "");
b2e698d2276c 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 10376
diff changeset
258 // index >= 0; FIXME: inline and customize set_direct_or_vtable_call
24170
0b85ccd62409 8168699: Validate special case invocations
coleenp
parents: 22991
diff changeset
259 set_direct_or_vtable_call(invoke_code, method, index, false);
12264
b2e698d2276c 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 10376
diff changeset
260 }
b2e698d2276c 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 10376
diff changeset
261
b2e698d2276c 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 10376
diff changeset
262 void ConstantPoolCacheEntry::set_itable_call(Bytecodes::Code invoke_code, methodHandle method, int index) {
b2e698d2276c 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 10376
diff changeset
263 assert(method->method_holder()->verify_itable_index(index), "");
b2e698d2276c 8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents: 10376
diff changeset
264 assert(invoke_code == Bytecodes::_invokeinterface, "");
6940
18fb7da42534 8000725: NPG: method_holder() and pool_holder() and pool_holder field should be InstanceKlass
coleenp
parents: 6822
diff changeset
265 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
266 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
267 assert(!method->is_final_method(), "interfaces do not have final methods; cannot link to one here");
0
a61af66fc99e Initial load
duke
parents:
diff changeset
268 set_f1(interf);
a61af66fc99e Initial load
duke
parents:
diff changeset
269 set_f2(index);
6266
1d7922586cf6 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 5898
diff changeset
270 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
271 0, // no option bits
1d7922586cf6 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 5898
diff changeset
272 method()->size_of_parameters());
0
a61af66fc99e Initial load
duke
parents:
diff changeset
273 set_bytecode_1(Bytecodes::_invokeinterface);
a61af66fc99e Initial load
duke
parents:
diff changeset
274 }
a61af66fc99e Initial load
duke
parents:
diff changeset
275
a61af66fc99e Initial load
duke
parents:
diff changeset
276
6942
64672b22ef05 8001658: No need to pass resolved_references as argument to ConstantPoolCacheEntry::set_method_handle_common
twisti
parents: 6822
diff changeset
277 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
278 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
279 }
083fde3b838e 6964498: JSR 292 invokedynamic sites need local bootstrap methods
jrose
parents: 1552
diff changeset
280
6942
64672b22ef05 8001658: No need to pass resolved_references as argument to ConstantPoolCacheEntry::set_method_handle_common
twisti
parents: 6822
diff changeset
281 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
282 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
283 }
083fde3b838e 6964498: JSR 292 invokedynamic sites need local bootstrap methods
jrose
parents: 1552
diff changeset
284
6737
4bfe8b33cf66 7196242: vm/mlvm/indy/stress/java/loopsAndThreads crashed
twisti
parents: 6725
diff changeset
285 void ConstantPoolCacheEntry::set_method_handle_common(constantPoolHandle cpool,
4bfe8b33cf66 7196242: vm/mlvm/indy/stress/java/loopsAndThreads crashed
twisti
parents: 6725
diff changeset
286 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
287 const CallInfo &call_info) {
6266
1d7922586cf6 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 5898
diff changeset
288 // 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
289 // 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
290 // 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
291 // 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
292 // Competing writers must acquire exclusive access via a lock.
4bfe8b33cf66 7196242: vm/mlvm/indy/stress/java/loopsAndThreads crashed
twisti
parents: 6725
diff changeset
293 // 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
294 // 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
295 // cache entry.
4bfe8b33cf66 7196242: vm/mlvm/indy/stress/java/loopsAndThreads crashed
twisti
parents: 6725
diff changeset
296
12946
b8860472c377 8014910: deadlock between JVM/TI ClassPrepare event handler and CompilerThread
iklam
parents: 12316
diff changeset
297 MonitorLockerEx ml(cpool->lock());
6737
4bfe8b33cf66 7196242: vm/mlvm/indy/stress/java/loopsAndThreads crashed
twisti
parents: 6725
diff changeset
298 if (!is_f1_null()) {
4bfe8b33cf66 7196242: vm/mlvm/indy/stress/java/loopsAndThreads crashed
twisti
parents: 6725
diff changeset
299 return;
4bfe8b33cf66 7196242: vm/mlvm/indy/stress/java/loopsAndThreads crashed
twisti
parents: 6725
diff changeset
300 }
6266
1d7922586cf6 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 5898
diff changeset
301
6942
64672b22ef05 8001658: No need to pass resolved_references as argument to ConstantPoolCacheEntry::set_method_handle_common
twisti
parents: 6822
diff changeset
302 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
303 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
304 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
305 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
306 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
307
6725
da91efe96a93 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 6266
diff changeset
308 // Write the flags.
6737
4bfe8b33cf66 7196242: vm/mlvm/indy/stress/java/loopsAndThreads crashed
twisti
parents: 6725
diff changeset
309 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
310 ((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
311 ((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
312 ( 1 << is_final_shift ),
6266
1d7922586cf6 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 5898
diff changeset
313 adapter->size_of_parameters());
1d7922586cf6 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 5898
diff changeset
314
1d7922586cf6 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 5898
diff changeset
315 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
316 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
317 invoke_code,
12316
190899198332 7195622: CheckUnhandledOops has limited usefulness now
hseigel
parents: 12264
diff changeset
318 (void *)appendix(), (has_appendix ? "" : " (unused)"),
190899198332 7195622: CheckUnhandledOops has limited usefulness now
hseigel
parents: 12264
diff changeset
319 (void *)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
320 (intptr_t)adapter());
1d7922586cf6 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 5898
diff changeset
321 adapter->print();
1d7922586cf6 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 5898
diff changeset
322 if (has_appendix) appendix()->print();
1d7922586cf6 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 5898
diff changeset
323 }
1d7922586cf6 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 5898
diff changeset
324
1d7922586cf6 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 5898
diff changeset
325 // 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
326 // 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
327 // 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
328 // 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
329 // 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
330 // 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
331 // (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
332 // 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
333 // 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
334 // 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
335 // 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
336 //
1d7922586cf6 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 5898
diff changeset
337 // 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
338 // 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
339 // not '(Ljava/lang/String;)Ljava/util/List;'.
6725
da91efe96a93 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 6266
diff changeset
340 // 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
341 // 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
342 //
6725
da91efe96a93 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 6266
diff changeset
343
6942
64672b22ef05 8001658: No need to pass resolved_references as argument to ConstantPoolCacheEntry::set_method_handle_common
twisti
parents: 6822
diff changeset
344 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
345 // Store appendix, if any.
6725
da91efe96a93 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 6266
diff changeset
346 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
347 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
348 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
349 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
350 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
351 }
f6b0eb4e44cf 7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents: 6737
diff changeset
352
f6b0eb4e44cf 7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents: 6737
diff changeset
353 // 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
354 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
355 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
356 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
357 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
358 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
359 }
da91efe96a93 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 6266
diff changeset
360
da91efe96a93 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 6266
diff changeset
361 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
362
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
363 // 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
364 // 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
365 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
366 NOT_PRODUCT(verify(tty));
1d7922586cf6 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 5898
diff changeset
367 if (TraceInvokeDynamic) {
1d7922586cf6 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 5898
diff changeset
368 this->print(tty, 0);
1d7922586cf6 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 5898
diff changeset
369 }
726
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 196
diff changeset
370 }
be93aad57795 6655646: dynamic languages need dynamically linked call sites
jrose
parents: 196
diff changeset
371
6725
da91efe96a93 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 6266
diff changeset
372 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
373 // 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
374 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
375 if (invoke_code != (Bytecodes::Code)0) {
14443
3205e78d8193 8029396: PPC64 (part 212): Several memory ordering fixes in C-code.
goetz
parents: 13056
diff changeset
376 Metadata* f1 = f1_ord();
3785
ddd894528dbc 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 3279
diff changeset
377 if (f1 != NULL) {
ddd894528dbc 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 3279
diff changeset
378 switch (invoke_code) {
ddd894528dbc 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 3279
diff changeset
379 case Bytecodes::_invokeinterface:
ddd894528dbc 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 3279
diff changeset
380 assert(f1->is_klass(), "");
6725
da91efe96a93 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 6266
diff changeset
381 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
382 case Bytecodes::_invokestatic:
ddd894528dbc 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 3279
diff changeset
383 case Bytecodes::_invokespecial:
6266
1d7922586cf6 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 5898
diff changeset
384 assert(!has_appendix(), "");
6725
da91efe96a93 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 6266
diff changeset
385 case Bytecodes::_invokehandle:
da91efe96a93 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 6266
diff changeset
386 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
387 assert(f1->is_method(), "");
6725
da91efe96a93 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 6266
diff changeset
388 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
389 }
ddd894528dbc 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 3279
diff changeset
390 }
ddd894528dbc 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 3279
diff changeset
391 }
6266
1d7922586cf6 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 5898
diff changeset
392 invoke_code = bytecode_2();
1d7922586cf6 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 5898
diff changeset
393 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
394 switch (invoke_code) {
ddd894528dbc 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 3279
diff changeset
395 case Bytecodes::_invokevirtual:
ddd894528dbc 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 3279
diff changeset
396 if (is_vfinal()) {
ddd894528dbc 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 3279
diff changeset
397 // invokevirtual
6725
da91efe96a93 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 6266
diff changeset
398 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
399 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
400 return m;
ddd894528dbc 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 3279
diff changeset
401 } else {
ddd894528dbc 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 3279
diff changeset
402 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
403 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
404 Klass* klass = cpool->resolved_klass_at(holder_index);
6983
070d523b96a7 8001471: Klass::cast() does nothing
hseigel
parents: 6940
diff changeset
405 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
406 klass = SystemDictionary::Object_klass();
6725
da91efe96a93 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 6266
diff changeset
407 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
408 }
ddd894528dbc 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 3279
diff changeset
409 }
6266
1d7922586cf6 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 5898
diff changeset
410 break;
3785
ddd894528dbc 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 3279
diff changeset
411 }
ddd894528dbc 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 3279
diff changeset
412 }
ddd894528dbc 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 3279
diff changeset
413 return NULL;
ddd894528dbc 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 3279
diff changeset
414 }
ddd894528dbc 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 3279
diff changeset
415
ddd894528dbc 7056328: JSR 292 invocation sometimes fails in adapters for types not on boot class path
jrose
parents: 3279
diff changeset
416
6725
da91efe96a93 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 6266
diff changeset
417 oop ConstantPoolCacheEntry::appendix_if_resolved(constantPoolHandle cpool) {
17977
0b9500028980 8029381: assert(is_method_type()) failed: bad cast
drchase
parents: 17937
diff changeset
418 if (!has_appendix())
6725
da91efe96a93 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 6266
diff changeset
419 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
420 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
421 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
422 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
423 }
f6b0eb4e44cf 7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents: 6737
diff changeset
424
f6b0eb4e44cf 7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents: 6737
diff changeset
425
f6b0eb4e44cf 7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents: 6737
diff changeset
426 oop ConstantPoolCacheEntry::method_type_if_resolved(constantPoolHandle cpool) {
17977
0b9500028980 8029381: assert(is_method_type()) failed: bad cast
drchase
parents: 17937
diff changeset
427 if (!has_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
428 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
429 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
430 objArrayOop resolved_references = cpool->resolved_references();
da91efe96a93 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 6266
diff changeset
431 return resolved_references->obj_at(ref_index);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
432 }
a61af66fc99e Initial load
duke
parents:
diff changeset
433
a61af66fc99e Initial load
duke
parents:
diff changeset
434
8021
8d9fc28831cc 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 6985
diff changeset
435 #if INCLUDE_JVMTI
0
a61af66fc99e Initial load
duke
parents:
diff changeset
436 // RedefineClasses() API support:
8021
8d9fc28831cc 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 6985
diff changeset
437 // If this ConstantPoolCacheEntry refers to old_method then update it
0
a61af66fc99e Initial load
duke
parents:
diff changeset
438 // to refer to new_method.
6725
da91efe96a93 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 6266
diff changeset
439 bool ConstantPoolCacheEntry::adjust_method_entry(Method* old_method,
da91efe96a93 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 6266
diff changeset
440 Method* new_method, bool * trace_name_printed) {
0
a61af66fc99e Initial load
duke
parents:
diff changeset
441
a61af66fc99e Initial load
duke
parents:
diff changeset
442 if (is_vfinal()) {
6266
1d7922586cf6 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 5898
diff changeset
443 // 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
444 if (f2_as_vfinal_method() == old_method) {
0
a61af66fc99e Initial load
duke
parents:
diff changeset
445 // 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
446 // NOTE: can't use set_f2_as_vfinal_method as it asserts on different values
0
a61af66fc99e Initial load
duke
parents:
diff changeset
447 _f2 = (intptr_t)new_method;
a61af66fc99e Initial load
duke
parents:
diff changeset
448 if (RC_TRACE_IN_RANGE(0x00100000, 0x00400000)) {
a61af66fc99e Initial load
duke
parents:
diff changeset
449 if (!(*trace_name_printed)) {
a61af66fc99e Initial load
duke
parents:
diff changeset
450 // RC_TRACE_MESG macro has an embedded ResourceMark
a61af66fc99e Initial load
duke
parents:
diff changeset
451 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
452 old_method->method_holder()->external_name()));
0
a61af66fc99e Initial load
duke
parents:
diff changeset
453 *trace_name_printed = true;
a61af66fc99e Initial load
duke
parents:
diff changeset
454 }
a61af66fc99e Initial load
duke
parents:
diff changeset
455 // RC_TRACE macro has an embedded ResourceMark
a61af66fc99e Initial load
duke
parents:
diff changeset
456 RC_TRACE(0x00400000, ("cpc vf-entry update: %s(%s)",
a61af66fc99e Initial load
duke
parents:
diff changeset
457 new_method->name()->as_C_string(),
a61af66fc99e Initial load
duke
parents:
diff changeset
458 new_method->signature()->as_C_string()));
a61af66fc99e Initial load
duke
parents:
diff changeset
459 }
a61af66fc99e Initial load
duke
parents:
diff changeset
460 return true;
a61af66fc99e Initial load
duke
parents:
diff changeset
461 }
a61af66fc99e Initial load
duke
parents:
diff changeset
462
a61af66fc99e Initial load
duke
parents:
diff changeset
463 // f1() is not used with virtual entries so bail out
a61af66fc99e Initial load
duke
parents:
diff changeset
464 return false;
a61af66fc99e Initial load
duke
parents:
diff changeset
465 }
a61af66fc99e Initial load
duke
parents:
diff changeset
466
6725
da91efe96a93 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 6266
diff changeset
467 if (_f1 == NULL) {
0
a61af66fc99e Initial load
duke
parents:
diff changeset
468 // NULL f1() means this is a virtual entry so bail out
a61af66fc99e Initial load
duke
parents:
diff changeset
469 // We are assuming that the vtable index does not need change.
a61af66fc99e Initial load
duke
parents:
diff changeset
470 return false;
a61af66fc99e Initial load
duke
parents:
diff changeset
471 }
a61af66fc99e Initial load
duke
parents:
diff changeset
472
6725
da91efe96a93 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 6266
diff changeset
473 if (_f1 == old_method) {
0
a61af66fc99e Initial load
duke
parents:
diff changeset
474 _f1 = new_method;
a61af66fc99e Initial load
duke
parents:
diff changeset
475 if (RC_TRACE_IN_RANGE(0x00100000, 0x00400000)) {
a61af66fc99e Initial load
duke
parents:
diff changeset
476 if (!(*trace_name_printed)) {
a61af66fc99e Initial load
duke
parents:
diff changeset
477 // RC_TRACE_MESG macro has an embedded ResourceMark
a61af66fc99e Initial load
duke
parents:
diff changeset
478 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
479 old_method->method_holder()->external_name()));
0
a61af66fc99e Initial load
duke
parents:
diff changeset
480 *trace_name_printed = true;
a61af66fc99e Initial load
duke
parents:
diff changeset
481 }
a61af66fc99e Initial load
duke
parents:
diff changeset
482 // RC_TRACE macro has an embedded ResourceMark
a61af66fc99e Initial load
duke
parents:
diff changeset
483 RC_TRACE(0x00400000, ("cpc entry update: %s(%s)",
a61af66fc99e Initial load
duke
parents:
diff changeset
484 new_method->name()->as_C_string(),
a61af66fc99e Initial load
duke
parents:
diff changeset
485 new_method->signature()->as_C_string()));
a61af66fc99e Initial load
duke
parents:
diff changeset
486 }
a61af66fc99e Initial load
duke
parents:
diff changeset
487 return true;
a61af66fc99e Initial load
duke
parents:
diff changeset
488 }
a61af66fc99e Initial load
duke
parents:
diff changeset
489
a61af66fc99e Initial load
duke
parents:
diff changeset
490 return false;
a61af66fc99e Initial load
duke
parents:
diff changeset
491 }
a61af66fc99e Initial load
duke
parents:
diff changeset
492
8021
8d9fc28831cc 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 6985
diff changeset
493 // 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
494 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
495 if (is_vfinal()) {
8021
8d9fc28831cc 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 6985
diff changeset
496 // 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
497 Metadata* f2 = (Metadata*)_f2;
8021
8d9fc28831cc 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 6985
diff changeset
498 // 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
499 // _f2 == NULL || !_f2->is_method() are just as unexpected here.
8d9fc28831cc 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 6985
diff changeset
500 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
501 !((Method*)f2)->is_old() && !((Method*)f2)->is_obsolete());
8d9fc28831cc 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 6985
diff changeset
502 } else if (_f1 == NULL ||
8d9fc28831cc 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 6985
diff changeset
503 (NOT_PRODUCT(_f1->is_valid() &&) !_f1->is_method())) {
8d9fc28831cc 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 6985
diff changeset
504 // _f1 == NULL || !_f1->is_method() are OK here
8d9fc28831cc 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 6985
diff changeset
505 return true;
6725
da91efe96a93 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 6266
diff changeset
506 }
22885
367427923e39 8042796: jvmtiRedefineClasses.cpp: guarantee(false) failed: OLD and/or OBSOLETE method(s) found
sspitsyn
parents: 20197
diff changeset
507 // return false if _f1 refers to a non-deleted old or obsolete method
8021
8d9fc28831cc 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 6985
diff changeset
508 return (NOT_PRODUCT(_f1->is_valid() &&) _f1->is_method() &&
22885
367427923e39 8042796: jvmtiRedefineClasses.cpp: guarantee(false) failed: OLD and/or OBSOLETE method(s) found
sspitsyn
parents: 20197
diff changeset
509 (f1_as_method()->is_deleted() ||
367427923e39 8042796: jvmtiRedefineClasses.cpp: guarantee(false) failed: OLD and/or OBSOLETE method(s) found
sspitsyn
parents: 20197
diff changeset
510 (!f1_as_method()->is_old() && !f1_as_method()->is_obsolete())));
6725
da91efe96a93 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 6266
diff changeset
511 }
da91efe96a93 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 6266
diff changeset
512
22886
fdde6a70ea85 8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents: 22885
diff changeset
513 Method* ConstantPoolCacheEntry::get_interesting_method_entry(Klass* k) {
0
a61af66fc99e Initial load
duke
parents:
diff changeset
514 if (!is_method_entry()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
515 // not a method entry so not interesting by default
22886
fdde6a70ea85 8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents: 22885
diff changeset
516 return NULL;
0
a61af66fc99e Initial load
duke
parents:
diff changeset
517 }
6725
da91efe96a93 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 6266
diff changeset
518 Method* m = NULL;
0
a61af66fc99e Initial load
duke
parents:
diff changeset
519 if (is_vfinal()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
520 // 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
521 m = f2_as_vfinal_method();
1d7922586cf6 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 5898
diff changeset
522 } else if (is_f1_null()) {
0
a61af66fc99e Initial load
duke
parents:
diff changeset
523 // NULL _f1 means this is a virtual entry so also not interesting
22886
fdde6a70ea85 8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents: 22885
diff changeset
524 return NULL;
0
a61af66fc99e Initial load
duke
parents:
diff changeset
525 } else {
6725
da91efe96a93 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 6266
diff changeset
526 if (!(_f1->is_method())) {
da91efe96a93 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 6266
diff changeset
527 // _f1 can also contain a Klass* for an interface
22886
fdde6a70ea85 8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents: 22885
diff changeset
528 return NULL;
0
a61af66fc99e Initial load
duke
parents:
diff changeset
529 }
6266
1d7922586cf6 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 5898
diff changeset
530 m = f1_as_method();
0
a61af66fc99e Initial load
duke
parents:
diff changeset
531 }
a61af66fc99e Initial load
duke
parents:
diff changeset
532 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
533 if (m == NULL || !m->is_method() || (k != NULL && m->method_holder() != k)) {
0
a61af66fc99e Initial load
duke
parents:
diff changeset
534 // robustness for above sanity checks or method is not in
a61af66fc99e Initial load
duke
parents:
diff changeset
535 // the interesting class
22886
fdde6a70ea85 8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents: 22885
diff changeset
536 return NULL;
0
a61af66fc99e Initial load
duke
parents:
diff changeset
537 }
a61af66fc99e Initial load
duke
parents:
diff changeset
538 // the method is in the interesting class so the entry is interesting
22886
fdde6a70ea85 8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents: 22885
diff changeset
539 return m;
0
a61af66fc99e Initial load
duke
parents:
diff changeset
540 }
8021
8d9fc28831cc 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 6985
diff changeset
541 #endif // INCLUDE_JVMTI
0
a61af66fc99e Initial load
duke
parents:
diff changeset
542
a61af66fc99e Initial load
duke
parents:
diff changeset
543 void ConstantPoolCacheEntry::print(outputStream* st, int index) const {
a61af66fc99e Initial load
duke
parents:
diff changeset
544 // print separator
5898
f096e1b74d85 7148126: ConstantPoolCacheEntry::print prints to wrong stream
dholmes
parents: 3938
diff changeset
545 if (index == 0) st->print_cr(" -------------");
0
a61af66fc99e Initial load
duke
parents:
diff changeset
546 // print entry
5898
f096e1b74d85 7148126: ConstantPoolCacheEntry::print prints to wrong stream
dholmes
parents: 3938
diff changeset
547 st->print("%3d ("PTR_FORMAT") ", index, (intptr_t)this);
8021
8d9fc28831cc 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 6985
diff changeset
548 st->print_cr("[%02x|%02x|%5d]", bytecode_2(), bytecode_1(),
8d9fc28831cc 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 6985
diff changeset
549 constant_pool_index());
6725
da91efe96a93 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 6266
diff changeset
550 st->print_cr(" [ "PTR_FORMAT"]", (intptr_t)_f1);
5898
f096e1b74d85 7148126: ConstantPoolCacheEntry::print prints to wrong stream
dholmes
parents: 3938
diff changeset
551 st->print_cr(" [ "PTR_FORMAT"]", (intptr_t)_f2);
f096e1b74d85 7148126: ConstantPoolCacheEntry::print prints to wrong stream
dholmes
parents: 3938
diff changeset
552 st->print_cr(" [ "PTR_FORMAT"]", (intptr_t)_flags);
f096e1b74d85 7148126: ConstantPoolCacheEntry::print prints to wrong stream
dholmes
parents: 3938
diff changeset
553 st->print_cr(" -------------");
0
a61af66fc99e Initial load
duke
parents:
diff changeset
554 }
a61af66fc99e Initial load
duke
parents:
diff changeset
555
a61af66fc99e Initial load
duke
parents:
diff changeset
556 void ConstantPoolCacheEntry::verify(outputStream* st) const {
a61af66fc99e Initial load
duke
parents:
diff changeset
557 // not implemented yet
a61af66fc99e Initial load
duke
parents:
diff changeset
558 }
a61af66fc99e Initial load
duke
parents:
diff changeset
559
a61af66fc99e Initial load
duke
parents:
diff changeset
560 // Implementation of ConstantPoolCache
a61af66fc99e Initial load
duke
parents:
diff changeset
561
8712
3efdfd6ddbf2 8003553: NPG: metaspace objects should be zeroed in constructors
coleenp
parents: 8025
diff changeset
562 ConstantPoolCache* ConstantPoolCache::allocate(ClassLoaderData* loader_data,
3efdfd6ddbf2 8003553: NPG: metaspace objects should be zeroed in constructors
coleenp
parents: 8025
diff changeset
563 const intStack& index_map,
13056
41cb10cbfb3c 8025937: assert(existing_f1 == NULL || existing_f1 == f1) failed: illegal field change
coleenp
parents: 12946
diff changeset
564 const intStack& invokedynamic_index_map,
8712
3efdfd6ddbf2 8003553: NPG: metaspace objects should be zeroed in constructors
coleenp
parents: 8025
diff changeset
565 const intStack& invokedynamic_map, TRAPS) {
13056
41cb10cbfb3c 8025937: assert(existing_f1 == NULL || existing_f1 == f1) failed: illegal field change
coleenp
parents: 12946
diff changeset
566
41cb10cbfb3c 8025937: assert(existing_f1 == NULL || existing_f1 == f1) failed: illegal field change
coleenp
parents: 12946
diff changeset
567 const int length = index_map.length() + invokedynamic_index_map.length();
6725
da91efe96a93 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 6266
diff changeset
568 int size = ConstantPoolCache::size(length);
da91efe96a93 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 6266
diff changeset
569
10376
a1ebd310d5c1 8014912: Restore PrintSharedSpaces functionality after NPG
iklam
parents: 10152
diff changeset
570 return new (loader_data, size, false, MetaspaceObj::ConstantPoolCacheType, THREAD)
13056
41cb10cbfb3c 8025937: assert(existing_f1 == NULL || existing_f1 == f1) failed: illegal field change
coleenp
parents: 12946
diff changeset
571 ConstantPoolCache(length, index_map, invokedynamic_index_map, invokedynamic_map);
6725
da91efe96a93 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 6266
diff changeset
572 }
da91efe96a93 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 6266
diff changeset
573
8712
3efdfd6ddbf2 8003553: NPG: metaspace objects should be zeroed in constructors
coleenp
parents: 8025
diff changeset
574 void ConstantPoolCache::initialize(const intArray& inverse_index_map,
13056
41cb10cbfb3c 8025937: assert(existing_f1 == NULL || existing_f1 == f1) failed: illegal field change
coleenp
parents: 12946
diff changeset
575 const intArray& invokedynamic_inverse_index_map,
8712
3efdfd6ddbf2 8003553: NPG: metaspace objects should be zeroed in constructors
coleenp
parents: 8025
diff changeset
576 const intArray& invokedynamic_references_map) {
13056
41cb10cbfb3c 8025937: assert(existing_f1 == NULL || existing_f1 == f1) failed: illegal field change
coleenp
parents: 12946
diff changeset
577 for (int i = 0; i < inverse_index_map.length(); i++) {
1059
389049f3f393 6858164: invokedynamic code needs some cleanup (post-6655638)
jrose
parents: 844
diff changeset
578 ConstantPoolCacheEntry* e = entry_at(i);
389049f3f393 6858164: invokedynamic code needs some cleanup (post-6655638)
jrose
parents: 844
diff changeset
579 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
580 e->initialize_entry(original_index);
6725
da91efe96a93 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 6266
diff changeset
581 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
582 }
13056
41cb10cbfb3c 8025937: assert(existing_f1 == NULL || existing_f1 == f1) failed: illegal field change
coleenp
parents: 12946
diff changeset
583
41cb10cbfb3c 8025937: assert(existing_f1 == NULL || existing_f1 == f1) failed: illegal field change
coleenp
parents: 12946
diff changeset
584 // Append invokedynamic entries at the end
41cb10cbfb3c 8025937: assert(existing_f1 == NULL || existing_f1 == f1) failed: illegal field change
coleenp
parents: 12946
diff changeset
585 int invokedynamic_offset = inverse_index_map.length();
41cb10cbfb3c 8025937: assert(existing_f1 == NULL || existing_f1 == f1) failed: illegal field change
coleenp
parents: 12946
diff changeset
586 for (int i = 0; i < invokedynamic_inverse_index_map.length(); i++) {
41cb10cbfb3c 8025937: assert(existing_f1 == NULL || existing_f1 == f1) failed: illegal field change
coleenp
parents: 12946
diff changeset
587 int offset = i + invokedynamic_offset;
41cb10cbfb3c 8025937: assert(existing_f1 == NULL || existing_f1 == f1) failed: illegal field change
coleenp
parents: 12946
diff changeset
588 ConstantPoolCacheEntry* e = entry_at(offset);
41cb10cbfb3c 8025937: assert(existing_f1 == NULL || existing_f1 == f1) failed: illegal field change
coleenp
parents: 12946
diff changeset
589 int original_index = invokedynamic_inverse_index_map[i];
41cb10cbfb3c 8025937: assert(existing_f1 == NULL || existing_f1 == f1) failed: illegal field change
coleenp
parents: 12946
diff changeset
590 e->initialize_entry(original_index);
41cb10cbfb3c 8025937: assert(existing_f1 == NULL || existing_f1 == f1) failed: illegal field change
coleenp
parents: 12946
diff changeset
591 assert(entry_at(offset) == e, "sanity");
41cb10cbfb3c 8025937: assert(existing_f1 == NULL || existing_f1 == f1) failed: illegal field change
coleenp
parents: 12946
diff changeset
592 }
41cb10cbfb3c 8025937: assert(existing_f1 == NULL || existing_f1 == f1) failed: illegal field change
coleenp
parents: 12946
diff changeset
593
6725
da91efe96a93 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 6266
diff changeset
594 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
595 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
596 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
597 #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
598 // 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
599 // 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
600 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
601 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
602 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
603 }
f6b0eb4e44cf 7200949: JSR 292: rubybench/bench/time/bench_base64.rb fails with jruby.jar not on boot class path
twisti
parents: 6737
diff changeset
604 #endif
6725
da91efe96a93 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 6266
diff changeset
605 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
606 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
607 }
1059
389049f3f393 6858164: invokedynamic code needs some cleanup (post-6655638)
jrose
parents: 844
diff changeset
608 }
0
a61af66fc99e Initial load
duke
parents:
diff changeset
609 }
a61af66fc99e Initial load
duke
parents:
diff changeset
610
8021
8d9fc28831cc 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 6985
diff changeset
611 #if INCLUDE_JVMTI
0
a61af66fc99e Initial load
duke
parents:
diff changeset
612 // RedefineClasses() API support:
8021
8d9fc28831cc 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 6985
diff changeset
613 // If any entry of this ConstantPoolCache points to any of
0
a61af66fc99e Initial load
duke
parents:
diff changeset
614 // old_methods, replace it with the corresponding new_method.
22886
fdde6a70ea85 8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents: 22885
diff changeset
615 void ConstantPoolCache::adjust_method_entries(InstanceKlass* holder, bool * trace_name_printed) {
fdde6a70ea85 8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents: 22885
diff changeset
616 for (int i = 0; i < length(); i++) {
fdde6a70ea85 8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents: 22885
diff changeset
617 ConstantPoolCacheEntry* entry = entry_at(i);
fdde6a70ea85 8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents: 22885
diff changeset
618 Method* old_method = entry->get_interesting_method_entry(holder);
fdde6a70ea85 8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents: 22885
diff changeset
619 if (old_method == NULL || !old_method->is_old()) {
fdde6a70ea85 8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents: 22885
diff changeset
620 continue; // skip uninteresting entries
fdde6a70ea85 8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents: 22885
diff changeset
621 }
fdde6a70ea85 8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents: 22885
diff changeset
622 if (old_method->is_deleted()) {
fdde6a70ea85 8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents: 22885
diff changeset
623 // clean up entries with deleted methods
fdde6a70ea85 8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents: 22885
diff changeset
624 entry->initialize_entry(entry->constant_pool_index());
fdde6a70ea85 8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents: 22885
diff changeset
625 continue;
fdde6a70ea85 8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents: 22885
diff changeset
626 }
fdde6a70ea85 8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents: 22885
diff changeset
627 Method* new_method = holder->method_with_idnum(old_method->orig_method_idnum());
fdde6a70ea85 8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents: 22885
diff changeset
628
fdde6a70ea85 8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents: 22885
diff changeset
629 assert(new_method != NULL, "method_with_idnum() should not be NULL");
fdde6a70ea85 8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents: 22885
diff changeset
630 assert(old_method != new_method, "sanity check");
fdde6a70ea85 8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents: 22885
diff changeset
631
fdde6a70ea85 8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents: 22885
diff changeset
632 entry_at(i)->adjust_method_entry(old_method, new_method, trace_name_printed);
fdde6a70ea85 8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents: 22885
diff changeset
633 }
fdde6a70ea85 8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents: 22885
diff changeset
634 }
fdde6a70ea85 8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents: 22885
diff changeset
635
8021
8d9fc28831cc 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 6985
diff changeset
636 // 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
637 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
638 for (int i = 1; i < length(); i++) {
22886
fdde6a70ea85 8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents: 22885
diff changeset
639 if (entry_at(i)->get_interesting_method_entry(NULL) != NULL &&
8021
8d9fc28831cc 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 6985
diff changeset
640 !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
641 return false;
da91efe96a93 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 6266
diff changeset
642 }
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 return true;
da91efe96a93 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 6266
diff changeset
645 }
8021
8d9fc28831cc 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 6985
diff changeset
646
8d9fc28831cc 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 6985
diff changeset
647 void ConstantPoolCache::dump_cache() {
8d9fc28831cc 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 6985
diff changeset
648 for (int i = 1; i < length(); i++) {
22886
fdde6a70ea85 8046246: the constantPoolCacheOopDesc::adjust_method_entries() used in RedefineClasses does not scale
sspitsyn
parents: 22885
diff changeset
649 if (entry_at(i)->get_interesting_method_entry(NULL) != NULL) {
8021
8d9fc28831cc 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 6985
diff changeset
650 entry_at(i)->print(tty, i);
8d9fc28831cc 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 6985
diff changeset
651 }
8d9fc28831cc 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 6985
diff changeset
652 }
8d9fc28831cc 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 6985
diff changeset
653 }
8d9fc28831cc 7182152: Instrumentation hot swap test incorrect monitor count
dcubed
parents: 6985
diff changeset
654 #endif // INCLUDE_JVMTI
6725
da91efe96a93 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 6266
diff changeset
655
da91efe96a93 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 6266
diff changeset
656
da91efe96a93 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 6266
diff changeset
657 // Printing
da91efe96a93 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 6266
diff changeset
658
da91efe96a93 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 6266
diff changeset
659 void ConstantPoolCache::print_on(outputStream* st) const {
da91efe96a93 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 6266
diff changeset
660 assert(is_constantPoolCache(), "obj must be constant pool cache");
17937
78bbf4d43a14 8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents: 14443
diff changeset
661 st->print_cr("%s", internal_name());
6725
da91efe96a93 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 6266
diff changeset
662 // print constant pool cache entries
da91efe96a93 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 6266
diff changeset
663 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
664 }
da91efe96a93 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 6266
diff changeset
665
da91efe96a93 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 6266
diff changeset
666 void ConstantPoolCache::print_value_on(outputStream* st) const {
da91efe96a93 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 6266
diff changeset
667 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
668 st->print("cache [%d]", length());
da91efe96a93 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 6266
diff changeset
669 print_address_on(st);
da91efe96a93 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 6266
diff changeset
670 st->print(" for ");
da91efe96a93 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 6266
diff changeset
671 constant_pool()->print_value_on(st);
da91efe96a93 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 6266
diff changeset
672 }
da91efe96a93 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 6266
diff changeset
673
da91efe96a93 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 6266
diff changeset
674
da91efe96a93 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 6266
diff changeset
675 // Verification
da91efe96a93 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 6266
diff changeset
676
da91efe96a93 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 6266
diff changeset
677 void ConstantPoolCache::verify_on(outputStream* st) {
da91efe96a93 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 6266
diff changeset
678 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
679 // print constant pool cache entries
da91efe96a93 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 6266
diff changeset
680 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
681 }