
<html>

  <head>


    <meta http-equiv="content-type" content="text/html; charset=utf-8">

  </head>

  <body bgcolor="#FFFFFF" text="#000000">

    Hi all,<br>

    <br>

    Could I have some reviews for this change that adds G1 pre-barrier?<br>

    <br>

    JvmtiTagHashmapEntry has a bare oop member and this is a weak

    reference. So any place that allows this oop to 'escape' needs the

    G1 pre-barrier.<br>

    JvmtiEnv::GetObjectsWithTags() provides such an escape path.<br>

    <br>

    For G1, in order to maintain the SATB invariants, reading the

    referent of a weak reference must ensure the referent is marked

    alive.<br>

    <br>

    So this proposal includes adding the pre-barrier at

    TagObjectCollector::do_entry(JvmtiTagHashmapEntry* entry) which I

    see the only place interacts(except 'peek' operations) with the bare

    oop member.<br>

    <br>

    As writing stable test could take some more time, Stefan Karlsson

    and I did some tests to provoke this problem.<br>

    ( Stefan Karlsson kindly provided the test, <a

      href="http://cr.openjdk.java.net/%7Estefank/8173013/reproducer/">http://cr.openjdk.java.net/~stefank/8173013/reproducer/</a>

    )<br>

    With this proposed patch, the problem goes away.<br>

    <br>

    CR: <a class="moz-txt-link-freetext" href="https://bugs.openjdk.java.net/browse/JDK-8173013">https://bugs.openjdk.java.net/browse/JDK-8173013</a><br>

    Webrev: <a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~sangheki/8173013/webrev.0">http://cr.openjdk.java.net/~sangheki/8173013/webrev.0</a><br>

    Testing: JPRT, some closed tests which use JVMTI and JDI.<br>

    <br>

    Thanks,<br>

    Sangheon<br>

    <br>

  </body>

</html>