From vladimir.kempik at oracle.com Mon Sep 1 13:55:51 2014 From: vladimir.kempik at oracle.com (vladimir.kempik at oracle.com) Date: Mon, 01 Sep 2014 13:55:51 +0000 Subject: hg: jdk7u/jdk7u-dev/hotspot: 6883953: java -client -XX:ValueMapInitialSize=0 crashes Message-ID: <201409011355.s81DtpkG011711@aojmv0008> Changeset: 80ffd20441a8 Author: vkempik Date: 2014-09-01 15:33 +0400 URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/hotspot/rev/80ffd20441a8 6883953: java -client -XX:ValueMapInitialSize=0 crashes Summary: Add min, max checks for ValueMapInitialSize Reviewed-by: kvn, sgabdura ! src/share/vm/runtime/arguments.cpp From aleksej.efimov at oracle.com Mon Sep 1 14:47:10 2014 From: aleksej.efimov at oracle.com (Aleksej Efimov) Date: Mon, 01 Sep 2014 18:47:10 +0400 Subject: [7u-dev] RFA: 8036981: JAXB not preserving formatting for xsd:any Mixed content Message-ID: <540486EE.20403@oracle.com> Hello, Please, approve a backport of JDK8 fix to JDK7. It is a straight forward backport. New regression test passes on JDK7 build with fix and fails without it. The following xml related test shows no failures: test/javax/xml test/com/sun/org/apache/xml Thank you, Aleksej [1] JDK8 changesets: http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jaxws/rev/121e938cb9c3 http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/c2aa565bf483 [2] Review thread: http://mail.openjdk.java.net/pipermail/jdk8u-dev/2014-August/001672.html [3] JBS bug: https://bugs.openjdk.java.net/browse/JDK-8036981 From sean.coffey at oracle.com Mon Sep 1 14:52:23 2014 From: sean.coffey at oracle.com (=?UTF-8?B?U2XDoW4gQ29mZmV5?=) Date: Mon, 01 Sep 2014 15:52:23 +0100 Subject: [7u-dev] RFA: 8036981: JAXB not preserving formatting for xsd:any Mixed content In-Reply-To: <540486EE.20403@oracle.com> References: <540486EE.20403@oracle.com> Message-ID: <54048827.9010809@oracle.com> Approved. regards, Sean. On 01/09/14 15:47, Aleksej Efimov wrote: > Hello, > > Please, approve a backport of JDK8 fix to JDK7. It is a straight > forward backport. > > New regression test passes on JDK7 build with fix and fails without > it. The following xml related test shows no failures: test/javax/xml > test/com/sun/org/apache/xml > > Thank you, > Aleksej > > [1] JDK8 changesets: > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jaxws/rev/121e938cb9c3 > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/c2aa565bf483 > [2] Review thread: > http://mail.openjdk.java.net/pipermail/jdk8u-dev/2014-August/001672.html > [3] JBS bug: https://bugs.openjdk.java.net/browse/JDK-8036981 From srinivasan.raghavan at oracle.com Mon Sep 1 16:33:52 2014 From: srinivasan.raghavan at oracle.com (srinivasan raghavan) Date: Mon, 01 Sep 2014 22:03:52 +0530 Subject: [7u-dev] Request for Approval and Review: 8057008: [TEST_BUG] Test java/awt/Focus/SortingFPT/JDK8048887.java fails with compilation error Message-ID: <54049FF0.2060307@oracle.com> Hello , Please review a fix for the issue 8057008 [TEST_BUG] Test java/awt/Focus/SortingFPT/JDK8048887.java fails with compilation error Test bug fix https://bugs.openjdk.java.net/browse/JDK-8057008 The webrev is http://cr.openjdk.java.net/~kshefov/8031422/webrev.00/ Thanks -Srinivasan Raghavan From srinivasan.raghavan at oracle.com Mon Sep 1 16:42:36 2014 From: srinivasan.raghavan at oracle.com (srinivasan raghavan) Date: Mon, 01 Sep 2014 22:12:36 +0530 Subject: [7u-dev] Request for Approval and Review: 8057008: [TEST_BUG] Test java/awt/Focus/SortingFPT/JDK8048887.java fails with compilation error In-Reply-To: <54049FF0.2060307@oracle.com> References: <54049FF0.2060307@oracle.com> Message-ID: <5404A1FC.6090905@oracle.com> Hello , Apologies for giving wrong web-rev link i am updating with the new one The webrev is http://cr.openjdk.java.net/~kshefov/8057008/webrev.00/ -Thanks Srinivasan Raghavan On 9/1/2014 10:03 PM, srinivasan raghavan wrote: > Hello , > > Please review a fix for the issue > > 8057008 [TEST_BUG] Test java/awt/Focus/SortingFPT/JDK8048887.java > fails with compilation error > > Test bug fix > > https://bugs.openjdk.java.net/browse/JDK-8057008 > > The webrev is http://cr.openjdk.java.net/~kshefov/8031422/webrev.00/ > > Thanks > -Srinivasan Raghavan From sean.coffey at oracle.com Mon Sep 1 16:48:29 2014 From: sean.coffey at oracle.com (=?UTF-8?B?U2XDoW4gQ29mZmV5?=) Date: Mon, 01 Sep 2014 17:48:29 +0100 Subject: [7u-dev] Request for Approval and Review: 8057008: [TEST_BUG] Test java/awt/Focus/SortingFPT/JDK8048887.java fails with compilation error In-Reply-To: <5404A1FC.6090905@oracle.com> References: <54049FF0.2060307@oracle.com> <5404A1FC.6090905@oracle.com> Message-ID: <5404A35D.8070208@oracle.com> Looks good to me Srinivasan but you'll have to get an official 7u reviewer. You could append the bug ID to the @bug tags and perhaps add noreg-self to bug labels. No excuse for pushing testcases that fail to compile..that's a hint for the original 7u80 contributor ;) Approved subject to review. regards, Sean. On 01/09/14 17:42, srinivasan raghavan wrote: > Hello , > > Apologies for giving wrong web-rev link i am updating with the new one > > The webrev is http://cr.openjdk.java.net/~kshefov/8057008/webrev.00/ > > -Thanks > Srinivasan Raghavan > On 9/1/2014 10:03 PM, srinivasan raghavan wrote: >> Hello , >> >> Please review a fix for the issue >> >> 8057008 [TEST_BUG] Test java/awt/Focus/SortingFPT/JDK8048887.java >> fails with compilation error >> >> Test bug fix >> >> https://bugs.openjdk.java.net/browse/JDK-8057008 >> >> The webrev is http://cr.openjdk.java.net/~kshefov/8031422/webrev.00/ >> >> Thanks >> -Srinivasan Raghavan > From aleksej.efimov at oracle.com Tue Sep 2 09:58:09 2014 From: aleksej.efimov at oracle.com (aleksej.efimov at oracle.com) Date: Tue, 02 Sep 2014 09:58:09 +0000 Subject: hg: jdk7u/jdk7u-dev/jaxws: 8036981: JAXB not preserving formatting for xsd:any Mixed content Message-ID: <201409020958.s829w9TM008936@aojmv0008> Changeset: d924f235e859 Author: aefimov Date: 2014-09-02 13:49 +0400 URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/jaxws/rev/d924f235e859 8036981: JAXB not preserving formatting for xsd:any Mixed content Reviewed-by: lancea, mkos ! src/share/jaxws_classes/com/sun/xml/internal/bind/v2/runtime/ArrayBeanInfoImpl.java ! src/share/jaxws_classes/com/sun/xml/internal/bind/v2/runtime/ElementBeanInfoImpl.java ! src/share/jaxws_classes/com/sun/xml/internal/bind/v2/runtime/ValueListBeanInfoImpl.java ! src/share/jaxws_classes/com/sun/xml/internal/bind/v2/runtime/property/ArrayERProperty.java ! src/share/jaxws_classes/com/sun/xml/internal/bind/v2/runtime/property/SingleMapNodeProperty.java ! src/share/jaxws_classes/com/sun/xml/internal/bind/v2/runtime/reflect/Accessor.java ! src/share/jaxws_classes/com/sun/xml/internal/bind/v2/runtime/unmarshaller/DefaultValueLoaderDecorator.java ! src/share/jaxws_classes/com/sun/xml/internal/bind/v2/runtime/unmarshaller/Discarder.java ! src/share/jaxws_classes/com/sun/xml/internal/bind/v2/runtime/unmarshaller/DomLoader.java ! src/share/jaxws_classes/com/sun/xml/internal/bind/v2/runtime/unmarshaller/LeafPropertyLoader.java ! src/share/jaxws_classes/com/sun/xml/internal/bind/v2/runtime/unmarshaller/LeafPropertyXsiLoader.java ! src/share/jaxws_classes/com/sun/xml/internal/bind/v2/runtime/unmarshaller/Loader.java ! src/share/jaxws_classes/com/sun/xml/internal/bind/v2/runtime/unmarshaller/ProxyLoader.java ! src/share/jaxws_classes/com/sun/xml/internal/bind/v2/runtime/unmarshaller/SAXConnector.java ! src/share/jaxws_classes/com/sun/xml/internal/bind/v2/runtime/unmarshaller/Scope.java ! src/share/jaxws_classes/com/sun/xml/internal/bind/v2/runtime/unmarshaller/StructureLoader.java ! src/share/jaxws_classes/com/sun/xml/internal/bind/v2/runtime/unmarshaller/TextLoader.java ! src/share/jaxws_classes/com/sun/xml/internal/bind/v2/runtime/unmarshaller/UnmarshallingContext.java ! src/share/jaxws_classes/com/sun/xml/internal/bind/v2/runtime/unmarshaller/ValuePropertyLoader.java ! src/share/jaxws_classes/com/sun/xml/internal/bind/v2/runtime/unmarshaller/XsiNilLoader.java ! src/share/jaxws_classes/com/sun/xml/internal/bind/v2/runtime/unmarshaller/XsiTypeLoader.java From aleksej.efimov at oracle.com Tue Sep 2 09:58:21 2014 From: aleksej.efimov at oracle.com (aleksej.efimov at oracle.com) Date: Tue, 02 Sep 2014 09:58:21 +0000 Subject: hg: jdk7u/jdk7u-dev/jdk: 8036981: JAXB not preserving formatting for xsd:any Mixed content Message-ID: <201409020958.s829wLco008996@aojmv0008> Changeset: 87ff88fdd832 Author: aefimov Date: 2014-09-02 13:57 +0400 URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/jdk/rev/87ff88fdd832 8036981: JAXB not preserving formatting for xsd:any Mixed content Reviewed-by: lancea, mkos + test/javax/xml/bind/marshal/8036981/Good.java + test/javax/xml/bind/marshal/8036981/Main.java + test/javax/xml/bind/marshal/8036981/ObjectFactory.java + test/javax/xml/bind/marshal/8036981/Root.java + test/javax/xml/bind/marshal/8036981/Test.java + test/javax/xml/bind/marshal/8036981/test.xml From sean.coffey at oracle.com Tue Sep 2 11:36:00 2014 From: sean.coffey at oracle.com (=?ISO-8859-1?Q?Se=E1n_Coffey?=) Date: Tue, 02 Sep 2014 12:36:00 +0100 Subject: [7u-dev] Request for approval : 8012625 : Incorrect handling of HTTP/1.1 " Expect: 100-continue " in HttpURLConnection Message-ID: <5405ABA0.5000704@oracle.com> I'd like to backport the following fix from JDK 8 to 7u-dev. It applies cleanly and could help to address some performance issues for applications using the HTTP 100 status. Bug report : https://bugs.openjdk.java.net/browse/JDK-8012625 JDK 8 changeset : http://hg.openjdk.java.net/jdk8/jdk8/jdk/rev/74f91b7f4b66 I can't find the original review thread from 2013. Test results are good. regards, Sean. From mala.bankal at oracle.com Tue Sep 2 11:52:06 2014 From: mala.bankal at oracle.com (mala bankal) Date: Tue, 02 Sep 2014 17:22:06 +0530 Subject: [7u-dev] Request for approval: 8054817: File ccache only recognizes Linux and Solaris defaults Message-ID: <5405AF66.80901@oracle.com> HI, Request to approve integration of the changes for the backport of 8054817 to jdk7u-dev. webrev : http://cr.openjdk.java.net/~mbankal/8054817/webrev.00/ JDK9 changeset : http://hg.openjdk.java.net/jdk9/dev/jdk/rev/ec98f141c757 JDK8 changeset : http://hg.openjdk.java.net/jdk9/jdk9/jdk/rev/ec98f141c757 The backport has been derived from JDK8 changes. Thanks. rgds mala From sean.coffey at oracle.com Tue Sep 2 13:16:00 2014 From: sean.coffey at oracle.com (=?UTF-8?B?U2XDoW4gQ29mZmV5?=) Date: Tue, 02 Sep 2014 14:16:00 +0100 Subject: [7u-dev] Request for approval: 8054817: File ccache only recognizes Linux and Solaris defaults In-Reply-To: <5405AF66.80901@oracle.com> References: <5405AF66.80901@oracle.com> Message-ID: <5405C310.8090409@oracle.com> Review thread : http://mail.openjdk.java.net/pipermail/security-dev/2014-August/010967.html Approved. regards, Sean. On 02/09/2014 12:52, mala bankal wrote: > HI, > > Request to approve integration of the changes for the backport of > 8054817 to jdk7u-dev. > > webrev : > http://cr.openjdk.java.net/~mbankal/8054817/webrev.00/ > > JDK9 changeset : > http://hg.openjdk.java.net/jdk9/dev/jdk/rev/ec98f141c757 > JDK8 changeset : > http://hg.openjdk.java.net/jdk9/jdk9/jdk/rev/ec98f141c757 > > The backport has been derived from JDK8 changes. > Thanks. > > rgds > mala From rob.mckenna at oracle.com Tue Sep 2 14:52:21 2014 From: rob.mckenna at oracle.com (Rob McKenna) Date: Tue, 02 Sep 2014 15:52:21 +0100 Subject: [7u-dev] Request for approval : 8012625 : Incorrect handling of HTTP/1.1 " Expect: 100-continue " in HttpURLConnection In-Reply-To: <5405ABA0.5000704@oracle.com> References: <5405ABA0.5000704@oracle.com> Message-ID: <5405D9A5.4030503@oracle.com> Approved. -Rob On 02/09/14 12:36, Se?n Coffey wrote: > I'd like to backport the following fix from JDK 8 to 7u-dev. It > applies cleanly > and could help to address some performance issues for applications using > the HTTP 100 status. > > Bug report : https://bugs.openjdk.java.net/browse/JDK-8012625 > JDK 8 changeset : > http://hg.openjdk.java.net/jdk8/jdk8/jdk/rev/74f91b7f4b66 > I can't find the original review thread from 2013. > > Test results are good. > > regards, > Sean. From sean.coffey at oracle.com Tue Sep 2 14:59:44 2014 From: sean.coffey at oracle.com (sean.coffey at oracle.com) Date: Tue, 02 Sep 2014 14:59:44 +0000 Subject: hg: jdk7u/jdk7u-dev/jdk: 2 new changesets Message-ID: <201409021459.s82ExiN1027620@aojmv0008> Changeset: 866ce3916733 Author: michaelm Date: 2013-05-16 17:28 +0100 URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/jdk/rev/866ce3916733 8012625: Incorrect handling of HTTP/1.1 " Expect: 100-continue " in HttpURLConnection Reviewed-by: alanb, chegar ! src/share/classes/sun/net/www/protocol/http/HttpURLConnection.java + test/sun/net/www/protocol/http/B8012625.java Changeset: 43d918d51967 Author: coffeys Date: 2014-09-02 15:56 +0100 URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/jdk/rev/43d918d51967 Merge From sean.coffey at oracle.com Tue Sep 2 21:30:25 2014 From: sean.coffey at oracle.com (=?ISO-8859-1?Q?Se=E1n_Coffey?=) Date: Tue, 02 Sep 2014 22:30:25 +0100 Subject: [7u-dev] Request for approval : 8054019 Keytool Error publicKey's is not X.509, but X509 Message-ID: <540636F1.50309@oracle.com> Looking to fix this in 7u-dev. Not applicable for JDK 8u or later. review thread : http://mail.openjdk.java.net/pipermail/security-dev/2014-September/011145.html bug report : https://bugs.openjdk.java.net/browse/JDK-8054019 regards, Sean. From rob.mckenna at oracle.com Wed Sep 3 00:47:15 2014 From: rob.mckenna at oracle.com (Rob McKenna) Date: Wed, 03 Sep 2014 01:47:15 +0100 Subject: [7u-dev] Request for approval : 8054019 Keytool Error publicKey's is not X.509, but X509 In-Reply-To: <540636F1.50309@oracle.com> References: <540636F1.50309@oracle.com> Message-ID: <54066513.4080608@oracle.com> Approved. -Rob On 02/09/14 22:30, Se?n Coffey wrote: > Looking to fix this in 7u-dev. Not applicable for JDK 8u or later. > > review thread : > http://mail.openjdk.java.net/pipermail/security-dev/2014-September/011145.html > bug report : https://bugs.openjdk.java.net/browse/JDK-8054019 > > regards, > Sean. From sean.coffey at oracle.com Wed Sep 3 07:53:27 2014 From: sean.coffey at oracle.com (sean.coffey at oracle.com) Date: Wed, 03 Sep 2014 07:53:27 +0000 Subject: hg: jdk7u/jdk7u-dev/jdk: 8054019: Keytool Error publicKey's is not X.509, but X509 Message-ID: <201409030753.s837rRHC012621@aojmv0008> Changeset: eec956f1670c Author: coffeys Date: 2014-09-02 22:25 +0100 URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/jdk/rev/eec956f1670c 8054019: Keytool Error publicKey's is not X.509, but X509 Reviewed-by: mullan, xuelei ! src/share/classes/sun/security/x509/CertAndKeyGen.java From sean.coffey at oracle.com Wed Sep 3 09:51:54 2014 From: sean.coffey at oracle.com (=?ISO-8859-1?Q?Se=E1n_Coffey?=) Date: Wed, 03 Sep 2014 10:51:54 +0100 Subject: [7u-dev] Request for approval : 7087021: TEST: com/sun/crypto/provider/Mac/MacClone.java failed on Solaris sparc 5.10 Message-ID: <5406E4BA.6080607@oracle.com> Would like to backport this fix from 8 to 7u80. Applies cleanly and helps improve the test pass rate. bug : https://bugs.openjdk.java.net/browse/JDK-7087021 jdk8 changeset : http://hg.openjdk.java.net/jdk8/jdk8/jdk/rev/d78c6095dc98 regards, Sean. From sergey.gabdurakhmanov at oracle.com Wed Sep 3 10:36:58 2014 From: sergey.gabdurakhmanov at oracle.com (Sergey Gabdurakhmanov) Date: Wed, 03 Sep 2014 14:36:58 +0400 Subject: [7u-dev] Request for approval 8022229: Intermittent test failures in sun/tools/jstatd Message-ID: <5406EF4A.7000102@oracle.com> Hello, Could you please approve the back-port of the fix for 8022229 to jdk7u-dev? Files from test/lib/testlibrary/jdk/testlibrary/ have been backported in bug 8036786 "Update jdk7 testlibrary to match jdk8". The bug: https://bugs.openjdk.java.net/browse/JDK-8022229 The webrev: http://cr.openjdk.java.net/~sgabdura/8022229/webrev.00/ The review thread: http://mail.openjdk.java.net/pipermail/hotspot-dev/2013-October/011444.html The jdk8 changeset: http://hg.openjdk.java.net/jdk8/jdk8/jdk/rev/b04b124418d8 BR, Sergey From sean.coffey at oracle.com Wed Sep 3 10:39:21 2014 From: sean.coffey at oracle.com (=?UTF-8?B?U2XDoW4gQ29mZmV5?=) Date: Wed, 03 Sep 2014 11:39:21 +0100 Subject: [7u-dev] Request for approval 8022229: Intermittent test failures in sun/tools/jstatd In-Reply-To: <5406EF4A.7000102@oracle.com> References: <5406EF4A.7000102@oracle.com> Message-ID: <5406EFD9.3010509@oracle.com> Good to have these tests fixed up. Approved. regards, Sean. On 03/09/14 11:36, Sergey Gabdurakhmanov wrote: > Hello, > > Could you please approve the back-port of the fix for 8022229 to > jdk7u-dev? > Files from test/lib/testlibrary/jdk/testlibrary/ have been backported > in bug 8036786 "Update jdk7 testlibrary to match jdk8". > > The bug: > https://bugs.openjdk.java.net/browse/JDK-8022229 > The webrev: > http://cr.openjdk.java.net/~sgabdura/8022229/webrev.00/ > The review thread: > http://mail.openjdk.java.net/pipermail/hotspot-dev/2013-October/011444.html > > The jdk8 changeset: > http://hg.openjdk.java.net/jdk8/jdk8/jdk/rev/b04b124418d8 > > BR, > Sergey From rob.mckenna at oracle.com Wed Sep 3 11:03:11 2014 From: rob.mckenna at oracle.com (Rob McKenna) Date: Wed, 03 Sep 2014 12:03:11 +0100 Subject: [7u-dev] Request for approval : 7087021: TEST: com/sun/crypto/provider/Mac/MacClone.java failed on Solaris sparc 5.10 In-Reply-To: <5406E4BA.6080607@oracle.com> References: <5406E4BA.6080607@oracle.com> Message-ID: <5406F56F.1050301@oracle.com> Approved. -Rob On 03/09/14 10:51, Se?n Coffey wrote: > Would like to backport this fix from 8 to 7u80. Applies cleanly and > helps improve the test pass rate. > > bug : https://bugs.openjdk.java.net/browse/JDK-7087021 > jdk8 changeset : > http://hg.openjdk.java.net/jdk8/jdk8/jdk/rev/d78c6095dc98 > > regards, > Sean. From sean.coffey at oracle.com Wed Sep 3 11:04:18 2014 From: sean.coffey at oracle.com (sean.coffey at oracle.com) Date: Wed, 03 Sep 2014 11:04:18 +0000 Subject: hg: jdk7u/jdk7u-dev/jdk: 2 new changesets Message-ID: <201409031104.s83B4J97016062@aojmv0008> Changeset: b6d1b9081cb3 Author: weijun Date: 2014-08-11 22:09 +0800 URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/jdk/rev/b6d1b9081cb3 8054817: File ccache only recognizes Linux and Solaris defaults Reviewed-by: mullan ! src/share/classes/sun/security/krb5/internal/ccache/FileCredentialsCache.java + test/sun/security/krb5/ccache/DefaultFile.java Changeset: e3161d87f66b Author: vinnie Date: 2012-05-02 14:50 +0100 URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/jdk/rev/e3161d87f66b 7087021: TEST: com/sun/crypto/provider/Mac/MacClone.java failed on Solaris sparc 5.10 Reviewed-by: mullan ! test/com/sun/crypto/provider/Mac/MacClone.java From sergey.gabdurakhmanov at oracle.com Wed Sep 3 12:06:31 2014 From: sergey.gabdurakhmanov at oracle.com (Sergey Gabdurakhmanov) Date: Wed, 03 Sep 2014 16:06:31 +0400 Subject: [7u-dev] Request for approval 8029190: VM_Version::determine_features() asserts on Fujitsu Sparc64 CPUs Message-ID: <54070447.4070103@oracle.com> Hello, Could you please approve the back-port of the fix for 8029190 to jdk7u-dev? The bug: https://bugs.openjdk.java.net/browse/JDK-8029190 The webrev: http://cr.openjdk.java.net/~sgabdura/8029190/webrev.00/ The review thread: http://mail.openjdk.java.net/pipermail/hotspot-dev/2013-December/011892.html The jdk8 changeset: http://hg.openjdk.java.net/jdk8/jdk8/hotspot/rev/eae426d683f6 BR, Sergey From sean.coffey at oracle.com Wed Sep 3 12:18:20 2014 From: sean.coffey at oracle.com (=?UTF-8?B?U2XDoW4gQ29mZmV5?=) Date: Wed, 03 Sep 2014 13:18:20 +0100 Subject: [7u-dev] Request for approval 8029190: VM_Version::determine_features() asserts on Fujitsu Sparc64 CPUs In-Reply-To: <54070447.4070103@oracle.com> References: <54070447.4070103@oracle.com> Message-ID: <5407070C.8060101@oracle.com> Approved. Looks like the bug report needs a noreg- label. regards, Sean. On 03/09/14 13:06, Sergey Gabdurakhmanov wrote: > Hello, > > Could you please approve the back-port of the fix for 8029190 to > jdk7u-dev? > > The bug: > https://bugs.openjdk.java.net/browse/JDK-8029190 > The webrev: > http://cr.openjdk.java.net/~sgabdura/8029190/webrev.00/ > The review thread: > http://mail.openjdk.java.net/pipermail/hotspot-dev/2013-December/011892.html > > The jdk8 changeset: > http://hg.openjdk.java.net/jdk8/jdk8/hotspot/rev/eae426d683f6 > > BR, > Sergey From coleen.phillimore at oracle.com Wed Sep 3 18:00:14 2014 From: coleen.phillimore at oracle.com (coleen.phillimore at oracle.com) Date: Wed, 03 Sep 2014 18:00:14 +0000 Subject: hg: jdk7u/jdk7u-dev/hotspot: 8046233: VerifyError on backward branch Message-ID: <201409031800.s83I0EbW026782@aojmv0008> Changeset: aad3fc05d972 Author: hseigel Date: 2014-08-29 12:40 -0400 URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/hotspot/rev/aad3fc05d972 8046233: VerifyError on backward branch Summary: Check that both blocks have same uninitialized object Reviewed-by: acorn, coleenp ! src/share/vm/classfile/stackMapFrame.cpp ! src/share/vm/classfile/stackMapFrame.hpp ! src/share/vm/classfile/stackMapTable.cpp From sean.coffey at oracle.com Thu Sep 4 08:12:07 2014 From: sean.coffey at oracle.com (=?ISO-8859-1?Q?Se=E1n_Coffey?=) Date: Thu, 04 Sep 2014 09:12:07 +0100 Subject: hg: jdk7u/jdk7u-dev/hotspot: 8046233: VerifyError on backward branch In-Reply-To: <201409031800.s83I0EbW026782@aojmv0008> References: <201409031800.s83I0EbW026782@aojmv0008> Message-ID: <54081ED7.2030906@oracle.com> Coleen, Harold, I didn't see an approval request for this one. Could you log one for records purposes ? http://openjdk.java.net/projects/jdk7u/groundrules.html regards, Sean. On 03/09/2014 19:00, coleen.phillimore at oracle.com wrote: > Changeset: aad3fc05d972 > Author: hseigel > Date: 2014-08-29 12:40 -0400 > URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/hotspot/rev/aad3fc05d972 > > 8046233: VerifyError on backward branch > Summary: Check that both blocks have same uninitialized object > Reviewed-by: acorn, coleenp > > ! src/share/vm/classfile/stackMapFrame.cpp > ! src/share/vm/classfile/stackMapFrame.hpp > ! src/share/vm/classfile/stackMapTable.cpp > From sergey.gabdurakhmanov at oracle.com Thu Sep 4 08:40:45 2014 From: sergey.gabdurakhmanov at oracle.com (sergey.gabdurakhmanov at oracle.com) Date: Thu, 04 Sep 2014 08:40:45 +0000 Subject: hg: jdk7u/jdk7u-dev/jdk: 8022229: Intermittent test failures in sun/tools/jstatd Message-ID: <201409040840.s848ekbA016389@aojmv0008> Changeset: 86bc5adbf7b7 Author: sgabdura Date: 2014-09-04 08:34 +0200 URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/jdk/rev/86bc5adbf7b7 8022229: Intermittent test failures in sun/tools/jstatd Reviewed-by: ykantser, sla, egahlin, jbachorik, allwin + test/sun/tools/jstatd/JstatGCUtilParser.java + test/sun/tools/jstatd/JstatdTest.java + test/sun/tools/jstatd/TestJstatdDefaults.java + test/sun/tools/jstatd/TestJstatdExternalRegistry.java + test/sun/tools/jstatd/TestJstatdPort.java + test/sun/tools/jstatd/TestJstatdPortAndServer.java + test/sun/tools/jstatd/TestJstatdServer.java + test/sun/tools/jstatd/TestJstatdUsage.java - test/sun/tools/jstatd/jpsOutput1.awk - test/sun/tools/jstatd/jstatGcutilOutput1.awk - test/sun/tools/jstatd/jstatdDefaults.sh - test/sun/tools/jstatd/jstatdExternalRegistry.sh - test/sun/tools/jstatd/jstatdPort.sh - test/sun/tools/jstatd/jstatdServerName.sh - test/sun/tools/jstatd/jstatdUsage1.sh - test/sun/tools/jstatd/usage.out From sergey.gabdurakhmanov at oracle.com Thu Sep 4 08:42:27 2014 From: sergey.gabdurakhmanov at oracle.com (sergey.gabdurakhmanov at oracle.com) Date: Thu, 04 Sep 2014 08:42:27 +0000 Subject: hg: jdk7u/jdk7u-dev/hotspot: 8029190: VM_Version::determine_features() asserts on Fujitsu Sparc64 CPUs Message-ID: <201409040842.s848gRX2016811@aojmv0008> Changeset: 6547c22e85df Author: sgabdura Date: 2014-09-04 09:04 +0200 URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/hotspot/rev/6547c22e85df 8029190: VM_Version::determine_features() asserts on Fujitsu Sparc64 CPUs Summary: fix code to allow testing on Fujitsu Sparc64 CPUs Reviewed-by: kvn ! src/cpu/sparc/vm/vm_version_sparc.cpp ! src/cpu/sparc/vm/vm_version_sparc.hpp ! src/share/vm/runtime/arguments.cpp From yekaterina.kantserova at oracle.com Thu Sep 4 06:20:01 2014 From: yekaterina.kantserova at oracle.com (Yekaterina Kantserova) Date: Thu, 04 Sep 2014 08:20:01 +0200 Subject: [7u-dev] Request for approval 8022229: Intermittent test failures in sun/tools/jstatd In-Reply-To: <5406EF4A.7000102@oracle.com> References: <5406EF4A.7000102@oracle.com> Message-ID: <54080491.3080406@oracle.com> Looks good! Best Regards, Katja On 09/03/2014 12:36 PM, Sergey Gabdurakhmanov wrote: > Hello, > > Could you please approve the back-port of the fix for 8022229 to > jdk7u-dev? > Files from test/lib/testlibrary/jdk/testlibrary/ have been backported > in bug 8036786 "Update jdk7 testlibrary to match jdk8". > > The bug: > https://bugs.openjdk.java.net/browse/JDK-8022229 > The webrev: > http://cr.openjdk.java.net/~sgabdura/8022229/webrev.00/ > The review thread: > http://mail.openjdk.java.net/pipermail/hotspot-dev/2013-October/011444.html > > The jdk8 changeset: > http://hg.openjdk.java.net/jdk8/jdk8/jdk/rev/b04b124418d8 > > BR, > Sergey From harold.seigel at oracle.com Thu Sep 4 13:05:00 2014 From: harold.seigel at oracle.com (harold seigel) Date: Thu, 04 Sep 2014 09:05:00 -0400 Subject: hg: jdk7u/jdk7u-dev/hotspot: 8046233: VerifyError on backward branch In-Reply-To: <54081ED7.2030906@oracle.com> References: <201409031800.s83I0EbW026782@aojmv0008> <54081ED7.2030906@oracle.com> Message-ID: <5408637C.7020101@oracle.com> Hi Sean, This was my fault. I thought only a code review was needed. I'll log the request. Harold On 9/4/2014 4:12 AM, Se?n Coffey wrote: > Coleen, Harold, > > I didn't see an approval request for this one. Could you log one for > records purposes ? > http://openjdk.java.net/projects/jdk7u/groundrules.html > > regards, > Sean. > > On 03/09/2014 19:00, coleen.phillimore at oracle.com wrote: >> Changeset: aad3fc05d972 >> Author: hseigel >> Date: 2014-08-29 12:40 -0400 >> URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/hotspot/rev/aad3fc05d972 >> >> 8046233: VerifyError on backward branch >> Summary: Check that both blocks have same uninitialized object >> Reviewed-by: acorn, coleenp >> >> ! src/share/vm/classfile/stackMapFrame.cpp >> ! src/share/vm/classfile/stackMapFrame.hpp >> ! src/share/vm/classfile/stackMapTable.cpp >> > From harold.seigel at oracle.com Thu Sep 4 13:14:56 2014 From: harold.seigel at oracle.com (harold seigel) Date: Thu, 04 Sep 2014 09:14:56 -0400 Subject: [7u-dev] Request for approval 8046233: VerifyError on backward branch Message-ID: <540865D0.4010103@oracle.com> Hi, Please approve this small fix for bug 8046233 for JDK7u-dev. Section 4.10.2.4 of the JVM-7 Spec says: A valid instruction sequence must not have an uninitialized object on the operand stack or in a local variable at the target of a backwards branch if the special type of the uninitialized object is merged with a special type other than itself ... Currently, the split verifier does not allow any backward branches into a block containing an uninitialized object. This fix allows such backward branches if the originating block contains an uninitialized object of the same special type. Bug: https://bugs.openjdk.java.net/browse/JDK-8046233 Open webrev: http://cr.openjdk.java.net/~hseigel/bug_8046233_7/ Review thread: https://www.mail-archive.com/jdk7u-dev at openjdk.java.net/msg09346.html The fix was tested with the JCK lang, vm, and api/java_lang tests, the JTREG hotspot tests, and tests that reproduced the problem. With this fix, the tests that reproduce the problem get the same result with both the old and the split verifier. This bug has already been fixed in JDK-8u and JDK-9. But, a straight back-port to JDK-7u could not be done because the JVM-8 Spec differs from the JVM-7 Spec for this case. Thanks, Harold From sean.coffey at oracle.com Thu Sep 4 13:19:36 2014 From: sean.coffey at oracle.com (=?UTF-8?B?U2XDoW4gQ29mZmV5?=) Date: Thu, 04 Sep 2014 14:19:36 +0100 Subject: [7u-dev] Request for approval 8046233: VerifyError on backward branch In-Reply-To: <540865D0.4010103@oracle.com> References: <540865D0.4010103@oracle.com> Message-ID: <540866E8.6080603@oracle.com> Thanks for following up Harold. Good to have the detail also. Approved. regards, Sean. On 04/09/2014 14:14, harold seigel wrote: > Hi, > > Please approve this small fix for bug 8046233 for JDK7u-dev. > > Section 4.10.2.4 > > of the JVM-7 Spec says: > > A valid instruction sequence must not have an uninitialized object > on the operand stack or in a local variable at the target of a > backwards branch if the special type of the uninitialized object is > merged with a special type other than itself ... > > Currently, the split verifier does not allow any backward branches > into a block containing an uninitialized object. This fix allows such > backward branches if the originating block contains an uninitialized > object of the same special type. > > Bug: https://bugs.openjdk.java.net/browse/JDK-8046233 > Open webrev: http://cr.openjdk.java.net/~hseigel/bug_8046233_7/ > Review thread: > https://www.mail-archive.com/jdk7u-dev at openjdk.java.net/msg09346.html > > The fix was tested with the JCK lang, vm, and api/java_lang tests, the > JTREG hotspot tests, and tests that reproduced the problem. With this > fix, the tests that reproduce the problem get the same result with > both the old and the split verifier. > > This bug has already been fixed in JDK-8u and JDK-9. But, a straight > back-port to JDK-7u could not be done because the JVM-8 Spec differs > from the JVM-7 Spec for this case. > > Thanks, Harold From alexey.ivanov at oracle.com Thu Sep 4 14:22:28 2014 From: alexey.ivanov at oracle.com (Alexey Ivanov) Date: Thu, 04 Sep 2014 18:22:28 +0400 Subject: [7u80] Request for approval for JDK-8056211: JCK test api/java_awt/Event/InputMethodEvent/serial fails Message-ID: <540875A4.8080600@oracle.com> Hello, Please approve the backport of the fix to jdk7u-dev. The changeset applies cleanly after path reshuffling. JBS bug: https://bugs.openjdk.java.net/browse/JDK-8056211 Webrev: http://cr.openjdk.java.net/~aivanov/8056211/jdk9/webrev.00/ Review: http://mail.openjdk.java.net/pipermail/awt-dev/2014-August/008408.html continued at http://mail.openjdk.java.net/pipermail/awt-dev/2014-September/008435.html Changeset: http://hg.openjdk.java.net/jdk9/client/jdk/rev/d2c998b7e553 Thank you, Alexey. From rob.mckenna at oracle.com Thu Sep 4 14:43:29 2014 From: rob.mckenna at oracle.com (Rob McKenna) Date: Thu, 04 Sep 2014 15:43:29 +0100 Subject: [7u80] Request for approval for JDK-8056211: JCK test api/java_awt/Event/InputMethodEvent/serial fails In-Reply-To: <540875A4.8080600@oracle.com> References: <540875A4.8080600@oracle.com> Message-ID: <54087A91.6050005@oracle.com> Might want to add noreg-jck to the bug. Approved. -Rob On 04/09/14 15:22, Alexey Ivanov wrote: > Hello, > > Please approve the backport of the fix to jdk7u-dev. > The changeset applies cleanly after path reshuffling. > > JBS bug: https://bugs.openjdk.java.net/browse/JDK-8056211 > Webrev: http://cr.openjdk.java.net/~aivanov/8056211/jdk9/webrev.00/ > Review: > http://mail.openjdk.java.net/pipermail/awt-dev/2014-August/008408.html > continued at > http://mail.openjdk.java.net/pipermail/awt-dev/2014-September/008435.html > Changeset: http://hg.openjdk.java.net/jdk9/client/jdk/rev/d2c998b7e553 > > > Thank you, > Alexey. From alexey.ivanov at oracle.com Thu Sep 4 14:47:31 2014 From: alexey.ivanov at oracle.com (Alexey Ivanov) Date: Thu, 04 Sep 2014 18:47:31 +0400 Subject: [7u80] Request for approval for JDK-8056211: JCK test api/java_awt/Event/InputMethodEvent/serial fails In-Reply-To: <54087A91.6050005@oracle.com> References: <540875A4.8080600@oracle.com> <54087A91.6050005@oracle.com> Message-ID: <54087B83.6000807@oracle.com> Sure! Thank you, Rob. Regards, Alexey. On 04.09.2014 18:43, Rob McKenna wrote: > Might want to add noreg-jck to the bug. > > Approved. > > -Rob > > On 04/09/14 15:22, Alexey Ivanov wrote: >> Hello, >> >> Please approve the backport of the fix to jdk7u-dev. >> The changeset applies cleanly after path reshuffling. >> >> JBS bug: https://bugs.openjdk.java.net/browse/JDK-8056211 >> Webrev: http://cr.openjdk.java.net/~aivanov/8056211/jdk9/webrev.00/ >> Review: >> http://mail.openjdk.java.net/pipermail/awt-dev/2014-August/008408.html >> continued at >> http://mail.openjdk.java.net/pipermail/awt-dev/2014-September/008435.html >> Changeset: http://hg.openjdk.java.net/jdk9/client/jdk/rev/d2c998b7e553 >> >> >> Thank you, >> Alexey. > From anton.litvinov at oracle.com Thu Sep 4 15:18:06 2014 From: anton.litvinov at oracle.com (anton.litvinov at oracle.com) Date: Thu, 04 Sep 2014 15:18:06 +0000 Subject: hg: jdk7u/jdk7u-dev/jdk: 8056211: api/java_awt/Event/InputMethodEvent/serial/index.html#Input[serial2002] failure Message-ID: <201409041518.s84FI6K3026370@aojmv0008> Changeset: 38181109e3d1 Author: aivanov Date: 2014-09-04 19:16 +0400 URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/jdk/rev/38181109e3d1 8056211: api/java_awt/Event/InputMethodEvent/serial/index.html#Input[serial2002] failure Reviewed-by: pchelko, alexsch ! src/share/classes/java/awt/event/InputMethodEvent.java From sergey.gabdurakhmanov at oracle.com Fri Sep 5 08:20:19 2014 From: sergey.gabdurakhmanov at oracle.com (Sergey Gabdurakhmanov) Date: Fri, 05 Sep 2014 12:20:19 +0400 Subject: [7u-dev] Request for approval 8025644: java/util/stream/test/org/openjdk/tests/java/util/stream/ToArrayOpTest.java fails with TestData$OfRef): failure java.lang.AssertionError: expected [true] but found [false] Message-ID: <54097243.4000103@oracle.com> Hello, Could you please approve the back-port of the fix for 8025644 to jdk7u-dev? The bug: https://bugs.openjdk.java.net/browse/JDK-8025644 The webrev: http://cr.openjdk.java.net/~sgabdura/8025644/webrev.00/ The review thread: http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/2014-January/013247.html The jdk8 changeset: http://hg.openjdk.java.net/jdk8u/jdk8u/hotspot/rev/f176b09780b0 BR, Sergey From sean.coffey at oracle.com Fri Sep 5 08:32:54 2014 From: sean.coffey at oracle.com (=?UTF-8?B?U2XDoW4gQ29mZmV5?=) Date: Fri, 05 Sep 2014 09:32:54 +0100 Subject: [7u-dev] Request for approval 8025644: java/util/stream/test/org/openjdk/tests/java/util/stream/ToArrayOpTest.java fails with TestData$OfRef): failure java.lang.AssertionError: expected [true] but found [false] In-Reply-To: <54097243.4000103@oracle.com> References: <54097243.4000103@oracle.com> Message-ID: <54097536.6040508@oracle.com> Approved. regards, Sean. On 05/09/2014 09:20, Sergey Gabdurakhmanov wrote: > Hello, > > Could you please approve the back-port of the fix for 8025644 to > jdk7u-dev? > > The bug: > https://bugs.openjdk.java.net/browse/JDK-8025644 > The webrev: > http://cr.openjdk.java.net/~sgabdura/8025644/webrev.00/ > The review thread: > http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/2014-January/013247.html > > The jdk8 changeset: > http://hg.openjdk.java.net/jdk8u/jdk8u/hotspot/rev/f176b09780b0 > > BR, > Sergey From Alan.Bateman at oracle.com Fri Sep 5 09:19:19 2014 From: Alan.Bateman at oracle.com (Alan Bateman) Date: Fri, 05 Sep 2014 10:19:19 +0100 Subject: [7u-dev] Request for approval 8025644: java/util/stream/test/org/openjdk/tests/java/util/stream/ToArrayOpTest.java fails with TestData$OfRef): failure java.lang.AssertionError: expected [true] but found [false] In-Reply-To: <54097243.4000103@oracle.com> References: <54097243.4000103@oracle.com> Message-ID: <54098017.9040007@oracle.com> On 05/09/2014 09:20, Sergey Gabdurakhmanov wrote: > Hello, > > Could you please approve the back-port of the fix for 8025644 to > jdk7u-dev? I wonder why the bug synopsis wasn't changed to something that reflects the actual issue. java.util.stream is new in Java SE 8 so a bit strange to have a fix with this in the synopsis going into JDK 7. -Alan. From sergey.gabdurakhmanov at oracle.com Fri Sep 5 09:23:56 2014 From: sergey.gabdurakhmanov at oracle.com (Sergey Gabdurakhmanov) Date: Fri, 05 Sep 2014 13:23:56 +0400 Subject: [7u-dev] Request for approval 8025644: java/util/stream/test/org/openjdk/tests/java/util/stream/ToArrayOpTest.java fails with TestData$OfRef): failure java.lang.AssertionError: expected [true] but found [false] In-Reply-To: <54098017.9040007@oracle.com> References: <54097243.4000103@oracle.com> <54098017.9040007@oracle.com> Message-ID: <5409812C.9010807@oracle.com> Alan, Here and in commit message of the backport I should provide bug number of the original bug. And I can not change synopsis of 8025644. BR, Sergey On 05.09.2014 13:19, Alan Bateman wrote: > On 05/09/2014 09:20, Sergey Gabdurakhmanov wrote: >> Hello, >> >> Could you please approve the back-port of the fix for 8025644 to >> jdk7u-dev? > I wonder why the bug synopsis wasn't changed to something that > reflects the actual issue. java.util.stream is new in Java SE 8 so a > bit strange to have a fix with this in the synopsis going into JDK 7. > > -Alan. From sergey.gabdurakhmanov at oracle.com Sun Sep 7 10:45:18 2014 From: sergey.gabdurakhmanov at oracle.com (sergey.gabdurakhmanov at oracle.com) Date: Sun, 07 Sep 2014 10:45:18 +0000 Subject: hg: jdk7u/jdk7u-dev/hotspot: 8025644: java/util/stream/test/org/openjdk/tests/java/util/stream/ToArrayOpTest.java fails with TestData): failure java.lang.AssertionError: expected [true] but found [false] Message-ID: <201409071045.s87AjJUs019330@aojmv0008> Changeset: c6b5db6389c9 Author: sgabdura Date: 2014-09-05 10:19 +0200 URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/hotspot/rev/c6b5db6389c9 8025644: java/util/stream/test/org/openjdk/tests/java/util/stream/ToArrayOpTest.java fails with TestData): failure java.lang.AssertionError: expected [true] but found [false] Summary: In GraphKit::gen_checkcast() provide only exact superklass to GraphKit::maybe_cast_profiled_receiver() Reviewed-by: iveresov ! src/share/vm/opto/graphKit.cpp From alexandr.scherbatiy at oracle.com Mon Sep 8 10:00:37 2014 From: alexandr.scherbatiy at oracle.com (Alexander Scherbatiy) Date: Mon, 08 Sep 2014 14:00:37 +0400 Subject: [7u-dev] Request for approval: 8057184 JCK8's api/javax_swing/JDesktopPane/descriptions.html#getset failed with GTKLookAndFeel on Linux and Solaris run v.s. JDK8+ Message-ID: <540D7E45.10707@oracle.com> Hello, Could you approve the direct backport of the fix to JDK 7u-dev. The bug: https://bugs.openjdk.java.net/browse/JDK-8057184 The webrev: http://cr.openjdk.java.net/~alexsch/8057184/webrev.01/ The review thread: http://mail.openjdk.java.net/pipermail/swing-dev/2014-September/003887.html The JDK 8 changeset: http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/ba77067a033a Thanks, Alexandr. From sean.coffey at oracle.com Mon Sep 8 10:35:34 2014 From: sean.coffey at oracle.com (=?ISO-8859-1?Q?Se=E1n_Coffey?=) Date: Mon, 08 Sep 2014 11:35:34 +0100 Subject: [7u-dev] Request for approval: 8057184 JCK8's api/javax_swing/JDesktopPane/descriptions.html#getset failed with GTKLookAndFeel on Linux and Solaris run v.s. JDK8+ In-Reply-To: <540D7E45.10707@oracle.com> References: <540D7E45.10707@oracle.com> Message-ID: <540D8676.8010204@oracle.com> Approved. regards, Sean. On 08/09/14 11:00, Alexander Scherbatiy wrote: > > Hello, > > Could you approve the direct backport of the fix to JDK 7u-dev. > > The bug: https://bugs.openjdk.java.net/browse/JDK-8057184 > The webrev: http://cr.openjdk.java.net/~alexsch/8057184/webrev.01/ > The review thread: > http://mail.openjdk.java.net/pipermail/swing-dev/2014-September/003887.html > The JDK 8 changeset: > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/ba77067a033a > > Thanks, > Alexandr. > From alexandr.scherbatiy at oracle.com Mon Sep 8 11:40:33 2014 From: alexandr.scherbatiy at oracle.com (alexandr.scherbatiy at oracle.com) Date: Mon, 08 Sep 2014 11:40:33 +0000 Subject: hg: jdk7u/jdk7u-dev/jdk: 8057184: JCK8's api/javax_swing/JDesktopPane/descriptions.html#getset failed with GTKLookAndFeel on Linux and Solaris Message-ID: <201409081140.s88BedFf008870@aojmv0008> Changeset: fdeb6abac0ea Author: alexsch Date: 2014-09-08 15:37 +0400 URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/jdk/rev/fdeb6abac0ea 8057184: JCK8's api/javax_swing/JDesktopPane/descriptions.html#getset failed with GTKLookAndFeel on Linux and Solaris Reviewed-by: ant, azvegint ! src/share/classes/javax/swing/JDesktopPane.java From sean.coffey at oracle.com Tue Sep 9 17:23:19 2014 From: sean.coffey at oracle.com (=?ISO-8859-1?Q?Se=E1n_Coffey?=) Date: Tue, 09 Sep 2014 18:23:19 +0100 Subject: [7u-dev] Request for approval : 8044725 :Bug in zlib 1.2.5 prevents inflation of some gzipped files (zlib 1.2.8 port) Message-ID: <540F3787.80804@oracle.com> I'd like to backport the zlib v1.2.8 libraries to JDK 7u. bug report : https://bugs.openjdk.java.net/browse/JDK-8044725 webrev : http://cr.openjdk.java.net/~coffeys/webrev.8044725.7u/webrev/ review thread : http://mail.openjdk.java.net/pipermail/core-libs-dev/2014-September/028482.html regards, Sean. From rob.mckenna at oracle.com Tue Sep 9 17:41:54 2014 From: rob.mckenna at oracle.com (Rob McKenna) Date: Tue, 09 Sep 2014 18:41:54 +0100 Subject: [7u-dev] Request for approval : 8044725 :Bug in zlib 1.2.5 prevents inflation of some gzipped files (zlib 1.2.8 port) In-Reply-To: <540F3787.80804@oracle.com> References: <540F3787.80804@oracle.com> Message-ID: <540F3BE2.4000708@oracle.com> Approved. -Rob On 09/09/14 18:23, Se?n Coffey wrote: > I'd like to backport the zlib v1.2.8 libraries to JDK 7u. > > bug report : https://bugs.openjdk.java.net/browse/JDK-8044725 > webrev : http://cr.openjdk.java.net/~coffeys/webrev.8044725.7u/webrev/ > review thread : > http://mail.openjdk.java.net/pipermail/core-libs-dev/2014-September/028482.html > > regards, > Sean. From sean.coffey at oracle.com Tue Sep 9 17:46:21 2014 From: sean.coffey at oracle.com (sean.coffey at oracle.com) Date: Tue, 09 Sep 2014 17:46:21 +0000 Subject: hg: jdk7u/jdk7u-dev/jdk: 8044725: Bug in zlib 1.2.5 prevents inflation of some gzipped files (zlib 1.2.8 port) Message-ID: <201409091746.s89HkMDN022105@aojmv0008> Changeset: 49b0ce28e2ca Author: coffeys Date: 2014-09-09 18:43 +0100 URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/jdk/rev/49b0ce28e2ca 8044725: Bug in zlib 1.2.5 prevents inflation of some gzipped files (zlib 1.2.8 port) Reviewed-by: sherman ! make/common/Defs.gmk ! make/common/Program.gmk ! make/java/zip/FILES_c.gmk ! make/sun/splashscreen/FILES_c.gmk - src/share/native/java/util/zip/zlib-1.2.3/ChangeLog - src/share/native/java/util/zip/zlib-1.2.3/README - src/share/native/java/util/zip/zlib-1.2.3/compress.c - src/share/native/java/util/zip/zlib-1.2.3/crc32.h - src/share/native/java/util/zip/zlib-1.2.3/deflate.c - src/share/native/java/util/zip/zlib-1.2.3/deflate.h - src/share/native/java/util/zip/zlib-1.2.3/gzio.c - src/share/native/java/util/zip/zlib-1.2.3/infback.c - src/share/native/java/util/zip/zlib-1.2.3/inffast.c - src/share/native/java/util/zip/zlib-1.2.3/inffast.h - src/share/native/java/util/zip/zlib-1.2.3/inffixed.h - src/share/native/java/util/zip/zlib-1.2.3/inflate.c - src/share/native/java/util/zip/zlib-1.2.3/inflate.h - src/share/native/java/util/zip/zlib-1.2.3/inftrees.c - src/share/native/java/util/zip/zlib-1.2.3/inftrees.h - src/share/native/java/util/zip/zlib-1.2.3/patches/ChangeLog_java - src/share/native/java/util/zip/zlib-1.2.3/patches/crc32.c.diff - src/share/native/java/util/zip/zlib-1.2.3/patches/inflate.c.diff - src/share/native/java/util/zip/zlib-1.2.3/patches/zconf.h.diff - src/share/native/java/util/zip/zlib-1.2.3/patches/zlib.h.diff - src/share/native/java/util/zip/zlib-1.2.3/trees.c - src/share/native/java/util/zip/zlib-1.2.3/trees.h - src/share/native/java/util/zip/zlib-1.2.3/uncompr.c - src/share/native/java/util/zip/zlib-1.2.3/zadler32.c - src/share/native/java/util/zip/zlib-1.2.3/zconf.h - src/share/native/java/util/zip/zlib-1.2.3/zcrc32.c - src/share/native/java/util/zip/zlib-1.2.3/zlib.h - src/share/native/java/util/zip/zlib-1.2.3/zutil.c - src/share/native/java/util/zip/zlib-1.2.3/zutil.h + src/share/native/java/util/zip/zlib-1.2.8/ChangeLog + src/share/native/java/util/zip/zlib-1.2.8/README + src/share/native/java/util/zip/zlib-1.2.8/compress.c + src/share/native/java/util/zip/zlib-1.2.8/crc32.h + src/share/native/java/util/zip/zlib-1.2.8/deflate.c + src/share/native/java/util/zip/zlib-1.2.8/deflate.h + src/share/native/java/util/zip/zlib-1.2.8/gzclose.c + src/share/native/java/util/zip/zlib-1.2.8/gzguts.h + src/share/native/java/util/zip/zlib-1.2.8/gzlib.c + src/share/native/java/util/zip/zlib-1.2.8/gzread.c + src/share/native/java/util/zip/zlib-1.2.8/gzwrite.c + src/share/native/java/util/zip/zlib-1.2.8/infback.c + src/share/native/java/util/zip/zlib-1.2.8/inffast.c + src/share/native/java/util/zip/zlib-1.2.8/inffast.h + src/share/native/java/util/zip/zlib-1.2.8/inffixed.h + src/share/native/java/util/zip/zlib-1.2.8/inflate.c + src/share/native/java/util/zip/zlib-1.2.8/inflate.h + src/share/native/java/util/zip/zlib-1.2.8/inftrees.c + src/share/native/java/util/zip/zlib-1.2.8/inftrees.h + src/share/native/java/util/zip/zlib-1.2.8/patches/ChangeLog_java + src/share/native/java/util/zip/zlib-1.2.8/trees.c + src/share/native/java/util/zip/zlib-1.2.8/trees.h + src/share/native/java/util/zip/zlib-1.2.8/uncompr.c + src/share/native/java/util/zip/zlib-1.2.8/zadler32.c + src/share/native/java/util/zip/zlib-1.2.8/zconf.h + src/share/native/java/util/zip/zlib-1.2.8/zcrc32.c + src/share/native/java/util/zip/zlib-1.2.8/zlib.h + src/share/native/java/util/zip/zlib-1.2.8/zutil.c + src/share/native/java/util/zip/zlib-1.2.8/zutil.h From lana.steuck at oracle.com Tue Sep 9 18:36:08 2014 From: lana.steuck at oracle.com (lana.steuck at oracle.com) Date: Tue, 09 Sep 2014 18:36:08 +0000 Subject: hg: jdk7u/jdk7u/hotspot: 12 new changesets Message-ID: <201409091836.s89Ia8Xo001980@aojmv0008> Changeset: 612e0c2ebea4 Author: morris Date: 2014-08-13 13:00 -0700 URL: http://hg.openjdk.java.net/jdk7u/jdk7u/hotspot/rev/612e0c2ebea4 8054530: C2: assert(res == old_res) failed: Inconsistency between old and new Summary: Fixed signedness problem with assertion. Reviewed-by: kvn ! src/share/vm/oops/objArrayOop.hpp Changeset: cd27a7c464de Author: sgabdura Date: 2014-08-18 08:01 +0200 URL: http://hg.openjdk.java.net/jdk7u/jdk7u/hotspot/rev/cd27a7c464de 8046516: Segmentation fault in JVM (easily reproducible) Summary: Place new nodes on the previous loop exit in reorg_offsets(). Reviewed-by: kvn, roland ! src/share/vm/opto/loopopts.cpp + test/compiler/loopopts/TestLogSum.java Changeset: 682dd189a958 Author: poonam Date: 2014-08-19 04:24 -0700 URL: http://hg.openjdk.java.net/jdk7u/jdk7u/hotspot/rev/682dd189a958 8044406: JVM crash with JDK8 (build 1.8.0-b132) with G1 GC Summary: Fill the last card that has been allocated into with a dummy object Reviewed-by: tschatzl, mgerdin ! src/share/vm/gc_implementation/g1/g1AllocRegion.hpp ! src/share/vm/gc_implementation/g1/g1CollectedHeap.cpp ! src/share/vm/gc_implementation/g1/g1CollectedHeap.hpp Changeset: 4d36da56075c Author: dbuck Date: 2014-08-19 10:03 -0700 URL: http://hg.openjdk.java.net/jdk7u/jdk7u/hotspot/rev/4d36da56075c 8054883: Segmentation error while running program Summary: Fix pattern matching of range check Reviewed-by: kvn ! src/share/vm/opto/ifnode.cpp Changeset: 1443b6a0483c Author: hseigel Date: 2014-08-12 20:29 -0400 URL: http://hg.openjdk.java.net/jdk7u/jdk7u/hotspot/rev/1443b6a0483c 8050485: super() in a try block in a ctor causes VerifyError Summary: Parse catch clause paths to ensure they end in throws Reviewed-by: dlong, acorn, kamg, ctornqvi, lfoltan ! src/share/vm/classfile/verifier.cpp ! src/share/vm/classfile/verifier.hpp Changeset: 02f03ffc0337 Author: sgabdura Date: 2014-08-21 11:13 +0200 URL: http://hg.openjdk.java.net/jdk7u/jdk7u/hotspot/rev/02f03ffc0337 8029302: Performance regression in Math.pow intrinsic Summary: Added special case for x^y where y == 2 Reviewed-by: adlertz, kvn ! src/cpu/x86/vm/assembler_x86.cpp ! src/share/vm/opto/library_call.cpp Changeset: 022dc2063c86 Author: sgabdura Date: 2014-08-21 15:49 +0200 URL: http://hg.openjdk.java.net/jdk7u/jdk7u/hotspot/rev/022dc2063c86 8042052: assert(t != NULL) failed: must set before get Summary: Added missing call to _gvn.transform to make sure the type of the PhiNode is recorded in the gvn. Reviewed-by: anoll, kvn ! src/share/vm/opto/library_call.cpp Changeset: aa8698c3b6c1 Author: sgabdura Date: 2014-08-26 09:23 +0200 URL: http://hg.openjdk.java.net/jdk7u/jdk7u/hotspot/rev/aa8698c3b6c1 8036823: Stack trace sometimes shows 'locked' instead of 'waiting to lock' Summary: Add a ! owner check for 'waiting to lock' to catch current_pending_monitor corner cases. Reviewed-by: dcubed, dholmes, sspitsyn, kmo, zgu ! src/share/vm/runtime/objectMonitor.cpp ! src/share/vm/runtime/vframe.cpp + test/runtime/Thread/TestThreadDumpMonitorContention.java Changeset: a35f4994a5fc Author: sgabdura Date: 2014-08-26 09:29 +0200 URL: http://hg.openjdk.java.net/jdk7u/jdk7u/hotspot/rev/a35f4994a5fc 8046287: [TESTBUG] runtime/Thread/TestThreadDumpMonitorContention.java failed error_cnt=12 Summary: Added a new header waiting pattern to catch the case where the target thread waiting on a condition (like a VM op); several other test improvements Reviewed-by: dcubed, sspitsyn, dholmes ! test/runtime/Thread/TestThreadDumpMonitorContention.java Changeset: 80ffd20441a8 Author: vkempik Date: 2014-09-01 15:33 +0400 URL: http://hg.openjdk.java.net/jdk7u/jdk7u/hotspot/rev/80ffd20441a8 6883953: java -client -XX:ValueMapInitialSize=0 crashes Summary: Add min, max checks for ValueMapInitialSize Reviewed-by: kvn, sgabdura ! src/share/vm/runtime/arguments.cpp Changeset: aad3fc05d972 Author: hseigel Date: 2014-08-29 12:40 -0400 URL: http://hg.openjdk.java.net/jdk7u/jdk7u/hotspot/rev/aad3fc05d972 8046233: VerifyError on backward branch Summary: Check that both blocks have same uninitialized object Reviewed-by: acorn, coleenp ! src/share/vm/classfile/stackMapFrame.cpp ! src/share/vm/classfile/stackMapFrame.hpp ! src/share/vm/classfile/stackMapTable.cpp Changeset: d638e49facb6 Author: lana Date: 2014-09-03 19:05 -0700 URL: http://hg.openjdk.java.net/jdk7u/jdk7u/hotspot/rev/d638e49facb6 Merge From lana.steuck at oracle.com Tue Sep 9 18:36:11 2014 From: lana.steuck at oracle.com (lana.steuck at oracle.com) Date: Tue, 09 Sep 2014 18:36:11 +0000 Subject: hg: jdk7u/jdk7u/jaxws: 2 new changesets Message-ID: <201409091836.s89IaBvN002127@aojmv0008> Changeset: d924f235e859 Author: aefimov Date: 2014-09-02 13:49 +0400 URL: http://hg.openjdk.java.net/jdk7u/jdk7u/jaxws/rev/d924f235e859 8036981: JAXB not preserving formatting for xsd:any Mixed content Reviewed-by: lancea, mkos ! src/share/jaxws_classes/com/sun/xml/internal/bind/v2/runtime/ArrayBeanInfoImpl.java ! src/share/jaxws_classes/com/sun/xml/internal/bind/v2/runtime/ElementBeanInfoImpl.java ! src/share/jaxws_classes/com/sun/xml/internal/bind/v2/runtime/ValueListBeanInfoImpl.java ! src/share/jaxws_classes/com/sun/xml/internal/bind/v2/runtime/property/ArrayERProperty.java ! src/share/jaxws_classes/com/sun/xml/internal/bind/v2/runtime/property/SingleMapNodeProperty.java ! src/share/jaxws_classes/com/sun/xml/internal/bind/v2/runtime/reflect/Accessor.java ! src/share/jaxws_classes/com/sun/xml/internal/bind/v2/runtime/unmarshaller/DefaultValueLoaderDecorator.java ! src/share/jaxws_classes/com/sun/xml/internal/bind/v2/runtime/unmarshaller/Discarder.java ! src/share/jaxws_classes/com/sun/xml/internal/bind/v2/runtime/unmarshaller/DomLoader.java ! src/share/jaxws_classes/com/sun/xml/internal/bind/v2/runtime/unmarshaller/LeafPropertyLoader.java ! src/share/jaxws_classes/com/sun/xml/internal/bind/v2/runtime/unmarshaller/LeafPropertyXsiLoader.java ! src/share/jaxws_classes/com/sun/xml/internal/bind/v2/runtime/unmarshaller/Loader.java ! src/share/jaxws_classes/com/sun/xml/internal/bind/v2/runtime/unmarshaller/ProxyLoader.java ! src/share/jaxws_classes/com/sun/xml/internal/bind/v2/runtime/unmarshaller/SAXConnector.java ! src/share/jaxws_classes/com/sun/xml/internal/bind/v2/runtime/unmarshaller/Scope.java ! src/share/jaxws_classes/com/sun/xml/internal/bind/v2/runtime/unmarshaller/StructureLoader.java ! src/share/jaxws_classes/com/sun/xml/internal/bind/v2/runtime/unmarshaller/TextLoader.java ! src/share/jaxws_classes/com/sun/xml/internal/bind/v2/runtime/unmarshaller/UnmarshallingContext.java ! src/share/jaxws_classes/com/sun/xml/internal/bind/v2/runtime/unmarshaller/ValuePropertyLoader.java ! src/share/jaxws_classes/com/sun/xml/internal/bind/v2/runtime/unmarshaller/XsiNilLoader.java ! src/share/jaxws_classes/com/sun/xml/internal/bind/v2/runtime/unmarshaller/XsiTypeLoader.java Changeset: 12905bf82bda Author: lana Date: 2014-09-03 19:06 -0700 URL: http://hg.openjdk.java.net/jdk7u/jdk7u/jaxws/rev/12905bf82bda Merge From lana.steuck at oracle.com Tue Sep 9 18:36:18 2014 From: lana.steuck at oracle.com (lana.steuck at oracle.com) Date: Tue, 09 Sep 2014 18:36:18 +0000 Subject: hg: jdk7u/jdk7u/jdk: 29 new changesets Message-ID: <201409091836.s89IaJJu002140@aojmv0008> Changeset: 0679be303315 Author: serb Date: 2013-12-12 16:30 +0400 URL: http://hg.openjdk.java.net/jdk7u/jdk7u/jdk/rev/0679be303315 8001472: api/java_awt/Window/indexTGF_* tests fail because expected colors aren't equal Reviewed-by: anthony, azvegint ! src/solaris/classes/sun/awt/X11/XWindow.java + test/java/awt/Window/BackgroundIsNotUpdated/BackgroundIsNotUpdated.java Changeset: aca7751320fc Author: mcherkas Date: 2014-08-12 19:02 +0400 URL: http://hg.openjdk.java.net/jdk7u/jdk7u/jdk/rev/aca7751320fc 8042835: Remove mnemonic character from open, save and open directory JFileChooser's buttons Reviewed-by: alexp, alexsch Contributed-by: artem.malinko at oracle.com ! src/share/classes/com/sun/swing/internal/plaf/basic/resources/basic_ja.properties ! src/share/classes/com/sun/swing/internal/plaf/basic/resources/basic_ko.properties ! src/share/classes/com/sun/swing/internal/plaf/basic/resources/basic_zh_CN.properties ! src/share/classes/com/sun/swing/internal/plaf/basic/resources/basic_zh_TW.properties Changeset: b25397be1e6a Author: naoto Date: 2014-08-15 12:46 -0700 URL: http://hg.openjdk.java.net/jdk7u/jdk7u/jdk/rev/b25397be1e6a 8041791: String.toLowerCase regression - violates Unicode standard Reviewed-by: peytoia Contributed-by: jeremymanson at google.com ! src/share/classes/java/lang/ConditionalSpecialCasing.java ! src/share/classes/java/lang/String.java ! test/java/lang/String/ToLowerCase.java Changeset: b64f4a90cee7 Author: aivanov Date: 2014-08-18 11:08 +0400 URL: http://hg.openjdk.java.net/jdk7u/jdk7u/jdk/rev/b64f4a90cee7 8046007: Java app receives javax.print.PrintException: Printer is not accepting job Reviewed-by: prr, bae ! src/windows/native/sun/windows/WPrinterJob.cpp Changeset: 434a2dde9c21 Author: igerasim Date: 2014-08-18 17:14 +0400 URL: http://hg.openjdk.java.net/jdk7u/jdk7u/jdk/rev/434a2dde9c21 8009258: TEST_BUG:java/io/pathNames/GeneralWin32.java fails intermittently Reviewed-by: dxu, alanb Contributed-by: yiming.wang at oracle.com ! test/java/io/pathNames/General.java ! test/java/io/pathNames/GeneralWin32.java Changeset: 74c83f186ab1 Author: igerasim Date: 2014-08-18 17:16 +0400 URL: http://hg.openjdk.java.net/jdk7u/jdk7u/jdk/rev/74c83f186ab1 8015628: Test Failure in closed/java/io/pathNames/GeneralSolaris.java Reviewed-by: alanb ! test/java/io/pathNames/General.java ! test/java/io/pathNames/GeneralWin32.java Changeset: b94e612ae7c5 Author: igerasim Date: 2014-08-18 17:18 +0400 URL: http://hg.openjdk.java.net/jdk7u/jdk7u/jdk/rev/b94e612ae7c5 4792059: test/java/io/pathNames/GeneralSolaris.java fails on symbolic links Summary: Exclude the possible usage of linked files or directories in the test Reviewed-by: alanb ! test/java/io/pathNames/General.java Changeset: 8a39734cca39 Author: igerasim Date: 2014-08-18 17:20 +0400 URL: http://hg.openjdk.java.net/jdk7u/jdk7u/jdk/rev/8a39734cca39 8028631: Improve the test coverage to the pathname handling on unix-like platforms Summary: Add GeneralSolaris.java testcase and fix the concurrency issue Reviewed-by: lancea, chegar, alanb ! test/java/io/pathNames/General.java + test/java/io/pathNames/GeneralSolaris.java ! test/java/io/pathNames/GeneralWin32.java Changeset: 9c832ae708b3 Author: aivanov Date: 2014-08-23 01:15 +0400 URL: http://hg.openjdk.java.net/jdk7u/jdk7u/jdk/rev/9c832ae708b3 8043610: Sorting columns in JFileChooser fails with AppContext NPE Reviewed-by: anthony, alexsch ! src/share/classes/com/sun/java/swing/SwingUtilities3.java ! src/share/classes/java/awt/Component.java ! src/share/classes/javax/swing/JComponent.java + test/javax/swing/JComponent/8043610/bug8043610.java Changeset: 140a7fe18d66 Author: igerasim Date: 2014-08-23 01:05 +0400 URL: http://hg.openjdk.java.net/jdk7u/jdk7u/jdk/rev/140a7fe18d66 8055731: sun/security/smartcardio/TestDirect.java throws java.lang.IndexOutOfBoundsException Reviewed-by: valeriep ! test/sun/security/smartcardio/TestDirect.java Changeset: 525b2ad2fc7c Author: bae Date: 2014-08-26 15:53 +0100 URL: http://hg.openjdk.java.net/jdk7u/jdk7u/jdk/rev/525b2ad2fc7c 8040617: [macosx] Large JTable cell results in a OutOfMemoryException Reviewed-by: serb, prr ! src/share/classes/sun/java2d/opengl/OGLSurfaceDataProxy.java + test/sun/java2d/OpenGL/DrawHugeImageTest.java Changeset: fd940a08ec2d Author: igerasim Date: 2014-08-26 15:55 +0400 URL: http://hg.openjdk.java.net/jdk7u/jdk7u/jdk/rev/fd940a08ec2d 8055421: (fs) bad error handling in java.base/unix/native/libnio/fs/UnixNativeDispatcher.c Reviewed-by: martin, alanb ! src/solaris/native/sun/nio/fs/UnixNativeDispatcher.c Changeset: 70b7e2766d4a Author: aivanov Date: 2014-08-28 17:14 +0400 URL: http://hg.openjdk.java.net/jdk7u/jdk7u/jdk/rev/70b7e2766d4a 8056156: [TEST_BUG] Test javax/swing/JFileChooser/8046391/bug8046391.java fails in Windows Reviewed-by: anthony, alexp ! test/javax/swing/JFileChooser/8046391/bug8046391.java Changeset: f40d035e338d Author: coffeys Date: 2014-08-22 16:10 +0100 URL: http://hg.openjdk.java.net/jdk7u/jdk7u/jdk/rev/f40d035e338d 8032808: Support Solaris SO_FLOW_SLA socket option Reviewed-by: michaelm ! make/Makefile ! make/common/shared/Defs-javadoc.gmk ! make/docs/Makefile ! make/docs/NON_CORE_PKGS.gmk ! make/java/net/FILES_c.gmk ! make/java/net/mapfile-vers + make/jdk/Makefile + make/jdk/net/FILES_java.gmk + make/jdk/net/Makefile ! make/sun/net/FILES_java.gmk ! src/share/classes/java/net/DatagramSocket.java ! src/share/classes/java/net/DatagramSocketImpl.java ! src/share/classes/java/net/SocketImpl.java + src/share/classes/java/net/SocketSecrets.java + src/share/classes/jdk/net/ExtendedSocketOptions.java + src/share/classes/jdk/net/NetworkPermission.java + src/share/classes/jdk/net/SocketFlow.java + src/share/classes/jdk/net/Sockets.java + src/share/classes/jdk/net/package-info.java + src/share/classes/sun/net/ExtendedOptionsImpl.java ! src/share/classes/sun/nio/ch/AsynchronousSocketChannelImpl.java ! src/share/classes/sun/nio/ch/DatagramChannelImpl.java ! src/share/classes/sun/nio/ch/Net.java ! src/share/classes/sun/nio/ch/SocketChannelImpl.java + src/share/native/java/net/jdk_net_SocketFlow.h ! src/share/native/java/net/net_util.h ! src/solaris/classes/java/net/PlainDatagramSocketImpl.java ! src/solaris/classes/java/net/PlainSocketImpl.java + src/solaris/native/java/net/ExtendedOptionsImpl.c ! src/solaris/native/java/net/net_util_md.h + src/windows/native/java/net/ExtendedOptionsImpl.c ! test/Makefile + test/jdk/net/Sockets/Test.java + test/jdk/net/Sockets/policy.fail + test/jdk/net/Sockets/policy.success Changeset: e036c254ef30 Author: michaelm Date: 2014-06-24 11:32 +0100 URL: http://hg.openjdk.java.net/jdk7u/jdk7u/jdk/rev/e036c254ef30 8047186: jdk.net.Sockets throws InvocationTargetException instead of original runtime exceptions Reviewed-by: michaelm Contributed-by: artem.smotrakov at oracle.com ! src/share/classes/jdk/net/Sockets.java Changeset: 75dceeafa4c8 Author: michaelm Date: 2014-08-15 14:50 +0100 URL: http://hg.openjdk.java.net/jdk7u/jdk7u/jdk/rev/75dceeafa4c8 8029607: Type of Service (TOS) cannot be set in IPv6 header Reviewed-by: alanb ! src/share/classes/sun/nio/ch/DatagramChannelImpl.java ! src/share/classes/sun/nio/ch/Net.java ! src/share/classes/sun/nio/ch/ServerSocketChannelImpl.java ! src/share/classes/sun/nio/ch/SocketChannelImpl.java ! src/share/native/sun/nio/ch/genSocketOptionRegistry.c ! src/solaris/native/java/net/net_util_md.c ! src/solaris/native/sun/nio/ch/Net.c ! src/windows/native/sun/nio/ch/Net.c Changeset: 8d0a1cb7b3b9 Author: michaelm Date: 2014-07-07 12:42 +0100 URL: http://hg.openjdk.java.net/jdk7u/jdk7u/jdk/rev/8d0a1cb7b3b9 8048212: Two tests failed with "java.net.SocketException: Bad protocol option" on Windows after 8029607 Reviewed-by: alanb ! src/windows/native/java/net/net_util_md.c ! src/windows/native/sun/nio/ch/Net.c Changeset: 6c199d604eef Author: coffeys Date: 2014-08-28 14:18 +0100 URL: http://hg.openjdk.java.net/jdk7u/jdk7u/jdk/rev/6c199d604eef Merge Changeset: 85eb739ea5a6 Author: weijun Date: 2012-12-17 12:18 +0800 URL: http://hg.openjdk.java.net/jdk7u/jdk7u/jdk/rev/85eb739ea5a6 7197159: accept different kvno if there no match Reviewed-by: xuelei ! src/share/classes/sun/security/krb5/EncryptionKey.java ! test/sun/security/krb5/auto/DynamicKeytab.java + test/sun/security/krb5/auto/KvnoNA.java ! test/sun/security/krb5/auto/MoreKvno.java Changeset: 2df9079cfbd9 Author: igerasim Date: 2014-08-28 18:39 +0400 URL: http://hg.openjdk.java.net/jdk7u/jdk7u/jdk/rev/2df9079cfbd9 8017212: File.createTempFile requires unnecessary "read" permission Reviewed-by: alanb ! src/share/classes/java/io/File.java + test/java/io/File/CheckPermission.java ! test/java/io/File/NulFile.java ! test/java/io/File/createTempFile/SpecialTempFile.java Changeset: 353c0eb4aace Author: robm Date: 2014-08-28 18:01 +0100 URL: http://hg.openjdk.java.net/jdk7u/jdk7u/jdk/rev/353c0eb4aace 7158636: InterfaceAddress.getBroadcast() returns invalid broadcast address on WLAN Summary: Update Windows native code to infer WLAN interface type in Windows Vista and later Reviewed-by: chegar, alanb ! src/windows/native/java/net/NetworkInterface.c ! src/windows/native/java/net/NetworkInterface.h Changeset: 0032f33fdc47 Author: igerasim Date: 2014-08-29 17:13 +0400 URL: http://hg.openjdk.java.net/jdk7u/jdk7u/jdk/rev/0032f33fdc47 7058832: com/sun/net/httpserver/bugs/B6373555.java failing intermittently Reviewed-by: alanb ! test/com/sun/net/httpserver/bugs/B6373555.java Changeset: 87ff88fdd832 Author: aefimov Date: 2014-09-02 13:57 +0400 URL: http://hg.openjdk.java.net/jdk7u/jdk7u/jdk/rev/87ff88fdd832 8036981: JAXB not preserving formatting for xsd:any Mixed content Reviewed-by: lancea, mkos + test/javax/xml/bind/marshal/8036981/Good.java + test/javax/xml/bind/marshal/8036981/Main.java + test/javax/xml/bind/marshal/8036981/ObjectFactory.java + test/javax/xml/bind/marshal/8036981/Root.java + test/javax/xml/bind/marshal/8036981/Test.java + test/javax/xml/bind/marshal/8036981/test.xml Changeset: 866ce3916733 Author: michaelm Date: 2013-05-16 17:28 +0100 URL: http://hg.openjdk.java.net/jdk7u/jdk7u/jdk/rev/866ce3916733 8012625: Incorrect handling of HTTP/1.1 " Expect: 100-continue " in HttpURLConnection Reviewed-by: alanb, chegar ! src/share/classes/sun/net/www/protocol/http/HttpURLConnection.java + test/sun/net/www/protocol/http/B8012625.java Changeset: 43d918d51967 Author: coffeys Date: 2014-09-02 15:56 +0100 URL: http://hg.openjdk.java.net/jdk7u/jdk7u/jdk/rev/43d918d51967 Merge Changeset: eec956f1670c Author: coffeys Date: 2014-09-02 22:25 +0100 URL: http://hg.openjdk.java.net/jdk7u/jdk7u/jdk/rev/eec956f1670c 8054019: Keytool Error publicKey's is not X.509, but X509 Reviewed-by: mullan, xuelei ! src/share/classes/sun/security/x509/CertAndKeyGen.java Changeset: b6d1b9081cb3 Author: weijun Date: 2014-08-11 22:09 +0800 URL: http://hg.openjdk.java.net/jdk7u/jdk7u/jdk/rev/b6d1b9081cb3 8054817: File ccache only recognizes Linux and Solaris defaults Reviewed-by: mullan ! src/share/classes/sun/security/krb5/internal/ccache/FileCredentialsCache.java + test/sun/security/krb5/ccache/DefaultFile.java Changeset: e3161d87f66b Author: vinnie Date: 2012-05-02 14:50 +0100 URL: http://hg.openjdk.java.net/jdk7u/jdk7u/jdk/rev/e3161d87f66b 7087021: TEST: com/sun/crypto/provider/Mac/MacClone.java failed on Solaris sparc 5.10 Reviewed-by: mullan ! test/com/sun/crypto/provider/Mac/MacClone.java Changeset: 1326b00e10d1 Author: lana Date: 2014-09-03 19:05 -0700 URL: http://hg.openjdk.java.net/jdk7u/jdk7u/jdk/rev/1326b00e10d1 Merge From lana.steuck at oracle.com Tue Sep 9 18:43:30 2014 From: lana.steuck at oracle.com (lana.steuck at oracle.com) Date: Tue, 9 Sep 2014 11:43:30 -0700 (PDT) Subject: jdk7u-b03: jdk7u-dev Message-ID: <201409091843.s89IhU36029313@jano-app.us.oracle.com> http://hg.openjdk.java.net/jdk7u/jdk7u/rev/32e87fe0ad53 http://hg.openjdk.java.net/jdk7u/jdk7u/langtools/rev/41b81b3e37cd http://hg.openjdk.java.net/jdk7u/jdk7u/jdk/rev/1326b00e10d1 http://hg.openjdk.java.net/jdk7u/jdk7u/jaxws/rev/12905bf82bda http://hg.openjdk.java.net/jdk7u/jdk7u/jaxp/rev/498ddd5ee22c http://hg.openjdk.java.net/jdk7u/jdk7u/hotspot/rev/d638e49facb6 http://hg.openjdk.java.net/jdk7u/jdk7u/corba/rev/34147ec06239 --- All the fixes will be tested during promotion (no PIT testing at this point): List of all fixes: =================== JDK-8046007 client-libs Java app receives javax.print.PrintException: Printer is not accepting job. JDK-8029217 client-libs Java2Demo AccessControlException JDK-8043610 client-libs Sorting columns in JFileChooser fails with AppContext NPE JDK-8042835 client-libs Unexpected mnemonic in JFileChooser JDK-8056156 client-libs [TEST_BUG] Test javax/swing/JFileChooser/8046391/bug8046391.java fails in Windows JDK-8001472 client-libs [linux-sparc] api/java_awt/Window/indexTGF_* tests fail because expected colors aren't equal JDK-8040617 client-libs [macosx] Large JTable cell results in a OutOfMemoryException JDK-8055421 core-libs (fs) bad error handling in java.base/unix/native/libnio/fs/UnixNativeDispatcher.c JDK-8017212 core-libs File.createTempFile requires unnecessary "read" permission JDK-8028631 core-libs Improve the test coverage to the pathname handling on unix-like platforms JDK-8012625 core-libs Incorrect handling of HTTP/1.1 " Expect: 100-continue " in HttpURLConnection JDK-7158636 core-libs InterfaceAddress.getBroadcast() returns invalid broadcast address on WLAN JDK-8041791 core-libs String.toLowerCase regression - violates Unicode standard JDK-8032808 core-libs Support Solaris SO_FLOW_SLA socket option JDK-8009258 core-libs TEST_BUG: java/io/pathNames/GeneralWin32.java fails intermittently JDK-8015628 core-libs Test Failure in closed/java/io/pathNames/GeneralSolaris.java JDK-8048212 core-libs Two tests failed with "java.net.SocketException: Bad protocol option" on Windows after 8029607 JDK-8029607 core-libs Type of Service (TOS) cannot be set in IPv6 header JDK-7058832 core-libs com/sun/net/httpserver/bugs/B6373555.java failing intermittently JDK-8047186 core-libs jdk.net.Sockets throws InvocationTargetException instead of original runtime exceptions JDK-4792059 core-libs test/java/io/pathNames/GeneralSolaris.java fails on symbolic links JDK-8028101 core-svc closed/sun/tracing/ProviderProxyTest.java fails with assertions enabled JDK-8037224 deploy Exception Site List: no informative tooltip for "Edit Site List" button JDK-8050016 deploy Improve Java Control Panel support for High DPI JDK-8020304 deploy Self-signed sandbox app causes "expired JRE" blocking window since 7u40 b29 ( Very High security level + expired JRE) JDK-8041499 deploy Unsigned jars in manifest of signed JNLP applet are not executed if host is added to ESL JDK-8044043 deploy Warn user of invalid elements and attributes in ruleset.xml JDK-8049336 deploy Warning always displayed on LiveConnect call in Java 7u60 JDK-8054530 hotspot C2: assert(res == old_res) failed: Inconsistency between old and new JDK-8044406 hotspot JVM crash with JDK8 (build 1.8.0-b132) with G1 GC JDK-8029302 hotspot Performance regression in Math.pow intrinsic JDK-8054883 hotspot Segmentation error while running program JDK-8046516 hotspot Segmentation fault in JVM (easily reproducible) JDK-8036823 hotspot Stack trace sometimes shows 'locked' instead of 'waiting to lock' JDK-8046233 hotspot VerifyError on backward branch JDK-8046287 hotspot [TESTBUG] runtime/Thread/TestThreadDumpMonitorContention.java failed error_cnt=12 JDK-8042052 hotspot assert(t != NULL) failed: must set before get JDK-6883953 hotspot java -client -XX:ValueMapInitialSize=0 crashes JDK-8050485 hotspot super() in a try block in a ctor may need to cause VerifyError JDK-8054817 security-libs File ccache only recognizes Linux and Solaris defaults JDK-8054019 security-libs Keytool Error publicKey's is not X.509, but X509 JDK-7087021 security-libs TEST: com/sun/crypto/provider/Mac/MacClone.java failed on Solaris sparc 5.10 JDK-7197159 security-libs accept different kvno if there no match JDK-8055731 security-libs sun/security/smartcardio/TestDirect.java throws java.lang.IndexOutOfBoundsException JDK-8036981 xml JAXB not preserving formatting for xsd:any Mixed content From lana.steuck at oracle.com Tue Sep 9 21:17:33 2014 From: lana.steuck at oracle.com (lana.steuck at oracle.com) Date: Tue, 09 Sep 2014 21:17:33 +0000 Subject: hg: jdk7u/jdk7u-dev: Added tag jdk7u80-b02 for changeset 36e8397bf04d Message-ID: <201409092117.s89LHXZp029093@aojmv0008> Changeset: 32e87fe0ad53 Author: katleman Date: 2014-08-20 12:12 -0700 URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/rev/32e87fe0ad53 Added tag jdk7u80-b02 for changeset 36e8397bf04d ! .hgtags From lana.steuck at oracle.com Tue Sep 9 21:17:32 2014 From: lana.steuck at oracle.com (lana.steuck at oracle.com) Date: Tue, 09 Sep 2014 21:17:32 +0000 Subject: hg: jdk7u/jdk7u-dev/corba: Added tag jdk7u80-b02 for changeset 2444fa7df7e3 Message-ID: <201409092117.s89LHWJ1029071@aojmv0008> Changeset: 34147ec06239 Author: katleman Date: 2014-08-20 12:12 -0700 URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/corba/rev/34147ec06239 Added tag jdk7u80-b02 for changeset 2444fa7df7e3 ! .hgtags From lana.steuck at oracle.com Tue Sep 9 21:17:34 2014 From: lana.steuck at oracle.com (lana.steuck at oracle.com) Date: Tue, 09 Sep 2014 21:17:34 +0000 Subject: hg: jdk7u/jdk7u-dev/hotspot: 3 new changesets Message-ID: <201409092117.s89LHZYw029096@aojmv0008> Changeset: ded432e6064b Author: katleman Date: 2014-08-20 12:12 -0700 URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/hotspot/rev/ded432e6064b Added tag jdk7u80-b02 for changeset bad107a5d096 ! .hgtags Changeset: d638e49facb6 Author: lana Date: 2014-09-03 19:05 -0700 URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/hotspot/rev/d638e49facb6 Merge Changeset: 5fd0a42e71aa Author: lana Date: 2014-09-09 11:57 -0700 URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/hotspot/rev/5fd0a42e71aa Merge From lana.steuck at oracle.com Tue Sep 9 21:17:39 2014 From: lana.steuck at oracle.com (lana.steuck at oracle.com) Date: Tue, 09 Sep 2014 21:17:39 +0000 Subject: hg: jdk7u/jdk7u-dev/jaxp: Added tag jdk7u80-b02 for changeset 25a1b88d7a47 Message-ID: <201409092117.s89LHdU7029207@aojmv0008> Changeset: 498ddd5ee22c Author: katleman Date: 2014-08-20 12:12 -0700 URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/jaxp/rev/498ddd5ee22c Added tag jdk7u80-b02 for changeset 25a1b88d7a47 ! .hgtags From lana.steuck at oracle.com Tue Sep 9 21:17:40 2014 From: lana.steuck at oracle.com (lana.steuck at oracle.com) Date: Tue, 09 Sep 2014 21:17:40 +0000 Subject: hg: jdk7u/jdk7u-dev/jaxws: 2 new changesets Message-ID: <201409092117.s89LHeag029274@aojmv0008> Changeset: ace21763f956 Author: katleman Date: 2014-08-20 12:12 -0700 URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/jaxws/rev/ace21763f956 Added tag jdk7u80-b02 for changeset 579128925dd9 ! .hgtags Changeset: 12905bf82bda Author: lana Date: 2014-09-03 19:06 -0700 URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/jaxws/rev/12905bf82bda Merge From lana.steuck at oracle.com Tue Sep 9 21:17:41 2014 From: lana.steuck at oracle.com (lana.steuck at oracle.com) Date: Tue, 09 Sep 2014 21:17:41 +0000 Subject: hg: jdk7u/jdk7u-dev/langtools: Added tag jdk7u80-b02 for changeset 5bd6f3adf690 Message-ID: <201409092117.s89LHfYD029290@aojmv0008> Changeset: 41b81b3e37cd Author: katleman Date: 2014-08-20 12:13 -0700 URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/langtools/rev/41b81b3e37cd Added tag jdk7u80-b02 for changeset 5bd6f3adf690 ! .hgtags From lana.steuck at oracle.com Tue Sep 9 21:17:42 2014 From: lana.steuck at oracle.com (lana.steuck at oracle.com) Date: Tue, 09 Sep 2014 21:17:42 +0000 Subject: hg: jdk7u/jdk7u-dev/jdk: 3 new changesets Message-ID: <201409092117.s89LHg5r029309@aojmv0008> Changeset: 828c4fedd29f Author: katleman Date: 2014-08-20 12:13 -0700 URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/jdk/rev/828c4fedd29f Added tag jdk7u80-b02 for changeset 2590a9c18fdb ! .hgtags Changeset: 1326b00e10d1 Author: lana Date: 2014-09-03 19:05 -0700 URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/jdk/rev/1326b00e10d1 Merge Changeset: 54503c218de8 Author: lana Date: 2014-09-09 11:57 -0700 URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/jdk/rev/54503c218de8 Merge - src/share/native/java/util/zip/zlib-1.2.3/ChangeLog - src/share/native/java/util/zip/zlib-1.2.3/README - src/share/native/java/util/zip/zlib-1.2.3/compress.c - src/share/native/java/util/zip/zlib-1.2.3/crc32.h - src/share/native/java/util/zip/zlib-1.2.3/deflate.c - src/share/native/java/util/zip/zlib-1.2.3/deflate.h - src/share/native/java/util/zip/zlib-1.2.3/gzio.c - src/share/native/java/util/zip/zlib-1.2.3/infback.c - src/share/native/java/util/zip/zlib-1.2.3/inffast.c - src/share/native/java/util/zip/zlib-1.2.3/inffast.h - src/share/native/java/util/zip/zlib-1.2.3/inffixed.h - src/share/native/java/util/zip/zlib-1.2.3/inflate.c - src/share/native/java/util/zip/zlib-1.2.3/inflate.h - src/share/native/java/util/zip/zlib-1.2.3/inftrees.c - src/share/native/java/util/zip/zlib-1.2.3/inftrees.h - src/share/native/java/util/zip/zlib-1.2.3/patches/ChangeLog_java - src/share/native/java/util/zip/zlib-1.2.3/patches/crc32.c.diff - src/share/native/java/util/zip/zlib-1.2.3/patches/inflate.c.diff - src/share/native/java/util/zip/zlib-1.2.3/patches/zconf.h.diff - src/share/native/java/util/zip/zlib-1.2.3/patches/zlib.h.diff - src/share/native/java/util/zip/zlib-1.2.3/trees.c - src/share/native/java/util/zip/zlib-1.2.3/trees.h - src/share/native/java/util/zip/zlib-1.2.3/uncompr.c - src/share/native/java/util/zip/zlib-1.2.3/zadler32.c - src/share/native/java/util/zip/zlib-1.2.3/zconf.h - src/share/native/java/util/zip/zlib-1.2.3/zcrc32.c - src/share/native/java/util/zip/zlib-1.2.3/zlib.h - src/share/native/java/util/zip/zlib-1.2.3/zutil.c - src/share/native/java/util/zip/zlib-1.2.3/zutil.h - test/sun/tools/jstatd/jpsOutput1.awk - test/sun/tools/jstatd/jstatGcutilOutput1.awk - test/sun/tools/jstatd/jstatdDefaults.sh - test/sun/tools/jstatd/jstatdExternalRegistry.sh - test/sun/tools/jstatd/jstatdPort.sh - test/sun/tools/jstatd/jstatdServerName.sh - test/sun/tools/jstatd/jstatdUsage1.sh - test/sun/tools/jstatd/usage.out From dmitry.markov at oracle.com Wed Sep 10 12:16:07 2014 From: dmitry.markov at oracle.com (dmitry markov) Date: Wed, 10 Sep 2014 16:16:07 +0400 Subject: [7u-dev] Request for approval 8028484: [TEST_BUG][macosx] closed/java/awt/MouseInfo/JContainerMousePositionTest fails Message-ID: <54104107.9070009@oracle.com> Hello, Could you approve the back-port of the fix for 8028484 to jdk7u-dev (7u80), please? The original fix and the back-port are slightly different. There are trivial changes in the regression test (lambda was replaced by anonymous inner class). The author and the reviewers of the original fix are in the CC list. The bug: https://bugs.openjdk.java.net/browse/JDK-8028484 The webrev: http://cr.openjdk.java.net/~dmarkov/8028484/open/webrev.00/ - add to open repo http://cr.openjdk.java.net/~dmarkov/8028484/closed/webrev.00/ - diff with the previous version of the test The review thread: http://mail.openjdk.java.net/pipermail/awt-dev/2013-December/006493.html The jdk9 changeset: http://hg.openjdk.java.net/jdk9/client/jdk/rev/1490b2b2af97 The jdk8u changeset: http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/1490b2b2af97 Thanks, Dmitry From sean.coffey at oracle.com Wed Sep 10 12:19:33 2014 From: sean.coffey at oracle.com (=?UTF-8?B?U2XDoW4gQ29mZmV5?=) Date: Wed, 10 Sep 2014 13:19:33 +0100 Subject: [7u-dev] Request for approval 8028484: [TEST_BUG][macosx] closed/java/awt/MouseInfo/JContainerMousePositionTest fails In-Reply-To: <54104107.9070009@oracle.com> References: <54104107.9070009@oracle.com> Message-ID: <541041D5.4090102@oracle.com> Approved but subject to code review. regards, Sean. On 10/09/14 13:16, dmitry markov wrote: > Hello, > > Could you approve the back-port of the fix for 8028484 to jdk7u-dev > (7u80), please? > The original fix and the back-port are slightly different. There are > trivial changes in the regression test (lambda was replaced by > anonymous inner class). The author and the reviewers of the original > fix are in the CC list. > > The bug: > https://bugs.openjdk.java.net/browse/JDK-8028484 > The webrev: > http://cr.openjdk.java.net/~dmarkov/8028484/open/webrev.00/ - add to > open repo > http://cr.openjdk.java.net/~dmarkov/8028484/closed/webrev.00/ - diff > with the previous version of the test > The review thread: > http://mail.openjdk.java.net/pipermail/awt-dev/2013-December/006493.html > The jdk9 changeset: > http://hg.openjdk.java.net/jdk9/client/jdk/rev/1490b2b2af97 > The jdk8u changeset: > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/1490b2b2af97 > > Thanks, > Dmitry From dmitry.markov at oracle.com Wed Sep 10 12:25:33 2014 From: dmitry.markov at oracle.com (dmitry markov) Date: Wed, 10 Sep 2014 16:25:33 +0400 Subject: [7u-dev] Request for approval 8028484: [TEST_BUG][macosx] closed/java/awt/MouseInfo/JContainerMousePositionTest fails In-Reply-To: <541041D5.4090102@oracle.com> References: <54104107.9070009@oracle.com> <541041D5.4090102@oracle.com> Message-ID: <5410433D.4060708@oracle.com> Thank you, Sean! Anthony, Sergey, Could you review this, please? Thanks, Dmitry On 10/09/2014 16:19, Se?n Coffey wrote: > Approved but subject to code review. > > regards, > Sean. > > On 10/09/14 13:16, dmitry markov wrote: >> Hello, >> >> Could you approve the back-port of the fix for 8028484 to jdk7u-dev >> (7u80), please? >> The original fix and the back-port are slightly different. There are >> trivial changes in the regression test (lambda was replaced by >> anonymous inner class). The author and the reviewers of the original >> fix are in the CC list. >> >> The bug: >> https://bugs.openjdk.java.net/browse/JDK-8028484 >> The webrev: >> http://cr.openjdk.java.net/~dmarkov/8028484/open/webrev.00/ - add to >> open repo >> http://cr.openjdk.java.net/~dmarkov/8028484/closed/webrev.00/ - diff >> with the previous version of the test >> The review thread: >> http://mail.openjdk.java.net/pipermail/awt-dev/2013-December/006493.html >> The jdk9 changeset: >> http://hg.openjdk.java.net/jdk9/client/jdk/rev/1490b2b2af97 >> The jdk8u changeset: >> http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/1490b2b2af97 >> >> Thanks, >> Dmitry > From kevin.walls at oracle.com Wed Sep 10 13:46:57 2014 From: kevin.walls at oracle.com (Kevin Walls) Date: Wed, 10 Sep 2014 14:46:57 +0100 Subject: [7u-dev] Request for approval: 8049684: pstack crashes on java core dump Message-ID: <54105651.2080505@oracle.com> Hi, Please approve a simple backport of this hotspot fix that's in 9 and 8u. bug: https://bugs.openjdk.java.net/browse/JDK-8049684 7u webrev: http://cr.openjdk.java.net/~kevinw/8049684/7u/ The 8u changeset imports cleanly into 7u/hotspot: http://hg.openjdk.java.net/jdk8u/hs-dev/hotspot/rev/fd94cbe7c5da Thanks Kevin From sean.coffey at oracle.com Wed Sep 10 13:53:38 2014 From: sean.coffey at oracle.com (=?ISO-8859-1?Q?Se=E1n_Coffey?=) Date: Wed, 10 Sep 2014 14:53:38 +0100 Subject: [7u-dev] Request for approval: 8049684: pstack crashes on java core dump In-Reply-To: <54105651.2080505@oracle.com> References: <54105651.2080505@oracle.com> Message-ID: <541057E2.6020008@oracle.com> Approved. regards, Sean. On 10/09/14 14:46, Kevin Walls wrote: > Hi, > > Please approve a simple backport of this hotspot fix that's in 9 and 8u. > > bug: > https://bugs.openjdk.java.net/browse/JDK-8049684 > > 7u webrev: > http://cr.openjdk.java.net/~kevinw/8049684/7u/ > > The 8u changeset imports cleanly into 7u/hotspot: > http://hg.openjdk.java.net/jdk8u/hs-dev/hotspot/rev/fd94cbe7c5da > > Thanks > Kevin From andreas.eriksson at oracle.com Wed Sep 10 13:58:44 2014 From: andreas.eriksson at oracle.com (Andreas Eriksson) Date: Wed, 10 Sep 2014 15:58:44 +0200 Subject: RFR: 8041980 + 6461635 In-Reply-To: <540061F9.9010104@oracle.com> References: <53FF401A.9000205@oracle.com> <24105B5B-9CDD-40D5-8715-A6B2AFEF78A4@oracle.com> <540061F9.9010104@oracle.com> Message-ID: <54105914.6050906@oracle.com> Any jdk7u Reviewers out there that could take a look at this? Regards, Andreas On 2014-08-29 13:20, Andreas Eriksson wrote: > Thanks Staffan! > > Could I get a Reviewer too look at this as well please? > > > > I got a question why the changesets didn't apply cleanly to jdk7, here > is a summary: > > For the HotSpot part: > There were no aix or bsd-darwin mapfiles in jdk7. > > For JDK: > The testlibraries changes in jdk8 were already backported to jdk7 as a > separate change. > The .sh-files that were removed were slightly different (jdk8 passed a > ${TESTVMOPTS} to the java process). > The context in problemlist.txt was different. > > The main logic in the changesets applied to jdk7 cleanly. > > Regards, > Andreas > > On 2014-08-29 13:09, Staffan Larsen wrote: >> Looks good! (not a Reviewer for jdk7u) >> >> Thanks, >> /Staffan >> >> On 28 aug 2014, at 16:43, Andreas Eriksson >> > wrote: >> >>> Summary: >>> This is a backport of a change that fixes problems with jvmstat, >>> where it can't find local JVM processes. >>> Also backported is a change that improves tests related to this area. >>> >>> I could not run the regression tests on solaris_sparcv9 or >>> solaris_x64 due to a JPRT limitation. >>> All other platforms show no failures on JPRT with the added and >>> modified regression test run as well. >>> >>> Bugs: >>> JDK-8041979 - >>> sun/jvmstat/monitor/MonitoredVm/CR6672135.java failing on all platforms >>> JDK-6461635 - >>> BasicTests.sh test fails intermittently. >>> >>> Webrev: >>> http://cr.openjdk.java.net/~aeriksso/6461635_8041979/webrev.00/ >>> >>> Regards, >>> Andreas >> > From kevin.walls at oracle.com Wed Sep 10 17:48:53 2014 From: kevin.walls at oracle.com (kevin.walls at oracle.com) Date: Wed, 10 Sep 2014 17:48:53 +0000 Subject: hg: jdk7u/jdk7u-dev/hotspot: 8049684: pstack crashes on java core dump Message-ID: <201409101748.s8AHmr2h021178@aojmv0008> Changeset: f0769f0382d7 Author: kevinw Date: 2014-07-21 10:40 +0100 URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/hotspot/rev/f0769f0382d7 8049684: pstack crashes on java core dump Reviewed-by: sundar, sspitsyn ! src/os/bsd/dtrace/libjvm_db.c ! src/os/solaris/dtrace/libjvm_db.c From ivan.gerasimov at oracle.com Thu Sep 11 10:02:42 2014 From: ivan.gerasimov at oracle.com (ivan.gerasimov at oracle.com) Date: Thu, 11 Sep 2014 10:02:42 +0000 Subject: hg: jdk7u/jdk7u-dev/jdk: 7010989: Duplicate closure of file descriptors leads to unexpected and incorrect closure of sockets Message-ID: <201409111002.s8BA2gA7024482@aojmv0008> Changeset: 22b4d5ebccb3 Author: igerasim Date: 2014-09-09 19:02 +0400 URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/jdk/rev/22b4d5ebccb3 7010989: Duplicate closure of file descriptors leads to unexpected and incorrect closure of sockets Reviewed-by: chegar ! src/windows/native/java/net/TwoStacksPlainDatagramSocketImpl.c ! src/windows/native/java/net/TwoStacksPlainSocketImpl.c ! src/windows/native/java/net/net_util_md.c From kevin.walls at oracle.com Fri Sep 12 08:33:11 2014 From: kevin.walls at oracle.com (Kevin Walls) Date: Fri, 12 Sep 2014 09:33:11 +0100 Subject: [7u-dev] Request for approval: 6642881: Improve performance of Class.getClassLoader() Message-ID: <5412AFC7.6050501@oracle.com> Hi, Please approve this backport into 7u of changes to speed up access to the classloader. There are hotspot and JDK components. Changes have been in jdk9 for nearly 2 months, these for 7u are almost identical to the 8u push which went in more recently. bug: https://bugs.openjdk.java.net/browse/JDK-6642881 webrevs for 7u: http://cr.openjdk.java.net/~kevinw/6642881/webrev.hotspot/ http://cr.openjdk.java.net/~kevinw/6642881/webrev.jdk/ Thanks Kevin From sean.coffey at oracle.com Fri Sep 12 08:35:45 2014 From: sean.coffey at oracle.com (=?ISO-8859-1?Q?Se=E1n_Coffey?=) Date: Fri, 12 Sep 2014 09:35:45 +0100 Subject: [7u-dev] Request for approval: 6642881: Improve performance of Class.getClassLoader() In-Reply-To: <5412AFC7.6050501@oracle.com> References: <5412AFC7.6050501@oracle.com> Message-ID: <5412B061.6020803@oracle.com> Approved. Please add the noreg-perf label to the master bug report. regards, Sean. On 12/09/2014 09:33, Kevin Walls wrote: > > Hi, > > Please approve this backport into 7u of changes to speed up access to > the classloader. There are hotspot and JDK components. Changes have > been in jdk9 for nearly 2 months, these for 7u are almost identical to > the 8u push which went in more recently. > > bug: > https://bugs.openjdk.java.net/browse/JDK-6642881 > > webrevs for 7u: > http://cr.openjdk.java.net/~kevinw/6642881/webrev.hotspot/ > http://cr.openjdk.java.net/~kevinw/6642881/webrev.jdk/ > > Thanks > Kevin > > From dmitry.markov at oracle.com Fri Sep 12 11:37:11 2014 From: dmitry.markov at oracle.com (dmitry markov) Date: Fri, 12 Sep 2014 15:37:11 +0400 Subject: [7u-dev] Request for approval 8048110: Using tables in JTextPane leads to infinite loop in FlowLayout.layoutRow Message-ID: <5412DAE7.5020203@oracle.com> Hello, Could you approve the straight back-port of the fix for 8048110 to jdk7u-dev (jdk7u80), please? The bug: https://bugs.openjdk.java.net/browse/JDK-8048110 The webrev: http://cr.openjdk.java.net/~dmarkov/8048110/jdk7u/webrev.00/ The review threads: http://mail.openjdk.java.net/pipermail/swing-dev/2014-August/003811.html http://mail.openjdk.java.net/pipermail/swing-dev/2014-September/003892.html The jdk9 changeset: http://hg.openjdk.java.net/jdk9/client/jdk/rev/c9b070f0a2e5 The jdk8u changeset: http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/eb4956a1974f Thanks, Dmitry From dmitry.markov at oracle.com Fri Sep 12 12:13:54 2014 From: dmitry.markov at oracle.com (dmitry markov) Date: Fri, 12 Sep 2014 16:13:54 +0400 Subject: [7u-dev] Request for approval 8015586: [macosx] Test closed/java/awt/print/PrinterJob/PrintToDir.java fails on MacOSX Message-ID: <5412E382.1070609@oracle.com> Hello, Could you approve the back-port of the fix for 8015586 to jdk7u-dev (7u80), please? I could not find the review thread for the original fix, but the changes apply cleanly. The bug: https://bugs.openjdk.java.net/browse/JDK-8015586 The webrev: http://cr.openjdk.java.net/~dmarkov/8015586/jdk7u/webrev.00/ - add to open repo http://cr.openjdk.java.net/~dmarkov/8015586/jdk7u/webrev.diff/ - diff with the previous version of the test The jdk9 changeset: http://hg.openjdk.java.net/jdk9/jdk9/jdk/rev/c9c945cea665 The jdk8u changeset: http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/c9c945cea665 Thanks, Dmitry From sean.coffey at oracle.com Fri Sep 12 12:38:49 2014 From: sean.coffey at oracle.com (=?UTF-8?B?U2XDoW4gQ29mZmV5?=) Date: Fri, 12 Sep 2014 13:38:49 +0100 Subject: [7u-dev] Request for approval 8048110: Using tables in JTextPane leads to infinite loop in FlowLayout.layoutRow In-Reply-To: <5412DAE7.5020203@oracle.com> References: <5412DAE7.5020203@oracle.com> Message-ID: <5412E959.8080603@oracle.com> Approved. regards, Sean. On 12/09/14 12:37, dmitry markov wrote: > Hello, > > Could you approve the straight back-port of the fix for 8048110 to > jdk7u-dev (jdk7u80), please? > > The bug: > https://bugs.openjdk.java.net/browse/JDK-8048110 > The webrev: > http://cr.openjdk.java.net/~dmarkov/8048110/jdk7u/webrev.00/ > The review threads: > http://mail.openjdk.java.net/pipermail/swing-dev/2014-August/003811.html > http://mail.openjdk.java.net/pipermail/swing-dev/2014-September/003892.html > > The jdk9 changeset: > http://hg.openjdk.java.net/jdk9/client/jdk/rev/c9b070f0a2e5 > The jdk8u changeset: > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/eb4956a1974f > > Thanks, > Dmitry From sean.coffey at oracle.com Fri Sep 12 12:40:15 2014 From: sean.coffey at oracle.com (=?UTF-8?B?U2XDoW4gQ29mZmV5?=) Date: Fri, 12 Sep 2014 13:40:15 +0100 Subject: [7u-dev] Request for approval 8015586: [macosx] Test closed/java/awt/print/PrinterJob/PrintToDir.java fails on MacOSX In-Reply-To: <5412E382.1070609@oracle.com> References: <5412E382.1070609@oracle.com> Message-ID: <5412E9AF.1080508@oracle.com> Approved. regards, Sean. On 12/09/14 13:13, dmitry markov wrote: > Hello, > > Could you approve the back-port of the fix for 8015586 to jdk7u-dev > (7u80), please? I could not find the review thread for the original > fix, but the changes apply cleanly. > > The bug: > https://bugs.openjdk.java.net/browse/JDK-8015586 > The webrev: > http://cr.openjdk.java.net/~dmarkov/8015586/jdk7u/webrev.00/ - add to > open repo > http://cr.openjdk.java.net/~dmarkov/8015586/jdk7u/webrev.diff/ - diff > with the previous version of the test > The jdk9 changeset: > http://hg.openjdk.java.net/jdk9/jdk9/jdk/rev/c9c945cea665 > The jdk8u changeset: > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/c9c945cea665 > > Thanks, > Dmitry From kevin.walls at oracle.com Fri Sep 12 13:39:34 2014 From: kevin.walls at oracle.com (kevin.walls at oracle.com) Date: Fri, 12 Sep 2014 13:39:34 +0000 Subject: hg: jdk7u/jdk7u-dev/jdk: 6642881: Improve performance of Class.getClassLoader() Message-ID: <201409121339.s8CDdZrv028639@aojmv0008> Changeset: ac93f5941f48 Author: coleenp Date: 2014-09-12 14:36 +0100 URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/jdk/rev/ac93f5941f48 6642881: Improve performance of Class.getClassLoader() Summary: Add classLoader to java/lang/Class instance for fast access Reviewed-by: fparain, lfoltan, alanb, mchung, dholmes, kevinw ! src/share/classes/java/lang/Class.java ! src/share/classes/java/lang/reflect/AccessibleObject.java ! src/share/javavm/export/jvm.h ! src/share/native/common/check_code.c ! src/share/native/java/lang/Class.c From aleksej.efimov at oracle.com Fri Sep 12 14:42:17 2014 From: aleksej.efimov at oracle.com (Aleksej Efimov) Date: Fri, 12 Sep 2014 18:42:17 +0400 Subject: [7u-dev] RFA: 8049343: (tz) Support tzdata2014g Message-ID: <54130649.7070200@oracle.com> Hello, Could you, please, approve a backport of tz2014g related changes to JDK7. The backport is not straight-forward one, there is no generic time zone names in JDK7 - because of that the tz names changes slightly differs from JDK8. Also, there was a temporary fix for JSR310 code and test failures in JDK8/9. This changes wasn't applied too - they are not relevant to JDK7. Because of such differences, I assume that its also a subject for review too (adding i18n-dev alias), Performed testing: JPRT testing: No tz related failures were observed in (jdk_util,jdk_other,jdk_text) JTREG testing: test/sun/util/resources/TimeZone test/sun/util/calendar test/closed/java/util/TimeZone test/java/util/TimeZone test/java/util/Calendar test/java/util/Formatter test/sun/util/resources/TimeZone test/closed/java/text/Format/DateFormat Thank you, Aleksej Webrev: http://cr.openjdk.java.net/~aefimov/8049343/7/webrev.00/ JDK9 review: http://mail.openjdk.java.net/pipermail/core-libs-dev/2014-September/028467.html JDK9 changeset: http://hg.openjdk.java.net/jdk9/dev/jdk/rev/f7de7da2eb58 JBS bug: https://bugs.openjdk.java.net/browse/JDK-8049343 From sean.coffey at oracle.com Fri Sep 12 15:22:14 2014 From: sean.coffey at oracle.com (=?UTF-8?B?U2XDoW4gQ29mZmV5?=) Date: Fri, 12 Sep 2014 16:22:14 +0100 Subject: [7u-dev] RFA: 8049343: (tz) Support tzdata2014g In-Reply-To: <54130649.7070200@oracle.com> References: <54130649.7070200@oracle.com> Message-ID: <54130FA6.2080304@oracle.com> Please get a reviewer to review the JDK7u changes. Approved but subject to review. regards, Sean. On 12/09/14 15:42, Aleksej Efimov wrote: > Hello, > > Could you, please, approve a backport of tz2014g related changes to > JDK7. The backport is not straight-forward one, there is no generic > time zone names in JDK7 - because of that the tz names changes > slightly differs from JDK8. > Also, there was a temporary fix for JSR310 code and test failures in > JDK8/9. This changes wasn't applied too - they are not relevant to JDK7. > Because of such differences, I assume that its also a subject for > review too (adding i18n-dev alias), > > Performed testing: > JPRT testing: No tz related failures were observed in > (jdk_util,jdk_other,jdk_text) > JTREG testing: test/sun/util/resources/TimeZone test/sun/util/calendar > test/closed/java/util/TimeZone test/java/util/TimeZone > test/java/util/Calendar > test/java/util/Formatter test/sun/util/resources/TimeZone > test/closed/java/text/Format/DateFormat > > Thank you, > Aleksej > > Webrev: http://cr.openjdk.java.net/~aefimov/8049343/7/webrev.00/ > JDK9 review: > http://mail.openjdk.java.net/pipermail/core-libs-dev/2014-September/028467.html > > JDK9 changeset: http://hg.openjdk.java.net/jdk9/dev/jdk/rev/f7de7da2eb58 > JBS bug: https://bugs.openjdk.java.net/browse/JDK-8049343 > From kevin.walls at oracle.com Fri Sep 12 15:31:07 2014 From: kevin.walls at oracle.com (kevin.walls at oracle.com) Date: Fri, 12 Sep 2014 15:31:07 +0000 Subject: hg: jdk7u/jdk7u-dev/hotspot: 6642881: Improve performance of Class.getClassLoader() Message-ID: <201409121531.s8CFV755016148@aojmv0008> Changeset: e47086077559 Author: coleenp Date: 2014-09-12 14:41 +0100 URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/hotspot/rev/e47086077559 6642881: Improve performance of Class.getClassLoader() Summary: Add classLoader to java/lang/Class instance for fast access Reviewed-by: fparain, lfoltan, alanb, mchung, dholmes, kevinw ! src/share/vm/classfile/classFileParser.cpp ! src/share/vm/classfile/javaClasses.cpp ! src/share/vm/classfile/javaClasses.hpp ! src/share/vm/classfile/vmSymbols.hpp From dmitry.markov at oracle.com Mon Sep 15 08:39:01 2014 From: dmitry.markov at oracle.com (dmitry.markov at oracle.com) Date: Mon, 15 Sep 2014 08:39:01 +0000 Subject: hg: jdk7u/jdk7u-dev/jdk: 8048110: Using tables in JTextPane leads to infinite loop in FlowLayout.layoutRow Message-ID: <201409150839.s8F8d14t007594@aojmv0008> Changeset: b4e500502985 Author: dmarkov Date: 2014-09-15 12:29 +0400 URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/jdk/rev/b4e500502985 8048110: Using tables in JTextPane leads to infinite loop in FlowLayout.layoutRow Reviewed-by: alexp, alexsch ! src/share/classes/javax/swing/text/FlowView.java ! src/share/classes/javax/swing/text/GlyphView.java ! src/share/classes/javax/swing/text/View.java + test/javax/swing/text/View/8048110/bug8048110.java From dmitry.markov at oracle.com Mon Sep 15 09:48:03 2014 From: dmitry.markov at oracle.com (dmitry.markov at oracle.com) Date: Mon, 15 Sep 2014 09:48:03 +0000 Subject: hg: jdk7u/jdk7u-dev/jdk: 8015586: [macosx] Test closed/java/awt/print/PrinterJob/PrintToDir.java fails on MacOSX Message-ID: <201409150948.s8F9m30w017298@aojmv0008> Changeset: 0d4f0a8a1ee5 Author: dmarkov Date: 2014-09-15 13:47 +0400 URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/jdk/rev/0d4f0a8a1ee5 8015586: [macosx] Test closed/java/awt/print/PrinterJob/PrintToDir.java fails on MacOSX Reviewed-by: prr, jchen ! src/macosx/classes/sun/lwawt/macosx/CPrinterJob.java ! src/share/classes/sun/print/RasterPrinterJob.java + test/java/awt/print/PrinterJob/PrintToDir.java From masayoshi.okutsu at oracle.com Tue Sep 16 09:10:44 2014 From: masayoshi.okutsu at oracle.com (Masayoshi Okutsu) Date: Tue, 16 Sep 2014 18:10:44 +0900 Subject: [7u-dev] RFA: 8049343: (tz) Support tzdata2014g In-Reply-To: <54130649.7070200@oracle.com> References: <54130649.7070200@oracle.com> Message-ID: <5417FE94.5050703@oracle.com> Looks good to me. Thanks, Masayoshi On 9/12/2014 11:42 PM, Aleksej Efimov wrote: > Hello, > > Could you, please, approve a backport of tz2014g related changes to > JDK7. The backport is not straight-forward one, there is no generic > time zone names in JDK7 - because of that the tz names changes > slightly differs from JDK8. > Also, there was a temporary fix for JSR310 code and test failures in > JDK8/9. This changes wasn't applied too - they are not relevant to JDK7. > Because of such differences, I assume that its also a subject for > review too (adding i18n-dev alias), > > Performed testing: > JPRT testing: No tz related failures were observed in > (jdk_util,jdk_other,jdk_text) > JTREG testing: test/sun/util/resources/TimeZone test/sun/util/calendar > test/closed/java/util/TimeZone test/java/util/TimeZone > test/java/util/Calendar > test/java/util/Formatter test/sun/util/resources/TimeZone > test/closed/java/text/Format/DateFormat > > Thank you, > Aleksej > > Webrev: http://cr.openjdk.java.net/~aefimov/8049343/7/webrev.00/ > JDK9 review: > http://mail.openjdk.java.net/pipermail/core-libs-dev/2014-September/028467.html > > JDK9 changeset: http://hg.openjdk.java.net/jdk9/dev/jdk/rev/f7de7da2eb58 > JBS bug: https://bugs.openjdk.java.net/browse/JDK-8049343 > From rjdkolb at gmail.com Wed Sep 17 06:03:32 2014 From: rjdkolb at gmail.com (Richard Kolb) Date: Wed, 17 Sep 2014 08:03:32 +0200 Subject: Compiling OpenJDK 7 on Arm Message-ID: Hello, I am one of the community members of AdoptOpenJDK. I've been able to build the OpenJDK 8 on my Arm (Raspberry pi) But since there is no configure, it seems the Makefile of 7 assumes x86. Can anyone tell me how to compile to target Arm (zero) and specify hard float. thanks, Richard. g++ -DLINUX -D_GNU_SOURCE -DIA32 -I/media/548da502-ebde-45c0-9ab2-de5e2431ee0b/home/jdk/jdk7u60/hotspot/src/share/vm/prims -I/media/548da502-ebde-45c0-9ab2-de5e2431ee0b/home/jdk/jdk7u60/hotspot/src/share/vm -I/media/548da502-ebde-45c0-9ab2-de5e2431ee0b/home/jdk/jdk7u60/hotspot/src/share/vm/precompiled -I/media/548da502-ebde-45c0-9ab2-de5e2431ee0b/home/jdk/jdk7u60/hotspot/src/cpu/x86/vm -I/media/548da502-ebde-45c0-9ab2-de5e2431ee0b/home/jdk/jdk7u60/hotspot/src/os_cpu/linux_x86/vm -I/media/548da502-ebde-45c0-9ab2-de5e2431ee0b/home/jdk/jdk7u60/hotspot/src/os/linux/vm -I/media/548da502-ebde-45c0-9ab2-de5e2431ee0b/home/jdk/jdk7u60/hotspot/src/os/posix/vm -I/media/548da502-ebde-45c0-9ab2-de5e2431ee0b/home/jdk/jdk7u60/hotspot/src/share/vm/adlc -I../generated -DASSERT -DTARGET_OS_FAMILY_linux -DTARGET_ARCH_x86 -DTARGET_ARCH_MODEL_x86_32 -DTARGET_OS_ARCH_linux_x86 -DTARGET_OS_ARCH_MODEL_linux_x86_32 -DTARGET_COMPILER_gcc -DCOMPILER2 -DCOMPILER1 -fno-rtti -fno-exceptions -D_REENTRANT -fcheck-new -fvisibility=hidden -m32 -march=i586 -pipe -Werror -g -c -o ../generated/adfiles/adlparse.o /media/548da502-ebde-45c0-9ab2-de5e2431ee0b/home/jdk/jdk7u60/hotspot/src/share/vm/adlc/adlparse.cpp Assembler messages: Error: unknown architecture `i586' Error: unrecognized option -march=i586 cc1plus: error: unrecognized command line option '-m32' cc1plus: error: bad value (i586) for -march switch make[6]: *** [../generated/adfiles/adlparse.o] Error 1 make[6]: Leaving directory `/media/548da502-ebde-45c0-9ab2-de5e2431ee0b/home/jdk/jdk7u60/build/linux-arm/hotspot/outputdir/linux_i486_compiler2/product' make[5]: *** [ad_stuff] Error 2 make[5]: Leaving directory `/media/548da502-ebde-45c0-9ab2-de5e2431ee0b/home/jdk/jdk7u60/build/linux-arm/hotspot/outputdir/linux_i486_compiler2/product' make[4]: *** [product] Error 2 make[4]: Leaving directory `/media/548da502-ebde-45c0-9ab2-de5e2431ee0b/home/jdk/jdk7u60/build/linux-arm/hotspot/outputdir' make[3]: *** [generic_build2] Error 2 make[3]: Leaving directory `/media/548da502-ebde-45c0-9ab2-de5e2431ee0b/home/jdk/jdk7u60/hotspot/make' make[2]: *** [product] Error 2 make[2]: Leaving directory `/media/548da502-ebde-45c0-9ab2-de5e2431ee0b/home/jdk/jdk7u60/hotspot/make' make[1]: *** [hotspot-build] Error 2 make[1]: Leaving directory `/media/548da502-ebde-45c0-9ab2-de5e2431ee0b/home/jdk/jdk7u60' make: *** [build_product_image] Error 2 From david.holmes at oracle.com Wed Sep 17 06:54:10 2014 From: david.holmes at oracle.com (David Holmes) Date: Wed, 17 Sep 2014 16:54:10 +1000 Subject: Compiling OpenJDK 7 on Arm In-Reply-To: References: Message-ID: <54193012.4060100@oracle.com> On 17/09/2014 4:03 PM, Richard Kolb wrote: > Hello, > > I am one of the community members of AdoptOpenJDK. > I've been able to build the OpenJDK 8 on my Arm (Raspberry pi) > > But since there is no configure, it seems the Makefile of 7 assumes x86. Not quite - it assumes you are targetting the platform you are building on. > Can anyone tell me how to compile to target Arm (zero) and specify hard > float. Try: make ARCH=arm ... for a start. Then you will need to pass the right args to your cross-compiler, for which EXTRA_CFLAGS and EXTRA_LDFLAGS can be used. NOt sure of the details of doing a zero build in OpenJDK 7 though. David > thanks, > Richard. > > > g++ -DLINUX -D_GNU_SOURCE -DIA32 > -I/media/548da502-ebde-45c0-9ab2-de5e2431ee0b/home/jdk/jdk7u60/hotspot/src/share/vm/prims > -I/media/548da502-ebde-45c0-9ab2-de5e2431ee0b/home/jdk/jdk7u60/hotspot/src/share/vm > -I/media/548da502-ebde-45c0-9ab2-de5e2431ee0b/home/jdk/jdk7u60/hotspot/src/share/vm/precompiled > -I/media/548da502-ebde-45c0-9ab2-de5e2431ee0b/home/jdk/jdk7u60/hotspot/src/cpu/x86/vm > -I/media/548da502-ebde-45c0-9ab2-de5e2431ee0b/home/jdk/jdk7u60/hotspot/src/os_cpu/linux_x86/vm > -I/media/548da502-ebde-45c0-9ab2-de5e2431ee0b/home/jdk/jdk7u60/hotspot/src/os/linux/vm > -I/media/548da502-ebde-45c0-9ab2-de5e2431ee0b/home/jdk/jdk7u60/hotspot/src/os/posix/vm > -I/media/548da502-ebde-45c0-9ab2-de5e2431ee0b/home/jdk/jdk7u60/hotspot/src/share/vm/adlc > -I../generated -DASSERT -DTARGET_OS_FAMILY_linux -DTARGET_ARCH_x86 > -DTARGET_ARCH_MODEL_x86_32 -DTARGET_OS_ARCH_linux_x86 > -DTARGET_OS_ARCH_MODEL_linux_x86_32 -DTARGET_COMPILER_gcc -DCOMPILER2 > -DCOMPILER1 -fno-rtti -fno-exceptions -D_REENTRANT -fcheck-new > -fvisibility=hidden -m32 -march=i586 -pipe -Werror -g -c -o > ../generated/adfiles/adlparse.o > /media/548da502-ebde-45c0-9ab2-de5e2431ee0b/home/jdk/jdk7u60/hotspot/src/share/vm/adlc/adlparse.cpp > > Assembler messages: > Error: unknown architecture `i586' > > Error: unrecognized option -march=i586 > cc1plus: error: unrecognized command line option '-m32' > cc1plus: error: bad value (i586) for -march switch > make[6]: *** [../generated/adfiles/adlparse.o] Error 1 > make[6]: Leaving directory > `/media/548da502-ebde-45c0-9ab2-de5e2431ee0b/home/jdk/jdk7u60/build/linux-arm/hotspot/outputdir/linux_i486_compiler2/product' > make[5]: *** [ad_stuff] Error 2 > make[5]: Leaving directory > `/media/548da502-ebde-45c0-9ab2-de5e2431ee0b/home/jdk/jdk7u60/build/linux-arm/hotspot/outputdir/linux_i486_compiler2/product' > make[4]: *** [product] Error 2 > make[4]: Leaving directory > `/media/548da502-ebde-45c0-9ab2-de5e2431ee0b/home/jdk/jdk7u60/build/linux-arm/hotspot/outputdir' > make[3]: *** [generic_build2] Error 2 > make[3]: Leaving directory > `/media/548da502-ebde-45c0-9ab2-de5e2431ee0b/home/jdk/jdk7u60/hotspot/make' > make[2]: *** [product] Error 2 > make[2]: Leaving directory > `/media/548da502-ebde-45c0-9ab2-de5e2431ee0b/home/jdk/jdk7u60/hotspot/make' > make[1]: *** [hotspot-build] Error 2 > make[1]: Leaving directory > `/media/548da502-ebde-45c0-9ab2-de5e2431ee0b/home/jdk/jdk7u60' > make: *** [build_product_image] Error 2 > From dmitry.markov at oracle.com Wed Sep 17 08:05:32 2014 From: dmitry.markov at oracle.com (dmitry markov) Date: Wed, 17 Sep 2014 12:05:32 +0400 Subject: [7u-dev] Request for approval 8051359: [headless] JPopupMenu creation in headless mode with JDK9b23 causes NPE Message-ID: <541940CC.5010101@oracle.com> Hello, Could you approve the straight back-port of the fix for 8051359 to jdk7u-dev (jdk7u80), please? The bug: https://bugs.openjdk.java.net/browse/JDK-8051359 The webrev: http://cr.openjdk.java.net/~dmarkov/8051359/webrev.00/ The review threads: http://mail.openjdk.java.net/pipermail/awt-dev/2014-August/008352.html The jdk9 changeset: http://hg.openjdk.java.net/jdk9/client/jdk/rev/eb2872ff0b83 The jdk8u changeset: http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/8375459f193f Thanks, Dmitry From sean.coffey at oracle.com Wed Sep 17 08:31:00 2014 From: sean.coffey at oracle.com (=?UTF-8?B?U2XDoW4gQ29mZmV5?=) Date: Wed, 17 Sep 2014 09:31:00 +0100 Subject: [7u-dev] Request for approval 8051359: [headless] JPopupMenu creation in headless mode with JDK9b23 causes NPE In-Reply-To: <541940CC.5010101@oracle.com> References: <541940CC.5010101@oracle.com> Message-ID: <541946C4.8040909@oracle.com> Approved. regards, Sean. On 17/09/2014 09:05, dmitry markov wrote: > Hello, > > Could you approve the straight back-port of the fix for 8051359 to > jdk7u-dev (jdk7u80), please? > > The bug: > https://bugs.openjdk.java.net/browse/JDK-8051359 > The webrev: > http://cr.openjdk.java.net/~dmarkov/8051359/webrev.00/ > The review threads: > http://mail.openjdk.java.net/pipermail/awt-dev/2014-August/008352.html > The jdk9 changeset: > http://hg.openjdk.java.net/jdk9/client/jdk/rev/eb2872ff0b83 > The jdk8u changeset: > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/8375459f193f > > Thanks, > Dmitry From dmitry.markov at oracle.com Wed Sep 17 09:00:06 2014 From: dmitry.markov at oracle.com (dmitry.markov at oracle.com) Date: Wed, 17 Sep 2014 09:00:06 +0000 Subject: hg: jdk7u/jdk7u-dev/jdk: 8051359: JPopupMenu creation in headless mode with JDK9b23 causes NPE Message-ID: <201409170900.s8H906Ox010502@aojmv0008> Changeset: ae7b0ff91866 Author: dmarkov Date: 2014-09-17 12:59 +0400 URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/jdk/rev/ae7b0ff91866 8051359: JPopupMenu creation in headless mode with JDK9b23 causes NPE Reviewed-by: serb, pchelko ! src/share/classes/sun/awt/SunToolkit.java From aleksej.efimov at oracle.com Wed Sep 17 09:23:02 2014 From: aleksej.efimov at oracle.com (aleksej.efimov at oracle.com) Date: Wed, 17 Sep 2014 09:23:02 +0000 Subject: hg: jdk7u/jdk7u-dev/jdk: 8049343: (tz) Support tzdata2014g Message-ID: <201409170923.s8H9N25V014532@aojmv0008> Changeset: fd488f5e328f Author: aefimov Date: 2014-09-12 14:23 +0400 URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/jdk/rev/fd488f5e328f 8049343: (tz) Support tzdata2014g Reviewed-by: mfang, okutsu ! make/sun/javazic/tzdata/VERSION ! make/sun/javazic/tzdata/africa ! make/sun/javazic/tzdata/antarctica ! make/sun/javazic/tzdata/asia ! make/sun/javazic/tzdata/australasia ! make/sun/javazic/tzdata/backward ! make/sun/javazic/tzdata/etcetera ! make/sun/javazic/tzdata/europe ! make/sun/javazic/tzdata/factory ! make/sun/javazic/tzdata/iso3166.tab ! make/sun/javazic/tzdata/leapseconds ! make/sun/javazic/tzdata/northamerica ! make/sun/javazic/tzdata/pacificnew ! make/sun/javazic/tzdata/southamerica ! make/sun/javazic/tzdata/systemv ! make/sun/javazic/tzdata/zone.tab ! src/share/classes/sun/util/resources/TimeZoneNames.java ! src/share/classes/sun/util/resources/TimeZoneNames_de.java ! src/share/classes/sun/util/resources/TimeZoneNames_es.java ! src/share/classes/sun/util/resources/TimeZoneNames_fr.java ! src/share/classes/sun/util/resources/TimeZoneNames_it.java ! src/share/classes/sun/util/resources/TimeZoneNames_ja.java ! src/share/classes/sun/util/resources/TimeZoneNames_ko.java ! src/share/classes/sun/util/resources/TimeZoneNames_pt_BR.java ! src/share/classes/sun/util/resources/TimeZoneNames_sv.java ! src/share/classes/sun/util/resources/TimeZoneNames_zh_CN.java ! src/share/classes/sun/util/resources/TimeZoneNames_zh_TW.java ! test/sun/util/resources/TimeZone/Bug6317929.java From sergey.gabdurakhmanov at oracle.com Wed Sep 17 10:49:35 2014 From: sergey.gabdurakhmanov at oracle.com (Sergey Gabdurakhmanov) Date: Wed, 17 Sep 2014 14:49:35 +0400 Subject: [7u-dev] Request for approval 8028280, 8038274: ParkEvent leak when running modified runThese which only loads classes Message-ID: <5419673F.3030800@oracle.com> Hello, Could you please approve the back-port of the fix for 8028280 and 8038274 to jdk7u-dev? Especially please check stubGenerator_sparc.cpp The bugs: https://bugs.openjdk.java.net/browse/JDK-8028280 https://bugs.openjdk.java.net/browse/JDK-8038274 The webrev: http://cr.openjdk.java.net/~sgabdura/8028280/webrev.00/ The review threads: 8028280 http://mail.openjdk.java.net/pipermail/hotspot-runtime-dev/2013-November/010131.html 8038274 http://mail.openjdk.java.net/pipermail/hotspot-dev/2014-February/012539.html The jdk8 changeset: 8028280 http://hg.openjdk.java.net/jdk8u/hs-dev/hotspot/rev/0e6af9b390af 8038274 http://hg.openjdk.java.net/jdk8u/hs-dev/hotspot/rev/8c6165f9ea8e BR, Sergey From sean.coffey at oracle.com Wed Sep 17 11:26:30 2014 From: sean.coffey at oracle.com (=?UTF-8?B?U2XDoW4gQ29mZmV5?=) Date: Wed, 17 Sep 2014 12:26:30 +0100 Subject: [7u-dev] Request for approval 8028280, 8038274: ParkEvent leak when running modified runThese which only loads classes In-Reply-To: <5419673F.3030800@oracle.com> References: <5419673F.3030800@oracle.com> Message-ID: <54196FE6.7070602@oracle.com> 1st prize goes to the hotspot team for failing to use noreg- labels! Please add them. Approved for jdk7u-dev but subject to peer code review. regards, Sean. On 17/09/14 11:49, Sergey Gabdurakhmanov wrote: > Hello, > > Could you please approve the back-port of the fix for 8028280 and > 8038274 to jdk7u-dev? > Especially please check stubGenerator_sparc.cpp > > The bugs: > https://bugs.openjdk.java.net/browse/JDK-8028280 > https://bugs.openjdk.java.net/browse/JDK-8038274 > The webrev: > http://cr.openjdk.java.net/~sgabdura/8028280/webrev.00/ > The review threads: > 8028280 > http://mail.openjdk.java.net/pipermail/hotspot-runtime-dev/2013-November/010131.html > > 8038274 > http://mail.openjdk.java.net/pipermail/hotspot-dev/2014-February/012539.html > > The jdk8 changeset: > 8028280 > http://hg.openjdk.java.net/jdk8u/hs-dev/hotspot/rev/0e6af9b390af > 8038274 > http://hg.openjdk.java.net/jdk8u/hs-dev/hotspot/rev/8c6165f9ea8e > > BR, > Sergey From gnu.andrew at redhat.com Wed Sep 17 17:32:24 2014 From: gnu.andrew at redhat.com (Andrew Hughes) Date: Wed, 17 Sep 2014 13:32:24 -0400 (EDT) Subject: [7u80] Request for approval for CR 4963723: Implement SHA-224 In-Reply-To: <64136507.24494470.1410973821008.JavaMail.zimbra@redhat.com> Message-ID: <155489540.24500766.1410975144648.JavaMail.zimbra@redhat.com> This is the first of three backports to 7u designed to retain SSL compatibility with servers implemented in other languages switching to larger key sizes (notably DH >=2048 in Apache 2.4.7 [0]). This patch is a per-requisite of the patch which brings NSA Suite B support to 7. It applies largely unchanged, bar the following: * Copyright header adjustment * Removal of change to java.security.spec.MGF1ParameterSpec to avoid introducing a new public variable. The SHA-224 variant is constructed directly in com.sun.crypto.provider.OAEPParameters instead. * A change to OAEPParameters is dropped as it was already incorporated in the backport of 7180907 & 8049480 (addition of SHA-224 to convertToStandardName) Bug: https://bugs.openjdk.java.net/browse/JDK-4963723 Webrev: http://cr.openjdk.java.net/~andrew/jdk7u/4963723/webrev.01/ [0] https://httpd.apache.org/docs/2.4/mod/mod_ssl.html Ok to push? Thanks, -- Andrew :) Free Java Software Engineer Red Hat, Inc. (http://www.redhat.com) PGP Key: 248BDC07 (https://keys.indymedia.org/) Fingerprint = EC5A 1F5E C0AD 1D15 8F1F 8F91 3B96 A578 248B DC07 From gnu.andrew at redhat.com Wed Sep 17 17:38:49 2014 From: gnu.andrew at redhat.com (Andrew Hughes) Date: Wed, 17 Sep 2014 13:38:49 -0400 (EDT) Subject: Compiling OpenJDK 7 on Arm In-Reply-To: References: Message-ID: <472768789.24502446.1410975529051.JavaMail.zimbra@redhat.com> ----- Original Message ----- > Hello, > > I am one of the community members of AdoptOpenJDK. > I've been able to build the OpenJDK 8 on my Arm (Raspberry pi) > > But since there is no configure, it seems the Makefile of 7 assumes x86. > > Can anyone tell me how to compile to target Arm (zero) and specify hard > float. > You need to pass ZERO_BUILD=true to make. Alternatively, you can use IcedTea [0] to build on ARM32 with ./configure ; make and it also includes a JIT. [0] http://bitly.com/it20502 > thanks, > Richard. > > > g++ -DLINUX -D_GNU_SOURCE -DIA32 > -I/media/548da502-ebde-45c0-9ab2-de5e2431ee0b/home/jdk/jdk7u60/hotspot/src/share/vm/prims > -I/media/548da502-ebde-45c0-9ab2-de5e2431ee0b/home/jdk/jdk7u60/hotspot/src/share/vm > -I/media/548da502-ebde-45c0-9ab2-de5e2431ee0b/home/jdk/jdk7u60/hotspot/src/share/vm/precompiled > -I/media/548da502-ebde-45c0-9ab2-de5e2431ee0b/home/jdk/jdk7u60/hotspot/src/cpu/x86/vm > -I/media/548da502-ebde-45c0-9ab2-de5e2431ee0b/home/jdk/jdk7u60/hotspot/src/os_cpu/linux_x86/vm > -I/media/548da502-ebde-45c0-9ab2-de5e2431ee0b/home/jdk/jdk7u60/hotspot/src/os/linux/vm > -I/media/548da502-ebde-45c0-9ab2-de5e2431ee0b/home/jdk/jdk7u60/hotspot/src/os/posix/vm > -I/media/548da502-ebde-45c0-9ab2-de5e2431ee0b/home/jdk/jdk7u60/hotspot/src/share/vm/adlc > -I../generated -DASSERT -DTARGET_OS_FAMILY_linux -DTARGET_ARCH_x86 > -DTARGET_ARCH_MODEL_x86_32 -DTARGET_OS_ARCH_linux_x86 > -DTARGET_OS_ARCH_MODEL_linux_x86_32 -DTARGET_COMPILER_gcc -DCOMPILER2 > -DCOMPILER1 -fno-rtti -fno-exceptions -D_REENTRANT -fcheck-new > -fvisibility=hidden -m32 -march=i586 -pipe -Werror -g -c -o > ../generated/adfiles/adlparse.o > /media/548da502-ebde-45c0-9ab2-de5e2431ee0b/home/jdk/jdk7u60/hotspot/src/share/vm/adlc/adlparse.cpp > > Assembler messages: > Error: unknown architecture `i586' > > Error: unrecognized option -march=i586 > cc1plus: error: unrecognized command line option '-m32' > cc1plus: error: bad value (i586) for -march switch > make[6]: *** [../generated/adfiles/adlparse.o] Error 1 > make[6]: Leaving directory > `/media/548da502-ebde-45c0-9ab2-de5e2431ee0b/home/jdk/jdk7u60/build/linux-arm/hotspot/outputdir/linux_i486_compiler2/product' > make[5]: *** [ad_stuff] Error 2 > make[5]: Leaving directory > `/media/548da502-ebde-45c0-9ab2-de5e2431ee0b/home/jdk/jdk7u60/build/linux-arm/hotspot/outputdir/linux_i486_compiler2/product' > make[4]: *** [product] Error 2 > make[4]: Leaving directory > `/media/548da502-ebde-45c0-9ab2-de5e2431ee0b/home/jdk/jdk7u60/build/linux-arm/hotspot/outputdir' > make[3]: *** [generic_build2] Error 2 > make[3]: Leaving directory > `/media/548da502-ebde-45c0-9ab2-de5e2431ee0b/home/jdk/jdk7u60/hotspot/make' > make[2]: *** [product] Error 2 > make[2]: Leaving directory > `/media/548da502-ebde-45c0-9ab2-de5e2431ee0b/home/jdk/jdk7u60/hotspot/make' > make[1]: *** [hotspot-build] Error 2 > make[1]: Leaving directory > `/media/548da502-ebde-45c0-9ab2-de5e2431ee0b/home/jdk/jdk7u60' > make: *** [build_product_image] Error 2 > -- Andrew :) Free Java Software Engineer Red Hat, Inc. (http://www.redhat.com) PGP Key: 248BDC07 (https://keys.indymedia.org/) Fingerprint = EC5A 1F5E C0AD 1D15 8F1F 8F91 3B96 A578 248B DC07 From rob.mckenna at oracle.com Wed Sep 17 21:17:51 2014 From: rob.mckenna at oracle.com (Rob McKenna) Date: Wed, 17 Sep 2014 22:17:51 +0100 Subject: [7u80] Request for approval for CR 4963723: Implement SHA-224 In-Reply-To: <155489540.24500766.1410975144648.JavaMail.zimbra@redhat.com> References: <155489540.24500766.1410975144648.JavaMail.zimbra@redhat.com> Message-ID: <5419FA7F.903@oracle.com> Hi Andrew, Sorry to be a pest, but given the scope of the change I'd feel more comfortable with an explicit codereview for the backport. -Rob On 17/09/14 18:32, Andrew Hughes wrote: > This is the first of three backports to 7u designed to retain SSL > compatibility with servers implemented in other languages switching > to larger key sizes (notably DH >=2048 in Apache 2.4.7 [0]). > > This patch is a per-requisite of the patch which brings NSA Suite B > support to 7. It applies largely unchanged, bar the following: > > * Copyright header adjustment > * Removal of change to java.security.spec.MGF1ParameterSpec to avoid > introducing a new public variable. The SHA-224 variant is constructed > directly in com.sun.crypto.provider.OAEPParameters instead. > * A change to OAEPParameters is dropped as it was already incorporated > in the backport of 7180907 & 8049480 (addition of SHA-224 to > convertToStandardName) > > Bug: https://bugs.openjdk.java.net/browse/JDK-4963723 > Webrev: http://cr.openjdk.java.net/~andrew/jdk7u/4963723/webrev.01/ > > [0] https://httpd.apache.org/docs/2.4/mod/mod_ssl.html > > Ok to push? > > Thanks, From gnu.andrew at redhat.com Wed Sep 17 23:21:52 2014 From: gnu.andrew at redhat.com (Andrew Hughes) Date: Wed, 17 Sep 2014 19:21:52 -0400 (EDT) Subject: [7u80] Request for approval for CR 4963723: Implement SHA-224 In-Reply-To: <5419FA7F.903@oracle.com> References: <155489540.24500766.1410975144648.JavaMail.zimbra@redhat.com> <5419FA7F.903@oracle.com> Message-ID: <965519084.24587907.1410996112530.JavaMail.zimbra@redhat.com> ----- Original Message ----- > Hi Andrew, > > Sorry to be a pest, but given the scope of the change I'd feel more > comfortable with an explicit codereview for the backport. > > -Rob > > On 17/09/14 18:32, Andrew Hughes wrote: > > This is the first of three backports to 7u designed to retain SSL > > compatibility with servers implemented in other languages switching > > to larger key sizes (notably DH >=2048 in Apache 2.4.7 [0]). > > > > This patch is a per-requisite of the patch which brings NSA Suite B > > support to 7. It applies largely unchanged, bar the following: > > > > * Copyright header adjustment > > * Removal of change to java.security.spec.MGF1ParameterSpec to avoid > > introducing a new public variable. The SHA-224 variant is constructed > > directly in com.sun.crypto.provider.OAEPParameters instead. > > * A change to OAEPParameters is dropped as it was already incorporated > > in the backport of 7180907 & 8049480 (addition of SHA-224 to > > convertToStandardName) > > > > Bug: https://bugs.openjdk.java.net/browse/JDK-4963723 > > Webrev: http://cr.openjdk.java.net/~andrew/jdk7u/4963723/webrev.01/ > > > > [0] https://httpd.apache.org/docs/2.4/mod/mod_ssl.html > > > > Ok to push? > > > > Thanks, > > Which is what I asked for, no? If I wasn't waiting on a review first, I'd have pushed the change. -- Andrew :) Free Java Software Engineer Red Hat, Inc. (http://www.redhat.com) PGP Key: 248BDC07 (https://keys.indymedia.org/) Fingerprint = EC5A 1F5E C0AD 1D15 8F1F 8F91 3B96 A578 248B DC07 From rob.mckenna at oracle.com Wed Sep 17 23:31:29 2014 From: rob.mckenna at oracle.com (Rob McKenna) Date: Thu, 18 Sep 2014 00:31:29 +0100 Subject: [7u80] Request for approval for CR 4963723: Implement SHA-224 In-Reply-To: <965519084.24587907.1410996112530.JavaMail.zimbra@redhat.com> References: <155489540.24500766.1410975144648.JavaMail.zimbra@redhat.com> <5419FA7F.903@oracle.com> <965519084.24587907.1410996112530.JavaMail.zimbra@redhat.com> Message-ID: <541A19D1.40606@oracle.com> Code changes generally require two approvals: codereview, performed by a reviewer, (in this case from security-dev) and push approval, performed by a gatekeeper. Given your email template matches the push approval template I understood that you intended the latter. Generally speaking codereview requests would say "Request for review" as opposed to "Request for approval" so a reviewer could overlook your mail if you intended the former. -Rob On 18/09/14 00:21, Andrew Hughes wrote: > > ----- Original Message ----- >> Hi Andrew, >> >> Sorry to be a pest, but given the scope of the change I'd feel more >> comfortable with an explicit codereview for the backport. >> >> -Rob >> >> On 17/09/14 18:32, Andrew Hughes wrote: >>> This is the first of three backports to 7u designed to retain SSL >>> compatibility with servers implemented in other languages switching >>> to larger key sizes (notably DH >=2048 in Apache 2.4.7 [0]). >>> >>> This patch is a per-requisite of the patch which brings NSA Suite B >>> support to 7. It applies largely unchanged, bar the following: >>> >>> * Copyright header adjustment >>> * Removal of change to java.security.spec.MGF1ParameterSpec to avoid >>> introducing a new public variable. The SHA-224 variant is constructed >>> directly in com.sun.crypto.provider.OAEPParameters instead. >>> * A change to OAEPParameters is dropped as it was already incorporated >>> in the backport of 7180907 & 8049480 (addition of SHA-224 to >>> convertToStandardName) >>> >>> Bug: https://bugs.openjdk.java.net/browse/JDK-4963723 >>> Webrev: http://cr.openjdk.java.net/~andrew/jdk7u/4963723/webrev.01/ >>> >>> [0] https://httpd.apache.org/docs/2.4/mod/mod_ssl.html >>> >>> Ok to push? >>> >>> Thanks, >> > Which is what I asked for, no? > > If I wasn't waiting on a review first, I'd have pushed the change. From ivan.gerasimov at oracle.com Thu Sep 18 07:40:55 2014 From: ivan.gerasimov at oracle.com (ivan.gerasimov at oracle.com) Date: Thu, 18 Sep 2014 07:40:55 +0000 Subject: hg: jdk7u/jdk7u-dev/jdk: 8055949: ByteArrayOutputStream capacity should be maximal array size permitted by VM Message-ID: <201409180740.s8I7ex4d011827@aojmv0008> Changeset: 6b53b16c1641 Author: igerasim Date: 2014-09-17 23:52 +0400 URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/jdk/rev/6b53b16c1641 8055949: ByteArrayOutputStream capacity should be maximal array size permitted by VM Summary: Try to resize to "well-known" hotspot max array size first. Reviewed-by: alanb, mduigou ! src/share/classes/java/io/ByteArrayOutputStream.java + test/java/io/ByteArrayOutputStream/MaxCapacity.java From rjdkolb at gmail.com Fri Sep 19 09:53:08 2014 From: rjdkolb at gmail.com (Richard Kolb) Date: Fri, 19 Sep 2014 11:53:08 +0200 Subject: Pre-compiled headers with OpenJDK Message-ID: Hello, I am trying to compile the OpenJDK 7u60 and because I have a newer version of GCC I am hitting a pre-compiled issue. How do I properly set USE_PRECOMPILED_HEADER ? I have tried 'make USE_PRECOMPILED_HEADER=0' 'export USE_PRECOMPILED_HEADER=0' and even hacking the gcc makefile with no effect. thanks, Richard. From david.simms at oracle.com Fri Sep 19 10:07:15 2014 From: david.simms at oracle.com (David Simms) Date: Fri, 19 Sep 2014 12:07:15 +0200 Subject: [7u-dev] Request for approval 8028280, 8038274: ParkEvent leak when running modified runThese which only loads classes In-Reply-To: <5419673F.3030800@oracle.com> References: <5419673F.3030800@oracle.com> Message-ID: <541C0053.6060701@oracle.com> Looks good On 17/09/2014 12:49 p.m., Sergey Gabdurakhmanov wrote: > Hello, > > Could you please approve the back-port of the fix for 8028280 and > 8038274 to jdk7u-dev? > Especially please check stubGenerator_sparc.cpp > > The bugs: > https://bugs.openjdk.java.net/browse/JDK-8028280 > https://bugs.openjdk.java.net/browse/JDK-8038274 > The webrev: > http://cr.openjdk.java.net/~sgabdura/8028280/webrev.00/ > The review threads: > 8028280 > http://mail.openjdk.java.net/pipermail/hotspot-runtime-dev/2013-November/010131.html > > 8038274 > http://mail.openjdk.java.net/pipermail/hotspot-dev/2014-February/012539.html > > The jdk8 changeset: > 8028280 > http://hg.openjdk.java.net/jdk8u/hs-dev/hotspot/rev/0e6af9b390af > 8038274 > http://hg.openjdk.java.net/jdk8u/hs-dev/hotspot/rev/8c6165f9ea8e > > BR, > Sergey From david.holmes at oracle.com Fri Sep 19 10:48:54 2014 From: david.holmes at oracle.com (David Holmes) Date: Fri, 19 Sep 2014 20:48:54 +1000 Subject: Pre-compiled headers with OpenJDK In-Reply-To: References: Message-ID: <541C0A16.7040703@oracle.com> On 19/09/2014 7:53 PM, Richard Kolb wrote: > Hello, > > I am trying to compile the OpenJDK 7u60 and because I have a newer version > of GCC I am hitting a pre-compiled issue. > > How do I properly set USE_PRECOMPILED_HEADER ? > > I have tried 'make USE_PRECOMPILED_HEADER=0' That should be all that is needed. What are you seeing? David > 'export USE_PRECOMPILED_HEADER=0' and even hacking the gcc makefile with no effect. > > > thanks, > Richard. > From rjdkolb at gmail.com Fri Sep 19 10:54:20 2014 From: rjdkolb at gmail.com (Richard Kolb) Date: Fri, 19 Sep 2014 12:54:20 +0200 Subject: Pre-compiled headers with OpenJDK In-Reply-To: <541C0A16.7040703@oracle.com> References: <541C0A16.7040703@oracle.com> Message-ID: On 19 September 2014 12:48, David Holmes wrote: > On 19/09/2014 7:53 PM, Richard Kolb wrote: > >> Hello, >> >> I am trying to compile the OpenJDK 7u60 and because I have a newer version >> of GCC I am hitting a pre-compiled issue. >> >> How do I properly set USE_PRECOMPILED_HEADER ? >> >> I have tried 'make USE_PRECOMPILED_HEADER=0' >> > > That should be all that is needed. > > What are you seeing? > It reports : hotspot/src/share/vm/prims/jni.h:876:13: note: the mangling of 'va_list' has changed in GCC 4.4 make[6]: *** [precompiled.hpp.gch] Error 1 regards, Richard. From david.holmes at oracle.com Fri Sep 19 12:01:34 2014 From: david.holmes at oracle.com (David Holmes) Date: Fri, 19 Sep 2014 22:01:34 +1000 Subject: Pre-compiled headers with OpenJDK In-Reply-To: References: <541C0A16.7040703@oracle.com> Message-ID: <541C1B1E.80400@oracle.com> On 19/09/2014 8:54 PM, Richard Kolb wrote: > > > On 19 September 2014 12:48, David Holmes > wrote: > > On 19/09/2014 7:53 PM, Richard Kolb wrote: > > Hello, > > I am trying to compile the OpenJDK 7u60 and because I have a > newer version > of GCC I am hitting a pre-compiled issue. > > How do I properly set USE_PRECOMPILED_HEADER ? > > I have tried 'make USE_PRECOMPILED_HEADER=0' > > > That should be all that is needed. > > What are you seeing? > > > It reports : > > hotspot/src/share/vm/prims/jni.h:876:13: note: the mangling of 'va_list' > has changed in GCC 4.4 > make[6]: *** [precompiled.hpp.gch] Error 1 Look further up the log - the "mangling" message is not an error. David > > > regards, > Richard. From rjdkolb at gmail.com Fri Sep 19 12:10:03 2014 From: rjdkolb at gmail.com (Richard Kolb) Date: Fri, 19 Sep 2014 14:10:03 +0200 Subject: Pre-compiled headers with OpenJDK In-Reply-To: <541C1B1E.80400@oracle.com> References: <541C0A16.7040703@oracle.com> <541C1B1E.80400@oracle.com> Message-ID: On 19 Sep 2014 14:01, "David Holmes" wrote: > > On 19/09/2014 8:54 PM, Richard Kolb wrote: >> >> >> >> On 19 September 2014 12:48, David Holmes > > wrote: >> >> On 19/09/2014 7:53 PM, Richard Kolb wrote: >> >> Hello, >> >> I am trying to compile the OpenJDK 7u60 and because I have a >> newer version >> of GCC I am hitting a pre-compiled issue. >> >> How do I properly set USE_PRECOMPILED_HEADER ? >> >> I have tried 'make USE_PRECOMPILED_HEADER=0' >> >> >> That should be all that is needed. >> >> What are you seeing? >> >> >> It reports : >> >> hotspot/src/share/vm/prims/jni.h:876:13: note: the mangling of 'va_list' >> has changed in GCC 4.4 >> make[6]: *** [precompiled.hpp.gch] Error 1 > > > Look further up the log - the "mangling" message is not an error. Thanks David, will look again. Regards, Richard. From daniel.daugherty at oracle.com Fri Sep 19 15:06:10 2014 From: daniel.daugherty at oracle.com (Daniel D. Daugherty) Date: Fri, 19 Sep 2014 09:06:10 -0600 Subject: [7u-dev] Request for approval 8028280, 8038274: ParkEvent leak when running modified runThese which only loads classes In-Reply-To: <54196FE6.7070602@oracle.com> References: <5419673F.3030800@oracle.com> <54196FE6.7070602@oracle.com> Message-ID: <541C4662.6020603@oracle.com> On 9/17/14 5:26 AM, Se?n Coffey wrote: > 1st prize goes to the hotspot team for failing to use noreg- labels! > Please add them. Yup. Probably has something to do with the fact that historically most HotSpot regression tests were created using the Tonga test harness and were handled completely differently than JavaTest/JTREG tests. However, that is changing and a majority of new HotSpot regression tests are written using JavaTest/JTREG... > Approved for jdk7u-dev but subject to peer code review. src/cpu/sparc/vm/stubGenerator_sparc.cpp Not in the JDK8u version, but I can see why the change is here. src/os/bsd/vm/os_bsd.cpp No comments. src/os/linux/vm/os_linux.cpp No comments. src/os/solaris/vm/os_solaris.cpp No comments. src/os/windows/vm/os_windows.cpp No comments. src/share/vm/runtime/objectMonitor.cpp Thanks for also backporting this one. src/share/vm/runtime/os.hpp No comments. src/share/vm/runtime/park.cpp No comments. src/share/vm/runtime/thread.cpp No comments. Thumbs up! Dan > > regards, > Sean. > > On 17/09/14 11:49, Sergey Gabdurakhmanov wrote: >> Hello, >> >> Could you please approve the back-port of the fix for 8028280 and >> 8038274 to jdk7u-dev? >> Especially please check stubGenerator_sparc.cpp >> >> The bugs: >> https://bugs.openjdk.java.net/browse/JDK-8028280 >> https://bugs.openjdk.java.net/browse/JDK-8038274 >> The webrev: >> http://cr.openjdk.java.net/~sgabdura/8028280/webrev.00/ >> The review threads: >> 8028280 >> http://mail.openjdk.java.net/pipermail/hotspot-runtime-dev/2013-November/010131.html >> >> 8038274 >> http://mail.openjdk.java.net/pipermail/hotspot-dev/2014-February/012539.html >> >> The jdk8 changeset: >> 8028280 >> http://hg.openjdk.java.net/jdk8u/hs-dev/hotspot/rev/0e6af9b390af >> 8038274 >> http://hg.openjdk.java.net/jdk8u/hs-dev/hotspot/rev/8c6165f9ea8e >> >> BR, >> Sergey > > > From sean.coffey at oracle.com Fri Sep 19 16:34:53 2014 From: sean.coffey at oracle.com (=?ISO-8859-1?Q?Se=E1n_Coffey?=) Date: Fri, 19 Sep 2014 17:34:53 +0100 Subject: [7u-dev] Request for approval : 8057813: Alterations to jdk_security3 test target Message-ID: <541C5B2D.7090800@oracle.com> Looking to push this test clean up to jdk7u-dev. webrev : http://cr.openjdk.java.net/~coffeys/webrev.7u.8057813/webrev/ bug report : https://bugs.openjdk.java.net/browse/JDK-8057813 review thread : http://mail.openjdk.java.net/pipermail/security-dev/2014-September/011229.html regards, Sean. From naoto.sato at oracle.com Fri Sep 19 16:48:42 2014 From: naoto.sato at oracle.com (Naoto Sato) Date: Fri, 19 Sep 2014 09:48:42 -0700 Subject: [7u-dev] Request for approval : 8057813: Alterations to jdk_security3 test target In-Reply-To: <541C5B2D.7090800@oracle.com> References: <541C5B2D.7090800@oracle.com> Message-ID: <541C5E6A.3050509@oracle.com> Approved. Naoto On 9/19/14, 9:34 AM, Se?n Coffey wrote: > Looking to push this test clean up to jdk7u-dev. > > webrev : http://cr.openjdk.java.net/~coffeys/webrev.7u.8057813/webrev/ > bug report : https://bugs.openjdk.java.net/browse/JDK-8057813 > review thread : > http://mail.openjdk.java.net/pipermail/security-dev/2014-September/011229.html > > > regards, > Sean. From sean.coffey at oracle.com Fri Sep 19 16:51:54 2014 From: sean.coffey at oracle.com (sean.coffey at oracle.com) Date: Fri, 19 Sep 2014 16:51:54 +0000 Subject: hg: jdk7u/jdk7u-dev: 8057813: Alterations to jdk_security3 test target Message-ID: <201409191651.s8JGptx1024070@aojmv0008> Changeset: ff44294a2be6 Author: coffeys Date: 2014-09-19 17:49 +0100 URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/rev/ff44294a2be6 8057813: Alterations to jdk_security3 test target Reviewed-by: mullan ! make/jprt.properties ! test/Makefile From sean.coffey at oracle.com Fri Sep 19 16:52:28 2014 From: sean.coffey at oracle.com (sean.coffey at oracle.com) Date: Fri, 19 Sep 2014 16:52:28 +0000 Subject: hg: jdk7u/jdk7u-dev/jdk: 8057813: Alterations to jdk_security3 test target Message-ID: <201409191652.s8JGqShD024195@aojmv0008> Changeset: 056ef4238bb4 Author: coffeys Date: 2014-09-19 17:49 +0100 URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/jdk/rev/056ef4238bb4 8057813: Alterations to jdk_security3 test target Reviewed-by: mullan ! make/jprt.properties ! test/Makefile From srinivasan.raghavan at oracle.com Mon Sep 22 05:01:19 2014 From: srinivasan.raghavan at oracle.com (srinivasan raghavan) Date: Mon, 22 Sep 2014 10:31:19 +0530 Subject: Reminder [7u-dev] Request for Approval and Review: 8057008: [TEST_BUG] Test java/awt/Focus/SortingFPT/JDK8048887.java fails with compilation error In-Reply-To: <5404A35D.8070208@oracle.com> References: <54049FF0.2060307@oracle.com> <5404A1FC.6090905@oracle.com> <5404A35D.8070208@oracle.com> Message-ID: <541FAD1F.3010602@oracle.com> Hi all Please help me in reviewing the fix for the issue -Thanks Srinivasan Raghavan On 9/1/2014 10:18 PM, Se?n Coffey wrote: > Looks good to me Srinivasan but you'll have to get an official 7u > reviewer. > > You could append the bug ID to the @bug tags and perhaps add > noreg-self to bug labels. > No excuse for pushing testcases that fail to compile..that's a hint > for the original 7u80 contributor ;) > > Approved subject to review. > > regards, > Sean. > > On 01/09/14 17:42, srinivasan raghavan wrote: >> Hello , >> >> Apologies for giving wrong web-rev link i am updating with the new one >> >> The webrev is http://cr.openjdk.java.net/~kshefov/8057008/webrev.00/ >> >> -Thanks >> Srinivasan Raghavan >> On 9/1/2014 10:03 PM, srinivasan raghavan wrote: >>> Hello , >>> >>> Please review a fix for the issue >>> >>> 8057008 [TEST_BUG] Test java/awt/Focus/SortingFPT/JDK8048887.java >>> fails with compilation error >>> >>> Test bug fix >>> >>> https://bugs.openjdk.java.net/browse/JDK-8057008 >>> >>> The webrev is http://cr.openjdk.java.net/~kshefov/8031422/webrev.00/ >>> >>> Thanks >>> -Srinivasan Raghavan >> > From gnu.andrew at redhat.com Mon Sep 22 14:25:41 2014 From: gnu.andrew at redhat.com (Andrew Hughes) Date: Mon, 22 Sep 2014 10:25:41 -0400 (EDT) Subject: [7u80] Request for review for CR 4963723: Implement SHA-224 In-Reply-To: <541A19D1.40606@oracle.com> References: <155489540.24500766.1410975144648.JavaMail.zimbra@redhat.com> <5419FA7F.903@oracle.com> <965519084.24587907.1410996112530.JavaMail.zimbra@redhat.com> <541A19D1.40606@oracle.com> Message-ID: <43988642.26705122.1411395941534.JavaMail.zimbra@redhat.com> ----- Original Message ----- > Code changes generally require two approvals: codereview, performed by a > reviewer, (in this case from security-dev) and push approval, performed > by a gatekeeper. Given your email template matches the push approval > template I understood that you intended the latter. Generally speaking > codereview requests would say "Request for review" as opposed to > "Request for approval" so a reviewer could overlook your mail if you > intended the former. > > -Rob > > On 18/09/14 00:21, Andrew Hughes wrote: > > > > ----- Original Message ----- > >> Hi Andrew, > >> > >> Sorry to be a pest, but given the scope of the change I'd feel more > >> comfortable with an explicit codereview for the backport. > >> > >> -Rob > >> > >> On 17/09/14 18:32, Andrew Hughes wrote: > >>> This is the first of three backports to 7u designed to retain SSL > >>> compatibility with servers implemented in other languages switching > >>> to larger key sizes (notably DH >=2048 in Apache 2.4.7 [0]). > >>> > >>> This patch is a per-requisite of the patch which brings NSA Suite B > >>> support to 7. It applies largely unchanged, bar the following: > >>> > >>> * Copyright header adjustment > >>> * Removal of change to java.security.spec.MGF1ParameterSpec to avoid > >>> introducing a new public variable. The SHA-224 variant is constructed > >>> directly in com.sun.crypto.provider.OAEPParameters instead. > >>> * A change to OAEPParameters is dropped as it was already incorporated > >>> in the backport of 7180907 & 8049480 (addition of SHA-224 to > >>> convertToStandardName) > >>> > >>> Bug: https://bugs.openjdk.java.net/browse/JDK-4963723 > >>> Webrev: http://cr.openjdk.java.net/~andrew/jdk7u/4963723/webrev.01/ > >>> > >>> [0] https://httpd.apache.org/docs/2.4/mod/mod_ssl.html > >>> > >>> Ok to push? > >>> > >>> Thanks, > >> > > Which is what I asked for, no? > > > > If I wasn't waiting on a review first, I'd have pushed the change. > > This was the only applicable template on: http://openjdk.java.net/projects/jdk7u/ Anyway, now including security-dev for review. -- Andrew :) Free Java Software Engineer Red Hat, Inc. (http://www.redhat.com) PGP Key: 248BDC07 (https://keys.indymedia.org/) Fingerprint = EC5A 1F5E C0AD 1D15 8F1F 8F91 3B96 A578 248B DC07 From sergey.gabdurakhmanov at oracle.com Tue Sep 23 07:14:33 2014 From: sergey.gabdurakhmanov at oracle.com (Sergey Gabdurakhmanov) Date: Tue, 23 Sep 2014 11:14:33 +0400 Subject: [7u-dev] Request for approval 8057564: JVM hangs at getAgentProperties after attaching to VM with lower integrity Message-ID: <54211DD9.6040608@oracle.com> Hello, Could you please approve the back-port of the fix for 8057564 to jdk7u-dev? Backport to JDK8 I'll prepare in nearest time. The bug: https://bugs.openjdk.java.net/browse/JDK-8057564 The webrev: http://cr.openjdk.java.net/~sgabdura/8057564/webrev.jdk7/ The review thread: http://mail.openjdk.java.net/pipermail/serviceability-dev/2014-September/015611.html The jdk9 changeset: http://hg.openjdk.java.net/jdk9/dev/jdk/rev/71c9aa685da5 BR, Sergey From sean.coffey at oracle.com Tue Sep 23 08:19:09 2014 From: sean.coffey at oracle.com (=?UTF-8?B?U2XDoW4gQ29mZmV5?=) Date: Tue, 23 Sep 2014 09:19:09 +0100 Subject: [7u-dev] Request for approval 8057564: JVM hangs at getAgentProperties after attaching to VM with lower integrity In-Reply-To: <54211DD9.6040608@oracle.com> References: <54211DD9.6040608@oracle.com> Message-ID: <54212CFD.4020301@oracle.com> Sergey, can you make the bug report public ? Please add the information from the code review thread into the description of the bug. Can you also create an 8u40 backport record so that it's not forgotten. Approved. regards, Sean. On 23/09/2014 08:14, Sergey Gabdurakhmanov wrote: > Hello, > > Could you please approve the back-port of the fix for 8057564 to > jdk7u-dev? > Backport to JDK8 I'll prepare in nearest time. > > The bug: > https://bugs.openjdk.java.net/browse/JDK-8057564 > The webrev: > http://cr.openjdk.java.net/~sgabdura/8057564/webrev.jdk7/ > The review thread: > http://mail.openjdk.java.net/pipermail/serviceability-dev/2014-September/015611.html > > The jdk9 changeset: > http://hg.openjdk.java.net/jdk9/dev/jdk/rev/71c9aa685da5 > > BR, > Sergey From sergey.gabdurakhmanov at oracle.com Tue Sep 23 08:29:27 2014 From: sergey.gabdurakhmanov at oracle.com (sergey.gabdurakhmanov at oracle.com) Date: Tue, 23 Sep 2014 08:29:27 +0000 Subject: hg: jdk7u/jdk7u-dev/jdk: 8057564: JVM hangs at getAgentProperties after attaching to VM with lower Message-ID: <201409230829.s8N8TR6b002838@aojmv0008> Changeset: 9ac8a27d7d19 Author: sgabdura Date: 2014-09-23 09:06 +0200 URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/jdk/rev/9ac8a27d7d19 8057564: JVM hangs at getAgentProperties after attaching to VM with lower Summary: Create custom Security Descriptor for Named Pipe. Reviewed-by: mgronlun, dsamersoff, uta ! src/windows/native/sun/tools/attach/WindowsVirtualMachine.c From sergey.gabdurakhmanov at oracle.com Tue Sep 23 09:10:23 2014 From: sergey.gabdurakhmanov at oracle.com (sergey.gabdurakhmanov at oracle.com) Date: Tue, 23 Sep 2014 09:10:23 +0000 Subject: hg: jdk7u/jdk7u-dev/hotspot: 2 new changesets Message-ID: <201409230910.s8N9ANn5009843@aojmv0008> Changeset: 34aea5177b9c Author: sgabdura Date: 2014-09-17 10:52 +0200 URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/hotspot/rev/34aea5177b9c 8028280: ParkEvent leak when running modified runThese which only loads classes Summary: Use spin lock to manage ParkEvent and PlatformEvent free lists. Reviewed-by: dsimms, dholmes, fparain ! src/cpu/sparc/vm/stubGenerator_sparc.cpp ! src/os/bsd/vm/os_bsd.cpp ! src/os/linux/vm/os_linux.cpp ! src/os/solaris/vm/os_solaris.cpp ! src/os/windows/vm/os_windows.cpp ! src/share/vm/runtime/os.hpp ! src/share/vm/runtime/park.cpp ! src/share/vm/runtime/thread.cpp Changeset: c296492ba8b4 Author: sgabdura Date: 2014-09-17 11:30 +0200 URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/hotspot/rev/c296492ba8b4 8038274: update 8u fix for 8028073 now that 8028280 is backported to 8u Reviewed-by: dcubed, coleenp, sspitsyn ! src/share/vm/runtime/objectMonitor.cpp From vladimir.kempik at oracle.com Tue Sep 23 10:55:03 2014 From: vladimir.kempik at oracle.com (Vladimir Kempik) Date: Tue, 23 Sep 2014 14:55:03 +0400 Subject: Request for Approval: 8012941, 8028159 and 8043413 Message-ID: <54215187.6060606@oracle.com> Hello! I'd like to get a approval to push these backports of 8012941, 8028159 and 8043413 into jdk7. I need to backport all three fixes to fix customer's issue. First one is a fix for c1, second one extends fix to c2, third one fixes bug in second fix. The webrevs for jdk7: 8012941 - http://cr.openjdk.java.net/~vkempik/8012941/webrev.00/ 8028159 - http://cr.openjdk.java.net/~vkempik/8028159/webrev.00/ 8043413 - http://cr.openjdk.java.net/~vkempik/8043413/webrev.00/ Review thread - http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/2014-September/015657.html 8012941 Bug: https://bugs.openjdk.java.net/browse/JDK-8012941 Jdk8 changeset: http://hg.openjdk.java.net/jdk8/jdk8/hotspot/rev/0d1661d63d70 8028159 Bug: https://bugs.openjdk.java.net/browse/JDK-8028159 Jdk8 changeset: http://hg.openjdk.java.net/jdk8/jdk8/hotspot/rev/e74074c34312 8043413 Bug: https://bugs.openjdk.java.net/browse/JDK-8043413 Jdk8 changeset: http://hg.openjdk.java.net/jdk8u/jdk8u/hotspot/rev/73c839dda17e The changes (every step) were tested with jprt on all supported platforms. Thanks, Vladimir. From rob.mckenna at oracle.com Tue Sep 23 12:38:07 2014 From: rob.mckenna at oracle.com (Rob McKenna) Date: Tue, 23 Sep 2014 13:38:07 +0100 Subject: Request for Approval: 8012941, 8028159 and 8043413 In-Reply-To: <54215187.6060606@oracle.com> References: <54215187.6060606@oracle.com> Message-ID: <542169AF.9040505@oracle.com> Approved - please add a suitable noreg keyword to 8043413. -Rob On 23/09/14 11:55, Vladimir Kempik wrote: > Hello! > > I'd like to get a approval to push these backports of 8012941, 8028159 > and 8043413 into jdk7. > > I need to backport all three fixes to fix customer's issue. > First one is a fix for c1, second one extends fix to c2, third one > fixes bug in second fix. > > The webrevs for jdk7: > > 8012941 - http://cr.openjdk.java.net/~vkempik/8012941/webrev.00/ > 8028159 - http://cr.openjdk.java.net/~vkempik/8028159/webrev.00/ > 8043413 - http://cr.openjdk.java.net/~vkempik/8043413/webrev.00/ > > Review thread - > http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/2014-September/015657.html > > 8012941 > Bug: https://bugs.openjdk.java.net/browse/JDK-8012941 > Jdk8 changeset: > http://hg.openjdk.java.net/jdk8/jdk8/hotspot/rev/0d1661d63d70 > > 8028159 > Bug: https://bugs.openjdk.java.net/browse/JDK-8028159 > Jdk8 changeset: > http://hg.openjdk.java.net/jdk8/jdk8/hotspot/rev/e74074c34312 > > 8043413 > Bug: https://bugs.openjdk.java.net/browse/JDK-8043413 > Jdk8 changeset: > http://hg.openjdk.java.net/jdk8u/jdk8u/hotspot/rev/73c839dda17e > > The changes (every step) were tested with jprt on all supported > platforms. > > Thanks, Vladimir. > From andrew.brygin at oracle.com Tue Sep 23 13:22:24 2014 From: andrew.brygin at oracle.com (Andrew Brygin) Date: Tue, 23 Sep 2014 17:22:24 +0400 Subject: [7u-dev] Request for approval: 8025917: JDK demo applets not running with >=7u40 or (JDK 8 and JDK 9) Message-ID: <54217410.9030801@oracle.com> Hello, please approve an identical backport of the fix to jdk7u-dev. The bug: https://bugs.openjdk.java.net/browse/JDK-8025917 jdk9 changeset: http://hg.openjdk.java.net/jdk9/client/jdk/rev/f497d36d25f8 jdk8 changeset: http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/dae860c8ddf1 Thanks, Andrew From naoto.sato at oracle.com Tue Sep 23 17:17:16 2014 From: naoto.sato at oracle.com (Naoto Sato) Date: Tue, 23 Sep 2014 10:17:16 -0700 Subject: [7u-dev] Request for approval: 8025917: JDK demo applets not running with >=7u40 or (JDK 8 and JDK 9) In-Reply-To: <54217410.9030801@oracle.com> References: <54217410.9030801@oracle.com> Message-ID: <5421AB1C.5060307@oracle.com> Approved. Naoto On 9/23/14, 6:22 AM, Andrew Brygin wrote: > Hello, > > please approve an identical backport of the fix to jdk7u-dev. > > The bug: https://bugs.openjdk.java.net/browse/JDK-8025917 > jdk9 changeset: http://hg.openjdk.java.net/jdk9/client/jdk/rev/f497d36d25f8 > jdk8 changeset: > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/dae860c8ddf1 > > Thanks, > Andrew From staffan.larsen at oracle.com Wed Sep 24 10:48:49 2014 From: staffan.larsen at oracle.com (Staffan Larsen) Date: Wed, 24 Sep 2014 12:48:49 +0200 Subject: [7u-dev] Request for approval for 8058936 hotspot/test/Makefile should use jtreg script from $JT_HOME/bin/jreg (instead of $JT_HOME/win32/bin/jtreg) Message-ID: <41B4D450-6230-40A3-9476-1FEDE0E7861B@oracle.com> Please approve this backport from 9 and 8u. The fix applies cleanly. bug: https://bugs.openjdk.java.net/browse/JDK-8058936 jdk9 changeset: http://hg.openjdk.java.net/jdk9/hs-rt/hotspot/rev/01961999c58a review thread: http://mail.openjdk.java.net/pipermail/hotspot-dev/2014-September/015409.html Thanks, /Staffan From sean.coffey at oracle.com Wed Sep 24 11:05:43 2014 From: sean.coffey at oracle.com (=?windows-1252?Q?Se=E1n_Coffey?=) Date: Wed, 24 Sep 2014 12:05:43 +0100 Subject: [7u-dev] Request for approval for 8058936 hotspot/test/Makefile should use jtreg script from $JT_HOME/bin/jreg (instead of $JT_HOME/win32/bin/jtreg) In-Reply-To: <41B4D450-6230-40A3-9476-1FEDE0E7861B@oracle.com> References: <41B4D450-6230-40A3-9476-1FEDE0E7861B@oracle.com> Message-ID: <5422A587.2020104@oracle.com> Approved. Please add a noreg- label. regards, Sean. On 24/09/14 11:48, Staffan Larsen wrote: > Please approve this backport from 9 and 8u. The fix applies cleanly. > > bug: https://bugs.openjdk.java.net/browse/JDK-8058936 > jdk9 changeset: http://hg.openjdk.java.net/jdk9/hs-rt/hotspot/rev/01961999c58a > review thread: http://mail.openjdk.java.net/pipermail/hotspot-dev/2014-September/015409.html > > Thanks, > /Staffan > > From vladimir.kempik at oracle.com Wed Sep 24 11:22:45 2014 From: vladimir.kempik at oracle.com (vladimir.kempik at oracle.com) Date: Wed, 24 Sep 2014 11:22:45 +0000 Subject: hg: jdk7u/jdk7u-dev/hotspot: 3 new changesets Message-ID: <201409241122.s8OBMjPE003852@aojmv0008> Changeset: 96d116f870b0 Author: vkempik Date: 2014-09-24 13:47 +0400 URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/hotspot/rev/96d116f870b0 8012941: JSR 292: too deep inlining might crash compiler because of stack overflow Reviewed-by: kvn, sgabdura ! src/share/vm/c1/c1_GraphBuilder.cpp ! src/share/vm/c1/c1_globals.hpp Changeset: c02c67eb03ae Author: vkempik Date: 2014-09-24 13:49 +0400 URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/hotspot/rev/c02c67eb03ae 8028159: C2: compiler stack overflow during inlining of @ForceInline methods Reviewed-by: kvn, sgabdura ! src/share/vm/c1/c1_globals.hpp ! src/share/vm/opto/bytecodeInfo.cpp ! src/share/vm/opto/callGenerator.cpp ! src/share/vm/runtime/globals.hpp Changeset: 40459e83f8c9 Author: roland Date: 2014-06-02 10:01 +0200 URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/hotspot/rev/40459e83f8c9 8043413: REGRESSION: Hotspot causes segmentation fault in jdk8ux, but not in jdk7ux Summary: long series of lambda form calls trigger stack overflow in c2 Reviewed-by: kvn, vlivanov ! src/share/vm/opto/bytecodeInfo.cpp From staffan.larsen at oracle.com Wed Sep 24 11:38:38 2014 From: staffan.larsen at oracle.com (staffan.larsen at oracle.com) Date: Wed, 24 Sep 2014 11:38:38 +0000 Subject: hg: jdk7u/jdk7u-dev/hotspot: 8058936: hotspot/test/Makefile should use jtreg script from $JT_HOME/bin/jreg (instead of $JT_HOME/win32/bin/jtreg) Message-ID: <201409241138.s8OBccO7006797@aojmv0008> Changeset: 4c218194cc6c Author: sla Date: 2014-09-24 09:49 +0200 URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/hotspot/rev/4c218194cc6c 8058936: hotspot/test/Makefile should use jtreg script from $JT_HOME/bin/jreg (instead of $JT_HOME/win32/bin/jtreg) Reviewed-by: dholmes, stefank ! test/Makefile From andrew.brygin at oracle.com Wed Sep 24 11:42:49 2014 From: andrew.brygin at oracle.com (andrew.brygin at oracle.com) Date: Wed, 24 Sep 2014 11:42:49 +0000 Subject: hg: jdk7u/jdk7u-dev/jdk: 8025917: JDK demo applets not running with >=7u40 or (JDK 8 and JDK 9) Message-ID: <201409241142.s8OBgnlp007319@aojmv0008> Changeset: 1ed30c084e3d Author: bae Date: 2014-09-24 12:40 +0100 URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/jdk/rev/1ed30c084e3d 8025917: JDK demo applets not running with >=7u40 or (JDK 8 and JDK 9) Reviewed-by: alexp ! src/share/demo/README From ivan.gerasimov at oracle.com Thu Sep 25 07:42:16 2014 From: ivan.gerasimov at oracle.com (Ivan Gerasimov) Date: Thu, 25 Sep 2014 11:42:16 +0400 Subject: RFR [9] 8059101: unshuffle_patch.sh should be able to deal with stdin/stdout Message-ID: <5423C758.3060904@oracle.com> Hello! This is a proposal to enhance the unshuffle_patch.sh script so that it will be able to read from stdin and write to stdout. This would let us use the script in a pipe chain. For example, the following line could be used to apply a patch directly from a remote repository: wget -q -O - http://path.to.the.raw.patch | bash ~/jdk9/common/bin/unshuffle_patch.sh jdk - - | hg patch - --no-commit (Note, that it would only work, if the repository provides the patches in the git format, which is not currently the case with hg.openjdk.java.net.) Would you please help review/approve this enhancement? BUGURL: https://bugs.openjdk.java.net/browse/JDK-8059101 WEBREV: http://cr.openjdk.java.net/~igerasim/8059101/0/webrev/ Sincerely yours, Ivan From daniel.fuchs at oracle.com Thu Sep 25 09:49:41 2014 From: daniel.fuchs at oracle.com (Daniel Fuchs) Date: Thu, 25 Sep 2014 11:49:41 +0200 Subject: RFR [9] 8059101: unshuffle_patch.sh should be able to deal with stdin/stdout In-Reply-To: <5423C758.3060904@oracle.com> References: <5423C758.3060904@oracle.com> Message-ID: <5423E535.90908@oracle.com> Hi Ivan, When setting output & input, I wonder if it would be simpler to use an 'if else if' construct in order to avoid the '-a' in the 'if' that follows. something like this (pseudo code) might be easier to read: if "x$output" is "x-" then substitute - with /dev/stdout else if $output file exists complain endif Also the script uses echo to print warnings & error. It should probably be changed to print those on stderr ( >&2 ) so that they don't mix with the patch when the output is '-'. Similarly - I feel that either verbose should redirect to stderr, or the script should complain and exit if output is '-' and verbose is on. best regards, -- daniel On 25/09/14 09:42, Ivan Gerasimov wrote: > Hello! > > This is a proposal to enhance the unshuffle_patch.sh script so that it > will be able to read from stdin and write to stdout. > This would let us use the script in a pipe chain. > > For example, the following line could be used to apply a patch directly > from a remote repository: > wget -q -O - http://path.to.the.raw.patch | bash > ~/jdk9/common/bin/unshuffle_patch.sh jdk - - | hg patch - --no-commit > > (Note, that it would only work, if the repository provides the patches > in the git format, which is not currently the case with > hg.openjdk.java.net.) > > Would you please help review/approve this enhancement? > > BUGURL: https://bugs.openjdk.java.net/browse/JDK-8059101 > WEBREV: http://cr.openjdk.java.net/~igerasim/8059101/0/webrev/ > > Sincerely yours, > Ivan > From ivan.gerasimov at oracle.com Thu Sep 25 10:12:43 2014 From: ivan.gerasimov at oracle.com (Ivan Gerasimov) Date: Thu, 25 Sep 2014 14:12:43 +0400 Subject: RFR [9] 8059101: unshuffle_patch.sh should be able to deal with stdin/stdout In-Reply-To: <5423E535.90908@oracle.com> References: <5423C758.3060904@oracle.com> <5423E535.90908@oracle.com> Message-ID: <5423EA9B.7010400@oracle.com> Thank you Daniel for the comments! On 25.09.2014 13:49, Daniel Fuchs wrote: > Hi Ivan, > > When setting output & input, I wonder if it would be simpler > to use an 'if else if' construct in order to avoid the '-a' > in the 'if' that follows. > We would still need that -a in if, as the user could pass /dev/stdin and /dev/stdout as the arguments to the script. Dashes are meant to be the shortcut for these long names, but we should allow the long names to be passed too. > something like this (pseudo code) might be easier to read: > > if "x$output" is "x-" then > substitute - with /dev/stdout > else if $output file exists > complain > endif > > Also the script uses echo to print warnings & error. > It should probably be changed to print those on stderr ( >&2 ) > so that they don't mix with the patch when the output is '-'. > Agreed! Redirected all the error messages to stderr. > Similarly - I feel that either verbose should redirect to > stderr, or the script should complain and exit if output > is '-' and verbose is on. > Done. Please see the updated webrev: http://cr.openjdk.java.net/~igerasim/8059101/1/webrev/ Sincerely yours, Ivan From daniel.fuchs at oracle.com Thu Sep 25 10:18:13 2014 From: daniel.fuchs at oracle.com (Daniel Fuchs) Date: Thu, 25 Sep 2014 12:18:13 +0200 Subject: RFR [9] 8059101: unshuffle_patch.sh should be able to deal with stdin/stdout In-Reply-To: <5423EA9B.7010400@oracle.com> References: <5423C758.3060904@oracle.com> <5423E535.90908@oracle.com> <5423EA9B.7010400@oracle.com> Message-ID: <5423EBE5.6020901@oracle.com> Hi Ivan, Should 'usage' also be redirected? best regards, -- daniel On 25/09/14 12:12, Ivan Gerasimov wrote: > Thank you Daniel for the comments! > > On 25.09.2014 13:49, Daniel Fuchs wrote: >> Hi Ivan, >> >> When setting output & input, I wonder if it would be simpler >> to use an 'if else if' construct in order to avoid the '-a' >> in the 'if' that follows. >> > We would still need that -a in if, as the user could pass /dev/stdin and > /dev/stdout as the arguments to the script. > Dashes are meant to be the shortcut for these long names, but we should > allow the long names to be passed too. > >> something like this (pseudo code) might be easier to read: >> >> if "x$output" is "x-" then >> substitute - with /dev/stdout >> else if $output file exists >> complain >> endif >> >> Also the script uses echo to print warnings & error. >> It should probably be changed to print those on stderr ( >&2 ) >> so that they don't mix with the patch when the output is '-'. >> > Agreed! > Redirected all the error messages to stderr. > >> Similarly - I feel that either verbose should redirect to >> stderr, or the script should complain and exit if output >> is '-' and verbose is on. >> > Done. > > Please see the updated webrev: > http://cr.openjdk.java.net/~igerasim/8059101/1/webrev/ > > Sincerely yours, > Ivan > From ivan.gerasimov at oracle.com Thu Sep 25 10:26:13 2014 From: ivan.gerasimov at oracle.com (Ivan Gerasimov) Date: Thu, 25 Sep 2014 14:26:13 +0400 Subject: RFR [9] 8059101: unshuffle_patch.sh should be able to deal with stdin/stdout In-Reply-To: <5423EBE5.6020901@oracle.com> References: <5423C758.3060904@oracle.com> <5423E535.90908@oracle.com> <5423EA9B.7010400@oracle.com> <5423EBE5.6020901@oracle.com> Message-ID: <5423EDC5.8030701@oracle.com> On 25.09.2014 14:18, Daniel Fuchs wrote: > Hi Ivan, > > Should 'usage' also be redirected? > This would be inconsistent with other command line utilities. They usually print help/usage to stdout. Sincerely yours, Ivan > best regards, > > -- daniel > > On 25/09/14 12:12, Ivan Gerasimov wrote: >> Thank you Daniel for the comments! >> >> On 25.09.2014 13:49, Daniel Fuchs wrote: >>> Hi Ivan, >>> >>> When setting output & input, I wonder if it would be simpler >>> to use an 'if else if' construct in order to avoid the '-a' >>> in the 'if' that follows. >>> >> We would still need that -a in if, as the user could pass /dev/stdin and >> /dev/stdout as the arguments to the script. >> Dashes are meant to be the shortcut for these long names, but we should >> allow the long names to be passed too. >> >>> something like this (pseudo code) might be easier to read: >>> >>> if "x$output" is "x-" then >>> substitute - with /dev/stdout >>> else if $output file exists >>> complain >>> endif >>> >>> Also the script uses echo to print warnings & error. >>> It should probably be changed to print those on stderr ( >&2 ) >>> so that they don't mix with the patch when the output is '-'. >>> >> Agreed! >> Redirected all the error messages to stderr. >> >>> Similarly - I feel that either verbose should redirect to >>> stderr, or the script should complain and exit if output >>> is '-' and verbose is on. >>> >> Done. >> >> Please see the updated webrev: >> http://cr.openjdk.java.net/~igerasim/8059101/1/webrev/ >> >> Sincerely yours, >> Ivan >> > > > From chris.hegarty at oracle.com Thu Sep 25 12:21:49 2014 From: chris.hegarty at oracle.com (Chris Hegarty) Date: Thu, 25 Sep 2014 05:21:49 -0700 Subject: RFR [9] 8059101: unshuffle_patch.sh should be able to deal with stdin/stdout In-Reply-To: <5423EA9B.7010400@oracle.com> References: <5423C758.3060904@oracle.com> <5423E535.90908@oracle.com> <5423EA9B.7010400@oracle.com> Message-ID: On 25 Sep 2014, at 03:12, Ivan Gerasimov wrote: > Thank you Daniel for the comments! > > On 25.09.2014 13:49, Daniel Fuchs wrote: >> Hi Ivan, >> >> When setting output & input, I wonder if it would be simpler >> to use an 'if else if' construct in order to avoid the '-a' >> in the 'if' that follows. >> > We would still need that -a in if, as the user could pass /dev/stdin and /dev/stdout as the arguments to the script. > Dashes are meant to be the shortcut for these long names, but we should allow the long names to be passed too. > >> something like this (pseudo code) might be easier to read: >> >> if "x$output" is "x-" then >> substitute - with /dev/stdout >> else if $output file exists >> complain >> endif >> >> Also the script uses echo to print warnings & error. >> It should probably be changed to print those on stderr ( >&2 ) >> so that they don't mix with the patch when the output is '-'. >> > Agreed! > Redirected all the error messages to stderr. > >> Similarly - I feel that either verbose should redirect to >> stderr, or the script should complain and exit if output >> is '-' and verbose is on. >> > Done. > > Please see the updated webrev: > http://cr.openjdk.java.net/~igerasim/8059101/1/webrev/ This looks ok to me. -Chris. > > Sincerely yours, > Ivan > From chris.hegarty at oracle.com Thu Sep 25 12:23:17 2014 From: chris.hegarty at oracle.com (Chris Hegarty) Date: Thu, 25 Sep 2014 05:23:17 -0700 Subject: RFR [9] 8059101: unshuffle_patch.sh should be able to deal with stdin/stdout In-Reply-To: <5423C758.3060904@oracle.com> References: <5423C758.3060904@oracle.com> Message-ID: <0EE3C3EF-AC19-45F7-A842-4D7CD7298125@oracle.com> On 25 Sep 2014, at 00:42, Ivan Gerasimov wrote: > Hello! > > This is a proposal to enhance the unshuffle_patch.sh script so that it will be able to read from stdin and write to stdout. > This would let us use the script in a pipe chain. > > For example, the following line could be used to apply a patch directly from a remote repository: > wget -q -O - http://path.to.the.raw.patch | bash ~/jdk9/common/bin/unshuffle_patch.sh jdk - - | hg patch - --no-commit > > (Note, that it would only work, if the repository provides the patches in the git format, which is not currently the case with hg.openjdk.java.net.) It would be nice to start a thread with ops at o.j.n to request that this be changed. -Chris. > Would you please help review/approve this enhancement? > > BUGURL: https://bugs.openjdk.java.net/browse/JDK-8059101 > WEBREV: http://cr.openjdk.java.net/~igerasim/8059101/0/webrev/ > > Sincerely yours, > Ivan > From daniel.fuchs at oracle.com Thu Sep 25 12:57:46 2014 From: daniel.fuchs at oracle.com (Daniel Fuchs) Date: Thu, 25 Sep 2014 14:57:46 +0200 Subject: RFR [9] 8059101: unshuffle_patch.sh should be able to deal with stdin/stdout In-Reply-To: <5423EDC5.8030701@oracle.com> References: <5423C758.3060904@oracle.com> <5423E535.90908@oracle.com> <5423EA9B.7010400@oracle.com> <5423EBE5.6020901@oracle.com> <5423EDC5.8030701@oracle.com> Message-ID: <5424114A.9030508@oracle.com> On 25/09/14 12:26, Ivan Gerasimov wrote: > > On 25.09.2014 14:18, Daniel Fuchs wrote: >> Hi Ivan, >> >> Should 'usage' also be redirected? >> > This would be inconsistent with other command line utilities. > They usually print help/usage to stdout. Right. I suspected something like that. This looks good! thanks, -- daniel > > Sincerely yours, > Ivan > >> best regards, >> >> -- daniel >> >> On 25/09/14 12:12, Ivan Gerasimov wrote: >>> Thank you Daniel for the comments! >>> >>> On 25.09.2014 13:49, Daniel Fuchs wrote: >>>> Hi Ivan, >>>> >>>> When setting output & input, I wonder if it would be simpler >>>> to use an 'if else if' construct in order to avoid the '-a' >>>> in the 'if' that follows. >>>> >>> We would still need that -a in if, as the user could pass /dev/stdin and >>> /dev/stdout as the arguments to the script. >>> Dashes are meant to be the shortcut for these long names, but we should >>> allow the long names to be passed too. >>> >>>> something like this (pseudo code) might be easier to read: >>>> >>>> if "x$output" is "x-" then >>>> substitute - with /dev/stdout >>>> else if $output file exists >>>> complain >>>> endif >>>> >>>> Also the script uses echo to print warnings & error. >>>> It should probably be changed to print those on stderr ( >&2 ) >>>> so that they don't mix with the patch when the output is '-'. >>>> >>> Agreed! >>> Redirected all the error messages to stderr. >>> >>>> Similarly - I feel that either verbose should redirect to >>>> stderr, or the script should complain and exit if output >>>> is '-' and verbose is on. >>>> >>> Done. >>> >>> Please see the updated webrev: >>> http://cr.openjdk.java.net/~igerasim/8059101/1/webrev/ >>> >>> Sincerely yours, >>> Ivan >>> >> >> >> > From ivan.gerasimov at oracle.com Thu Sep 25 13:25:54 2014 From: ivan.gerasimov at oracle.com (Ivan Gerasimov) Date: Thu, 25 Sep 2014 17:25:54 +0400 Subject: RFR [9] 8059101: unshuffle_patch.sh should be able to deal with stdin/stdout In-Reply-To: References: <5423C758.3060904@oracle.com> <5423E535.90908@oracle.com> <5423EA9B.7010400@oracle.com> Message-ID: <542417E2.4050204@oracle.com> Thank you Daniel and Chris for your review! Sincerely yous, Ivan Please see the updated webrev: http://cr.openjdk.java.net/~igerasim/8059101/1/webrev/ > This looks ok to me. > From vladimir.kozlov at oracle.com Thu Sep 25 18:42:12 2014 From: vladimir.kozlov at oracle.com (Vladimir Kozlov) Date: Thu, 25 Sep 2014 11:42:12 -0700 Subject: [7u-dev] Request for approval for 8050022: linux-sparcv9: assert(SharedSkipVerify || obj->is_oop()) failed: sanity check Message-ID: <54246204.1000507@oracle.com> Backport request. Changes was pushed into jdk9 week ago. Nighties are fine. Changes are applied cleanly. Bug: https://bugs.openjdk.java.net/browse/JDK-8050022 jdk9 webrev: http://cr.openjdk.java.net/~morris/JDK-8050022.05 Review thread: http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/2014-September/015562.html jdk9 changeset: http://hg.openjdk.java.net/jdk9/hs-comp/hotspot/rev/ca010d2665ca Thanks, Vladimir From sean.coffey at oracle.com Thu Sep 25 18:49:45 2014 From: sean.coffey at oracle.com (=?windows-1252?Q?Se=E1n_Coffey?=) Date: Thu, 25 Sep 2014 19:49:45 +0100 Subject: [7u-dev] Request for approval for 8050022: linux-sparcv9: assert(SharedSkipVerify || obj->is_oop()) failed: sanity check In-Reply-To: <54246204.1000507@oracle.com> References: <54246204.1000507@oracle.com> Message-ID: <542463C9.6030007@oracle.com> Vladimir, this needs to be pushed to the jdk8u code line first. http://openjdk.java.net/projects/jdk7u/groundrules.html : Rule 1 regards, Sean. On 25/09/2014 19:42, Vladimir Kozlov wrote: > Backport request. Changes was pushed into jdk9 week ago. Nighties are > fine. Changes are applied cleanly. > > Bug: > https://bugs.openjdk.java.net/browse/JDK-8050022 > jdk9 webrev: > http://cr.openjdk.java.net/~morris/JDK-8050022.05 > Review thread: > http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/2014-September/015562.html > > jdk9 changeset: > http://hg.openjdk.java.net/jdk9/hs-comp/hotspot/rev/ca010d2665ca > > Thanks, > Vladimir From vladimir.kozlov at oracle.com Thu Sep 25 19:06:19 2014 From: vladimir.kozlov at oracle.com (Vladimir Kozlov) Date: Thu, 25 Sep 2014 12:06:19 -0700 Subject: [7u-dev] Request for approval for 8050022: linux-sparcv9: assert(SharedSkipVerify || obj->is_oop()) failed: sanity check In-Reply-To: <542463C9.6030007@oracle.com> References: <54246204.1000507@oracle.com> <542463C9.6030007@oracle.com> Message-ID: <542467AB.9000405@oracle.com> On 9/25/14 11:49 AM, Se?n Coffey wrote: > Vladimir, > > this needs to be pushed to the jdk8u code line first. > http://openjdk.java.net/projects/jdk7u/groundrules.html : Rule 1 Yes, I also sent the request to hotspot-dev (since it is Hotspot changes) for backport to 8u40. Can I push to 7u after I push into 8u without an other 7u request? Thanks, Vladimir > > regards, > Sean. > > On 25/09/2014 19:42, Vladimir Kozlov wrote: >> Backport request. Changes was pushed into jdk9 week ago. Nighties are >> fine. Changes are applied cleanly. >> >> Bug: >> https://bugs.openjdk.java.net/browse/JDK-8050022 >> jdk9 webrev: >> http://cr.openjdk.java.net/~morris/JDK-8050022.05 >> Review thread: >> http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/2014-September/015562.html >> >> jdk9 changeset: >> http://hg.openjdk.java.net/jdk9/hs-comp/hotspot/rev/ca010d2665ca >> >> Thanks, >> Vladimir > From sean.coffey at oracle.com Thu Sep 25 19:14:35 2014 From: sean.coffey at oracle.com (=?windows-1252?Q?Se=E1n_Coffey?=) Date: Thu, 25 Sep 2014 20:14:35 +0100 Subject: [7u-dev] Request for approval for 8050022: linux-sparcv9: assert(SharedSkipVerify || obj->is_oop()) failed: sanity check In-Reply-To: <542467AB.9000405@oracle.com> References: <54246204.1000507@oracle.com> <542463C9.6030007@oracle.com> <542467AB.9000405@oracle.com> Message-ID: <5424699B.8090404@oracle.com> On 25/09/2014 20:06, Vladimir Kozlov wrote: > On 9/25/14 11:49 AM, Se?n Coffey wrote: >> Vladimir, >> >> this needs to be pushed to the jdk8u code line first. >> http://openjdk.java.net/projects/jdk7u/groundrules.html : Rule 1 > > Yes, I also sent the request to hotspot-dev (since it is Hotspot > changes) for backport to 8u40. > Can I push to 7u after I push into 8u without an other 7u request? Thanks for update. I just needed to know that 8u plans were in progress. Approved. regards, Sean. > > Thanks, > Vladimir > >> >> regards, >> Sean. >> >> On 25/09/2014 19:42, Vladimir Kozlov wrote: >>> Backport request. Changes was pushed into jdk9 week ago. Nighties are >>> fine. Changes are applied cleanly. >>> >>> Bug: >>> https://bugs.openjdk.java.net/browse/JDK-8050022 >>> jdk9 webrev: >>> http://cr.openjdk.java.net/~morris/JDK-8050022.05 >>> Review thread: >>> http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/2014-September/015562.html >>> >>> >>> jdk9 changeset: >>> http://hg.openjdk.java.net/jdk9/hs-comp/hotspot/rev/ca010d2665ca >>> >>> Thanks, >>> Vladimir >> From vladimir.kozlov at oracle.com Thu Sep 25 19:56:33 2014 From: vladimir.kozlov at oracle.com (Vladimir Kozlov) Date: Thu, 25 Sep 2014 12:56:33 -0700 Subject: [7u-dev] Request for approval for 8050022: linux-sparcv9: assert(SharedSkipVerify || obj->is_oop()) failed: sanity check In-Reply-To: <5424699B.8090404@oracle.com> References: <54246204.1000507@oracle.com> <542463C9.6030007@oracle.com> <542467AB.9000405@oracle.com> <5424699B.8090404@oracle.com> Message-ID: <54247371.8000904@oracle.com> Thank you, Sean. Vladimir On 9/25/14 12:14 PM, Se?n Coffey wrote: > > On 25/09/2014 20:06, Vladimir Kozlov wrote: >> On 9/25/14 11:49 AM, Se?n Coffey wrote: >>> Vladimir, >>> >>> this needs to be pushed to the jdk8u code line first. >>> http://openjdk.java.net/projects/jdk7u/groundrules.html : Rule 1 >> >> Yes, I also sent the request to hotspot-dev (since it is Hotspot >> changes) for backport to 8u40. >> Can I push to 7u after I push into 8u without an other 7u request? > Thanks for update. I just needed to know that 8u plans were in progress. > > Approved. > > regards, > Sean. > >> >> Thanks, >> Vladimir >> >>> >>> regards, >>> Sean. >>> >>> On 25/09/2014 19:42, Vladimir Kozlov wrote: >>>> Backport request. Changes was pushed into jdk9 week ago. Nighties are >>>> fine. Changes are applied cleanly. >>>> >>>> Bug: >>>> https://bugs.openjdk.java.net/browse/JDK-8050022 >>>> jdk9 webrev: >>>> http://cr.openjdk.java.net/~morris/JDK-8050022.05 >>>> Review thread: >>>> http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/2014-September/015562.html >>>> >>>> >>>> jdk9 changeset: >>>> http://hg.openjdk.java.net/jdk9/hs-comp/hotspot/rev/ca010d2665ca >>>> >>>> Thanks, >>>> Vladimir >>> > From vladimir.kozlov at oracle.com Thu Sep 25 22:27:30 2014 From: vladimir.kozlov at oracle.com (vladimir.kozlov at oracle.com) Date: Thu, 25 Sep 2014 22:27:30 +0000 Subject: hg: jdk7u/jdk7u-dev/hotspot: 8050022: linux-sparcv9: assert(SharedSkipVerify || obj->is_oop()) failed: sanity check Message-ID: <201409252227.s8PMRUR4006165@aojmv0008> Changeset: c7f740ad8991 Author: morris Date: 2014-09-18 11:46 -0700 URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/hotspot/rev/c7f740ad8991 8050022: linux-sparcv9: assert(SharedSkipVerify || obj->is_oop()) failed: sanity check Summary: Provide promoted stack slots for floating-point registers in the SPARC c_calling_convention. Reviewed-by: kvn, jrose, drchase ! src/cpu/sparc/vm/sharedRuntime_sparc.cpp ! src/cpu/sparc/vm/sparc.ad From anton.nashatyrev at oracle.com Fri Sep 26 10:28:41 2014 From: anton.nashatyrev at oracle.com (Anton Nashatyrev) Date: Fri, 26 Sep 2014 14:28:41 +0400 Subject: [8u-dev][7u-dev] Backport request for 8058473: "Comparison method violates its general contract" when using Clipboard In-Reply-To: <54253A71.9040804@oracle.com> References: <54253A71.9040804@oracle.com> Message-ID: <54253FD9.901@oracle.com> Hello, please approve the backport to 8u-dev and 7u-dev (the patch is absolutely the same for 8u-dev and 7u-dev) JBS: https://bugs.openjdk.java.net/browse/JDK-8058473 JDK9 changeset: http://hg.openjdk.java.net/jdk9/client/jdk/rev/c87d6fdf8376 Backport to 8u: http://cr.openjdk.java.net/%7Eanashaty/8058473/8/webrev.00/ Review thread: http://mail.openjdk.java.net/pipermail/awt-dev/2014-September/008516.html Thanks! Anton. Hello Anton, the backport looks fine. Thanks, Andrew On 9/25/2014 4:16 PM, anton nashatyrev wrote: > Hi Sergey, Andrew, > > could you please additionally review the fix backport to 8u-dev > (the fixed util method as been moved to another class) ? > > JDK9 fix: http://cr.openjdk.java.net/%7Eanashaty/8058473/9/webrev.00/ > Backport to 8u: > http://cr.openjdk.java.net/%7Eanashaty/8058473/8/webrev.00/ > > Thanks! > Anton. > > On 24.09.2014 18:15, Andrew Brygin wrote: >> Hello Anton, >> >> the fix looks fine to me. >> >> Thanks, >> Andrew >> >> On 9/24/2014 1:51 PM, anton nashatyrev wrote: >>> Sergey, thanks for review! >>> >>> Any other volunteer? >>> >>> On 19.09.2014 17:27, Sergey Bylokhov wrote: >>>> Hi, Anton. >>>> Then the fix looks good, thanks for clarification. >>>> >>>> On 19.09.2014 17:25, anton nashatyrev wrote: >>>>> We don't treat the 'text/plain; class=java.lang.Object' as >>>>> TextType (DataFlavor.isFlavorTextType()) since it has unknown >>>>> representation class. Such flavors fall into 'other' category. >>>>> >>>>> On 19.09.2014 17:16, Sergey Bylokhov wrote: >>>>>> Hi, Anton. >>>>>> Why "unknown/flavor" is before "text/plain" in the fixed version? >>>>>> >>>>>> text/plain; class=java.io.Serializable; charset=unicode >>>>>> text/html; class=java.io.Serializable; charset=unicode >>>>>> unknown/flavor; class=java.io.InputStream >>>>>> unknown/flavor; class=java.lang.Object >>>>>> text/unknown; class=java.lang.Object; charset=unicode >>>>>> text/unknown; class=java.lang.Object >>>>>> text/plain; class=java.lang.Object; charset=unicode >>>>>> text/plain; class=java.lang.Object >>>>>> >>>>>> >>>>>> >>>>>> On 19.09.2014 15:58, anton nashatyrev wrote: >>>>>>> Hi Sergey, >>>>>>> >>>>>>> On 18.09.2014 17:30, Sergey Bylokhov wrote: >>>>>>>> - Can you try to change sort TimSort algorithm to mergerSort. >>>>>>>> What the order will be in this case ?(see JDK-8048887). >>>>>>> >>>>>>> The mergeSort seems to be more relaxed regarding the Comparator >>>>>>> correctness (BTW TimSort also doesn't fail each time - this >>>>>>> depends on initial elements positions). The order of course is >>>>>>> not correct with the incorrect comparator: >>>>>>> >>>>>>> >>>>>>> MergeSort with current comparator Merge/TimSort with fixed >>>>>>> Comparator >>>>>>> --------------------------------- >>>>>>> ----------------------------------- >>>>>>> application/unknown; class=java.io.InputStream >>>>>>> application/unknown; class=java.io.InputStream >>>>>>> application/unknown; class=java.io.InputStream >>>>>>> application/unknown; class=java.io.InputStream >>>>>>> application/unknown; class=java.lang.Object >>>>>>> application/unknown; class=java.lang.Object >>>>>>> application/x-java-jvm-local-objectref; >>>>>>> class=java.io.InputStream >>>>>>> application/x-java-jvm-local-objectref; class=java.io.InputStream >>>>>>> application/x-java-jvm-local-objectref; >>>>>>> class=java.io.InputStream >>>>>>> application/x-java-jvm-local-objectref; class=java.io.InputStream >>>>>>> application/x-java-jvm-local-objectref; class=java.lang.Object >>>>>>> application/x-java-jvm-local-objectref; class=java.lang.Object >>>>>>> text/unknown; class=java.io.Serializable; >>>>>>> charset=unicode text/html; class=java.io.Reader >>>>>>> text/plain; class=java.io.Serializable; >>>>>>> charset=unicode text/html; class=java.lang.String >>>>>>> text/html; class=java.io.Serializable; >>>>>>> charset=unicode text/html; >>>>>>> class=java.io.InputStream; charset=unicode >>>>>>> unknown/flavor; class=java.io.InputStream text/html; >>>>>>> class=java.io.InputStream; charset=unicode >>>>>>> unknown/flavor; class=java.io.InputStream text/html; >>>>>>> class=java.io.InputStream >>>>>>> text/html; class=java.io.Reader text/html; >>>>>>> class=java.io.InputStream; charset=cp1251 >>>>>>> text/html; class=java.lang.String text/plain; >>>>>>> class=java.io.Reader >>>>>>> text/html; class=java.io.InputStream; >>>>>>> charset=unicode text/plain; class=java.lang.String >>>>>>> text/html; class=java.io.InputStream; >>>>>>> charset=unicode text/plain; >>>>>>> class=java.io.InputStream; charset=unicode >>>>>>> text/html; class=java.io.InputStream text/plain; >>>>>>> class=java.io.InputStream; charset=unicode >>>>>>> text/html; class=java.io.InputStream; >>>>>>> charset=cp1251 text/plain; >>>>>>> class=java.io.InputStream >>>>>>> text/plain; class=java.io.InputStream; >>>>>>> charset=unicode text/plain; >>>>>>> class=java.io.InputStream; charset=cp1251 >>>>>>> text/plain; class=java.io.InputStream; >>>>>>> charset=unicode text/unknown; class=java.io.Reader >>>>>>> text/plain; class=java.io.InputStream text/unknown; >>>>>>> class=java.lang.String >>>>>>> text/plain; class=java.io.InputStream; >>>>>>> charset=cp1251 text/unknown; >>>>>>> class=java.io.InputStream; charset=unicode >>>>>>> text/unknown; class=java.io.InputStream; >>>>>>> charset=unicode text/unknown; >>>>>>> class=java.io.InputStream; charset=unicode >>>>>>> text/unknown; class=java.io.InputStream; >>>>>>> charset=cp1251 text/unknown; class=java.io.InputStream >>>>>>> unknown/flavor; class=java.lang.Object text/unknown; >>>>>>> class=java.io.InputStream; charset=cp1251 >>>>>>> text/unknown; class=java.lang.Object; >>>>>>> charset=unicode text/unknown; >>>>>>> class=java.io.Serializable; charset=unicode >>>>>>> text/unknown; class=java.io.Reader text/plain; >>>>>>> class=java.io.Serializable; charset=unicode >>>>>>> text/unknown; class=java.lang.String text/html; >>>>>>> class=java.io.Serializable; charset=unicode >>>>>>> text/unknown; class=java.io.InputStream; >>>>>>> charset=unicode unknown/flavor; >>>>>>> class=java.io.InputStream >>>>>>> text/unknown; class=java.io.InputStream unknown/flavor; >>>>>>> class=java.io.InputStream >>>>>>> text/unknown; class=java.lang.Object unknown/flavor; >>>>>>> class=java.lang.Object >>>>>>> text/plain; class=java.lang.String text/unknown; >>>>>>> class=java.lang.Object; charset=unicode >>>>>>> text/plain; class=java.lang.Object; >>>>>>> charset=unicode text/unknown; >>>>>>> class=java.lang.Object >>>>>>> text/plain; class=java.lang.Object text/plain; >>>>>>> class=java.lang.Object; charset=unicode >>>>>>> text/plain; class=java.io.Reader text/plain; >>>>>>> class=java.lang.Object >>>>>>> text/html; class=java.lang.Object; >>>>>>> charset=unicode text/html; >>>>>>> class=java.lang.Object; charset=unicode >>>>>>> text/html; class=java.lang.Object text/html; >>>>>>> class=java.lang.Object >>>>>>> >>>>>>> >>>>>>>> - Can you run all related regression and jck tests. >>>>>>> >>>>>>> Executed all 'datatransfer' regression and JCK tests - no >>>>>>> regressions detected. >>>>>>> >>>>>>>> Thanks. >>>>>>>> >>>>>>>> On 18.09.2014 17:04, anton nashatyrev wrote: >>>>>>>>> Hello, >>>>>>>>> could you please review the following fix: >>>>>>>>> >>>>>>>>> fix: http://cr.openjdk.java.net/~anashaty/8058473/9/webrev.00/ >>>>>>>>> >>>>>>>>> bug: https://bugs.openjdk.java.net/browse/JDK-8058473 >>>>>>>>> >>>>>>>>> Problem: DataFlavorComparator violates Comparator contract. >>>>>>>>> >>>>>>>>> Fix: make the DataFlavorComparator transitive conformant >>>>>>>>> (please see the bug comment for details) >>>>>>>>> >>>>>>>>> Thanks! >>>>>>>>> Anton. >>>>>>>> >>>>>>>> >>>>>>> >>>>>> >>>>>> >>>>> >>>> >>>> >>> >> > From sean.coffey at oracle.com Fri Sep 26 11:13:49 2014 From: sean.coffey at oracle.com (=?UTF-8?B?U2XDoW4gQ29mZmV5?=) Date: Fri, 26 Sep 2014 12:13:49 +0100 Subject: [8u-dev][7u-dev] Backport request for 8058473: "Comparison method violates its general contract" when using Clipboard In-Reply-To: <54253FD9.901@oracle.com> References: <54253A71.9040804@oracle.com> <54253FD9.901@oracle.com> Message-ID: <54254A6D.2020706@oracle.com> Approved for 8u-dev and 7u-dev. regards, Sean. On 26/09/2014 11:28, Anton Nashatyrev wrote: > Hello, > > please approve the backport to 8u-dev and 7u-dev (the patch is > absolutely the same for 8u-dev and 7u-dev) > > JBS: https://bugs.openjdk.java.net/browse/JDK-8058473 > JDK9 changeset: > http://hg.openjdk.java.net/jdk9/client/jdk/rev/c87d6fdf8376 > Backport to 8u: > http://cr.openjdk.java.net/%7Eanashaty/8058473/8/webrev.00/ > Review thread: > http://mail.openjdk.java.net/pipermail/awt-dev/2014-September/008516.html > > Thanks! > Anton. > > Hello Anton, > > the backport looks fine. > > Thanks, > Andrew > > On 9/25/2014 4:16 PM, anton nashatyrev wrote: >> Hi Sergey, Andrew, >> >> could you please additionally review the fix backport to 8u-dev >> (the fixed util method as been moved to another class) ? >> >> JDK9 fix: http://cr.openjdk.java.net/%7Eanashaty/8058473/9/webrev.00/ >> Backport to 8u: >> http://cr.openjdk.java.net/%7Eanashaty/8058473/8/webrev.00/ >> >> Thanks! >> Anton. >> >> On 24.09.2014 18:15, Andrew Brygin wrote: >>> Hello Anton, >>> >>> the fix looks fine to me. >>> >>> Thanks, >>> Andrew >>> >>> On 9/24/2014 1:51 PM, anton nashatyrev wrote: >>>> Sergey, thanks for review! >>>> >>>> Any other volunteer? >>>> >>>> On 19.09.2014 17:27, Sergey Bylokhov wrote: >>>>> Hi, Anton. >>>>> Then the fix looks good, thanks for clarification. >>>>> >>>>> On 19.09.2014 17:25, anton nashatyrev wrote: >>>>>> We don't treat the 'text/plain; class=java.lang.Object' as >>>>>> TextType (DataFlavor.isFlavorTextType()) since it has unknown >>>>>> representation class. Such flavors fall into 'other' category. >>>>>> >>>>>> On 19.09.2014 17:16, Sergey Bylokhov wrote: >>>>>>> Hi, Anton. >>>>>>> Why "unknown/flavor" is before "text/plain" in the fixed version? >>>>>>> >>>>>>> text/plain; class=java.io.Serializable; charset=unicode >>>>>>> text/html; class=java.io.Serializable; charset=unicode >>>>>>> unknown/flavor; class=java.io.InputStream >>>>>>> unknown/flavor; class=java.lang.Object >>>>>>> text/unknown; class=java.lang.Object; charset=unicode >>>>>>> text/unknown; class=java.lang.Object >>>>>>> text/plain; class=java.lang.Object; charset=unicode >>>>>>> text/plain; class=java.lang.Object >>>>>>> >>>>>>> >>>>>>> >>>>>>> On 19.09.2014 15:58, anton nashatyrev wrote: >>>>>>>> Hi Sergey, >>>>>>>> >>>>>>>> On 18.09.2014 17:30, Sergey Bylokhov wrote: >>>>>>>>> - Can you try to change sort TimSort algorithm to mergerSort. >>>>>>>>> What the order will be in this case ?(see JDK-8048887). >>>>>>>> >>>>>>>> The mergeSort seems to be more relaxed regarding the Comparator >>>>>>>> correctness (BTW TimSort also doesn't fail each time - this >>>>>>>> depends on initial elements positions). The order of course is >>>>>>>> not correct with the incorrect comparator: >>>>>>>> >>>>>>>> >>>>>>>> MergeSort with current comparator Merge/TimSort with fixed >>>>>>>> Comparator >>>>>>>> --------------------------------- >>>>>>>> ----------------------------------- >>>>>>>> application/unknown; class=java.io.InputStream >>>>>>>> application/unknown; class=java.io.InputStream >>>>>>>> application/unknown; class=java.io.InputStream >>>>>>>> application/unknown; class=java.io.InputStream >>>>>>>> application/unknown; class=java.lang.Object >>>>>>>> application/unknown; class=java.lang.Object >>>>>>>> application/x-java-jvm-local-objectref; >>>>>>>> class=java.io.InputStream >>>>>>>> application/x-java-jvm-local-objectref; class=java.io.InputStream >>>>>>>> application/x-java-jvm-local-objectref; >>>>>>>> class=java.io.InputStream >>>>>>>> application/x-java-jvm-local-objectref; class=java.io.InputStream >>>>>>>> application/x-java-jvm-local-objectref; class=java.lang.Object >>>>>>>> application/x-java-jvm-local-objectref; class=java.lang.Object >>>>>>>> text/unknown; class=java.io.Serializable; >>>>>>>> charset=unicode text/html; class=java.io.Reader >>>>>>>> text/plain; class=java.io.Serializable; >>>>>>>> charset=unicode text/html; class=java.lang.String >>>>>>>> text/html; class=java.io.Serializable; >>>>>>>> charset=unicode text/html; >>>>>>>> class=java.io.InputStream; charset=unicode >>>>>>>> unknown/flavor; class=java.io.InputStream text/html; >>>>>>>> class=java.io.InputStream; charset=unicode >>>>>>>> unknown/flavor; class=java.io.InputStream text/html; >>>>>>>> class=java.io.InputStream >>>>>>>> text/html; class=java.io.Reader text/html; >>>>>>>> class=java.io.InputStream; charset=cp1251 >>>>>>>> text/html; class=java.lang.String text/plain; >>>>>>>> class=java.io.Reader >>>>>>>> text/html; class=java.io.InputStream; >>>>>>>> charset=unicode text/plain; class=java.lang.String >>>>>>>> text/html; class=java.io.InputStream; >>>>>>>> charset=unicode text/plain; >>>>>>>> class=java.io.InputStream; charset=unicode >>>>>>>> text/html; class=java.io.InputStream text/plain; >>>>>>>> class=java.io.InputStream; charset=unicode >>>>>>>> text/html; class=java.io.InputStream; >>>>>>>> charset=cp1251 text/plain; >>>>>>>> class=java.io.InputStream >>>>>>>> text/plain; class=java.io.InputStream; >>>>>>>> charset=unicode text/plain; >>>>>>>> class=java.io.InputStream; charset=cp1251 >>>>>>>> text/plain; class=java.io.InputStream; >>>>>>>> charset=unicode text/unknown; class=java.io.Reader >>>>>>>> text/plain; class=java.io.InputStream text/unknown; >>>>>>>> class=java.lang.String >>>>>>>> text/plain; class=java.io.InputStream; >>>>>>>> charset=cp1251 text/unknown; >>>>>>>> class=java.io.InputStream; charset=unicode >>>>>>>> text/unknown; class=java.io.InputStream; >>>>>>>> charset=unicode text/unknown; >>>>>>>> class=java.io.InputStream; charset=unicode >>>>>>>> text/unknown; class=java.io.InputStream; >>>>>>>> charset=cp1251 text/unknown; >>>>>>>> class=java.io.InputStream >>>>>>>> unknown/flavor; class=java.lang.Object text/unknown; >>>>>>>> class=java.io.InputStream; charset=cp1251 >>>>>>>> text/unknown; class=java.lang.Object; >>>>>>>> charset=unicode text/unknown; >>>>>>>> class=java.io.Serializable; charset=unicode >>>>>>>> text/unknown; class=java.io.Reader text/plain; >>>>>>>> class=java.io.Serializable; charset=unicode >>>>>>>> text/unknown; class=java.lang.String text/html; >>>>>>>> class=java.io.Serializable; charset=unicode >>>>>>>> text/unknown; class=java.io.InputStream; >>>>>>>> charset=unicode unknown/flavor; >>>>>>>> class=java.io.InputStream >>>>>>>> text/unknown; class=java.io.InputStream unknown/flavor; >>>>>>>> class=java.io.InputStream >>>>>>>> text/unknown; class=java.lang.Object unknown/flavor; >>>>>>>> class=java.lang.Object >>>>>>>> text/plain; class=java.lang.String text/unknown; >>>>>>>> class=java.lang.Object; charset=unicode >>>>>>>> text/plain; class=java.lang.Object; >>>>>>>> charset=unicode text/unknown; >>>>>>>> class=java.lang.Object >>>>>>>> text/plain; class=java.lang.Object text/plain; >>>>>>>> class=java.lang.Object; charset=unicode >>>>>>>> text/plain; class=java.io.Reader text/plain; >>>>>>>> class=java.lang.Object >>>>>>>> text/html; class=java.lang.Object; >>>>>>>> charset=unicode text/html; >>>>>>>> class=java.lang.Object; charset=unicode >>>>>>>> text/html; class=java.lang.Object text/html; >>>>>>>> class=java.lang.Object >>>>>>>> >>>>>>>> >>>>>>>>> - Can you run all related regression and jck tests. >>>>>>>> >>>>>>>> Executed all 'datatransfer' regression and JCK tests - no >>>>>>>> regressions detected. >>>>>>>> >>>>>>>>> Thanks. >>>>>>>>> >>>>>>>>> On 18.09.2014 17:04, anton nashatyrev wrote: >>>>>>>>>> Hello, >>>>>>>>>> could you please review the following fix: >>>>>>>>>> >>>>>>>>>> fix: http://cr.openjdk.java.net/~anashaty/8058473/9/webrev.00/ >>>>>>>>>> >>>>>>>>>> bug: https://bugs.openjdk.java.net/browse/JDK-8058473 >>>>>>>>>> >>>>>>>>>> Problem: DataFlavorComparator violates Comparator contract. >>>>>>>>>> >>>>>>>>>> Fix: make the DataFlavorComparator transitive conformant >>>>>>>>>> (please see the bug comment for details) >>>>>>>>>> >>>>>>>>>> Thanks! >>>>>>>>>> Anton. >>>>>>>>> >>>>>>>>> >>>>>>>> >>>>>>> >>>>>>> >>>>>> >>>>> >>>>> >>>> >>> >> > > > From anton.nashatyrev at oracle.com Fri Sep 26 11:41:54 2014 From: anton.nashatyrev at oracle.com (anton.nashatyrev at oracle.com) Date: Fri, 26 Sep 2014 11:41:54 +0000 Subject: hg: jdk7u/jdk7u-dev/jdk: 8058473: "Comparison method violates its general contract" when using Clipboard Message-ID: <201409261141.s8QBfsRt019208@aojmv0008> Changeset: 219a2687c3f9 Author: anashaty Date: 2014-09-26 15:40 +0400 URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/jdk/rev/219a2687c3f9 8058473: "Comparison method violates its general contract" when using Clipboard Reviewed-by: serb, bae ! src/share/classes/sun/awt/datatransfer/DataTransferer.java + test/sun/awt/datatransfer/DataFlavorComparatorTest1.java From alexander.potochkin at oracle.com Mon Sep 29 12:11:04 2014 From: alexander.potochkin at oracle.com (Alexander Potochkin) Date: Mon, 29 Sep 2014 16:11:04 +0400 Subject: [7u-dev] Request for approval 8028484: [TEST_BUG][macosx] closed/java/awt/MouseInfo/JContainerMousePositionTest fails In-Reply-To: <5410433D.4060708@oracle.com> References: <54104107.9070009@oracle.com> <541041D5.4090102@oracle.com> <5410433D.4060708@oracle.com> Message-ID: <54294C58.50709@oracle.com> Hello Dmitry The changes look good! Thanks alexp On 9/10/2014 4:25 PM, dmitry markov wrote: > Thank you, Sean! > > Anthony, Sergey, > Could you review this, please? > > Thanks, > Dmitry > On 10/09/2014 16:19, Se?n Coffey wrote: >> Approved but subject to code review. >> >> regards, >> Sean. >> >> On 10/09/14 13:16, dmitry markov wrote: >>> Hello, >>> >>> Could you approve the back-port of the fix for 8028484 to jdk7u-dev >>> (7u80), please? >>> The original fix and the back-port are slightly different. There are >>> trivial changes in the regression test (lambda was replaced by >>> anonymous inner class). The author and the reviewers of the original >>> fix are in the CC list. >>> >>> The bug: >>> https://bugs.openjdk.java.net/browse/JDK-8028484 >>> The webrev: >>> http://cr.openjdk.java.net/~dmarkov/8028484/open/webrev.00/ - add to >>> open repo >>> http://cr.openjdk.java.net/~dmarkov/8028484/closed/webrev.00/ - diff >>> with the previous version of the test >>> The review thread: >>> http://mail.openjdk.java.net/pipermail/awt-dev/2013-December/006493.html >>> >>> The jdk9 changeset: >>> http://hg.openjdk.java.net/jdk9/client/jdk/rev/1490b2b2af97 >>> The jdk8u changeset: >>> http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/1490b2b2af97 >>> >>> Thanks, >>> Dmitry >> > From Sergey.Bylokhov at oracle.com Mon Sep 29 12:24:59 2014 From: Sergey.Bylokhov at oracle.com (Sergey Bylokhov) Date: Mon, 29 Sep 2014 16:24:59 +0400 Subject: [7u-dev] Request for approval 8028484: [TEST_BUG][macosx] closed/java/awt/MouseInfo/JContainerMousePositionTest fails In-Reply-To: <54294C58.50709@oracle.com> References: <54104107.9070009@oracle.com> <541041D5.4090102@oracle.com> <5410433D.4060708@oracle.com> <54294C58.50709@oracle.com> Message-ID: <54294F9B.8050800@oracle.com> Looks fine. On 29.09.2014 16:11, Alexander Potochkin wrote: > Hello Dmitry > > The changes look good! > > Thanks > alexp > > On 9/10/2014 4:25 PM, dmitry markov wrote: >> Thank you, Sean! >> >> Anthony, Sergey, >> Could you review this, please? >> >> Thanks, >> Dmitry >> On 10/09/2014 16:19, Se?n Coffey wrote: >>> Approved but subject to code review. >>> >>> regards, >>> Sean. >>> >>> On 10/09/14 13:16, dmitry markov wrote: >>>> Hello, >>>> >>>> Could you approve the back-port of the fix for 8028484 to jdk7u-dev >>>> (7u80), please? >>>> The original fix and the back-port are slightly different. There >>>> are trivial changes in the regression test (lambda was replaced by >>>> anonymous inner class). The author and the reviewers of the >>>> original fix are in the CC list. >>>> >>>> The bug: >>>> https://bugs.openjdk.java.net/browse/JDK-8028484 >>>> The webrev: >>>> http://cr.openjdk.java.net/~dmarkov/8028484/open/webrev.00/ - add >>>> to open repo >>>> http://cr.openjdk.java.net/~dmarkov/8028484/closed/webrev.00/ - >>>> diff with the previous version of the test >>>> The review thread: >>>> http://mail.openjdk.java.net/pipermail/awt-dev/2013-December/006493.html >>>> >>>> The jdk9 changeset: >>>> http://hg.openjdk.java.net/jdk9/client/jdk/rev/1490b2b2af97 >>>> The jdk8u changeset: >>>> http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/1490b2b2af97 >>>> >>>> Thanks, >>>> Dmitry >>> >> > -- Best regards, Sergey. From lana.steuck at oracle.com Mon Sep 29 18:22:26 2014 From: lana.steuck at oracle.com (lana.steuck at oracle.com) Date: Mon, 29 Sep 2014 18:22:26 +0000 Subject: hg: jdk7u/jdk7u: 8057813: Alterations to jdk_security3 test target Message-ID: <201409291822.s8TIMR78026266@aojmv0008> Changeset: ff44294a2be6 Author: coffeys Date: 2014-09-19 17:49 +0100 URL: http://hg.openjdk.java.net/jdk7u/jdk7u/rev/ff44294a2be6 8057813: Alterations to jdk_security3 test target Reviewed-by: mullan ! make/jprt.properties ! test/Makefile From lana.steuck at oracle.com Mon Sep 29 18:22:28 2014 From: lana.steuck at oracle.com (lana.steuck at oracle.com) Date: Mon, 29 Sep 2014 18:22:28 +0000 Subject: hg: jdk7u/jdk7u/hotspot: 11 new changesets Message-ID: <201409291822.s8TIMTbD026272@aojmv0008> Changeset: 6547c22e85df Author: sgabdura Date: 2014-09-04 09:04 +0200 URL: http://hg.openjdk.java.net/jdk7u/jdk7u/hotspot/rev/6547c22e85df 8029190: VM_Version::determine_features() asserts on Fujitsu Sparc64 CPUs Summary: fix code to allow testing on Fujitsu Sparc64 CPUs Reviewed-by: kvn ! src/cpu/sparc/vm/vm_version_sparc.cpp ! src/cpu/sparc/vm/vm_version_sparc.hpp ! src/share/vm/runtime/arguments.cpp Changeset: c6b5db6389c9 Author: sgabdura Date: 2014-09-05 10:19 +0200 URL: http://hg.openjdk.java.net/jdk7u/jdk7u/hotspot/rev/c6b5db6389c9 8025644: java/util/stream/test/org/openjdk/tests/java/util/stream/ToArrayOpTest.java fails with TestData): failure java.lang.AssertionError: expected [true] but found [false] Summary: In GraphKit::gen_checkcast() provide only exact superklass to GraphKit::maybe_cast_profiled_receiver() Reviewed-by: iveresov ! src/share/vm/opto/graphKit.cpp Changeset: 5fd0a42e71aa Author: lana Date: 2014-09-09 11:57 -0700 URL: http://hg.openjdk.java.net/jdk7u/jdk7u/hotspot/rev/5fd0a42e71aa Merge Changeset: f0769f0382d7 Author: kevinw Date: 2014-07-21 10:40 +0100 URL: http://hg.openjdk.java.net/jdk7u/jdk7u/hotspot/rev/f0769f0382d7 8049684: pstack crashes on java core dump Reviewed-by: sundar, sspitsyn ! src/os/bsd/dtrace/libjvm_db.c ! src/os/solaris/dtrace/libjvm_db.c Changeset: e47086077559 Author: coleenp Date: 2014-09-12 14:41 +0100 URL: http://hg.openjdk.java.net/jdk7u/jdk7u/hotspot/rev/e47086077559 6642881: Improve performance of Class.getClassLoader() Summary: Add classLoader to java/lang/Class instance for fast access Reviewed-by: fparain, lfoltan, alanb, mchung, dholmes, kevinw ! src/share/vm/classfile/classFileParser.cpp ! src/share/vm/classfile/javaClasses.cpp ! src/share/vm/classfile/javaClasses.hpp ! src/share/vm/classfile/vmSymbols.hpp Changeset: 34aea5177b9c Author: sgabdura Date: 2014-09-17 10:52 +0200 URL: http://hg.openjdk.java.net/jdk7u/jdk7u/hotspot/rev/34aea5177b9c 8028280: ParkEvent leak when running modified runThese which only loads classes Summary: Use spin lock to manage ParkEvent and PlatformEvent free lists. Reviewed-by: dsimms, dholmes, fparain ! src/cpu/sparc/vm/stubGenerator_sparc.cpp ! src/os/bsd/vm/os_bsd.cpp ! src/os/linux/vm/os_linux.cpp ! src/os/solaris/vm/os_solaris.cpp ! src/os/windows/vm/os_windows.cpp ! src/share/vm/runtime/os.hpp ! src/share/vm/runtime/park.cpp ! src/share/vm/runtime/thread.cpp Changeset: c296492ba8b4 Author: sgabdura Date: 2014-09-17 11:30 +0200 URL: http://hg.openjdk.java.net/jdk7u/jdk7u/hotspot/rev/c296492ba8b4 8038274: update 8u fix for 8028073 now that 8028280 is backported to 8u Reviewed-by: dcubed, coleenp, sspitsyn ! src/share/vm/runtime/objectMonitor.cpp Changeset: 96d116f870b0 Author: vkempik Date: 2014-09-24 13:47 +0400 URL: http://hg.openjdk.java.net/jdk7u/jdk7u/hotspot/rev/96d116f870b0 8012941: JSR 292: too deep inlining might crash compiler because of stack overflow Reviewed-by: kvn, sgabdura ! src/share/vm/c1/c1_GraphBuilder.cpp ! src/share/vm/c1/c1_globals.hpp Changeset: c02c67eb03ae Author: vkempik Date: 2014-09-24 13:49 +0400 URL: http://hg.openjdk.java.net/jdk7u/jdk7u/hotspot/rev/c02c67eb03ae 8028159: C2: compiler stack overflow during inlining of @ForceInline methods Reviewed-by: kvn, sgabdura ! src/share/vm/c1/c1_globals.hpp ! src/share/vm/opto/bytecodeInfo.cpp ! src/share/vm/opto/callGenerator.cpp ! src/share/vm/runtime/globals.hpp Changeset: 40459e83f8c9 Author: roland Date: 2014-06-02 10:01 +0200 URL: http://hg.openjdk.java.net/jdk7u/jdk7u/hotspot/rev/40459e83f8c9 8043413: REGRESSION: Hotspot causes segmentation fault in jdk8ux, but not in jdk7ux Summary: long series of lambda form calls trigger stack overflow in c2 Reviewed-by: kvn, vlivanov ! src/share/vm/opto/bytecodeInfo.cpp Changeset: 4c218194cc6c Author: sla Date: 2014-09-24 09:49 +0200 URL: http://hg.openjdk.java.net/jdk7u/jdk7u/hotspot/rev/4c218194cc6c 8058936: hotspot/test/Makefile should use jtreg script from $JT_HOME/bin/jreg (instead of $JT_HOME/win32/bin/jtreg) Reviewed-by: dholmes, stefank ! test/Makefile From lana.steuck at oracle.com Mon Sep 29 18:22:37 2014 From: lana.steuck at oracle.com (lana.steuck at oracle.com) Date: Mon, 29 Sep 2014 18:22:37 +0000 Subject: hg: jdk7u/jdk7u/jdk: 15 new changesets Message-ID: <201409291822.s8TIMcNL026481@aojmv0008> Changeset: 86bc5adbf7b7 Author: sgabdura Date: 2014-09-04 08:34 +0200 URL: http://hg.openjdk.java.net/jdk7u/jdk7u/jdk/rev/86bc5adbf7b7 8022229: Intermittent test failures in sun/tools/jstatd Reviewed-by: ykantser, sla, egahlin, jbachorik, allwin + test/sun/tools/jstatd/JstatGCUtilParser.java + test/sun/tools/jstatd/JstatdTest.java + test/sun/tools/jstatd/TestJstatdDefaults.java + test/sun/tools/jstatd/TestJstatdExternalRegistry.java + test/sun/tools/jstatd/TestJstatdPort.java + test/sun/tools/jstatd/TestJstatdPortAndServer.java + test/sun/tools/jstatd/TestJstatdServer.java + test/sun/tools/jstatd/TestJstatdUsage.java - test/sun/tools/jstatd/jpsOutput1.awk - test/sun/tools/jstatd/jstatGcutilOutput1.awk - test/sun/tools/jstatd/jstatdDefaults.sh - test/sun/tools/jstatd/jstatdExternalRegistry.sh - test/sun/tools/jstatd/jstatdPort.sh - test/sun/tools/jstatd/jstatdServerName.sh - test/sun/tools/jstatd/jstatdUsage1.sh - test/sun/tools/jstatd/usage.out Changeset: 38181109e3d1 Author: aivanov Date: 2014-09-04 19:16 +0400 URL: http://hg.openjdk.java.net/jdk7u/jdk7u/jdk/rev/38181109e3d1 8056211: api/java_awt/Event/InputMethodEvent/serial/index.html#Input[serial2002] failure Reviewed-by: pchelko, alexsch ! src/share/classes/java/awt/event/InputMethodEvent.java Changeset: fdeb6abac0ea Author: alexsch Date: 2014-09-08 15:37 +0400 URL: http://hg.openjdk.java.net/jdk7u/jdk7u/jdk/rev/fdeb6abac0ea 8057184: JCK8's api/javax_swing/JDesktopPane/descriptions.html#getset failed with GTKLookAndFeel on Linux and Solaris Reviewed-by: ant, azvegint ! src/share/classes/javax/swing/JDesktopPane.java Changeset: 49b0ce28e2ca Author: coffeys Date: 2014-09-09 18:43 +0100 URL: http://hg.openjdk.java.net/jdk7u/jdk7u/jdk/rev/49b0ce28e2ca 8044725: Bug in zlib 1.2.5 prevents inflation of some gzipped files (zlib 1.2.8 port) Reviewed-by: sherman ! make/common/Defs.gmk ! make/common/Program.gmk ! make/java/zip/FILES_c.gmk ! make/sun/splashscreen/FILES_c.gmk - src/share/native/java/util/zip/zlib-1.2.3/ChangeLog - src/share/native/java/util/zip/zlib-1.2.3/README - src/share/native/java/util/zip/zlib-1.2.3/compress.c - src/share/native/java/util/zip/zlib-1.2.3/crc32.h - src/share/native/java/util/zip/zlib-1.2.3/deflate.c - src/share/native/java/util/zip/zlib-1.2.3/deflate.h - src/share/native/java/util/zip/zlib-1.2.3/gzio.c - src/share/native/java/util/zip/zlib-1.2.3/infback.c - src/share/native/java/util/zip/zlib-1.2.3/inffast.c - src/share/native/java/util/zip/zlib-1.2.3/inffast.h - src/share/native/java/util/zip/zlib-1.2.3/inffixed.h - src/share/native/java/util/zip/zlib-1.2.3/inflate.c - src/share/native/java/util/zip/zlib-1.2.3/inflate.h - src/share/native/java/util/zip/zlib-1.2.3/inftrees.c - src/share/native/java/util/zip/zlib-1.2.3/inftrees.h - src/share/native/java/util/zip/zlib-1.2.3/patches/ChangeLog_java - src/share/native/java/util/zip/zlib-1.2.3/patches/crc32.c.diff - src/share/native/java/util/zip/zlib-1.2.3/patches/inflate.c.diff - src/share/native/java/util/zip/zlib-1.2.3/patches/zconf.h.diff - src/share/native/java/util/zip/zlib-1.2.3/patches/zlib.h.diff - src/share/native/java/util/zip/zlib-1.2.3/trees.c - src/share/native/java/util/zip/zlib-1.2.3/trees.h - src/share/native/java/util/zip/zlib-1.2.3/uncompr.c - src/share/native/java/util/zip/zlib-1.2.3/zadler32.c - src/share/native/java/util/zip/zlib-1.2.3/zconf.h - src/share/native/java/util/zip/zlib-1.2.3/zcrc32.c - src/share/native/java/util/zip/zlib-1.2.3/zlib.h - src/share/native/java/util/zip/zlib-1.2.3/zutil.c - src/share/native/java/util/zip/zlib-1.2.3/zutil.h + src/share/native/java/util/zip/zlib-1.2.8/ChangeLog + src/share/native/java/util/zip/zlib-1.2.8/README + src/share/native/java/util/zip/zlib-1.2.8/compress.c + src/share/native/java/util/zip/zlib-1.2.8/crc32.h + src/share/native/java/util/zip/zlib-1.2.8/deflate.c + src/share/native/java/util/zip/zlib-1.2.8/deflate.h + src/share/native/java/util/zip/zlib-1.2.8/gzclose.c + src/share/native/java/util/zip/zlib-1.2.8/gzguts.h + src/share/native/java/util/zip/zlib-1.2.8/gzlib.c + src/share/native/java/util/zip/zlib-1.2.8/gzread.c + src/share/native/java/util/zip/zlib-1.2.8/gzwrite.c + src/share/native/java/util/zip/zlib-1.2.8/infback.c + src/share/native/java/util/zip/zlib-1.2.8/inffast.c + src/share/native/java/util/zip/zlib-1.2.8/inffast.h + src/share/native/java/util/zip/zlib-1.2.8/inffixed.h + src/share/native/java/util/zip/zlib-1.2.8/inflate.c + src/share/native/java/util/zip/zlib-1.2.8/inflate.h + src/share/native/java/util/zip/zlib-1.2.8/inftrees.c + src/share/native/java/util/zip/zlib-1.2.8/inftrees.h + src/share/native/java/util/zip/zlib-1.2.8/patches/ChangeLog_java + src/share/native/java/util/zip/zlib-1.2.8/trees.c + src/share/native/java/util/zip/zlib-1.2.8/trees.h + src/share/native/java/util/zip/zlib-1.2.8/uncompr.c + src/share/native/java/util/zip/zlib-1.2.8/zadler32.c + src/share/native/java/util/zip/zlib-1.2.8/zconf.h + src/share/native/java/util/zip/zlib-1.2.8/zcrc32.c + src/share/native/java/util/zip/zlib-1.2.8/zlib.h + src/share/native/java/util/zip/zlib-1.2.8/zutil.c + src/share/native/java/util/zip/zlib-1.2.8/zutil.h Changeset: 54503c218de8 Author: lana Date: 2014-09-09 11:57 -0700 URL: http://hg.openjdk.java.net/jdk7u/jdk7u/jdk/rev/54503c218de8 Merge - src/share/native/java/util/zip/zlib-1.2.3/ChangeLog - src/share/native/java/util/zip/zlib-1.2.3/README - src/share/native/java/util/zip/zlib-1.2.3/compress.c - src/share/native/java/util/zip/zlib-1.2.3/crc32.h - src/share/native/java/util/zip/zlib-1.2.3/deflate.c - src/share/native/java/util/zip/zlib-1.2.3/deflate.h - src/share/native/java/util/zip/zlib-1.2.3/gzio.c - src/share/native/java/util/zip/zlib-1.2.3/infback.c - src/share/native/java/util/zip/zlib-1.2.3/inffast.c - src/share/native/java/util/zip/zlib-1.2.3/inffast.h - src/share/native/java/util/zip/zlib-1.2.3/inffixed.h - src/share/native/java/util/zip/zlib-1.2.3/inflate.c - src/share/native/java/util/zip/zlib-1.2.3/inflate.h - src/share/native/java/util/zip/zlib-1.2.3/inftrees.c - src/share/native/java/util/zip/zlib-1.2.3/inftrees.h - src/share/native/java/util/zip/zlib-1.2.3/patches/ChangeLog_java - src/share/native/java/util/zip/zlib-1.2.3/patches/crc32.c.diff - src/share/native/java/util/zip/zlib-1.2.3/patches/inflate.c.diff - src/share/native/java/util/zip/zlib-1.2.3/patches/zconf.h.diff - src/share/native/java/util/zip/zlib-1.2.3/patches/zlib.h.diff - src/share/native/java/util/zip/zlib-1.2.3/trees.c - src/share/native/java/util/zip/zlib-1.2.3/trees.h - src/share/native/java/util/zip/zlib-1.2.3/uncompr.c - src/share/native/java/util/zip/zlib-1.2.3/zadler32.c - src/share/native/java/util/zip/zlib-1.2.3/zconf.h - src/share/native/java/util/zip/zlib-1.2.3/zcrc32.c - src/share/native/java/util/zip/zlib-1.2.3/zlib.h - src/share/native/java/util/zip/zlib-1.2.3/zutil.c - src/share/native/java/util/zip/zlib-1.2.3/zutil.h - test/sun/tools/jstatd/jpsOutput1.awk - test/sun/tools/jstatd/jstatGcutilOutput1.awk - test/sun/tools/jstatd/jstatdDefaults.sh - test/sun/tools/jstatd/jstatdExternalRegistry.sh - test/sun/tools/jstatd/jstatdPort.sh - test/sun/tools/jstatd/jstatdServerName.sh - test/sun/tools/jstatd/jstatdUsage1.sh - test/sun/tools/jstatd/usage.out Changeset: 22b4d5ebccb3 Author: igerasim Date: 2014-09-09 19:02 +0400 URL: http://hg.openjdk.java.net/jdk7u/jdk7u/jdk/rev/22b4d5ebccb3 7010989: Duplicate closure of file descriptors leads to unexpected and incorrect closure of sockets Reviewed-by: chegar ! src/windows/native/java/net/TwoStacksPlainDatagramSocketImpl.c ! src/windows/native/java/net/TwoStacksPlainSocketImpl.c ! src/windows/native/java/net/net_util_md.c Changeset: ac93f5941f48 Author: coleenp Date: 2014-09-12 14:36 +0100 URL: http://hg.openjdk.java.net/jdk7u/jdk7u/jdk/rev/ac93f5941f48 6642881: Improve performance of Class.getClassLoader() Summary: Add classLoader to java/lang/Class instance for fast access Reviewed-by: fparain, lfoltan, alanb, mchung, dholmes, kevinw ! src/share/classes/java/lang/Class.java ! src/share/classes/java/lang/reflect/AccessibleObject.java ! src/share/javavm/export/jvm.h ! src/share/native/common/check_code.c ! src/share/native/java/lang/Class.c Changeset: b4e500502985 Author: dmarkov Date: 2014-09-15 12:29 +0400 URL: http://hg.openjdk.java.net/jdk7u/jdk7u/jdk/rev/b4e500502985 8048110: Using tables in JTextPane leads to infinite loop in FlowLayout.layoutRow Reviewed-by: alexp, alexsch ! src/share/classes/javax/swing/text/FlowView.java ! src/share/classes/javax/swing/text/GlyphView.java ! src/share/classes/javax/swing/text/View.java + test/javax/swing/text/View/8048110/bug8048110.java Changeset: 0d4f0a8a1ee5 Author: dmarkov Date: 2014-09-15 13:47 +0400 URL: http://hg.openjdk.java.net/jdk7u/jdk7u/jdk/rev/0d4f0a8a1ee5 8015586: [macosx] Test closed/java/awt/print/PrinterJob/PrintToDir.java fails on MacOSX Reviewed-by: prr, jchen ! src/macosx/classes/sun/lwawt/macosx/CPrinterJob.java ! src/share/classes/sun/print/RasterPrinterJob.java + test/java/awt/print/PrinterJob/PrintToDir.java Changeset: ae7b0ff91866 Author: dmarkov Date: 2014-09-17 12:59 +0400 URL: http://hg.openjdk.java.net/jdk7u/jdk7u/jdk/rev/ae7b0ff91866 8051359: JPopupMenu creation in headless mode with JDK9b23 causes NPE Reviewed-by: serb, pchelko ! src/share/classes/sun/awt/SunToolkit.java Changeset: fd488f5e328f Author: aefimov Date: 2014-09-12 14:23 +0400 URL: http://hg.openjdk.java.net/jdk7u/jdk7u/jdk/rev/fd488f5e328f 8049343: (tz) Support tzdata2014g Reviewed-by: mfang, okutsu ! make/sun/javazic/tzdata/VERSION ! make/sun/javazic/tzdata/africa ! make/sun/javazic/tzdata/antarctica ! make/sun/javazic/tzdata/asia ! make/sun/javazic/tzdata/australasia ! make/sun/javazic/tzdata/backward ! make/sun/javazic/tzdata/etcetera ! make/sun/javazic/tzdata/europe ! make/sun/javazic/tzdata/factory ! make/sun/javazic/tzdata/iso3166.tab ! make/sun/javazic/tzdata/leapseconds ! make/sun/javazic/tzdata/northamerica ! make/sun/javazic/tzdata/pacificnew ! make/sun/javazic/tzdata/southamerica ! make/sun/javazic/tzdata/systemv ! make/sun/javazic/tzdata/zone.tab ! src/share/classes/sun/util/resources/TimeZoneNames.java ! src/share/classes/sun/util/resources/TimeZoneNames_de.java ! src/share/classes/sun/util/resources/TimeZoneNames_es.java ! src/share/classes/sun/util/resources/TimeZoneNames_fr.java ! src/share/classes/sun/util/resources/TimeZoneNames_it.java ! src/share/classes/sun/util/resources/TimeZoneNames_ja.java ! src/share/classes/sun/util/resources/TimeZoneNames_ko.java ! src/share/classes/sun/util/resources/TimeZoneNames_pt_BR.java ! src/share/classes/sun/util/resources/TimeZoneNames_sv.java ! src/share/classes/sun/util/resources/TimeZoneNames_zh_CN.java ! src/share/classes/sun/util/resources/TimeZoneNames_zh_TW.java ! test/sun/util/resources/TimeZone/Bug6317929.java Changeset: 6b53b16c1641 Author: igerasim Date: 2014-09-17 23:52 +0400 URL: http://hg.openjdk.java.net/jdk7u/jdk7u/jdk/rev/6b53b16c1641 8055949: ByteArrayOutputStream capacity should be maximal array size permitted by VM Summary: Try to resize to "well-known" hotspot max array size first. Reviewed-by: alanb, mduigou ! src/share/classes/java/io/ByteArrayOutputStream.java + test/java/io/ByteArrayOutputStream/MaxCapacity.java Changeset: 056ef4238bb4 Author: coffeys Date: 2014-09-19 17:49 +0100 URL: http://hg.openjdk.java.net/jdk7u/jdk7u/jdk/rev/056ef4238bb4 8057813: Alterations to jdk_security3 test target Reviewed-by: mullan ! make/jprt.properties ! test/Makefile Changeset: 9ac8a27d7d19 Author: sgabdura Date: 2014-09-23 09:06 +0200 URL: http://hg.openjdk.java.net/jdk7u/jdk7u/jdk/rev/9ac8a27d7d19 8057564: JVM hangs at getAgentProperties after attaching to VM with lower Summary: Create custom Security Descriptor for Named Pipe. Reviewed-by: mgronlun, dsamersoff, uta ! src/windows/native/sun/tools/attach/WindowsVirtualMachine.c Changeset: 1ed30c084e3d Author: bae Date: 2014-09-24 12:40 +0100 URL: http://hg.openjdk.java.net/jdk7u/jdk7u/jdk/rev/1ed30c084e3d 8025917: JDK demo applets not running with >=7u40 or (JDK 8 and JDK 9) Reviewed-by: alexp ! src/share/demo/README From lana.steuck at oracle.com Mon Sep 29 18:27:10 2014 From: lana.steuck at oracle.com (lana.steuck at oracle.com) Date: Mon, 29 Sep 2014 11:27:10 -0700 (PDT) Subject: jdk7u-b03: jdk7u-dev Message-ID: <201409291827.s8TIRA9J010187@jano-app.us.oracle.com> http://hg.openjdk.java.net/jdk7u/jdk7u/rev/ff44294a2be6 http://hg.openjdk.java.net/jdk7u/jdk7u/langtools/rev/41b81b3e37cd http://hg.openjdk.java.net/jdk7u/jdk7u/jdk/rev/1ed30c084e3d http://hg.openjdk.java.net/jdk7u/jdk7u/jaxws/rev/12905bf82bda http://hg.openjdk.java.net/jdk7u/jdk7u/jaxp/rev/498ddd5ee22c http://hg.openjdk.java.net/jdk7u/jdk7u/hotspot/rev/4c218194cc6c http://hg.openjdk.java.net/jdk7u/jdk7u/corba/rev/34147ec06239 --- All the fixes will be tested during promotion (no PIT testing at this point): List of all fixes: =================== JDK-8057184 client-libs JCK8's api/javax_swing/JDesktopPane/descriptions.html#getset failed with GTKLookAndFeel on Linux and Solaris run JDK-8048110 client-libs Using tables in JTextPane leads to infinite loop in FlowLayout.layoutRow JDK-8051359 client-libs [headless] JPopupMenu creation in headless mode with JDK9b23 causes NPE JDK-8015586 client-libs [macosx] Test closed/java/awt/print/PrinterJob/PrintToDir.java fails on MacOSX JDK-8056211 client-libs api/java_awt/Event/InputMethodEvent/serial/index.html#Input[serial2002] failure JDK-8049343 core-libs (tz) Support tzdata2014g JDK-8044725 core-libs Bug in zlib 1.2.5 prevents inflation of some gzipped files (zlib 1.2.8 port) JDK-8055949 core-libs ByteArrayOutputStream capacity should be maximal array size permitted by VM JDK-7010989 core-libs Duplicate closure of file descriptors leads to unexpected and incorrect closure of sockets JDK-8057747 core-libs Several test failing after update to tzdata2014g JDK-8039936 core-libs Test closed/java/net/SocketPermission/BindTest.java fails in JDK 8u20 nightly build intermittently on Solaris amd64, hard to reproduce. JDK-8049618 core-libs [Test Bug ] Test closed/java/net/SocketPermission/BindTest.java fails on Sctp Channel JDK-8029322 core-libs closed/java/net/SocketPermission/BindTest.java fails intermittently after JDK-8028819 was fixed JDK-8022229 core-svc Intermittent test failures in sun/tools/jstatd JDK-8025917 deploy JDK demo applets not running with >=7u40 or (JDK 8 and JDK 9) JDK-8051890 deploy Java Web Start raises "Unable to create a shortcut for " dialog JDK-8057142 deploy Revert accidental changes in Deployment.java JDK-8055823 deploy [nightly] property keys instead of values in Java Control Panel security tab JDK-8028159 hotspot C2: compiler stack overflow during inlining of @ForceInline methods JDK-6642881 hotspot Improve performance of Class.getClassLoader() JDK-8012941 hotspot JSR 292: too deep inlining might crash compiler because of stack overflow JDK-8057564 hotspot JVM hangs at getAgentProperties after attaching to VM with lower JDK-8028280 hotspot ParkEvent leak when running modified runThese which only loads classes JDK-8043413 hotspot REGRESSION: Hotspot causes segmentation fault in jdk8ux, but not in jdk7ux JDK-8029190 hotspot VM_Version::determine_features() asserts on Fujitsu Sparc64 CPUs JDK-8058936 hotspot hotspot/test/Makefile should use jtreg script from $JT_HOME/bin/jreg (instead of $JT_HOME/win32/bin/jtreg) JDK-8025644 hotspot java/util/stream/test/org/openjdk/tests/java/util/stream/ToArrayOpTest.java fails with TestData$OfRef): failure java.lang.AssertionError: expected [true] but found [false] JDK-8049684 hotspot pstack crashes on java core dump JDK-8038274 hotspot update 8u fix for 8028073 now that 8028280 is backported to 8u JDK-8057813 security-libs Alterations to jdk_security3 test target From dmitry.markov at oracle.com Tue Sep 30 06:11:30 2014 From: dmitry.markov at oracle.com (dmitry.markov at oracle.com) Date: Tue, 30 Sep 2014 06:11:30 +0000 Subject: hg: jdk7u/jdk7u-dev/jdk: 8028484: [TEST_BUG][macosx] closed/java/awt/MouseInfo/JContainerMousePositionTest fails Message-ID: <201409300611.s8U6BUXP018155@aojmv0008> Changeset: a4ad23be4ef7 Author: dmarkov Date: 2014-09-30 10:10 +0400 URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/jdk/rev/a4ad23be4ef7 8028484: [TEST_BUG][macosx] closed/java/awt/MouseInfo/JContainerMousePositionTest fails Reviewed-by: alexp, serb + test/java/awt/MouseInfo/JContainerMousePositionTest.java From sean.coffey at oracle.com Tue Sep 30 08:28:05 2014 From: sean.coffey at oracle.com (=?UTF-8?B?U2XDoW4gQ29mZmV5?=) Date: Tue, 30 Sep 2014 09:28:05 +0100 Subject: [7u-dev] Request for approval : 7085757 7195759 8021121 8055222 8006748 7077119 7028073 6931564 8027695 Message-ID: <542A6995.4090606@oracle.com> Looking to port some currency data and minor Locale updates to jdk7u-dev. All have already been fixed in JDK 8. 7085757: Currency Data: ISO 4217 Amendment 152 7195759: ISO 4217 Amendment 154 8021121: ISO 4217 Amendment Number 156 8055222: Currency update needed for ISO 4217 Amendment #159 8006748: getISO3Country() returns wrong value 7077119: remove past transition dates from CurrencyData.properties file 7028073: The currency symbol for Peru is wrong 6931564: Incorrect display name of Locale for south africa 8027695: There should be a space before % sign in Swedish locale review thread : http://mail.openjdk.java.net/pipermail/i18n-dev/2014-September/001539.html regards, Sean. From anton.nashatyrev at oracle.com Tue Sep 30 11:55:09 2014 From: anton.nashatyrev at oracle.com (Anton Nashatyrev) Date: Tue, 30 Sep 2014 15:55:09 +0400 Subject: [7u-dev] Backport request: 8056914: Right Click Menu for Paste not showing after upgrading to java 7 Message-ID: <542A9A1D.7010203@oracle.com> Hello, please approve the fix backport from jdk8 to jdk7u-dev. The patch was applied clearly with the exception of one change which has failed due to lines shifting, this change has been applied by hands. Bug: https://bugs.openjdk.java.net/browse/JDK-8056914 Jdk8 Changeset: http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/7b4db47b6d68 Jdk9 Review Thread: http://mail.openjdk.java.net/pipermail/awt-dev/2014-September/008473.html Thanks! Anton. From sean.coffey at oracle.com Tue Sep 30 11:56:38 2014 From: sean.coffey at oracle.com (=?UTF-8?B?U2XDoW4gQ29mZmV5?=) Date: Tue, 30 Sep 2014 12:56:38 +0100 Subject: [7u-dev] Backport request: 8056914: Right Click Menu for Paste not showing after upgrading to java 7 In-Reply-To: <542A9A1D.7010203@oracle.com> References: <542A9A1D.7010203@oracle.com> Message-ID: <542A9A76.3030208@oracle.com> Approved. regards, Sean. On 30/09/2014 12:55, Anton Nashatyrev wrote: > Hello, > > please approve the fix backport from jdk8 to jdk7u-dev. The patch > was applied clearly with the exception of one change which has failed > due to lines shifting, this change has been applied by hands. > > Bug: https://bugs.openjdk.java.net/browse/JDK-8056914 > Jdk8 Changeset: > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/7b4db47b6d68 > Jdk9 Review Thread: > http://mail.openjdk.java.net/pipermail/awt-dev/2014-September/008473.html > > Thanks! > Anton. From stefan.johansson at oracle.com Tue Sep 30 12:51:06 2014 From: stefan.johansson at oracle.com (Stefan Johansson) Date: Tue, 30 Sep 2014 14:51:06 +0200 Subject: [7u80] Request for approval 8026303: CMS: JVM intermittently crashes with "FreeList of size 258 violates Conservation Principle" assert Message-ID: <542AA73A.1070203@oracle.com> Hi, Please approve the backport of the fix for JDK-8026303 to jdk7u-dev. This fix resolves the issue reported in JDK-8055730: https://bugs.openjdk.java.net/browse/JDK-8055730 The changeset applies without any conflicts after the changeset has been modified to match JDK-7 source. Two main changes was done to the changeset, see this webrev for details: http://cr.openjdk.java.net/~sjohanss/8026303/webrev.changeset/ Full webrev: http://cr.openjdk.java.net/~sjohanss/8026303/webrev.00/ Original bug: https://bugs.openjdk.java.net/browse/JDK-8026303 Original webrev: http://cr.openjdk.java.net/~jmasa/8026303/webrev.00/ Original changeset: http://hg.openjdk.java.net/jdk9/hs-gc/hotspot/rev/0ade41b2dbeb Review: http://mail.openjdk.java.net/pipermail/hotspot-gc-dev/2013-November/008976.html Thanks, Stefan From anton.nashatyrev at oracle.com Tue Sep 30 13:13:39 2014 From: anton.nashatyrev at oracle.com (anton.nashatyrev at oracle.com) Date: Tue, 30 Sep 2014 13:13:39 +0000 Subject: hg: jdk7u/jdk7u-dev/jdk: 8056914: Right Click Menu for Paste not showing after upgrading to java 7 Message-ID: <201409301313.s8UDDdn6001007@aojmv0008> Changeset: 08ee676353cc Author: anashaty Date: 2014-09-30 17:12 +0400 URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/jdk/rev/08ee676353cc 8056914: Right Click Menu for Paste not showing after upgrading to java 7 Reviewed-by: serb, bae ! src/windows/native/sun/windows/awt_TextArea.cpp ! src/windows/native/sun/windows/awt_TextArea.h ! src/windows/native/sun/windows/awt_TextComponent.cpp ! src/windows/native/sun/windows/awt_TextComponent.h ! src/windows/native/sun/windows/awt_TextField.cpp From rob.mckenna at oracle.com Tue Sep 30 14:40:23 2014 From: rob.mckenna at oracle.com (Rob McKenna) Date: Tue, 30 Sep 2014 15:40:23 +0100 Subject: [7u-dev] Request for approval : 7085757 7195759 8021121 8055222 8006748 7077119 7028073 6931564 8027695 In-Reply-To: <542A6995.4090606@oracle.com> References: <542A6995.4090606@oracle.com> Message-ID: <542AC0D7.4000407@oracle.com> Approved. -Rob On 30/09/14 09:28, Se?n Coffey wrote: > Looking to port some currency data and minor Locale updates to > jdk7u-dev. All have already been fixed in JDK 8. > > 7085757: Currency Data: ISO 4217 Amendment 152 > 7195759: ISO 4217 Amendment 154 > 8021121: ISO 4217 Amendment Number 156 > 8055222: Currency update needed for ISO 4217 Amendment #159 > 8006748: getISO3Country() returns wrong value > 7077119: remove past transition dates from CurrencyData.properties file > 7028073: The currency symbol for Peru is wrong > > 6931564: Incorrect display name of Locale for south africa > 8027695: There should be a space before % sign in Swedish locale > > review thread : > http://mail.openjdk.java.net/pipermail/i18n-dev/2014-September/001539.html > > regards, > Sean. From sean.coffey at oracle.com Tue Sep 30 15:10:53 2014 From: sean.coffey at oracle.com (sean.coffey at oracle.com) Date: Tue, 30 Sep 2014 15:10:53 +0000 Subject: hg: jdk7u/jdk7u-dev/jdk: 3 new changesets Message-ID: <201409301510.s8UFAsoe023511@aojmv0008> Changeset: 74a70385c21d Author: coffeys Date: 2014-09-29 17:36 +0100 URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/jdk/rev/74a70385c21d 7085757: Currency Data: ISO 4217 Amendment 152 7195759: ISO 4217 Amendment 154 8021121: ISO 4217 Amendment Number 156 8055222: Currency update needed for ISO 4217 Amendment #159 8006748: getISO3Country() returns wrong value 7077119: remove past transition dates from CurrencyData.properties file 7028073: The currency symbol for Peru is wrong Reviewed-by: naoto ! src/share/classes/java/util/CurrencyData.properties ! src/share/classes/java/util/LocaleISOData.java ! src/share/classes/sun/util/resources/CurrencyNames.properties ! src/share/classes/sun/util/resources/CurrencyNames_es_PE.properties ! src/share/classes/sun/util/resources/CurrencyNames_lt_LT.properties ! src/share/classes/sun/util/resources/CurrencyNames_lv_LV.properties ! src/share/classes/sun/util/resources/LocaleNames.properties ! test/java/util/Currency/ValidateISO4217.java ! test/java/util/Currency/tablea1.txt ! test/java/util/Locale/LocaleTest.java ! test/sun/text/resources/LocaleData ! test/sun/text/resources/LocaleDataTest.java Changeset: 2c00191a86c3 Author: coffeys Date: 2014-09-29 17:39 +0100 URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/jdk/rev/2c00191a86c3 6931564: Incorrect display name of Locale for south africa 8027695: There should be a space before % sign in Swedish locale Reviewed-by: naoto ! src/share/classes/sun/text/resources/FormatData_sv_SE.java ! src/share/classes/sun/util/resources/LocaleNames_sv.properties ! test/sun/text/resources/LocaleData ! test/sun/text/resources/LocaleDataTest.java Changeset: e975546b12ad Author: coffeys Date: 2014-09-30 16:07 +0100 URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/jdk/rev/e975546b12ad Merge From sean.coffey at oracle.com Tue Sep 30 17:46:35 2014 From: sean.coffey at oracle.com (=?UTF-8?B?U2XDoW4gQ29mZmV5?=) Date: Tue, 30 Sep 2014 18:46:35 +0100 Subject: [7u-dev] Request for approval : 8052406: SSLv2Hello protocol may be filtered out unexpectedly Message-ID: <542AEC7B.3000906@oracle.com> Looking to backport this change from JDK 8u. JDK 8u changeset applies cleanly. bug report : https://bugs.openjdk.java.net/browse/JDK-8052406 changeset : http://hg.openjdk.java.net/jdk8u/jdk8u/jdk/rev/ddba61b06470 regards, Sean. From rob.mckenna at oracle.com Tue Sep 30 18:06:18 2014 From: rob.mckenna at oracle.com (Rob McKenna) Date: Tue, 30 Sep 2014 19:06:18 +0100 Subject: [7u-dev] Request for approval : 8052406: SSLv2Hello protocol may be filtered out unexpectedly In-Reply-To: <542AEC7B.3000906@oracle.com> References: <542AEC7B.3000906@oracle.com> Message-ID: <542AF11A.40309@oracle.com> Approved. -Rob On 30/09/14 18:46, Se?n Coffey wrote: > Looking to backport this change from JDK 8u. JDK 8u changeset applies > cleanly. > > bug report : https://bugs.openjdk.java.net/browse/JDK-8052406 > changeset : http://hg.openjdk.java.net/jdk8u/jdk8u/jdk/rev/ddba61b06470 > > regards, > Sean. >