[prev in list] [next in list] [prev in thread] [next in thread]
List: openjdk-serviceability-dev
Subject: Re: RFR (XS): 8173936 [TESTBUG] test/serviceability/jvmti/ModuleAwareAgents/ClassFileLoadHook/MAACla
From: "serguei.spitsyn () oracle ! com" <serguei ! spitsyn () oracle ! com>
Date: 2017-10-17 14:48:44
Message-ID: 2026c017-ca45-7dfb-f4fb-86d0d19e1a61 () oracle ! com
[Download RAW message or body]
<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
</head>
<body text="#000000" bgcolor="#FFFFFF">
<div class="moz-cite-prefix">Hi George,<br>
<br>
Thank you a lot for review!<br>
Serguei<br>
<br>
<br>
On 10/17/17 05:40, George Triantafillou wrote:<br>
</div>
<blockquote type="cite"
cite="mid:5b245360-bc77-9595-0f2c-7071e115dbd8@oracle.com">
<meta http-equiv="Context-Type" content="text/html; charset=utf-8">
<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"
moz-do-not-send="true">serguei.spitsyn@oracle.com</a> wrote:<br>
</div>
<blockquote type="cite"
cite="mid:86cb693e-85d2-7ccc-52c5-d2769b77cca3@oracle.com">
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>
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><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>
</blockquote>
<br>
</body>
</html>
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic