|
Hi George,
Thank you a lot for review!
Serguei
On 10/17/17 05:40, George Triantafillou wrote:
Hi Serguei,
This looks good.
-George
Please, review a fix for the test bug:
https://bugs.openjdk.java.net/browse/JDK-8173936
Webrev:
http://cr.openjdk.java.net/~sspitsyn/webrevs/2017/hotspot/8173936-MAA-cflh.1/
Summary:
This test expects CFLH events in the JVMTI start phase but it
no longer gets these events because
the Jigsaw implementation has changed in a way that no longer
loads any classes in this phase
unless the capability can_generate_early_vmstart is enabled.
The fix is to expect CFLH events in the JVMTI start phase only
if this capability is enabled.
Testing:
The fixed test ClassFileLoadHook/MAAClassFileLoadHook.java is
passed now.
Thanks,
Serguei
|
