From erik.joelsson at oracle.com Thu Oct 1 13:16:08 2015 From: erik.joelsson at oracle.com (Erik Joelsson) Date: Thu, 1 Oct 2015 15:16:08 +0200 Subject: [8u] Request for Approval and Review: JDK-8136980: build for 8u65 and 8u66 for solaris platforms is failing In-Reply-To: <560BD900.9000308@oracle.com> References: <560BD900.9000308@oracle.com> Message-ID: <560D3218.5070205@oracle.com> Still looking for an official review from 8u reviewer for this. Ideally someone from Hotspot. /Erik On 2015-09-30 14:43, Erik Joelsson wrote: > Hello, > > Please approve and review this fix for 8u. > > My last fix for this issue, JDK-8136691, was not enough. I made a > mistake while verifying the fix and the problem is still there. > > There are more discrepancies between Solaris and the other platform > makefiles. The excludeSrc.gmk file is not included anywhere when > building on Solaris. The variable Src_Files_EXCLUDE is overwritten in > vm.make instead of appended to, so even when including excludeSrc.gmk, > it wasn't enough. The other platform specific buildtree.make files > also set INCLUDE_TRACE in the generated makefile. > > Bug: https://bugs.openjdk.java.net/browse/JDK-8136980 > Webrev: http://cr.openjdk.java.net/~erikj/8136980/webrev.hotspot.01/ > > /Erik From mikael.vidstedt at oracle.com Thu Oct 1 18:35:22 2015 From: mikael.vidstedt at oracle.com (Mikael Vidstedt) Date: Thu, 1 Oct 2015 11:35:22 -0700 Subject: [8u] Request for Approval and Review: JDK-8136980: build for 8u65 and 8u66 for solaris platforms is failing In-Reply-To: <560D3218.5070205@oracle.com> References: <560BD900.9000308@oracle.com> <560D3218.5070205@oracle.com> Message-ID: <560D7CEA.5050302@oracle.com> Looks ok. (This really needs to be unified, but we know that) Cheers, Mikael On 2015-10-01 06:16, Erik Joelsson wrote: > Still looking for an official review from 8u reviewer for this. > Ideally someone from Hotspot. > > /Erik > > On 2015-09-30 14:43, Erik Joelsson wrote: >> Hello, >> >> Please approve and review this fix for 8u. >> >> My last fix for this issue, JDK-8136691, was not enough. I made a >> mistake while verifying the fix and the problem is still there. >> >> There are more discrepancies between Solaris and the other platform >> makefiles. The excludeSrc.gmk file is not included anywhere when >> building on Solaris. The variable Src_Files_EXCLUDE is overwritten in >> vm.make instead of appended to, so even when including >> excludeSrc.gmk, it wasn't enough. The other platform specific >> buildtree.make files also set INCLUDE_TRACE in the generated makefile. >> >> Bug: https://bugs.openjdk.java.net/browse/JDK-8136980 >> Webrev: http://cr.openjdk.java.net/~erikj/8136980/webrev.hotspot.01/ >> >> /Erik > From sundararajan.athijegannathan at oracle.com Fri Oct 2 03:28:26 2015 From: sundararajan.athijegannathan at oracle.com (Sundararajan Athijegannathan) Date: Fri, 2 Oct 2015 08:58:26 +0530 Subject: [8u] approval request for 8138616: invokeFunction fails if function calls a function defined in GLOBAL_SCOPE Message-ID: <560DF9DA.3000307@oracle.com> Please approve. bug: https://bugs.openjdk.java.net/browse/JDK-8138616 jdk9 review thread: http://mail.openjdk.java.net/pipermail/nashorn-dev/2015-October/005356.html jdk8u-dev webrev: http://cr.openjdk.java.net/~sundar/8138616/8u/ Backported 'as is' except for the modular source layout difference. Thanks, -Sundar From david.buck at oracle.com Fri Oct 2 03:37:14 2015 From: david.buck at oracle.com (david buck) Date: Fri, 2 Oct 2015 12:37:14 +0900 Subject: [8u] approval request for 8138616: invokeFunction fails if function calls a function defined in GLOBAL_SCOPE In-Reply-To: <560DF9DA.3000307@oracle.com> References: <560DF9DA.3000307@oracle.com> Message-ID: <560DFBEA.3080504@oracle.com> approved for 8u-dev backport Cheers, -Buck On 2015/10/02 12:28, Sundararajan Athijegannathan wrote: > Please approve. > > bug: https://bugs.openjdk.java.net/browse/JDK-8138616 > jdk9 review thread: > http://mail.openjdk.java.net/pipermail/nashorn-dev/2015-October/005356.html > jdk8u-dev webrev: http://cr.openjdk.java.net/~sundar/8138616/8u/ > > Backported 'as is' except for the modular source layout difference. > > Thanks, > -Sundar From alexandr.scherbatiy at oracle.com Fri Oct 2 12:30:15 2015 From: alexandr.scherbatiy at oracle.com (Alexander Scherbatiy) Date: Fri, 02 Oct 2015 15:30:15 +0300 Subject: [8u-dev] Request for approval: 8134917: [macosx] JOptionPane doesn't receive mouse events when opened from a drop event Message-ID: <560E78D7.7000505@oracle.com> Hello, Could you approve the direct backport of the fix to JDK 8u-dev. The bug: https://bugs.openjdk.java.net/browse/JDK-8134917 The webrev: http://cr.openjdk.java.net/~alexsch/8134917/webrev.00 The review thread: http://mail.openjdk.java.net/pipermail/awt-dev/2015-September/010082.html The JDK 9 changeset: http://hg.openjdk.java.net/jdk9/client/jdk/rev/34c93fd32c20 Thanks, Alexandr. From rob.mckenna at oracle.com Fri Oct 2 12:53:42 2015 From: rob.mckenna at oracle.com (Rob McKenna) Date: Fri, 2 Oct 2015 13:53:42 +0100 Subject: [8u-dev] Request for approval: 8134917: [macosx] JOptionPane doesn't receive mouse events when opened from a drop event In-Reply-To: <560E78D7.7000505@oracle.com> References: <560E78D7.7000505@oracle.com> Message-ID: <560E7E56.6030608@oracle.com> Approved -Rob On 02/10/15 13:30, Alexander Scherbatiy wrote: > > Hello, > > Could you approve the direct backport of the fix to JDK 8u-dev. > > The bug: https://bugs.openjdk.java.net/browse/JDK-8134917 > The webrev: http://cr.openjdk.java.net/~alexsch/8134917/webrev.00 > The review thread: > http://mail.openjdk.java.net/pipermail/awt-dev/2015-September/010082.html > The JDK 9 changeset: > http://hg.openjdk.java.net/jdk9/client/jdk/rev/34c93fd32c20 > > Thanks, > Alexandr. > From hannes.wallnoefer at oracle.com Fri Oct 2 14:31:29 2015 From: hannes.wallnoefer at oracle.com (Hannes Wallnoefer) Date: Fri, 2 Oct 2015 16:31:29 +0200 Subject: [8u-dev] Request for Approval: 8138632: Sparse array does not handle growth of underlying dense array Message-ID: <560E9541.1070709@oracle.com> Please approve backport of 8138632 to 8u-dev. Bug: https://bugs.openjdk.java.net/browse/JDK-8138632 Webrev: http://cr.openjdk.java.net/~hannesw/8138632/webrev/ Review thread: http://mail.openjdk.java.net/pipermail/nashorn-dev/2015-September/005353.html Changes apply cleanly to jdk8u-dev after path reshuffling. Thanks, Hannes From hannes.wallnoefer at oracle.com Fri Oct 2 14:34:20 2015 From: hannes.wallnoefer at oracle.com (Hannes Wallnoefer) Date: Fri, 2 Oct 2015 16:34:20 +0200 Subject: [8u-dev] Request for Approval: 8137281: OutOfMemoryError with large numeric keys in JSON.parse Message-ID: <560E95EC.9070101@oracle.com> Please approve backport of 8137281 to 8u-dev. Bug: https://bugs.openjdk.java.net/browse/JDK-8137281 Webrev: http://cr.openjdk.java.net/~hannesw/8137281/webrev/ Review thread: http://mail.openjdk.java.net/pipermail/nashorn-dev/2015-October/005361.html Changes apply cleanly to jdk8u-dev after path reshuffling. Thanks, Hannes From rob.mckenna at oracle.com Fri Oct 2 14:58:05 2015 From: rob.mckenna at oracle.com (Rob McKenna) Date: Fri, 2 Oct 2015 15:58:05 +0100 Subject: [8u-dev] Request for Approval: 8137281: OutOfMemoryError with large numeric keys in JSON.parse In-Reply-To: <560E95EC.9070101@oracle.com> References: <560E95EC.9070101@oracle.com> Message-ID: <560E9B7D.7020909@oracle.com> Approved -Rob On 02/10/15 15:34, Hannes Wallnoefer wrote: > Please approve backport of 8137281 to 8u-dev. > > Bug: https://bugs.openjdk.java.net/browse/JDK-8137281 > Webrev: http://cr.openjdk.java.net/~hannesw/8137281/webrev/ > Review thread: > http://mail.openjdk.java.net/pipermail/nashorn-dev/2015-October/005361.html > > Changes apply cleanly to jdk8u-dev after path reshuffling. > > Thanks, > Hannes From rob.mckenna at oracle.com Fri Oct 2 14:57:58 2015 From: rob.mckenna at oracle.com (Rob McKenna) Date: Fri, 2 Oct 2015 07:57:58 -0700 (PDT) Subject: [8u-dev] Request for Approval: 8138632: Sparse array does not handle growth of underlying dense array In-Reply-To: <560E9541.1070709@oracle.com> References: <560E9541.1070709@oracle.com> Message-ID: <560E9B76.3090609@oracle.com> Approved -Rob On 02/10/15 15:31, Hannes Wallnoefer wrote: > Please approve backport of 8138632 to 8u-dev. > > Bug: https://bugs.openjdk.java.net/browse/JDK-8138632 > Webrev: http://cr.openjdk.java.net/~hannesw/8138632/webrev/ > Review thread: > http://mail.openjdk.java.net/pipermail/nashorn-dev/2015-September/005353.html > > > Changes apply cleanly to jdk8u-dev after path reshuffling. > > Thanks, > Hannes From hannes.wallnoefer at oracle.com Fri Oct 2 15:51:51 2015 From: hannes.wallnoefer at oracle.com (Hannes Wallnoefer) Date: Fri, 2 Oct 2015 17:51:51 +0200 Subject: [8u-dev] Request for Approval: 8137281: OutOfMemoryError with large numeric keys in JSON.parse In-Reply-To: <560E9B7D.7020909@oracle.com> References: <560E95EC.9070101@oracle.com> <560E9B7D.7020909@oracle.com> Message-ID: <560EA817.9030507@oracle.com> Thanks, Rob. Actually, I spoke too early. The patch applies to 8u-dev, but Java 7 does not support removal of type parameters in the TreeMap constructor. Since that was just cleanup I uploaded a new webrev for 8u-dev without it: http://cr.openjdk.java.net/~hannesw/8137281/ Please review/approve. Thanks, Hannes Am 2015-10-02 um 16:58 schrieb Rob McKenna: > Approved > > -Rob > > On 02/10/15 15:34, Hannes Wallnoefer wrote: >> Please approve backport of 8137281 to 8u-dev. >> >> Bug: https://bugs.openjdk.java.net/browse/JDK-8137281 >> Webrev: http://cr.openjdk.java.net/~hannesw/8137281/webrev/ >> Review thread: >> http://mail.openjdk.java.net/pipermail/nashorn-dev/2015-October/005361.html >> >> >> Changes apply cleanly to jdk8u-dev after path reshuffling. >> >> Thanks, >> Hannes From erik.joelsson at oracle.com Mon Oct 5 09:38:09 2015 From: erik.joelsson at oracle.com (Erik Joelsson) Date: Mon, 5 Oct 2015 11:38:09 +0200 Subject: [8u] Request for Approval and Review: JDK-8136980: build for 8u65 and 8u66 for solaris platforms is failing In-Reply-To: <560D3218.5070205@oracle.com> References: <560BD900.9000308@oracle.com> <560D3218.5070205@oracle.com> Message-ID: <56124501.5090702@oracle.com> Another friendly bump for an 8u (R)eviewer for this. /Erik On 2015-10-01 15:16, Erik Joelsson wrote: > Still looking for an official review from 8u reviewer for this. > Ideally someone from Hotspot. > > /Erik > > On 2015-09-30 14:43, Erik Joelsson wrote: >> Hello, >> >> Please approve and review this fix for 8u. >> >> My last fix for this issue, JDK-8136691, was not enough. I made a >> mistake while verifying the fix and the problem is still there. >> >> There are more discrepancies between Solaris and the other platform >> makefiles. The excludeSrc.gmk file is not included anywhere when >> building on Solaris. The variable Src_Files_EXCLUDE is overwritten in >> vm.make instead of appended to, so even when including >> excludeSrc.gmk, it wasn't enough. The other platform specific >> buildtree.make files also set INCLUDE_TRACE in the generated makefile. >> >> Bug: https://bugs.openjdk.java.net/browse/JDK-8136980 >> Webrev: http://cr.openjdk.java.net/~erikj/8136980/webrev.hotspot.01/ >> >> /Erik > From aleksej.efimov at oracle.com Mon Oct 5 15:34:06 2015 From: aleksej.efimov at oracle.com (Aleksej Efimov) Date: Mon, 5 Oct 2015 18:34:06 +0300 Subject: [8u-dev] Request for Approval: 8138716: (tz) Support tzdata2015g Message-ID: <5612986E.4070304@oracle.com> Hi, Please approve backport of tzdata2015g integration (8138716) to 8u-dev. Changes apply cleanly to jdk8u-dev after path reshuffling. Testing shows no tz related test failures. Bug: https://bugs.openjdk.java.net/browse/JDK-8138716 Review thread: http://mail.openjdk.java.net/pipermail/core-libs-dev/2015-October/035594.html JDK9 changeset: http://hg.openjdk.java.net/jdk9/dev/jdk/rev/3066806b394d Thank you, Aleksej From rob.mckenna at oracle.com Mon Oct 5 15:38:58 2015 From: rob.mckenna at oracle.com (Rob McKenna) Date: Mon, 5 Oct 2015 16:38:58 +0100 Subject: [8u-dev] Request for Approval: 8138716: (tz) Support tzdata2015g In-Reply-To: <5612986E.4070304@oracle.com> References: <5612986E.4070304@oracle.com> Message-ID: <56129992.8050608@oracle.com> Approved -Rob On 05/10/15 16:34, Aleksej Efimov wrote: > Hi, > > Please approve backport of tzdata2015g integration (8138716) to 8u-dev. > Changes apply cleanly to jdk8u-dev after path reshuffling. Testing shows > no tz related test failures. > > Bug: https://bugs.openjdk.java.net/browse/JDK-8138716 > Review thread: > http://mail.openjdk.java.net/pipermail/core-libs-dev/2015-October/035594.html > > JDK9 changeset: http://hg.openjdk.java.net/jdk9/dev/jdk/rev/3066806b394d > > Thank you, > Aleksej From aleksej.efimov at oracle.com Mon Oct 5 15:50:45 2015 From: aleksej.efimov at oracle.com (Aleksej Efimov) Date: Mon, 5 Oct 2015 18:50:45 +0300 Subject: [8u-dev] Request for Approval: 8081794: ParsePosition getErrorIndex returns 0 for TimeZone parsing problem Message-ID: <56129C55.8030807@oracle.com> Hi, Please approve a straight-forward backport of JDK-8081794 fix. Changes apply cleanly to jdk8u-dev after path reshuffling. Bug: https://bugs.openjdk.java.net/browse/JDK-8081794 Review thread: http://mail.openjdk.java.net/pipermail/i18n-dev/2015-September/001761.html JDK9 changeset: http://hg.openjdk.java.net/jdk9/dev/jdk/rev/2a3c2bd933a5 Thank you, Aleksej From rob.mckenna at oracle.com Mon Oct 5 16:14:19 2015 From: rob.mckenna at oracle.com (Rob McKenna) Date: Mon, 5 Oct 2015 17:14:19 +0100 Subject: [8u-dev] Request for Approval: 8081794: ParsePosition getErrorIndex returns 0 for TimeZone parsing problem In-Reply-To: <56129C55.8030807@oracle.com> References: <56129C55.8030807@oracle.com> Message-ID: <5612A1DB.3050201@oracle.com> Approved -Rob On 05/10/15 16:50, Aleksej Efimov wrote: > Hi, > > Please approve a straight-forward backport of JDK-8081794 fix. > Changes apply cleanly to jdk8u-dev after path reshuffling. > > Bug: https://bugs.openjdk.java.net/browse/JDK-8081794 > Review thread: > http://mail.openjdk.java.net/pipermail/i18n-dev/2015-September/001761.html > JDK9 changeset: http://hg.openjdk.java.net/jdk9/dev/jdk/rev/2a3c2bd933a5 > > Thank you, > Aleksej From mikhail.cherkasov at oracle.com Tue Oct 6 07:52:24 2015 From: mikhail.cherkasov at oracle.com (mikhail cherkasov) Date: Tue, 06 Oct 2015 10:52:24 +0300 Subject: [8u-dev] Request for Approval: 8086038 [macosx] No available data flavors when copying from Microsoft Word for Mac Message-ID: <56137DB8.8050702@oracle.com> Hi all, Could you please approve a direct backport of 8086038 to 8u-dev? Bug: https://bugs.openjdk.java.net/browse/JDK-8086038 Review threads: http://mail.openjdk.java.net/pipermail/awt-dev/2015-September/010091.html http://mail.openjdk.java.net/pipermail/awt-dev/2015-October/010097.html JDK9 changeset: http://hg.openjdk.java.net/jdk9/client/jdk/rev/eafaa1778c63 Thanks, Mikhail. From sean.coffey at oracle.com Tue Oct 6 08:10:29 2015 From: sean.coffey at oracle.com (=?UTF-8?Q?Se=c3=a1n_Coffey?=) Date: Tue, 6 Oct 2015 09:10:29 +0100 Subject: [8u-dev] Request for Approval: 8086038 [macosx] No available data flavors when copying from Microsoft Word for Mac In-Reply-To: <56137DB8.8050702@oracle.com> References: <56137DB8.8050702@oracle.com> Message-ID: <561381F5.4090401@oracle.com> Please add a suitable noreg- label to the bug report. Approved. Regards, Sean. On 06/10/2015 08:52, mikhail cherkasov wrote: > Hi all, > > Could you please approve a direct backport of 8086038 to 8u-dev? > > Bug: > https://bugs.openjdk.java.net/browse/JDK-8086038 > > Review threads: > http://mail.openjdk.java.net/pipermail/awt-dev/2015-September/010091.html > http://mail.openjdk.java.net/pipermail/awt-dev/2015-October/010097.html > > JDK9 changeset: > http://hg.openjdk.java.net/jdk9/client/jdk/rev/eafaa1778c63 > > Thanks, > Mikhail. From bengt.rutisson at oracle.com Tue Oct 6 10:01:29 2015 From: bengt.rutisson at oracle.com (Bengt Rutisson) Date: Tue, 06 Oct 2015 12:01:29 +0200 Subject: [8u] Request for Approval and Review: JDK-8136980: build for 8u65 and 8u66 for solaris platforms is failing In-Reply-To: <56124501.5090702@oracle.com> References: <560BD900.9000308@oracle.com> <560D3218.5070205@oracle.com> <56124501.5090702@oracle.com> Message-ID: <56139BF9.4030902@oracle.com> Hi Erik, On 2015-10-05 11:38, Erik Joelsson wrote: > Another friendly bump for an 8u (R)eviewer for this. Rubber stamped. (Thanks for walking me through the patch, Erik!) Bengt > > /Erik > > On 2015-10-01 15:16, Erik Joelsson wrote: >> Still looking for an official review from 8u reviewer for this. >> Ideally someone from Hotspot. >> >> /Erik >> >> On 2015-09-30 14:43, Erik Joelsson wrote: >>> Hello, >>> >>> Please approve and review this fix for 8u. >>> >>> My last fix for this issue, JDK-8136691, was not enough. I made a >>> mistake while verifying the fix and the problem is still there. >>> >>> There are more discrepancies between Solaris and the other platform >>> makefiles. The excludeSrc.gmk file is not included anywhere when >>> building on Solaris. The variable Src_Files_EXCLUDE is overwritten >>> in vm.make instead of appended to, so even when including >>> excludeSrc.gmk, it wasn't enough. The other platform specific >>> buildtree.make files also set INCLUDE_TRACE in the generated makefile. >>> >>> Bug: https://bugs.openjdk.java.net/browse/JDK-8136980 >>> Webrev: http://cr.openjdk.java.net/~erikj/8136980/webrev.hotspot.01/ >>> >>> /Erik >> > From daniel.daugherty at oracle.com Tue Oct 6 14:38:31 2015 From: daniel.daugherty at oracle.com (Daniel D. Daugherty) Date: Tue, 6 Oct 2015 08:38:31 -0600 Subject: [8u] Request for Approval and Review: JDK-8136980: build for 8u65 and 8u66 for solaris platforms is failing In-Reply-To: <560BD900.9000308@oracle.com> References: <560BD900.9000308@oracle.com> Message-ID: <5613DCE7.1080706@oracle.com> Sorry for missing this review request. I'm still learning to deal with my new e-mail filters (restored from memory after losing my MacBook Pro env). On 9/30/15 6:43 AM, Erik Joelsson wrote: > Hello, > > Please approve and review this fix for 8u. > > My last fix for this issue, JDK-8136691, was not enough. I made a > mistake while verifying the fix and the problem is still there. > > There are more discrepancies between Solaris and the other platform > makefiles. The excludeSrc.gmk file is not included anywhere when > building on Solaris. The variable Src_Files_EXCLUDE is overwritten in > vm.make instead of appended to, so even when including excludeSrc.gmk, > it wasn't enough. The other platform specific buildtree.make files > also set INCLUDE_TRACE in the generated makefile. > > Bug: https://bugs.openjdk.java.net/browse/JDK-8136980 > Webrev: http://cr.openjdk.java.net/~erikj/8136980/webrev.hotspot.01/ make/solaris/makefiles/buildtree.make No comments. make/solaris/makefiles/vm.make No comments. Thumbs up. Dan > > /Erik > From tobias.hartmann at oracle.com Wed Oct 7 10:27:10 2015 From: tobias.hartmann at oracle.com (Tobias Hartmann) Date: Wed, 07 Oct 2015 12:27:10 +0200 Subject: [8u] Request for approval and review: 8075805: Crash while trying to release CompiledICHolder Message-ID: <5614F37E.8090208@oracle.com> Hi, please approve and review the following backport to 8u. 8075805: Crash while trying to release CompiledICHolder https://bugs.openjdk.java.net/browse/JDK-8075805 http://hg.openjdk.java.net/jdk9/hs-comp/hotspot/rev/c0ea5537dc8b http://cr.openjdk.java.net/~thartmann/8075805/webrev.01/ I would also like to backport this follow up change that fixes an issue with the fix for 8075805: 8134493: Cleaning inline caches of unloaded nmethods should be done in sweeper https://bugs.openjdk.java.net/browse/JDK-8134493 http://hg.openjdk.java.net/jdk9/hs-comp/hotspot/rev/9df4555d2d7d http://cr.openjdk.java.net/~thartmann/8134493/webrev.01/ Unfortunately, both changesets do not apply cleanly to 8u. I merged the fixes, here are the new webrevs: JDK-8075805: http://cr.openjdk.java.net/~thartmann/8075805_8u/webrev.00/ JDK-8134493 (incremental): http://cr.openjdk.java.net/~thartmann/8134493_8u/webrev.00/ Both changes: http://cr.openjdk.java.net/~thartmann/8075805_8134493_8u/webrev.00/ Both fixes were pushed some months ago and testing showed no problems. Thanks, Tobias From david.buck at oracle.com Wed Oct 7 10:41:50 2015 From: david.buck at oracle.com (david buck) Date: Wed, 7 Oct 2015 19:41:50 +0900 Subject: [8u] Request for approval and review: 8075805: Crash while trying to release CompiledICHolder In-Reply-To: <5614F37E.8090208@oracle.com> References: <5614F37E.8090208@oracle.com> Message-ID: <5614F6EE.9000501@oracle.com> approved for 8u-dev backport pending successful completion of code review Please add the appropriate noreg label to JDK-8134493. Cheers, -Buck On 2015/10/07 19:27, Tobias Hartmann wrote: > Hi, > > please approve and review the following backport to 8u. > > 8075805: Crash while trying to release CompiledICHolder > https://bugs.openjdk.java.net/browse/JDK-8075805 > http://hg.openjdk.java.net/jdk9/hs-comp/hotspot/rev/c0ea5537dc8b > http://cr.openjdk.java.net/~thartmann/8075805/webrev.01/ > > I would also like to backport this follow up change that fixes an issue with the fix for 8075805: > > 8134493: Cleaning inline caches of unloaded nmethods should be done in sweeper > https://bugs.openjdk.java.net/browse/JDK-8134493 > http://hg.openjdk.java.net/jdk9/hs-comp/hotspot/rev/9df4555d2d7d > http://cr.openjdk.java.net/~thartmann/8134493/webrev.01/ > > Unfortunately, both changesets do not apply cleanly to 8u. I merged the fixes, here are the new webrevs: > > JDK-8075805: > http://cr.openjdk.java.net/~thartmann/8075805_8u/webrev.00/ > > JDK-8134493 (incremental): > http://cr.openjdk.java.net/~thartmann/8134493_8u/webrev.00/ > > Both changes: > http://cr.openjdk.java.net/~thartmann/8075805_8134493_8u/webrev.00/ > > Both fixes were pushed some months ago and testing showed no problems. > > Thanks, > Tobias > From tobias.hartmann at oracle.com Wed Oct 7 11:48:33 2015 From: tobias.hartmann at oracle.com (Tobias Hartmann) Date: Wed, 07 Oct 2015 13:48:33 +0200 Subject: [8u] Request for approval and review: 8075805: Crash while trying to release CompiledICHolder In-Reply-To: <5614F6EE.9000501@oracle.com> References: <5614F37E.8090208@oracle.com> <5614F6EE.9000501@oracle.com> Message-ID: <56150691.9030407@oracle.com> Thanks, David. On 07.10.2015 12:41, david buck wrote: > approved for 8u-dev backport pending successful completion of code review > > Please add the appropriate noreg label to JDK-8134493. Done (noreg-hard). Best, Tobias > > Cheers, > -Buck > > On 2015/10/07 19:27, Tobias Hartmann wrote: >> Hi, >> >> please approve and review the following backport to 8u. >> >> 8075805: Crash while trying to release CompiledICHolder >> https://bugs.openjdk.java.net/browse/JDK-8075805 >> http://hg.openjdk.java.net/jdk9/hs-comp/hotspot/rev/c0ea5537dc8b >> http://cr.openjdk.java.net/~thartmann/8075805/webrev.01/ >> >> I would also like to backport this follow up change that fixes an issue with the fix for 8075805: >> >> 8134493: Cleaning inline caches of unloaded nmethods should be done in sweeper >> https://bugs.openjdk.java.net/browse/JDK-8134493 >> http://hg.openjdk.java.net/jdk9/hs-comp/hotspot/rev/9df4555d2d7d >> http://cr.openjdk.java.net/~thartmann/8134493/webrev.01/ >> >> Unfortunately, both changesets do not apply cleanly to 8u. I merged the fixes, here are the new webrevs: >> >> JDK-8075805: >> http://cr.openjdk.java.net/~thartmann/8075805_8u/webrev.00/ >> >> JDK-8134493 (incremental): >> http://cr.openjdk.java.net/~thartmann/8134493_8u/webrev.00/ >> >> Both changes: >> http://cr.openjdk.java.net/~thartmann/8075805_8134493_8u/webrev.00/ >> >> Both fixes were pushed some months ago and testing showed no problems. >> >> Thanks, >> Tobias >> From vladimir.kozlov at oracle.com Wed Oct 7 12:43:20 2015 From: vladimir.kozlov at oracle.com (Vladimir Kozlov) Date: Wed, 7 Oct 2015 20:43:20 +0800 Subject: [8u] Request for approval and review: 8075805: Crash while trying to release CompiledICHolder In-Reply-To: <5614F37E.8090208@oracle.com> References: <5614F37E.8090208@oracle.com> Message-ID: <56151368.3030405@oracle.com> Changes are matching jdk9 changes. Tobias, should we backport 8058737 changes too? Next code is missing in 8u now: + MutexLocker cl(CompiledIC_lock); + nm->clear_ic_stubs(); It was P2. Why we did not backport it? Thanks, Vladimir On 10/7/15 6:27 PM, Tobias Hartmann wrote: > Hi, > > please approve and review the following backport to 8u. > > 8075805: Crash while trying to release CompiledICHolder > https://bugs.openjdk.java.net/browse/JDK-8075805 > http://hg.openjdk.java.net/jdk9/hs-comp/hotspot/rev/c0ea5537dc8b > http://cr.openjdk.java.net/~thartmann/8075805/webrev.01/ > > I would also like to backport this follow up change that fixes an issue with the fix for 8075805: > > 8134493: Cleaning inline caches of unloaded nmethods should be done in sweeper > https://bugs.openjdk.java.net/browse/JDK-8134493 > http://hg.openjdk.java.net/jdk9/hs-comp/hotspot/rev/9df4555d2d7d > http://cr.openjdk.java.net/~thartmann/8134493/webrev.01/ > > Unfortunately, both changesets do not apply cleanly to 8u. I merged the fixes, here are the new webrevs: > > JDK-8075805: > http://cr.openjdk.java.net/~thartmann/8075805_8u/webrev.00/ > > JDK-8134493 (incremental): > http://cr.openjdk.java.net/~thartmann/8134493_8u/webrev.00/ > > Both changes: > http://cr.openjdk.java.net/~thartmann/8075805_8134493_8u/webrev.00/ > > Both fixes were pushed some months ago and testing showed no problems. > > Thanks, > Tobias > From tobias.hartmann at oracle.com Wed Oct 7 14:19:48 2015 From: tobias.hartmann at oracle.com (Tobias Hartmann) Date: Wed, 07 Oct 2015 16:19:48 +0200 Subject: [8u] Request for approval and review: 8075805: Crash while trying to release CompiledICHolder In-Reply-To: <56151368.3030405@oracle.com> References: <5614F37E.8090208@oracle.com> <56151368.3030405@oracle.com> Message-ID: <56152A04.8040006@oracle.com> Hi Vladimir, On 07.10.2015 14:43, Vladimir Kozlov wrote: > Changes are matching jdk9 changes. > Tobias, should we backport 8058737 changes too? Next code is missing in 8u now: > > + MutexLocker cl(CompiledIC_lock); > + nm->clear_ic_stubs(); > > It was P2. Why we did not backport it? Yes, you are right. It seems like I missed to backport that one because it never showed up with 8. However, we should backport it. The patch for JDK-8058737 applies cleanly. I updated the other webrevs: JDK-8058737: CodeCache::find_blob fails with 'unsafe access to zombie method' https://bugs.openjdk.java.net/browse/JDK-8058737 http://cr.openjdk.java.net/~thartmann/8058737/webrev.01/ JDK-8075805: Crash while trying to release CompiledICHolder https://bugs.openjdk.java.net/browse/JDK-8075805 http://cr.openjdk.java.net/~thartmann/8075805_8u/webrev.01/ JDK-8134493: Cleaning inline caches of unloaded nmethods should be done in sweeper https://bugs.openjdk.java.net/browse/JDK-8134493 http://cr.openjdk.java.net/~thartmann/8134493_8u/webrev.01/ All changes (now also including 8058737) : http://cr.openjdk.java.net/~thartmann/8075805_8134493_8u/webrev.01 Thanks, Tobias > Thanks, > Vladimir > > On 10/7/15 6:27 PM, Tobias Hartmann wrote: >> Hi, >> >> please approve and review the following backport to 8u. >> >> 8075805: Crash while trying to release CompiledICHolder >> https://bugs.openjdk.java.net/browse/JDK-8075805 >> http://hg.openjdk.java.net/jdk9/hs-comp/hotspot/rev/c0ea5537dc8b >> http://cr.openjdk.java.net/~thartmann/8075805/webrev.01/ >> >> I would also like to backport this follow up change that fixes an issue with the fix for 8075805: >> >> 8134493: Cleaning inline caches of unloaded nmethods should be done in sweeper >> https://bugs.openjdk.java.net/browse/JDK-8134493 >> http://hg.openjdk.java.net/jdk9/hs-comp/hotspot/rev/9df4555d2d7d >> http://cr.openjdk.java.net/~thartmann/8134493/webrev.01/ >> >> Unfortunately, both changesets do not apply cleanly to 8u. I merged the fixes, here are the new webrevs: >> >> JDK-8075805: >> http://cr.openjdk.java.net/~thartmann/8075805_8u/webrev.00/ >> >> JDK-8134493 (incremental): >> http://cr.openjdk.java.net/~thartmann/8134493_8u/webrev.00/ >> >> Both changes: >> http://cr.openjdk.java.net/~thartmann/8075805_8134493_8u/webrev.00/ >> >> Both fixes were pushed some months ago and testing showed no problems. >> >> Thanks, >> Tobias >> From vladimir.kozlov at oracle.com Wed Oct 7 14:23:13 2015 From: vladimir.kozlov at oracle.com (Vladimir Kozlov) Date: Wed, 7 Oct 2015 22:23:13 +0800 Subject: [8u] Request for approval and review: 8075805: Crash while trying to release CompiledICHolder In-Reply-To: <56152A04.8040006@oracle.com> References: <5614F37E.8090208@oracle.com> <56151368.3030405@oracle.com> <56152A04.8040006@oracle.com> Message-ID: <56152AD1.6040600@oracle.com> Nice. Looks good. Thanks, Vladimir On 10/7/15 10:19 PM, Tobias Hartmann wrote: > Hi Vladimir, > > On 07.10.2015 14:43, Vladimir Kozlov wrote: >> Changes are matching jdk9 changes. >> Tobias, should we backport 8058737 changes too? Next code is missing in 8u now: >> >> + MutexLocker cl(CompiledIC_lock); >> + nm->clear_ic_stubs(); >> >> It was P2. Why we did not backport it? > > Yes, you are right. It seems like I missed to backport that one because it never showed up with 8. However, we should backport it. The patch for JDK-8058737 applies cleanly. I updated the other webrevs: > > JDK-8058737: CodeCache::find_blob fails with 'unsafe access to zombie method' > https://bugs.openjdk.java.net/browse/JDK-8058737 > http://cr.openjdk.java.net/~thartmann/8058737/webrev.01/ > > JDK-8075805: Crash while trying to release CompiledICHolder > https://bugs.openjdk.java.net/browse/JDK-8075805 > http://cr.openjdk.java.net/~thartmann/8075805_8u/webrev.01/ > > JDK-8134493: Cleaning inline caches of unloaded nmethods should be done in sweeper > https://bugs.openjdk.java.net/browse/JDK-8134493 > http://cr.openjdk.java.net/~thartmann/8134493_8u/webrev.01/ > > All changes (now also including 8058737) : > http://cr.openjdk.java.net/~thartmann/8075805_8134493_8u/webrev.01 > > Thanks, > Tobias > > >> Thanks, >> Vladimir >> >> On 10/7/15 6:27 PM, Tobias Hartmann wrote: >>> Hi, >>> >>> please approve and review the following backport to 8u. >>> >>> 8075805: Crash while trying to release CompiledICHolder >>> https://bugs.openjdk.java.net/browse/JDK-8075805 >>> http://hg.openjdk.java.net/jdk9/hs-comp/hotspot/rev/c0ea5537dc8b >>> http://cr.openjdk.java.net/~thartmann/8075805/webrev.01/ >>> >>> I would also like to backport this follow up change that fixes an issue with the fix for 8075805: >>> >>> 8134493: Cleaning inline caches of unloaded nmethods should be done in sweeper >>> https://bugs.openjdk.java.net/browse/JDK-8134493 >>> http://hg.openjdk.java.net/jdk9/hs-comp/hotspot/rev/9df4555d2d7d >>> http://cr.openjdk.java.net/~thartmann/8134493/webrev.01/ >>> >>> Unfortunately, both changesets do not apply cleanly to 8u. I merged the fixes, here are the new webrevs: >>> >>> JDK-8075805: >>> http://cr.openjdk.java.net/~thartmann/8075805_8u/webrev.00/ >>> >>> JDK-8134493 (incremental): >>> http://cr.openjdk.java.net/~thartmann/8134493_8u/webrev.00/ >>> >>> Both changes: >>> http://cr.openjdk.java.net/~thartmann/8075805_8134493_8u/webrev.00/ >>> >>> Both fixes were pushed some months ago and testing showed no problems. >>> >>> Thanks, >>> Tobias >>> From tobias.hartmann at oracle.com Thu Oct 8 06:58:24 2015 From: tobias.hartmann at oracle.com (Tobias Hartmann) Date: Thu, 08 Oct 2015 08:58:24 +0200 Subject: [8u] Request for approval and review: 8075805: Crash while trying to release CompiledICHolder In-Reply-To: <56152AD1.6040600@oracle.com> References: <5614F37E.8090208@oracle.com> <56151368.3030405@oracle.com> <56152A04.8040006@oracle.com> <56152AD1.6040600@oracle.com> Message-ID: <56161410.4080207@oracle.com> Thanks, Vladimir. David, are you fine with backporting JDK-8058737 as well? Best, Tobias On 07.10.2015 16:23, Vladimir Kozlov wrote: > Nice. Looks good. > > Thanks, > Vladimir > > On 10/7/15 10:19 PM, Tobias Hartmann wrote: >> Hi Vladimir, >> >> On 07.10.2015 14:43, Vladimir Kozlov wrote: >>> Changes are matching jdk9 changes. >>> Tobias, should we backport 8058737 changes too? Next code is missing in 8u now: >>> >>> + MutexLocker cl(CompiledIC_lock); >>> + nm->clear_ic_stubs(); >>> >>> It was P2. Why we did not backport it? >> >> Yes, you are right. It seems like I missed to backport that one because it never showed up with 8. However, we should backport it. The patch for JDK-8058737 applies cleanly. I updated the other webrevs: >> >> JDK-8058737: CodeCache::find_blob fails with 'unsafe access to zombie method' >> https://bugs.openjdk.java.net/browse/JDK-8058737 >> http://cr.openjdk.java.net/~thartmann/8058737/webrev.01/ >> >> JDK-8075805: Crash while trying to release CompiledICHolder >> https://bugs.openjdk.java.net/browse/JDK-8075805 >> http://cr.openjdk.java.net/~thartmann/8075805_8u/webrev.01/ >> >> JDK-8134493: Cleaning inline caches of unloaded nmethods should be done in sweeper >> https://bugs.openjdk.java.net/browse/JDK-8134493 >> http://cr.openjdk.java.net/~thartmann/8134493_8u/webrev.01/ >> >> All changes (now also including 8058737) : >> http://cr.openjdk.java.net/~thartmann/8075805_8134493_8u/webrev.01 >> >> Thanks, >> Tobias >> >> >>> Thanks, >>> Vladimir >>> >>> On 10/7/15 6:27 PM, Tobias Hartmann wrote: >>>> Hi, >>>> >>>> please approve and review the following backport to 8u. >>>> >>>> 8075805: Crash while trying to release CompiledICHolder >>>> https://bugs.openjdk.java.net/browse/JDK-8075805 >>>> http://hg.openjdk.java.net/jdk9/hs-comp/hotspot/rev/c0ea5537dc8b >>>> http://cr.openjdk.java.net/~thartmann/8075805/webrev.01/ >>>> >>>> I would also like to backport this follow up change that fixes an issue with the fix for 8075805: >>>> >>>> 8134493: Cleaning inline caches of unloaded nmethods should be done in sweeper >>>> https://bugs.openjdk.java.net/browse/JDK-8134493 >>>> http://hg.openjdk.java.net/jdk9/hs-comp/hotspot/rev/9df4555d2d7d >>>> http://cr.openjdk.java.net/~thartmann/8134493/webrev.01/ >>>> >>>> Unfortunately, both changesets do not apply cleanly to 8u. I merged the fixes, here are the new webrevs: >>>> >>>> JDK-8075805: >>>> http://cr.openjdk.java.net/~thartmann/8075805_8u/webrev.00/ >>>> >>>> JDK-8134493 (incremental): >>>> http://cr.openjdk.java.net/~thartmann/8134493_8u/webrev.00/ >>>> >>>> Both changes: >>>> http://cr.openjdk.java.net/~thartmann/8075805_8134493_8u/webrev.00/ >>>> >>>> Both fixes were pushed some months ago and testing showed no problems. >>>> >>>> Thanks, >>>> Tobias >>>> From david.buck at oracle.com Thu Oct 8 07:07:38 2015 From: david.buck at oracle.com (David Buck) Date: Thu, 8 Oct 2015 16:07:38 +0900 Subject: [8u] Request for approval and review: 8075805: Crash while trying to release CompiledICHolder In-Reply-To: <56161410.4080207@oracle.com> References: <5614F37E.8090208@oracle.com> <56151368.3030405@oracle.com> <56152A04.8040006@oracle.com> <56152AD1.6040600@oracle.com> <56161410.4080207@oracle.com> Message-ID: Yes, I also approve JDK-8058737 for backport to jdk8u-dev. Cheers, -Buck > On Oct 8, 2015, at 15:58, Tobias Hartmann wrote: > > Thanks, Vladimir. > > David, are you fine with backporting JDK-8058737 as well? > > Best, > Tobias > > On 07.10.2015 16:23, Vladimir Kozlov wrote: >> Nice. Looks good. >> >> Thanks, >> Vladimir >> >> On 10/7/15 10:19 PM, Tobias Hartmann wrote: >>> Hi Vladimir, >>> >>> On 07.10.2015 14:43, Vladimir Kozlov wrote: >>>> Changes are matching jdk9 changes. >>>> Tobias, should we backport 8058737 changes too? Next code is missing in 8u now: >>>> >>>> + MutexLocker cl(CompiledIC_lock); >>>> + nm->clear_ic_stubs(); >>>> >>>> It was P2. Why we did not backport it? >>> >>> Yes, you are right. It seems like I missed to backport that one because it never showed up with 8. However, we should backport it. The patch for JDK-8058737 applies cleanly. I updated the other webrevs: >>> >>> JDK-8058737: CodeCache::find_blob fails with 'unsafe access to zombie method' >>> https://bugs.openjdk.java.net/browse/JDK-8058737 >>> http://cr.openjdk.java.net/~thartmann/8058737/webrev.01/ >>> >>> JDK-8075805: Crash while trying to release CompiledICHolder >>> https://bugs.openjdk.java.net/browse/JDK-8075805 >>> http://cr.openjdk.java.net/~thartmann/8075805_8u/webrev.01/ >>> >>> JDK-8134493: Cleaning inline caches of unloaded nmethods should be done in sweeper >>> https://bugs.openjdk.java.net/browse/JDK-8134493 >>> http://cr.openjdk.java.net/~thartmann/8134493_8u/webrev.01/ >>> >>> All changes (now also including 8058737) : >>> http://cr.openjdk.java.net/~thartmann/8075805_8134493_8u/webrev.01 >>> >>> Thanks, >>> Tobias >>> >>> >>>> Thanks, >>>> Vladimir >>>> >>>> On 10/7/15 6:27 PM, Tobias Hartmann wrote: >>>>> Hi, >>>>> >>>>> please approve and review the following backport to 8u. >>>>> >>>>> 8075805: Crash while trying to release CompiledICHolder >>>>> https://bugs.openjdk.java.net/browse/JDK-8075805 >>>>> http://hg.openjdk.java.net/jdk9/hs-comp/hotspot/rev/c0ea5537dc8b >>>>> http://cr.openjdk.java.net/~thartmann/8075805/webrev.01/ >>>>> >>>>> I would also like to backport this follow up change that fixes an issue with the fix for 8075805: >>>>> >>>>> 8134493: Cleaning inline caches of unloaded nmethods should be done in sweeper >>>>> https://bugs.openjdk.java.net/browse/JDK-8134493 >>>>> http://hg.openjdk.java.net/jdk9/hs-comp/hotspot/rev/9df4555d2d7d >>>>> http://cr.openjdk.java.net/~thartmann/8134493/webrev.01/ >>>>> >>>>> Unfortunately, both changesets do not apply cleanly to 8u. I merged the fixes, here are the new webrevs: >>>>> >>>>> JDK-8075805: >>>>> http://cr.openjdk.java.net/~thartmann/8075805_8u/webrev.00/ >>>>> >>>>> JDK-8134493 (incremental): >>>>> http://cr.openjdk.java.net/~thartmann/8134493_8u/webrev.00/ >>>>> >>>>> Both changes: >>>>> http://cr.openjdk.java.net/~thartmann/8075805_8134493_8u/webrev.00/ >>>>> >>>>> Both fixes were pushed some months ago and testing showed no problems. >>>>> >>>>> Thanks, >>>>> Tobias >>>>> From tobias.hartmann at oracle.com Thu Oct 8 07:10:15 2015 From: tobias.hartmann at oracle.com (Tobias Hartmann) Date: Thu, 08 Oct 2015 09:10:15 +0200 Subject: [8u] Request for approval and review: 8075805: Crash while trying to release CompiledICHolder In-Reply-To: References: <5614F37E.8090208@oracle.com> <56151368.3030405@oracle.com> <56152A04.8040006@oracle.com> <56152AD1.6040600@oracle.com> <56161410.4080207@oracle.com> Message-ID: <561616D7.3050708@oracle.com> Thanks, David! Best, Tobias On 08.10.2015 09:07, David Buck wrote: > Yes, I also approve JDK-8058737 for backport to jdk8u-dev. > > Cheers, > -Buck > >> On Oct 8, 2015, at 15:58, Tobias Hartmann wrote: >> >> Thanks, Vladimir. >> >> David, are you fine with backporting JDK-8058737 as well? >> >> Best, >> Tobias >> >> On 07.10.2015 16:23, Vladimir Kozlov wrote: >>> Nice. Looks good. >>> >>> Thanks, >>> Vladimir >>> >>> On 10/7/15 10:19 PM, Tobias Hartmann wrote: >>>> Hi Vladimir, >>>> >>>> On 07.10.2015 14:43, Vladimir Kozlov wrote: >>>>> Changes are matching jdk9 changes. >>>>> Tobias, should we backport 8058737 changes too? Next code is missing in 8u now: >>>>> >>>>> + MutexLocker cl(CompiledIC_lock); >>>>> + nm->clear_ic_stubs(); >>>>> >>>>> It was P2. Why we did not backport it? >>>> >>>> Yes, you are right. It seems like I missed to backport that one because it never showed up with 8. However, we should backport it. The patch for JDK-8058737 applies cleanly. I updated the other webrevs: >>>> >>>> JDK-8058737: CodeCache::find_blob fails with 'unsafe access to zombie method' >>>> https://bugs.openjdk.java.net/browse/JDK-8058737 >>>> http://cr.openjdk.java.net/~thartmann/8058737/webrev.01/ >>>> >>>> JDK-8075805: Crash while trying to release CompiledICHolder >>>> https://bugs.openjdk.java.net/browse/JDK-8075805 >>>> http://cr.openjdk.java.net/~thartmann/8075805_8u/webrev.01/ >>>> >>>> JDK-8134493: Cleaning inline caches of unloaded nmethods should be done in sweeper >>>> https://bugs.openjdk.java.net/browse/JDK-8134493 >>>> http://cr.openjdk.java.net/~thartmann/8134493_8u/webrev.01/ >>>> >>>> All changes (now also including 8058737) : >>>> http://cr.openjdk.java.net/~thartmann/8075805_8134493_8u/webrev.01 >>>> >>>> Thanks, >>>> Tobias >>>> >>>> >>>>> Thanks, >>>>> Vladimir >>>>> >>>>> On 10/7/15 6:27 PM, Tobias Hartmann wrote: >>>>>> Hi, >>>>>> >>>>>> please approve and review the following backport to 8u. >>>>>> >>>>>> 8075805: Crash while trying to release CompiledICHolder >>>>>> https://bugs.openjdk.java.net/browse/JDK-8075805 >>>>>> http://hg.openjdk.java.net/jdk9/hs-comp/hotspot/rev/c0ea5537dc8b >>>>>> http://cr.openjdk.java.net/~thartmann/8075805/webrev.01/ >>>>>> >>>>>> I would also like to backport this follow up change that fixes an issue with the fix for 8075805: >>>>>> >>>>>> 8134493: Cleaning inline caches of unloaded nmethods should be done in sweeper >>>>>> https://bugs.openjdk.java.net/browse/JDK-8134493 >>>>>> http://hg.openjdk.java.net/jdk9/hs-comp/hotspot/rev/9df4555d2d7d >>>>>> http://cr.openjdk.java.net/~thartmann/8134493/webrev.01/ >>>>>> >>>>>> Unfortunately, both changesets do not apply cleanly to 8u. I merged the fixes, here are the new webrevs: >>>>>> >>>>>> JDK-8075805: >>>>>> http://cr.openjdk.java.net/~thartmann/8075805_8u/webrev.00/ >>>>>> >>>>>> JDK-8134493 (incremental): >>>>>> http://cr.openjdk.java.net/~thartmann/8134493_8u/webrev.00/ >>>>>> >>>>>> Both changes: >>>>>> http://cr.openjdk.java.net/~thartmann/8075805_8134493_8u/webrev.00/ >>>>>> >>>>>> Both fixes were pushed some months ago and testing showed no problems. >>>>>> >>>>>> Thanks, >>>>>> Tobias >>>>>> > From christoph.langer at sap.com Fri Oct 9 09:34:49 2015 From: christoph.langer at sap.com (Langer, Christoph) Date: Fri, 9 Oct 2015 09:34:49 +0000 Subject: [8u-dev] Request for Approval: 8134505 Cleanup of "TimeZone_md.c" Message-ID: <0DFD2E72402C9243A8630A7759B27E4338F1EDA8@DEWDFEMB12B.global.corp.sap> Hi, can you please approve a direct backport of 8134505 to 8u-dev? Bug: https://bugs.openjdk.java.net/browse/JDK-8134505 Review thread of JDK9: http://mail.openjdk.java.net/pipermail/jdk9-dev/2015-September/002642.html JDK9 changeset: http://hg.openjdk.java.net/jdk9/dev/jdk/rev/4b948e5a3e77 I did a new webrev because the patch would not simply apply after running the unshuffle script: http://cr.openjdk.java.net/~clanger/webrevs/8134505.8udev/ Thanks, Christoph From ivan.gerasimov at oracle.com Fri Oct 9 10:26:52 2015 From: ivan.gerasimov at oracle.com (Ivan Gerasimov) Date: Fri, 9 Oct 2015 13:26:52 +0300 Subject: [8u-dev] Request for approval to backport: 8137121, 8137230 ((fc) Infinite loop FileChannel.truncate) Message-ID: <5617966C.6030600@oracle.com> Hello! Would you please approve the backport of these two fixes: 8137121: (fc) Infinite loop FileChannel.truncate 8137230: TEST_BUG: java/nio/channels/FileChannel/LoopingTruncate.java timed out The fixes apply cleanly after unshuffling. Bugs: https://bugs.openjdk.java.net/browse/JDK-8137121 https://bugs.openjdk.java.net/browse/JDK-8137230 Changes: http://hg.openjdk.java.net/jdk9/jdk9/jdk/rev/551959336e36 http://hg.openjdk.java.net/jdk9/jdk9/jdk/rev/4b901f8d4b61 Reviews: http://mail.openjdk.java.net/pipermail/nio-dev/2015-September/003327.html http://mail.openjdk.java.net/pipermail/nio-dev/2015-September/003330.html Sincerely yours, Ivan From david.buck at oracle.com Fri Oct 9 10:48:37 2015 From: david.buck at oracle.com (David Buck) Date: Fri, 9 Oct 2015 19:48:37 +0900 Subject: [8u-dev] Request for approval to backport: 8137121, 8137230 ((fc) Infinite loop FileChannel.truncate) In-Reply-To: <5617966C.6030600@oracle.com> References: <5617966C.6030600@oracle.com> Message-ID: approved for backport to 8u-dev Cheers, -Buck > On Oct 9, 2015, at 19:26, Ivan Gerasimov wrote: > > Hello! > > Would you please approve the backport of these two fixes: > 8137121: (fc) Infinite loop FileChannel.truncate > 8137230: TEST_BUG: java/nio/channels/FileChannel/LoopingTruncate.java timed out > > The fixes apply cleanly after unshuffling. > > Bugs: > https://bugs.openjdk.java.net/browse/JDK-8137121 > https://bugs.openjdk.java.net/browse/JDK-8137230 > Changes: > http://hg.openjdk.java.net/jdk9/jdk9/jdk/rev/551959336e36 > http://hg.openjdk.java.net/jdk9/jdk9/jdk/rev/4b901f8d4b61 > Reviews: > http://mail.openjdk.java.net/pipermail/nio-dev/2015-September/003327.html > http://mail.openjdk.java.net/pipermail/nio-dev/2015-September/003330.html > > Sincerely yours, > Ivan > From sean.coffey at oracle.com Fri Oct 9 14:44:51 2015 From: sean.coffey at oracle.com (=?UTF-8?Q?Se=c3=a1n_Coffey?=) Date: Fri, 9 Oct 2015 15:44:51 +0100 Subject: [8u-dev] Request for Approval: 8134505 Cleanup of "TimeZone_md.c" In-Reply-To: <0DFD2E72402C9243A8630A7759B27E4338F1EDA8@DEWDFEMB12B.global.corp.sap> References: <0DFD2E72402C9243A8630A7759B27E4338F1EDA8@DEWDFEMB12B.global.corp.sap> Message-ID: <5617D2E3.4060704@oracle.com> Approved for jdk8u-dev but subject to peer review just to ensure the changes came back ok. Can you also please add a suitable noreg label to the bug report ? http://openjdk.java.net/guide/changePlanning.html#noreg Regards, Sean. On 09/10/15 10:34, Langer, Christoph wrote: > Hi, > > can you please approve a direct backport of 8134505 to 8u-dev? > > Bug: > https://bugs.openjdk.java.net/browse/JDK-8134505 > > Review thread of JDK9: > http://mail.openjdk.java.net/pipermail/jdk9-dev/2015-September/002642.html > > JDK9 changeset: > http://hg.openjdk.java.net/jdk9/dev/jdk/rev/4b948e5a3e77 > > I did a new webrev because the patch would not simply apply after running the unshuffle script: > http://cr.openjdk.java.net/~clanger/webrevs/8134505.8udev/ > > > Thanks, > Christoph > From rob.mckenna at oracle.com Fri Oct 9 15:41:20 2015 From: rob.mckenna at oracle.com (Rob McKenna) Date: Fri, 9 Oct 2015 16:41:20 +0100 Subject: [8u communication] Approaching JDK 8 Updates milestone : 8u72 RDP2 Message-ID: <5617E020.3010507@oracle.com> As per the preliminary 8u72 timeline [1], the 8u72 Rampdown 2 milestone is approaching. Please aim to have all 8u72 planned fixes addressed by Oct 23rd. The 8u72 RDP2 build is planned for Oct 26th and fixes need to be in the jdk8u-dev forest slightly beforehand for PIT testing. Once we enter RDP2, we'll fork the JDK8u master forest to create a JDK 8u72 stabilization forest. Only showstopper bugs should be considered for fixing in RDP2 after that milestone. -Rob [1] http://openjdk.java.net/projects/jdk8u/releases/8u72.html From dmitry.samersoff at oracle.com Fri Oct 9 22:10:24 2015 From: dmitry.samersoff at oracle.com (Dmitry Samersoff) Date: Sat, 10 Oct 2015 01:10:24 +0300 Subject: [8u-dev] Request for Approval: 8134505 Cleanup of "TimeZone_md.c" In-Reply-To: <0DFD2E72402C9243A8630A7759B27E4338F1EDA8@DEWDFEMB12B.global.corp.sap> References: <0DFD2E72402C9243A8630A7759B27E4338F1EDA8@DEWDFEMB12B.global.corp.sap> Message-ID: <56183B50.8040503@oracle.com> Christoph, Looks good for me! -Dmitry On 2015-10-09 12:34, Langer, Christoph wrote: > Hi, > > > > can you please approve a direct backport of 8134505 to 8u-dev? > > > > Bug: > > https://bugs.openjdk.java.net/browse/JDK-8134505 > > > > Review thread of JDK9: > > http://mail.openjdk.java.net/pipermail/jdk9-dev/2015-September/002642.html > > > > JDK9 changeset: > > http://hg.openjdk.java.net/jdk9/dev/jdk/rev/4b948e5a3e77 > > > > I did a new webrev because the patch would not simply apply after > running the unshuffle script: > > http://cr.openjdk.java.net/~clanger/webrevs/8134505.8udev/ > > > > > > Thanks, > > Christoph > > > -- Dmitry Samersoff Oracle Java development team, Saint Petersburg, Russia * I would love to change the world, but they won't give me the sources. From aleksej.efimov at oracle.com Sun Oct 11 12:13:29 2015 From: aleksej.efimov at oracle.com (Aleksej Efimov) Date: Sun, 11 Oct 2015 15:13:29 +0300 Subject: [8u-dev] Request for approval: 8139107: DateTimeFormatter with Locale.UK throw a NullPointerException when parsing zone Message-ID: <561A5269.3080902@oracle.com> Hi, Please approve backport of JDK-8139107 to 8u-dev. Changes apply cleanly to jdk8u-dev after path reshuffling. Testing shows no test failures. Bug: https://bugs.openjdk.java.net/browse/JDK-8139107 Review thread: http://mail.openjdk.java.net/pipermail/i18n-dev/2015-October/001774.html JDK9 changeset: http://hg.openjdk.java.net/jdk9/dev/jdk/rev/e860e54043fd Thank you, Aleksej From david.buck at oracle.com Sun Oct 11 13:16:18 2015 From: david.buck at oracle.com (david buck) Date: Sun, 11 Oct 2015 22:16:18 +0900 Subject: [8u-dev] Request for approval: 8139107: DateTimeFormatter with Locale.UK throw a NullPointerException when parsing zone In-Reply-To: <561A5269.3080902@oracle.com> References: <561A5269.3080902@oracle.com> Message-ID: <561A6122.8080207@oracle.com> approved for backport to 8u-dev Cheers, -Buck On 2015/10/11 21:13, Aleksej Efimov wrote: > Hi, > > Please approve backport of JDK-8139107 to 8u-dev. > Changes apply cleanly to jdk8u-dev after path reshuffling. Testing shows > no test failures. > > Bug: https://bugs.openjdk.java.net/browse/JDK-8139107 > Review thread: > http://mail.openjdk.java.net/pipermail/i18n-dev/2015-October/001774.html > JDK9 changeset: http://hg.openjdk.java.net/jdk9/dev/jdk/rev/e860e54043fd > > Thank you, > Aleksej From aleksej.efimov at oracle.com Sun Oct 11 19:51:20 2015 From: aleksej.efimov at oracle.com (Aleksej Efimov) Date: Sun, 11 Oct 2015 22:51:20 +0300 Subject: [8u-dev] Request for approval: 8139107: DateTimeFormatter with Locale.UK throw a NullPointerException when parsing zone In-Reply-To: <561A6122.8080207@oracle.com> References: <561A5269.3080902@oracle.com> <561A6122.8080207@oracle.com> Message-ID: <561ABDB8.6050104@oracle.com> Thank you David! -Aleksej On 10/11/2015 04:16 PM, david buck wrote: > approved for backport to 8u-dev > > Cheers, > -Buck > > On 2015/10/11 21:13, Aleksej Efimov wrote: >> Hi, >> >> Please approve backport of JDK-8139107 to 8u-dev. >> Changes apply cleanly to jdk8u-dev after path reshuffling. Testing shows >> no test failures. >> >> Bug: https://bugs.openjdk.java.net/browse/JDK-8139107 >> Review thread: >> http://mail.openjdk.java.net/pipermail/i18n-dev/2015-October/001774.html >> JDK9 changeset: http://hg.openjdk.java.net/jdk9/dev/jdk/rev/e860e54043fd >> >> Thank you, >> Aleksej From roland.westrelin at oracle.com Mon Oct 12 12:18:55 2015 From: roland.westrelin at oracle.com (Roland Westrelin) Date: Mon, 12 Oct 2015 14:18:55 +0200 Subject: [8u] Request for approval: 8134031: Incorrect JIT compilation of complex code with inlining and escape analysis Message-ID: Hi, Please approve the following backport to 8u: 8134031: Incorrect JIT compilation of complex code with inlining and escape analysis https://bugs.openjdk.java.net/browse/JDK-8134031 http://hg.openjdk.java.net/jdk9/jdk9/hotspot/rev/979c4f71a3c8 The change applies cleanly to 8u. It was pushed to 9 a month and a half ago and testing hasn?t shown any problem with it. Thanks, Roland. From sean.coffey at oracle.com Mon Oct 12 13:08:03 2015 From: sean.coffey at oracle.com (=?UTF-8?Q?Se=c3=a1n_Coffey?=) Date: Mon, 12 Oct 2015 14:08:03 +0100 Subject: [8u] Request for approval: 8134031: Incorrect JIT compilation of complex code with inlining and escape analysis In-Reply-To: References: Message-ID: <561BB0B3.60100@oracle.com> Approved. Regards, Sean. On 12/10/15 13:18, Roland Westrelin wrote: > Hi, > > Please approve the following backport to 8u: > > 8134031: Incorrect JIT compilation of complex code with inlining and escape analysis > https://bugs.openjdk.java.net/browse/JDK-8134031 > http://hg.openjdk.java.net/jdk9/jdk9/hotspot/rev/979c4f71a3c8 > > The change applies cleanly to 8u. It was pushed to 9 a month and a half ago and testing hasn?t shown any problem with it. > > Thanks, > Roland. From vladimir.kozlov at oracle.com Mon Oct 12 14:11:48 2015 From: vladimir.kozlov at oracle.com (Vladimir Kozlov) Date: Mon, 12 Oct 2015 22:11:48 +0800 Subject: [8u] Request for approval: 8134031: Incorrect JIT compilation of complex code with inlining and escape analysis In-Reply-To: References: Message-ID: <561BBFA4.2040406@oracle.com> Looks good. Thank you for backporting. Vladimir On 10/12/15 8:18 PM, Roland Westrelin wrote: > Hi, > > Please approve the following backport to 8u: > > 8134031: Incorrect JIT compilation of complex code with inlining and escape analysis > https://bugs.openjdk.java.net/browse/JDK-8134031 > http://hg.openjdk.java.net/jdk9/jdk9/hotspot/rev/979c4f71a3c8 > > The change applies cleanly to 8u. It was pushed to 9 a month and a half ago and testing hasn?t shown any problem with it. > > Thanks, > Roland. > From roland.westrelin at oracle.com Mon Oct 12 14:41:14 2015 From: roland.westrelin at oracle.com (Roland Westrelin) Date: Mon, 12 Oct 2015 16:41:14 +0200 Subject: [8u] Request for approval: 8134031: Incorrect JIT compilation of complex code with inlining and escape analysis In-Reply-To: <561BBFA4.2040406@oracle.com> References: <561BBFA4.2040406@oracle.com> Message-ID: <987E0D5B-DC15-4E54-8129-12A1FB676032@oracle.com> Thanks Vladimir & Sean. Roland. > On Oct 12, 2015, at 4:11 PM, Vladimir Kozlov wrote: > > Looks good. Thank you for backporting. > > Vladimir > > On 10/12/15 8:18 PM, Roland Westrelin wrote: >> Hi, >> >> Please approve the following backport to 8u: >> >> 8134031: Incorrect JIT compilation of complex code with inlining and escape analysis >> https://bugs.openjdk.java.net/browse/JDK-8134031 >> http://hg.openjdk.java.net/jdk9/jdk9/hotspot/rev/979c4f71a3c8 >> >> The change applies cleanly to 8u. It was pushed to 9 a month and a half ago and testing hasn?t shown any problem with it. >> >> Thanks, >> Roland. >> From ivan.gerasimov at oracle.com Tue Oct 13 07:57:01 2015 From: ivan.gerasimov at oracle.com (Ivan Gerasimov) Date: Tue, 13 Oct 2015 10:57:01 +0300 Subject: [8u-dev] Request for approval to backport: 8138819: (se) File descriptor leak when Selector.open fails Message-ID: <561CB94D.8020707@oracle.com> Hi! Would you please approve backport of this fix to 8u-dev? The fix applies smoothly after unshuffling. Bug: https://bugs.openjdk.java.net/browse/JDK-8138819 Jdk9 change: http://hg.openjdk.java.net/jdk9/jdk9/jdk/rev/44548e78cce5 Jdk9 review: http://mail.openjdk.java.net/pipermail/nio-dev/2015-October/003332.html Sincerely yours, Ivan From david.buck at oracle.com Tue Oct 13 08:18:26 2015 From: david.buck at oracle.com (david buck) Date: Tue, 13 Oct 2015 17:18:26 +0900 Subject: [8u-dev] Request for approval to backport: 8138819: (se) File descriptor leak when Selector.open fails In-Reply-To: <561CB94D.8020707@oracle.com> References: <561CB94D.8020707@oracle.com> Message-ID: <561CBE52.8000707@oracle.com> approved for backport to jdk8u-dev Cheers, -Buck On 2015/10/13 16:57, Ivan Gerasimov wrote: > Hi! > > Would you please approve backport of this fix to 8u-dev? > The fix applies smoothly after unshuffling. > > Bug: https://bugs.openjdk.java.net/browse/JDK-8138819 > Jdk9 change: http://hg.openjdk.java.net/jdk9/jdk9/jdk/rev/44548e78cce5 > Jdk9 review: > http://mail.openjdk.java.net/pipermail/nio-dev/2015-October/003332.html > > Sincerely yours, > Ivan > From volker.simonis at gmail.com Tue Oct 13 08:35:00 2015 From: volker.simonis at gmail.com (Volker Simonis) Date: Tue, 13 Oct 2015 10:35:00 +0200 Subject: [8u-dev] Request for Approval: 8134505 Cleanup of "TimeZone_md.c" In-Reply-To: <56183B50.8040503@oracle.com> References: <0DFD2E72402C9243A8630A7759B27E4338F1EDA8@DEWDFEMB12B.global.corp.sap> <56183B50.8040503@oracle.com> Message-ID: Thanks a lot Sean and Dmitry! I'll sponsor this change. Regards, Volker On Sat, Oct 10, 2015 at 12:10 AM, Dmitry Samersoff < dmitry.samersoff at oracle.com> wrote: > Christoph, > > Looks good for me! > > -Dmitry > > On 2015-10-09 12:34, Langer, Christoph wrote: > > Hi, > > > > > > > > can you please approve a direct backport of 8134505 to 8u-dev? > > > > > > > > Bug: > > > > https://bugs.openjdk.java.net/browse/JDK-8134505 > > > > > > > > Review thread of JDK9: > > > > > http://mail.openjdk.java.net/pipermail/jdk9-dev/2015-September/002642.html > > > > > > > > JDK9 changeset: > > > > http://hg.openjdk.java.net/jdk9/dev/jdk/rev/4b948e5a3e77 > > > > > > > > I did a new webrev because the patch would not simply apply after > > running the unshuffle script: > > > > http://cr.openjdk.java.net/~clanger/webrevs/8134505.8udev/ > > > > > > > > > > > > Thanks, > > > > Christoph > > > > > > > > > -- > Dmitry Samersoff > Oracle Java development team, Saint Petersburg, Russia > * I would love to change the world, but they won't give me the sources. > From cheleswer.sahu at oracle.com Tue Oct 13 13:47:44 2015 From: cheleswer.sahu at oracle.com (cheleswer sahu) Date: Tue, 13 Oct 2015 19:17:44 +0530 Subject: [8u-dev] Request for approval: 8074368: ThreadMXBean.getThreadInfo() corrupts memory when called with empty array for thread ids Message-ID: <561D0B80.9070008@oracle.com> Hi, May I please have approval to backport this fix from JDK9 to JDK8. I have built the JDK8 and tested already. JDK9 fix applies cleanly to JDK8u after reshuffling. As I do not have account for OPENJDK, Kevin walls will push this fix into JDK8u repository. Bug: https://bugs.openjdk.java.net/browse/JDK-8074368 Jdk9 change: http://hg.openjdk.java.net/jdk9/jdk9/hotspot/rev/bd08f461c7e4 http://hg.openjdk.java.net/jdk9/jdk9/jdk/rev/4b6014c70710 Jdk9 review: http://mail.openjdk.java.net/pipermail/serviceability-dev/2015-April/017060.html Regards, Cheleswer From rob.mckenna at oracle.com Tue Oct 13 14:00:50 2015 From: rob.mckenna at oracle.com (Rob McKenna) Date: Tue, 13 Oct 2015 15:00:50 +0100 Subject: [8u-dev] Request for approval: 8074368: ThreadMXBean.getThreadInfo() corrupts memory when called with empty array for thread ids In-Reply-To: <561D0B80.9070008@oracle.com> References: <561D0B80.9070008@oracle.com> Message-ID: <561D0E92.3030007@oracle.com> Approved. -Rob On 13/10/15 14:47, cheleswer sahu wrote: > Hi, > May I please have approval to backport this fix from JDK9 to JDK8. I > have built the JDK8 and tested already. JDK9 fix applies cleanly to > JDK8u after reshuffling. As I do not have account for OPENJDK, Kevin > walls will push this fix into JDK8u repository. > > Bug: https://bugs.openjdk.java.net/browse/JDK-8074368 > Jdk9 change: http://hg.openjdk.java.net/jdk9/jdk9/hotspot/rev/bd08f461c7e4 > http://hg.openjdk.java.net/jdk9/jdk9/jdk/rev/4b6014c70710 > Jdk9 review: > http://mail.openjdk.java.net/pipermail/serviceability-dev/2015-April/017060.html > > > > > Regards, > Cheleswer From tobias.hartmann at oracle.com Wed Oct 14 12:02:30 2015 From: tobias.hartmann at oracle.com (Tobias Hartmann) Date: Wed, 14 Oct 2015 14:02:30 +0200 Subject: [8u] Request for approval and review: 8139150: ClassVerifier frees exception message while it's still in use Message-ID: <561E4456.1090205@oracle.com> Hi, please approve and review the following backport to 8u. https://bugs.openjdk.java.net/browse/JDK-8139150 http://cr.openjdk.java.net/~thartmann/8139150/webrev.00/ http://hg.openjdk.java.net/jdk9/hs-comp/hotspot/rev/926d9bae67d3 Nightly testing showed no problems. The change applies cleanly to 8u. Thanks, Tobias From vladimir.kozlov at oracle.com Wed Oct 14 12:09:49 2015 From: vladimir.kozlov at oracle.com (Vladimir Kozlov) Date: Wed, 14 Oct 2015 20:09:49 +0800 Subject: [8u] Request for approval and review: 8139150: ClassVerifier frees exception message while it's still in use In-Reply-To: <561E4456.1090205@oracle.com> References: <561E4456.1090205@oracle.com> Message-ID: <561E460D.8060802@oracle.com> Good. Thanks, Vladimir On 10/14/15 8:02 PM, Tobias Hartmann wrote: > Hi, > > please approve and review the following backport to 8u. > > https://bugs.openjdk.java.net/browse/JDK-8139150 > http://cr.openjdk.java.net/~thartmann/8139150/webrev.00/ > http://hg.openjdk.java.net/jdk9/hs-comp/hotspot/rev/926d9bae67d3 > > Nightly testing showed no problems. The change applies cleanly to 8u. > > Thanks, > Tobias > From david.buck at oracle.com Wed Oct 14 12:27:17 2015 From: david.buck at oracle.com (David Buck) Date: Wed, 14 Oct 2015 21:27:17 +0900 Subject: [8u] Request for approval and review: 8139150: ClassVerifier frees exception message while it's still in use In-Reply-To: <561E4456.1090205@oracle.com> References: <561E4456.1090205@oracle.com> Message-ID: JDK 9 review thread: http://mail.openjdk.java.net/pipermail/hotspot-dev/2015-October/020206.html approved for backport to jdk8u-dev Cheers, -Buck > On Oct 14, 2015, at 21:02, Tobias Hartmann wrote: > > Hi, > > please approve and review the following backport to 8u. > > https://bugs.openjdk.java.net/browse/JDK-8139150 > http://cr.openjdk.java.net/~thartmann/8139150/webrev.00/ > http://hg.openjdk.java.net/jdk9/hs-comp/hotspot/rev/926d9bae67d3 > > Nightly testing showed no problems. The change applies cleanly to 8u. > > Thanks, > Tobias From tobias.hartmann at oracle.com Wed Oct 14 12:26:51 2015 From: tobias.hartmann at oracle.com (Tobias Hartmann) Date: Wed, 14 Oct 2015 14:26:51 +0200 Subject: [8u] Request for approval and review: 8139150: ClassVerifier frees exception message while it's still in use In-Reply-To: <561E460D.8060802@oracle.com> References: <561E4456.1090205@oracle.com> <561E460D.8060802@oracle.com> Message-ID: <561E4A0B.9010408@oracle.com> Thanks, Vladimir. Best, Tobias On 14.10.2015 14:09, Vladimir Kozlov wrote: > Good. > > Thanks, > Vladimir > > On 10/14/15 8:02 PM, Tobias Hartmann wrote: >> Hi, >> >> please approve and review the following backport to 8u. >> >> https://bugs.openjdk.java.net/browse/JDK-8139150 >> http://cr.openjdk.java.net/~thartmann/8139150/webrev.00/ >> http://hg.openjdk.java.net/jdk9/hs-comp/hotspot/rev/926d9bae67d3 >> >> Nightly testing showed no problems. The change applies cleanly to 8u. >> >> Thanks, >> Tobias >> From tobias.hartmann at oracle.com Wed Oct 14 13:43:54 2015 From: tobias.hartmann at oracle.com (Tobias Hartmann) Date: Wed, 14 Oct 2015 15:43:54 +0200 Subject: [8u] Request for approval and review: 8139150: ClassVerifier frees exception message while it's still in use In-Reply-To: References: <561E4456.1090205@oracle.com> Message-ID: <561E5C1A.3030701@oracle.com> Thanks, David. Best, Tobias On 14.10.2015 14:27, David Buck wrote: > JDK 9 review thread: http://mail.openjdk.java.net/pipermail/hotspot-dev/2015-October/020206.html > > approved for backport to jdk8u-dev > > Cheers, > -Buck > > >> On Oct 14, 2015, at 21:02, Tobias Hartmann wrote: >> >> Hi, >> >> please approve and review the following backport to 8u. >> >> https://bugs.openjdk.java.net/browse/JDK-8139150 >> http://cr.openjdk.java.net/~thartmann/8139150/webrev.00/ >> http://hg.openjdk.java.net/jdk9/hs-comp/hotspot/rev/926d9bae67d3 >> >> Nightly testing showed no problems. The change applies cleanly to 8u. >> >> Thanks, >> Tobias > From sean.coffey at oracle.com Wed Oct 14 15:47:10 2015 From: sean.coffey at oracle.com (=?UTF-8?Q?Se=c3=a1n_Coffey?=) Date: Wed, 14 Oct 2015 16:47:10 +0100 Subject: Request for approval :8136534: Loading JKS keystore using non-null InputStream results in closed stream Message-ID: <561E78FE.4000708@oracle.com> Looking to push this fix to jdk8u-dev. bug report : https://bugs.openjdk.java.net/browse/JDK-8136534 webrev : http://cr.openjdk.java.net/~coffeys/webrev.8136534.8u/webrev/ review thread : http://mail.openjdk.java.net/pipermail/security-dev/2015-October/012944.html -- Regards, Sean. From rob.mckenna at oracle.com Wed Oct 14 15:49:54 2015 From: rob.mckenna at oracle.com (Rob McKenna) Date: Wed, 14 Oct 2015 16:49:54 +0100 Subject: Request for approval :8136534: Loading JKS keystore using non-null InputStream results in closed stream In-Reply-To: <561E78FE.4000708@oracle.com> References: <561E78FE.4000708@oracle.com> Message-ID: <561E79A2.3000905@oracle.com> Approved -Rob On 14/10/15 16:47, Se?n Coffey wrote: > Looking to push this fix to jdk8u-dev. > > bug report : https://bugs.openjdk.java.net/browse/JDK-8136534 > webrev : http://cr.openjdk.java.net/~coffeys/webrev.8136534.8u/webrev/ > review thread : > http://mail.openjdk.java.net/pipermail/security-dev/2015-October/012944.html > > From rob.mckenna at oracle.com Thu Oct 15 11:50:24 2015 From: rob.mckenna at oracle.com (Rob McKenna) Date: Thu, 15 Oct 2015 12:50:24 +0100 Subject: [8u-dev] Request for approval: 8135305 & 8133015 Message-ID: <561F9300.30304@oracle.com> Hi folks, Looking to backport these changes to 8. Backports apply cleanly post shuffle: 8133015: InetAddress.isReachable(tmout) returning wrong value on Windows for IPv6 https://bugs.openjdk.java.net/browse/JDK-8133015 http://hg.openjdk.java.net/jdk9/jdk9/jdk/rev/59ff6cd9535d http://mail.openjdk.java.net/pipermail/net-dev/2015-August/009066.html 8135305: api/java_net/InetAddress/complement_isReachable failed https://bugs.openjdk.java.net/browse/JDK-8135305 http://hg.openjdk.java.net/jdk9/jdk9/jdk/rev/2dc2f2b7f1ac http://mail.openjdk.java.net/pipermail/net-dev/2015-September/009165.html -Rob From sean.coffey at oracle.com Thu Oct 15 12:39:49 2015 From: sean.coffey at oracle.com (=?UTF-8?Q?Se=c3=a1n_Coffey?=) Date: Thu, 15 Oct 2015 13:39:49 +0100 Subject: [8u-dev] Request for approval: 8135305 & 8133015 In-Reply-To: <561F9300.30304@oracle.com> References: <561F9300.30304@oracle.com> Message-ID: <561F9E95.1050801@oracle.com> Approved. Regards, Sean. On 15/10/2015 12:50, Rob McKenna wrote: > Hi folks, > > Looking to backport these changes to 8. Backports apply cleanly post > shuffle: > > 8133015: InetAddress.isReachable(tmout) returning wrong value on > Windows for IPv6 > https://bugs.openjdk.java.net/browse/JDK-8133015 > http://hg.openjdk.java.net/jdk9/jdk9/jdk/rev/59ff6cd9535d > http://mail.openjdk.java.net/pipermail/net-dev/2015-August/009066.html > > 8135305: api/java_net/InetAddress/complement_isReachable failed > https://bugs.openjdk.java.net/browse/JDK-8135305 > http://hg.openjdk.java.net/jdk9/jdk9/jdk/rev/2dc2f2b7f1ac > http://mail.openjdk.java.net/pipermail/net-dev/2015-September/009165.html > > -Rob > From rob.mckenna at oracle.com Fri Oct 16 11:49:17 2015 From: rob.mckenna at oracle.com (Rob McKenna) Date: Fri, 16 Oct 2015 12:49:17 +0100 Subject: [8u-dev] Request for review & approval - 8133249: Occasional SIGSEGV: non thread-safe use of strerr in getLastErrorString Message-ID: <5620E43D.6090703@oracle.com> Hi folks, This fix required minor changes to the JDK9 version due to small differences between repos. Webrev: http://cr.openjdk.java.net/~robm/8133249/webrev.jdk8/ https://bugs.openjdk.java.net/browse/JDK-8133249 http://hg.openjdk.java.net/jdk9/jdk9/jdk/rev/5f8661991f73 http://mail.openjdk.java.net/pipermail/core-libs-dev/2015-September/035311.html -Rob From ivan.gerasimov at oracle.com Fri Oct 16 12:05:55 2015 From: ivan.gerasimov at oracle.com (Ivan Gerasimov) Date: Fri, 16 Oct 2015 15:05:55 +0300 Subject: [8u-dev] Request for review & approval - 8133249: Occasional SIGSEGV: non thread-safe use of strerr in getLastErrorString In-Reply-To: <5620E43D.6090703@oracle.com> References: <5620E43D.6090703@oracle.com> Message-ID: <5620E823.1000000@oracle.com> Hi Rob! Looks good. Just a couple of missing spaces in src/java.base/unix/native/libnet/PlainDatagramSocketImpl.c, not a big deal, I think. I'm not a reviewer in jdk8u-dev project, so someone else will also need to express his/her opinion. Sincerely yours, Ivan On 16.10.2015 14:49, Rob McKenna wrote: > Hi folks, > > This fix required minor changes to the JDK9 version due to small > differences between repos. Webrev: > > http://cr.openjdk.java.net/~robm/8133249/webrev.jdk8/ > > https://bugs.openjdk.java.net/browse/JDK-8133249 > http://hg.openjdk.java.net/jdk9/jdk9/jdk/rev/5f8661991f73 > http://mail.openjdk.java.net/pipermail/core-libs-dev/2015-September/035311.html > > > -Rob > From sean.coffey at oracle.com Fri Oct 16 14:38:23 2015 From: sean.coffey at oracle.com (=?UTF-8?Q?Se=c3=a1n_Coffey?=) Date: Fri, 16 Oct 2015 15:38:23 +0100 Subject: [8u-dev] Request for review & approval - 8133249: Occasional SIGSEGV: non thread-safe use of strerr in getLastErrorString In-Reply-To: <5620E43D.6090703@oracle.com> References: <5620E43D.6090703@oracle.com> Message-ID: <56210BDF.8000409@oracle.com> Looks good. Approved. Regards, Sean. On 16/10/15 12:49, Rob McKenna wrote: > Hi folks, > > This fix required minor changes to the JDK9 version due to small > differences between repos. Webrev: > > http://cr.openjdk.java.net/~robm/8133249/webrev.jdk8/ > > https://bugs.openjdk.java.net/browse/JDK-8133249 > http://hg.openjdk.java.net/jdk9/jdk9/jdk/rev/5f8661991f73 > http://mail.openjdk.java.net/pipermail/core-libs-dev/2015-September/035311.html > > > -Rob From aleksej.efimov at oracle.com Mon Oct 19 16:36:28 2015 From: aleksej.efimov at oracle.com (Aleksej Efimov) Date: Mon, 19 Oct 2015 19:36:28 +0300 Subject: [8u-dev] Request for approval: 8073519: schemagen does not report errors while generating xsd files Message-ID: <56251C0C.8050006@oracle.com> Hi, Please approve backport of JDK-8073519 to 8u-dev. Changes applies cleanly to jdk8u-dev after path reshuffling. Testing (JPRT and JTREG) shows no test failures. Bug: https://bugs.openjdk.java.net/browse/JDK-8073519 Review thread: http://mail.openjdk.java.net/pipermail/core-libs-dev/2015-October/035781.html JDK9 changesets: http://hg.openjdk.java.net/jdk9/dev/jaxws/rev/3d533243505d http://hg.openjdk.java.net/jdk9/dev/jdk/rev/0440acded788 Thank you, Alekse From rob.mckenna at oracle.com Mon Oct 19 16:39:58 2015 From: rob.mckenna at oracle.com (Rob McKenna) Date: Mon, 19 Oct 2015 17:39:58 +0100 Subject: [8u-dev] Request for approval: 8073519: schemagen does not report errors while generating xsd files In-Reply-To: <56251C0C.8050006@oracle.com> References: <56251C0C.8050006@oracle.com> Message-ID: <56251CDE.50409@oracle.com> Approved -Rob On 19/10/15 17:36, Aleksej Efimov wrote: > Hi, > > Please approve backport of JDK-8073519 to 8u-dev. > Changes applies cleanly to jdk8u-dev after path reshuffling. Testing > (JPRT and JTREG) shows > no test failures. > > Bug: > https://bugs.openjdk.java.net/browse/JDK-8073519 > > Review thread: > http://mail.openjdk.java.net/pipermail/core-libs-dev/2015-October/035781.html > > > JDK9 changesets: > http://hg.openjdk.java.net/jdk9/dev/jaxws/rev/3d533243505d > http://hg.openjdk.java.net/jdk9/dev/jdk/rev/0440acded788 > > Thank you, > Alekse From erik.joelsson at oracle.com Tue Oct 20 15:31:18 2015 From: erik.joelsson at oracle.com (Erik Joelsson) Date: Tue, 20 Oct 2015 17:31:18 +0200 Subject: [8u-dev]: Request for approval: 8139813: Base heap size on type of boot jdk, not architecture of build machine Message-ID: <56265E46.5030401@oracle.com> Hello, Please approve backport of JDK-8139813 to 8u-dev. Changes for both JDK 9 and JDK 8u have been reviewed together as the same patch would not quite apply. Bug: https://bugs.openjdk.java.net/browse/JDK-8139813 Review thread: http://mail.openjdk.java.net/pipermail/build-dev/2015-October/015722.html JDK 9 Changeset: http://hg.openjdk.java.net/jdk9/dev/rev/cb3f10185e63 /Erik From sean.coffey at oracle.com Tue Oct 20 15:38:48 2015 From: sean.coffey at oracle.com (=?UTF-8?Q?Se=c3=a1n_Coffey?=) Date: Tue, 20 Oct 2015 16:38:48 +0100 Subject: [8u-dev]: Request for approval: 8139813: Base heap size on type of boot jdk, not architecture of build machine In-Reply-To: <56265E46.5030401@oracle.com> References: <56265E46.5030401@oracle.com> Message-ID: <56266008.6070004@oracle.com> Approved. Regards, Sean. On 20/10/2015 16:31, Erik Joelsson wrote: > Hello, > > Please approve backport of JDK-8139813 to 8u-dev. Changes for both JDK > 9 and JDK 8u have been reviewed together as the same patch would not > quite apply. > > Bug: https://bugs.openjdk.java.net/browse/JDK-8139813 > > Review thread: > http://mail.openjdk.java.net/pipermail/build-dev/2015-October/015722.html > > JDK 9 Changeset: http://hg.openjdk.java.net/jdk9/dev/rev/cb3f10185e63 > > /Erik From abhi.saha at oracle.com Tue Oct 20 20:26:05 2015 From: abhi.saha at oracle.com (Abhi Saha) Date: Tue, 20 Oct 2015 13:26:05 -0700 Subject: [8u72] Request for approval for bulk changes from Oct 2015 CPU fixes into 8u72 Message-ID: <5626A35D.2050402@oracle.com> 8u65/8u66 has been released earlier today [1]. Requesting approval to sync up the changes into the jdk8u forest. webrev : http://cr.openjdk.java.net/~asaha/openJDK.8u66-8u72.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.buck at oracle.com Tue Oct 20 20:43:10 2015 From: david.buck at oracle.com (david buck) Date: Tue, 20 Oct 2015 13:43:10 -0700 Subject: [8u72] Request for approval for bulk changes from Oct 2015 CPU fixes into 8u72 In-Reply-To: <5626A35D.2050402@oracle.com> References: <5626A35D.2050402@oracle.com> Message-ID: <5626A75E.9040509@oracle.com> approved for backport to 8u-dev Cheers, -Buck On 2015/10/20 13:26, Abhi Saha wrote: > 8u65/8u66 has been released earlier today [1]. Requesting approval to > sync up the changes into the jdk8u forest. > > webrev : http://cr.openjdk.java.net/~asaha/openJDK.8u66-8u72.sync/webrev > > > Thanks > Abhijit > > > [1] http://www.oracle.com/technetwork/java/javase/downloads/index.html > > From sean.coffey at oracle.com Wed Oct 21 10:35:08 2015 From: sean.coffey at oracle.com (=?UTF-8?Q?Se=c3=a1n_Coffey?=) Date: Wed, 21 Oct 2015 11:35:08 +0100 Subject: [8u-communication] JDK 8u66 is now available! Message-ID: <56276A5C.1070601@oracle.com> JDK 8u66 was released yesterday [0]. Thanks to all those who have contributed towards it. OpenJDK 8u66 source code is available from the jdk8u master forest via use of the 'jdk8u66-b17' mercurial tag. I plan to update the OpenJDK 8u project page with latest status. If you're packaging this release, it could be useful to let subscribed members know about it via communication on this mailing list. Please continue to contribute fixes back to the jdk8u-dev forest [1] which is already gathering changes for 8u72. Note that 8u72 based early access binaries from Oracle are already available [2] regards, Sean. [0] http://www.oracle.com/technetwork/java/javase/downloads/index.html [1] http://hg.openjdk.java.net/jdk8u/jdk8u-dev [2] https://jdk8.java.net/download.html -- Regards, Sean. From sean.coffey at oracle.com Wed Oct 21 10:41:18 2015 From: sean.coffey at oracle.com (=?UTF-8?Q?Se=c3=a1n_Coffey?=) Date: Wed, 21 Oct 2015 11:41:18 +0100 Subject: [8u-dev] Request for approval : 6907252, 8038502 Message-ID: <56276BCE.9040603@oracle.com> Looking to push these two fixes to the jdk8u-dev forest : https://bugs.openjdk.java.net/browse/JDK-6907252 6907252: ZipFileInputStream Not Thread-Safe JDK 9 review thread : http://mail.openjdk.java.net/pipermail/core-libs-dev/2015-October/035867.html https://bugs.openjdk.java.net/browse/JDK-8038502 8038502: Deflater.needsInput() should use synchronization JDK 9 review thread : http://mail.openjdk.java.net/pipermail/core-libs-dev/2015-October/035913.html Both changes apply cleanly after modular path unshuffling. -- Regards, Sean. From dalibor.topic at oracle.com Wed Oct 21 11:01:54 2015 From: dalibor.topic at oracle.com (dalibor topic) Date: Wed, 21 Oct 2015 13:01:54 +0200 Subject: [8u-dev] Request for approval : 6907252, 8038502 In-Reply-To: <56276BCE.9040603@oracle.com> References: <56276BCE.9040603@oracle.com> Message-ID: <562770A2.3070609@oracle.com> Approved. On 21.10.2015 12:41, Se?n Coffey wrote: > Looking to push these two fixes to the jdk8u-dev forest : > > https://bugs.openjdk.java.net/browse/JDK-6907252 > 6907252: ZipFileInputStream Not Thread-Safe > JDK 9 review thread : > http://mail.openjdk.java.net/pipermail/core-libs-dev/2015-October/035867.html > > > https://bugs.openjdk.java.net/browse/JDK-8038502 > 8038502: Deflater.needsInput() should use synchronization > JDK 9 review thread : > http://mail.openjdk.java.net/pipermail/core-libs-dev/2015-October/035913.html > > > Both changes apply cleanly after modular path unshuffling. > -- 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, Astrid Kepper, Val Maher Oracle is committed to developing practices and products that help protect the environment From alexey.ivanov at oracle.com Wed Oct 21 14:32:19 2015 From: alexey.ivanov at oracle.com (Alexey Ivanov) Date: Wed, 21 Oct 2015 17:32:19 +0300 Subject: [8u-dev] Request for approval for 8130136: Swing window sometimes fails to repaint partially when it becomes exposed Message-ID: <5627A1F3.70407@oracle.com> Hello, Could you please approve the following backport of the fix to 8u-dev? The patch from JDK9 applies cleanly after paths unshuffling. JBS: https://bugs.openjdk.java.net/browse/JDK-8130136 Webrev: http://cr.openjdk.java.net/~aivanov/8130136/jdk9/webrev.01/ Review threads: http://mail.openjdk.java.net/pipermail/2d-dev/2015-October/005717.html http://mail.openjdk.java.net/pipermail/awt-dev/2015-October/010141.html JDK9 changeset: http://hg.openjdk.java.net/jdk9/client/jdk/rev/4d719805b1f1 Thanks, Alexey From sean.coffey at oracle.com Wed Oct 21 15:20:12 2015 From: sean.coffey at oracle.com (=?UTF-8?Q?Se=c3=a1n_Coffey?=) Date: Wed, 21 Oct 2015 16:20:12 +0100 Subject: [8u-dev] Request for approval for 8130136: Swing window sometimes fails to repaint partially when it becomes exposed In-Reply-To: <5627A1F3.70407@oracle.com> References: <5627A1F3.70407@oracle.com> Message-ID: <5627AD2C.4030900@oracle.com> Approved. Regards, Sean. On 21/10/15 15:32, Alexey Ivanov wrote: > Hello, > > Could you please approve the following backport of the fix to 8u-dev? > The patch from JDK9 applies cleanly after paths unshuffling. > > JBS: https://bugs.openjdk.java.net/browse/JDK-8130136 > Webrev: http://cr.openjdk.java.net/~aivanov/8130136/jdk9/webrev.01/ > > Review threads: > http://mail.openjdk.java.net/pipermail/2d-dev/2015-October/005717.html > http://mail.openjdk.java.net/pipermail/awt-dev/2015-October/010141.html > > JDK9 changeset: > http://hg.openjdk.java.net/jdk9/client/jdk/rev/4d719805b1f1 > > Thanks, > Alexey From mikhail.cherkasov at oracle.com Wed Oct 21 16:18:32 2015 From: mikhail.cherkasov at oracle.com (mikhail cherkasov) Date: Wed, 21 Oct 2015 19:18:32 +0300 Subject: [8u-dev] Request for Approval: 8136763,[macosx] java always returns only one value for "text/uri-list" dataflavor even if several files were copied Message-ID: <5627BAD8.5090705@oracle.com> Hi all, Could you please approve a direct backport of 8136763 to 8u-dev? Bug: https://bugs.openjdk.java.net/browse/JDK-8136763 Review threads: http://mail.openjdk.java.net/pipermail/awt-dev/2015-October/010116.html JDK9 changeset: http://hg.openjdk.java.net/jdk9/client/jdk/rev/3fdbedd9ff1b Thanks, Mikhail. From sean.coffey at oracle.com Wed Oct 21 16:31:46 2015 From: sean.coffey at oracle.com (=?UTF-8?Q?Se=c3=a1n_Coffey?=) Date: Wed, 21 Oct 2015 17:31:46 +0100 Subject: [8u-dev] Request for Approval: 8136763,[macosx] java always returns only one value for "text/uri-list" dataflavor even if several files were copied In-Reply-To: <5627BAD8.5090705@oracle.com> References: <5627BAD8.5090705@oracle.com> Message-ID: <5627BDF2.1040405@oracle.com> Approved. Regards, Sean. On 21/10/15 17:18, mikhail cherkasov wrote: > Hi all, > > Could you please approve a direct backport of 8136763 to 8u-dev? > > Bug: > https://bugs.openjdk.java.net/browse/JDK-8136763 > > Review threads: > http://mail.openjdk.java.net/pipermail/awt-dev/2015-October/010116.html > > JDK9 changeset: > http://hg.openjdk.java.net/jdk9/client/jdk/rev/3fdbedd9ff1b > > > Thanks, > Mikhail. From sean.coffey at oracle.com Wed Oct 21 16:32:59 2015 From: sean.coffey at oracle.com (=?UTF-8?Q?Se=c3=a1n_Coffey?=) Date: Wed, 21 Oct 2015 17:32:59 +0100 Subject: [8u-dev] Request for Approval: 8086038 [macosx] No available data flavors when copying from Microsoft Word for Mac In-Reply-To: <561381F5.4090401@oracle.com> References: <56137DB8.8050702@oracle.com> <561381F5.4090401@oracle.com> Message-ID: <5627BE3B.1000000@oracle.com> Mikhail - can you follow up on this request and please add a noreg- label to master bug report ? Regards, Sean. On 06/10/15 09:10, Se?n Coffey wrote: > Please add a suitable noreg- label to the bug report. > > Approved. > > Regards, > Sean. > > On 06/10/2015 08:52, mikhail cherkasov wrote: >> Hi all, >> >> Could you please approve a direct backport of 8086038 to 8u-dev? >> >> Bug: >> https://bugs.openjdk.java.net/browse/JDK-8086038 >> >> Review threads: >> http://mail.openjdk.java.net/pipermail/awt-dev/2015-September/010091.html >> >> http://mail.openjdk.java.net/pipermail/awt-dev/2015-October/010097.html >> >> JDK9 changeset: >> http://hg.openjdk.java.net/jdk9/client/jdk/rev/eafaa1778c63 >> >> Thanks, >> Mikhail. > From mikhail.cherkasov at oracle.com Wed Oct 21 16:48:13 2015 From: mikhail.cherkasov at oracle.com (mikhail cherkasov) Date: Wed, 21 Oct 2015 19:48:13 +0300 Subject: [8u-dev] Request for Approval: 8086038 [macosx] No available data flavors when copying from Microsoft Word for Mac In-Reply-To: <5627BE3B.1000000@oracle.com> References: <56137DB8.8050702@oracle.com> <561381F5.4090401@oracle.com> <5627BE3B.1000000@oracle.com> Message-ID: <5627C1CD.5050205@oracle.com> Hi Sean, The label has been added. Thanks, Mikhail. On 10/21/2015 19:32, Se?n Coffey wrote: > Mikhail - > > can you follow up on this request and please add a noreg- label to > master bug report ? > > Regards, > Sean. > > On 06/10/15 09:10, Se?n Coffey wrote: >> Please add a suitable noreg- label to the bug report. >> >> Approved. >> >> Regards, >> Sean. >> >> On 06/10/2015 08:52, mikhail cherkasov wrote: >>> Hi all, >>> >>> Could you please approve a direct backport of 8086038 to 8u-dev? >>> >>> Bug: >>> https://bugs.openjdk.java.net/browse/JDK-8086038 >>> >>> Review threads: >>> http://mail.openjdk.java.net/pipermail/awt-dev/2015-September/010091.html >>> >>> http://mail.openjdk.java.net/pipermail/awt-dev/2015-October/010097.html >>> >>> JDK9 changeset: >>> http://hg.openjdk.java.net/jdk9/client/jdk/rev/eafaa1778c63 >>> >>> Thanks, >>> Mikhail. >> > From Sergey.Bylokhov at oracle.com Fri Oct 23 01:09:40 2015 From: Sergey.Bylokhov at oracle.com (Sergey Bylokhov) Date: Fri, 23 Oct 2015 04:09:40 +0300 Subject: [8u-dev] Request for approval: 8138764, 8041900 Message-ID: <562988D4.2040803@oracle.com> Hello, These are a direct back ports from jdk 9 to jdk 8u-dev. 8138764: In some cases the usage of TreeLock can be replaced by other synchronization Bug: https://bugs.openjdk.java.net/browse/JDK-8138764 Webrev can be found at: http://cr.openjdk.java.net/~serb/8138764/webrev.00 jdk9 changeset: http://hg.openjdk.java.net/jdk9/client/jdk/rev/c62b791a8960 Review: http://mail.openjdk.java.net/pipermail/awt-dev/2015-October/010119.html Reviewers: Alexander Potochkin, Alexander Scherbatiy 8041900: [macosx] Java forces the use of discrete GPU Bug: https://bugs.openjdk.java.net/browse/JDK-8041900 Webrev can be found at: http://cr.openjdk.java.net/~serb/8041900/webrev.00 jdk9 changeset: http://hg.openjdk.java.net/jdk9/client/jdk/rev/0746c20f1365 Review: http://mail.openjdk.java.net/pipermail/2d-dev/2015-October/005747.html Reviewers: Alexander Scherbatiy, Semyon Sadetsky -- Best regards, Sergey. From david.buck at oracle.com Thu Oct 22 22:11:17 2015 From: david.buck at oracle.com (david buck) Date: Thu, 22 Oct 2015 15:11:17 -0700 Subject: [8u-dev] Request for approval: 8138764, 8041900 In-Reply-To: <562988D4.2040803@oracle.com> References: <562988D4.2040803@oracle.com> Message-ID: <56295F05.6080600@oracle.com> approved for backport to jdk8u-dev Cheers, -Buck On 2015/10/22 18:09, Sergey Bylokhov wrote: > Hello, > These are a direct back ports from jdk 9 to jdk 8u-dev. > > 8138764: In some cases the usage of TreeLock can be replaced by other > synchronization > Bug: https://bugs.openjdk.java.net/browse/JDK-8138764 > Webrev can be found at: http://cr.openjdk.java.net/~serb/8138764/webrev.00 > jdk9 changeset: http://hg.openjdk.java.net/jdk9/client/jdk/rev/c62b791a8960 > Review: > http://mail.openjdk.java.net/pipermail/awt-dev/2015-October/010119.html > Reviewers: Alexander Potochkin, Alexander Scherbatiy > > > 8041900: [macosx] Java forces the use of discrete GPU > Bug: https://bugs.openjdk.java.net/browse/JDK-8041900 > Webrev can be found at: http://cr.openjdk.java.net/~serb/8041900/webrev.00 > jdk9 changeset: http://hg.openjdk.java.net/jdk9/client/jdk/rev/0746c20f1365 > Review: > http://mail.openjdk.java.net/pipermail/2d-dev/2015-October/005747.html > Reviewers: Alexander Scherbatiy, Semyon Sadetsky > From zoltan.majo at oracle.com Fri Oct 23 08:27:08 2015 From: zoltan.majo at oracle.com (=?UTF-8?B?Wm9sdMOhbiBNYWrDsw==?=) Date: Fri, 23 Oct 2015 10:27:08 +0200 Subject: [8u] Request for approval and review: 8080650: Enable stubs to use frame pointers correctly Message-ID: <5629EF5C.9000106@oracle.com> Hi, please approve and review the following backport to 8u. Original (9) bug: https://bugs.openjdk.java.net/browse/JDK-8080650 Original (9) webrev: http://cr.openjdk.java.net/~zmajo/8080650/webrev.01/ Changeset: http://hg.openjdk.java.net/jdk9/hs-comp/hotspot/rev/cf43bef12125 The change cleanly applies to 8u. Nightly testing showed no problems. I tested the patch applied to jdk8u-dev with: - JPRT (testset hotspot), all tests pass; - locally (linux-x64) executed all hotspot tests, all tests pass that pass with an unmodified VM. Thank you and best regards, Zoltan From sean.coffey at oracle.com Fri Oct 23 09:42:43 2015 From: sean.coffey at oracle.com (=?UTF-8?Q?Se=c3=a1n_Coffey?=) Date: Fri, 23 Oct 2015 10:42:43 +0100 Subject: [8u] Request for approval and review: 8080650: Enable stubs to use frame pointers correctly In-Reply-To: <5629EF5C.9000106@oracle.com> References: <5629EF5C.9000106@oracle.com> Message-ID: <562A0113.9070206@oracle.com> Approved for jdk8u-dev. Regards, Sean. On 23/10/15 09:27, Zolt?n Maj? wrote: > Hi, > > > please approve and review the following backport to 8u. > > Original (9) bug: https://bugs.openjdk.java.net/browse/JDK-8080650 > Original (9) webrev: http://cr.openjdk.java.net/~zmajo/8080650/webrev.01/ > Changeset: > http://hg.openjdk.java.net/jdk9/hs-comp/hotspot/rev/cf43bef12125 > > The change cleanly applies to 8u. Nightly testing showed no problems. > > I tested the patch applied to jdk8u-dev with: > - JPRT (testset hotspot), all tests pass; > - locally (linux-x64) executed all hotspot tests, all tests pass that > pass with an unmodified VM. > > Thank you and best regards, > > > Zoltan > From vladimir.kozlov at oracle.com Fri Oct 23 09:44:09 2015 From: vladimir.kozlov at oracle.com (Vladimir Kozlov) Date: Fri, 23 Oct 2015 17:44:09 +0800 Subject: [8u] Request for approval and review: 8080650: Enable stubs to use frame pointers correctly In-Reply-To: <5629EF5C.9000106@oracle.com> References: <5629EF5C.9000106@oracle.com> Message-ID: <562A0169.9020901@oracle.com> Looks good. Thanks, Vladimir On 10/23/15 4:27 PM, Zolt?n Maj? wrote: > Hi, > > > please approve and review the following backport to 8u. > > Original (9) bug: https://bugs.openjdk.java.net/browse/JDK-8080650 > Original (9) webrev: http://cr.openjdk.java.net/~zmajo/8080650/webrev.01/ > Changeset: http://hg.openjdk.java.net/jdk9/hs-comp/hotspot/rev/cf43bef12125 > > The change cleanly applies to 8u. Nightly testing showed no problems. > > I tested the patch applied to jdk8u-dev with: > - JPRT (testset hotspot), all tests pass; > - locally (linux-x64) executed all hotspot tests, all tests pass that > pass with an unmodified VM. > > Thank you and best regards, > > > Zoltan > From zoltan.majo at oracle.com Fri Oct 23 10:15:45 2015 From: zoltan.majo at oracle.com (=?UTF-8?B?Wm9sdMOhbiBNYWrDsw==?=) Date: Fri, 23 Oct 2015 12:15:45 +0200 Subject: [8u] Request for approval and review: 8080650: Enable stubs to use frame pointers correctly In-Reply-To: <562A0169.9020901@oracle.com> References: <5629EF5C.9000106@oracle.com> <562A0169.9020901@oracle.com> Message-ID: <562A08D1.6010409@oracle.com> Thank you, Se?n and Vladimir, for the approval and the review, respectively! Best regards, Zolt?n On 10/23/2015 11:44 AM, Vladimir Kozlov wrote: > Looks good. > > Thanks, > Vladimir > > On 10/23/15 4:27 PM, Zolt?n Maj? wrote: >> Hi, >> >> >> please approve and review the following backport to 8u. >> >> Original (9) bug: https://bugs.openjdk.java.net/browse/JDK-8080650 >> Original (9) webrev: >> http://cr.openjdk.java.net/~zmajo/8080650/webrev.01/ >> Changeset: >> http://hg.openjdk.java.net/jdk9/hs-comp/hotspot/rev/cf43bef12125 >> >> The change cleanly applies to 8u. Nightly testing showed no problems. >> >> I tested the patch applied to jdk8u-dev with: >> - JPRT (testset hotspot), all tests pass; >> - locally (linux-x64) executed all hotspot tests, all tests pass that >> pass with an unmodified VM. >> >> Thank you and best regards, >> >> >> Zoltan >> From andreas.eriksson at oracle.com Tue Oct 27 12:12:56 2015 From: andreas.eriksson at oracle.com (Andreas Eriksson) Date: Tue, 27 Oct 2015 13:12:56 +0100 Subject: [8u-dev] Request for approval for CR JDK-8134759 - jdb: Incorrect stepping inside finally block Message-ID: <562F6A48.8020701@oracle.com> Hi, I'd like approval for a JDK 8 backport of JDK-8134759 - jdb: Incorrect stepping inside finally block. The changeset from JDK 9 applies cleanly after running it through unshuffle_patch.sh. JDK 9 Changeset: http://hg.openjdk.java.net/jdk9/dev/langtools/rev/16873e56156e JDK 9 review thread: http://mail.openjdk.java.net/pipermail/compiler-dev/2015-October/009819.html Regards, Andreas From rob.mckenna at oracle.com Tue Oct 27 12:43:43 2015 From: rob.mckenna at oracle.com (Rob McKenna) Date: Tue, 27 Oct 2015 12:43:43 +0000 Subject: [8u-dev] Request for approval for CR JDK-8134759 - jdb: Incorrect stepping inside finally block In-Reply-To: <562F6A48.8020701@oracle.com> References: <562F6A48.8020701@oracle.com> Message-ID: <562F717F.2020207@oracle.com> Approved. -Rob On 27/10/15 12:12, Andreas Eriksson wrote: > Hi, > > I'd like approval for a JDK 8 backport of JDK-8134759 > - jdb: Incorrect > stepping inside finally block. > The changeset from JDK 9 applies cleanly after running it through > unshuffle_patch.sh. > > JDK 9 Changeset: > http://hg.openjdk.java.net/jdk9/dev/langtools/rev/16873e56156e > JDK 9 review thread: > http://mail.openjdk.java.net/pipermail/compiler-dev/2015-October/009819.html > > > Regards, > Andreas From rob.mckenna at oracle.com Tue Oct 27 17:41:56 2015 From: rob.mckenna at oracle.com (Rob McKenna) Date: Tue, 27 Oct 2015 17:41:56 +0000 Subject: JDK 8u76 timeline Message-ID: <562FB764.7020106@oracle.com> I've published a proposal for the key milestone dates relating to the 8u76 release: http://openjdk.java.net/projects/jdk8u/releases/8u76.html Oct 2015 8u-dev forests begin collecting 8u76 fixes (completed) January 2016 RampDown 2 (Fork for the stabilization forests) April 2016 GA Note that these timelines are preliminary only and are subject to change. Please refer to the JDK 8 milestones definition page [1] for a summary of what each milestone consists of. Please submit feedback by Friday the 30th of October 2015. Thanks, -Rob [1] http://openjdk.java.net/projects/jdk8/milestones From rob.mckenna at oracle.com Wed Oct 28 13:11:27 2015 From: rob.mckenna at oracle.com (Rob McKenna) Date: Wed, 28 Oct 2015 13:11:27 +0000 Subject: [8u-communication] JDK 8u76 timeline In-Reply-To: <562FB764.7020106@oracle.com> References: <562FB764.7020106@oracle.com> Message-ID: <5630C97F.9090708@oracle.com> Adjusting subject line. -Rob On 27/10/15 17:41, Rob McKenna wrote: > I've published a proposal for the key milestone dates relating to the > 8u76 release: > > http://openjdk.java.net/projects/jdk8u/releases/8u76.html > > Oct 2015 8u-dev forests begin collecting 8u76 fixes (completed) > January 2016 RampDown 2 (Fork for the stabilization forests) > April 2016 GA > > Note that these timelines are preliminary only and are subject to > change. Please refer to the JDK 8 milestones definition page [1] for a > summary of what each milestone consists of. Please submit feedback by > Friday the 30th of October 2015. > > Thanks, > > -Rob > > [1] http://openjdk.java.net/projects/jdk8/milestones From sean.coffey at oracle.com Wed Oct 28 17:58:17 2015 From: sean.coffey at oracle.com (=?UTF-8?Q?Se=c3=a1n_Coffey?=) Date: Wed, 28 Oct 2015 17:58:17 +0000 Subject: [8u-dev] Request for review & approval: 8068033 JNI exception pending in jdk/src/share/bin/java.c Message-ID: <56310CB9.8060207@oracle.com> I'd like to backport this fix to jdk8u-dev. bug report : https://bugs.openjdk.java.net/browse/JDK-8068033 webrev : http://cr.openjdk.java.net/~coffeys/webrev.8068033.jdk8u/webrev/ jdk 9 code review : http://mail.openjdk.java.net/pipermail/core-libs-dev/2015-February/031211.html The net code change is identical to that of jdk9 fix but the CHECK_EXCEPTION_RETURN() macro was not present in the jdk8u java.c file and hence didn't need removing. Kumar, can I ask you to confirm that the changes are ok ? JPRT is green. -- Regards, Sean. From michael.haupt at oracle.com Thu Oct 29 10:58:15 2015 From: michael.haupt at oracle.com (Michael Haupt) Date: Thu, 29 Oct 2015 11:58:15 +0100 Subject: RFR(XS): 8139863: [TESTBUG] Need to port tests for JDK-8134903 to 8u-dev Message-ID: <2E8BB4F7-4208-4038-B207-63BE2CADBCDA@oracle.com> Dear all, cross-posted to jdk8u-dev and core-libs-dev. Please review this change. Issue: https://bugs.openjdk.java.net/browse/JDK-8139863 Webrev: http://cr.openjdk.java.net/~mhaupt/8139863/webrev.00 This adds a missing test to 8u-dev that was not pushed along with the feature backport it belongs to. The backport was approved; see http://mail.openjdk.java.net/pipermail/jdk8u-dev/2015-September/004185.html. Thanks, Michael -- Dr. Michael Haupt | Principal Member of Technical Staff Phone: +49 331 200 7277 | Fax: +49 331 200 7561 Oracle Java Platform Group | LangTools Team | Nashorn Oracle Deutschland B.V. & Co. KG, Schiffbauergasse 14 | 14467 Potsdam, Germany Oracle is committed to developing practices and products that help protect the environment From alexey.ivanov at oracle.com Thu Oct 29 15:57:11 2015 From: alexey.ivanov at oracle.com (Alexey Ivanov) Date: Thu, 29 Oct 2015 18:57:11 +0300 Subject: [8u-dev] Request for approval for 8134828: Scrollbar thumb disappears with Nimbus L&F Message-ID: <563241D7.8030607@oracle.com> Hello, Could you please approve the following backport of the fix to 8u-dev? The patch from JDK9 applies cleanly after paths unshuffling. JBS: https://bugs.openjdk.java.net/browse/JDK-8134828 Webrev: http://cr.openjdk.java.net/~ssadetsky/8134828/webrev.00/ Code review: http://mail.openjdk.java.net/pipermail/swing-dev/2015-September/004908.html http://mail.openjdk.java.net/pipermail/swing-dev/2015-October/005072.html JDK9 changeset: http://hg.openjdk.java.net/jdk9/client/jdk/rev/b14984244f11 Thanks, Alexey From sean.coffey at oracle.com Thu Oct 29 16:12:00 2015 From: sean.coffey at oracle.com (=?UTF-8?Q?Se=c3=a1n_Coffey?=) Date: Thu, 29 Oct 2015 16:12:00 +0000 Subject: [8u-dev] Request for approval for 8134828: Scrollbar thumb disappears with Nimbus L&F In-Reply-To: <563241D7.8030607@oracle.com> References: <563241D7.8030607@oracle.com> Message-ID: <56324550.8090608@oracle.com> Approved. Regards, Sean. On 29/10/2015 15:57, Alexey Ivanov wrote: > Hello, > > Could you please approve the following backport of the fix to 8u-dev? > The patch from JDK9 applies cleanly after paths unshuffling. > > JBS: https://bugs.openjdk.java.net/browse/JDK-8134828 > Webrev: http://cr.openjdk.java.net/~ssadetsky/8134828/webrev.00/ > > Code review: > http://mail.openjdk.java.net/pipermail/swing-dev/2015-September/004908.html > > http://mail.openjdk.java.net/pipermail/swing-dev/2015-October/005072.html > > JDK9 changeset: > http://hg.openjdk.java.net/jdk9/client/jdk/rev/b14984244f11 > > > Thanks, > Alexey From ivan.gerasimov at oracle.com Thu Oct 29 19:20:39 2015 From: ivan.gerasimov at oracle.com (Ivan Gerasimov) Date: Thu, 29 Oct 2015 22:20:39 +0300 Subject: [8u-dev] Request for approval to backport: 8139373: [TEST_BUG] java/net/MulticastSocket/MultiDead.java failed with timeout Message-ID: <56327187.2080902@oracle.com> Hi! Would you please approve backporting of this test-stabilization fix to jdk8u-dev? The fix applies cleanly. Bug: https://bugs.openjdk.java.net/browse/JDK-8139373 Jdk9 change: http://hg.openjdk.java.net/jdk9/jdk9/jdk/rev/9a84eb7c34e1 Jdk9 review: http://mail.openjdk.java.net/pipermail/net-dev/2015-October/009285.html Sincerely yours, Ivan From sean.coffey at oracle.com Thu Oct 29 19:23:51 2015 From: sean.coffey at oracle.com (=?UTF-8?Q?Se=c3=a1n_Coffey?=) Date: Thu, 29 Oct 2015 19:23:51 +0000 Subject: [8u-dev] Request for approval to backport: 8139373: [TEST_BUG] java/net/MulticastSocket/MultiDead.java failed with timeout In-Reply-To: <56327187.2080902@oracle.com> References: <56327187.2080902@oracle.com> Message-ID: <56327247.7040701@oracle.com> Approved. Regards, Sean. On 29/10/2015 19:20, Ivan Gerasimov wrote: > Hi! > > Would you please approve backporting of this test-stabilization fix to > jdk8u-dev? > The fix applies cleanly. > > Bug: https://bugs.openjdk.java.net/browse/JDK-8139373 > Jdk9 change: http://hg.openjdk.java.net/jdk9/jdk9/jdk/rev/9a84eb7c34e1 > Jdk9 review: > http://mail.openjdk.java.net/pipermail/net-dev/2015-October/009285.html > > Sincerely yours, > Ivan > From kumar.x.srinivasan at oracle.com Thu Oct 29 21:34:42 2015 From: kumar.x.srinivasan at oracle.com (Kumar Srinivasan) Date: Thu, 29 Oct 2015 14:34:42 -0700 Subject: [8u-dev] Request for review & approval: 8068033 JNI exception pending in jdk/src/share/bin/java.c In-Reply-To: <56310CB9.8060207@oracle.com> References: <56310CB9.8060207@oracle.com> Message-ID: <563290F2.1050703@oracle.com> Hi Sean, Approved. Thanks Kumar On 10/28/2015 10:58 AM, Se?n Coffey wrote: > I'd like to backport this fix to jdk8u-dev. > > bug report : https://bugs.openjdk.java.net/browse/JDK-8068033 > webrev : http://cr.openjdk.java.net/~coffeys/webrev.8068033.jdk8u/webrev/ > jdk 9 code review : > http://mail.openjdk.java.net/pipermail/core-libs-dev/2015-February/031211.html > > The net code change is identical to that of jdk9 fix but the > CHECK_EXCEPTION_RETURN() macro was not present in the jdk8u java.c > file and hence didn't need removing. > > Kumar, can I ask you to confirm that the changes are ok ? JPRT is green. > -- > Regards, > Sean. From rob.mckenna at oracle.com Thu Oct 29 23:13:25 2015 From: rob.mckenna at oracle.com (Rob McKenna) Date: Thu, 29 Oct 2015 23:13:25 +0000 Subject: [8u-dev] Request for review & approval: 8068033 JNI exception pending in jdk/src/share/bin/java.c In-Reply-To: <563290F2.1050703@oracle.com> References: <56310CB9.8060207@oracle.com> <563290F2.1050703@oracle.com> Message-ID: <5632A815.7030508@oracle.com> Approved. -Rob On 29/10/15 21:34, Kumar Srinivasan wrote: > Hi Sean, > > Approved. > > Thanks > > Kumar > > On 10/28/2015 10:58 AM, Se?n Coffey wrote: >> I'd like to backport this fix to jdk8u-dev. >> >> bug report : https://bugs.openjdk.java.net/browse/JDK-8068033 >> webrev : http://cr.openjdk.java.net/~coffeys/webrev.8068033.jdk8u/webrev/ >> jdk 9 code review : >> http://mail.openjdk.java.net/pipermail/core-libs-dev/2015-February/031211.html >> >> >> The net code change is identical to that of jdk9 fix but the >> CHECK_EXCEPTION_RETURN() macro was not present in the jdk8u java.c >> file and hence didn't need removing. >> >> Kumar, can I ask you to confirm that the changes are ok ? JPRT is green. >> -- >> Regards, >> Sean. > From michael.haupt at oracle.com Fri Oct 30 07:12:50 2015 From: michael.haupt at oracle.com (Michael Haupt) Date: Fri, 30 Oct 2015 08:12:50 +0100 Subject: [8u-dev] RFR(XS): 8139863: [TESTBUG] Need to port tests for JDK-8134903 to 8u-dev In-Reply-To: <2E8BB4F7-4208-4038-B207-63BE2CADBCDA@oracle.com> References: <2E8BB4F7-4208-4038-B207-63BE2CADBCDA@oracle.com> Message-ID: <7DA08B0B-7532-4695-B89B-B4FC75CC057E@oracle.com> (reposted with corrected subject line) > Am 29.10.2015 um 11:58 schrieb Michael Haupt : > > Dear all, > > cross-posted to jdk8u-dev and core-libs-dev. > > Please review this change. > Issue: https://bugs.openjdk.java.net/browse/JDK-8139863 > Webrev: http://cr.openjdk.java.net/~mhaupt/8139863/webrev.00 > > This adds a missing test to 8u-dev that was not pushed along with the feature backport it belongs to. The backport was approved; see http://mail.openjdk.java.net/pipermail/jdk8u-dev/2015-September/004185.html. > > Thanks, > > Michael > -- Dr. Michael Haupt | Principal Member of Technical Staff Phone: +49 331 200 7277 | Fax: +49 331 200 7561 Oracle Java Platform Group | LangTools Team | Nashorn Oracle Deutschland B.V. & Co. KG, Schiffbauergasse 14 | 14467 Potsdam, Germany Oracle is committed to developing practices and products that help protect the environment From anton.litvinov at oracle.com Fri Oct 30 11:08:09 2015 From: anton.litvinov at oracle.com (Anton Litvinov) Date: Fri, 30 Oct 2015 14:08:09 +0300 Subject: [8u-dev] Request for approval for CR 8132503: [macosx] Chinese full stop symbol cannot be entered with Pinyin IM on OS X Message-ID: <56334F99.7070500@oracle.com> Hello, I would like to request for approval to push a backport of the fix from JDK 9 to JDK 8. The backport fix is identical to the original fix. Bug: https://bugs.openjdk.java.net/browse/JDK-8132503 JDK 9 changeset: http://hg.openjdk.java.net/jdk9/client/jdk/rev/1a8daf81fc7a JDK 9 review thread: Approval 1 - http://mail.openjdk.java.net/pipermail/awt-dev/2015-October/010248.html Approval 2 - http://mail.openjdk.java.net/pipermail/awt-dev/2015-October/010251.html Reviewers: alexsch, kizune The patch from JDK 9 applies cleanly after correction of a file path. Thank you, Anton From david.buck at oracle.com Fri Oct 30 11:19:37 2015 From: david.buck at oracle.com (david buck) Date: Fri, 30 Oct 2015 04:19:37 -0700 Subject: [8u-dev] Request for approval for CR 8132503: [macosx] Chinese full stop symbol cannot be entered with Pinyin IM on OS X In-Reply-To: <56334F99.7070500@oracle.com> References: <56334F99.7070500@oracle.com> Message-ID: <56335249.7090605@oracle.com> approved for backport to 8u-dev Cheers, -Buck On 2015/10/30 4:08, Anton Litvinov wrote: > Hello, > > I would like to request for approval to push a backport of the fix from > JDK 9 to JDK 8. The backport fix is identical to the original fix. > > Bug: https://bugs.openjdk.java.net/browse/JDK-8132503 > JDK 9 changeset: > http://hg.openjdk.java.net/jdk9/client/jdk/rev/1a8daf81fc7a > JDK 9 review thread: > Approval 1 - > http://mail.openjdk.java.net/pipermail/awt-dev/2015-October/010248.html > Approval 2 - > http://mail.openjdk.java.net/pipermail/awt-dev/2015-October/010251.html > Reviewers: alexsch, kizune > > The patch from JDK 9 applies cleanly after correction of a file path. > > Thank you, > Anton From vincent at josm.openstreetmap.de Sat Oct 31 14:14:00 2015 From: vincent at josm.openstreetmap.de (Vincent Privat) Date: Sat, 31 Oct 2015 15:14:00 +0100 Subject: [8u72] Backport of JDK-8058175: XML - Attribute values with supplemental characters are being corrupted Message-ID: Hi, Could we please get JDK-8058175 backported to 8u72? There is no known workaround and it's a major problem for JOSM, see JDK-8141097. By the way, is it possible to add label "josm-found" to JDK-8141097 and resolve it as a duplicate of JDK-8058175? We have verified that our original problem is solved in JDK9 thanks to this correction. The fix appears to be quite simple, see http://hg.openjdk.java.net/jdk9/dev/jaxp/rev/705a3999eb77 Best regards, Vincent