Re: RFR 7162400: Intermittent java.io.IOException: Bad file number during HotSpotVirtualMachine.executeCommand

2013-07-11 Thread David Holmes
On 12/07/2013 1:17 AM, Dmitry Samersoff wrote: Peter, As UNIX_PATH_MAX is sort (108 bytes) we *must* check return value of snprintf (e.g. ll.446 of attachListener_linux.cpp) and make sure it's less or equal UNIX_PATH_MAX, otherwise we unlink wrong file in case of UNIX_PATH_MAX overflow. As all

hg: jdk8/tl/jdk: 7 new changesets

2013-07-11 Thread valerie . peng
Changeset: f225da733291 Author:valeriep Date: 2013-07-05 13:53 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/f225da733291 8012637: Adjust CipherInputStream class to work in AEAD/GCM mode Summary: Ensure the Cipher.doFinal() is called only once Reviewed-by: xuelei ! src/shar

hg: hsx/hotspot-rt/langtools: 32 new changesets

2013-07-11 Thread john . coomes
Changeset: 6debfa63a4a1 Author:vromero Date: 2013-06-20 08:45 +0100 URL: http://hg.openjdk.java.net/hsx/hotspot-rt/langtools/rev/6debfa63a4a1 8016613: javac should avoid source 8 only analysis when compiling for source 7 Reviewed-by: jjg Contributed-by: maurizio.cimadam...@oracle.co

hg: hsx/hotspot-rt/nashorn: 33 new changesets

2013-07-11 Thread john . coomes
Changeset: 6a75a505301f Author:sundar Date: 2013-06-18 18:43 +0530 URL: http://hg.openjdk.java.net/hsx/hotspot-rt/nashorn/rev/6a75a505301f 8012698: [nashorn] tests fail to run with agentvm or samevm Reviewed-by: hannesw, jlaskey ! test/src/jdk/nashorn/api/javaaccess/BooleanAccessTe

hg: hsx/hotspot-rt/jdk: 86 new changesets

2013-07-11 Thread john . coomes
Changeset: 5cfcd545ce4a Author:vadim Date: 2013-06-26 13:49 +0400 URL: http://hg.openjdk.java.net/hsx/hotspot-rt/jdk/rev/5cfcd545ce4a 8016254: several sun/java2d/OpenGL tests failed with SIGFPE Reviewed-by: prr, bae ! src/share/native/sun/java2d/opengl/OGLContext.c Changeset: 3ffa

hg: jdk8/tl/jdk: 8017212: File.createTempFile requires unnecessary "read" permission

2013-07-11 Thread dan . xu
Changeset: 10d2a4b1e576 Author:dxu Date: 2013-07-11 13:40 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/10d2a4b1e576 8017212: File.createTempFile requires unnecessary "read" permission Summary: Directly call FileSystem method to check a file existence. Also reviewed by tom.

hg: hsx/hotspot-rt/jaxp: Added tag jdk8-b98 for changeset 15e5bb51bc0c

2013-07-11 Thread john . coomes
Changeset: adf49c3ef83c Author:katleman Date: 2013-07-11 10:13 -0700 URL: http://hg.openjdk.java.net/hsx/hotspot-rt/jaxp/rev/adf49c3ef83c Added tag jdk8-b98 for changeset 15e5bb51bc0c ! .hgtags

hg: hsx/hotspot-rt/jaxws: Added tag jdk8-b98 for changeset b1fb4612a2ca

2013-07-11 Thread john . coomes
Changeset: 8ef83d4b23c9 Author:katleman Date: 2013-07-11 10:13 -0700 URL: http://hg.openjdk.java.net/hsx/hotspot-rt/jaxws/rev/8ef83d4b23c9 Added tag jdk8-b98 for changeset b1fb4612a2ca ! .hgtags

hg: hsx/hotspot-rt/corba: Added tag jdk8-b98 for changeset 3370fb6146e4

2013-07-11 Thread john . coomes
Changeset: 3f67804ab613 Author:katleman Date: 2013-07-11 10:13 -0700 URL: http://hg.openjdk.java.net/hsx/hotspot-rt/corba/rev/3f67804ab613 Added tag jdk8-b98 for changeset 3370fb6146e4 ! .hgtags

hg: hsx/hotspot-rt: 2 new changesets

2013-07-11 Thread john . coomes
Changeset: 0d0c983a817b Author:tbell Date: 2013-07-09 08:35 -0700 URL: http://hg.openjdk.java.net/hsx/hotspot-rt/rev/0d0c983a817b 8009315: F# on PATH breaks Cygwin tools (mkdir, echo, mktemp ...) Reviewed-by: erikj ! common/autoconf/generated-configure.sh ! common/autoconf/toolchai

hg: jdk8/tl/jdk: 8019799: api/java_util/jar/Pack200 test failed with compactX profiles.

2013-07-11 Thread kumar . x . srinivasan
Changeset: 0bd48087e2dc Author:ksrini Date: 2013-07-11 11:14 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/0bd48087e2dc 8019799: api/java_util/jar/Pack200 test failed with compactX profiles. Reviewed-by: dholmes ! src/share/classes/com/sun/java/util/jar/pack/UnpackerImpl.ja

hg: jdk8/tl/jdk: 8010285: Enforce the requirement of Management Interfaces being public

2013-07-11 Thread jaroslav . bachorik
Changeset: f3211af79339 Author:jbachorik Date: 2013-07-11 21:11 +0200 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/f3211af79339 8010285: Enforce the requirement of Management Interfaces being public Reviewed-by: sjiang, dfuchs, mchung ! src/share/classes/com/sun/jmx/mbeanserver/

hg: jdk8/tl/jdk: 8020321: Problem in PKCS11 regression test TestRSAKeyLength

2013-07-11 Thread valerie . peng
Changeset: 162c015c434a Author:valeriep Date: 2013-07-11 11:43 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/162c015c434a 8020321: Problem in PKCS11 regression test TestRSAKeyLength Summary: Corrected the "isValidKeyLength" array Reviewed-by: xuelei ! test/sun/security/pkcs

Re: RFR 7162400: Intermittent java.io.IOException: Bad file number during HotSpotVirtualMachine.executeCommand

2013-07-11 Thread Dmitry Samersoff
Peter, As UNIX_PATH_MAX is sort (108 bytes) we *must* check return value of snprintf (e.g. ll.446 of attachListener_linux.cpp) and make sure it's less or equal UNIX_PATH_MAX, otherwise we unlink wrong file in case of UNIX_PATH_MAX overflow. -Dmitry On 2013-07-11 18:14, Peter Allwin wrote: > Hel

RE: RFR 7162400: Intermittent java.io.IOException: Bad file number during HotSpotVirtualMachine.executeCommand

2013-07-11 Thread Christian Tornqvist
Looks good, thanks for taking the time to write a test for this! Thanks, Christian -Original Message- From: hotspot-runtime-dev-boun...@openjdk.java.net [mailto:hotspot-runtime-dev-boun...@openjdk.java.net] On Behalf Of Peter Allwin Sent: den 11 juli 2013 10:14 To: serviceability-dev@open

hg: jdk8/tl/langtools: 8013404: Unclear spec for target typing with conditional operator (?:)

2013-07-11 Thread maurizio . cimadamore
Changeset: 87a951c88a33 Author:mcimadamore Date: 2013-07-11 15:37 +0100 URL: http://hg.openjdk.java.net/jdk8/tl/langtools/rev/87a951c88a33 8013404: Unclear spec for target typing with conditional operator (?:) Summary: Fix previously ignored test Reviewed-by: jjg, vromero ! test/to

RE: RFR 7162400: Intermittent java.io.IOException: Bad file number during HotSpotVirtualMachine.executeCommand

2013-07-11 Thread Peter Allwin
Hello, Thank you everyone for the feedback, I've incorporated the recommendations into a new revision: http://cr.openjdk.java.net/~allwin/7162400/webrev.02/ Changes: - Fixed speling misteaks - Added jtreg regression test using Mikael's excellent suggestion of -XX:+PauseAtStartup, tested locall

hg: jdk8/tl/jdk: 7187144: JavaDoc for ScriptEngineFactory.getProgram() contains an error

2013-07-11 Thread sundararajan . athijegannathan
Changeset: dadcfd84d33f Author:sundar Date: 2013-07-11 18:50 +0530 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/dadcfd84d33f 7187144: JavaDoc for ScriptEngineFactory.getProgram() contains an error Reviewed-by: mcimadamore, jlaskey, hannesw, attila ! src/share/classes/javax/scrip

RFR: 8019584 javax/management/remote/mandatory/loading/MissingClassTest.java failed in nightly against jdk7u45: java.io.InvalidObjectException: Invalid notification: null

2013-07-11 Thread Jaroslav Bachorik
Please, review the change. http://cr.openjdk.java.net/~jbachorik/8019584/webrev.00/ The combination of the fix for JDK-8014085 and ObjectInputStream.readFields() not throwing CNFE when trying to deserialize an object graph containing references to non-available classes makes an InvalidObjectExcep

hg: jdk8/tl/jdk: 8019484: Sync j.u.c.ConcurrentHashMap from 166 to tl

2013-07-11 Thread paul . sandoz
Changeset: 05b164788aab Author:psandoz Date: 2013-07-11 13:07 +0200 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/05b164788aab 8019484: Sync j.u.c.ConcurrentHashMap from 166 to tl Reviewed-by: martin Contributed-by: Doug Lea ! src/share/classes/java/util/concurrent/ConcurrentHas

hg: jdk8/tl/jdk: 8019826: Test com/sun/management/HotSpotDiagnosticMXBean/SetVMOption.java fails with NPE

2013-07-11 Thread jaroslav . bachorik
Changeset: a46982a212e0 Author:jbachorik Date: 2013-07-11 09:21 +0200 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/a46982a212e0 8019826: Test com/sun/management/HotSpotDiagnosticMXBean/SetVMOption.java fails with NPE Reviewed-by: sjiang, dholmes, mchung ! test/com/sun/managemen