You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@harmony.apache.org by "Artem Aliev (JIRA)" <ji...@apache.org> on 2006/09/21 12:04:23 UTC

[jira] Created: (HARMONY-1518) [DRLVM] Fix jvmti monitor info for JIT mode debuger

[DRLVM] Fix jvmti monitor info for JIT mode debuger
---------------------------------------------------

                 Key: HARMONY-1518
                 URL: http://issues.apache.org/jira/browse/HARMONY-1518
             Project: Harmony
          Issue Type: Bug
          Components: DRLVM
            Reporter: Artem Aliev


    - Remove monitor fast path in case of TI to record all monitors.
    - fix the get_recursion to works correctly with reservation locks
    - set contended monitor in wait calls
    - add safe points to stop in correct jvmti places

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

        

[jira] Assigned: (HARMONY-1518) [DRLVM] Fix jvmti monitor info for JIT mode debuger

Posted by "Geir Magnusson Jr (JIRA)" <ji...@apache.org>.
     [ http://issues.apache.org/jira/browse/HARMONY-1518?page=all ]

Geir Magnusson Jr reassigned HARMONY-1518:
------------------------------------------

    Assignee: Geir Magnusson Jr

> [DRLVM] Fix jvmti monitor info for JIT mode debuger
> ---------------------------------------------------
>
>                 Key: HARMONY-1518
>                 URL: http://issues.apache.org/jira/browse/HARMONY-1518
>             Project: Harmony
>          Issue Type: Bug
>          Components: DRLVM
>            Reporter: Artem Aliev
>         Assigned To: Geir Magnusson Jr
>         Attachments: monitor_info.patch
>
>
>     - Remove monitor fast path in case of TI to record all monitors.
>     - fix the get_recursion to works correctly with reservation locks
>     - set contended monitor in wait calls
>     - add safe points to stop in correct jvmti places

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

        

[jira] Closed: (HARMONY-1518) [DRLVM] Fix jvmti monitor info for JIT mode debuger

Posted by "Geir Magnusson Jr (JIRA)" <ji...@apache.org>.
     [ http://issues.apache.org/jira/browse/HARMONY-1518?page=all ]

Geir Magnusson Jr closed HARMONY-1518.
--------------------------------------


> [DRLVM] Fix jvmti monitor info for JIT mode debuger
> ---------------------------------------------------
>
>                 Key: HARMONY-1518
>                 URL: http://issues.apache.org/jira/browse/HARMONY-1518
>             Project: Harmony
>          Issue Type: Bug
>          Components: DRLVM
>            Reporter: Artem Aliev
>         Assigned To: Geir Magnusson Jr
>         Attachments: monitor_info.patch, monitor_info_updated.patch, monitor_info_updated2.patch
>
>
>     - Remove monitor fast path in case of TI to record all monitors.
>     - fix the get_recursion to works correctly with reservation locks
>     - set contended monitor in wait calls
>     - add safe points to stop in correct jvmti places

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

        

[jira] Updated: (HARMONY-1518) [DRLVM] Fix jvmti monitor info for JIT mode debuger

Posted by "Nikolay Kuznetsov (JIRA)" <ji...@apache.org>.
     [ http://issues.apache.org/jira/browse/HARMONY-1518?page=all ]

Nikolay Kuznetsov updated HARMONY-1518:
---------------------------------------

    Attachment: monitor_info_updated.patch

Updated patch to be applied standalone instead of previous one.
Monitor enter cleared contended monitors incorrectly.


> [DRLVM] Fix jvmti monitor info for JIT mode debuger
> ---------------------------------------------------
>
>                 Key: HARMONY-1518
>                 URL: http://issues.apache.org/jira/browse/HARMONY-1518
>             Project: Harmony
>          Issue Type: Bug
>          Components: DRLVM
>            Reporter: Artem Aliev
>         Assigned To: Geir Magnusson Jr
>         Attachments: monitor_info.patch, monitor_info_updated.patch
>
>
>     - Remove monitor fast path in case of TI to record all monitors.
>     - fix the get_recursion to works correctly with reservation locks
>     - set contended monitor in wait calls
>     - add safe points to stop in correct jvmti places

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

        

[jira] Commented: (HARMONY-1518) [DRLVM] Fix jvmti monitor info for JIT mode debuger

Posted by "Nikolay Kuznetsov (JIRA)" <ji...@apache.org>.
    [ http://issues.apache.org/jira/browse/HARMONY-1518?page=comments#action_12436826 ] 
            
Nikolay Kuznetsov commented on HARMONY-1518:
--------------------------------------------

It turns out that my patch also include patch from HARMONY-1519 issue, I can clean this out if needed or
applying this patch will also cover 1519

> [DRLVM] Fix jvmti monitor info for JIT mode debuger
> ---------------------------------------------------
>
>                 Key: HARMONY-1518
>                 URL: http://issues.apache.org/jira/browse/HARMONY-1518
>             Project: Harmony
>          Issue Type: Bug
>          Components: DRLVM
>            Reporter: Artem Aliev
>         Assigned To: Geir Magnusson Jr
>         Attachments: monitor_info.patch, monitor_info_updated.patch
>
>
>     - Remove monitor fast path in case of TI to record all monitors.
>     - fix the get_recursion to works correctly with reservation locks
>     - set contended monitor in wait calls
>     - add safe points to stop in correct jvmti places

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

        

[jira] Updated: (HARMONY-1518) [DRLVM] Fix jvmti monitor info for JIT mode debuger

Posted by "Artem Aliev (JIRA)" <ji...@apache.org>.
     [ http://issues.apache.org/jira/browse/HARMONY-1518?page=all ]

Artem Aliev updated HARMONY-1518:
---------------------------------

    Attachment: monitor_info.patch

Fix attached

> [DRLVM] Fix jvmti monitor info for JIT mode debuger
> ---------------------------------------------------
>
>                 Key: HARMONY-1518
>                 URL: http://issues.apache.org/jira/browse/HARMONY-1518
>             Project: Harmony
>          Issue Type: Bug
>          Components: DRLVM
>            Reporter: Artem Aliev
>         Attachments: monitor_info.patch
>
>
>     - Remove monitor fast path in case of TI to record all monitors.
>     - fix the get_recursion to works correctly with reservation locks
>     - set contended monitor in wait calls
>     - add safe points to stop in correct jvmti places

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

        

[jira] Updated: (HARMONY-1518) [DRLVM] Fix jvmti monitor info for JIT mode debuger

Posted by "Nikolay Kuznetsov (JIRA)" <ji...@apache.org>.
     [ http://issues.apache.org/jira/browse/HARMONY-1518?page=all ]

Nikolay Kuznetsov updated HARMONY-1518:
---------------------------------------

    Attachment: monitor_info_updated2.patch

I've talked to Artem, he is currently preparing new patch for HARMONY-1519, and looks like it would be more convenient to have separate patched, so I've removed patch 1519 from this one.

Attached monitor_info_updated2.patch is a separate one.

> [DRLVM] Fix jvmti monitor info for JIT mode debuger
> ---------------------------------------------------
>
>                 Key: HARMONY-1518
>                 URL: http://issues.apache.org/jira/browse/HARMONY-1518
>             Project: Harmony
>          Issue Type: Bug
>          Components: DRLVM
>            Reporter: Artem Aliev
>         Assigned To: Geir Magnusson Jr
>         Attachments: monitor_info.patch, monitor_info_updated.patch, monitor_info_updated2.patch
>
>
>     - Remove monitor fast path in case of TI to record all monitors.
>     - fix the get_recursion to works correctly with reservation locks
>     - set contended monitor in wait calls
>     - add safe points to stop in correct jvmti places

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

        

[jira] Resolved: (HARMONY-1518) [DRLVM] Fix jvmti monitor info for JIT mode debuger

Posted by "Geir Magnusson Jr (JIRA)" <ji...@apache.org>.
     [ http://issues.apache.org/jira/browse/HARMONY-1518?page=all ]

Geir Magnusson Jr resolved HARMONY-1518.
----------------------------------------

    Resolution: Fixed

r449021

ubuntu 6, smoke and c-unit, most kernel

> [DRLVM] Fix jvmti monitor info for JIT mode debuger
> ---------------------------------------------------
>
>                 Key: HARMONY-1518
>                 URL: http://issues.apache.org/jira/browse/HARMONY-1518
>             Project: Harmony
>          Issue Type: Bug
>          Components: DRLVM
>            Reporter: Artem Aliev
>         Assigned To: Geir Magnusson Jr
>         Attachments: monitor_info.patch, monitor_info_updated.patch, monitor_info_updated2.patch
>
>
>     - Remove monitor fast path in case of TI to record all monitors.
>     - fix the get_recursion to works correctly with reservation locks
>     - set contended monitor in wait calls
>     - add safe points to stop in correct jvmti places

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

        

[jira] Commented: (HARMONY-1518) [DRLVM] Fix jvmti monitor info for JIT mode debuger

Posted by "Geir Magnusson Jr (JIRA)" <ji...@apache.org>.
    [ http://issues.apache.org/jira/browse/HARMONY-1518?page=comments#action_12436677 ] 
            
Geir Magnusson Jr commented on HARMONY-1518:
--------------------------------------------

When applied, this patch causes a c-unit test to fail : 

     [echo] test_java_monitors: /home/geir/dev/apache/harmony/enhanced/trunk/working_vm/vm/tests/unit/thread/utils/thread_unit_test_vm_emulator.c:133: NewGlobalRef: Assertion `object' failed.
     [echo] ## TEST FAILED


> [DRLVM] Fix jvmti monitor info for JIT mode debuger
> ---------------------------------------------------
>
>                 Key: HARMONY-1518
>                 URL: http://issues.apache.org/jira/browse/HARMONY-1518
>             Project: Harmony
>          Issue Type: Bug
>          Components: DRLVM
>            Reporter: Artem Aliev
>         Assigned To: Geir Magnusson Jr
>         Attachments: monitor_info.patch
>
>
>     - Remove monitor fast path in case of TI to record all monitors.
>     - fix the get_recursion to works correctly with reservation locks
>     - set contended monitor in wait calls
>     - add safe points to stop in correct jvmti places

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