You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@harmony.apache.org by "Alexey Varlamov (JIRA)" <ji...@apache.org> on 2006/11/17 15:21:39 UTC

[jira] Resolved: (HARMONY-2217) [drlvm] Improve interrupt() implementation

     [ http://issues.apache.org/jira/browse/HARMONY-2217?page=all ]

Alexey Varlamov resolved HARMONY-2217.
--------------------------------------

    Resolution: Fixed
      Assignee: Alexey Varlamov

Applied at r476148, please verify.

> [drlvm] Improve interrupt() implementation
> ------------------------------------------
>
>                 Key: HARMONY-2217
>                 URL: http://issues.apache.org/jira/browse/HARMONY-2217
>             Project: Harmony
>          Issue Type: Sub-task
>          Components: DRLVM
>            Reporter: Salikh Zakirov
>         Assigned To: Alexey Varlamov
>            Priority: Minor
>         Attachments: 0001-removed-interrupter-thread.patch, 0002-refactored-wait_count-to-be-modified-only-from-wait.patch, 0003-replaced-HyThread.monitor-with-HyThread.current_condition.patch
>
>
> This issue contains improvements of interrupt() implementation on top of H-1789-updated.patch from HARMONY-1789:
> 0001-removed-interrupter-thread.patch replaces the auxiiliary interrupter thread with direct call to hycond_notify_all(), since this call does not require the monitor to be held
> 0002-refactored-wait_count-to-be-modified-only-from-wait.patch fixes wait_count handling, which is broken a bit by 0001 patch. IMHO, modifying wait_count outside of threads waiting on the monitor was not correct in the first place.
> 0003-replaced-HyThread.monitor-with-HyThread.current_condition.patch further simplifies interrupt() implementation by removing special case for a thread waiting on a monitor, since it boils down to waiting on a conditional variable.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira