Webrev with the fix suggested by Mikael G. :
http://cr.openjdk.java.net/~sspitsyn/webrevs/2013/hotspot/8013945-JVMTI-JSR292.2/

I'm going to integrate it after testing, but a couple of extra eyes to double-check it would be nice.

Thanks,
Serguei


On 5/24/13 2:29 AM, serguei.spit...@oracle.com wrote:
Please, review the fix for:
  bug: http://bugs.sun.com/view_bug.do?bug_id=8013945
  jbs: https://jbs.oracle.com/bugs/browse/JDK-8013945

Open webrev:
http://cr.openjdk.java.net/~sspitsyn/webrevs/2013/hotspot/8013945-JVMTI-JSR292.1/

Summary:
  CMS calls InstanceKlass::release_C_heap_structures() concurrently.
The "delete mnt" needs to take MemberNameTable_lock if !SafepointSynchronize::is_at_safepoint().

Testing:
  The vm/mlvm and Nashorn tests, the tests listed in the bug report

Thanks,
Serguei

Reply via email to