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