Return-Path: Delivered-To: apmail-harmony-commits-archive@www.apache.org Received: (qmail 95683 invoked from network); 25 May 2007 11:01:05 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (140.211.11.2) by minotaur.apache.org with SMTP; 25 May 2007 11:01:05 -0000 Received: (qmail 17144 invoked by uid 500); 25 May 2007 11:01:10 -0000 Delivered-To: apmail-harmony-commits-archive@harmony.apache.org Received: (qmail 17129 invoked by uid 500); 25 May 2007 11:01:10 -0000 Mailing-List: contact commits-help@harmony.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@harmony.apache.org Delivered-To: mailing list commits@harmony.apache.org Received: (qmail 17120 invoked by uid 99); 25 May 2007 11:01:10 -0000 Received: from herse.apache.org (HELO herse.apache.org) (140.211.11.133) by apache.org (qpsmtpd/0.29) with ESMTP; Fri, 25 May 2007 04:01:10 -0700 X-ASF-Spam-Status: No, hits=-100.0 required=10.0 tests=ALL_TRUSTED X-Spam-Check-By: apache.org Received: from [140.211.11.4] (HELO brutus.apache.org) (140.211.11.4) by apache.org (qpsmtpd/0.29) with ESMTP; Fri, 25 May 2007 04:01:04 -0700 Received: from brutus (localhost [127.0.0.1]) by brutus.apache.org (Postfix) with ESMTP id 2224E714063 for ; Fri, 25 May 2007 04:00:43 -0700 (PDT) Message-ID: <6643852.1180090843137.JavaMail.jira@brutus> Date: Fri, 25 May 2007 04:00:43 -0700 (PDT) From: "Ilya Leviev (JIRA)" To: commits@harmony.apache.org Subject: [jira] Updated: (HARMONY-3974) [drlvm][gc_gen] Race conditions at mspace_alloc.cpp":39: at mspace_alloc_block() function In-Reply-To: <17304766.1180085356290.JavaMail.jira@brutus> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-Virus-Checked: Checked by ClamAV on apache.org [ https://issues.apache.org/jira/browse/HARMONY-3974?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ilya Leviev updated HARMONY-3974: --------------------------------- Attachment: HARMONY-3974_add_markup.patch added patch that mark race by special API for prevention of further alarms > [drlvm][gc_gen] Race conditions at mspace_alloc.cpp":39: at mspace_alloc_block() function > ----------------------------------------------------------------------------------------- > > Key: HARMONY-3974 > URL: https://issues.apache.org/jira/browse/HARMONY-3974 > Project: Harmony > Issue Type: Bug > Components: DRLVM > Reporter: Ilya Leviev > Assigned To: Xiao-Feng Li > Attachments: HARMONY-3974_add_markup.patch, SourceViewScreenshot-1.jpg > > > TC report on thread unsafe access that result in race condition that occur during concurrent execution of mspace_alloc_block() function > if it not affect correctness of execution I will mark it by special API for prevention of further alarms on this race. > Write -> Write data-race > Memory write at "mspace_alloc.cpp":39 conflicts with a prior memory write at "mspace_alloc.cpp":39 > Stack Trace: > Context > Function thread_start_proc "thread_native_basic.c":711 > Function collector_thread_func "collector.cpp":181 > Function void nongen_forward_pool(struct Collector *) "fspace_nongen_forward_pool.cpp":220 > Function collector_trace_rootsets "fspace_nongen_forward_pool.cpp":178 > Function trace_object "fspace_nongen_forward_pool.cpp":123 > Function forward_object "fspace_nongen_forward_pool.cpp":94 > Function struct Partial_Reveal_Object * collector_forward_object(struct Collector *,struct Partial_Reveal_Object *) "collector_alloc.h":51 > Function void * mos_alloc(unsigned int,struct Allocator *) "gen.cpp":273 > Function void * mspace_alloc(unsigned int,struct Allocator *) "mspace_alloc.cpp":69 > Function mspace_alloc_block "mspace_alloc.cpp":24 > 1st Access > Function thread_start_proc "thread_native_basic.c":711 > Function collector_thread_func "collector.cpp":181 > Function void nongen_forward_pool(struct Collector *) "fspace_nongen_forward_pool.cpp":220 > Function collector_trace_rootsets "fspace_nongen_forward_pool.cpp":178 > Function trace_object "fspace_nongen_forward_pool.cpp":123 > Function forward_object "fspace_nongen_forward_pool.cpp":94 > Function struct Partial_Reveal_Object * collector_forward_object(struct Collector *,struct Partial_Reveal_Object *) "collector_alloc.h":51 > Function void * mos_alloc(unsigned int,struct Allocator *) "gen.cpp":273 > Function void * mspace_alloc(unsigned int,struct Allocator *) "mspace_alloc.cpp":69 > Function mspace_alloc_block "mspace_alloc.cpp":39 > "36" "" " }" > "37" "" " /* ok, got one */" > "38" "" " Block_Header* alloc_block = (Block_Header*)&(mspace->blocks[allocated_idx - mspace->first_block_idx]);" > "39" "*" " mspace->alloced_size += GC_BLOCK_SIZE_BYTES;" > "40" "" " " > "41" "" " allocator_init_free_block(allocator, alloc_block);" > 2nd Access > Function thread_start_proc "thread_native_basic.c":711 > Function collector_thread_func "collector.cpp":181 > Function void nongen_forward_pool(struct Collector *) "fspace_nongen_forward_pool.cpp":220 > Function collector_trace_rootsets "fspace_nongen_forward_pool.cpp":178 > Function trace_object "fspace_nongen_forward_pool.cpp":123 > Function forward_object "fspace_nongen_forward_pool.cpp":94 > Function struct Partial_Reveal_Object * collector_forward_object(struct Collector *,struct Partial_Reveal_Object *) "collector_alloc.h":51 > Function void * mos_alloc(unsigned int,struct Allocator *) "gen.cpp":273 > Function void * mspace_alloc(unsigned int,struct Allocator *) "mspace_alloc.cpp":69 > Function mspace_alloc_block "mspace_alloc.cpp":39 > "36" "" " }" > "37" "" " /* ok, got one */" > "38" "" " Block_Header* alloc_block = (Block_Header*)&(mspace->blocks[allocated_idx - mspace->first_block_idx]);" > "39" "*" " mspace->alloced_size += GC_BLOCK_SIZE_BYTES;" > "40" "" " " > "41" "" " allocator_init_free_block(allocator, alloc_block);" > 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.