From gromero at linux.vnet.ibm.com Mon Oct 2 17:11:54 2017 From: gromero at linux.vnet.ibm.com (Gustavo Romero) Date: Mon, 2 Oct 2017 14:11:54 -0300 Subject: [8u] RFA for backport of 8168318: PPC64: Use cmpldi instead of li/cmpld In-Reply-To: <4f1b4d73-38fb-9b32-8d29-aad4a427ecdb@oracle.com> References: <59CEBDD9.7090305@linux.vnet.ibm.com> <4f1b4d73-38fb-9b32-8d29-aad4a427ecdb@oracle.com> Message-ID: <59D2735A.8080405@linux.vnet.ibm.com> Hi David! On 29-09-2017 19:37, David Buck wrote: > Hi Gustavo! > > As this would be a backport of an enhancement, you will need to file a separate enhancement backport request first. Please see rule 3 [0] for details. Thanks, I'll send a separate enhancement backport request. Regards, Gustavo > Cheers, > -Buck > > [0] http://openjdk.java.net/projects/jdk8u/groundrules.html > > On 2017/09/29 14:40, Gustavo Romero wrote: >> Hi, >> >> Could the following backport be approved please? >> >> It's a PPC64-only change and applies cleanly. >> >> An improvement of ~1% was observed on some Neo4j benchmarks. >> >> The original author, Igor, was from Eldorado organization which has already >> signed the OCA [1] accordingly. >> >> Bug : https://bugs.openjdk.java.net/browse/JDK-8168318 >> Changeset : http://hg.openjdk.java.net/jdk9/jdk9/hotspot/rev/622d3fe587f2 >> Review thread : http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/2016-October/024771.html >> >> Thank you and best regards, >> Gustavo >> >> [1] http://www.oracle.com/technetwork/community/oca-486395.html "Instituto de Pesquisas Eldorado - OpenJDK" >> > From gromero at linux.vnet.ibm.com Mon Oct 2 17:20:18 2017 From: gromero at linux.vnet.ibm.com (Gustavo Romero) Date: Mon, 2 Oct 2017 14:20:18 -0300 Subject: [8u] Request for enhancement backport approval for 8168318: PPC64: Use cmpldi instead of li/cmpld Message-ID: <59D27552.4030004@linux.vnet.ibm.com> Hi, Could the following enhancement backport be approved, please? Bug : https://bugs.openjdk.java.net/browse/JDK-8168318 Review : http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/2016-October/024771.html Changeset : http://hg.openjdk.java.net/jdk9/jdk9/hotspot/rev/622d3fe587f2 An improvement of ~1% was observed on some Neo4j benchmarks with that change applied. This is a significant ppc64-only performance improvement and applies cleanly. It's low risk and does not affect any of the Oracle platforms in any way. The proposed integration date is as soon as granted since there is no remaining work to be done. Thank you. Best regards, Gustavo From gromero at linux.vnet.ibm.com Mon Oct 2 17:24:32 2017 From: gromero at linux.vnet.ibm.com (Gustavo Romero) Date: Mon, 2 Oct 2017 14:24:32 -0300 Subject: [8u] Request for enhancement backport approval for 8170328: PPC64: Use andis instead of lis/and Message-ID: <59D27650.3040604@linux.vnet.ibm.com> Hi, Could the following enhancement backport be approved, please? Bug : https://bugs.openjdk.java.net/browse/JDK-8170328 Review : http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/2016-November/024997.html Changeset : http://hg.openjdk.java.net/jdk9/jdk9/hotspot/rev/3866c59ee901 Some Neo4j workloads (like LDBC benchmark) can get a benefit from that change. This is a significant ppc64-only performance improvement which improves JITed code and it applies cleanly. It's low risk and does not affect any of the Oracle platforms in any way. The proposed integration date is as soon as granted since there is no remaining work to be done. Thank you. Best regards, Gustavo From rob.mckenna at oracle.com Tue Oct 3 13:16:31 2017 From: rob.mckenna at oracle.com (Rob McKenna) Date: Tue, 3 Oct 2017 14:16:31 +0100 Subject: [8u] Request for enhancement backport approval for 8168318: PPC64: Use cmpldi instead of li/cmpld In-Reply-To: <59D27552.4030004@linux.vnet.ibm.com> References: <59D27552.4030004@linux.vnet.ibm.com> Message-ID: <20171003131631.GA2937@vimes> This request has been approved. -Rob On 02/10/17 14:20, Gustavo Romero wrote: > Hi, > > Could the following enhancement backport be approved, please? > > Bug : https://bugs.openjdk.java.net/browse/JDK-8168318 > Review : http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/2016-October/024771.html > Changeset : http://hg.openjdk.java.net/jdk9/jdk9/hotspot/rev/622d3fe587f2 > > An improvement of ~1% was observed on some Neo4j benchmarks with that change > applied. This is a significant ppc64-only performance improvement and applies > cleanly. It's low risk and does not affect any of the Oracle platforms in any > way. The proposed integration date is as soon as granted since there is no > remaining work to be done. > > Thank you. > > > Best regards, > Gustavo > From rob.mckenna at oracle.com Tue Oct 3 13:16:43 2017 From: rob.mckenna at oracle.com (Rob McKenna) Date: Tue, 3 Oct 2017 14:16:43 +0100 Subject: [8u] Request for enhancement backport approval for 8170328: PPC64: Use andis instead of lis/and In-Reply-To: <59D27650.3040604@linux.vnet.ibm.com> References: <59D27650.3040604@linux.vnet.ibm.com> Message-ID: <20171003131643.GB2937@vimes> This request has been approved. -Rob On 02/10/17 14:24, Gustavo Romero wrote: > Hi, > > Could the following enhancement backport be approved, please? > > Bug : https://bugs.openjdk.java.net/browse/JDK-8170328 > Review : http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/2016-November/024997.html > Changeset : http://hg.openjdk.java.net/jdk9/jdk9/hotspot/rev/3866c59ee901 > > Some Neo4j workloads (like LDBC benchmark) can get a benefit from that change. > This is a significant ppc64-only performance improvement which improves JITed > code and it applies cleanly. It's low risk and does not affect any of the Oracle > platforms in any way. The proposed integration date is as soon as granted since > there is no remaining work to be done. > > Thank you. > > > Best regards, > Gustavo > From rob.mckenna at oracle.com Tue Oct 3 13:17:13 2017 From: rob.mckenna at oracle.com (Rob McKenna) Date: Tue, 3 Oct 2017 14:17:13 +0100 Subject: [8u] RFA for backport of 8168318: PPC64: Use cmpldi instead of li/cmpld In-Reply-To: <59D2735A.8080405@linux.vnet.ibm.com> References: <59CEBDD9.7090305@linux.vnet.ibm.com> <4f1b4d73-38fb-9b32-8d29-aad4a427ecdb@oracle.com> <59D2735A.8080405@linux.vnet.ibm.com> Message-ID: <20171003131713.GC2937@vimes> Approved -Rob On 02/10/17 14:11, Gustavo Romero wrote: > Hi David! > > On 29-09-2017 19:37, David Buck wrote: > > Hi Gustavo! > > > > As this would be a backport of an enhancement, you will need to file a separate enhancement backport request first. Please see rule 3 [0] for details. > > Thanks, I'll send a separate enhancement backport request. > > Regards, > Gustavo > > > Cheers, > > -Buck > > > > [0] http://openjdk.java.net/projects/jdk8u/groundrules.html > > > > On 2017/09/29 14:40, Gustavo Romero wrote: > >> Hi, > >> > >> Could the following backport be approved please? > >> > >> It's a PPC64-only change and applies cleanly. > >> > >> An improvement of ~1% was observed on some Neo4j benchmarks. > >> > >> The original author, Igor, was from Eldorado organization which has already > >> signed the OCA [1] accordingly. > >> > >> Bug : https://bugs.openjdk.java.net/browse/JDK-8168318 > >> Changeset : http://hg.openjdk.java.net/jdk9/jdk9/hotspot/rev/622d3fe587f2 > >> Review thread : http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/2016-October/024771.html > >> > >> Thank you and best regards, > >> Gustavo > >> > >> [1] http://www.oracle.com/technetwork/community/oca-486395.html "Instituto de Pesquisas Eldorado - OpenJDK" > >> > > > From rob.mckenna at oracle.com Tue Oct 3 13:17:46 2017 From: rob.mckenna at oracle.com (Rob McKenna) Date: Tue, 3 Oct 2017 14:17:46 +0100 Subject: [8u] RFA for backport of 8170328: PPC64: Use andis instead of lis/and In-Reply-To: <7bc7522a-6360-3d1f-96ae-99c93938416d@oracle.com> References: <59CEC345.4070705@linux.vnet.ibm.com> <7bc7522a-6360-3d1f-96ae-99c93938416d@oracle.com> Message-ID: <20171003131746.GD2937@vimes> Approved -Rob On 29/09/17 15:38, David Buck wrote: > Hi Gustavo! > > As this would be a backport of an enhancement, you will need to file a > separate enhancement backport request first. Please see rule 3 [0] for > details. > > Cheers, > -Buck > > [0] http://openjdk.java.net/projects/jdk8u/groundrules.html > > On 2017/09/29 15:03, Gustavo Romero wrote: > >Hi, > > > >Could the following backport be approved please? > > > >It's a PPC64-only change and it applies cleanly. > > > >The original author, Igor, was from Eldorado organization which has already > >signed the OCA [1] accordingly. > > > >Bug : https://bugs.openjdk.java.net/browse/JDK-8170328 > >Changeset : http://hg.openjdk.java.net/jdk9/jdk9/hotspot/rev/3866c59ee901 > >Review thread : http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/2016-November/024997.html > > > >Thank you and best regards, > >Gustavo > > > >[1] http://www.oracle.com/technetwork/community/oca-486395.html "Instituto de Pesquisas Eldorado - OpenJDK" > > From gromero at linux.vnet.ibm.com Tue Oct 3 13:36:53 2017 From: gromero at linux.vnet.ibm.com (Gustavo Romero) Date: Tue, 3 Oct 2017 10:36:53 -0300 Subject: [8u] RFA for backport of 8168318: PPC64: Use cmpldi instead of li/cmpld In-Reply-To: <20171003131713.GC2937@vimes> References: <59CEBDD9.7090305@linux.vnet.ibm.com> <4f1b4d73-38fb-9b32-8d29-aad4a427ecdb@oracle.com> <59D2735A.8080405@linux.vnet.ibm.com> <20171003131713.GC2937@vimes> Message-ID: <59D39275.2070400@linux.vnet.ibm.com> Hi Rob, On 03-10-2017 10:17, Rob McKenna wrote: > Approved Thank you. I need a sponsor for that change. Could somebody sponsor it please? Regards, Gustavo From gromero at linux.vnet.ibm.com Tue Oct 3 13:37:42 2017 From: gromero at linux.vnet.ibm.com (Gustavo Romero) Date: Tue, 3 Oct 2017 10:37:42 -0300 Subject: [8u] RFA for backport of 8170328: PPC64: Use andis instead of lis/and In-Reply-To: <20171003131746.GD2937@vimes> References: <59CEC345.4070705@linux.vnet.ibm.com> <7bc7522a-6360-3d1f-96ae-99c93938416d@oracle.com> <20171003131746.GD2937@vimes> Message-ID: <59D392A6.6020007@linux.vnet.ibm.com> Hi Rob, On 03-10-2017 10:17, Rob McKenna wrote: > Approved Thank you. I need a sponsor for that change. Could somebody sponsor it please? Regards, Gustavo From sreeprakash.s at oracle.com Fri Oct 6 14:09:36 2017 From: sreeprakash.s at oracle.com (Sreeprakash Sreedharan) Date: Fri, 6 Oct 2017 07:09:36 -0700 (PDT) Subject: RFA for backport of JDK-8184016: Text in native popup is not always updated with Sogou IME Message-ID: <46509143-51e4-4dab-a343-c72f95d3066c@default> Hi, May I get the approval for backport of 'JDK-8184016:: Text in native popup is not always updated with Sogou IME' to jdk8u-dev. The patch from JDK-10 was applied cleanly. JDK10 bug: https://bugs.openjdk.java.net/browse/JDK-8184016 JDK10 changeset: hg.openjdk.java.net/jdk10/jdk10/jdk/rev/72c480992841 Ensured that there are no failures related to this on JPRT (both west and Stockholm) Queues. Regards, Sreeprakash From rob.mckenna at oracle.com Fri Oct 6 14:20:37 2017 From: rob.mckenna at oracle.com (Rob McKenna) Date: Fri, 6 Oct 2017 15:20:37 +0100 Subject: RFA for backport of JDK-8184016: Text in native popup is not always updated with Sogou IME In-Reply-To: <46509143-51e4-4dab-a343-c72f95d3066c@default> References: <46509143-51e4-4dab-a343-c72f95d3066c@default> Message-ID: <20171006142037.GA3940@vimes> Approved -Rob On 06/10/17 07:09, Sreeprakash Sreedharan wrote: > Hi, > > > > May I get the approval for backport of 'JDK-8184016:: Text in native popup is not always updated with Sogou IME' to jdk8u-dev. > > > > The patch from JDK-10 was applied cleanly. > > > > JDK10 bug: https://bugs.openjdk.java.net/browse/JDK-8184016 > > JDK10 changeset: hg.openjdk.java.net/jdk10/jdk10/jdk/rev/72c480992841 > > > > Ensured that there are no failures related to this on JPRT (both west and Stockholm) Queues. > > > > Regards, > > Sreeprakash > > From sreeprakash.s at oracle.com Mon Oct 9 12:26:01 2017 From: sreeprakash.s at oracle.com (Sreeprakash Sreedharan) Date: Mon, 9 Oct 2017 05:26:01 -0700 (PDT) Subject: [8u-dev] RFA: JDK8U Backport of 8180370: Characters are skipped on input of Korean text on OS X Message-ID: <50f6a54e-8878-460f-b668-afae6367ebf7@default> Hi, May I get the approval for backport of 'JDK-8180370:Characters are skipped on input of Korean text on OS X' Main Bug: https://bugs.openjdk.java.net/browse/JDK-8180370 JDK10 review thread: http://mail.openjdk.java.net/pipermail/awt-dev/2017-June/012881.html JDK10 changeset: http://hg.openjdk.java.net/jdk10/jdk10/jdk/rev/7d3e0c5b5e25 Changes were not applied cleanly and jdk8u-dev review has been done in a separate thread. Jdk8u webrev: http://cr.openjdk.java.net/~ssreedharan/8180370/jdk8u-dev/webrev.00/ Jdk8u review thread: http://mail.openjdk.java.net/pipermail/awt-dev/2017-October/013173.html I have tested out all the relevant Swing and AWT JTREG tests. Also, ensured that there are no failures related to this on JPRT (both west and Stockholm) Queues. Regards, Sreeprakash From david.buck at oracle.com Mon Oct 9 13:56:24 2017 From: david.buck at oracle.com (David Buck) Date: Mon, 9 Oct 2017 06:56:24 -0700 Subject: [8u-dev] RFA: JDK8U Backport of 8180370: Characters are skipped on input of Korean text on OS X In-Reply-To: <50f6a54e-8878-460f-b668-afae6367ebf7@default> References: <50f6a54e-8878-460f-b668-afae6367ebf7@default> Message-ID: <6b8c7342-0760-2ca8-8dc4-23366eaa0228@oracle.com> approved for push to 8u-dev Cheers, -Buck On 2017/10/09 5:26, Sreeprakash Sreedharan wrote: > Hi, > > > > May I get the approval for backport of 'JDK-8180370:Characters are skipped on input of Korean text on OS X' > > > > Main Bug: https://bugs.openjdk.java.net/browse/JDK-8180370 > > JDK10 review thread: http://mail.openjdk.java.net/pipermail/awt-dev/2017-June/012881.html > > JDK10 changeset: http://hg.openjdk.java.net/jdk10/jdk10/jdk/rev/7d3e0c5b5e25 > > > > Changes were not applied cleanly and jdk8u-dev review has been done in a separate thread. > > > > Jdk8u webrev: http://cr.openjdk.java.net/~ssreedharan/8180370/jdk8u-dev/webrev.00/ > > Jdk8u review thread: http://mail.openjdk.java.net/pipermail/awt-dev/2017-October/013173.html > > > > I have tested out all the relevant Swing and AWT JTREG tests. > > Also, ensured that there are no failures related to this on JPRT (both west and Stockholm) Queues. > > > > Regards, > > Sreeprakash > > > From dmitry.markov at oracle.com Mon Oct 9 16:17:22 2017 From: dmitry.markov at oracle.com (Dmitry Markov) Date: Mon, 9 Oct 2017 17:17:22 +0100 Subject: [8u-dev] Request for approval 8139218 & 8159432 Message-ID: <5FF94162-7C4F-4DCA-9D0C-8C6F86CDBA13@oracle.com> Hello, Can I get an approval to backport the following fixes: 8139218 and 8159432 to 8u-dev, please? Both patches apply cleanly. Bug: https://bugs.openjdk.java.net/browse/JDK-8139218 (Dialog that opens and closes quickly changes focus in original focus owner) Review: http://mail.openjdk.java.net/pipermail/awt-dev/2015-December/010467.html http://mail.openjdk.java.net/pipermail/awt-dev/2016-February/010648.html jdk9 changeset: http://hg.openjdk.java.net/jdk9/jdk9/jdk/rev/209dfceab2e3 Bug: https://bugs.openjdk.java.net/browse/JDK-8159432 ([PIT][macosx] StackOverflow in closed/java/awt/Dialog/DialogDeadlock/DialogDeadlockTest) Review: http://mail.openjdk.java.net/pipermail/awt-dev/2016-August/011824.html http://mail.openjdk.java.net/pipermail/awt-dev/2016-November/012252.html jdk9 changeset: http://hg.openjdk.java.net/jdk9/jdk9/jdk/rev/55c54a8c7206 Cumulative jdk8u webrev: http://cr.openjdk.java.net/~dmarkov/8139218_and_8159432/jdk8u/webrev.00/ Thanks, Dmitry From ivan.gerasimov at oracle.com Mon Oct 9 21:14:12 2017 From: ivan.gerasimov at oracle.com (Ivan Gerasimov) Date: Mon, 9 Oct 2017 14:14:12 -0700 Subject: [8u-dev] Request for Approval to Backport: 8031661: java/net/Authenticator/B4769350.java failed intermittently Message-ID: <304fe185-94fc-e235-950a-ec4054918c35@oracle.com> Hello! May I ask for approval to backport this test-only fix? The patch applies cleanly. Bug: https://bugs.openjdk.java.net/browse/JDK-8031661 Jdk9 change: http://hg.openjdk.java.net/jdk9/jdk9/jdk/rev/3a102a5cc9d1 Jdk9 review: http://mail.openjdk.java.net/pipermail/net-dev/2014-February/008223.html Thanks in advance! -- With kind regards, Ivan Gerasimov From david.buck at oracle.com Mon Oct 9 21:16:49 2017 From: david.buck at oracle.com (David Buck) Date: Mon, 9 Oct 2017 17:16:49 -0400 Subject: [8u-dev] Request for Approval to Backport: 8031661: java/net/Authenticator/B4769350.java failed intermittently In-Reply-To: <304fe185-94fc-e235-950a-ec4054918c35@oracle.com> References: <304fe185-94fc-e235-950a-ec4054918c35@oracle.com> Message-ID: <730fb421-5e71-597a-3c25-5d560d62f183@oracle.com> approved for backport to 8u-dev Cheers, -Buck On 2017/10/09 17:14, Ivan Gerasimov wrote: > Hello! > > May I ask for approval to backport this test-only fix? > The patch applies cleanly. > > Bug: https://bugs.openjdk.java.net/browse/JDK-8031661 > Jdk9 change: http://hg.openjdk.java.net/jdk9/jdk9/jdk/rev/3a102a5cc9d1 > Jdk9 review: > http://mail.openjdk.java.net/pipermail/net-dev/2014-February/008223.html > > Thanks in advance! > From david.buck at oracle.com Tue Oct 10 10:49:08 2017 From: david.buck at oracle.com (David Buck) Date: Tue, 10 Oct 2017 06:49:08 -0400 Subject: [8u-dev] Request for approval 8139218 & 8159432 In-Reply-To: <5FF94162-7C4F-4DCA-9D0C-8C6F86CDBA13@oracle.com> References: <5FF94162-7C4F-4DCA-9D0C-8C6F86CDBA13@oracle.com> Message-ID: <5cae4d05-3289-db97-f2fc-ccd7ae28c0f5@oracle.com> both approved for backport to 8u-dev Cheers, -Buck On 2017/10/09 12:17, Dmitry Markov wrote: > Hello, > > Can I get an approval to backport the following fixes: 8139218 and 8159432 to 8u-dev, please? Both patches apply cleanly. > > Bug: > https://bugs.openjdk.java.net/browse/JDK-8139218 > (Dialog that opens and closes quickly changes focus in original focus owner) > Review: > http://mail.openjdk.java.net/pipermail/awt-dev/2015-December/010467.html > http://mail.openjdk.java.net/pipermail/awt-dev/2016-February/010648.html > jdk9 changeset: > http://hg.openjdk.java.net/jdk9/jdk9/jdk/rev/209dfceab2e3 > > Bug: > https://bugs.openjdk.java.net/browse/JDK-8159432 > ([PIT][macosx] StackOverflow in closed/java/awt/Dialog/DialogDeadlock/DialogDeadlockTest) > Review: > http://mail.openjdk.java.net/pipermail/awt-dev/2016-August/011824.html > http://mail.openjdk.java.net/pipermail/awt-dev/2016-November/012252.html > jdk9 changeset: > http://hg.openjdk.java.net/jdk9/jdk9/jdk/rev/55c54a8c7206 > > Cumulative jdk8u webrev: > http://cr.openjdk.java.net/~dmarkov/8139218_and_8159432/jdk8u/webrev.00/ > > Thanks, > Dmitry > From dmitry.markov at oracle.com Tue Oct 10 13:59:25 2017 From: dmitry.markov at oracle.com (Dmitry Markov) Date: Tue, 10 Oct 2017 14:59:25 +0100 Subject: [8u-dev] Request for approval 8155197: Focus transition issue Message-ID: <703CCC9C-7566-4B4E-9FE4-DC8611F6356B@oracle.com> Hello, Can I get an approval to port the fix for 8155197 to jdk8u-dev, please? The jdk10 patch applies cleanly. JBS: https://bugs.openjdk.java.net/browse/JDK-8155197 Review thread: http://mail.openjdk.java.net/pipermail/awt-dev/2017-September/013098.html http://mail.openjdk.java.net/pipermail/awt-dev/2017-October/013156.html jdk10 changeset: http://hg.openjdk.java.net/jdk10/client/rev/40ba3aa34970 Thanks, Dmitry From rob.mckenna at oracle.com Tue Oct 10 14:05:01 2017 From: rob.mckenna at oracle.com (Rob McKenna) Date: Tue, 10 Oct 2017 15:05:01 +0100 Subject: [8u-dev] Request for approval 8155197: Focus transition issue In-Reply-To: <703CCC9C-7566-4B4E-9FE4-DC8611F6356B@oracle.com> References: <703CCC9C-7566-4B4E-9FE4-DC8611F6356B@oracle.com> Message-ID: <20171010140501.GB3324@vimes> Approved -Rob On 10/10/17 14:59, Dmitry Markov wrote: > Hello, > > Can I get an approval to port the fix for 8155197 to jdk8u-dev, please? The jdk10 patch applies cleanly. > > JBS: https://bugs.openjdk.java.net/browse/JDK-8155197 > Review thread: http://mail.openjdk.java.net/pipermail/awt-dev/2017-September/013098.html > http://mail.openjdk.java.net/pipermail/awt-dev/2017-October/013156.html > jdk10 changeset: http://hg.openjdk.java.net/jdk10/client/rev/40ba3aa34970 > > Thanks, > Dmitry From ivan.gerasimov at oracle.com Tue Oct 10 19:51:01 2017 From: ivan.gerasimov at oracle.com (Ivan Gerasimov) Date: Tue, 10 Oct 2017 12:51:01 -0700 Subject: [8u-dev] Request for Approval to Backport: 8187023: Cannot read pkcs11 config file in UTF-16 environment Message-ID: Hello! Would you please approve the backport to jdk8u-dev? The unshuffled fix applies cleanly. Bug: https://bugs.openjdk.java.net/browse/JDK-8187023 Jdk10 change: http://hg.openjdk.java.net/jdk10/jdk10/jdk/rev/ddc25f646c2e Jdk10 review: http://mail.openjdk.java.net/pipermail/security-dev/2017-August/016282.html Thanks in advance! -- With kind regards, Ivan Gerasimov From david.buck at oracle.com Tue Oct 10 19:59:22 2017 From: david.buck at oracle.com (David Buck) Date: Tue, 10 Oct 2017 15:59:22 -0400 Subject: [8u-dev] Request for Approval to Backport: 8187023: Cannot read pkcs11 config file in UTF-16 environment In-Reply-To: References: Message-ID: <35d02e4c-716c-632b-66ca-42241b2b817f@oracle.com> approved for backport to 8u-dev Cheers, -Buck On 2017/10/10 15:51, Ivan Gerasimov wrote: > Hello! > > Would you please approve the backport to jdk8u-dev? > The unshuffled fix applies cleanly. > > Bug: https://bugs.openjdk.java.net/browse/JDK-8187023 > Jdk10 change: http://hg.openjdk.java.net/jdk10/jdk10/jdk/rev/ddc25f646c2e > Jdk10 review: > http://mail.openjdk.java.net/pipermail/security-dev/2017-August/016282.html > > Thanks in advance! > From Horvath.Bystrik at scheidt-bachmann.sk Wed Oct 11 06:40:56 2017 From: Horvath.Bystrik at scheidt-bachmann.sk (Horvath.Bystrik) Date: Wed, 11 Oct 2017 06:40:56 +0000 Subject: [8u-dev] Question regarding JDK-8129861 Message-ID: <9828b56bd89145e89be9d620bc4134e3@SZA-EX2013.sb-sk.local> Hello, I would like to ask why the issue JDK-8129861 : High processor load for ScheduledThreadPoolExecutor with 0 core threads (http://bugs.java.com/view_bug.do?bug_id=8129861) is not part of any of JDK8 update. It is fixed in JDK9, but not officially backported. There are several issues from java.util.concurrent treated like this. The backport exists as a part of enhancements of JSR166 - http://gee.cs.oswego.edu/cgi-bin/viewcvs.cgi/jsr166/src/jdk8/java/util/ , but it is not a part of any update of JDK8. Thank you in advance for the answer. Best regards, Bystrik From david.holmes at oracle.com Wed Oct 11 06:50:24 2017 From: david.holmes at oracle.com (David Holmes) Date: Wed, 11 Oct 2017 16:50:24 +1000 Subject: [8u-dev] Question regarding JDK-8129861 In-Reply-To: <9828b56bd89145e89be9d620bc4134e3@SZA-EX2013.sb-sk.local> References: <9828b56bd89145e89be9d620bc4134e3@SZA-EX2013.sb-sk.local> Message-ID: Hi, On 11/10/2017 4:40 PM, Horvath.Bystrik wrote: > Hello, > > I would like to ask why the issue JDK-8129861 : High processor load for ScheduledThreadPoolExecutor with 0 core threads (http://bugs.java.com/view_bug.do?bug_id=8129861) is not part of any of JDK8 update. It is fixed in JDK9, but not officially backported. There are several issues from java.util.concurrent treated like this. The backport exists as a part of enhancements of JSR166 - http://gee.cs.oswego.edu/cgi-bin/viewcvs.cgi/jsr166/src/jdk8/java/util/ , but it is not a part of any update of JDK8. The changes were done as a bulk update for 9: https://bugs.openjdk.java.net/browse/JDK-8134853 which includes some slight specification changes that may not be suitable for an update release. Individually backporting those changes that can be backported is doable but you have to find someone to actually do it. Not all bug fixes are automatically backported. David > Thank you in advance for the answer. > > Best regards, > Bystrik > From gnu.andrew at redhat.com Thu Oct 12 17:42:27 2017 From: gnu.andrew at redhat.com (Andrew Hughes) Date: Thu, 12 Oct 2017 18:42:27 +0100 Subject: [8u-dev] Request for backport approval: 8145913: PPC64: add Montgomery multiply intrinsic In-Reply-To: <20170929173807.GA2854@vimes> References: <20170927152236.GC6441@vimes> <8f85be8893c14c6f8e106250a93e84b0@sap.com> <20170929173807.GA2854@vimes> Message-ID: On 29 September 2017 at 18:38, Rob McKenna wrote: > Hi Gustavo, > > I'm happy with Martins review. Thanks, > > -Rob > Built on ppc64 and ppc64le with RHEL 7, and pushed on Gustavo's behalf: http://hg.openjdk.java.net/jdk8u/jdk8u-dev/hotspot/rev/9575483cce09 -- Andrew :) Senior Free Java Software Engineer Red Hat, Inc. (http://www.redhat.com) Web Site: http://fuseyism.com Twitter: https://twitter.com/gnu_andrew_java PGP Key: ed25519/0xCFDA0F9B35964222 (hkp://keys.gnupg.net) Fingerprint = 5132 579D D154 0ED2 3E04 C5A0 CFDA 0F9B 3596 4222 From gnu.andrew at redhat.com Thu Oct 12 17:43:39 2017 From: gnu.andrew at redhat.com (Andrew Hughes) Date: Thu, 12 Oct 2017 18:43:39 +0100 Subject: [8u] RFA for backport of 8168318: PPC64: Use cmpldi instead of li/cmpld In-Reply-To: <59D39275.2070400@linux.vnet.ibm.com> References: <59CEBDD9.7090305@linux.vnet.ibm.com> <4f1b4d73-38fb-9b32-8d29-aad4a427ecdb@oracle.com> <59D2735A.8080405@linux.vnet.ibm.com> <20171003131713.GC2937@vimes> <59D39275.2070400@linux.vnet.ibm.com> Message-ID: On 3 October 2017 at 14:36, Gustavo Romero wrote: > Hi Rob, > > On 03-10-2017 10:17, Rob McKenna wrote: >> Approved > > Thank you. > > I need a sponsor for that change. Could somebody sponsor it please? > > > Regards, > Gustavo > Built on ppc64 and ppc64le with RHEL 7 and pushed on Gustavo's behalf: http://hg.openjdk.java.net/jdk8u/jdk8u-dev/hotspot/rev/75f5e49c6187 -- Andrew :) Senior Free Java Software Engineer Red Hat, Inc. (http://www.redhat.com) Web Site: http://fuseyism.com Twitter: https://twitter.com/gnu_andrew_java PGP Key: ed25519/0xCFDA0F9B35964222 (hkp://keys.gnupg.net) Fingerprint = 5132 579D D154 0ED2 3E04 C5A0 CFDA 0F9B 3596 4222 From gnu.andrew at redhat.com Thu Oct 12 17:44:28 2017 From: gnu.andrew at redhat.com (Andrew Hughes) Date: Thu, 12 Oct 2017 18:44:28 +0100 Subject: [8u] RFA for backport of 8170328: PPC64: Use andis instead of lis/and In-Reply-To: <59D392A6.6020007@linux.vnet.ibm.com> References: <59CEC345.4070705@linux.vnet.ibm.com> <7bc7522a-6360-3d1f-96ae-99c93938416d@oracle.com> <20171003131746.GD2937@vimes> <59D392A6.6020007@linux.vnet.ibm.com> Message-ID: On 3 October 2017 at 14:37, Gustavo Romero wrote: > Hi Rob, > > On 03-10-2017 10:17, Rob McKenna wrote: >> Approved > > Thank you. > > I need a sponsor for that change. Could somebody sponsor it please? > > > Regards, > Gustavo > Built on ppc64 and ppc64le with RHEL 7 and pushed on Gustavo's behalf: http://hg.openjdk.java.net/jdk8u/jdk8u-dev/hotspot/rev/7c3dc80542b6 -- Andrew :) Senior Free Java Software Engineer Red Hat, Inc. (http://www.redhat.com) Web Site: http://fuseyism.com Twitter: https://twitter.com/gnu_andrew_java PGP Key: ed25519/0xCFDA0F9B35964222 (hkp://keys.gnupg.net) Fingerprint = 5132 579D D154 0ED2 3E04 C5A0 CFDA 0F9B 3596 4222 From gnu.andrew at redhat.com Thu Oct 12 17:45:26 2017 From: gnu.andrew at redhat.com (Andrew Hughes) Date: Thu, 12 Oct 2017 18:45:26 +0100 Subject: [8u] RFA for backport of 8181810: PPC64: Leverage extrdi for bitfield extract In-Reply-To: <59CE64FB.7090103@linux.vnet.ibm.com> References: <20170928135843.GA3741@vimes> <59CD33EF.6070002@linux.vnet.ibm.com> <59CE64FB.7090103@linux.vnet.ibm.com> Message-ID: On 29 September 2017 at 16:21, Gustavo Romero wrote: > Hi, > > On 28-09-2017 14:39, Gustavo Romero wrote: >> Hello Rob, >> >> On 28-09-2017 10:58, Rob McKenna wrote: >>> Approved. Please add an appropriate noreg label to the bug. >> >> I've add the 'noreg' label accordingly. >> >> Could somebody sponsor that change and push it please? > > I forgot to mention that it's a PPC64 only change. Let me know > if anything else is missing. > > Thank you and best regards, > Gustavo > Built on ppc64 and ppc64le with RHEL 7 and pushed on Gustavo's behalf: http://hg.openjdk.java.net/jdk8u/jdk8u-dev/hotspot/rev/584eac5794ff -- Andrew :) Senior Free Java Software Engineer Red Hat, Inc. (http://www.redhat.com) Web Site: http://fuseyism.com Twitter: https://twitter.com/gnu_andrew_java PGP Key: ed25519/0xCFDA0F9B35964222 (hkp://keys.gnupg.net) Fingerprint = 5132 579D D154 0ED2 3E04 C5A0 CFDA 0F9B 3596 4222 From kevin.walls at oracle.com Fri Oct 13 22:09:16 2017 From: kevin.walls at oracle.com (Kevin Walls) Date: Fri, 13 Oct 2017 23:09:16 +0100 Subject: [8u-dev] Request for approval: 8164954: split_if creates empty phi and region nodes In-Reply-To: References: Message-ID: Hi, I'd like to get backport approval for 8u-dev for: bug: https://bugs.openjdk.java.net/browse/JDK-8164954 9 changeset: http://hg.openjdk.java.net/jdk9/jdk9/hotspot/rev/38f38c10a11d 9 Review thread: http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/2017-March/025773.html Review of this 8u backport: http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/2017-October/027253.html It doesn't quite hg import cleanly as the surrounding code is a little different, but the code change itself is unchanged. The 8u change has been through jprt testing and also tested with the testsuite of a Java-based product which was seen hitting the same assert as in this bug.? hg diff of the proposed 8u change is below. Thanks! Kevin -------- Forwarded Message -------- Subject: Re: [8u] RFF(S): 8164954: split_if creates empty phi and region nodes Date: Fri, 13 Oct 2017 10:57:46 -0700 From: Vladimir Kozlov To: Kevin Walls , hotspot-compiler-dev at openjdk.java.net 8u change looks good. Thanks, Vladimir On 10/13/17 2:25 AM, Kevin Walls wrote: > Hi, > > I'd like to get a review of a backport to 8u. > > bug: https://bugs.openjdk.java.net/browse/JDK-8164954 > > 9 changeset: http://hg.openjdk.java.net/jdk9/jdk9/hotspot/rev/38f38c10a11d > > Review thread: http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/2017-March/025773.html > > > It doesn't hg import cleanly as the surrounding code is a little different: this change adds a condition in split_if() > which may make that method return earlier, but 8u does not have the block after the change, beginning "if > (nb_predicate_proj > 1) {", that comes in with 8078426. > > The 8u change has been through jprt testing and also tested with the testsuite of a Java-based product which was seen > hitting the same assert as in this bug.? hg diff of the proposed 8u change is below, I think that's enough but can offer > a webrev if anybody needs one. > > Thanks! > Kevin > > > bash-4.2$ hg diff src/share/vm/opto/ifnode.cpp > diff -r c89173159237 src/share/vm/opto/ifnode.cpp > --- a/src/share/vm/opto/ifnode.cpp????? Thu Sep 07 10:15:21 2017 -0400 > +++ b/src/share/vm/opto/ifnode.cpp????? Fri Oct 13 02:03:00 2017 -0700 > @@ -234,6 +234,13 @@ > ?????? predicate_proj = proj; > ???? } > ?? } > + > +? // If all the defs of the phi are the same constant, we already have the desired end state. > +? // Skip the split that would create empty phi and region nodes. > +? if((r->req() - req_c) == 1) { > +??? return NULL; > +? } > + > ?? Node* predicate_c = NULL; > ?? Node* predicate_x = NULL; > ?? bool counted_loop = r->is_CountedLoop(); > > From rob.mckenna at oracle.com Fri Oct 13 22:38:18 2017 From: rob.mckenna at oracle.com (Rob McKenna) Date: Fri, 13 Oct 2017 23:38:18 +0100 Subject: [8u-dev] Request for approval: 8164954: split_if creates empty phi and region nodes In-Reply-To: References: Message-ID: <20171013223818.GJ9161@vimes> Approved. Please add an appropriate noreg label. -Rob On 13/10/17 23:09, Kevin Walls wrote: > Hi, > I'd like to get backport approval for 8u-dev for: > bug: https://bugs.openjdk.java.net/browse/JDK-8164954 > > 9 changeset: http://hg.openjdk.java.net/jdk9/jdk9/hotspot/rev/38f38c10a11d > > 9 Review thread: http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/2017-March/025773.html > > Review of this 8u backport: http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/2017-October/027253.html > > > It doesn't quite hg import cleanly as the surrounding code is a little different, but the code change itself is unchanged. > > The 8u change has been through jprt testing and also tested with the testsuite of a Java-based product which was seen > hitting the same assert as in this bug.? hg diff of the proposed 8u change is below. > > Thanks! > Kevin > > > > -------- Forwarded Message -------- > Subject: Re: [8u] RFF(S): 8164954: split_if creates empty phi and region > nodes > Date: Fri, 13 Oct 2017 10:57:46 -0700 > From: Vladimir Kozlov > To: Kevin Walls , > hotspot-compiler-dev at openjdk.java.net > > > > 8u change looks good. > > Thanks, > Vladimir > > On 10/13/17 2:25 AM, Kevin Walls wrote: > >Hi, > > > >I'd like to get a review of a backport to 8u. > > > >bug: https://bugs.openjdk.java.net/browse/JDK-8164954 > > > >9 changeset: http://hg.openjdk.java.net/jdk9/jdk9/hotspot/rev/38f38c10a11d > > > >Review thread: http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/2017-March/025773.html > > > > > >It doesn't hg import cleanly as the surrounding code is a little > >different: this change adds a condition in split_if() which may make that > >method return earlier, but 8u does not have the block after the change, > >beginning "if (nb_predicate_proj > 1) {", that comes in with 8078426. > > > >The 8u change has been through jprt testing and also tested with the > >testsuite of a Java-based product which was seen hitting the same assert > >as in this bug.? hg diff of the proposed 8u change is below, I think > >that's enough but can offer a webrev if anybody needs one. > > > >Thanks! > >Kevin > > > > > >bash-4.2$ hg diff src/share/vm/opto/ifnode.cpp > >diff -r c89173159237 src/share/vm/opto/ifnode.cpp > >--- a/src/share/vm/opto/ifnode.cpp????? Thu Sep 07 10:15:21 2017 -0400 > >+++ b/src/share/vm/opto/ifnode.cpp????? Fri Oct 13 02:03:00 2017 -0700 > >@@ -234,6 +234,13 @@ > > ?????? predicate_proj = proj; > > ???? } > > ?? } > >+ > >+? // If all the defs of the phi are the same constant, we already have the desired end state. > >+? // Skip the split that would create empty phi and region nodes. > >+? if((r->req() - req_c) == 1) { > >+??? return NULL; > >+? } > >+ > > ?? Node* predicate_c = NULL; > > ?? Node* predicate_x = NULL; > > ?? bool counted_loop = r->is_CountedLoop(); > > > > > From kevin.walls at oracle.com Sat Oct 14 07:40:18 2017 From: kevin.walls at oracle.com (Kevin Walls) Date: Sat, 14 Oct 2017 08:40:18 +0100 Subject: [8u-dev] Request for approval: 8164954: split_if creates empty phi and region nodes In-Reply-To: <20171013223818.GJ9161@vimes> References: <20171013223818.GJ9161@vimes> Message-ID: Thanks Rob, done! On 13/10/2017 23:38, Rob McKenna wrote: > Approved. Please add an appropriate noreg label. > > -Rob > > On 13/10/17 23:09, Kevin Walls wrote: >> Hi, >> I'd like to get backport approval for 8u-dev for: >> bug: https://bugs.openjdk.java.net/browse/JDK-8164954 >> >> 9 changeset: http://hg.openjdk.java.net/jdk9/jdk9/hotspot/rev/38f38c10a11d >> >> 9 Review thread: http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/2017-March/025773.html >> >> Review of this 8u backport: http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/2017-October/027253.html >> >> >> It doesn't quite hg import cleanly as the surrounding code is a little different, but the code change itself is unchanged. >> >> The 8u change has been through jprt testing and also tested with the testsuite of a Java-based product which was seen >> hitting the same assert as in this bug.? hg diff of the proposed 8u change is below. >> >> Thanks! >> Kevin >> >> >> >> -------- Forwarded Message -------- >> Subject: Re: [8u] RFF(S): 8164954: split_if creates empty phi and region >> nodes >> Date: Fri, 13 Oct 2017 10:57:46 -0700 >> From: Vladimir Kozlov >> To: Kevin Walls , >> hotspot-compiler-dev at openjdk.java.net >> >> >> >> 8u change looks good. >> >> Thanks, >> Vladimir >> >> On 10/13/17 2:25 AM, Kevin Walls wrote: >>> Hi, >>> >>> I'd like to get a review of a backport to 8u. >>> >>> bug: https://bugs.openjdk.java.net/browse/JDK-8164954 >>> >>> 9 changeset: http://hg.openjdk.java.net/jdk9/jdk9/hotspot/rev/38f38c10a11d >>> >>> Review thread: http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/2017-March/025773.html >>> >>> >>> It doesn't hg import cleanly as the surrounding code is a little >>> different: this change adds a condition in split_if() which may make that >>> method return earlier, but 8u does not have the block after the change, >>> beginning "if (nb_predicate_proj > 1) {", that comes in with 8078426. >>> >>> The 8u change has been through jprt testing and also tested with the >>> testsuite of a Java-based product which was seen hitting the same assert >>> as in this bug.? hg diff of the proposed 8u change is below, I think >>> that's enough but can offer a webrev if anybody needs one. >>> >>> Thanks! >>> Kevin >>> >>> >>> bash-4.2$ hg diff src/share/vm/opto/ifnode.cpp >>> diff -r c89173159237 src/share/vm/opto/ifnode.cpp >>> --- a/src/share/vm/opto/ifnode.cpp????? Thu Sep 07 10:15:21 2017 -0400 >>> +++ b/src/share/vm/opto/ifnode.cpp????? Fri Oct 13 02:03:00 2017 -0700 >>> @@ -234,6 +234,13 @@ >>> ?????? predicate_proj = proj; >>> ???? } >>> ?? } >>> + >>> +? // If all the defs of the phi are the same constant, we already have the desired end state. >>> +? // Skip the split that would create empty phi and region nodes. >>> +? if((r->req() - req_c) == 1) { >>> +??? return NULL; >>> +? } >>> + >>> ?? Node* predicate_c = NULL; >>> ?? Node* predicate_x = NULL; >>> ?? bool counted_loop = r->is_CountedLoop(); >>> >>> From hariprasad705 at gmail.com Tue Oct 17 07:31:03 2017 From: hariprasad705 at gmail.com (Hari Prasad) Date: Tue, 17 Oct 2017 13:01:03 +0530 Subject: 8u152 Release Message-ID: Hi As per https://bugs.openjdk.java.net/projects/JDK/versions/18805 "8u152 " should be released on 2017-10-16, but it's not. I am looking for native memory fix that is " https://bugs.openjdk.java.net/browse/JDK-8164293" can you please help me when "8u152" will be available to the public? Thanks -- Hariprasad.B From dalibor.topic at oracle.com Tue Oct 17 08:35:41 2017 From: dalibor.topic at oracle.com (dalibor topic) Date: Tue, 17 Oct 2017 10:35:41 +0200 Subject: 8u152 Release In-Reply-To: References: Message-ID: <42880347-6e6d-372a-6dc7-f848df5ac544@oracle.com> On 17.10.2017 09:31, Hari Prasad wrote: > Hi > > As per https://bugs.openjdk.java.net/projects/JDK/versions/18805 > "8u152 " should be released on 2017-10-16, but it's not. > The schedule for 8u152 can be found here: http://openjdk.java.net/projects/jdk8u/releases/8u152.html While you may certainly find other things on the Internet, the only relevant schedule for this project is the link above. cheers, dalibor topic -- Dalibor Topic | Principal Product Manager Phone: +494089091214 | Mobile: +491737185961 ORACLE Deutschland B.V. & Co. KG | K?hneh?fe 5 | 22761 Hamburg ORACLE Deutschland B.V. & Co. KG Hauptverwaltung: Riesstr. 25, D-80992 M?nchen Registergericht: Amtsgericht M?nchen, HRA 95603 Komplement?rin: ORACLE Deutschland Verwaltung B.V. Hertogswetering 163/167, 3543 AS Utrecht, Niederlande Handelsregister der Handelskammer Midden-Niederlande, Nr. 30143697 Gesch?ftsf?hrer: Alexander van der Ven, Jan Schultheiss, Val Maher Oracle is committed to developing practices and products that help protect the environment From dalibor.topic at oracle.com Tue Oct 17 13:58:32 2017 From: dalibor.topic at oracle.com (dalibor topic) Date: Tue, 17 Oct 2017 15:58:32 +0200 Subject: 8u152 Release In-Reply-To: References: <42880347-6e6d-372a-6dc7-f848df5ac544@oracle.com> Message-ID: On 17.10.2017 12:01, Hari Prasad wrote: > This means will be released in Oct-2017,?but no exact date decided yet. ? Generally speaking, we don't specify exact GA dates for OpenJDK JDK 8 Update releases. Since 8u152 contains the changes from 8u151, it can only be released once that CPU has been released. The Oracle CPU schedule is public. cheers, dalibor topic -- Dalibor Topic | Principal Product Manager Phone: +494089091214 | Mobile: +491737185961 ORACLE Deutschland B.V. & Co. KG | K?hneh?fe 5 | 22761 Hamburg ORACLE Deutschland B.V. & Co. KG Hauptverwaltung: Riesstr. 25, D-80992 M?nchen Registergericht: Amtsgericht M?nchen, HRA 95603 Komplement?rin: ORACLE Deutschland Verwaltung B.V. Hertogswetering 163/167, 3543 AS Utrecht, Niederlande Handelsregister der Handelskammer Midden-Niederlande, Nr. 30143697 Gesch?ftsf?hrer: Alexander van der Ven, Jan Schultheiss, Val Maher Oracle is committed to developing practices and products that help protect the environment From hariprasad705 at gmail.com Tue Oct 17 14:06:30 2017 From: hariprasad705 at gmail.com (Hari Prasad) Date: Tue, 17 Oct 2017 19:36:30 +0530 Subject: 8u152 Release In-Reply-To: References: <42880347-6e6d-372a-6dc7-f848df5ac544@oracle.com> Message-ID: Ok, thanks for the clarification. On 17-Oct-2017 7:28 PM, "dalibor topic" wrote: > On 17.10.2017 12:01, Hari Prasad wrote: > >> This means will be released in Oct-2017, but no exact date decided yet. ? >> > > Generally speaking, we don't specify exact GA dates for OpenJDK JDK 8 > Update releases. > > Since 8u152 contains the changes from 8u151, it can only be released once > that CPU has been released. The Oracle CPU schedule is public. > > cheers, > dalibor topic > > -- > Dalibor Topic | Principal Product Manager > Phone: +494089091214 | Mobile: +491737185961 > > > ORACLE Deutschland B.V. & Co. KG | K?hneh?fe 5 | 22761 Hambu > > rg > > ORACLE Deutschland B.V. & Co. KG > Hauptverwaltung: Riesstr. > 25, D-80992 > M?nchen > Registergericht: Amtsgericht M?nchen, HRA 95603 > > Komplement?rin: ORACLE Deutschland Verwaltung B.V. > Hertogswetering 163/167, 3543 AS Utrecht, Niederlande > Handelsregister der Handelskammer Midden-Niederlande, Nr. 30143697 > Gesch?ftsf?hrer: Alexander van der Ven, Jan Schultheiss, Val Maher > > Oracle is committed to developing > practices and products that help protect the environment > From abhi.saha at oracle.com Tue Oct 17 17:22:12 2017 From: abhi.saha at oracle.com (Abhijit Saha) Date: Tue, 17 Oct 2017 10:22:12 -0700 Subject: [8u] Request for approval for bulk changes from 8u152 fixes into 8u Message-ID: <500b0592-52d5-0176-a98d-b842b20b321a@oracle.com> 8u152 has been released earlier today [1]. Requesting approval to sync up the 8u152 changes into the jdk8u forest. Webrev: http://cr.openjdk.java.net/~asaha/openJDK.8u152-8u162.sync/webrev Thanks Abhijit [1] http://www.oracle.com/technetwork/java/javase/downloads/index.html -- Java Platform Group Oracle Corporation. (408)276-7564 From rob.mckenna at oracle.com Tue Oct 17 17:32:31 2017 From: rob.mckenna at oracle.com (Rob McKenna) Date: Tue, 17 Oct 2017 18:32:31 +0100 Subject: [8u] Request for approval for bulk changes from 8u152 fixes into 8u In-Reply-To: <500b0592-52d5-0176-a98d-b842b20b321a@oracle.com> References: <500b0592-52d5-0176-a98d-b842b20b321a@oracle.com> Message-ID: <20171017173231.GA10374@vimes> Approved -Rob On 17/10/17 10:22, Abhijit Saha wrote: > 8u152 has been released earlier today [1]. Requesting approval to sync up > the 8u152 changes into the jdk8u forest. > > Webrev: http://cr.openjdk.java.net/~asaha/openJDK.8u152-8u162.sync/webrev > > > Thanks > Abhijit > > > [1] http://www.oracle.com/technetwork/java/javase/downloads/index.html > > > -- > Java Platform Group > Oracle Corporation. > (408)276-7564 > From david.holmes at oracle.com Tue Oct 17 21:17:49 2017 From: david.holmes at oracle.com (David Holmes) Date: Wed, 18 Oct 2017 07:17:49 +1000 Subject: 8u152 Release In-Reply-To: References: <42880347-6e6d-372a-6dc7-f848df5ac544@oracle.com> Message-ID: <4e046985-b5a0-69a7-4fbe-2e67ec18fcb0@oracle.com> And as chance would have it both 8u151 and 8u152 are now available. :) David On 18/10/2017 12:06 AM, Hari Prasad wrote: > Ok, thanks for the clarification. > > On 17-Oct-2017 7:28 PM, "dalibor topic" wrote: > >> On 17.10.2017 12:01, Hari Prasad wrote: >> >>> This means will be released in Oct-2017, but no exact date decided yet. ? >>> >> >> Generally speaking, we don't specify exact GA dates for OpenJDK JDK 8 >> Update releases. >> >> Since 8u152 contains the changes from 8u151, it can only be released once >> that CPU has been released. The Oracle CPU schedule is public. >> >> cheers, >> dalibor topic >> >> -- >> Dalibor Topic | Principal Product Manager >> Phone: +494089091214 | Mobile: +491737185961 >> >> >> ORACLE Deutschland B.V. & Co. KG | K?hneh?fe 5 | 22761 Hambu >> >> rg >> >> ORACLE Deutschland B.V. & Co. KG >> Hauptverwaltung: Riesstr. >> 25, D-80992 >> M?nchen >> Registergericht: Amtsgericht M?nchen, HRA 95603 >> >> Komplement?rin: ORACLE Deutschland Verwaltung B.V. >> Hertogswetering 163/167, 3543 AS Utrecht, Niederlande >> Handelsregister der Handelskammer Midden-Niederlande, Nr. 30143697 >> Gesch?ftsf?hrer: Alexander van der Ven, Jan Schultheiss, Val Maher >> >> Oracle is committed to developing >> practices and products that help protect the environment >> From hariprasad705 at gmail.com Wed Oct 18 05:27:55 2017 From: hariprasad705 at gmail.com (Hari Prasad) Date: Wed, 18 Oct 2017 10:57:55 +0530 Subject: 8u152 Release In-Reply-To: <4e046985-b5a0-69a7-4fbe-2e67ec18fcb0@oracle.com> References: <42880347-6e6d-372a-6dc7-f848df5ac544@oracle.com> <4e046985-b5a0-69a7-4fbe-2e67ec18fcb0@oracle.com> Message-ID: its great to hear such a good news. Thanks David On Wed, Oct 18, 2017 at 2:47 AM, David Holmes wrote: > And as chance would have it both 8u151 and 8u152 are now available. :) > > David > > On 18/10/2017 12:06 AM, Hari Prasad wrote: > >> Ok, thanks for the clarification. >> >> On 17-Oct-2017 7:28 PM, "dalibor topic" wrote: >> >> On 17.10.2017 12:01, Hari Prasad wrote: >>> >>> This means will be released in Oct-2017, but no exact date decided yet. ? >>>> >>>> >>> Generally speaking, we don't specify exact GA dates for OpenJDK JDK 8 >>> Update releases. >>> >>> Since 8u152 contains the changes from 8u151, it can only be released once >>> that CPU has been released. The Oracle CPU schedule is public. >>> >>> cheers, >>> dalibor topic >>> >>> -- >>> Dalibor Topic | Principal Product Manager >>> Phone: +494089091214 | Mobile: +491737185961 >>> >>> >>> ORACLE Deutschland B.V. & Co. KG | K?hneh?fe 5 | 22761 Hambu >>> >> 761+Hambu&entry=gmail&source=g> >>> rg >>> >>> ORACLE Deutschland B.V. & Co. KG >>> Hauptverwaltung: Riesstr. >>> 25, D-80992 >>> M?nchen >>> Registergericht: Amtsgericht M?nchen, HRA 95603 >>> >>> Komplement?rin: ORACLE Deutschland Verwaltung B.V. >>> Hertogswetering 163/167, 3543 AS Utrecht, Niederlande >>> Handelsregister der Handelskammer Midden-Niederlande, Nr. 30143697 >>> Gesch?ftsf?hrer: Alexander van der Ven, Jan Schultheiss, Val Maher >>> >>> Oracle is committed to developing >>> practices and products that help protect the environment >>> >>> -- Hariprasad.B From raduandritoiu at gmail.com Wed Oct 18 07:18:31 2017 From: raduandritoiu at gmail.com (Radu Andritoiu) Date: Wed, 18 Oct 2017 10:18:31 +0300 Subject: Cross compile jdk8u on PPC32 Message-ID: Hello, I am new to the community and want to thank you in advance. I am trying to cross compile from the latest revision on branch jdk8u162 a 'client' jvm-variant for PPC32 bit. Here are some of the errors I am getting: /home/build/build_java/jdk8u_5/hotspot/src/share/vm/runtime/globals.hpp:114:31: error: c1_globals_ppc.hpp: No such file or directory ... /home/build/build_java/jdk8u_5/hotspot/src/os_cpu/linux_ppc/vm/atomic_linux_ppc.inline.hpp:34:2: error: #error "Atomic currently only implemented for PPC64" ... /home/build/build_java/jdk8u_5/hotspot/src/os_cpu/linux_ppc/vm/orderAccess_linux_ppc.inline.hpp:33:2: error: #error "OrderAccess currently only implemented for PPC64" ... /home/build/build_java/jdk8u_5/hotspot/src/cpu/ppc/vm/copy_ppc.hpp:30:2: error: #error "copy currently only implemented for PPC64" ... /home/build/build_java/jdk8u_5/hotspot/src/cpu/ppc/vm/jniTypes_ppc.hpp:49:2: error: #error "ppc32 support currently not implemented!!!" ... My conclusion was that 'client' jvm-variant is not ported for PPC32. But the 'servert' jvm-variant did not work either. My questions are: 1. Is there a way to compile a 'client' jvm-variant for PPC32? am I doing something wrong or is this not supported at this time? Should I use a different revision that jdk8u162? 2. Is there a way to compile a 'server' jvm-variant for PPC32? Is this also not supported at this time? 3. Is there another project/branch besides 'jdk8u' that should work in cross compiling a jvm for PPC32 bit? Thank you, Radu Andritoiu From fridrich.strba at suse.com Wed Oct 18 07:24:29 2017 From: fridrich.strba at suse.com (Fridrich Strba) Date: Wed, 18 Oct 2017 09:24:29 +0200 Subject: Cross compile jdk8u on PPC32 In-Reply-To: References: Message-ID: For PPC32, use jvm-variant "zero" Cheers Fridrich On 18/10/17 09:18, Radu Andritoiu wrote: > Hello, > > I am new to the community and want to thank you in advance. > > I am trying to cross compile from the latest revision on branch jdk8u162 a > 'client' jvm-variant for PPC32 bit. > Here are some of the errors I am getting: > > /home/build/build_java/jdk8u_5/hotspot/src/share/vm/runtime/globals.hpp:114:31: > error: c1_globals_ppc.hpp: No such file or directory > ... > /home/build/build_java/jdk8u_5/hotspot/src/os_cpu/linux_ppc/vm/atomic_linux_ppc.inline.hpp:34:2: > error: #error "Atomic currently only implemented for PPC64" > ... > /home/build/build_java/jdk8u_5/hotspot/src/os_cpu/linux_ppc/vm/orderAccess_linux_ppc.inline.hpp:33:2: > error: #error "OrderAccess currently only implemented for PPC64" > ... > /home/build/build_java/jdk8u_5/hotspot/src/cpu/ppc/vm/copy_ppc.hpp:30:2: > error: #error "copy currently only implemented for PPC64" > ... > /home/build/build_java/jdk8u_5/hotspot/src/cpu/ppc/vm/jniTypes_ppc.hpp:49:2: > error: #error "ppc32 support currently not implemented!!!" > ... > > My conclusion was that 'client' jvm-variant is not ported for PPC32. But > the 'servert' jvm-variant did not work either. > My questions are: > 1. Is there a way to compile a 'client' jvm-variant for PPC32? am I doing > something wrong or is this not supported at this time? Should I use a > different revision that jdk8u162? > 2. Is there a way to compile a 'server' jvm-variant for PPC32? Is this > also not supported at this time? > 3. Is there another project/branch besides 'jdk8u' that should work in > cross compiling a jvm for PPC32 bit? > > Thank you, > Radu Andritoiu > From raduandritoiu at gmail.com Wed Oct 18 07:46:44 2017 From: raduandritoiu at gmail.com (Radu Andritoiu) Date: Wed, 18 Oct 2017 10:46:44 +0300 Subject: Cross compile jdk8u on PPC32 In-Reply-To: References: Message-ID: Thank you for the quick reply. I will try it now and post the results. Thank you, Radu On Wed, Oct 18, 2017 at 10:24 AM, Fridrich Strba wrote: > For PPC32, use jvm-variant "zero" > > Cheers > > Fridrich > > > On 18/10/17 09:18, Radu Andritoiu wrote: > > Hello, > > > > I am new to the community and want to thank you in advance. > > > > I am trying to cross compile from the latest revision on branch jdk8u162 > a > > 'client' jvm-variant for PPC32 bit. > > Here are some of the errors I am getting: > > > > /home/build/build_java/jdk8u_5/hotspot/src/share/vm/ > runtime/globals.hpp:114:31: > > error: c1_globals_ppc.hpp: No such file or directory > > ... > > /home/build/build_java/jdk8u_5/hotspot/src/os_cpu/linux_ > ppc/vm/atomic_linux_ppc.inline.hpp:34:2: > > error: #error "Atomic currently only implemented for PPC64" > > ... > > /home/build/build_java/jdk8u_5/hotspot/src/os_cpu/linux_ > ppc/vm/orderAccess_linux_ppc.inline.hpp:33:2: > > error: #error "OrderAccess currently only implemented for PPC64" > > ... > > /home/build/build_java/jdk8u_5/hotspot/src/cpu/ppc/vm/copy_ppc.hpp:30:2: > > error: #error "copy currently only implemented for PPC64" > > ... > > /home/build/build_java/jdk8u_5/hotspot/src/cpu/ppc/vm/ > jniTypes_ppc.hpp:49:2: > > error: #error "ppc32 support currently not implemented!!!" > > ... > > > > My conclusion was that 'client' jvm-variant is not ported for PPC32. But > > the 'servert' jvm-variant did not work either. > > My questions are: > > 1. Is there a way to compile a 'client' jvm-variant for PPC32? am I > doing > > something wrong or is this not supported at this time? Should I use a > > different revision that jdk8u162? > > 2. Is there a way to compile a 'server' jvm-variant for PPC32? Is this > > also not supported at this time? > > 3. Is there another project/branch besides 'jdk8u' that should work in > > cross compiling a jvm for PPC32 bit? > > > > Thank you, > > Radu Andritoiu > > > > > From volker.simonis at gmail.com Wed Oct 18 11:40:17 2017 From: volker.simonis at gmail.com (Volker Simonis) Date: Wed, 18 Oct 2017 11:40:17 +0000 Subject: Cross compile jdk8u on PPC32 In-Reply-To: References: Message-ID: Hi Radu, ppc32 is currently not supported in the main OpenJDK repositories. You can of course use the zero variant, but that's interpreter only, so the performance won't be great. As far as I know, Azul supports ppc32 but I'm not sure where they are hosting their sources. CC'ed Ivan from Azul who may know. Regards, Volker Radu Andritoiu schrieb am Mi. 18. Okt. 2017 um 09:18: > Hello, > > I am new to the community and want to thank you in advance. > > I am trying to cross compile from the latest revision on branch jdk8u162 a > 'client' jvm-variant for PPC32 bit. > Here are some of the errors I am getting: > > > /home/build/build_java/jdk8u_5/hotspot/src/share/vm/runtime/globals.hpp:114:31: > error: c1_globals_ppc.hpp: No such file or directory > ... > > /home/build/build_java/jdk8u_5/hotspot/src/os_cpu/linux_ppc/vm/atomic_linux_ppc.inline.hpp:34:2: > error: #error "Atomic currently only implemented for PPC64" > ... > > /home/build/build_java/jdk8u_5/hotspot/src/os_cpu/linux_ppc/vm/orderAccess_linux_ppc.inline.hpp:33:2: > error: #error "OrderAccess currently only implemented for PPC64" > ... > /home/build/build_java/jdk8u_5/hotspot/src/cpu/ppc/vm/copy_ppc.hpp:30:2: > error: #error "copy currently only implemented for PPC64" > ... > > /home/build/build_java/jdk8u_5/hotspot/src/cpu/ppc/vm/jniTypes_ppc.hpp:49:2: > error: #error "ppc32 support currently not implemented!!!" > ... > > My conclusion was that 'client' jvm-variant is not ported for PPC32. But > the 'servert' jvm-variant did not work either. > My questions are: > 1. Is there a way to compile a 'client' jvm-variant for PPC32? am I doing > something wrong or is this not supported at this time? Should I use a > different revision that jdk8u162? > 2. Is there a way to compile a 'server' jvm-variant for PPC32? Is this > also not supported at this time? > 3. Is there another project/branch besides 'jdk8u' that should work in > cross compiling a jvm for PPC32 bit? > > Thank you, > Radu Andritoiu > From raduandritoiu at gmail.com Wed Oct 18 12:20:54 2017 From: raduandritoiu at gmail.com (Radu Andritoiu) Date: Wed, 18 Oct 2017 15:20:54 +0300 Subject: Cross compile jdk8u on PPC32 In-Reply-To: References: Message-ID: I am trying to compile jdk8u using jvm-variant "zero" and it asks for 'libffi'. I read that it uses 'libffi' for JNA to bridge between the interpreter and native code. It does not use JNI. My question is if the final VM will have support for JNI, because I will need to run a java project that is already coded and uses JNI generated by SWIG. Also thank you for the help and info about Azul. Thank you, Radu On Wed, Oct 18, 2017 at 2:40 PM, Volker Simonis wrote: > Hi Radu, > > ppc32 is currently not supported in the main OpenJDK repositories. You can > of course use the zero variant, but that's interpreter only, so the > performance won't be great. > > As far as I know, Azul supports ppc32 but I'm not sure where they are > hosting their sources. CC'ed Ivan from Azul who may know. > > Regards, > Volker > > Radu Andritoiu schrieb am Mi. 18. Okt. 2017 um > 09:18: > >> Hello, >> >> I am new to the community and want to thank you in advance. >> >> I am trying to cross compile from the latest revision on branch jdk8u162 a >> 'client' jvm-variant for PPC32 bit. >> Here are some of the errors I am getting: >> >> /home/build/build_java/jdk8u_5/hotspot/src/share/vm/ >> runtime/globals.hpp:114:31: >> error: c1_globals_ppc.hpp: No such file or directory >> ... >> /home/build/build_java/jdk8u_5/hotspot/src/os_cpu/linux_ >> ppc/vm/atomic_linux_ppc.inline.hpp:34:2: >> error: #error "Atomic currently only implemented for PPC64" >> ... >> /home/build/build_java/jdk8u_5/hotspot/src/os_cpu/linux_ >> ppc/vm/orderAccess_linux_ppc.inline.hpp:33:2: >> error: #error "OrderAccess currently only implemented for PPC64" >> ... >> /home/build/build_java/jdk8u_5/hotspot/src/cpu/ppc/vm/copy_ppc.hpp:30:2: >> error: #error "copy currently only implemented for PPC64" >> ... >> /home/build/build_java/jdk8u_5/hotspot/src/cpu/ppc/vm/ >> jniTypes_ppc.hpp:49:2: >> error: #error "ppc32 support currently not implemented!!!" >> ... >> >> My conclusion was that 'client' jvm-variant is not ported for PPC32. But >> the 'servert' jvm-variant did not work either. >> My questions are: >> 1. Is there a way to compile a 'client' jvm-variant for PPC32? am I doing >> something wrong or is this not supported at this time? Should I use a >> different revision that jdk8u162? >> 2. Is there a way to compile a 'server' jvm-variant for PPC32? Is this >> also not supported at this time? >> 3. Is there another project/branch besides 'jdk8u' that should work in >> cross compiling a jvm for PPC32 bit? >> >> Thank you, >> Radu Andritoiu >> > From gnu.andrew at redhat.com Wed Oct 18 15:26:41 2017 From: gnu.andrew at redhat.com (Andrew Hughes) Date: Wed, 18 Oct 2017 16:26:41 +0100 Subject: Cross compile jdk8u on PPC32 In-Reply-To: References: Message-ID: On 18 October 2017 at 08:24, Fridrich Strba wrote: > For PPC32, use jvm-variant "zero" > > Cheers > > Fridrich > > That should be the default for PPC32. If not, I'd say that's a bug. -- Andrew :) Senior Free Java Software Engineer Red Hat, Inc. (http://www.redhat.com) Web Site: http://fuseyism.com Twitter: https://twitter.com/gnu_andrew_java PGP Key: ed25519/0xCFDA0F9B35964222 (hkp://keys.gnupg.net) Fingerprint = 5132 579D D154 0ED2 3E04 C5A0 CFDA 0F9B 3596 4222 From yasuenag at gmail.com Mon Oct 23 22:43:13 2017 From: yasuenag at gmail.com (Yasumasa Suenaga) Date: Tue, 24 Oct 2017 07:43:13 +0900 Subject: [8u] RFA for 8189599: InitialBootClassLoaderMetaspaceSize and CompressedClassSpaceSize should be checked consistent from MaxMetaspaceSize Message-ID: <97a3b207-b469-42ad-1db0-13811d51e77f@gmail.com> Hi all, I want to backport JDK-8087291 change to 8u because this issue will affect all Java 8 or later users. Could you approve it? JBS: https://bugs.openjdk.java.net/browse/JDK-8189599 Review thread: http://mail.openjdk.java.net/pipermail/hotspot-runtime-dev/2017-October/024770.html http://mail.openjdk.java.net/pipermail/hotspot-runtime-dev/2017-October/024927.html jdk10 changeset: http://hg.openjdk.java.net/jdk10/hs/rev/dbd1f4f276ba Thanks, Yasumasa From ningsheng.jian at linaro.org Tue Oct 24 02:03:12 2017 From: ningsheng.jian at linaro.org (Ningsheng Jian) Date: Tue, 24 Oct 2017 10:03:12 +0800 Subject: RFA for backport of 8035496: G1 ARM: missing remset entry noticed by VerifyAfterGC for vm/gc/concurrent/lp50yp10rp70mr30st0 Message-ID: OK. Thank you Andrew. Forwarding to jdk8u-dev. ---- Hi, I found the following patch is missing in JDK8u: http://hg.openjdk.java.net/jdk9/dev/hotspot/rev/86b7a6d44894 I cannot view JDK-8035496 bug details, but I think this fix is correct for AArch64 (and probably other weak ordering architectures). It should be backported to JDK8u. I found this fix when we were trying to figure out the root cause of a random G1GC crash in one AArch64 platform. The backport webrev is: http://cr.openjdk.java.net/~njian/8035496-backport/webrev/ Is it OK for jdk8u? If yes, could someone please help to push the changeset? Thanks, Ningsheng On 23 October 2017 at 23:30, Andrew Hughes wrote: > On 23 October 2017 at 07:04, Ningsheng Jian wrote: >> Hi, >> >> I found the following patch is missing in AArch64 JDK8u: >> >> http://hg.openjdk.java.net/jdk9/dev/hotspot/rev/86b7a6d44894 >> >> I cannot view the bug details, but I think this fix is correct for >> AArch64 and should be backported to JDK8u. I found this fix when we >> were trying to figure out the root cause of a random G1GC crash in one >> AArch64 platform. >> >> The backport webrev is: >> >> http://cr.openjdk.java.net/~njian/8035496-backport/webrev/ >> >> Is it OK for aarch64-port/jdk8u? >> >> Thanks, >> Ningsheng > > > As this is a backport from 9, it should really be requested for the > main 8u tree: > > http://openjdk.java.net/projects/jdk8u/codereview.html > http://mail.openjdk.java.net/mailman/listinfo/jdk8u-dev > > aarch64/jdk8u would then pick it up on merge. > -- > Andrew :) > > Senior Free Java Software Engineer > Red Hat, Inc. (http://www.redhat.com) > > Web Site: http://fuseyism.com > Twitter: https://twitter.com/gnu_andrew_java > PGP Key: ed25519/0xCFDA0F9B35964222 (hkp://keys.gnupg.net) > Fingerprint = 5132 579D D154 0ED2 3E04 C5A0 CFDA 0F9B 3596 4222 From aph at redhat.com Tue Oct 24 08:46:48 2017 From: aph at redhat.com (Andrew Haley) Date: Tue, 24 Oct 2017 09:46:48 +0100 Subject: Cross compile jdk8u on PPC32 In-Reply-To: References: Message-ID: <9b00b55d-7683-ce7f-dff0-f162347e94f4@redhat.com> On 18/10/17 13:20, Radu Andritoiu wrote: > I am trying to compile jdk8u using jvm-variant "zero" and it asks for > 'libffi'. > I read that it uses 'libffi' for JNA to bridge between the interpreter and > native code. It does not use JNI. That's wrong. Zero supports JNI just fine. -- Andrew Haley Java Platform Lead Engineer Red Hat UK Ltd. EAC8 43EB D3EF DB98 CC77 2FAD A5CD 6035 332F A671 From raduandritoiu at gmail.com Tue Oct 24 09:00:47 2017 From: raduandritoiu at gmail.com (Radu Andritoiu) Date: Tue, 24 Oct 2017 12:00:47 +0300 Subject: Cross compile jdk8u on PPC32 In-Reply-To: <9b00b55d-7683-ce7f-dff0-f162347e94f4@redhat.com> References: <9b00b55d-7683-ce7f-dff0-f162347e94f4@redhat.com> Message-ID: Hello, I am still having trouble compiling the 'zero' jvm-variant. I get C/C++ compiling errors I am trying to fix. I looked in 'hotspot/src/cpu' and saw there are source-code folders for different cpu architectures: 'ppc', 'sparc', 'x86', 'zero'. If I use jvm-variant 'zero' when compiling it will try to use the code from 'hotspot/src/cpu/zero'. I looked in the mercurial commit logs and saw (I guess) commits for PPC being done in the folder 'hotspot/src/cpu/ppc'. My question are to build for PPC 32 what source-code folder should I use 'hotspot/src/cpu/zero' or 'hotspot/src/cpu/ppc'? Do I try to build/compile using jvm-variant 'zero' and force it to use the code from 'hotspot/src/cpu/ppc'? Is the source-code from 'hotspot/src/cpu/ppc' onlu for PPC 64? Thank you very much for the help, Radu On Tue, Oct 24, 2017 at 11:46 AM, Andrew Haley wrote: > On 18/10/17 13:20, Radu Andritoiu wrote: > > I am trying to compile jdk8u using jvm-variant "zero" and it asks for > > 'libffi'. > > I read that it uses 'libffi' for JNA to bridge between the interpreter > and > > native code. It does not use JNI. > > That's wrong. Zero supports JNI just fine. > > -- > Andrew Haley > Java Platform Lead Engineer > Red Hat UK Ltd. > EAC8 43EB D3EF DB98 CC77 2FAD A5CD 6035 332F A671 > From martin.doerr at sap.com Tue Oct 24 09:08:42 2017 From: martin.doerr at sap.com (Doerr, Martin) Date: Tue, 24 Oct 2017 09:08:42 +0000 Subject: Cross compile jdk8u on PPC32 In-Reply-To: References: <9b00b55d-7683-ce7f-dff0-f162347e94f4@redhat.com> Message-ID: <5f61da5183944c5eb8c3e3d870ffea64@sap.com> Hi Radu, 'hotspot/src/cpu/ppc' only supports PPC64. I?m not so familiar with the zero build, but I?ve heard about people using zero on PPC32 (e.g. for Debian). Maybe one of them can provide hints. Best regards, Martin From: ppc-aix-port-dev [mailto:ppc-aix-port-dev-bounces at openjdk.java.net] On Behalf Of Radu Andritoiu Sent: Dienstag, 24. Oktober 2017 11:01 To: Andrew Haley Cc: jdk8u-dev at openjdk.java.net; ppc-aix-port-dev at openjdk.java.net; Ivan Krylov Subject: Re: Cross compile jdk8u on PPC32 Hello, I am still having trouble compiling the 'zero' jvm-variant. I get C/C++ compiling errors I am trying to fix. I looked in 'hotspot/src/cpu' and saw there are source-code folders for different cpu architectures: 'ppc', 'sparc', 'x86', 'zero'. If I use jvm-variant 'zero' when compiling it will try to use the code from 'hotspot/src/cpu/zero'. I looked in the mercurial commit logs and saw (I guess) commits for PPC being done in the folder 'hotspot/src/cpu/ppc'. My question are to build for PPC 32 what source-code folder should I use 'hotspot/src/cpu/zero' or 'hotspot/src/cpu/ppc'? Do I try to build/compile using jvm-variant 'zero' and force it to use the code from 'hotspot/src/cpu/ppc'? Is the source-code from 'hotspot/src/cpu/ppc' onlu for PPC 64? Thank you very much for the help, Radu On Tue, Oct 24, 2017 at 11:46 AM, Andrew Haley > wrote: On 18/10/17 13:20, Radu Andritoiu wrote: > I am trying to compile jdk8u using jvm-variant "zero" and it asks for > 'libffi'. > I read that it uses 'libffi' for JNA to bridge between the interpreter and > native code. It does not use JNI. That's wrong. Zero supports JNI just fine. -- Andrew Haley Java Platform Lead Engineer Red Hat UK Ltd. EAC8 43EB D3EF DB98 CC77 2FAD A5CD 6035 332F A671 From david.holmes at oracle.com Tue Oct 24 09:11:35 2017 From: david.holmes at oracle.com (David Holmes) Date: Tue, 24 Oct 2017 19:11:35 +1000 Subject: Cross compile jdk8u on PPC32 In-Reply-To: References: <9b00b55d-7683-ce7f-dff0-f162347e94f4@redhat.com> Message-ID: On 24/10/2017 7:00 PM, Radu Andritoiu wrote: > Hello, > > I am still having trouble compiling the 'zero' jvm-variant. I get C/C++ > compiling errors I am trying to fix. > > I looked in 'hotspot/src/cpu' and saw there are source-code folders for > different cpu architectures: 'ppc', 'sparc', 'x86', 'zero'. > If I use jvm-variant 'zero' when compiling it will try to use the code from > 'hotspot/src/cpu/zero'. > I looked in the mercurial commit logs and saw (I guess) commits for PPC > being done in the folder 'hotspot/src/cpu/ppc'. > > My question are to build for PPC 32 what source-code folder should I use > 'hotspot/src/cpu/zero' or 'hotspot/src/cpu/ppc'? You don't "use" a specific folder the build takes care of that for you. When you build zero it uses src/cpu/zero. > Do I try to build/compile using jvm-variant 'zero' and force it to use the > code from 'hotspot/src/cpu/ppc'? No. > Is the source-code from 'hotspot/src/cpu/ppc' onlu for PPC 64? Yes. David > > Thank you very much for the help, > Radu > > > > On Tue, Oct 24, 2017 at 11:46 AM, Andrew Haley wrote: > >> On 18/10/17 13:20, Radu Andritoiu wrote: >>> I am trying to compile jdk8u using jvm-variant "zero" and it asks for >>> 'libffi'. >>> I read that it uses 'libffi' for JNA to bridge between the interpreter >> and >>> native code. It does not use JNI. >> >> That's wrong. Zero supports JNI just fine. >> >> -- >> Andrew Haley >> Java Platform Lead Engineer >> Red Hat UK Ltd. >> EAC8 43EB D3EF DB98 CC77 2FAD A5CD 6035 332F A671 >> From rob.mckenna at oracle.com Tue Oct 24 09:49:55 2017 From: rob.mckenna at oracle.com (Rob McKenna) Date: Tue, 24 Oct 2017 10:49:55 +0100 Subject: RFA for backport of 8035496: G1 ARM: missing remset entry noticed by VerifyAfterGC for vm/gc/concurrent/lp50yp10rp70mr30st0 In-Reply-To: References: Message-ID: <20171024094955.GB3531@tecra> Hi Ninsheng, Unless the patch applies cleanly this will need a codereview from somoone on hotspot-dev. -Rob On 24/10/17 10:03, Ningsheng Jian wrote: > OK. Thank you Andrew. Forwarding to jdk8u-dev. > ---- > Hi, > > I found the following patch is missing in JDK8u: > > http://hg.openjdk.java.net/jdk9/dev/hotspot/rev/86b7a6d44894 > > I cannot view JDK-8035496 bug details, but I think this fix is correct > for AArch64 (and probably other weak ordering architectures). It > should be backported to JDK8u. > > I found this fix when we were trying to figure out the root cause of a > random G1GC crash in one AArch64 platform. > > The backport webrev is: > > http://cr.openjdk.java.net/~njian/8035496-backport/webrev/ > > Is it OK for jdk8u? If yes, could someone please help to push the changeset? > > Thanks, > Ningsheng > > > > On 23 October 2017 at 23:30, Andrew Hughes wrote: > > On 23 October 2017 at 07:04, Ningsheng Jian wrote: > >> Hi, > >> > >> I found the following patch is missing in AArch64 JDK8u: > >> > >> http://hg.openjdk.java.net/jdk9/dev/hotspot/rev/86b7a6d44894 > >> > >> I cannot view the bug details, but I think this fix is correct for > >> AArch64 and should be backported to JDK8u. I found this fix when we > >> were trying to figure out the root cause of a random G1GC crash in one > >> AArch64 platform. > >> > >> The backport webrev is: > >> > >> http://cr.openjdk.java.net/~njian/8035496-backport/webrev/ > >> > >> Is it OK for aarch64-port/jdk8u? > >> > >> Thanks, > >> Ningsheng > > > > > > As this is a backport from 9, it should really be requested for the > > main 8u tree: > > > > http://openjdk.java.net/projects/jdk8u/codereview.html > > http://mail.openjdk.java.net/mailman/listinfo/jdk8u-dev > > > > aarch64/jdk8u would then pick it up on merge. > > -- > > Andrew :) > > > > Senior Free Java Software Engineer > > Red Hat, Inc. (http://www.redhat.com) > > > > Web Site: http://fuseyism.com > > Twitter: https://twitter.com/gnu_andrew_java > > PGP Key: ed25519/0xCFDA0F9B35964222 (hkp://keys.gnupg.net) > > Fingerprint = 5132 579D D154 0ED2 3E04 C5A0 CFDA 0F9B 3596 4222 From rob.mckenna at oracle.com Tue Oct 24 09:55:25 2017 From: rob.mckenna at oracle.com (Rob McKenna) Date: Tue, 24 Oct 2017 10:55:25 +0100 Subject: [8u] RFA for 8087291: InitialBootClassLoaderMetaspaceSize and CompressedClassSpaceSize should be checked consistent from MaxMetaspaceSize In-Reply-To: <97a3b207-b469-42ad-1db0-13811d51e77f@gmail.com> References: <97a3b207-b469-42ad-1db0-13811d51e77f@gmail.com> Message-ID: <20171024095525.GC3531@tecra> Hi Yasumasa, Please use the main bug id in future requests. (and the main bug link) Approved, -Rob On 24/10/17 07:43, Yasumasa Suenaga wrote: > Hi all, > > I want to backport JDK-8087291 change to 8u because this issue will affect all Java 8 or later users. > Could you approve it? > > > JBS: > https://bugs.openjdk.java.net/browse/JDK-8189599 > > Review thread: > http://mail.openjdk.java.net/pipermail/hotspot-runtime-dev/2017-October/024770.html > http://mail.openjdk.java.net/pipermail/hotspot-runtime-dev/2017-October/024927.html > > jdk10 changeset: > http://hg.openjdk.java.net/jdk10/hs/rev/dbd1f4f276ba > > > Thanks, > > Yasumasa From david.holmes at oracle.com Tue Oct 24 10:08:46 2017 From: david.holmes at oracle.com (David Holmes) Date: Tue, 24 Oct 2017 20:08:46 +1000 Subject: RFA for backport of 8035496: G1 ARM: missing remset entry noticed by VerifyAfterGC for vm/gc/concurrent/lp50yp10rp70mr30st0 In-Reply-To: <20171024094955.GB3531@tecra> References: <20171024094955.GB3531@tecra> Message-ID: Hi Rob, I had a look at this earlier and the patch does apply cleanly. Thanks, David On 24/10/2017 7:49 PM, Rob McKenna wrote: > Hi Ninsheng, > > Unless the patch applies cleanly this will need a codereview from > somoone on hotspot-dev. > > -Rob > > On 24/10/17 10:03, Ningsheng Jian wrote: >> OK. Thank you Andrew. Forwarding to jdk8u-dev. >> ---- >> Hi, >> >> I found the following patch is missing in JDK8u: >> >> http://hg.openjdk.java.net/jdk9/dev/hotspot/rev/86b7a6d44894 >> >> I cannot view JDK-8035496 bug details, but I think this fix is correct >> for AArch64 (and probably other weak ordering architectures). It >> should be backported to JDK8u. >> >> I found this fix when we were trying to figure out the root cause of a >> random G1GC crash in one AArch64 platform. >> >> The backport webrev is: >> >> http://cr.openjdk.java.net/~njian/8035496-backport/webrev/ >> >> Is it OK for jdk8u? If yes, could someone please help to push the changeset? >> >> Thanks, >> Ningsheng >> >> >> >> On 23 October 2017 at 23:30, Andrew Hughes wrote: >>> On 23 October 2017 at 07:04, Ningsheng Jian wrote: >>>> Hi, >>>> >>>> I found the following patch is missing in AArch64 JDK8u: >>>> >>>> http://hg.openjdk.java.net/jdk9/dev/hotspot/rev/86b7a6d44894 >>>> >>>> I cannot view the bug details, but I think this fix is correct for >>>> AArch64 and should be backported to JDK8u. I found this fix when we >>>> were trying to figure out the root cause of a random G1GC crash in one >>>> AArch64 platform. >>>> >>>> The backport webrev is: >>>> >>>> http://cr.openjdk.java.net/~njian/8035496-backport/webrev/ >>>> >>>> Is it OK for aarch64-port/jdk8u? >>>> >>>> Thanks, >>>> Ningsheng >>> >>> >>> As this is a backport from 9, it should really be requested for the >>> main 8u tree: >>> >>> http://openjdk.java.net/projects/jdk8u/codereview.html >>> http://mail.openjdk.java.net/mailman/listinfo/jdk8u-dev >>> >>> aarch64/jdk8u would then pick it up on merge. >>> -- >>> Andrew :) >>> >>> Senior Free Java Software Engineer >>> Red Hat, Inc. (http://www.redhat.com) >>> >>> Web Site: http://fuseyism.com >>> Twitter: https://twitter.com/gnu_andrew_java >>> PGP Key: ed25519/0xCFDA0F9B35964222 (hkp://keys.gnupg.net) >>> Fingerprint = 5132 579D D154 0ED2 3E04 C5A0 CFDA 0F9B 3596 4222 From rob.mckenna at oracle.com Tue Oct 24 10:18:22 2017 From: rob.mckenna at oracle.com (Rob McKenna) Date: Tue, 24 Oct 2017 11:18:22 +0100 Subject: RFA for backport of 8035496: G1 ARM: missing remset entry noticed by VerifyAfterGC for vm/gc/concurrent/lp50yp10rp70mr30st0 In-Reply-To: References: <20171024094955.GB3531@tecra> Message-ID: <20171024101822.GD3531@tecra> Thanks David, Approved. -Rob On 24/10/17 20:08, David Holmes wrote: > Hi Rob, > > I had a look at this earlier and the patch does apply cleanly. > > Thanks, > David > > On 24/10/2017 7:49 PM, Rob McKenna wrote: > >Hi Ninsheng, > > > >Unless the patch applies cleanly this will need a codereview from > >somoone on hotspot-dev. > > > > -Rob > > > >On 24/10/17 10:03, Ningsheng Jian wrote: > >>OK. Thank you Andrew. Forwarding to jdk8u-dev. > >>---- > >>Hi, > >> > >>I found the following patch is missing in JDK8u: > >> > >>http://hg.openjdk.java.net/jdk9/dev/hotspot/rev/86b7a6d44894 > >> > >>I cannot view JDK-8035496 bug details, but I think this fix is correct > >>for AArch64 (and probably other weak ordering architectures). It > >>should be backported to JDK8u. > >> > >>I found this fix when we were trying to figure out the root cause of a > >>random G1GC crash in one AArch64 platform. > >> > >>The backport webrev is: > >> > >>http://cr.openjdk.java.net/~njian/8035496-backport/webrev/ > >> > >>Is it OK for jdk8u? If yes, could someone please help to push the changeset? > >> > >>Thanks, > >>Ningsheng > >> > >> > >> > >>On 23 October 2017 at 23:30, Andrew Hughes wrote: > >>>On 23 October 2017 at 07:04, Ningsheng Jian wrote: > >>>>Hi, > >>>> > >>>>I found the following patch is missing in AArch64 JDK8u: > >>>> > >>>>http://hg.openjdk.java.net/jdk9/dev/hotspot/rev/86b7a6d44894 > >>>> > >>>>I cannot view the bug details, but I think this fix is correct for > >>>>AArch64 and should be backported to JDK8u. I found this fix when we > >>>>were trying to figure out the root cause of a random G1GC crash in one > >>>>AArch64 platform. > >>>> > >>>>The backport webrev is: > >>>> > >>>>http://cr.openjdk.java.net/~njian/8035496-backport/webrev/ > >>>> > >>>>Is it OK for aarch64-port/jdk8u? > >>>> > >>>>Thanks, > >>>>Ningsheng > >>> > >>> > >>>As this is a backport from 9, it should really be requested for the > >>>main 8u tree: > >>> > >>>http://openjdk.java.net/projects/jdk8u/codereview.html > >>>http://mail.openjdk.java.net/mailman/listinfo/jdk8u-dev > >>> > >>>aarch64/jdk8u would then pick it up on merge. > >>>-- > >>>Andrew :) > >>> > >>>Senior Free Java Software Engineer > >>>Red Hat, Inc. (http://www.redhat.com) > >>> > >>>Web Site: http://fuseyism.com > >>>Twitter: https://twitter.com/gnu_andrew_java > >>>PGP Key: ed25519/0xCFDA0F9B35964222 (hkp://keys.gnupg.net) > >>>Fingerprint = 5132 579D D154 0ED2 3E04 C5A0 CFDA 0F9B 3596 4222 From yasuenag at gmail.com Tue Oct 24 11:30:08 2017 From: yasuenag at gmail.com (Yasumasa Suenaga) Date: Tue, 24 Oct 2017 20:30:08 +0900 Subject: [8u] RFA for 8087291: InitialBootClassLoaderMetaspaceSize and CompressedClassSpaceSize should be checked consistent from MaxMetaspaceSize In-Reply-To: <20171024095525.GC3531@tecra> References: <97a3b207-b469-42ad-1db0-13811d51e77f@gmail.com> <20171024095525.GC3531@tecra> Message-ID: Hi Rob, Thank you for your approval, and sorry fo my incorrect bug id. Yasumasa 2017/10/24 ??6:55 "Rob McKenna" : > Hi Yasumasa, > > Please use the main bug id in future requests. (and the main bug link) > > Approved, > > -Rob > > On 24/10/17 07:43, Yasumasa Suenaga wrote: > > Hi all, > > > > I want to backport JDK-8087291 change to 8u because this issue will > affect all Java 8 or later users. > > Could you approve it? > > > > > > JBS: > > https://bugs.openjdk.java.net/browse/JDK-8189599 > > > > Review thread: > > http://mail.openjdk.java.net/pipermail/hotspot-runtime-dev/ > 2017-October/024770.html > > http://mail.openjdk.java.net/pipermail/hotspot-runtime-dev/ > 2017-October/024927.html > > > > jdk10 changeset: > > http://hg.openjdk.java.net/jdk10/hs/rev/dbd1f4f276ba > > > > > > Thanks, > > > > Yasumasa > From bob.vandette at oracle.com Tue Oct 24 12:34:14 2017 From: bob.vandette at oracle.com (Bob Vandette) Date: Tue, 24 Oct 2017 08:34:14 -0400 Subject: Cross compile jdk8u on PPC32 In-Reply-To: <9b00b55d-7683-ce7f-dff0-f162347e94f4@redhat.com> References: <9b00b55d-7683-ce7f-dff0-f162347e94f4@redhat.com> Message-ID: As others have mentioned, you need to target ZERO and not PPC32 when trying to build a non supported cpu architecture with the Zero interpreter. You do need to have a libffi.so binary available for the build. Zero uses this to manage the native stack frames. Bob. > On Oct 24, 2017, at 4:46 AM, Andrew Haley wrote: > > On 18/10/17 13:20, Radu Andritoiu wrote: >> I am trying to compile jdk8u using jvm-variant "zero" and it asks for >> 'libffi'. >> I read that it uses 'libffi' for JNA to bridge between the interpreter and >> native code. It does not use JNI. > > That's wrong. Zero supports JNI just fine. > > -- > Andrew Haley > Java Platform Lead Engineer > Red Hat UK Ltd. > EAC8 43EB D3EF DB98 CC77 2FAD A5CD 6035 332F A671 From ningsheng.jian at linaro.org Wed Oct 25 05:45:08 2017 From: ningsheng.jian at linaro.org (Ningsheng Jian) Date: Wed, 25 Oct 2017 13:45:08 +0800 Subject: RFA for backport of 8035496: G1 ARM: missing remset entry noticed by VerifyAfterGC for vm/gc/concurrent/lp50yp10rp70mr30st0 In-Reply-To: <20171024101822.GD3531@tecra> References: <20171024094955.GB3531@tecra> <20171024101822.GD3531@tecra> Message-ID: Thank you Rob and David! Can someone please help to push the patch? Thanks, Ningsheng On 24 October 2017 at 18:18, Rob McKenna wrote: > Thanks David, > > Approved. > > -Rob > > On 24/10/17 20:08, David Holmes wrote: >> Hi Rob, >> >> I had a look at this earlier and the patch does apply cleanly. >> >> Thanks, >> David >> >> On 24/10/2017 7:49 PM, Rob McKenna wrote: >> >Hi Ninsheng, >> > >> >Unless the patch applies cleanly this will need a codereview from >> >somoone on hotspot-dev. >> > >> > -Rob >> > >> >On 24/10/17 10:03, Ningsheng Jian wrote: >> >>OK. Thank you Andrew. Forwarding to jdk8u-dev. >> >>---- >> >>Hi, >> >> >> >>I found the following patch is missing in JDK8u: >> >> >> >>http://hg.openjdk.java.net/jdk9/dev/hotspot/rev/86b7a6d44894 >> >> >> >>I cannot view JDK-8035496 bug details, but I think this fix is correct >> >>for AArch64 (and probably other weak ordering architectures). It >> >>should be backported to JDK8u. >> >> >> >>I found this fix when we were trying to figure out the root cause of a >> >>random G1GC crash in one AArch64 platform. >> >> >> >>The backport webrev is: >> >> >> >>http://cr.openjdk.java.net/~njian/8035496-backport/webrev/ >> >> >> >>Is it OK for jdk8u? If yes, could someone please help to push the changeset? >> >> >> >>Thanks, >> >>Ningsheng >> >> >> >> >> >> >> >>On 23 October 2017 at 23:30, Andrew Hughes wrote: >> >>>On 23 October 2017 at 07:04, Ningsheng Jian wrote: >> >>>>Hi, >> >>>> >> >>>>I found the following patch is missing in AArch64 JDK8u: >> >>>> >> >>>>http://hg.openjdk.java.net/jdk9/dev/hotspot/rev/86b7a6d44894 >> >>>> >> >>>>I cannot view the bug details, but I think this fix is correct for >> >>>>AArch64 and should be backported to JDK8u. I found this fix when we >> >>>>were trying to figure out the root cause of a random G1GC crash in one >> >>>>AArch64 platform. >> >>>> >> >>>>The backport webrev is: >> >>>> >> >>>>http://cr.openjdk.java.net/~njian/8035496-backport/webrev/ >> >>>> >> >>>>Is it OK for aarch64-port/jdk8u? >> >>>> >> >>>>Thanks, >> >>>>Ningsheng >> >>> >> >>> >> >>>As this is a backport from 9, it should really be requested for the >> >>>main 8u tree: >> >>> >> >>>http://openjdk.java.net/projects/jdk8u/codereview.html >> >>>http://mail.openjdk.java.net/mailman/listinfo/jdk8u-dev >> >>> >> >>>aarch64/jdk8u would then pick it up on merge. >> >>>-- >> >>>Andrew :) >> >>> >> >>>Senior Free Java Software Engineer >> >>>Red Hat, Inc. (http://www.redhat.com) >> >>> >> >>>Web Site: http://fuseyism.com >> >>>Twitter: https://twitter.com/gnu_andrew_java >> >>>PGP Key: ed25519/0xCFDA0F9B35964222 (hkp://keys.gnupg.net) >> >>>Fingerprint = 5132 579D D154 0ED2 3E04 C5A0 CFDA 0F9B 3596 4222 From gnu.andrew at redhat.com Thu Oct 26 18:23:01 2017 From: gnu.andrew at redhat.com (Andrew Hughes) Date: Thu, 26 Oct 2017 19:23:01 +0100 Subject: RFA for backport of 8035496: G1 ARM: missing remset entry noticed by VerifyAfterGC for vm/gc/concurrent/lp50yp10rp70mr30st0 In-Reply-To: References: <20171024094955.GB3531@tecra> <20171024101822.GD3531@tecra> Message-ID: On 25 October 2017 at 06:45, Ningsheng Jian wrote: > Thank you Rob and David! > > Can someone please help to push the patch? > > Thanks, > Ningsheng > I'm happy to push it on your behalf. -- Andrew :) Senior Free Java Software Engineer Red Hat, Inc. (http://www.redhat.com) Web Site: http://fuseyism.com Twitter: https://twitter.com/gnu_andrew_java PGP Key: ed25519/0xCFDA0F9B35964222 (hkp://keys.gnupg.net) Fingerprint = 5132 579D D154 0ED2 3E04 C5A0 CFDA 0F9B 3596 4222 From gnu.andrew at redhat.com Thu Oct 26 19:19:23 2017 From: gnu.andrew at redhat.com (Andrew Hughes) Date: Thu, 26 Oct 2017 20:19:23 +0100 Subject: RFA for backport of 8035496: G1 ARM: missing remset entry noticed by VerifyAfterGC for vm/gc/concurrent/lp50yp10rp70mr30st0 In-Reply-To: References: <20171024094955.GB3531@tecra> <20171024101822.GD3531@tecra> Message-ID: On 26 October 2017 at 19:23, Andrew Hughes wrote: > On 25 October 2017 at 06:45, Ningsheng Jian wrote: >> Thank you Rob and David! >> >> Can someone please help to push the patch? >> >> Thanks, >> Ningsheng >> > > I'm happy to push it on your behalf. > -- > Andrew :) > > Senior Free Java Software Engineer > Red Hat, Inc. (http://www.redhat.com) > > Web Site: http://fuseyism.com > Twitter: https://twitter.com/gnu_andrew_java > PGP Key: ed25519/0xCFDA0F9B35964222 (hkp://keys.gnupg.net) > Fingerprint = 5132 579D D154 0ED2 3E04 C5A0 CFDA 0F9B 3596 4222 Built and pushed: http://hg.openjdk.java.net/jdk8u/jdk8u-dev/hotspot/rev/4edb0f406a2c For the record, the 8 and 9 patches are identical, bar the change in path from the more succinct 'src/share/vm/gc/' in 9 to 'src/share/vm/gc_implementation/' in 8. -- Andrew :) Senior Free Java Software Engineer Red Hat, Inc. (http://www.redhat.com) Web Site: http://fuseyism.com Twitter: https://twitter.com/gnu_andrew_java PGP Key: ed25519/0xCFDA0F9B35964222 (hkp://keys.gnupg.net) Fingerprint = 5132 579D D154 0ED2 3E04 C5A0 CFDA 0F9B 3596 4222 From ningsheng.jian at linaro.org Fri Oct 27 01:43:21 2017 From: ningsheng.jian at linaro.org (Ningsheng Jian) Date: Fri, 27 Oct 2017 09:43:21 +0800 Subject: RFA for backport of 8035496: G1 ARM: missing remset entry noticed by VerifyAfterGC for vm/gc/concurrent/lp50yp10rp70mr30st0 In-Reply-To: References: <20171024094955.GB3531@tecra> <20171024101822.GD3531@tecra> Message-ID: Thank you Andrew! > > Built and pushed: > > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/hotspot/rev/4edb0f406a2c > > For the record, the 8 and 9 patches are identical, bar the change in > path from the more succinct 'src/share/vm/gc/' in 9 to > 'src/share/vm/gc_implementation/' in 8. > Yes, that's why I prepared the trivial webrev changeset. Ningsheng From rwestrel at redhat.com Fri Oct 27 13:51:34 2017 From: rwestrel at redhat.com (Roland Westrelin) Date: Fri, 27 Oct 2017 15:51:34 +0200 Subject: [8u] request for approval: 8187822: C2 conditonal move optimization might create broken graph Message-ID: Hi, I'd like to backport 8187822 to 8u as it's an issue reported by one of our (Red Hat) users. The patch applies cleanly. It was pushed to jdk10 3 weeks ago and there was no follow up bugs that I'm aware of. https://bugs.openjdk.java.net/browse/JDK-8187822 http://hg.openjdk.java.net/jdk10/hs/rev/b3fd664e5af2 Roland. From rob.mckenna at oracle.com Fri Oct 27 15:03:35 2017 From: rob.mckenna at oracle.com (Rob McKenna) Date: Fri, 27 Oct 2017 16:03:35 +0100 Subject: [8u] request for approval: 8187822: C2 conditonal move optimization might create broken graph In-Reply-To: References: Message-ID: <20171027150335.GD3576@vimes> Approved -Rob On 27/10/17 15:51, Roland Westrelin wrote: > > Hi, > > I'd like to backport 8187822 to 8u as it's an issue reported by one of > our (Red Hat) users. The patch applies cleanly. It was pushed to jdk10 3 > weeks ago and there was no follow up bugs that I'm aware of. > > https://bugs.openjdk.java.net/browse/JDK-8187822 > http://hg.openjdk.java.net/jdk10/hs/rev/b3fd664e5af2 > > Roland.