Re: RFR: JDK-8147388: Add diagnostic commands to attach JVMTI agent.

2016-02-17 Thread serguei.spit...@oracle.com
On 2/17/16 03:42, Yasumasa Suenaga wrote: Hi all, Jaroslav found that a patch for JDK-8147388 cannot build on Windows. So I fix it in new webrev: http://cr.openjdk.java.net/~ysuenaga/JDK-8147388/webrev.05/ jtreg test for this feature is passed on Windows i586 and Fedora23 x86_64. Could you

Re: RFR[9u-dev] 8130425: libjvm crash due to stack overflow in executables with 32k tbss/tdata

2016-02-17 Thread Roger Riggs
Hi, ok. A release note will be needed to document the new property. Please add the label 'release-note' and a separate comment with the proposed release note text. Thanks, Roger On 2/17/2016 10:17 AM, cheleswer sahu wrote: Hi, I have made changes in the property name (jdk.lang.processReape

RFR:8150067:Quarantine serviceability/tmtools/jstat/GcCapacityTest.java test

2016-02-17 Thread Alexander Kulyakhtin
Hi, Could you, please, review this trivial test-only change (quarantining the test): CR: https://bugs.openjdk.java.net/browse/JDK-8150067 Webrev: http://cr.openjdk.java.net/~akulyakh/8150067/test/serviceability/tmtools/jstat/GcCapacityTest.java.udiff.html The test needs to be quarantined until

Re: RFR[9u-dev] 8130425: libjvm crash due to stack overflow in executables with 32k tbss/tdata

2016-02-17 Thread cheleswer sahu
Hi, I have made changes in the property name (jdk.lang.processReaperUseDefaultStackSize) and code as suggested in the earlier reviews. --- old/src/java.base/share/classes/java/lang/ProcessHandleImpl.java 2016-02-17 18:48:10.54563 +0530 +++ new/src/java.base/share/classes/java/lang/Proces

Re: jmx-dev [ping] Re: RFR 8145919: sun/management/jmxremote/bootstrap/RmiSslBootstrapTest failed with Connection failed for no credentials (Round 2)

2016-02-17 Thread Daniel Fuchs
Thanks Jaroslav, that's great! If these certificates have an expiration date then it's possible that we will have to regenerate them from time to time... cheers, -- daniel On 17/02/16 13:13, Jaroslav Bachorik wrote: On 16.2.2016 11:56, Daniel Fuchs wrote: Hi Jaroslav, I have no objection to

Re: jmx-dev [ping] Re: RFR 8145919: sun/management/jmxremote/bootstrap/RmiSslBootstrapTest failed with Connection failed for no credentials (Round 2)

2016-02-17 Thread Jaroslav Bachorik
On 16.2.2016 11:56, Daniel Fuchs wrote: Hi Jaroslav, I have no objection to this change. Could you add a comment somewhere to explain how you generated the truststore and keystore - in case we need to tweak that again in the future? I've added a simple readme file next to the keystores. http

Re: RFR: JDK-8147388: Add diagnostic commands to attach JVMTI agent.

2016-02-17 Thread Yasumasa Suenaga
Hi all, Jaroslav found that a patch for JDK-8147388 cannot build on Windows. So I fix it in new webrev: http://cr.openjdk.java.net/~ysuenaga/JDK-8147388/webrev.05/ jtreg test for this feature is passed on Windows i586 and Fedora23 x86_64. Could you review it again? Thanks, Yasumasa On 20