<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
</head>
<body text="#000000" bgcolor="#FFFFFF">
<p>Hi Serguei,</p>
<p>This looks good.</p>
<p>-George<br>
</p>
<div class="moz-cite-prefix">On 10/17/2017 1:35 AM,
<a class="moz-txt-link-abbreviated" href="mailto:serguei.spitsyn@oracle.com">serguei.spitsyn@oracle.com</a> wrote:<br>
</div>
<blockquote type="cite"
cite="mid:86cb693e-85d2-7ccc-52c5-d2769b77cca3@oracle.com">
<meta http-equiv="content-type" content="text/html; charset=utf-8">
Please, review a fix for the test bug:<br>
 <a class="moz-txt-link-freetext"
href="https://bugs.openjdk.java.net/browse/JDK-8173936"
moz-do-not-send="true">https://bugs.openjdk.java.net/browse/JDK-8173936</a><br>
<br>
<br>
<font face="Monaco">Webrev:<br>
Â
<a class="moz-txt-link-freetext"
href="http://cr.openjdk.java.net/%7Esspitsyn/webrevs/2017/hotspot/8173936-MAA-cflh.1/"
moz-do-not-send="true">http://cr.openjdk.java.net/~sspitsyn/webrevs/2017/hotspot/8173936-MAA-cflh.1/</a></font><br>
<br>
<br>
Summary:<br>
 This test expects CFLH events in the JVMTI start phase but it no
longer gets these events because<br>
 the Jigsaw implementation has changed in a way that no longer
loads any classes in this phase<br>
 unless the capability can_generate_early_vmstart is enabled.<br>
 The fix is to expect CFLH events in the JVMTI start phase only
if this capability is enabled.<br>
<br>
<br>
Testing:<br>
 The fixed test ClassFileLoadHook/MAAClassFileLoadHook.java is
passed now.<br>
<br>
Thanks,<br>
Serguei<br>
</blockquote>
<br>
</body>
</html>