Re: RFR(XXS): 8156777: [TESTBUG] test/testlibrary_tests/SimpleClassFileLoadHookTest.java requires non minimal VM

2016-05-11 Thread mikhailo
Fix looks good to me. Thank you, Misha On 05/11/2016 09:36 AM, Leonid Mesnik wrote: Hi Could you please review this extra small fix which add @requires to the single test. This test is not compatible with minimal VM even it doesn't require an

RFR(XXS): 8156777: [TESTBUG] test/testlibrary_tests/SimpleClassFileLoadHookTest.java requires non minimal VM

2016-05-11 Thread Leonid Mesnik
Hi Could you please review this extra small fix which add @requires to the single test. This test is not compatible with minimal VM even it doesn't require any specific modules outside compact2. So I've just added corresponding @requires. Bug

RE: RFR[9u-dev]: 8150900: Implement diagnostic_pd

2016-05-11 Thread Cheleswer Sahu
Thanks Christian for review. I will correct the alignment.   Regards, Cheleswer   From: Christian Thalinger Sent: Wednesday, May 11, 2016 1:00 AM To: Cheleswer Sahu Cc: Kevin Walls; Gerard Ziemski; serviceability-dev@openjdk.java.net; hotspot-runtime-...@openjdk.java.net Subject: Re: RFR[9u-