harmony-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Ilya Leviev (JIRA)" <j...@apache.org>
Subject [jira] Created: (HARMONY-3896) [drlvm][thread] Race condition at thread_java_monitors.c":345 and thread_native_suspend.c":189
Date Thu, 17 May 2007 12:43:16 GMT
[drlvm][thread] Race condition at thread_java_monitors.c":345 and thread_native_suspend.c":189
----------------------------------------------------------------------------------------------

                 Key: HARMONY-3896
                 URL: https://issues.apache.org/jira/browse/HARMONY-3896
             Project: Harmony
          Issue Type: Bug
          Components: DRLVM
            Reporter: Ilya Leviev


[drlvm][thread] Race condition at thread_java_monitors.c":345 and thread_native_suspend.c":189

TC report on thread unsafe access to "state" field that result in race condition that occur
during concurrent execution of jthread_monitor_timed_wait and wait_safe_region_event functions.

Write -> Write data-race	
Memory write at "thread_native_suspend.c":189 conflicts with a prior memory write at "thread_java_monitors.c":345


Stack Trace: 

Context
	Function void vm_execute_java_method_array(struct _jmethodID *,union jvalue *,union jvalue
*) "ini.cpp":60
	Function ExecuteMethod "em_intf.cpp":43
	Function void DrlEMImpl::executeMethod(struct _jmethodID *,union jvalue *,union jvalue *)
"drlemimpl.cpp":509
	Function void JIT_execute_method_default(void *,struct _jmethodID *,union jvalue *,union
jvalue *) "ini_ia32.cpp":199
	Function vm_invoke_native_array_stub "ini_ia32.cpp":76
	Function hythread_thin_monitor_exit "thread_native_thin_monitor.c":416
	Function hythread_thin_monitor_try_enter "thread_native_thin_monitor.c":323
	Function unreserve_lock "thread_native_thin_monitor.c":162
	Function hythread_suspend_other "thread_native_suspend.c":255
	Function wait_safe_region_event "thread_native_suspend.c":169

1st Access
	Function class VM_thread * get_thread_ptr_stub(void) "thread_manager.cpp":138
	Function m2n_free_local_handles "m2n_ia32.cpp":268
	Function hythread_self "hythread.h":465
	Function vm_get_gc_thread_local "c_interface.cpp":2334
	Function gc_alloc_fast "gc_for_vm.cpp":178
	Function hythread_self "hythread.h":465
	Function class VM_thread * get_thread_ptr_stub(void) "thread_manager.cpp":138
	Function hythread_suspend_enable "hythread_ext.h":370
	Function Java_java_lang_VMThreadManager_wait "java_lang_vmthreadmanager.cpp":202
	Function jthread_monitor_timed_wait "thread_java_monitors.c":345


function jthread_monitor_timed_wait "thread_java_monitors.c":345
"343"	""	"        tm_native_thread->state &= ~TM_THREAD_STATE_WAITING_INDEFINITELY;"
"344"	""	"     }"
"345"	"*"	"     tm_native_thread->state |= TM_THREAD_STATE_RUNNABLE;"  
"346"	""	" "




2nd Access
	Function void vm_execute_java_method_array(struct _jmethodID *,union jvalue *,union jvalue
*) "ini.cpp":60
	Function ExecuteMethod "em_intf.cpp":43
	Function void DrlEMImpl::executeMethod(struct _jmethodID *,union jvalue *,union jvalue *)
"drlemimpl.cpp":509
	Function void JIT_execute_method_default(void *,struct _jmethodID *,union jvalue *,union
jvalue *) "ini_ia32.cpp":199
	Function vm_invoke_native_array_stub "ini_ia32.cpp":76
	Function hythread_thin_monitor_exit "thread_native_thin_monitor.c":416
	Function hythread_thin_monitor_try_enter "thread_native_thin_monitor.c":323
	Function unreserve_lock "thread_native_thin_monitor.c":162
	Function hythread_suspend_other "thread_native_suspend.c":255
	Function wait_safe_region_event "thread_native_suspend.c":189


function wait_safe_region_event "thread_native_suspend.c":189

"187"	""	"            thread, thread->suspend_count, thread->request));"
"188"	""	"     hymutex_lock(&thread->mutex);"
"189"	"*"	"     thread->state |= TM_THREAD_STATE_SUSPENDED;"
"190"	""	"     hymutex_unlock(&thread->mutex);"
"191"	""	"     return TM_ERROR_NONE;"


If it not affect correctness of execution I will mark it by special API for prevention of
further alarms on this race. 





-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


Mime
View raw message