truffle: src/share/vm/runtime/sharedRuntime.cpp comparison

comparison src/share/vm/runtime/sharedRuntime.cpp @ 17795:a9becfeecd1b

Merge

author	kvn
date	Wed, 22 Jan 2014 17:42:23 -0800
parents	5da8bb64b370 7b9127b17b7a
children	cf9f24de0b93

comparison

equal deleted inserted replaced

-:3514ee402842
+:a9becfeecd1b
 thread->set_is_method_handle_return(nm->is_method_handle_return(return_address));
 // native nmethods don't have exception handlers
 assert(!nm->is_native_method(), "no exception handler");
 assert(nm->header_begin() != nm->exception_begin(), "no exception handler");
 if (nm->is_deopt_pc(return_address)) {
+// If we come here because of a stack overflow, the stack may be
+// unguarded. Reguard the stack otherwise if we return to the
+// deopt blob and the stack bang causes a stack overflow we
+// crash.
+bool guard_pages_enabled = thread->stack_yellow_zone_enabled();
+if (!guard_pages_enabled) guard_pages_enabled = thread->reguard_stack();
+assert(guard_pages_enabled, "stack banging in deopt blob may cause crash");
 return SharedRuntime::deopt_blob()->unpack_with_exception();
 } else {
 return nm->exception_begin();
 }
 }
 assert(ic_miss != NULL, "must have handler");
 ResourceMark rm;
 NOT_PRODUCT(int insts_size);
-AdapterBlob* B = NULL;
+AdapterBlob* new_adapter = NULL;
 AdapterHandlerEntry* entry = NULL;
 AdapterFingerPrint* fingerprint = NULL;
 {
 MutexLocker mu(AdapterHandlerLibrary_lock);
 // make sure data structure is initialized
 // Lookup method signature's fingerprint
 entry = _adapters->lookup(total_args_passed, sig_bt);
 #ifdef ASSERT
 AdapterHandlerEntry* shared_entry = NULL;
-if (VerifyAdapterSharing && entry != NULL) {
+// Start adapter sharing verification only after the VM is booted.
+if (VerifyAdapterSharing && (entry != NULL)) {
 shared_entry = entry;
 entry = NULL;
 }
 #endif
 int comp_args_on_stack = SharedRuntime::java_calling_convention(sig_bt, regs, total_args_passed, false);
 // Make a C heap allocated version of the fingerprint to store in the adapter
 fingerprint = new AdapterFingerPrint(total_args_passed, sig_bt);
+// StubRoutines::code2() is initialized after this function can be called. As a result,
+// VerifyAdapterCalls and VerifyAdapterSharing can fail if we re-use code that generated
+// prior to StubRoutines::code2() being set. Checks refer to checks generated in an I2C
+// stub that ensure that an I2C stub is called from an interpreter frame.
+bool contains_all_checks = StubRoutines::code2() != NULL;
 // Create I2C & C2I handlers
 BufferBlob* buf = buffer_blob(); // the temporary code buffer in CodeCache
 if (buf != NULL) {
 CodeBuffer buffer(buf);
 short buffer_locs[20];
 buffer.insts()->initialize_shared_locs((relocInfo*)buffer_locs,
 sizeof(buffer_locs)/sizeof(relocInfo));
 MacroAssembler _masm(&buffer);
 entry = SharedRuntime::generate_i2c2i_adapters(&_masm,
 total_args_passed,
 comp_args_on_stack,
 sig_bt,
 regs,
 fingerprint);
 #ifdef ASSERT
 if (VerifyAdapterSharing) {
 if (shared_entry != NULL) {
-assert(shared_entry->compare_code(buf->code_begin(), buffer.insts_size(), total_args_passed, sig_bt),
+assert(shared_entry->compare_code(buf->code_begin(), buffer.insts_size()), "code must match");
-"code must match");
 // Release the one just created and return the original
 _adapters->free_entry(entry);
 return shared_entry;
 } else  {
-entry->save_code(buf->code_begin(), buffer.insts_size(), total_args_passed, sig_bt);
+entry->save_code(buf->code_begin(), buffer.insts_size());
 }
 }
 #endif
-B = AdapterBlob::create(&buffer);
+new_adapter = AdapterBlob::create(&buffer);
 NOT_PRODUCT(insts_size = buffer.insts_size());
 }
-if (B == NULL) {
+if (new_adapter == NULL) {
 // CodeCache is full, disable compilation
 // Ought to log this but compile log is only per compile thread
 // and we're some non descript Java thread.
 MutexUnlocker mu(AdapterHandlerLibrary_lock);
 CompileBroker::handle_full_code_cache();
 return NULL; // Out of CodeCache space
 }
-entry->relocate(B->content_begin());
+entry->relocate(new_adapter->content_begin());
 #ifndef PRODUCT
 // debugging suppport
 if (PrintAdapterHandlers || PrintStubCode) {
 ttyLocker ttyl;
 entry->print_adapter_on(tty);
 tty->cr();
 }
 }
 }
 #endif
+// Add the entry only if the entry contains all required checks (see sharedRuntime_xxx.cpp)
-_adapters->add(entry);
+// The checks are inserted only if -XX:+VerifyAdapterCalls is specified.
+if (contains_all_checks || !VerifyAdapterCalls) {
+_adapters->add(entry);
+}
 }
 // Outside of the lock
-if (B != NULL) {
+if (new_adapter != NULL) {
 char blob_id[256];
 jio_snprintf(blob_id,
 sizeof(blob_id),
 "%s(%s)@" PTR_FORMAT,
-B->name(),
+new_adapter->name(),
 fingerprint->as_string(),
-B->content_begin());
+new_adapter->content_begin());
-Forte::register_stub(blob_id, B->content_begin(), B->content_end());
+Forte::register_stub(blob_id, new_adapter->content_begin(),new_adapter->content_end());
 if (JvmtiExport::should_post_dynamic_code_generated()) {
-JvmtiExport::post_dynamic_code_generated(blob_id, B->content_begin(), B->content_end());
+JvmtiExport::post_dynamic_code_generated(blob_id, new_adapter->content_begin(), new_adapter->content_end());
 }
 }
 return entry;
 }
 void AdapterHandlerEntry::deallocate() {
 delete _fingerprint;
 #ifdef ASSERT
 if (_saved_code) FREE_C_HEAP_ARRAY(unsigned char, _saved_code, mtCode);
-if (_saved_sig)  FREE_C_HEAP_ARRAY(Basictype, _saved_sig, mtCode);
 #endif
 }
 #ifdef ASSERT
 // Capture the code before relocation so that it can be compared
 // against other versions.  If the code is captured after relocation
 // then relative instructions won't be equivalent.
-void AdapterHandlerEntry::save_code(unsigned char* buffer, int length, int total_args_passed, BasicType* sig_bt) {
+void AdapterHandlerEntry::save_code(unsigned char* buffer, int length) {
 _saved_code = NEW_C_HEAP_ARRAY(unsigned char, length, mtCode);
-_code_length = length;
+_saved_code_length = length;
 memcpy(_saved_code, buffer, length);
-_total_args_passed = total_args_passed;
+}
-_saved_sig = NEW_C_HEAP_ARRAY(BasicType, _total_args_passed, mtCode);
-memcpy(_saved_sig, sig_bt, _total_args_passed * sizeof(BasicType));
-}
+bool AdapterHandlerEntry::compare_code(unsigned char* buffer, int length) {
+if (length != _saved_code_length) {
-bool AdapterHandlerEntry::compare_code(unsigned char* buffer, int length, int total_args_passed, BasicType* sig_bt) {
-if (length != _code_length) {
 return false;
 }
-for (int i = 0; i < length; i++) {
-if (buffer[i] != _saved_code[i]) {
+return (memcmp(buffer, _saved_code, length) == 0) ? true : false;
-return false;
+}
-}
+#endif
-}
-return true;
-}
+/**
-#endif
+* Create a native wrapper for this native method.  The wrapper converts the
+* Java-compiled calling convention to the native convention, handles
+* arguments, and transitions to native.  On return from the native we transition
-// Create a native wrapper for this native method.  The wrapper converts the
+* back to java blocking if a safepoint is in progress.
-// java compiled calling convention to the native convention, handlizes
+*/
-// arguments, and transitions to native.  On return from the native we transition
+void AdapterHandlerLibrary::create_native_wrapper(methodHandle method) {
-// back to java blocking if a safepoint is in progress.
-nmethod *AdapterHandlerLibrary::create_native_wrapper(methodHandle method, int compile_id) {
 ResourceMark rm;
 nmethod* nm = NULL;
 assert(method->is_native(), "must be native");
 assert(method->is_method_handle_intrinsic() ||
 method->has_native_function(), "must have something valid to call!");
 {
-// perform the work while holding the lock, but perform any printing outside the lock
+// Perform the work while holding the lock, but perform any printing outside the lock
 MutexLocker mu(AdapterHandlerLibrary_lock);
 // See if somebody beat us to it
 nm = method->code();
-if (nm) {
+if (nm != NULL) {
-return nm;
+return;
 }
+const int compile_id = CompileBroker::assign_compile_id(method, CompileBroker::standard_entry_bci);
+assert(compile_id > 0, "Must generate native wrapper");
 ResourceMark rm;
 BufferBlob*  buf = buffer_blob(); // the temporary code buffer in CodeCache
 if (buf != NULL) {
 CodeBuffer buffer(buf);
 double locs_buf[20];
 buffer.insts()->initialize_shared_locs((relocInfo*)locs_buf, sizeof(locs_buf) / sizeof(relocInfo));
 // are just trampolines so the argument registers must be outgoing ones.
 const bool is_outgoing = method->is_method_handle_intrinsic();
 int comp_args_on_stack = SharedRuntime::java_calling_convention(sig_bt, regs, total_args_passed, is_outgoing);
 // Generate the compiled-to-native wrapper code
-nm = SharedRuntime::generate_native_wrapper(&_masm,
+nm = SharedRuntime::generate_native_wrapper(&_masm, method, compile_id, sig_bt, regs, ret_type);
-method,
-compile_id,
+if (nm != NULL) {
-sig_bt,
+method->set_code(method, nm);
-regs,
+}
-ret_type);
+}
-}
+} // Unlock AdapterHandlerLibrary_lock
-}
-// Must unlock before calling set_code
 // Install the generated code.
 if (nm != NULL) {
 if (PrintCompilation) {
 ttyLocker ttyl;
 CompileTask::print_compilation(tty, nm, method->is_static() ? "(static)" : "");
 }
-method->set_code(method, nm);
 nm->post_compiled_method_load_event();
 } else {
 // CodeCache is full, disable compilation
 CompileBroker::handle_full_code_cache();
 }
-return nm;
 }
 JRT_ENTRY_NO_ASYNC(void, SharedRuntime::block_for_jni_critical(JavaThread* thread))
 assert(thread == JavaThread::current(), "must be");
 // The code is about to enter a JNI lazy critical native method and

Mercurial > hg > truffle

comparison src/share/vm/runtime/sharedRuntime.cpp @ 17795:a9becfeecd1b