RFR (XS): 8173936 [TESTBUG] test/serviceability/jvmti/ModuleAwareAgents/ClassFileLoadHook/MAAClassFileLoadHook.java needs to be re-examined

2017-10-16 Thread serguei.spit...@oracle.com
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 bu

Re: RFR (XS): 8173936 [TESTBUG] test/serviceability/jvmti/ModuleAwareAgents/ClassFileLoadHook/MAAClassFileLoadHook.java needs to be re-examined

2017-10-16 Thread David Holmes
Hi Serguei, On 17/10/2017 3:35 PM, serguei.spit...@oracle.com wrote: 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

Re: RFR (XS): 8173936 [TESTBUG] test/serviceability/jvmti/ModuleAwareAgents/ClassFileLoadHook/MAAClassFileLoadHook.java needs to be re-examined

2017-10-16 Thread serguei.spit...@oracle.com
Hi David, On 10/16/17 23:24, David Holmes wrote: Hi Serguei, On 17/10/2017 3:35 PM, serguei.spit...@oracle.com wrote: 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/

Re: RFR (XS): 8173936 [TESTBUG] test/serviceability/jvmti/ModuleAwareAgents/ClassFileLoadHook/MAAClassFileLoadHook.java needs to be re-examined

2017-10-17 Thread George Triantafillou
Hi Serguei, This looks good. -George On 10/17/2017 1:35 AM, serguei.spit...@oracle.com wrote: 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 e

Re: RFR (XS): 8173936 [TESTBUG] test/serviceability/jvmti/ModuleAwareAgents/ClassFileLoadHook/MAAClassFileLoadHook.java needs to be re-examined

2017-10-17 Thread serguei.spit...@oracle.com
Hi George, Thank you a lot for review! Serguei On 10/17/17 05:40, George Triantafillou wrote: Hi Serguei, This looks good. -George On 10/17/2017 1:35 AM, serguei.spit...@oracle.com wrote: