<html>

  <head>

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

  </head>

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

    <p>Hi Muthusamy,</p>

    <p>To skip running a test if "-XX:-ClassUnloading" is given it's

      better to say:</p>

    <pre><span class="new">@requires vm.opt.ClassUnloading != false</span></pre>

    This will allow to keep the test running if no 'ClassUnloading' VM

    flag is specified.<br>

    <br>

    Thanks,<br>

    Dima<br>

    <br>

    <br>

    <div class="moz-cite-prefix">On 09.08.2017 14:36, Muthusamy

      Chinnathambi wrote:<br>

    </div>

    <blockquote type="cite"

      cite="mid:69263e19-e475-4a9a-811e-7fbaf5641b45@default">

      <pre wrap="">Hi,

Please review this trivial change for the fix of bug 'JDK-8180366 [TESTBUG] gc/g1/humongousObjects/TestHumongousClassLoader should not be run with class unloading disabled'

Summary:

This testcase(TestHumongousClassLoader.java) checks that unreachable classes and unreachable humongous class loader are unloaded after GC.  Hence we need to have class unloading enabled for the correctness of the test. The fix is to prohibit execution of the test if class unloading is disabled.

jdk10 bug: <a class="moz-txt-link-freetext" href="https://bugs.openjdk.java.net/browse/JDK-8180366">https://bugs.openjdk.java.net/browse/JDK-8180366</a>

webrev link: <a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~shshahma/Muthu/8180366/webrev.00/">http://cr.openjdk.java.net/~shshahma/Muthu/8180366/webrev.00/</a>

Testing: I have manually confirmed that jtreg will no longer run the test if -XX:-ClassUnloading is used.

Regards,

Muthusamy C

</pre>

    </blockquote>

    <br>

  </body>

</html>