You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@harmony.apache.org by "Chengrui Deng (JIRA)" <ji...@apache.org> on 2008/09/03 11:25:44 UTC

[jira] Commented: (HARMONY-5360) [drlvm][thread] Suspend enable / disable mismatch in jthread function.

    [ https://issues.apache.org/jira/browse/HARMONY-5360?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12627942#action_12627942 ] 

Chengrui Deng commented on HARMONY-5360:
----------------------------------------

Below is the code of jthread_throw_exception:

IDATA jthread_throw_exception(char *name, char *message)
{
    assert(hythread_is_suspend_enabled());
    jobject jthe = exn_create(name);
    return jthread_throw_exception_object(jthe);
}

We can see that assertion is used to assure jthread_throw_exception should be called in suspend enable mode only. So I think this issue is actually not an issue.

> [drlvm][thread] Suspend enable / disable mismatch in jthread function.
> ----------------------------------------------------------------------
>
>                 Key: HARMONY-5360
>                 URL: https://issues.apache.org/jira/browse/HARMONY-5360
>             Project: Harmony
>          Issue Type: Bug
>          Components: DRLVM
>            Reporter: Pavel Afremov
>            Priority: Minor
>
> Suspend enable / disable mismatch in jthread function.
> I found following bug during ...\vm\vmcore\src\thread\thread_java_monitors.cpp code review.
> Function jthread_throw_exception should be called in suspend disable mode only. But it can be called form jthread_monitor_exit, which can works in suspend disable and in suspend enable mode.

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