harmony-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Gregory Shimansky (JIRA)" <j...@apache.org>
Subject [jira] Resolved: (HARMONY-3949) [drlvm][class_support] Known/proven race conditions markup at "method_lookup.cpp":234 at CodeChunkInfo * Method_Lookup_Table::find(void *,bool)
Date Wed, 23 May 2007 21:10:16 GMT

     [ https://issues.apache.org/jira/browse/HARMONY-3949?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Gregory Shimansky resolved HARMONY-3949.
----------------------------------------

    Resolution: Fixed

Patch applied at 541078. Please check that it was applied as expected.

> [drlvm][class_support] Known/proven race conditions markup at "method_lookup.cpp":234
at CodeChunkInfo * Method_Lookup_Table::find(void *,bool)
> -----------------------------------------------------------------------------------------------------------------------------------------------
>
>                 Key: HARMONY-3949
>                 URL: https://issues.apache.org/jira/browse/HARMONY-3949
>             Project: Harmony
>          Issue Type: Bug
>          Components: DRLVM
>            Reporter: Ilya Leviev
>         Assigned To: Gregory Shimansky
>         Attachments: HARMONY-3949_add_markup.patch, SourceViewScreenshot-1.jpg
>
>
> TC report on thread unsafe access that result in race condition that occur during concurrent
execution of find function at "method_lookup.cpp":234.
> As issues related to vm statistic data collection in debug mode and it not affect correctness
of execution I mark it by special API for prevention of further alarms on this race.
> Write -> Write data-race	
> Memory write at "method_lookup.cpp":234 conflicts with a prior memory write at "method_lookup.cpp":234
> Stack Trace: 
> Context
> 	Function is_class_initialized "jit_runtime_support.cpp":702
> 	Function class VM_thread * get_thread_ptr_stub(void) "thread_manager.cpp":138
> 	Function m2n_pop_local_handles "m2n_ia32.cpp":257
> 	Function vm_get_gc_thread_local "c_interface.cpp":2334
> 	Function gc_alloc_fast "gc_for_vm.cpp":178
> 	Function class VM_thread * get_thread_ptr_stub(void) "thread_manager.cpp":138
> 	Function hythread_suspend_enable "hythread_ext.h":370
> 	Function Java_org_apache_harmony_vm_VMStack_getStackTrace "org_apache_harmony_vm_vmstack.cpp":378
> 	Function get_file_and_line "stack_trace.cpp":62
> 	Function class CodeChunkInfo * Method_Lookup_Table::find(void *,bool) "method_lookup.cpp":215
> 1st Access
> 	Function vm_get_gc_thread_local "c_interface.cpp":2334
> 	Function gc_alloc_fast "gc_for_vm.cpp":178
> 	Function vm_get_gc_thread_local "c_interface.cpp":2334
> 	Function gc_alloc_fast "gc_for_vm.cpp":178
> 	Function class VM_thread * get_thread_ptr_stub(void) "thread_manager.cpp":138
> 	Function hythread_suspend_enable "hythread_ext.h":370
> 	Function Java_org_apache_harmony_vm_VMStack_getCallerClass "org_apache_harmony_vm_vmstack.cpp":59
> 	Function st_get_frame "stack_trace.cpp":137
> 	Function void si_goto_previous(struct StackIterator *,bool) "stack_iterator_ia32.cpp":321
> 	Function class CodeChunkInfo * Method_Lookup_Table::find(void *,bool) "method_lookup.cpp":234
> 	"232"	""	"         if ((addr >= guess_start) && (addr < guess_end)) {"
> 	"233"	""	" #ifdef VM_STATS"
> 	"234"	"*"	"             VM_Statistics::get_vm_stats().num_method_lookup_cache_hit++;"
> 	"235"	""	" #endif //VM_STATS"
> 	"236"	""	"             return guess;"
> 2nd Access
> 	Function is_class_initialized "jit_runtime_support.cpp":702
> 	Function class VM_thread * get_thread_ptr_stub(void) "thread_manager.cpp":138
> 	Function m2n_pop_local_handles "m2n_ia32.cpp":257
> 	Function vm_get_gc_thread_local "c_interface.cpp":2334
> 	Function gc_alloc_fast "gc_for_vm.cpp":178
> 	Function class VM_thread * get_thread_ptr_stub(void) "thread_manager.cpp":138
> 	Function hythread_suspend_enable "hythread_ext.h":370
> 	Function Java_org_apache_harmony_vm_VMStack_getStackTrace "org_apache_harmony_vm_vmstack.cpp":378
> 	Function get_file_and_line "stack_trace.cpp":62
> 	Function class CodeChunkInfo * Method_Lookup_Table::find(void *,bool) "method_lookup.cpp":234
> 	"232"	""	"         if ((addr >= guess_start) && (addr < guess_end)) {"
> 	"233"	""	" #ifdef VM_STATS"
> 	"234"	"*"	"             VM_Statistics::get_vm_stats().num_method_lookup_cache_hit++;"
> 	"235"	""	" #endif //VM_STATS"
> 	"236"	""	"             return guess;"
> See also Source View screenshots.

-- 
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