author | pliden |
Wed, 20 Nov 2019 10:37:46 +0100 | |
changeset 59152 | 59272e9e0635 |
parent 59056 | 15936b142f86 |
permissions | -rw-r--r-- |
1 | 1 |
/* |
53582 | 2 |
* Copyright (c) 1997, 2019, Oracle and/or its affiliates. All rights reserved. |
1 | 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 |
* |
|
5547
f4b087cbb361
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
4571
diff
changeset
|
19 |
* Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA |
f4b087cbb361
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
4571
diff
changeset
|
20 |
* or visit www.oracle.com if you need additional information or have any |
f4b087cbb361
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
4571
diff
changeset
|
21 |
* questions. |
1 | 22 |
* |
23 |
*/ |
|
24 |
||
7397 | 25 |
#include "precompiled.hpp" |
26 |
#include "classfile/javaClasses.hpp" |
|
54786 | 27 |
#include "classfile/symbolTable.hpp" |
7397 | 28 |
#include "classfile/systemDictionary.hpp" |
29 |
#include "classfile/vmSymbols.hpp" |
|
59016
3b9eeae66fa0
8232735: Convert PrintJNIResolving to Unified Logging
coleenp
parents:
58722
diff
changeset
|
30 |
#include "logging/log.hpp" |
3b9eeae66fa0
8232735: Convert PrintJNIResolving to Unified Logging
coleenp
parents:
58722
diff
changeset
|
31 |
#include "logging/logTag.hpp" |
7397 | 32 |
#include "memory/oopFactory.hpp" |
33 |
#include "memory/resourceArea.hpp" |
|
34 |
#include "oops/instanceKlass.hpp" |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13396
diff
changeset
|
35 |
#include "oops/method.hpp" |
7397 | 36 |
#include "oops/oop.inline.hpp" |
8076
96d498ec7ae1
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
7405
diff
changeset
|
37 |
#include "oops/symbol.hpp" |
7397 | 38 |
#include "prims/jvm_misc.hpp" |
39 |
#include "prims/nativeLookup.hpp" |
|
36551 | 40 |
#include "prims/unsafe.hpp" |
7397 | 41 |
#include "runtime/arguments.hpp" |
42 |
#include "runtime/handles.inline.hpp" |
|
57710
05ff6e27de45
8229236: CriticalJNINatives: dll handling should be done in native thread state
mdoerr
parents:
57600
diff
changeset
|
43 |
#include "runtime/interfaceSupport.inline.hpp" |
7397 | 44 |
#include "runtime/javaCalls.hpp" |
53582 | 45 |
#include "runtime/os.inline.hpp" |
7397 | 46 |
#include "runtime/sharedRuntime.hpp" |
47 |
#include "runtime/signature.hpp" |
|
15482
470d0b0c09f1
8005915: Unify SERIALGC and INCLUDE_ALTERNATE_GCS
jprovino
parents:
14488
diff
changeset
|
48 |
#include "utilities/macros.hpp" |
54347
235883996bc7
8221698: Remove redundant includes from popular header files
iklam
parents:
53582
diff
changeset
|
49 |
#include "utilities/utf8.hpp" |
50113 | 50 |
#if INCLUDE_JFR |
51 |
#include "jfr/jfr.hpp" |
|
36508 | 52 |
#endif |
1 | 53 |
|
8076
96d498ec7ae1
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
7405
diff
changeset
|
54 |
static void mangle_name_on(outputStream* st, Symbol* name, int begin, int end) { |
1 | 55 |
char* bytes = (char*)name->bytes() + begin; |
56 |
char* end_bytes = (char*)name->bytes() + end; |
|
57 |
while (bytes < end_bytes) { |
|
58 |
jchar c; |
|
59 |
bytes = UTF8::next(bytes, &c); |
|
60 |
if (c <= 0x7f && isalnum(c)) { |
|
61 |
st->put((char) c); |
|
62 |
} else { |
|
63 |
if (c == '_') st->print("_1"); |
|
64 |
else if (c == '/') st->print("_"); |
|
65 |
else if (c == ';') st->print("_2"); |
|
66 |
else if (c == '[') st->print("_3"); |
|
67 |
else st->print("_%.5x", c); |
|
68 |
} |
|
69 |
} |
|
70 |
} |
|
71 |
||
72 |
||
8076
96d498ec7ae1
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
7405
diff
changeset
|
73 |
static void mangle_name_on(outputStream* st, Symbol* name) { |
1 | 74 |
mangle_name_on(st, name, 0, name->utf8_length()); |
75 |
} |
|
76 |
||
77 |
||
33593
60764a78fa5c
8140274: methodHandles and constantPoolHandles should be passed as const references
coleenp
parents:
33160
diff
changeset
|
78 |
char* NativeLookup::pure_jni_name(const methodHandle& method) { |
1 | 79 |
stringStream st; |
80 |
// Prefix |
|
81 |
st.print("Java_"); |
|
82 |
// Klass name |
|
83 |
mangle_name_on(&st, method->klass_name()); |
|
84 |
st.print("_"); |
|
85 |
// Method name |
|
86 |
mangle_name_on(&st, method->name()); |
|
87 |
return st.as_string(); |
|
88 |
} |
|
89 |
||
90 |
||
33593
60764a78fa5c
8140274: methodHandles and constantPoolHandles should be passed as const references
coleenp
parents:
33160
diff
changeset
|
91 |
char* NativeLookup::critical_jni_name(const methodHandle& method) { |
11637
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
10565
diff
changeset
|
92 |
stringStream st; |
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
10565
diff
changeset
|
93 |
// Prefix |
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
10565
diff
changeset
|
94 |
st.print("JavaCritical_"); |
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
10565
diff
changeset
|
95 |
// Klass name |
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
10565
diff
changeset
|
96 |
mangle_name_on(&st, method->klass_name()); |
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
10565
diff
changeset
|
97 |
st.print("_"); |
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
10565
diff
changeset
|
98 |
// Method name |
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
10565
diff
changeset
|
99 |
mangle_name_on(&st, method->name()); |
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
10565
diff
changeset
|
100 |
return st.as_string(); |
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
10565
diff
changeset
|
101 |
} |
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
10565
diff
changeset
|
102 |
|
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
10565
diff
changeset
|
103 |
|
33593
60764a78fa5c
8140274: methodHandles and constantPoolHandles should be passed as const references
coleenp
parents:
33160
diff
changeset
|
104 |
char* NativeLookup::long_jni_name(const methodHandle& method) { |
1 | 105 |
// Signature ignore the wrapping parenteses and the trailing return type |
106 |
stringStream st; |
|
8076
96d498ec7ae1
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
7405
diff
changeset
|
107 |
Symbol* signature = method->signature(); |
1 | 108 |
st.print("__"); |
109 |
// find ')' |
|
110 |
int end; |
|
58722
cba8afa5cfed
8231844: Enhance type signature characters in classfile_constants.h and improve the JVM to use type signature characters more consistently
lfoltan
parents:
57710
diff
changeset
|
111 |
for (end = 0; end < signature->utf8_length() && signature->char_at(end) != JVM_SIGNATURE_ENDFUNC; end++); |
1 | 112 |
// skip first '(' |
113 |
mangle_name_on(&st, signature, 1, end); |
|
114 |
return st.as_string(); |
|
115 |
} |
|
116 |
||
117 |
extern "C" { |
|
2534 | 118 |
void JNICALL JVM_RegisterMethodHandleMethods(JNIEnv *env, jclass unsafecls); |
1 | 119 |
void JNICALL JVM_RegisterPerfMethods(JNIEnv *env, jclass perfclass); |
12095 | 120 |
void JNICALL JVM_RegisterWhiteBoxMethods(JNIEnv *env, jclass wbclass); |
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
28731
diff
changeset
|
121 |
#if INCLUDE_JVMCI |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
28731
diff
changeset
|
122 |
jobject JNICALL JVM_GetJVMCIRuntime(JNIEnv *env, jclass c); |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
28731
diff
changeset
|
123 |
void JNICALL JVM_RegisterJVMCINatives(JNIEnv *env, jclass compilerToVMClass); |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
28731
diff
changeset
|
124 |
#endif |
1 | 125 |
} |
126 |
||
8675
e9fef2a9bef7
6839872: remove implementation inheritance from JSR 292 APIs
jrose
parents:
8076
diff
changeset
|
127 |
#define CC (char*) /* cast a literal from (const char*) */ |
e9fef2a9bef7
6839872: remove implementation inheritance from JSR 292 APIs
jrose
parents:
8076
diff
changeset
|
128 |
#define FN_PTR(f) CAST_FROM_FN_PTR(void*, &f) |
e9fef2a9bef7
6839872: remove implementation inheritance from JSR 292 APIs
jrose
parents:
8076
diff
changeset
|
129 |
|
e9fef2a9bef7
6839872: remove implementation inheritance from JSR 292 APIs
jrose
parents:
8076
diff
changeset
|
130 |
static JNINativeMethod lookup_special_native_methods[] = { |
35103
d19354bcbfa6
8143628: Fork sun.misc.Unsafe and jdk.internal.misc.Unsafe native method tables
psandoz
parents:
33606
diff
changeset
|
131 |
{ CC"Java_jdk_internal_misc_Unsafe_registerNatives", NULL, FN_PTR(JVM_RegisterJDKInternalMiscUnsafeMethods) }, |
8676
9098d4e927e1
7012648: move JSR 292 to package java.lang.invoke and adjust names
jrose
parents:
8675
diff
changeset
|
132 |
{ CC"Java_java_lang_invoke_MethodHandleNatives_registerNatives", NULL, FN_PTR(JVM_RegisterMethodHandleMethods) }, |
34928
23cee048ab91
8146736: Move sun.misc performance counters to jdk.internal.perf
chegar
parents:
33606
diff
changeset
|
133 |
{ CC"Java_jdk_internal_perf_Perf_registerNatives", NULL, FN_PTR(JVM_RegisterPerfMethods) }, |
12095 | 134 |
{ CC"Java_sun_hotspot_WhiteBox_registerNatives", NULL, FN_PTR(JVM_RegisterWhiteBoxMethods) }, |
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
28731
diff
changeset
|
135 |
#if INCLUDE_JVMCI |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
28731
diff
changeset
|
136 |
{ CC"Java_jdk_vm_ci_runtime_JVMCI_initializeRuntime", NULL, FN_PTR(JVM_GetJVMCIRuntime) }, |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
28731
diff
changeset
|
137 |
{ CC"Java_jdk_vm_ci_hotspot_CompilerToVM_registerNatives", NULL, FN_PTR(JVM_RegisterJVMCINatives) }, |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
28731
diff
changeset
|
138 |
#endif |
50113 | 139 |
#if INCLUDE_JFR |
140 |
{ CC"Java_jdk_jfr_internal_JVM_registerNatives", NULL, FN_PTR(jfr_register_natives) }, |
|
36508 | 141 |
#endif |
8675
e9fef2a9bef7
6839872: remove implementation inheritance from JSR 292 APIs
jrose
parents:
8076
diff
changeset
|
142 |
}; |
e9fef2a9bef7
6839872: remove implementation inheritance from JSR 292 APIs
jrose
parents:
8076
diff
changeset
|
143 |
|
57600
3d44a84147cc
8193042: NativeLookup::lookup_critical_entry() should only load shared library once
thartmann
parents:
54847
diff
changeset
|
144 |
static address lookup_special_native(const char* jni_name) { |
8675
e9fef2a9bef7
6839872: remove implementation inheritance from JSR 292 APIs
jrose
parents:
8076
diff
changeset
|
145 |
int count = sizeof(lookup_special_native_methods) / sizeof(JNINativeMethod); |
20381
ed1047a3ac21
8024087: Remove dead JVM_{Get,Set}PrimitiveFieldValues functions
mikael
parents:
17381
diff
changeset
|
146 |
for (int i = 0; i < count; i++) { |
8675
e9fef2a9bef7
6839872: remove implementation inheritance from JSR 292 APIs
jrose
parents:
8076
diff
changeset
|
147 |
// NB: To ignore the jni prefix and jni postfix strstr is used matching. |
e9fef2a9bef7
6839872: remove implementation inheritance from JSR 292 APIs
jrose
parents:
8076
diff
changeset
|
148 |
if (strstr(jni_name, lookup_special_native_methods[i].name) != NULL) { |
e9fef2a9bef7
6839872: remove implementation inheritance from JSR 292 APIs
jrose
parents:
8076
diff
changeset
|
149 |
return CAST_FROM_FN_PTR(address, lookup_special_native_methods[i].fnPtr); |
1 | 150 |
} |
151 |
} |
|
152 |
return NULL; |
|
153 |
} |
|
154 |
||
33593
60764a78fa5c
8140274: methodHandles and constantPoolHandles should be passed as const references
coleenp
parents:
33160
diff
changeset
|
155 |
address NativeLookup::lookup_style(const methodHandle& method, char* pure_name, const char* long_name, int args_size, bool os_style, bool& in_base_library, TRAPS) { |
1 | 156 |
address entry; |
57600
3d44a84147cc
8193042: NativeLookup::lookup_critical_entry() should only load shared library once
thartmann
parents:
54847
diff
changeset
|
157 |
const char* jni_name = compute_complete_jni_name(pure_name, long_name, args_size, os_style); |
3d44a84147cc
8193042: NativeLookup::lookup_critical_entry() should only load shared library once
thartmann
parents:
54847
diff
changeset
|
158 |
|
1 | 159 |
|
160 |
// If the loader is null we have a system class, so we attempt a lookup in |
|
161 |
// the native Java library. This takes care of any bootstrapping problems. |
|
162 |
// Note: It is critical for bootstrapping that Java_java_lang_ClassLoader_00024NativeLibrary_find |
|
163 |
// gets found the first time around - otherwise an infinite loop can occure. This is |
|
164 |
// another VM/library dependency |
|
14391
df0a1573d5bd
8000725: NPG: method_holder() and pool_holder() and pool_holder field should be InstanceKlass
coleenp
parents:
13975
diff
changeset
|
165 |
Handle loader(THREAD, method->method_holder()->class_loader()); |
1 | 166 |
if (loader.is_null()) { |
167 |
entry = lookup_special_native(jni_name); |
|
168 |
if (entry == NULL) { |
|
7405
e6fc8d3926f8
6348631: remove the use of the HPI library from Hotspot
ikrylov
parents:
7397
diff
changeset
|
169 |
entry = (address) os::dll_lookup(os::native_java_library(), jni_name); |
1 | 170 |
} |
171 |
if (entry != NULL) { |
|
172 |
in_base_library = true; |
|
173 |
return entry; |
|
174 |
} |
|
175 |
} |
|
176 |
||
177 |
// Otherwise call static method findNative in ClassLoader |
|
46329
53ccc37bda19
8155672: Remove instanceKlassHandles and KlassHandles
coleenp
parents:
36615
diff
changeset
|
178 |
Klass* klass = SystemDictionary::ClassLoader_klass(); |
1 | 179 |
Handle name_arg = java_lang_String::create_from_str(jni_name, CHECK_NULL); |
180 |
||
181 |
JavaValue result(T_LONG); |
|
182 |
JavaCalls::call_static(&result, |
|
183 |
klass, |
|
8076
96d498ec7ae1
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
7405
diff
changeset
|
184 |
vmSymbols::findNative_name(), |
96d498ec7ae1
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
7405
diff
changeset
|
185 |
vmSymbols::classloader_string_long_signature(), |
1 | 186 |
// Arguments |
187 |
loader, |
|
188 |
name_arg, |
|
189 |
CHECK_NULL); |
|
190 |
entry = (address) (intptr_t) result.get_jlong(); |
|
191 |
||
192 |
if (entry == NULL) { |
|
193 |
// findNative didn't find it, if there are any agent libraries look in them |
|
194 |
AgentLibrary* agent; |
|
195 |
for (agent = Arguments::agents(); agent != NULL; agent = agent->next()) { |
|
7405
e6fc8d3926f8
6348631: remove the use of the HPI library from Hotspot
ikrylov
parents:
7397
diff
changeset
|
196 |
entry = (address) os::dll_lookup(agent->os_lib(), jni_name); |
1 | 197 |
if (entry != NULL) { |
198 |
return entry; |
|
199 |
} |
|
200 |
} |
|
201 |
} |
|
202 |
||
203 |
return entry; |
|
204 |
} |
|
205 |
||
57600
3d44a84147cc
8193042: NativeLookup::lookup_critical_entry() should only load shared library once
thartmann
parents:
54847
diff
changeset
|
206 |
const char* NativeLookup::compute_complete_jni_name(const char* pure_name, const char* long_name, int args_size, bool os_style) { |
3d44a84147cc
8193042: NativeLookup::lookup_critical_entry() should only load shared library once
thartmann
parents:
54847
diff
changeset
|
207 |
stringStream st; |
3d44a84147cc
8193042: NativeLookup::lookup_critical_entry() should only load shared library once
thartmann
parents:
54847
diff
changeset
|
208 |
if (os_style) { |
3d44a84147cc
8193042: NativeLookup::lookup_critical_entry() should only load shared library once
thartmann
parents:
54847
diff
changeset
|
209 |
os::print_jni_name_prefix_on(&st, args_size); |
11637
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
10565
diff
changeset
|
210 |
} |
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
10565
diff
changeset
|
211 |
|
57600
3d44a84147cc
8193042: NativeLookup::lookup_critical_entry() should only load shared library once
thartmann
parents:
54847
diff
changeset
|
212 |
st.print_raw(pure_name); |
3d44a84147cc
8193042: NativeLookup::lookup_critical_entry() should only load shared library once
thartmann
parents:
54847
diff
changeset
|
213 |
st.print_raw(long_name); |
3d44a84147cc
8193042: NativeLookup::lookup_critical_entry() should only load shared library once
thartmann
parents:
54847
diff
changeset
|
214 |
if (os_style) { |
3d44a84147cc
8193042: NativeLookup::lookup_critical_entry() should only load shared library once
thartmann
parents:
54847
diff
changeset
|
215 |
os::print_jni_name_suffix_on(&st, args_size); |
11637
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
10565
diff
changeset
|
216 |
} |
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
10565
diff
changeset
|
217 |
|
57600
3d44a84147cc
8193042: NativeLookup::lookup_critical_entry() should only load shared library once
thartmann
parents:
54847
diff
changeset
|
218 |
return st.as_string(); |
11637
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
10565
diff
changeset
|
219 |
} |
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
10565
diff
changeset
|
220 |
|
57600
3d44a84147cc
8193042: NativeLookup::lookup_critical_entry() should only load shared library once
thartmann
parents:
54847
diff
changeset
|
221 |
address NativeLookup::lookup_critical_style(void* dll, const char* pure_name, const char* long_name, int args_size, bool os_style) { |
3d44a84147cc
8193042: NativeLookup::lookup_critical_entry() should only load shared library once
thartmann
parents:
54847
diff
changeset
|
222 |
const char* jni_name = compute_complete_jni_name(pure_name, long_name, args_size, os_style); |
3d44a84147cc
8193042: NativeLookup::lookup_critical_entry() should only load shared library once
thartmann
parents:
54847
diff
changeset
|
223 |
assert(dll != NULL, "dll must be loaded"); |
3d44a84147cc
8193042: NativeLookup::lookup_critical_entry() should only load shared library once
thartmann
parents:
54847
diff
changeset
|
224 |
return (address)os::dll_lookup(dll, jni_name); |
3d44a84147cc
8193042: NativeLookup::lookup_critical_entry() should only load shared library once
thartmann
parents:
54847
diff
changeset
|
225 |
} |
11637
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
10565
diff
changeset
|
226 |
|
1 | 227 |
// Check all the formats of native implementation name to see if there is one |
228 |
// for the specified method. |
|
33593
60764a78fa5c
8140274: methodHandles and constantPoolHandles should be passed as const references
coleenp
parents:
33160
diff
changeset
|
229 |
address NativeLookup::lookup_entry(const methodHandle& method, bool& in_base_library, TRAPS) { |
1 | 230 |
address entry = NULL; |
231 |
in_base_library = false; |
|
232 |
// Compute pure name |
|
233 |
char* pure_name = pure_jni_name(method); |
|
234 |
||
235 |
// Compute argument size |
|
236 |
int args_size = 1 // JNIEnv |
|
237 |
+ (method->is_static() ? 1 : 0) // class for static methods |
|
238 |
+ method->size_of_parameters(); // actual parameters |
|
239 |
||
240 |
// 1) Try JNI short style |
|
241 |
entry = lookup_style(method, pure_name, "", args_size, true, in_base_library, CHECK_NULL); |
|
242 |
if (entry != NULL) return entry; |
|
243 |
||
244 |
// Compute long name |
|
245 |
char* long_name = long_jni_name(method); |
|
246 |
||
247 |
// 2) Try JNI long style |
|
248 |
entry = lookup_style(method, pure_name, long_name, args_size, true, in_base_library, CHECK_NULL); |
|
249 |
if (entry != NULL) return entry; |
|
250 |
||
251 |
// 3) Try JNI short style without os prefix/suffix |
|
252 |
entry = lookup_style(method, pure_name, "", args_size, false, in_base_library, CHECK_NULL); |
|
253 |
if (entry != NULL) return entry; |
|
254 |
||
255 |
// 4) Try JNI long style without os prefix/suffix |
|
256 |
entry = lookup_style(method, pure_name, long_name, args_size, false, in_base_library, CHECK_NULL); |
|
257 |
||
258 |
return entry; // NULL indicates not found |
|
259 |
} |
|
260 |
||
11637
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
10565
diff
changeset
|
261 |
// Check all the formats of native implementation name to see if there is one |
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
10565
diff
changeset
|
262 |
// for the specified method. |
33593
60764a78fa5c
8140274: methodHandles and constantPoolHandles should be passed as const references
coleenp
parents:
33160
diff
changeset
|
263 |
address NativeLookup::lookup_critical_entry(const methodHandle& method) { |
57710
05ff6e27de45
8229236: CriticalJNINatives: dll handling should be done in native thread state
mdoerr
parents:
57600
diff
changeset
|
264 |
assert(CriticalJNINatives, "or should not be here"); |
11637
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
10565
diff
changeset
|
265 |
|
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
10565
diff
changeset
|
266 |
if (method->is_synchronized() || |
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
10565
diff
changeset
|
267 |
!method->is_static()) { |
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
10565
diff
changeset
|
268 |
// Only static non-synchronized methods are allowed |
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
10565
diff
changeset
|
269 |
return NULL; |
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
10565
diff
changeset
|
270 |
} |
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
10565
diff
changeset
|
271 |
|
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
10565
diff
changeset
|
272 |
ResourceMark rm; |
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
10565
diff
changeset
|
273 |
|
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
10565
diff
changeset
|
274 |
Symbol* signature = method->signature(); |
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
10565
diff
changeset
|
275 |
for (int end = 0; end < signature->utf8_length(); end++) { |
51997
9ce37fa2e179
8209138: Symbol constructor uses u1 as the element type of its name argument
hseigel
parents:
50113
diff
changeset
|
276 |
if (signature->char_at(end) == 'L') { |
11637
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
10565
diff
changeset
|
277 |
// Don't allow object types |
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
10565
diff
changeset
|
278 |
return NULL; |
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
10565
diff
changeset
|
279 |
} |
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
10565
diff
changeset
|
280 |
} |
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
10565
diff
changeset
|
281 |
|
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
10565
diff
changeset
|
282 |
// Compute argument size |
47782 | 283 |
int args_size = method->size_of_parameters(); |
284 |
for (SignatureStream ss(signature); !ss.at_return_type(); ss.next()) { |
|
285 |
if (ss.is_array()) { |
|
286 |
args_size += T_INT_size; // array length parameter |
|
287 |
} |
|
288 |
} |
|
11637
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
10565
diff
changeset
|
289 |
|
57710
05ff6e27de45
8229236: CriticalJNINatives: dll handling should be done in native thread state
mdoerr
parents:
57600
diff
changeset
|
290 |
// dll handling requires I/O. Don't do that while in _thread_in_vm (safepoint may get requested). |
05ff6e27de45
8229236: CriticalJNINatives: dll handling should be done in native thread state
mdoerr
parents:
57600
diff
changeset
|
291 |
ThreadToNativeFromVM thread_in_native(JavaThread::current()); |
05ff6e27de45
8229236: CriticalJNINatives: dll handling should be done in native thread state
mdoerr
parents:
57600
diff
changeset
|
292 |
|
57600
3d44a84147cc
8193042: NativeLookup::lookup_critical_entry() should only load shared library once
thartmann
parents:
54847
diff
changeset
|
293 |
void* dll = dll_load(method); |
3d44a84147cc
8193042: NativeLookup::lookup_critical_entry() should only load shared library once
thartmann
parents:
54847
diff
changeset
|
294 |
address entry = NULL; |
3d44a84147cc
8193042: NativeLookup::lookup_critical_entry() should only load shared library once
thartmann
parents:
54847
diff
changeset
|
295 |
|
3d44a84147cc
8193042: NativeLookup::lookup_critical_entry() should only load shared library once
thartmann
parents:
54847
diff
changeset
|
296 |
if (dll != NULL) { |
3d44a84147cc
8193042: NativeLookup::lookup_critical_entry() should only load shared library once
thartmann
parents:
54847
diff
changeset
|
297 |
entry = lookup_critical_style(dll, method, args_size); |
3d44a84147cc
8193042: NativeLookup::lookup_critical_entry() should only load shared library once
thartmann
parents:
54847
diff
changeset
|
298 |
// Close the handle to avoid keeping the library alive if the native method holder is unloaded. |
3d44a84147cc
8193042: NativeLookup::lookup_critical_entry() should only load shared library once
thartmann
parents:
54847
diff
changeset
|
299 |
// This is fine because the library is still kept alive by JNI (see JVM_LoadLibrary). As soon |
3d44a84147cc
8193042: NativeLookup::lookup_critical_entry() should only load shared library once
thartmann
parents:
54847
diff
changeset
|
300 |
// as the holder class and the library are unloaded (see JVM_UnloadLibrary), the native wrapper |
3d44a84147cc
8193042: NativeLookup::lookup_critical_entry() should only load shared library once
thartmann
parents:
54847
diff
changeset
|
301 |
// that calls 'critical_entry' becomes unreachable and is unloaded as well. |
3d44a84147cc
8193042: NativeLookup::lookup_critical_entry() should only load shared library once
thartmann
parents:
54847
diff
changeset
|
302 |
os::dll_unload(dll); |
3d44a84147cc
8193042: NativeLookup::lookup_critical_entry() should only load shared library once
thartmann
parents:
54847
diff
changeset
|
303 |
} |
3d44a84147cc
8193042: NativeLookup::lookup_critical_entry() should only load shared library once
thartmann
parents:
54847
diff
changeset
|
304 |
|
3d44a84147cc
8193042: NativeLookup::lookup_critical_entry() should only load shared library once
thartmann
parents:
54847
diff
changeset
|
305 |
return entry; // NULL indicates not found |
3d44a84147cc
8193042: NativeLookup::lookup_critical_entry() should only load shared library once
thartmann
parents:
54847
diff
changeset
|
306 |
} |
3d44a84147cc
8193042: NativeLookup::lookup_critical_entry() should only load shared library once
thartmann
parents:
54847
diff
changeset
|
307 |
|
3d44a84147cc
8193042: NativeLookup::lookup_critical_entry() should only load shared library once
thartmann
parents:
54847
diff
changeset
|
308 |
void* NativeLookup::dll_load(const methodHandle& method) { |
3d44a84147cc
8193042: NativeLookup::lookup_critical_entry() should only load shared library once
thartmann
parents:
54847
diff
changeset
|
309 |
if (method->has_native_function()) { |
3d44a84147cc
8193042: NativeLookup::lookup_critical_entry() should only load shared library once
thartmann
parents:
54847
diff
changeset
|
310 |
|
3d44a84147cc
8193042: NativeLookup::lookup_critical_entry() should only load shared library once
thartmann
parents:
54847
diff
changeset
|
311 |
address current_entry = method->native_function(); |
3d44a84147cc
8193042: NativeLookup::lookup_critical_entry() should only load shared library once
thartmann
parents:
54847
diff
changeset
|
312 |
|
3d44a84147cc
8193042: NativeLookup::lookup_critical_entry() should only load shared library once
thartmann
parents:
54847
diff
changeset
|
313 |
char dll_name[JVM_MAXPATHLEN]; |
3d44a84147cc
8193042: NativeLookup::lookup_critical_entry() should only load shared library once
thartmann
parents:
54847
diff
changeset
|
314 |
int offset; |
3d44a84147cc
8193042: NativeLookup::lookup_critical_entry() should only load shared library once
thartmann
parents:
54847
diff
changeset
|
315 |
if (os::dll_address_to_library_name(current_entry, dll_name, sizeof(dll_name), &offset)) { |
3d44a84147cc
8193042: NativeLookup::lookup_critical_entry() should only load shared library once
thartmann
parents:
54847
diff
changeset
|
316 |
char ebuf[32]; |
3d44a84147cc
8193042: NativeLookup::lookup_critical_entry() should only load shared library once
thartmann
parents:
54847
diff
changeset
|
317 |
return os::dll_load(dll_name, ebuf, sizeof(ebuf)); |
3d44a84147cc
8193042: NativeLookup::lookup_critical_entry() should only load shared library once
thartmann
parents:
54847
diff
changeset
|
318 |
} |
3d44a84147cc
8193042: NativeLookup::lookup_critical_entry() should only load shared library once
thartmann
parents:
54847
diff
changeset
|
319 |
} |
3d44a84147cc
8193042: NativeLookup::lookup_critical_entry() should only load shared library once
thartmann
parents:
54847
diff
changeset
|
320 |
|
3d44a84147cc
8193042: NativeLookup::lookup_critical_entry() should only load shared library once
thartmann
parents:
54847
diff
changeset
|
321 |
return NULL; |
3d44a84147cc
8193042: NativeLookup::lookup_critical_entry() should only load shared library once
thartmann
parents:
54847
diff
changeset
|
322 |
} |
3d44a84147cc
8193042: NativeLookup::lookup_critical_entry() should only load shared library once
thartmann
parents:
54847
diff
changeset
|
323 |
|
3d44a84147cc
8193042: NativeLookup::lookup_critical_entry() should only load shared library once
thartmann
parents:
54847
diff
changeset
|
324 |
address NativeLookup::lookup_critical_style(void* dll, const methodHandle& method, int args_size) { |
3d44a84147cc
8193042: NativeLookup::lookup_critical_entry() should only load shared library once
thartmann
parents:
54847
diff
changeset
|
325 |
address entry = NULL; |
3d44a84147cc
8193042: NativeLookup::lookup_critical_entry() should only load shared library once
thartmann
parents:
54847
diff
changeset
|
326 |
const char* critical_name = critical_jni_name(method); |
3d44a84147cc
8193042: NativeLookup::lookup_critical_entry() should only load shared library once
thartmann
parents:
54847
diff
changeset
|
327 |
|
11637
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
10565
diff
changeset
|
328 |
// 1) Try JNI short style |
57600
3d44a84147cc
8193042: NativeLookup::lookup_critical_entry() should only load shared library once
thartmann
parents:
54847
diff
changeset
|
329 |
entry = lookup_critical_style(dll, critical_name, "", args_size, true); |
3d44a84147cc
8193042: NativeLookup::lookup_critical_entry() should only load shared library once
thartmann
parents:
54847
diff
changeset
|
330 |
if (entry != NULL) { |
3d44a84147cc
8193042: NativeLookup::lookup_critical_entry() should only load shared library once
thartmann
parents:
54847
diff
changeset
|
331 |
return entry; |
3d44a84147cc
8193042: NativeLookup::lookup_critical_entry() should only load shared library once
thartmann
parents:
54847
diff
changeset
|
332 |
} |
11637
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
10565
diff
changeset
|
333 |
|
57600
3d44a84147cc
8193042: NativeLookup::lookup_critical_entry() should only load shared library once
thartmann
parents:
54847
diff
changeset
|
334 |
const char* long_name = long_jni_name(method); |
11637
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
10565
diff
changeset
|
335 |
|
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
10565
diff
changeset
|
336 |
// 2) Try JNI long style |
57600
3d44a84147cc
8193042: NativeLookup::lookup_critical_entry() should only load shared library once
thartmann
parents:
54847
diff
changeset
|
337 |
entry = lookup_critical_style(dll, critical_name, long_name, args_size, true); |
3d44a84147cc
8193042: NativeLookup::lookup_critical_entry() should only load shared library once
thartmann
parents:
54847
diff
changeset
|
338 |
if (entry != NULL) { |
3d44a84147cc
8193042: NativeLookup::lookup_critical_entry() should only load shared library once
thartmann
parents:
54847
diff
changeset
|
339 |
return entry; |
3d44a84147cc
8193042: NativeLookup::lookup_critical_entry() should only load shared library once
thartmann
parents:
54847
diff
changeset
|
340 |
} |
11637
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
10565
diff
changeset
|
341 |
|
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
10565
diff
changeset
|
342 |
// 3) Try JNI short style without os prefix/suffix |
57600
3d44a84147cc
8193042: NativeLookup::lookup_critical_entry() should only load shared library once
thartmann
parents:
54847
diff
changeset
|
343 |
entry = lookup_critical_style(dll, critical_name, "", args_size, false); |
3d44a84147cc
8193042: NativeLookup::lookup_critical_entry() should only load shared library once
thartmann
parents:
54847
diff
changeset
|
344 |
if (entry != NULL) { |
3d44a84147cc
8193042: NativeLookup::lookup_critical_entry() should only load shared library once
thartmann
parents:
54847
diff
changeset
|
345 |
return entry; |
3d44a84147cc
8193042: NativeLookup::lookup_critical_entry() should only load shared library once
thartmann
parents:
54847
diff
changeset
|
346 |
} |
11637
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
10565
diff
changeset
|
347 |
|
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
10565
diff
changeset
|
348 |
// 4) Try JNI long style without os prefix/suffix |
57600
3d44a84147cc
8193042: NativeLookup::lookup_critical_entry() should only load shared library once
thartmann
parents:
54847
diff
changeset
|
349 |
return lookup_critical_style(dll, critical_name, long_name, args_size, false); |
11637
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
10565
diff
changeset
|
350 |
} |
030466036615
7013347: allow crypto functions to be called inline to enhance performance
never
parents:
10565
diff
changeset
|
351 |
|
1 | 352 |
// Check if there are any JVM TI prefixes which have been applied to the native method name. |
353 |
// If any are found, remove them before attemping the look up of the |
|
354 |
// native implementation again. |
|
355 |
// See SetNativeMethodPrefix in the JVM TI Spec for more details. |
|
33593
60764a78fa5c
8140274: methodHandles and constantPoolHandles should be passed as const references
coleenp
parents:
33160
diff
changeset
|
356 |
address NativeLookup::lookup_entry_prefixed(const methodHandle& method, bool& in_base_library, TRAPS) { |
13975
2f7431485cfa
7189254: Change makefiles for more flexibility to override defaults
jprovino
parents:
13728
diff
changeset
|
357 |
#if INCLUDE_JVMTI |
1 | 358 |
ResourceMark rm(THREAD); |
359 |
||
360 |
int prefix_count; |
|
361 |
char** prefixes = JvmtiExport::get_all_native_method_prefixes(&prefix_count); |
|
362 |
char* in_name = method->name()->as_C_string(); |
|
363 |
char* wrapper_name = in_name; |
|
364 |
// last applied prefix will be first -- go backwards |
|
365 |
for (int i = prefix_count-1; i >= 0; i--) { |
|
366 |
char* prefix = prefixes[i]; |
|
367 |
size_t prefix_len = strlen(prefix); |
|
368 |
if (strncmp(prefix, wrapper_name, prefix_len) == 0) { |
|
369 |
// has this prefix remove it |
|
370 |
wrapper_name += prefix_len; |
|
371 |
} |
|
372 |
} |
|
373 |
if (wrapper_name != in_name) { |
|
374 |
// we have a name for a wrapping method |
|
375 |
int wrapper_name_len = (int)strlen(wrapper_name); |
|
8076
96d498ec7ae1
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
7405
diff
changeset
|
376 |
TempNewSymbol wrapper_symbol = SymbolTable::probe(wrapper_name, wrapper_name_len); |
96d498ec7ae1
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
7405
diff
changeset
|
377 |
if (wrapper_symbol != NULL) { |
46329
53ccc37bda19
8155672: Remove instanceKlassHandles and KlassHandles
coleenp
parents:
36615
diff
changeset
|
378 |
Klass* k = method->method_holder(); |
53ccc37bda19
8155672: Remove instanceKlassHandles and KlassHandles
coleenp
parents:
36615
diff
changeset
|
379 |
Method* wrapper_method = k->lookup_method(wrapper_symbol, method->signature()); |
1 | 380 |
if (wrapper_method != NULL && !wrapper_method->is_native()) { |
381 |
// we found a wrapper method, use its native entry |
|
382 |
method->set_is_prefixed_native(); |
|
59056
15936b142f86
8233913: Remove implicit conversion from Method* to methodHandle
coleenp
parents:
59016
diff
changeset
|
383 |
return lookup_entry(methodHandle(THREAD, wrapper_method), in_base_library, THREAD); |
1 | 384 |
} |
385 |
} |
|
386 |
} |
|
13975
2f7431485cfa
7189254: Change makefiles for more flexibility to override defaults
jprovino
parents:
13728
diff
changeset
|
387 |
#endif // INCLUDE_JVMTI |
1 | 388 |
return NULL; |
389 |
} |
|
390 |
||
33593
60764a78fa5c
8140274: methodHandles and constantPoolHandles should be passed as const references
coleenp
parents:
33160
diff
changeset
|
391 |
address NativeLookup::lookup_base(const methodHandle& method, bool& in_base_library, TRAPS) { |
1 | 392 |
address entry = NULL; |
393 |
ResourceMark rm(THREAD); |
|
394 |
||
395 |
entry = lookup_entry(method, in_base_library, THREAD); |
|
396 |
if (entry != NULL) return entry; |
|
397 |
||
398 |
// standard native method resolution has failed. Check if there are any |
|
399 |
// JVM TI prefixes which have been applied to the native method name. |
|
400 |
entry = lookup_entry_prefixed(method, in_base_library, THREAD); |
|
401 |
if (entry != NULL) return entry; |
|
402 |
||
403 |
// Native function not found, throw UnsatisfiedLinkError |
|
54432
532e88de77eb
8221470: Print methods in exception messages in java-like Syntax.
goetz
parents:
54347
diff
changeset
|
404 |
stringStream ss; |
532e88de77eb
8221470: Print methods in exception messages in java-like Syntax.
goetz
parents:
54347
diff
changeset
|
405 |
ss.print("'"); |
532e88de77eb
8221470: Print methods in exception messages in java-like Syntax.
goetz
parents:
54347
diff
changeset
|
406 |
method->print_external_name(&ss); |
532e88de77eb
8221470: Print methods in exception messages in java-like Syntax.
goetz
parents:
54347
diff
changeset
|
407 |
ss.print("'"); |
532e88de77eb
8221470: Print methods in exception messages in java-like Syntax.
goetz
parents:
54347
diff
changeset
|
408 |
THROW_MSG_0(vmSymbols::java_lang_UnsatisfiedLinkError(), ss.as_string()); |
1 | 409 |
} |
410 |
||
411 |
||
33593
60764a78fa5c
8140274: methodHandles and constantPoolHandles should be passed as const references
coleenp
parents:
33160
diff
changeset
|
412 |
address NativeLookup::lookup(const methodHandle& method, bool& in_base_library, TRAPS) { |
1 | 413 |
if (!method->has_native_function()) { |
17381
a53ac5bf4850
7196277: JSR 292: Two jck/runtime tests crash on java.lang.invoke.MethodHandle.invokeExact
twisti
parents:
15482
diff
changeset
|
414 |
address entry = lookup_base(method, in_base_library, CHECK_NULL); |
1 | 415 |
method->set_native_function(entry, |
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13396
diff
changeset
|
416 |
Method::native_bind_event_is_interesting); |
1 | 417 |
// -verbose:jni printing |
59016
3b9eeae66fa0
8232735: Convert PrintJNIResolving to Unified Logging
coleenp
parents:
58722
diff
changeset
|
418 |
if (log_is_enabled(Debug, jni, resolve)) { |
1 | 419 |
ResourceMark rm(THREAD); |
59016
3b9eeae66fa0
8232735: Convert PrintJNIResolving to Unified Logging
coleenp
parents:
58722
diff
changeset
|
420 |
log_debug(jni, resolve)("[Dynamic-linking native method %s.%s ... JNI]", |
3b9eeae66fa0
8232735: Convert PrintJNIResolving to Unified Logging
coleenp
parents:
58722
diff
changeset
|
421 |
method->method_holder()->external_name(), |
3b9eeae66fa0
8232735: Convert PrintJNIResolving to Unified Logging
coleenp
parents:
58722
diff
changeset
|
422 |
method->name()->as_C_string()); |
1 | 423 |
} |
424 |
} |
|
425 |
return method->native_function(); |
|
426 |
} |
|
427 |
||
428 |
address NativeLookup::base_library_lookup(const char* class_name, const char* method_name, const char* signature) { |
|
429 |
EXCEPTION_MARK; |
|
430 |
bool in_base_library = true; // SharedRuntime inits some math methods. |
|
54847
59ea39bb2809
8223657: Remove unused THREAD argument from SymbolTable functions
coleenp
parents:
54786
diff
changeset
|
431 |
TempNewSymbol c_name = SymbolTable::new_symbol(class_name); |
59ea39bb2809
8223657: Remove unused THREAD argument from SymbolTable functions
coleenp
parents:
54786
diff
changeset
|
432 |
TempNewSymbol m_name = SymbolTable::new_symbol(method_name); |
59ea39bb2809
8223657: Remove unused THREAD argument from SymbolTable functions
coleenp
parents:
54786
diff
changeset
|
433 |
TempNewSymbol s_name = SymbolTable::new_symbol(signature); |
1 | 434 |
|
435 |
// Find the class |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13396
diff
changeset
|
436 |
Klass* k = SystemDictionary::resolve_or_fail(c_name, true, CATCH); |
46329
53ccc37bda19
8155672: Remove instanceKlassHandles and KlassHandles
coleenp
parents:
36615
diff
changeset
|
437 |
InstanceKlass* klass = InstanceKlass::cast(k); |
1 | 438 |
|
439 |
// Find method and invoke standard lookup |
|
440 |
methodHandle method (THREAD, |
|
28731
f7339cba0a6a
8067480: Crash in klassItable::initialize_itable_for_interface when running vm.runtime.defmeth.StaticMethodsTest.
lfoltan
parents:
25468
diff
changeset
|
441 |
klass->uncached_lookup_method(m_name, s_name, Klass::find_overpass)); |
1 | 442 |
address result = lookup(method, in_base_library, CATCH); |
443 |
assert(in_base_library, "must be in basic library"); |
|
444 |
guarantee(result != NULL, "must be non NULL"); |
|
445 |
return result; |
|
446 |
} |