From j.bachorik at gmail.com Mon Oct 3 14:06:07 2016 From: j.bachorik at gmail.com (Jaroslav Bachorik) Date: Mon, 3 Oct 2016 16:06:07 +0200 Subject: [jdk8u-dev] Request for Review and Approval : 8139870:sun.management.LazyCompositeData.isTypeMatched() fails for composite types with items of ArrayType Message-ID: Hi Sean, >Looking to backport this to jdk8u-dev. > >Bug report : https://bugs.openjdk.java.net/browse/JDK-8139870 >jdk8u-dev webrev : >http://cr.openjdk.java.net/~coffeys/webrev.8139870.jdk8u/webrev/ > >Some minor tweaks to the src code were necessary due to method access >method modifier changes introduced for modules in JDK 9. (getBoolean, >getLong getInt). The change looks good! Thanks for taking care of the backport! -JB- > >regards, >Sean. From rob.mckenna at oracle.com Mon Oct 3 17:53:31 2016 From: rob.mckenna at oracle.com (Rob McKenna) Date: Mon, 3 Oct 2016 18:53:31 +0100 Subject: [jdk8u-dev] Request for Review and Approval : 8139870:sun.management.LazyCompositeData.isTypeMatched() fails for composite types with items of ArrayType In-Reply-To: References: Message-ID: <20161003175331.GD3328@vimes> Approved -Rob On 03/10/16 04:06, Jaroslav Bachorik wrote: > Hi Sean, > > >Looking to backport this to jdk8u-dev. > > > >Bug report : https://bugs.openjdk.java.net/browse/JDK-8139870 > >jdk8u-dev webrev : > >http://cr.openjdk.java.net/~coffeys/webrev.8139870.jdk8u/webrev/ > > > >Some minor tweaks to the src code were necessary due to method access > >method modifier changes introduced for modules in JDK 9. (getBoolean, > >getLong getInt). > > The change looks good! Thanks for taking care of the backport! > > -JB- > > > > >regards, > >Sean. From shafi.s.ahmad at oracle.com Wed Oct 5 08:48:37 2016 From: shafi.s.ahmad at oracle.com (Shafi Ahmad) Date: Wed, 5 Oct 2016 01:48:37 -0700 (PDT) Subject: [8u] RFA for JDK-8166872: GPL header in /hotspot/src/share/vm/gc_implementation/g1/g1RemSetSummary.cpp Message-ID: <92723ada-bbbb-42ec-8e01-42eff871856e@default> Hi, Please approve the code change for 'JDK-8166872: GPL header in /hotspot/src/share/vm/gc_implementation/g1/g1RemSetSummary.cpp' for jdk8u. Review link: http://mail.openjdk.java.net/pipermail/hotspot-dev/2016-October/024755.html Open webrev: http://cr.openjdk.java.net/~shshahma/8166872/webrev.00/ Bug link: https://bugs.openjdk.java.net/browse/JDK-8166872 Regards, Shafi From rob.mckenna at oracle.com Wed Oct 5 13:49:50 2016 From: rob.mckenna at oracle.com (Rob McKenna) Date: Wed, 5 Oct 2016 14:49:50 +0100 Subject: [8u] RFA for JDK-8166872: GPL header in /hotspot/src/share/vm/gc_implementation/g1/g1RemSetSummary.cpp In-Reply-To: <92723ada-bbbb-42ec-8e01-42eff871856e@default> References: <92723ada-bbbb-42ec-8e01-42eff871856e@default> Message-ID: <20161005134950.GA2869@vimes> Approved -Rob On 05/10/16 01:48, Shafi Ahmad wrote: > Hi, > > Please approve the code change for 'JDK-8166872: GPL header in /hotspot/src/share/vm/gc_implementation/g1/g1RemSetSummary.cpp' for jdk8u. > > Review link: http://mail.openjdk.java.net/pipermail/hotspot-dev/2016-October/024755.html > Open webrev: http://cr.openjdk.java.net/~shshahma/8166872/webrev.00/ > Bug link: https://bugs.openjdk.java.net/browse/JDK-8166872 > > Regards, > Shafi From alexandr.scherbatiy at oracle.com Wed Oct 5 15:07:01 2016 From: alexandr.scherbatiy at oracle.com (Alexander Scherbatiy) Date: Wed, 5 Oct 2016 19:07:01 +0400 Subject: [8u-dev] Request for approval: 8166591 [macos 10.12] Trackpad scrolling of text on OS X 10.12 Sierra is very fast (Trackpad, Retina only) Message-ID: Hello, Could you approve the backport of the fix to JDK 8u-dev. There are some style difference between JDK 8u-dev and JDK 9 changes so it is not the direct back port. The updated webrev resolves this. It does not changes the fix logic so the logic is just the same. The bug: https://bugs.openjdk.java.net/browse/JDK-8166591 The JDK 8u-dev webrev: http://cr.openjdk.java.net/~alexsch/8166591/webrev-jdk8.00 The JDK 9 webrev: http://cr.openjdk.java.net/~alexsch/8166591/webrev.09 The review thread: http://mail.openjdk.java.net/pipermail/awt-dev/2016-October/012086.html The JDK 9 changeset: http://hg.openjdk.java.net/jdk9/client/jdk/rev/ff61a6fd0349 Thanks, Alexandr. From ajit.ghaisas at oracle.com Wed Oct 5 16:01:15 2016 From: ajit.ghaisas at oracle.com (Ajit Ghaisas) Date: Wed, 5 Oct 2016 09:01:15 -0700 (PDT) Subject: FW: [OpenJDK 2D-Dev] [8u Backport] Fix for JDK-8158356 : SIGSEGV when attempting to rotate BufferedImage using AffineTransform by NaN degrees In-Reply-To: References: <0316a26c-b41b-4323-8558-f7855358b238@default> <7b0b187d-979d-4ad9-bc64-e5480f57ed37@default> <57EBFB2B.8030305@oracle.com> Message-ID: <5e00bb26-c775-4de2-8b80-df4cb4e4a6ea@default> Hi, Please approve this 8u backport request : Changes are same as done for JDK-9. Bug : https://bugs.openjdk.java.net/browse/JDK-8158356 JDK 8u Webrev : http://cr.openjdk.java.net/~aghaisas/8158356/8u_backport/webrev.00/ JDK8 backport review thread : http://mail.openjdk.java.net/pipermail/2d-dev/2016-September/007725.html JDK 9 review thread : http://mail.openjdk.java.net/pipermail/2d-dev/2016-August/007536.html JDK 9 changeset : http://hg.openjdk.java.net/jdk9/client/jdk/rev/ca2ebc3de28c Regards, Ajit -----Original Message----- From: Jim Graham Sent: Friday, September 30, 2016 2:58 AM To: Philip Race; Ajit Ghaisas Cc: 2d-dev Subject: Re: [OpenJDK 2D-Dev] [8u Backport] Fix for JDK-8158356 : SIGSEGV when attempting to rotate BufferedImage using AffineTransform by NaN degrees +1 ...jim On 9/28/16 10:17 AM, Philip Race wrote: > +1 > > -phil > > On 9/14/16, 2:52 AM, Ajit Ghaisas wrote: >> Hi Jim, >> >> I agree upon aborting the operation in case the non-finite slope is not due to divide by 0 case. >> As this is the 8u backport of the fix done in 9, I will address the review comment in JDK-8166009. >> >> The original backport webrev is still the same. >> >> http://cr.openjdk.java.net/~aghaisas/8158356/8u_backport/webrev.00/ >> >> Request you to approve this 8u backport. >> >> Regards, >> Ajit >> >> -----Original Message----- >> From: Jim Graham >> Sent: Friday, September 02, 2016 1:16 AM >> To: Ajit Ghaisas; Philip Race; 2d-dev >> Subject: Re: [8u Backport] Fix for JDK-8158356 : SIGSEGV when >> attempting to rotate BufferedImage using AffineTransform by NaN >> degrees >> >> Hi Ajit, >> >> In the cases where you "continue" on a non-finite slope, doesn't that >> mean that the edges will be mismatched? If you can't determine the bounding polygon, perhaps the entire operation should be aborted instead...? >> >> It's different from the case of dy1==dy2 which also results in a >> "continue" because in that case there are no edges to insert in the >> array because that edge of the bounding box doesn't occupy any >> vertical space. In the case of an infinite slope, there could be edges expected to be produced (in fact there should be because we've already tested that dy1 != dy2), but the loop doesn't produce any edge values and stale data is left in the array. >> >> ...jim >> >> On 9/1/16 2:14 AM, Ajit Ghaisas wrote: >>> Hi, >>> >>> >>> >>> This is a review request for 8u back-port of the fix done in JDK-9. >>> >>> >>> >>> Bug : >>> >>> https://bugs.openjdk.java.net/browse/JDK-8158356 >>> >>> >>> >>> Root cause : >>> >>> The root cause of the crash is - NaN is converted to an integer and used as array index in mlib_ImageScanPoly.c. >>> >>> >>> >>> Fix : >>> >>> The native method previously did not check the validity of the >>> input double argument. Now, I have added a check for finite double values. >>> >>> >>> >>> Webrev : >>> >>> >>> http://cr.openjdk.java.net/~aghaisas/8158356/8u_backport/webrev.00/ >>> >>> >>> >>> Regards, >>> >>> Ajit >>> From rob.mckenna at oracle.com Wed Oct 5 16:04:01 2016 From: rob.mckenna at oracle.com (Rob McKenna) Date: Wed, 5 Oct 2016 17:04:01 +0100 Subject: [8u-dev] Request for approval: 8166591 [macos 10.12] Trackpad scrolling of text on OS X 10.12 Sierra is very fast (Trackpad, Retina only) In-Reply-To: References: Message-ID: <20161005160401.GC2869@vimes> Hi Alexandr, If the fix has changed, you will need to request a codereview from the appropriate alias. -Rob On 05/10/16 07:07, Alexander Scherbatiy wrote: > > Hello, > > Could you approve the backport of the fix to JDK 8u-dev. > There are some style difference between JDK 8u-dev and JDK 9 changes so it > is not the direct back port. The updated webrev resolves this. It does not > changes the fix logic so the logic is just the same. > > The bug: https://bugs.openjdk.java.net/browse/JDK-8166591 > The JDK 8u-dev webrev: > http://cr.openjdk.java.net/~alexsch/8166591/webrev-jdk8.00 > The JDK 9 webrev: http://cr.openjdk.java.net/~alexsch/8166591/webrev.09 > The review thread: > http://mail.openjdk.java.net/pipermail/awt-dev/2016-October/012086.html > The JDK 9 changeset: > http://hg.openjdk.java.net/jdk9/client/jdk/rev/ff61a6fd0349 > > Thanks, > Alexandr. From rob.mckenna at oracle.com Wed Oct 5 16:20:07 2016 From: rob.mckenna at oracle.com (Rob McKenna) Date: Wed, 5 Oct 2016 17:20:07 +0100 Subject: FW: [OpenJDK 2D-Dev] [8u Backport] Fix for JDK-8158356 : SIGSEGV when attempting to rotate BufferedImage using AffineTransform by NaN degrees In-Reply-To: <5e00bb26-c775-4de2-8b80-df4cb4e4a6ea@default> References: <0316a26c-b41b-4323-8558-f7855358b238@default> <7b0b187d-979d-4ad9-bc64-e5480f57ed37@default> <57EBFB2B.8030305@oracle.com> <5e00bb26-c775-4de2-8b80-df4cb4e4a6ea@default> Message-ID: <20161005162007.GE2869@vimes> Approved. Please follow the push approval request template for your subject line for future requests: http://openjdk.java.net/projects/jdk8u/approval-template.html -Rob On 05/10/16 09:01, Ajit Ghaisas wrote: > Hi, > > Please approve this 8u backport request : > Changes are same as done for JDK-9. > > Bug : https://bugs.openjdk.java.net/browse/JDK-8158356 > > JDK 8u Webrev : http://cr.openjdk.java.net/~aghaisas/8158356/8u_backport/webrev.00/ > > JDK8 backport review thread : http://mail.openjdk.java.net/pipermail/2d-dev/2016-September/007725.html > > > JDK 9 review thread : http://mail.openjdk.java.net/pipermail/2d-dev/2016-August/007536.html > > JDK 9 changeset : http://hg.openjdk.java.net/jdk9/client/jdk/rev/ca2ebc3de28c > > Regards, > Ajit > > -----Original Message----- > From: Jim Graham > Sent: Friday, September 30, 2016 2:58 AM > To: Philip Race; Ajit Ghaisas > Cc: 2d-dev > Subject: Re: [OpenJDK 2D-Dev] [8u Backport] Fix for JDK-8158356 : SIGSEGV when attempting to rotate BufferedImage using AffineTransform by NaN degrees > > +1 > > ...jim > > On 9/28/16 10:17 AM, Philip Race wrote: > > +1 > > > > -phil > > > > On 9/14/16, 2:52 AM, Ajit Ghaisas wrote: > >> Hi Jim, > >> > >> I agree upon aborting the operation in case the non-finite slope is not due to divide by 0 case. > >> As this is the 8u backport of the fix done in 9, I will address the review comment in JDK-8166009. > >> > >> The original backport webrev is still the same. > >> > >> http://cr.openjdk.java.net/~aghaisas/8158356/8u_backport/webrev.00/ > >> > >> Request you to approve this 8u backport. > >> > >> Regards, > >> Ajit > >> > >> -----Original Message----- > >> From: Jim Graham > >> Sent: Friday, September 02, 2016 1:16 AM > >> To: Ajit Ghaisas; Philip Race; 2d-dev > >> Subject: Re: [8u Backport] Fix for JDK-8158356 : SIGSEGV when > >> attempting to rotate BufferedImage using AffineTransform by NaN > >> degrees > >> > >> Hi Ajit, > >> > >> In the cases where you "continue" on a non-finite slope, doesn't that > >> mean that the edges will be mismatched? If you can't determine the bounding polygon, perhaps the entire operation should be aborted instead...? > >> > >> It's different from the case of dy1==dy2 which also results in a > >> "continue" because in that case there are no edges to insert in the > >> array because that edge of the bounding box doesn't occupy any > >> vertical space. In the case of an infinite slope, there could be edges expected to be produced (in fact there should be because we've already tested that dy1 != dy2), but the loop doesn't produce any edge values and stale data is left in the array. > >> > >> ...jim > >> > >> On 9/1/16 2:14 AM, Ajit Ghaisas wrote: > >>> Hi, > >>> > >>> > >>> > >>> This is a review request for 8u back-port of the fix done in JDK-9. > >>> > >>> > >>> > >>> Bug : > >>> > >>> https://bugs.openjdk.java.net/browse/JDK-8158356 > >>> > >>> > >>> > >>> Root cause : > >>> > >>> The root cause of the crash is - NaN is converted to an integer and used as array index in mlib_ImageScanPoly.c. > >>> > >>> > >>> > >>> Fix : > >>> > >>> The native method previously did not check the validity of the > >>> input double argument. Now, I have added a check for finite double values. > >>> > >>> > >>> > >>> Webrev : > >>> > >>> > >>> http://cr.openjdk.java.net/~aghaisas/8158356/8u_backport/webrev.00/ > >>> > >>> > >>> > >>> Regards, > >>> > >>> Ajit > >>> From jayathirth.d.v at oracle.com Thu Oct 6 11:03:43 2016 From: jayathirth.d.v at oracle.com (Jayathirth D V) Date: Thu, 6 Oct 2016 04:03:43 -0700 (PDT) Subject: [8u]Request for approval to backport: 8162461 & 8166685 Message-ID: <92dd5ec4-53ba-4efb-86bd-84026cd7eed6@default> Hi, Please approve this 8u backport request : the fix is combination of two fixes in JDK 9(JDK-8162461 & JDK-8166685). Bug : https://bugs.openjdk.java.net/browse/JDK-8162461 & https://bugs.openjdk.java.net/browse/JDK-8166685 JDK 8u Webrev : http://cr.openjdk.java.net/~jdv/8162461.8u/webrev.01/ JDK8 backport review thread : http://mail.openjdk.java.net/pipermail/2d-dev/2016-October/007761.html JDK 9 review thread : http://mail.openjdk.java.net/pipermail/2d-dev/2016-September/007601.html & http://mail.openjdk.java.net/pipermail/2d-dev/2016-September/007695.html JDK 9 changeset : http://hg.openjdk.java.net/jdk9/client/jdk/rev/ddfa2c1cdeb5 & http://hg.openjdk.java.net/jdk9/client/jdk/rev/f46ed6f7c785 Thanks, Jay From ramanand.patil at oracle.com Thu Oct 6 11:52:14 2016 From: ramanand.patil at oracle.com (Ramanand Patil) Date: Thu, 6 Oct 2016 04:52:14 -0700 (PDT) Subject: [8u-dev] Request for Approval: Backport of 8166875: (tz) Support tzdata2016g Message-ID: <4bc2f0ee-86d7-4e7b-97e8-b70490c6ccab@default> Hi, Please review and approve the backport of 8166875 to 8u-dev. Bug: https://bugs.openjdk.java.net/browse/JDK-8166875 JDK9 Changeset: http://hg.openjdk.java.net/jdk9/dev/jdk/rev/9caaac90a127 JDK9 Review Thread: http://mail.openjdk.java.net/pipermail/i18n-dev/2016-October/002144.html JDK8u-dev Webrev for reference: http://cr.openjdk.java.net/~rpatil/8167255/webrev.00/ Changes apply cleanly to jdk8u-dev after path reshuffling and after ignoring one test case (test/java/util/TimeZone/TimeZoneTest.java) which is not present in jdk8u-dev repo. Regards, Ramanand. From rob.mckenna at oracle.com Thu Oct 6 13:12:47 2016 From: rob.mckenna at oracle.com (Rob McKenna) Date: Thu, 6 Oct 2016 14:12:47 +0100 Subject: [8u-dev] Request for Approval: Backport of 8166875: (tz) Support tzdata2016g In-Reply-To: <4bc2f0ee-86d7-4e7b-97e8-b70490c6ccab@default> References: <4bc2f0ee-86d7-4e7b-97e8-b70490c6ccab@default> Message-ID: <20161006131247.GD2793@vimes> Approved -Rob On 06/10/16 04:52, Ramanand Patil wrote: > Hi, > Please review and approve the backport of 8166875 to 8u-dev. > Bug: https://bugs.openjdk.java.net/browse/JDK-8166875 > JDK9 Changeset: http://hg.openjdk.java.net/jdk9/dev/jdk/rev/9caaac90a127 > JDK9 Review Thread: http://mail.openjdk.java.net/pipermail/i18n-dev/2016-October/002144.html > JDK8u-dev Webrev for reference: http://cr.openjdk.java.net/~rpatil/8167255/webrev.00/ > > Changes apply cleanly to jdk8u-dev after path reshuffling and after ignoring one test case (test/java/util/TimeZone/TimeZoneTest.java) which is not present in jdk8u-dev repo. > > > Regards, > Ramanand. From rob.mckenna at oracle.com Thu Oct 6 13:13:19 2016 From: rob.mckenna at oracle.com (Rob McKenna) Date: Thu, 6 Oct 2016 14:13:19 +0100 Subject: [8u]Request for approval to backport: 8162461 & 8166685 In-Reply-To: <92dd5ec4-53ba-4efb-86bd-84026cd7eed6@default> References: <92dd5ec4-53ba-4efb-86bd-84026cd7eed6@default> Message-ID: <20161006131319.GE2793@vimes> Approved -Rob On 06/10/16 04:03, Jayathirth D V wrote: > Hi, > > > > Please approve this 8u backport request : the fix is combination of two fixes in JDK 9(JDK-8162461 & JDK-8166685). > > Bug : https://bugs.openjdk.java.net/browse/JDK-8162461 & https://bugs.openjdk.java.net/browse/JDK-8166685 > > JDK 8u Webrev : http://cr.openjdk.java.net/~jdv/8162461.8u/webrev.01/ > > JDK8 backport review thread : http://mail.openjdk.java.net/pipermail/2d-dev/2016-October/007761.html > > JDK 9 review thread : http://mail.openjdk.java.net/pipermail/2d-dev/2016-September/007601.html & http://mail.openjdk.java.net/pipermail/2d-dev/2016-September/007695.html > > JDK 9 changeset : http://hg.openjdk.java.net/jdk9/client/jdk/rev/ddfa2c1cdeb5 & http://hg.openjdk.java.net/jdk9/client/jdk/rev/f46ed6f7c785 > > > > Thanks, > > Jay > > From alexandr.scherbatiy at oracle.com Fri Oct 7 06:52:24 2016 From: alexandr.scherbatiy at oracle.com (Alexandr Scherbatiy) Date: Fri, 7 Oct 2016 09:52:24 +0300 Subject: [8u-dev] Request for approval: 8166591 [macos 10.12] Trackpad scrolling of text on OS X 10.12 Sierra is very fast (Trackpad, Retina only) In-Reply-To: <20161005160401.GC2869@vimes> References: <20161005160401.GC2869@vimes> Message-ID: On 10/5/2016 7:04 PM, Rob McKenna wrote: > Hi Alexandr, > > If the fix has changed, you will need to request a codereview from the appropriate alias. The backport has been approved on the awt-dev alias: [1] [2]. [1] http://mail.openjdk.java.net/pipermail/awt-dev/2016-October/012105.html [2] http://mail.openjdk.java.net/pipermail/awt-dev/2016-October/012110.html Thanks, Alexandr. > > -Rob > > On 05/10/16 07:07, Alexander Scherbatiy wrote: >> Hello, >> >> Could you approve the backport of the fix to JDK 8u-dev. >> There are some style difference between JDK 8u-dev and JDK 9 changes so it >> is not the direct back port. The updated webrev resolves this. It does not >> changes the fix logic so the logic is just the same. >> >> The bug: https://bugs.openjdk.java.net/browse/JDK-8166591 >> The JDK 8u-dev webrev: >> http://cr.openjdk.java.net/~alexsch/8166591/webrev-jdk8.00 >> The JDK 9 webrev: http://cr.openjdk.java.net/~alexsch/8166591/webrev.09 >> The review thread: >> http://mail.openjdk.java.net/pipermail/awt-dev/2016-October/012086.html >> The JDK 9 changeset: >> http://hg.openjdk.java.net/jdk9/client/jdk/rev/ff61a6fd0349 >> >> Thanks, >> Alexandr. From rob.mckenna at oracle.com Fri Oct 7 11:30:32 2016 From: rob.mckenna at oracle.com (Rob McKenna) Date: Fri, 7 Oct 2016 12:30:32 +0100 Subject: [8u-dev] Request for approval: 8166591 [macos 10.12] Trackpad scrolling of text on OS X 10.12 Sierra is very fast (Trackpad, Retina only) In-Reply-To: References: <20161005160401.GC2869@vimes> Message-ID: <20161007113032.GA2557@vimes> Approved -Rob On 07/10/16 09:52, Alexandr Scherbatiy wrote: > On 10/5/2016 7:04 PM, Rob McKenna wrote: > >Hi Alexandr, > > > >If the fix has changed, you will need to request a codereview from the appropriate alias. > The backport has been approved on the awt-dev alias: [1] [2]. > > [1] > http://mail.openjdk.java.net/pipermail/awt-dev/2016-October/012105.html > [2] > http://mail.openjdk.java.net/pipermail/awt-dev/2016-October/012110.html > > Thanks, > Alexandr. > > > > > -Rob > > > >On 05/10/16 07:07, Alexander Scherbatiy wrote: > >>Hello, > >> > >>Could you approve the backport of the fix to JDK 8u-dev. > >>There are some style difference between JDK 8u-dev and JDK 9 changes so it > >>is not the direct back port. The updated webrev resolves this. It does not > >>changes the fix logic so the logic is just the same. > >> > >>The bug: https://bugs.openjdk.java.net/browse/JDK-8166591 > >>The JDK 8u-dev webrev: > >>http://cr.openjdk.java.net/~alexsch/8166591/webrev-jdk8.00 > >>The JDK 9 webrev: http://cr.openjdk.java.net/~alexsch/8166591/webrev.09 > >>The review thread: > >>http://mail.openjdk.java.net/pipermail/awt-dev/2016-October/012086.html > >>The JDK 9 changeset: > >>http://hg.openjdk.java.net/jdk9/client/jdk/rev/ff61a6fd0349 > >> > >>Thanks, > >>Alexandr. > From coleen.phillimore at oracle.com Fri Oct 7 20:01:59 2016 From: coleen.phillimore at oracle.com (Coleen Phillimore) Date: Fri, 7 Oct 2016 16:01:59 -0400 Subject: [8u-dev] Request for approval: 8162766: Unsafe_DefineClass0 accesses raw oops while in _thread_in_native Message-ID: Summary: Go to thread_in_vm to get the class_loader oop. This error was found by Leela Venati. Tested with jtreg hotspot/test. open webrev at http://cr.openjdk.java.net/~coleenp/8162766.01/webrev bug link https://bugs.openjdk.java.net/browse/JDK-8162766 Thanks, Coleen From coleen.phillimore at oracle.com Fri Oct 7 20:04:20 2016 From: coleen.phillimore at oracle.com (Coleen Phillimore) Date: Fri, 7 Oct 2016 16:04:20 -0400 Subject: [8u-dev] Request for approval: 8162766: Unsafe_DefineClass0 accesses raw oops while in _thread_in_native In-Reply-To: References: Message-ID: <6cc75d31-d334-ae1b-685a-327de80a3d72@oracle.com> I should point out that this bug wasn't in jdk9. On 10/7/16 4:01 PM, Coleen Phillimore wrote: > Summary: Go to thread_in_vm to get the class_loader oop. > > This error was found by Leela Venati. Tested with jtreg hotspot/test. > > open webrev at http://cr.openjdk.java.net/~coleenp/8162766.01/webrev > bug link https://bugs.openjdk.java.net/browse/JDK-8162766 > > Thanks, > Coleen From coleen.phillimore at oracle.com Fri Oct 7 20:45:48 2016 From: coleen.phillimore at oracle.com (Coleen Phillimore) Date: Fri, 7 Oct 2016 16:45:48 -0400 Subject: [8u-dev] Request for approval: 8162766: Unsafe_DefineClass0 accesses raw oops while in _thread_in_native In-Reply-To: References: Message-ID: <6799aeef-212d-13ca-8ad4-a8c95001602e@oracle.com> Yes, we can use JVM_GetClassLoader. I thought I'd deleted that function but it must have been deleted in 9. Coleen On 10/7/16 4:19 PM, Leela Mohan wrote: > Hi Coleen, > > Sorry i never came back after we found the issue. We are certainly > doing many unnecessary transitions from native to vm. It seems like, > we can certainly avoid them if we have UNSAFE_LEAF (equivalent to > JVM_LEAF) . But that might be larger change. > > Coming back to your change, I am wondering if we can use > "JVM_GetClassLoader" instead. > > Thanks, > Leela > > On Fri, Oct 7, 2016 at 1:01 PM, Coleen Phillimore > > > wrote: > > Summary: Go to thread_in_vm to get the class_loader oop. > > This error was found by Leela Venati. Tested with jtreg hotspot/test. > > open webrev at > http://cr.openjdk.java.net/~coleenp/8162766.01/webrev > > bug link https://bugs.openjdk.java.net/browse/JDK-8162766 > > > Thanks, > Coleen > > From coleen.phillimore at oracle.com Fri Oct 7 21:20:01 2016 From: coleen.phillimore at oracle.com (Coleen Phillimore) Date: Fri, 7 Oct 2016 17:20:01 -0400 Subject: [8u-dev] Request for approval: 8162766: Unsafe_DefineClass0 accesses raw oops while in _thread_in_native In-Reply-To: References: Message-ID: <8a836f20-c732-7fcc-a404-7b1a83b014a8@oracle.com> Here's the (retested) fix using JVM_GetClassLoader. open webrev at http://cr.openjdk.java.net/~coleenp/8162766.02/webrev From the history it seems that this part was backported with the Class.getClassLoader() speedup change, and the code was corrected in jdk9. Thanks, Coleen On 10/7/16 4:19 PM, Leela Mohan wrote: > Hi Coleen, > > Sorry i never came back after we found the issue. We are certainly > doing many unnecessary transitions from native to vm. It seems like, > we can certainly avoid them if we have UNSAFE_LEAF (equivalent to > JVM_LEAF) . But that might be larger change. > > Coming back to your change, I am wondering if we can use > "JVM_GetClassLoader" instead. > > Thanks, > Leela > > On Fri, Oct 7, 2016 at 1:01 PM, Coleen Phillimore > > > wrote: > > Summary: Go to thread_in_vm to get the class_loader oop. > > This error was found by Leela Venati. Tested with jtreg hotspot/test. > > open webrev at > http://cr.openjdk.java.net/~coleenp/8162766.01/webrev > > bug link https://bugs.openjdk.java.net/browse/JDK-8162766 > > > Thanks, > Coleen > > From coleen.phillimore at oracle.com Fri Oct 7 21:28:32 2016 From: coleen.phillimore at oracle.com (Coleen Phillimore) Date: Fri, 7 Oct 2016 17:28:32 -0400 Subject: [8u-dev] Request for approval: 8162766: Unsafe_DefineClass0 accesses raw oops while in _thread_in_native In-Reply-To: References: <8a836f20-c732-7fcc-a404-7b1a83b014a8@oracle.com> Message-ID: <23b4c462-be54-2c6d-be5e-2050cf000d4f@oracle.com> Thanks! Thanks for pointing out that JVM_GetClassLoader wasn't deleted. Coleen On 10/7/16 5:22 PM, Leela Mohan wrote: > Looks good. > > Thanks, > Leela > > On Fri, Oct 7, 2016 at 2:20 PM, Coleen Phillimore > > > wrote: > > > Here's the (retested) fix using JVM_GetClassLoader. > > open webrev at > http://cr.openjdk.java.net/~coleenp/8162766.02/webrev > > > From the history it seems that this part was backported with the > Class.getClassLoader() speedup change, and the code was corrected > in jdk9. > > Thanks, > Coleen > > > On 10/7/16 4:19 PM, Leela Mohan wrote: >> Hi Coleen, >> >> Sorry i never came back after we found the issue. We are >> certainly doing many unnecessary transitions from native to vm. >> It seems like, we can certainly avoid them if we have UNSAFE_LEAF >> (equivalent to JVM_LEAF) . But that might be larger change. >> >> Coming back to your change, I am wondering if we can use >> "JVM_GetClassLoader" instead. >> >> Thanks, >> Leela >> >> On Fri, Oct 7, 2016 at 1:01 PM, Coleen Phillimore >> > > wrote: >> >> Summary: Go to thread_in_vm to get the class_loader oop. >> >> This error was found by Leela Venati. Tested with jtreg >> hotspot/test. >> >> open webrev at >> http://cr.openjdk.java.net/~coleenp/8162766.01/webrev >> >> bug link https://bugs.openjdk.java.net/browse/JDK-8162766 >> >> >> Thanks, >> Coleen >> >> > > From serguei.spitsyn at oracle.com Sat Oct 8 21:20:01 2016 From: serguei.spitsyn at oracle.com (serguei.spitsyn at oracle.com) Date: Sat, 8 Oct 2016 14:20:01 -0700 Subject: [8u-dev] Request for approval: 8162766: Unsafe_DefineClass0 accesses raw oops while in _thread_in_native In-Reply-To: References: Message-ID: <1fd1b3f2-66fa-1f9c-dee5-c3212e22b491@oracle.com> If you need a reviewer then the fix looks good. Thanks, Serguei On 10/7/16 13:01, Coleen Phillimore wrote: > Summary: Go to thread_in_vm to get the class_loader oop. > > This error was found by Leela Venati. Tested with jtreg hotspot/test. > > open webrev at http://cr.openjdk.java.net/~coleenp/8162766.01/webrev > bug link https://bugs.openjdk.java.net/browse/JDK-8162766 > > Thanks, > Coleen From rob.mckenna at oracle.com Mon Oct 10 14:54:25 2016 From: rob.mckenna at oracle.com (Rob McKenna) Date: Mon, 10 Oct 2016 15:54:25 +0100 Subject: [8u-dev] Request for approval: 8162766: Unsafe_DefineClass0 accesses raw oops while in _thread_in_native In-Reply-To: <1fd1b3f2-66fa-1f9c-dee5-c3212e22b491@oracle.com> References: <1fd1b3f2-66fa-1f9c-dee5-c3212e22b491@oracle.com> Message-ID: <20161010145425.GA4425@tecra> Approved. Please add an appropriate noreg label to the bug. -Rob On 08/10/16 02:20, serguei.spitsyn at oracle.com wrote: > If you need a reviewer then the fix looks good. > > Thanks, > Serguei > > > On 10/7/16 13:01, Coleen Phillimore wrote: > >Summary: Go to thread_in_vm to get the class_loader oop. > > > >This error was found by Leela Venati. Tested with jtreg hotspot/test. > > > >open webrev at http://cr.openjdk.java.net/~coleenp/8162766.01/webrev > >bug link https://bugs.openjdk.java.net/browse/JDK-8162766 > > > >Thanks, > >Coleen > > From sgehwolf at redhat.com Mon Oct 10 15:27:06 2016 From: sgehwolf at redhat.com (Severin Gehwolf) Date: Mon, 10 Oct 2016 17:27:06 +0200 Subject: [8u-dev] Request for approval - 8153711: [REDO] JDWP: Memory Leak: GlobalRefs never deleted when processing invokeMethod command Message-ID: <1476113226.2885.51.camel@redhat.com> Hi, Please approve this backport request for JDK-8153711, which fixes a memory leak in JDWP. The fix has been in JDK 9[1] a couple of weeks now and no regressions have been discovered since it was pushed. It's a clean backport of the JDK 9 fix (after unshuffle) with minimal changes to the test which is needed due to differences in test libraries. Webrev:?http://cr.openjdk.java.net/~sgehwolf/webrevs/JDK-8153711/webrev.jdk8.01/ HG-export: http://cr.openjdk.java.net/~sgehwolf/webrevs/JDK-8153711/JDK-8153711-jdk8-jdk-final.export.patch Bug: https://bugs.openjdk.java.net/browse/JDK-8153711 Original review thread: http://mail.openjdk.java.net/pipermail/serviceability-dev/2016-September/020407.html Once approved, I'd need somebody who can push this changeset for me please! Thanks, Severin [1] http://hg.openjdk.java.net/jdk9/jdk9/jdk/rev/4c843eb35b8a From rob.mckenna at oracle.com Mon Oct 10 17:24:44 2016 From: rob.mckenna at oracle.com (Rob McKenna) Date: Mon, 10 Oct 2016 18:24:44 +0100 Subject: [8u-dev] Request for approval - 8153711: [REDO] JDWP: Memory Leak: GlobalRefs never deleted when processing invokeMethod command In-Reply-To: <1476113226.2885.51.camel@redhat.com> References: <1476113226.2885.51.camel@redhat.com> Message-ID: <20161010172444.GA2834@vimes> Approved -Rob On 10/10/16 05:27, Severin Gehwolf wrote: > Hi, > > Please approve this backport request for JDK-8153711, which fixes a > memory leak in JDWP. The fix has been in JDK 9[1] a couple of weeks now > and no regressions have been discovered since it was pushed. > > It's a clean backport of the JDK 9 fix (after unshuffle) with minimal > changes to the test which is needed due to differences in test > libraries. > > Webrev:?http://cr.openjdk.java.net/~sgehwolf/webrevs/JDK-8153711/webrev.jdk8.01/ > HG-export: http://cr.openjdk.java.net/~sgehwolf/webrevs/JDK-8153711/JDK-8153711-jdk8-jdk-final.export.patch > Bug: https://bugs.openjdk.java.net/browse/JDK-8153711 > Original review thread: http://mail.openjdk.java.net/pipermail/serviceability-dev/2016-September/020407.html > > Once approved, I'd need somebody who can push this changeset for me please! > > Thanks, > Severin > > [1] http://hg.openjdk.java.net/jdk9/jdk9/jdk/rev/4c843eb35b8a From iris.clark at oracle.com Mon Oct 10 17:40:01 2016 From: iris.clark at oracle.com (Iris Clark) Date: Mon, 10 Oct 2016 10:40:01 -0700 (PDT) Subject: [8u-dev] Request for aproval for 8166799 - ASSEMBLY_EXCEPTION uses historical company name Message-ID: <547ca8e9-f87a-4dd1-a0e4-4ad629d41bf5@default> Hi. Please approve backport of the following changes: Bug 8166799: ASSEMBLY_EXCEPTION contains historical company name https://bugs.openjdk.java.net/browse/JDK-8166799 webrev http://cr.openjdk.java.net/~iris/8166799/webrev/ Review http://mail.openjdk.java.net/pipermail/jdk9-dev/2016-October/004979.html Thanks, Iris From rob.mckenna at oracle.com Mon Oct 10 23:15:03 2016 From: rob.mckenna at oracle.com (Rob McKenna) Date: Tue, 11 Oct 2016 00:15:03 +0100 Subject: [8u-dev] Request for aproval for 8166799 - ASSEMBLY_EXCEPTION uses historical company name In-Reply-To: <547ca8e9-f87a-4dd1-a0e4-4ad629d41bf5@default> References: <547ca8e9-f87a-4dd1-a0e4-4ad629d41bf5@default> Message-ID: <20161010231503.GB2834@vimes> Approved -Rob On 10/10/16 10:40, Iris Clark wrote: > Hi. > > > > Please approve backport of the following changes: > > > > Bug > > 8166799: ASSEMBLY_EXCEPTION contains historical company name > https://bugs.openjdk.java.net/browse/JDK-8166799 > > > > webrev > > http://cr.openjdk.java.net/~iris/8166799/webrev/ > > > > Review > > http://mail.openjdk.java.net/pipermail/jdk9-dev/2016-October/004979.html > > > > Thanks, > > Iris > > From vladimir.kempik at oracle.com Tue Oct 11 06:09:45 2016 From: vladimir.kempik at oracle.com (Vladimir Kempik) Date: Tue, 11 Oct 2016 09:09:45 +0300 Subject: [8u] RFR 8154945: Enable 8130150 and 8081778 intrinsics by default Message-ID: Hello I'd like to port this fix into 8u-dev. This patch makes intrinsics introduced in 8130150 and 8081778 to be enabled by default. Basically it reverts https://bugs.openjdk.java.net/browse/JDK-8151522 (without reverting test changes of 8151522) Bug: https://bugs.openjdk.java.net/browse/JDK-8154945 Webrev: http://cr.openjdk.java.net/~vkempik/8154945/webrev.00/ Review Thread: http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/2016-October/024654.html Thanks -Vladimir From vladimir.kempik at oracle.com Tue Oct 11 06:23:55 2016 From: vladimir.kempik at oracle.com (Vladimir Kempik) Date: Tue, 11 Oct 2016 09:23:55 +0300 Subject: [8u] RFA 8154945: Enable 8130150 and 8081778 intrinsics by default In-Reply-To: References: Message-ID: <1b4d1223-46c5-8703-f13f-87481b6f7aca@oracle.com> RFA, not RFR 11.10.16 9:09, Vladimir Kempik ?????: > Hello > > I'd like to port this fix into 8u-dev. > > This patch makes intrinsics introduced in 8130150 and 8081778 to be > enabled by default. > > Basically it reverts https://bugs.openjdk.java.net/browse/JDK-8151522 > (without reverting test changes of 8151522) > > > Bug: https://bugs.openjdk.java.net/browse/JDK-8154945 > Webrev: http://cr.openjdk.java.net/~vkempik/8154945/webrev.00/ > Review Thread: > http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/2016-October/024654.html > Thanks > -Vladimir > From rob.mckenna at oracle.com Tue Oct 11 12:51:01 2016 From: rob.mckenna at oracle.com (Rob McKenna) Date: Tue, 11 Oct 2016 13:51:01 +0100 Subject: [8u] RFA 8154945: Enable 8130150 and 8081778 intrinsics by default In-Reply-To: <1b4d1223-46c5-8703-f13f-87481b6f7aca@oracle.com> References: <1b4d1223-46c5-8703-f13f-87481b6f7aca@oracle.com> Message-ID: <20161011125101.GA28015@vimes> Approved. Please add a suitable noreg label. -Rob On 11/10/16 09:23, Vladimir Kempik wrote: > RFA, not RFR > > > 11.10.16 9:09, Vladimir Kempik ?????: > >Hello > > > >I'd like to port this fix into 8u-dev. > > > >This patch makes intrinsics introduced in 8130150 and 8081778 to be > >enabled by default. > > > >Basically it reverts https://bugs.openjdk.java.net/browse/JDK-8151522 > >(without reverting test changes of 8151522) > > > > > >Bug: https://bugs.openjdk.java.net/browse/JDK-8154945 > >Webrev: http://cr.openjdk.java.net/~vkempik/8154945/webrev.00/ > >Review Thread: http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/2016-October/024654.html > >Thanks > >-Vladimir > > > From coleen.phillimore at oracle.com Tue Oct 11 18:01:32 2016 From: coleen.phillimore at oracle.com (Coleen Phillimore) Date: Tue, 11 Oct 2016 14:01:32 -0400 Subject: [8u-dev] Request for approval: 8162766: Unsafe_DefineClass0 accesses raw oops while in _thread_in_native In-Reply-To: <1fd1b3f2-66fa-1f9c-dee5-c3212e22b491@oracle.com> References: <1fd1b3f2-66fa-1f9c-dee5-c3212e22b491@oracle.com> Message-ID: <3738a155-9946-e61a-4826-bae82ee12b3f@oracle.com> Thank you, Serguei. Coleen On 10/8/16 5:20 PM, serguei.spitsyn at oracle.com wrote: > If you need a reviewer then the fix looks good. > > Thanks, > Serguei > > > On 10/7/16 13:01, Coleen Phillimore wrote: >> Summary: Go to thread_in_vm to get the class_loader oop. >> >> This error was found by Leela Venati. Tested with jtreg hotspot/test. >> >> open webrev at http://cr.openjdk.java.net/~coleenp/8162766.01/webrev >> bug link https://bugs.openjdk.java.net/browse/JDK-8162766 >> >> Thanks, >> Coleen > > From gnu.andrew at redhat.com Tue Oct 11 19:27:49 2016 From: gnu.andrew at redhat.com (Andrew Hughes) Date: Tue, 11 Oct 2016 15:27:49 -0400 (EDT) Subject: [8u-dev] Request for approval - 8153711: [REDO] JDWP: Memory Leak: GlobalRefs never deleted when processing invokeMethod command In-Reply-To: <20161010172444.GA2834@vimes> References: <1476113226.2885.51.camel@redhat.com> <20161010172444.GA2834@vimes> Message-ID: <578041548.342463.1476214069738.JavaMail.zimbra@redhat.com> ----- Original Message ----- > Approved > > -Rob > And pushed: http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/3243d893b0b2 > On 10/10/16 05:27, Severin Gehwolf wrote: > > Hi, > > > > Please approve this backport request for JDK-8153711, which fixes a > > memory leak in JDWP. The fix has been in JDK 9[1] a couple of weeks now > > and no regressions have been discovered since it was pushed. > > > > It's a clean backport of the JDK 9 fix (after unshuffle) with minimal > > changes to the test which is needed due to differences in test > > libraries. > > > > Webrev: > > ?http://cr.openjdk.java.net/~sgehwolf/webrevs/JDK-8153711/webrev.jdk8.01/ > > HG-export: > > http://cr.openjdk.java.net/~sgehwolf/webrevs/JDK-8153711/JDK-8153711-jdk8-jdk-final.export.patch > > Bug: https://bugs.openjdk.java.net/browse/JDK-8153711 > > Original review thread: > > http://mail.openjdk.java.net/pipermail/serviceability-dev/2016-September/020407.html > > > > Once approved, I'd need somebody who can push this changeset for me please! > > > > Thanks, > > Severin > > > > [1] http://hg.openjdk.java.net/jdk9/jdk9/jdk/rev/4c843eb35b8a > -- Andrew :) Senior Free Java Software Engineer Red Hat, Inc. (http://www.redhat.com) PGP Key: ed25519/0xCFDA0F9B35964222 (hkp://keys.gnupg.net) Fingerprint = 5132 579D D154 0ED2 3E04 C5A0 CFDA 0F9B 3596 4222 From ramanand.patil at oracle.com Thu Oct 13 11:07:10 2016 From: ramanand.patil at oracle.com (Ramanand Patil) Date: Thu, 13 Oct 2016 04:07:10 -0700 (PDT) Subject: [8u-dev] Request for Approval: Backport of 8167459: Add debug output for indicating if a chosen ciphersuite was legacy Message-ID: Hi, Please approve the backport of 8167459 to 8u-dev. Bug: https://bugs.openjdk.java.net/browse/JDK-8167459 JDK9 Changeset: http://hg.openjdk.java.net/jdk9/dev/jdk/rev/5cdd35a1baf8 JDK9 Review Thread: http://mail.openjdk.java.net/pipermail/security-dev/2016-October/014965.html Changes apply cleanly to jdk8u-dev. Regards, Ramanand. From coleen.phillimore at oracle.com Thu Oct 13 12:29:19 2016 From: coleen.phillimore at oracle.com (Coleen Phillimore) Date: Thu, 13 Oct 2016 08:29:19 -0400 Subject: [8u-dev] Request for approval: 8163969: Cyclic interface initialization causes JVM crash Message-ID: <803f87d1-6963-49e7-49a2-8d0efdbf97f3@oracle.com> This has been reviewed by sspitsyn and dholmes. Now requesting approval for the backport. Thanks, Coleen --- Summary: Backport change to correct interface initialization. There were too many changes to instanceKlass.cpp for a clean backport. Also in JDK8, this corrects interface initialization to not initialize the whole interface hierarchy if an interface, not class, initializes initialization. This is to correctly follow JLS 12.4.2 step 7. I filed a compatibility request (in review) to document the difference in behavior, which I believe will not be noticed. Tested with JPRT, including runtime jtreg lambda-features tests, and JCK tests. open webrev at http://cr.openjdk.java.net/~coleenp/8163969.8.01/webrev bug link https://bugs.openjdk.java.net/browse/JDK-8163969.8 From soho123.2012 at gmail.com Thu Oct 13 12:51:38 2016 From: soho123.2012 at gmail.com (Diaz Soho) Date: Thu, 13 Oct 2016 20:51:38 +0800 Subject: mips embedded platform how to Message-ID: Hi all, I am new in openJDK for mips embedded platform. I can not find any solution about running configure script for cross-compile mips cpu platform. Does anyone have idea? Best Regards, DiazSoho From dalibor.topic at oracle.com Thu Oct 13 13:13:19 2016 From: dalibor.topic at oracle.com (dalibor topic) Date: Thu, 13 Oct 2016 15:13:19 +0200 Subject: mips embedded platform how to In-Reply-To: References: Message-ID: On 13.10.2016 14:51, Diaz Soho wrote: > I am new in openJDK for mips embedded platform. I can not find any > solution about running configure script for cross-compile mips cpu > platform. Does anyone have idea? Hi, there is no specific MIPS port in JDK 8 Updates Project. You might be able to build the zero interpreter engine for your platform instead. Please see http://hg.openjdk.java.net/jdk8u/build/raw-file/tip/README-builds.html for details. cheers, dalibor topic -- Dalibor Topic | Principal Product Manager Phone: +494089091214 | Mobile: +491737185961 ORACLE Deutschland B.V. & Co. KG | K?hneh?fe 5 | 22761 Hamburg ORACLE Deutschland B.V. & Co. KG Hauptverwaltung: Riesstr. 25, D-80992 M?nchen Registergericht: Amtsgericht M?nchen, HRA 95603 Komplement?rin: ORACLE Deutschland Verwaltung B.V. Hertogswetering 163/167, 3543 AS Utrecht, Niederlande Handelsregister der Handelskammer Midden-Niederlande, Nr. 30143697 Gesch?ftsf?hrer: Alexander van der Ven, Jan Schultheiss, Val Maher Oracle is committed to developing practices and products that help protect the environment From soho123.2012 at gmail.com Thu Oct 13 13:17:04 2016 From: soho123.2012 at gmail.com (Diaz Soho) Date: Thu, 13 Oct 2016 21:17:04 +0800 Subject: mips embedded platform how to In-Reply-To: References: Message-ID: Hi , then which cpu platform supported by JDK 8 Updates Project? 2016-10-13 21:13 GMT+08:00 dalibor topic : > On 13.10.2016 14:51, Diaz Soho wrote: >> >> I am new in openJDK for mips embedded platform. I can not find any >> solution about running configure script for cross-compile mips cpu >> platform. Does anyone have idea? > > > Hi, > > there is no specific MIPS port in JDK 8 Updates Project. You might be able > to build the zero interpreter engine for your platform instead. > > Please see > http://hg.openjdk.java.net/jdk8u/build/raw-file/tip/README-builds.html for > details. > > cheers, > dalibor topic > -- > Dalibor Topic | Principal Product Manager > Phone: +494089091214 | Mobile: +491737185961 > > > ORACLE Deutschland B.V. & Co. KG | K?hneh?fe 5 | 22761 Hamburg > > ORACLE Deutschland B.V. & Co. KG > Hauptverwaltung: Riesstr. 25, D-80992 M?nchen > Registergericht: Amtsgericht M?nchen, HRA 95603 > > Komplement?rin: ORACLE Deutschland Verwaltung B.V. > Hertogswetering 163/167, 3543 AS Utrecht, Niederlande > Handelsregister der Handelskammer Midden-Niederlande, Nr. 30143697 > Gesch?ftsf?hrer: Alexander van der Ven, Jan Schultheiss, Val Maher > > Oracle is committed to developing > practices and products that help protect the environment From dalibor.topic at oracle.com Thu Oct 13 14:00:35 2016 From: dalibor.topic at oracle.com (dalibor topic) Date: Thu, 13 Oct 2016 16:00:35 +0200 Subject: mips embedded platform how to In-Reply-To: References: Message-ID: Please see http://hg.openjdk.java.net/jdk8u/jdk8u/hotspot/file/b0f2296a0bc8/src/os_cpu for a list of HotSpot ports in the JDK 8 Updates Project's source code tree. cheers, dalibor topic On 13.10.2016 15:17, Diaz Soho wrote: > Hi , > > then which cpu platform supported by JDK 8 Updates Project? > > 2016-10-13 21:13 GMT+08:00 dalibor topic : >> On 13.10.2016 14:51, Diaz Soho wrote: >>> >>> I am new in openJDK for mips embedded platform. I can not find any >>> solution about running configure script for cross-compile mips cpu >>> platform. Does anyone have idea? >> >> >> Hi, >> >> there is no specific MIPS port in JDK 8 Updates Project. You might be able >> to build the zero interpreter engine for your platform instead. >> >> Please see >> http://hg.openjdk.java.net/jdk8u/build/raw-file/tip/README-builds.html for >> details. >> >> cheers, >> dalibor topic >> -- >> Dalibor Topic | Principal Product Manager >> Phone: +494089091214 | Mobile: +491737185961 >> >> >> ORACLE Deutschland B.V. & Co. KG | K?hneh?fe 5 | 22761 Hamburg >> >> ORACLE Deutschland B.V. & Co. KG >> Hauptverwaltung: Riesstr. 25, D-80992 M?nchen >> Registergericht: Amtsgericht M?nchen, HRA 95603 >> >> Komplement?rin: ORACLE Deutschland Verwaltung B.V. >> Hertogswetering 163/167, 3543 AS Utrecht, Niederlande >> Handelsregister der Handelskammer Midden-Niederlande, Nr. 30143697 >> Gesch?ftsf?hrer: Alexander van der Ven, Jan Schultheiss, Val Maher >> >> Oracle is committed to developing >> practices and products that help protect the environment -- Dalibor Topic | Principal Product Manager Phone: +494089091214 | Mobile: +491737185961 ORACLE Deutschland B.V. & Co. KG | K?hneh?fe 5 | 22761 Hamburg ORACLE Deutschland B.V. & Co. KG Hauptverwaltung: Riesstr. 25, D-80992 M?nchen Registergericht: Amtsgericht M?nchen, HRA 95603 Komplement?rin: ORACLE Deutschland Verwaltung B.V. Hertogswetering 163/167, 3543 AS Utrecht, Niederlande Handelsregister der Handelskammer Midden-Niederlande, Nr. 30143697 Gesch?ftsf?hrer: Alexander van der Ven, Jan Schultheiss, Val Maher Oracle is committed to developing practices and products that help protect the environment From rob.mckenna at oracle.com Thu Oct 13 14:13:07 2016 From: rob.mckenna at oracle.com (Rob McKenna) Date: Thu, 13 Oct 2016 15:13:07 +0100 Subject: [8u-dev] Request for Approval: Backport of 8167459: Add debug output for indicating if a chosen ciphersuite was legacy In-Reply-To: References: Message-ID: <20161013141307.GB3354@vimes> Approved -Rob On 13/10/16 04:07, Ramanand Patil wrote: > Hi, > Please approve the backport of 8167459 to 8u-dev. > Bug: https://bugs.openjdk.java.net/browse/JDK-8167459 > JDK9 Changeset: http://hg.openjdk.java.net/jdk9/dev/jdk/rev/5cdd35a1baf8 > JDK9 Review Thread: http://mail.openjdk.java.net/pipermail/security-dev/2016-October/014965.html > > Changes apply cleanly to jdk8u-dev. > > Regards, > Ramanand. From soho123.2012 at gmail.com Thu Oct 13 14:30:36 2016 From: soho123.2012 at gmail.com (Diaz Soho) Date: Thu, 13 Oct 2016 22:30:36 +0800 Subject: mips embedded platform how to In-Reply-To: References: Message-ID: hi, if I just like to use jdk only, then does it still have cpu arch depend? where is the code that is cpu arch depend except hotspot? 2016-10-13 22:00 GMT+08:00 dalibor topic : > Please see > http://hg.openjdk.java.net/jdk8u/jdk8u/hotspot/file/b0f2296a0bc8/src/os_cpu > for a list of HotSpot ports in the JDK 8 Updates Project's source code tree. > > cheers, > dalibor topic > > On 13.10.2016 15:17, Diaz Soho wrote: >> >> Hi , >> >> then which cpu platform supported by JDK 8 Updates Project? >> >> 2016-10-13 21:13 GMT+08:00 dalibor topic : >>> >>> On 13.10.2016 14:51, Diaz Soho wrote: >>>> >>>> >>>> I am new in openJDK for mips embedded platform. I can not find any >>>> solution about running configure script for cross-compile mips cpu >>>> platform. Does anyone have idea? >>> >>> >>> >>> Hi, >>> >>> there is no specific MIPS port in JDK 8 Updates Project. You might be >>> able >>> to build the zero interpreter engine for your platform instead. >>> >>> Please see >>> http://hg.openjdk.java.net/jdk8u/build/raw-file/tip/README-builds.html >>> for >>> details. >>> >>> cheers, >>> dalibor topic >>> -- >>> Dalibor Topic | Principal Product Manager >>> Phone: +494089091214 | Mobile: +491737185961 >>> >>> >>> ORACLE Deutschland B.V. & Co. KG | K?hneh?fe 5 | 22761 Hamburg >>> >>> ORACLE Deutschland B.V. & Co. KG >>> Hauptverwaltung: Riesstr. 25, D-80992 M?nchen >>> Registergericht: Amtsgericht M?nchen, HRA 95603 >>> >>> Komplement?rin: ORACLE Deutschland Verwaltung B.V. >>> Hertogswetering 163/167, 3543 AS Utrecht, Niederlande >>> Handelsregister der Handelskammer Midden-Niederlande, Nr. 30143697 >>> Gesch?ftsf?hrer: Alexander van der Ven, Jan Schultheiss, Val Maher >>> >>> Oracle is committed to developing >>> practices and products that help protect the environment > > > -- > Dalibor Topic | Principal Product Manager > Phone: +494089091214 | Mobile: +491737185961 > > > ORACLE Deutschland B.V. & Co. KG | K?hneh?fe 5 | 22761 Hamburg > > ORACLE Deutschland B.V. & Co. KG > Hauptverwaltung: Riesstr. 25, D-80992 M?nchen > Registergericht: Amtsgericht M?nchen, HRA 95603 > > Komplement?rin: ORACLE Deutschland Verwaltung B.V. > Hertogswetering 163/167, 3543 AS Utrecht, Niederlande > Handelsregister der Handelskammer Midden-Niederlande, Nr. 30143697 > Gesch?ftsf?hrer: Alexander van der Ven, Jan Schultheiss, Val Maher > > Oracle is committed to developing > practices and products that help protect the environment From dalibor.topic at oracle.com Thu Oct 13 14:40:41 2016 From: dalibor.topic at oracle.com (dalibor topic) Date: Thu, 13 Oct 2016 16:40:41 +0200 Subject: mips embedded platform how to In-Reply-To: References: Message-ID: <62a7ae34-0d31-2749-7a3b-103b97449800@oracle.com> On 13.10.2016 16:30, Diaz Soho wrote: > hi, > if I just like to use jdk only, then does it still have cpu arch depend? > > where is the code that is cpu arch depend except hotspot? That's hard to say in general. Some of the class libraries use native code, which may or may not be deliberately or inadvertently CPU dependent due to being implemented in C and/or C++ and the platform-specific idiosyncrasies of their compilers. You'll have to try it for yourself, basically. cheers, dalibor topic -- Dalibor Topic | Principal Product Manager Phone: +494089091214 | Mobile: +491737185961 ORACLE Deutschland B.V. & Co. KG | K?hneh?fe 5 | 22761 Hamburg ORACLE Deutschland B.V. & Co. KG Hauptverwaltung: Riesstr. 25, D-80992 M?nchen Registergericht: Amtsgericht M?nchen, HRA 95603 Komplement?rin: ORACLE Deutschland Verwaltung B.V. Hertogswetering 163/167, 3543 AS Utrecht, Niederlande Handelsregister der Handelskammer Midden-Niederlande, Nr. 30143697 Gesch?ftsf?hrer: Alexander van der Ven, Jan Schultheiss, Val Maher Oracle is committed to developing practices and products that help protect the environment From david.holmes at oracle.com Thu Oct 13 22:15:02 2016 From: david.holmes at oracle.com (David Holmes) Date: Fri, 14 Oct 2016 08:15:02 +1000 Subject: mips embedded platform how to In-Reply-To: References: Message-ID: <5b1f1ca6-0165-0310-9a7d-b98a41184a3f@oracle.com> On 14/10/2016 12:30 AM, Diaz Soho wrote: > hi, > if I just like to use jdk only, then does it still have cpu arch depend? While there is unlikely to be any CPU dependent code in the non-hotspot sources, you can't just take the 'jdk' (by which I presume you mean rt.jar) and use it with some other VM. A JDK is rt.jar+hotspot - they don't work independently. David > where is the code that is cpu arch depend except hotspot? > > 2016-10-13 22:00 GMT+08:00 dalibor topic : >> Please see >> http://hg.openjdk.java.net/jdk8u/jdk8u/hotspot/file/b0f2296a0bc8/src/os_cpu >> for a list of HotSpot ports in the JDK 8 Updates Project's source code tree. >> >> cheers, >> dalibor topic >> >> On 13.10.2016 15:17, Diaz Soho wrote: >>> >>> Hi , >>> >>> then which cpu platform supported by JDK 8 Updates Project? >>> >>> 2016-10-13 21:13 GMT+08:00 dalibor topic : >>>> >>>> On 13.10.2016 14:51, Diaz Soho wrote: >>>>> >>>>> >>>>> I am new in openJDK for mips embedded platform. I can not find any >>>>> solution about running configure script for cross-compile mips cpu >>>>> platform. Does anyone have idea? >>>> >>>> >>>> >>>> Hi, >>>> >>>> there is no specific MIPS port in JDK 8 Updates Project. You might be >>>> able >>>> to build the zero interpreter engine for your platform instead. >>>> >>>> Please see >>>> http://hg.openjdk.java.net/jdk8u/build/raw-file/tip/README-builds.html >>>> for >>>> details. >>>> >>>> cheers, >>>> dalibor topic >>>> -- >>>> Dalibor Topic | Principal Product Manager >>>> Phone: +494089091214 | Mobile: +491737185961 >>>> >>>> >>>> ORACLE Deutschland B.V. & Co. KG | K?hneh?fe 5 | 22761 Hamburg >>>> >>>> ORACLE Deutschland B.V. & Co. KG >>>> Hauptverwaltung: Riesstr. 25, D-80992 M?nchen >>>> Registergericht: Amtsgericht M?nchen, HRA 95603 >>>> >>>> Komplement?rin: ORACLE Deutschland Verwaltung B.V. >>>> Hertogswetering 163/167, 3543 AS Utrecht, Niederlande >>>> Handelsregister der Handelskammer Midden-Niederlande, Nr. 30143697 >>>> Gesch?ftsf?hrer: Alexander van der Ven, Jan Schultheiss, Val Maher >>>> >>>> Oracle is committed to developing >>>> practices and products that help protect the environment >> >> >> -- >> Dalibor Topic | Principal Product Manager >> Phone: +494089091214 | Mobile: +491737185961 >> >> >> ORACLE Deutschland B.V. & Co. KG | K?hneh?fe 5 | 22761 Hamburg >> >> ORACLE Deutschland B.V. & Co. KG >> Hauptverwaltung: Riesstr. 25, D-80992 M?nchen >> Registergericht: Amtsgericht M?nchen, HRA 95603 >> >> Komplement?rin: ORACLE Deutschland Verwaltung B.V. >> Hertogswetering 163/167, 3543 AS Utrecht, Niederlande >> Handelsregister der Handelskammer Midden-Niederlande, Nr. 30143697 >> Gesch?ftsf?hrer: Alexander van der Ven, Jan Schultheiss, Val Maher >> >> Oracle is committed to developing >> practices and products that help protect the environment From david.holmes at oracle.com Fri Oct 14 02:47:21 2016 From: david.holmes at oracle.com (David Holmes) Date: Fri, 14 Oct 2016 12:47:21 +1000 Subject: mips embedded platform how to In-Reply-To: References: <5b1f1ca6-0165-0310-9a7d-b98a41184a3f@oracle.com> Message-ID: <2082ddf9-bd70-5493-223e-d3faf6d4353c@oracle.com> On 14/10/2016 11:54 AM, Diaz Soho wrote: > hi , > > even I set the configure option --with-jvm-variants=zero > --with-target-bits=32, and --openjdk-target=mips-linux, then > I still get the message "error: unsupported cpu mips" > > does anyone have idea why zero vm can not accept? It isn't zero that can't accept it, it is the build system checking for supported platforms. You need to edit common/autoconf/platform.m4 to add MIPS and regenerate the autogenerated_configure.sh file to get past that. David > > > 2016-10-14 6:15 GMT+08:00 David Holmes : >> On 14/10/2016 12:30 AM, Diaz Soho wrote: >>> >>> hi, >>> if I just like to use jdk only, then does it still have cpu arch depend? >> >> >> While there is unlikely to be any CPU dependent code in the non-hotspot >> sources, you can't just take the 'jdk' (by which I presume you mean rt.jar) >> and use it with some other VM. A JDK is rt.jar+hotspot - they don't work >> independently. >> >> David >> >>> where is the code that is cpu arch depend except hotspot? >>> >>> 2016-10-13 22:00 GMT+08:00 dalibor topic : >>>> >>>> Please see >>>> >>>> http://hg.openjdk.java.net/jdk8u/jdk8u/hotspot/file/b0f2296a0bc8/src/os_cpu >>>> for a list of HotSpot ports in the JDK 8 Updates Project's source code >>>> tree. >>>> >>>> cheers, >>>> dalibor topic >>>> >>>> On 13.10.2016 15:17, Diaz Soho wrote: >>>>> >>>>> >>>>> Hi , >>>>> >>>>> then which cpu platform supported by JDK 8 Updates Project? >>>>> >>>>> 2016-10-13 21:13 GMT+08:00 dalibor topic : >>>>>> >>>>>> >>>>>> On 13.10.2016 14:51, Diaz Soho wrote: >>>>>>> >>>>>>> >>>>>>> >>>>>>> I am new in openJDK for mips embedded platform. I can not find any >>>>>>> solution about running configure script for cross-compile mips cpu >>>>>>> platform. Does anyone have idea? >>>>>> >>>>>> >>>>>> >>>>>> >>>>>> Hi, >>>>>> >>>>>> there is no specific MIPS port in JDK 8 Updates Project. You might be >>>>>> able >>>>>> to build the zero interpreter engine for your platform instead. >>>>>> >>>>>> Please see >>>>>> http://hg.openjdk.java.net/jdk8u/build/raw-file/tip/README-builds.html >>>>>> for >>>>>> details. >>>>>> >>>>>> cheers, >>>>>> dalibor topic >>>>>> -- >>>>>> Dalibor Topic | Principal Product Manager >>>>>> Phone: +494089091214 | Mobile: +491737185961 >>>>>> >>>>>> >>>>>> ORACLE Deutschland B.V. & Co. KG | K?hneh?fe 5 | 22761 Hamburg >>>>>> >>>>>> ORACLE Deutschland B.V. & Co. KG >>>>>> Hauptverwaltung: Riesstr. 25, D-80992 M?nchen >>>>>> Registergericht: Amtsgericht M?nchen, HRA 95603 >>>>>> >>>>>> Komplement?rin: ORACLE Deutschland Verwaltung B.V. >>>>>> Hertogswetering 163/167, 3543 AS Utrecht, Niederlande >>>>>> Handelsregister der Handelskammer Midden-Niederlande, Nr. 30143697 >>>>>> Gesch?ftsf?hrer: Alexander van der Ven, Jan Schultheiss, Val Maher >>>>>> >>>>>> Oracle is committed to developing >>>>>> practices and products that help protect the environment >>>> >>>> >>>> >>>> -- >>>> Dalibor Topic | Principal Product Manager >>>> Phone: +494089091214 | Mobile: +491737185961 >>>> >>>> >>>> ORACLE Deutschland B.V. & Co. KG | K?hneh?fe 5 | 22761 Hamburg >>>> >>>> ORACLE Deutschland B.V. & Co. KG >>>> Hauptverwaltung: Riesstr. 25, D-80992 M?nchen >>>> Registergericht: Amtsgericht M?nchen, HRA 95603 >>>> >>>> Komplement?rin: ORACLE Deutschland Verwaltung B.V. >>>> Hertogswetering 163/167, 3543 AS Utrecht, Niederlande >>>> Handelsregister der Handelskammer Midden-Niederlande, Nr. 30143697 >>>> Gesch?ftsf?hrer: Alexander van der Ven, Jan Schultheiss, Val Maher >>>> >>>> Oracle is committed to developing >>>> practices and products that help protect the environment From soho123.2012 at gmail.com Fri Oct 14 07:10:34 2016 From: soho123.2012 at gmail.com (Diaz Soho) Date: Fri, 14 Oct 2016 15:10:34 +0800 Subject: why I can not set --without-freetype Message-ID: hi, why I can not set --without-freetype when run configure script? since I use openjdk for my embedded platform that does not support any graphic function. that is not necessary for me, but I can not disable anyone have idea? From david.holmes at oracle.com Fri Oct 14 08:39:18 2016 From: david.holmes at oracle.com (David Holmes) Date: Fri, 14 Oct 2016 18:39:18 +1000 Subject: why I can not set --without-freetype In-Reply-To: References: Message-ID: <5fb63741-30e6-2101-2a88-6cfd86471258@oracle.com> On 14/10/2016 5:10 PM, Diaz Soho wrote: > hi, > > why I can not set --without-freetype when run configure script? > since I use openjdk for my embedded platform that does not support any > graphic function. that is not necessary for me, but I can not disable > > anyone have idea? The build is not set up to cover every possible scenario, and requires freetype because it will build parts of the JDK that depend on it. Build requirements are covered in README-builds.html at the top of the repo. Build specific queries should go to build-dev at openjdk.java.net - though this query comes up a bit so you should be able to find a link via google. David From soho123.2012 at gmail.com Mon Oct 17 06:55:29 2016 From: soho123.2012 at gmail.com (Diaz Soho) Date: Mon, 17 Oct 2016 14:55:29 +0800 Subject: build error 'ZeroEntry' was not declared in this scope Message-ID: hi , when I use the option "--with-jvm-variants=zero " to compile jdk8u, I get the error: jdk8u/hotspot/src/cpu/zero/vm/cppInterpreterGenerator_zero.hpp:36:5: error: 'ZeroEntry' was not declared in this scope does anyome have ideas? From sgehwolf at redhat.com Mon Oct 17 08:31:42 2016 From: sgehwolf at redhat.com (Severin Gehwolf) Date: Mon, 17 Oct 2016 10:31:42 +0200 Subject: build error 'ZeroEntry' was not declared in this scope In-Reply-To: References: Message-ID: <1476693102.3047.1.camel@redhat.com> Hi, On Mon, 2016-10-17 at 14:55 +0800, Diaz Soho wrote: > hi , > > when I use the option "--with-jvm-variants=zero " to compile jdk8u, I > get the error: > jdk8u/hotspot/src/cpu/zero/vm/cppInterpreterGenerator_zero.hpp:36:5: > error: 'ZeroEntry' was not declared in this scope > > does anyome have ideas? Debian builds Zero on mips{,el} AFAIK[1]. You could have a look at their patches. Cheers, Severin [1]?https://packages.debian.org/jessie-backports/openjdk-8-jdk From christoph.langer at sap.com Mon Oct 17 08:38:31 2016 From: christoph.langer at sap.com (Langer, Christoph) Date: Mon, 17 Oct 2016 08:38:31 +0000 Subject: [8u-dev] Request for Review + Approval: Backport of 8163181: Further improvements for Unix NetworkInterface native implementation Message-ID: <866c93003d63482bb147736a5aa9f058@DEWDFE13DE11.global.corp.sap> Hi, please review (@Chris or Mark) and approve the following downport. Bug: https://bugs.openjdk.java.net/browse/JDK-8163181 Original Change: http://hg.openjdk.java.net/jdk9/dev/jdk/rev/40c3550625a2 New Webrev: http://cr.openjdk.java.net/~clanger/webrevs/8163181.8udev/ Mail discussion: http://mail.openjdk.java.net/pipermail/net-dev/2016-September/010207.html The main motivation for this downport is the bug in the AIX version of enumIPv6Interfaces. As the whole change is merely formatting, I decided to ask for a full downport. The patch does not apply cleanly, I had to resolve a little bit and also added some further formatting cleanups to be mostly identical with the JDK 9 version of NetworkInterface.c now. Thanks and best regards Christoph From philip.race at oracle.com Mon Oct 17 21:08:40 2016 From: philip.race at oracle.com (Philip Race) Date: Mon, 17 Oct 2016 14:08:40 -0700 Subject: [8u] Request for approval to backport 8089573: [macosx] Incorrect char to glyph mapping printing on OSX 10.10 Message-ID: <58053DD8.2010409@oracle.com> Hello, Please approve the backport of 8089573 to 8u-dev. This bug badly affects JavaFX printing on MacOS. Bug: https://bugs.openjdk.java.net/browse/JDK-8089573 JDK9 changeset : http://hg.openjdk.java.net/jdk9/client/jdk/rev/f950cc6004c6 JDK9 review thread :http://mail.openjdk.java.net/pipermail/2d-dev/2016-October/007806.html 8u webrev : http://cr.openjdk.java.net/~prr/8089573.8u/ -phil PS JDK8u122 RDP2 is fast approaching but I have not found any email on the exact cut-off .. From rob.mckenna at oracle.com Mon Oct 17 21:46:12 2016 From: rob.mckenna at oracle.com (Rob McKenna) Date: Mon, 17 Oct 2016 22:46:12 +0100 Subject: [8u] Request for approval to backport 8089573: [macosx] Incorrect char to glyph mapping printing on OSX 10.10 In-Reply-To: <58053DD8.2010409@oracle.com> References: <58053DD8.2010409@oracle.com> Message-ID: <20161017214612.GE5174@vimes> Approved -Rob On 17/10/16 02:08, Philip Race wrote: > Hello, > > Please approve the backport of 8089573 to 8u-dev. > This bug badly affects JavaFX printing on MacOS. > > Bug: https://bugs.openjdk.java.net/browse/JDK-8089573 > JDK9 changeset : http://hg.openjdk.java.net/jdk9/client/jdk/rev/f950cc6004c6 > JDK9 review thread > :http://mail.openjdk.java.net/pipermail/2d-dev/2016-October/007806.html > 8u webrev : http://cr.openjdk.java.net/~prr/8089573.8u/ > > -phil > > PS JDK8u122 RDP2 is fast approaching but I have not found any > email on the exact cut-off .. > From soho123.2012 at gmail.com Tue Oct 18 07:59:21 2016 From: soho123.2012 at gmail.com (Diaz Soho) Date: Tue, 18 Oct 2016 15:59:21 +0800 Subject: link error when make jdk8u with zero Message-ID: hi all, the error when build jdk8u, users/jamvm/jdk8u/build/linux-mips-normal-zero-fastdebug/jdk/btnative/genSocketOptionRegistry/genSocketOptionRegistry.o: In function `out': genSocketOptionRegistry.c:(.text+0xd): undefined reference to `puts' does anyone have idea? From chris.hegarty at oracle.com Tue Oct 18 10:52:52 2016 From: chris.hegarty at oracle.com (Chris Hegarty) Date: Tue, 18 Oct 2016 11:52:52 +0100 Subject: [8u-dev] Request for Review + Approval: Backport of 8163181: Further improvements for Unix NetworkInterface native implementation In-Reply-To: <866c93003d63482bb147736a5aa9f058@DEWDFE13DE11.global.corp.sap> References: <866c93003d63482bb147736a5aa9f058@DEWDFE13DE11.global.corp.sap> Message-ID: <4F20171F-1396-49E6-82A3-5056C76E4AF2@oracle.com> > On 17 Oct 2016, at 09:38, Langer, Christoph wrote: > > Hi, > > please review (@Chris or Mark) and approve the following downpour. From my point of view, I?m ok with this. You still need 8u maintainer approval. > Bug: https://bugs.openjdk.java.net/browse/JDK-8163181 > Original Change: http://hg.openjdk.java.net/jdk9/dev/jdk/rev/40c3550625a2 > New Webrev: http://cr.openjdk.java.net/~clanger/webrevs/8163181.8udev/ > Mail discussion: http://mail.openjdk.java.net/pipermail/net-dev/2016-September/010207.html > > The main motivation for this downport is the bug in the AIX version of enumIPv6Interfaces. > > As the whole change is merely formatting, I decided to ask for a full downport. The patch does not apply cleanly, I had to resolve a little bit and also added some further formatting cleanups to be mostly identical with the JDK 9 version of NetworkInterface.c now. > > Thanks and best regards > Christoph -Chris. From christoph.langer at sap.com Tue Oct 18 10:54:45 2016 From: christoph.langer at sap.com (Langer, Christoph) Date: Tue, 18 Oct 2016 10:54:45 +0000 Subject: [8u-dev] Request for Review + Approval: Backport of 8163181: Further improvements for Unix NetworkInterface native implementation In-Reply-To: <4F20171F-1396-49E6-82A3-5056C76E4AF2@oracle.com> References: <866c93003d63482bb147736a5aa9f058@DEWDFE13DE11.global.corp.sap> <4F20171F-1396-49E6-82A3-5056C76E4AF2@oracle.com> Message-ID: <350e9c4d534349d28b4c33ef09f634c5@DEWDFE13DE11.global.corp.sap> Thanks, Chris. So then I'm asking for approval now... Best regards Christoph > -----Original Message----- > From: Chris Hegarty [mailto:chris.hegarty at oracle.com] > Sent: Dienstag, 18. Oktober 2016 12:53 > To: Langer, Christoph > Cc: jdk8u-dev at openjdk.java.net; Mark Sheppard > ; net-dev at openjdk.java.net > Subject: Re: [8u-dev] Request for Review + Approval: Backport of 8163181: > Further improvements for Unix NetworkInterface native implementation > > > > On 17 Oct 2016, at 09:38, Langer, Christoph > wrote: > > > > Hi, > > > > please review (@Chris or Mark) and approve the following downpour. > > From my point of view, I?m ok with this. You still need 8u maintainer approval. > > > Bug: https://bugs.openjdk.java.net/browse/JDK-8163181 > > Original Change: http://hg.openjdk.java.net/jdk9/dev/jdk/rev/40c3550625a2 > > New Webrev: http://cr.openjdk.java.net/~clanger/webrevs/8163181.8udev/ > > Mail discussion: http://mail.openjdk.java.net/pipermail/net-dev/2016- > September/010207.html > > > > The main motivation for this downport is the bug in the AIX version of > enumIPv6Interfaces. > > > > As the whole change is merely formatting, I decided to ask for a full > downport. The patch does not apply cleanly, I had to resolve a little bit and also > added some further formatting cleanups to be mostly identical with the JDK 9 > version of NetworkInterface.c now. > > > > Thanks and best regards > > Christoph > > -Chris. From abhi.saha at oracle.com Tue Oct 18 19:14:34 2016 From: abhi.saha at oracle.com (Abhijit Saha) Date: Tue, 18 Oct 2016 12:14:34 -0700 Subject: [8u122] Request for approval for bulk changes from Oct 2016 PSU 8u112 fixes into 8u Message-ID: 8u111/8u112 has been released earlier today [1]. Requesting approval to sync up the 8u112 changes into the jdk8u forest. webrev : http://cr.openjdk.java.net/~asaha/openJDK.8u112-8u122.sync/webrev Thanks Abhijit [1] http://www.oracle.com/technetwork/java/javase/downloads/index.html -- Java Platform Group Oracle Corporation. (408)276-7564 From rob.mckenna at oracle.com Tue Oct 18 19:40:23 2016 From: rob.mckenna at oracle.com (Rob McKenna) Date: Tue, 18 Oct 2016 20:40:23 +0100 Subject: [8u122] Request for approval for bulk changes from Oct 2016 PSU 8u112 fixes into 8u In-Reply-To: References: Message-ID: <20161018194023.GC2701@vimes> Approved -Rob On 18/10/16 12:14, Abhijit Saha wrote: > 8u111/8u112 has been released earlier today [1]. Requesting approval to sync > up the 8u112 changes into the jdk8u forest. > > webrev : http://cr.openjdk.java.net/~asaha/openJDK.8u112-8u122.sync/webrev > > > Thanks > Abhijit > > > [1] http://www.oracle.com/technetwork/java/javase/downloads/index.html > > > -- > Java Platform Group > Oracle Corporation. > (408)276-7564 > From david.holmes at oracle.com Wed Oct 19 01:01:41 2016 From: david.holmes at oracle.com (David Holmes) Date: Wed, 19 Oct 2016 11:01:41 +1000 Subject: link error when make jdk8u with zero In-Reply-To: References: Message-ID: <181d1e1f-180b-b2cd-e3b7-b4f3258f5485@oracle.com> Hi, I suggest taking this to mips-port at openjdk.java.net Cheers, David On 18/10/2016 5:59 PM, Diaz Soho wrote: > hi all, > > the error when build jdk8u, > users/jamvm/jdk8u/build/linux-mips-normal-zero-fastdebug/jdk/btnative/genSocketOptionRegistry/genSocketOptionRegistry.o: > In function `out': > genSocketOptionRegistry.c:(.text+0xd): undefined reference to `puts' > > does anyone have idea? > From hannes.wallnoefer at oracle.com Wed Oct 19 14:04:43 2016 From: hannes.wallnoefer at oracle.com (=?utf-8?Q?Hannes_Walln=C3=B6fer?=) Date: Wed, 19 Oct 2016 16:04:43 +0200 Subject: [8u-dev] Request for Approval: 8166902: Nested object literal property maps not reset in optimistic recompilation Message-ID: Hi, Please approve back port of 8166902 to 8u-dev. Bug: https://bugs.openjdk.java.net/browse/JDK-8166902 Webrev: http://cr.openjdk.java.net/~hannesw/8166902/webrev.00/ Review thread: http://mail.openjdk.java.net/pipermail/nashorn-dev/2016-September/006513.html Changes apply cleanly to jdk8u-dev after path reshuffling. Thanks, Hannes From soho123.2012 at gmail.com Wed Oct 19 14:22:10 2016 From: soho123.2012 at gmail.com (Diaz Soho) Date: Wed, 19 Oct 2016 22:22:10 +0800 Subject: where is the output for jamVM? Message-ID: Hi all, I have finish build jdk8u with --openjdk-target=mips-linux --with-jvm-variants=zero. but I can not make sure where is the output for my jamVM ? I guess: jdk8u\build\linux-mips-normal-zero-fastdebug\hotspot\linux_mips_zero\fastdebug\libjvm.so, and anyother? like : jdk8u\build\linux-mips-normal-zero-fastdebug\jdk\lib\mips\*.so files? or others? From rob.mckenna at oracle.com Wed Oct 19 15:34:26 2016 From: rob.mckenna at oracle.com (Rob McKenna) Date: Wed, 19 Oct 2016 16:34:26 +0100 Subject: [8u-dev] Request for Approval: 8166902: Nested object literal property maps not reset in optimistic recompilation In-Reply-To: References: Message-ID: <20161019153426.GB3920@vimes> Approved -Rob On 19/10/16 04:04, Hannes Walln?fer wrote: > Hi, > > Please approve back port of 8166902 to 8u-dev. > > Bug: https://bugs.openjdk.java.net/browse/JDK-8166902 > Webrev: http://cr.openjdk.java.net/~hannesw/8166902/webrev.00/ > Review thread: http://mail.openjdk.java.net/pipermail/nashorn-dev/2016-September/006513.html > > Changes apply cleanly to jdk8u-dev after path reshuffling. > > Thanks, > Hannes From christoph.langer at sap.com Wed Oct 19 19:40:06 2016 From: christoph.langer at sap.com (Langer, Christoph) Date: Wed, 19 Oct 2016 19:40:06 +0000 Subject: [8u-dev] Request for Review + Approval: Backport of 8163181: Further improvements for Unix NetworkInterface native implementation References: <866c93003d63482bb147736a5aa9f058@DEWDFE13DE11.global.corp.sap> <4F20171F-1396-49E6-82A3-5056C76E4AF2@oracle.com> Message-ID: <3cafc07fc1ce484498200ae43a5998d5@dewdfe13de07.global.corp.sap> Hi, ping: Can I get approval for this downport? Thanks Christoph > -----Original Message----- > From: Langer, Christoph > Sent: Dienstag, 18. Oktober 2016 12:55 > To: jdk8u-dev at openjdk.java.net > Cc: Mark Sheppard ; net-dev at openjdk.java.net; > 'Chris Hegarty' > Subject: RE: [8u-dev] Request for Review + Approval: Backport of 8163181: > Further improvements for Unix NetworkInterface native implementation > > Thanks, Chris. > > So then I'm asking for approval now... > > Best regards > Christoph > > > -----Original Message----- > > From: Chris Hegarty [mailto:chris.hegarty at oracle.com] > > Sent: Dienstag, 18. Oktober 2016 12:53 > > To: Langer, Christoph > > Cc: jdk8u-dev at openjdk.java.net; Mark Sheppard > > ; net-dev at openjdk.java.net > > Subject: Re: [8u-dev] Request for Review + Approval: Backport of 8163181: > > Further improvements for Unix NetworkInterface native implementation > > > > > > > On 17 Oct 2016, at 09:38, Langer, Christoph > > wrote: > > > > > > Hi, > > > > > > please review (@Chris or Mark) and approve the following downpour. > > > > From my point of view, I?m ok with this. You still need 8u maintainer > approval. > > > > > Bug: https://bugs.openjdk.java.net/browse/JDK-8163181 > > > Original Change: > http://hg.openjdk.java.net/jdk9/dev/jdk/rev/40c3550625a2 > > > New Webrev: http://cr.openjdk.java.net/~clanger/webrevs/8163181.8udev/ > > > Mail discussion: http://mail.openjdk.java.net/pipermail/net-dev/2016- > > September/010207.html > > > > > > The main motivation for this downport is the bug in the AIX version of > > enumIPv6Interfaces. > > > > > > As the whole change is merely formatting, I decided to ask for a full > > downport. The patch does not apply cleanly, I had to resolve a little bit and > also > > added some further formatting cleanups to be mostly identical with the JDK 9 > > version of NetworkInterface.c now. > > > > > > Thanks and best regards > > > Christoph > > > > -Chris. From rob.mckenna at oracle.com Wed Oct 19 19:53:59 2016 From: rob.mckenna at oracle.com (Rob McKenna) Date: Wed, 19 Oct 2016 20:53:59 +0100 Subject: [8u-dev] Request for Review + Approval: Backport of 8163181: Further improvements for Unix NetworkInterface native implementation In-Reply-To: <3cafc07fc1ce484498200ae43a5998d5@dewdfe13de07.global.corp.sap> References: <866c93003d63482bb147736a5aa9f058@DEWDFE13DE11.global.corp.sap> <4F20171F-1396-49E6-82A3-5056C76E4AF2@oracle.com> <3cafc07fc1ce484498200ae43a5998d5@dewdfe13de07.global.corp.sap> Message-ID: <20161019195359.GC3920@vimes> Yup. Please add an appropriate noreg label to the bug though. Approved. -Rob On 19/10/16 07:40, Langer, Christoph wrote: > Hi, > > ping: Can I get approval for this downport? > > Thanks > Christoph > > > -----Original Message----- > > From: Langer, Christoph > > Sent: Dienstag, 18. Oktober 2016 12:55 > > To: jdk8u-dev at openjdk.java.net > > Cc: Mark Sheppard ; net-dev at openjdk.java.net; > > 'Chris Hegarty' > > Subject: RE: [8u-dev] Request for Review + Approval: Backport of 8163181: > > Further improvements for Unix NetworkInterface native implementation > > > > Thanks, Chris. > > > > So then I'm asking for approval now... > > > > Best regards > > Christoph > > > > > -----Original Message----- > > > From: Chris Hegarty [mailto:chris.hegarty at oracle.com] > > > Sent: Dienstag, 18. Oktober 2016 12:53 > > > To: Langer, Christoph > > > Cc: jdk8u-dev at openjdk.java.net; Mark Sheppard > > > ; net-dev at openjdk.java.net > > > Subject: Re: [8u-dev] Request for Review + Approval: Backport of 8163181: > > > Further improvements for Unix NetworkInterface native implementation > > > > > > > > > > On 17 Oct 2016, at 09:38, Langer, Christoph > > > wrote: > > > > > > > > Hi, > > > > > > > > please review (@Chris or Mark) and approve the following downpour. > > > > > > From my point of view, I?m ok with this. You still need 8u maintainer > > approval. > > > > > > > Bug: https://bugs.openjdk.java.net/browse/JDK-8163181 > > > > Original Change: > > http://hg.openjdk.java.net/jdk9/dev/jdk/rev/40c3550625a2 > > > > New Webrev: http://cr.openjdk.java.net/~clanger/webrevs/8163181.8udev/ > > > > Mail discussion: http://mail.openjdk.java.net/pipermail/net-dev/2016- > > > September/010207.html > > > > > > > > The main motivation for this downport is the bug in the AIX version of > > > enumIPv6Interfaces. > > > > > > > > As the whole change is merely formatting, I decided to ask for a full > > > downport. The patch does not apply cleanly, I had to resolve a little bit and > > also > > > added some further formatting cleanups to be mostly identical with the JDK 9 > > > version of NetworkInterface.c now. > > > > > > > > Thanks and best regards > > > > Christoph > > > > > > -Chris. From david.holmes at oracle.com Thu Oct 20 00:20:54 2016 From: david.holmes at oracle.com (David Holmes) Date: Thu, 20 Oct 2016 10:20:54 +1000 Subject: where is the output for jamVM? In-Reply-To: References: Message-ID: Hi, This is not the right mailing list for these build queries. If you continue to have build queries please use build-dev at openjdk.java.net, or else discuss on the mips-port list. On 20/10/2016 12:22 AM, Diaz Soho wrote: > Hi all, > > I have finish build jdk8u with --openjdk-target=mips-linux > --with-jvm-variants=zero. > > but I can not make sure where is the output for my jamVM ? > I guess: > jdk8u\build\linux-mips-normal-zero-fastdebug\hotspot\linux_mips_zero\fastdebug\libjvm.so, > and anyother? > > like : > jdk8u\build\linux-mips-normal-zero-fastdebug\jdk\lib\mips\*.so files? > > or others? What make target did you specify? If you do "make images" then you will get a full JDK and JRE under build\linux-mips-normal-zero-fastdebug\images David From dalibor.topic at oracle.com Thu Oct 20 01:02:37 2016 From: dalibor.topic at oracle.com (dalibor topic) Date: Thu, 20 Oct 2016 03:02:37 +0200 Subject: where is the output for jamVM? In-Reply-To: References: Message-ID: <3f725d5b-5a92-dd1b-a154-8bbf49e127b4@oracle.com> On 19.10.2016 16:22, Diaz Soho wrote: > Hi all, > > I have finish build jdk8u with --openjdk-target=mips-linux > --with-jvm-variants=zero. > > but I can not make sure where is the output for my jamVM ? There is no JamVM in OpenJDK. cheers, dalibor topic -- Dalibor Topic | Principal Product Manager Phone: +494089091214 | Mobile: +491737185961 ORACLE Deutschland B.V. & Co. KG | K?hneh?fe 5 | 22761 Hamburg ORACLE Deutschland B.V. & Co. KG Hauptverwaltung: Riesstr. 25, D-80992 M?nchen Registergericht: Amtsgericht M?nchen, HRA 95603 Komplement?rin: ORACLE Deutschland Verwaltung B.V. Hertogswetering 163/167, 3543 AS Utrecht, Niederlande Handelsregister der Handelskammer Midden-Niederlande, Nr. 30143697 Gesch?ftsf?hrer: Alexander van der Ven, Jan Schultheiss, Val Maher Oracle is committed to developing practices and products that help protect the environment From david.holmes at oracle.com Thu Oct 20 01:19:07 2016 From: david.holmes at oracle.com (David Holmes) Date: Thu, 20 Oct 2016 11:19:07 +1000 Subject: where is the output for jamVM? In-Reply-To: <3f725d5b-5a92-dd1b-a154-8bbf49e127b4@oracle.com> References: <3f725d5b-5a92-dd1b-a154-8bbf49e127b4@oracle.com> Message-ID: <47a85ab2-c261-77b1-756f-a995dde8ca87@oracle.com> On 20/10/2016 11:02 AM, dalibor topic wrote: > On 19.10.2016 16:22, Diaz Soho wrote: >> Hi all, >> >> I have finish build jdk8u with --openjdk-target=mips-linux >> --with-jvm-variants=zero. >> >> but I can not make sure where is the output for my jamVM ? > > There is no JamVM in OpenJDK. :) I had assumed this was a typo for Java VM Cheers, David > cheers, > dalibor topic > From morethanair at coupang.com Thu Oct 20 00:49:01 2016 From: morethanair at coupang.com (=?utf-8?Q?=22Khan_N=28=EC=B9=B8/=EB=82=A8=EA=B7=9C=ED=95=9C=29_?= =?utf-8?Q?=5BCloud_Platform_Tribe=5D=22?=) Date: Thu, 20 Oct 2016 09:49:01 +0900 Subject: Regarding JDK-8049846 Bug Message-ID: <143DD2F5-F451-4FF7-86BE-0730212F5485@coupang.com> Hi, We are migrating our system from Java 7 to Java 8. One of our teams could not upgrade to Java 8 because of JDK-8049846 issue. I found the resolved date for the issue was Sep 13th, 2016. Can we know when the expected release date and version for this issue? Best Regards, Khan -- ??? ( Khan ) ? Technical Program Manager @ eCommerce Platform Tel : 02-6150-5377 Mobile : 010-9340-0416 e-mail : morethanair at coupang.com From volker.simonis at gmail.com Thu Oct 20 08:56:51 2016 From: volker.simonis at gmail.com (Volker Simonis) Date: Thu, 20 Oct 2016 10:56:51 +0200 Subject: Regarding JDK-8049846 Bug In-Reply-To: <143DD2F5-F451-4FF7-86BE-0730212F5485@coupang.com> References: <143DD2F5-F451-4FF7-86BE-0730212F5485@coupang.com> Message-ID: Hi Khan, as you can see in the bug, there has been no downport of 8049846 to jdk8u until now. Take a look at bug https://bugs.openjdk.java.net/browse/JDK-8072463 for example to see how a bug entry looks like which has been downported to previous Java versions. There's no automatism for downporting bug fixes. You should request it from the distributor of your Java binaries (i.e. Oracle if you use Oracle JDK, RedHat if you're using OpenJDK on RHEL/Fedora, etc...). Regards, Volker On Thu, Oct 20, 2016 at 2:49 AM, "Khan N(?/???) [Cloud Platform Tribe]" wrote: > Hi, > > We are migrating our system from Java 7 to Java 8. > One of our teams could not upgrade to Java 8 because of JDK-8049846 issue. > > I found the resolved date for the issue was Sep 13th, 2016. > Can we know when the expected release date and version for this issue? > > Best Regards, > Khan > > -- > ??? ( Khan ) ? > Technical Program Manager @ eCommerce Platform > > Tel : 02-6150-5377 > Mobile : 010-9340-0416 > e-mail : morethanair at coupang.com > > > From morethanair at coupang.com Thu Oct 20 11:19:29 2016 From: morethanair at coupang.com (=?utf-8?Q?=22Khan_N=28=EC=B9=B8/=EB=82=A8=EA=B7=9C=ED=95=9C=29_?= =?utf-8?Q?=5BTPMs=5D_=C2=AD=22?=) Date: Thu, 20 Oct 2016 20:19:29 +0900 Subject: Regarding JDK-8049846 Bug In-Reply-To: References: <143DD2F5-F451-4FF7-86BE-0730212F5485@coupang.com> Message-ID: Hello, Volker Thank you for your answer. You mean I need to contact to Oracle for this issue, right? Could you elaborate what 'downporting' means? Regards, Khan > On Oct 20, 2016, at 5:56 PM, Volker Simonis wrote: > > Hi Khan, > > as you can see in the bug, there has been no downport of 8049846 to > jdk8u until now. Take a look at bug > https://bugs.openjdk.java.net/browse/JDK-8072463 for example to see > how a bug entry looks like which has been downported to previous Java > versions. > > There's no automatism for downporting bug fixes. You should request it > from the distributor of your Java binaries (i.e. Oracle if you use > Oracle JDK, RedHat if you're using OpenJDK on RHEL/Fedora, etc...). > > Regards, > Volker > > > On Thu, Oct 20, 2016 at 2:49 AM, "Khan N(?/???) [Cloud Platform > Tribe]" > wrote: >> Hi, >> >> We are migrating our system from Java 7 to Java 8. >> One of our teams could not upgrade to Java 8 because of JDK-8049846 issue. >> >> I found the resolved date for the issue was Sep 13th, 2016. >> Can we know when the expected release date and version for this issue? >> >> Best Regards, >> Khan >> >> -- >> ??? ( Khan ) ? >> Technical Program Manager @ eCommerce Platform >> >> Tel : 02-6150-5377 >> Mobile : 010-9340-0416 >> e-mail : morethanair at coupang.com > From soho123.2012 at gmail.com Thu Oct 20 13:28:59 2016 From: soho123.2012 at gmail.com (Diaz Soho) Date: Thu, 20 Oct 2016 21:28:59 +0800 Subject: the Minimum set when install jdk8u Message-ID: Hi all, when I finish build jdk8u, and try to install to a local directory, I find the folder /usr/local/jvm/ include a lots of files , the size is very large. is there any minimun set that I can select to install to my embedded platform? for example: if I would like to run "java -jar " on my embedded platform, then how to figure out the minimum set that should be installed ? Thanks! Best Regards, DiazSoho From david.holmes at oracle.com Thu Oct 20 13:38:36 2016 From: david.holmes at oracle.com (David Holmes) Date: Thu, 20 Oct 2016 23:38:36 +1000 Subject: the Minimum set when install jdk8u In-Reply-To: References: Message-ID: <38a4eca4-2dd0-2034-1139-a778e713a522@oracle.com> On 20/10/2016 11:28 PM, Diaz Soho wrote: > Hi all, > > when I finish build jdk8u, and try to install to a local directory, > I find the folder /usr/local/jvm/ include a lots of files , the size > is very large. > > is there any minimun set that I can select to install to my embedded platform? > for example: > if I would like to run "java -jar " on my embedded > platform, then how to figure out the minimum set that should be > installed ? The Compact Profiles support smaller runtime images (subsetting rt.jar), as does the Minimal VM (stripped down with some services removed). But there are only three defined profiles, you can't arbitrarily remove things from rt.jar David > Thanks! > > Best Regards, > DiazSoho > From volker.simonis at gmail.com Thu Oct 20 13:54:06 2016 From: volker.simonis at gmail.com (Volker Simonis) Date: Thu, 20 Oct 2016 15:54:06 +0200 Subject: Regarding JDK-8049846 Bug In-Reply-To: References: <143DD2F5-F451-4FF7-86BE-0730212F5485@coupang.com> Message-ID: On Thu, Oct 20, 2016 at 1:19 PM, "Khan N(?/???) [TPMs] ?" wrote: > Hello, Volker > > Thank you for your answer. > You mean I need to contact to Oracle for this issue, right? > You need to contact the one who builds your jdk binaries. If you are using the Oracle JDK that's obviously Oracle. > Could you elaborate what 'downporting' means? Downporting is the process of integrating a change from the head revision (in this case jdk9 to a lower release, e.g. jdk8 or jdk7). > > Regards, > Khan > > On Oct 20, 2016, at 5:56 PM, Volker Simonis > wrote: > > Hi Khan, > > as you can see in the bug, there has been no downport of 8049846 to > jdk8u until now. Take a look at bug > https://bugs.openjdk.java.net/browse/JDK-8072463 for example to see > how a bug entry looks like which has been downported to previous Java > versions. > > There's no automatism for downporting bug fixes. You should request it > from the distributor of your Java binaries (i.e. Oracle if you use > Oracle JDK, RedHat if you're using OpenJDK on RHEL/Fedora, etc...). > > Regards, > Volker > > > On Thu, Oct 20, 2016 at 2:49 AM, "Khan N(?/???) [Cloud Platform > Tribe]" wrote: > > Hi, > > We are migrating our system from Java 7 to Java 8. > One of our teams could not upgrade to Java 8 because of JDK-8049846 issue. > > I found the resolved date for the issue was Sep 13th, 2016. > Can we know when the expected release date and version for this issue? > > Best Regards, > Khan > > -- > ??? ( Khan ) ? > Technical Program Manager @ eCommerce Platform > > Tel : 02-6150-5377 > Mobile : 010-9340-0416 > e-mail : morethanair at coupang.com > > From morethanair at coupang.com Thu Oct 20 13:55:38 2016 From: morethanair at coupang.com (=?utf-8?Q?=22Khan_N=28=EC=B9=B8/=EB=82=A8=EA=B7=9C=ED=95=9C=29_?= =?utf-8?Q?=5BTPMs=5D_=C2=AD=22?=) Date: Thu, 20 Oct 2016 22:55:38 +0900 Subject: Regarding JDK-8049846 Bug In-Reply-To: References: <143DD2F5-F451-4FF7-86BE-0730212F5485@coupang.com> Message-ID: Hi, Volker I appreciate your kind reply. Regards, Khan > On Oct 20, 2016, at 10:54 PM, Volker Simonis wrote: > > On Thu, Oct 20, 2016 at 1:19 PM, "Khan N(?/???) [TPMs] ?" > wrote: >> Hello, Volker >> >> Thank you for your answer. >> You mean I need to contact to Oracle for this issue, right? >> > > You need to contact the one who builds your jdk binaries. If you are > using the Oracle JDK that's obviously Oracle. > >> Could you elaborate what 'downporting' means? > > Downporting is the process of integrating a change from the head > revision (in this case jdk9 to a lower release, e.g. jdk8 or jdk7). > >> >> Regards, >> Khan >> >> On Oct 20, 2016, at 5:56 PM, Volker Simonis >> wrote: >> >> Hi Khan, >> >> as you can see in the bug, there has been no downport of 8049846 to >> jdk8u until now. Take a look at bug >> https://bugs.openjdk.java.net/browse/JDK-8072463 for example to see >> how a bug entry looks like which has been downported to previous Java >> versions. >> >> There's no automatism for downporting bug fixes. You should request it >> from the distributor of your Java binaries (i.e. Oracle if you use >> Oracle JDK, RedHat if you're using OpenJDK on RHEL/Fedora, etc...). >> >> Regards, >> Volker >> >> >> On Thu, Oct 20, 2016 at 2:49 AM, "Khan N(?/???) [Cloud Platform >> Tribe]" wrote: >> >> Hi, >> >> We are migrating our system from Java 7 to Java 8. >> One of our teams could not upgrade to Java 8 because of JDK-8049846 issue. >> >> I found the resolved date for the issue was Sep 13th, 2016. >> Can we know when the expected release date and version for this issue? >> >> Best Regards, >> Khan >> >> -- >> ??? ( Khan ) ? >> Technical Program Manager @ eCommerce Platform >> >> Tel : 02-6150-5377 >> Mobile : 010-9340-0416 >> e-mail : morethanair at coupang.com >> >> From dalibor.topic at oracle.com Thu Oct 20 14:07:17 2016 From: dalibor.topic at oracle.com (dalibor topic) Date: Thu, 20 Oct 2016 16:07:17 +0200 Subject: link error when make jdk8u with zero In-Reply-To: References: Message-ID: <3ff43033-070c-aeb7-bd9e-7cc2f9040e8a@oracle.com> On 18.10.2016 09:59, Diaz Soho wrote: > hi all, > > the error when build jdk8u, > users/jamvm/jdk8u/build/linux-mips-normal-zero-fastdebug/jdk/btnative/genSocketOptionRegistry/genSocketOptionRegistry.o: > In function `out': > genSocketOptionRegistry.c:(.text+0xd): undefined reference to `puts' > > does anyone have idea? > You might be using the wrong linker, assuming you're trying to cross-compile, rather than building on the platform you intend to run on. cheers, dalibor topic -- Dalibor Topic | Principal Product Manager Phone: +494089091214 | Mobile: +491737185961 ORACLE Deutschland B.V. & Co. KG | K?hneh?fe 5 | 22761 Hamburg ORACLE Deutschland B.V. & Co. KG Hauptverwaltung: Riesstr. 25, D-80992 M?nchen Registergericht: Amtsgericht M?nchen, HRA 95603 Komplement?rin: ORACLE Deutschland Verwaltung B.V. Hertogswetering 163/167, 3543 AS Utrecht, Niederlande Handelsregister der Handelskammer Midden-Niederlande, Nr. 30143697 Gesch?ftsf?hrer: Alexander van der Ven, Jan Schultheiss, Val Maher Oracle is committed to developing practices and products that help protect the environment From ivan.gerasimov at oracle.com Thu Oct 20 18:15:24 2016 From: ivan.gerasimov at oracle.com (Ivan Gerasimov) Date: Thu, 20 Oct 2016 21:15:24 +0300 Subject: [8u-dev] Request for Review and Approval to backport: 8167591: Add MD5 to signed JAR restrictions Message-ID: Hello! Would you please help review and give the approval to backport this fix? The changes in the backport, comparing to the fix in 9, are due to different file structure and the different previous value of the property. Bug: https://bugs.openjdk.java.net/browse/JDK-8167591 Jdk9 change: http://hg.openjdk.java.net/jdk9/dev/jdk/rev/aabbe4707f1f Jdk9 review: http://mail.openjdk.java.net/pipermail/security-dev/2016-October/014995.html Jdk8 webrev: http://cr.openjdk.java.net/~igerasim/8167591/00/webrev/ With kind regards, Ivan From david.holmes at oracle.com Thu Oct 20 20:47:47 2016 From: david.holmes at oracle.com (David Holmes) Date: Fri, 21 Oct 2016 06:47:47 +1000 Subject: the Minimum set when install jdk8u In-Reply-To: References: <38a4eca4-2dd0-2034-1139-a778e713a522@oracle.com> Message-ID: <09da47ce-9ac5-bb8e-c874-81c1e08c667f@oracle.com> Hi, Please keep things on the mailing list rather than private emails. On 20/10/2016 11:49 PM, Diaz Soho wrote: > Hi David, > > currently, the VM default is server, > but how about the Minima VM setting ? Minimal VM is built by specifying it as one of the jvm variants to configure eg: ./configure.sh --with-jvm-variants server,client,minimal1 note it is called minimal1 at the build level. The Minimal VM is a variant of the client VM so only available for JIT supported platforms (not via Zero) and only for 32-bit. If your JDK image has been built with the Minimal VM then it can be executed using "java -minimal ..." Compact profiles are built by using "make profiles". David > 2016-10-20 21:38 GMT+08:00 David Holmes : >> On 20/10/2016 11:28 PM, Diaz Soho wrote: >>> >>> Hi all, >>> >>> when I finish build jdk8u, and try to install to a local directory, >>> I find the folder /usr/local/jvm/ include a lots of files , the size >>> is very large. >>> >>> is there any minimun set that I can select to install to my embedded >>> platform? >>> for example: >>> if I would like to run "java -jar " on my embedded >>> platform, then how to figure out the minimum set that should be >>> installed ? >> >> >> The Compact Profiles support smaller runtime images (subsetting rt.jar), as >> does the Minimal VM (stripped down with some services removed). But there >> are only three defined profiles, you can't arbitrarily remove things from >> rt.jar >> >> David >> >>> Thanks! >>> >>> Best Regards, >>> DiazSoho >>> >> From david.holmes at oracle.com Thu Oct 20 20:58:54 2016 From: david.holmes at oracle.com (David Holmes) Date: Fri, 21 Oct 2016 06:58:54 +1000 Subject: Regarding JDK-8049846 Bug In-Reply-To: References: <143DD2F5-F451-4FF7-86BE-0730212F5485@coupang.com> Message-ID: On 20/10/2016 11:54 PM, Volker Simonis wrote: > On Thu, Oct 20, 2016 at 1:19 PM, "Khan N(?/???) [TPMs] ?" > wrote: >> Hello, Volker >> >> Thank you for your answer. >> You mean I need to contact to Oracle for this issue, right? >> > > You need to contact the one who builds your jdk binaries. If you are > using the Oracle JDK that's obviously Oracle. > >> Could you elaborate what 'downporting' means? > > Downporting is the process of integrating a change from the head > revision (in this case jdk9 to a lower release, e.g. jdk8 or jdk7). "we" also refer to this as backporting. JBS can create backport issues for things that should be backported. David >> >> Regards, >> Khan >> >> On Oct 20, 2016, at 5:56 PM, Volker Simonis >> wrote: >> >> Hi Khan, >> >> as you can see in the bug, there has been no downport of 8049846 to >> jdk8u until now. Take a look at bug >> https://bugs.openjdk.java.net/browse/JDK-8072463 for example to see >> how a bug entry looks like which has been downported to previous Java >> versions. >> >> There's no automatism for downporting bug fixes. You should request it >> from the distributor of your Java binaries (i.e. Oracle if you use >> Oracle JDK, RedHat if you're using OpenJDK on RHEL/Fedora, etc...). >> >> Regards, >> Volker >> >> >> On Thu, Oct 20, 2016 at 2:49 AM, "Khan N(?/???) [Cloud Platform >> Tribe]" wrote: >> >> Hi, >> >> We are migrating our system from Java 7 to Java 8. >> One of our teams could not upgrade to Java 8 because of JDK-8049846 issue. >> >> I found the resolved date for the issue was Sep 13th, 2016. >> Can we know when the expected release date and version for this issue? >> >> Best Regards, >> Khan >> >> -- >> ??? ( Khan ) ? >> Technical Program Manager @ eCommerce Platform >> >> Tel : 02-6150-5377 >> Mobile : 010-9340-0416 >> e-mail : morethanair at coupang.com >> >> From anthony.scarpino at oracle.com Thu Oct 20 21:06:43 2016 From: anthony.scarpino at oracle.com (Anthony Scarpino) Date: Thu, 20 Oct 2016 14:06:43 -0700 Subject: [8u-dev] Request for Review and Approval to backport: 8167591: Add MD5 to signed JAR restrictions In-Reply-To: References: Message-ID: <580931E3.9010506@oracle.com> On 10/20/2016 11:15 AM, Ivan Gerasimov wrote: > Hello! > > Would you please help review and give the approval to backport this fix? > > The changes in the backport, comparing to the fix in 9, are due to > different file structure and the different previous value of the property. > > Bug: https://bugs.openjdk.java.net/browse/JDK-8167591 > Jdk9 change: http://hg.openjdk.java.net/jdk9/dev/jdk/rev/aabbe4707f1f > Jdk9 review: > http://mail.openjdk.java.net/pipermail/security-dev/2016-October/014995.html > > Jdk8 webrev: http://cr.openjdk.java.net/~igerasim/8167591/00/webrev/ > > With kind regards, > Ivan > This looks fine to me. Tony From sean.coffey at oracle.com Thu Oct 20 21:16:46 2016 From: sean.coffey at oracle.com (=?UTF-8?Q?Se=c3=a1n_Coffey?=) Date: Thu, 20 Oct 2016 22:16:46 +0100 Subject: [8u-dev] Request for Review and Approval to backport: 8167591: Add MD5 to signed JAR restrictions In-Reply-To: <580931E3.9010506@oracle.com> References: <580931E3.9010506@oracle.com> Message-ID: <597123d4-e1db-1633-0eb0-df9000b1671c@oracle.com> Approved. Regards, Sean. On 20/10/2016 22:06, Anthony Scarpino wrote: > On 10/20/2016 11:15 AM, Ivan Gerasimov wrote: >> Hello! >> >> Would you please help review and give the approval to backport this fix? >> >> The changes in the backport, comparing to the fix in 9, are due to >> different file structure and the different previous value of the >> property. >> >> Bug: https://bugs.openjdk.java.net/browse/JDK-8167591 >> Jdk9 change: http://hg.openjdk.java.net/jdk9/dev/jdk/rev/aabbe4707f1f >> Jdk9 review: >> http://mail.openjdk.java.net/pipermail/security-dev/2016-October/014995.html >> >> >> Jdk8 webrev: http://cr.openjdk.java.net/~igerasim/8167591/00/webrev/ >> >> With kind regards, >> Ivan >> > > This looks fine to me. > > Tony > From rob.mckenna at oracle.com Fri Oct 21 02:31:45 2016 From: rob.mckenna at oracle.com (Rob McKenna) Date: Fri, 21 Oct 2016 03:31:45 +0100 Subject: [8u-dev] Request for Review & Approval - 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar Message-ID: <20161021023145.GB2435@vimes> Hi folks, Looking for a codereview and push approval for the following: bug: https://bugs.openjdk.java.net/browse/JDK-8163304 9 changeset: http://hg.openjdk.java.net/jdk9/dev/jdk/rev/7a25dbe45e61 8 webrev: http://cr.openjdk.java.net/~robm/8163304/webrev.02/ Thanks, -Rob From sean.coffey at oracle.com Fri Oct 21 05:30:28 2016 From: sean.coffey at oracle.com (=?ISO-8859-1?Q?Se=E1n_Coffey?=) Date: Fri, 21 Oct 2016 06:30:28 +0100 Subject: [8u-dev] Request for Review & Approval - 8163304: jarsigner -verbose -verify should print the algorithms used to sign the jar In-Reply-To: <3D18888E-185F-4CF7-92C2-D5014E6544F9@oracle.com> References: <20161021023145.GB2435@vimes> <3D18888E-185F-4CF7-92C2-D5014E6544F9@oracle.com> Message-ID: Approved. Regards, Sean. On 21 October 2016 03:38:20 GMT+01:00, Wang Weijun wrote: > > >On Oct 21, 2016, at 10:31 AM, Rob McKenna >wrote: > >Hi folks, > >Looking for a codereview and push approval for the following: > >bug: https://bugs.openjdk.java.net/browse/JDK-8163304 >9 changeset: http://hg.openjdk.java.net/jdk9/dev/jdk/rev/7a25dbe45e61 >8 webrev: http://cr.openjdk.java.net/~robm/8163304/webrev.02/ > > >The code change looks good to me. > >Thanks >Max > > >Thanks, > > -Rob From vladimir.kempik at oracle.com Fri Oct 21 16:36:06 2016 From: vladimir.kempik at oracle.com (Vladimir Kempik) Date: Fri, 21 Oct 2016 19:36:06 +0300 Subject: [8u] RFA 8157184: java/lang/invoke/LFCaching/LFMultiThreadCachingTest.java failed with a fatal error Message-ID: <780bcb7a-fcc5-c754-34bd-580adfa60478@oracle.com> Hello I'd like to port this fix into 8u-dev. Bug: https://bugs.openjdk.java.net/browse/JDK-8157184 Webrev: http://cr.openjdk.java.net/~vkempik/8157184/webrev.00/ Original jdk9 changeset - http://hg.openjdk.java.net/jdk9/jdk9/hotspot/rev/35e7b0314d18 Review Thread: http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/2016-October/024775.html Thanks -Vladimir From rob.mckenna at oracle.com Fri Oct 21 17:12:52 2016 From: rob.mckenna at oracle.com (Rob McKenna) Date: Fri, 21 Oct 2016 18:12:52 +0100 Subject: [8u] RFA 8157184: java/lang/invoke/LFCaching/LFMultiThreadCachingTest.java failed with a fatal error In-Reply-To: <780bcb7a-fcc5-c754-34bd-580adfa60478@oracle.com> References: <780bcb7a-fcc5-c754-34bd-580adfa60478@oracle.com> Message-ID: <20161021171252.GA2252@vimes> Approved -Rob On 21/10/16 07:36, Vladimir Kempik wrote: > Hello > > I'd like to port this fix into 8u-dev. > > > Bug: https://bugs.openjdk.java.net/browse/JDK-8157184 > Webrev: http://cr.openjdk.java.net/~vkempik/8157184/webrev.00/ > Original jdk9 changeset - > http://hg.openjdk.java.net/jdk9/jdk9/hotspot/rev/35e7b0314d18 > Review Thread: http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/2016-October/024775.html > > Thanks > -Vladimir > From vivi.an at oracle.com Fri Oct 21 18:45:05 2016 From: vivi.an at oracle.com (Vivi An) Date: Fri, 21 Oct 2016 11:45:05 -0700 Subject: [8u] Backport Request: 7172652: With JDK 1.7 text field does not obtain focus when using mnemonic Alt/Key combin Message-ID: <580A6231.7030805@oracle.com> Hello, Could you please approve straight backport for below bug fix to 8udev ? Description: This is a bug in Label UI's key release event processing routine for key mnemonics: only one release event is captured instead of two(Alt release and the mnemonic key release). The Alt release event goes up on hierarchy and is captured by the internal frame menu for which Alt release means selection change event. / /Bug: https://bugs.openjdk.java.net/browse/JDK-7172652 JDK9 review thread: http://mail.openjdk.java.net/pipermail/jdk9-client-changes/2015-May/001219.html JDK9 changeset: http://hg.openjdk.java.net/jdk9/client/jdk/rev/f4fb05e323a0 Thank you Regards, ~ Vivi From aleksej.efimov at oracle.com Mon Oct 24 10:31:48 2016 From: aleksej.efimov at oracle.com (Aleks Efimov) Date: Mon, 24 Oct 2016 13:31:48 +0300 Subject: [8u-dev] Request for approval: 8167179: Make XSL generated namespace prefixes local to transformation process Message-ID: <60b3c3ae-9a06-bff1-cd0b-48b9704172a6@oracle.com> Hi, I'd like to backport JDK-8167179 fix to JDK8u-dev. The source changes are identical and were applied cleanly after reshuffling. Minor changes was made to regression test: It was moved to jdk repo and the execution mode with security manager was removed from jtreg header - it was added to JDK9/jaxp repo only. Test functionality has not been changed. Build with these changes was tested with JPRT and JCK tests: no failures observed JBS: https://bugs.openjdk.java.net/browse/JDK-8167179 JDK9 Changeset: http://hg.openjdk.java.net/jdk9/dev/jaxp/rev/bdafa0cc34a9 JDK9 Review Thread: http://mail.openjdk.java.net/pipermail/core-libs-dev/2016-October/044239.html Webrev with JDK8 changes: http://cr.openjdk.java.net/~aefimov/8167179/8/00 With Best Regards, Aleksej From david.buck at oracle.com Mon Oct 24 11:17:45 2016 From: david.buck at oracle.com (david buck) Date: Mon, 24 Oct 2016 20:17:45 +0900 Subject: [8u-dev] Request for approval: 8167179: Make XSL generated namespace prefixes local to transformation process In-Reply-To: <60b3c3ae-9a06-bff1-cd0b-48b9704172a6@oracle.com> References: <60b3c3ae-9a06-bff1-cd0b-48b9704172a6@oracle.com> Message-ID: <8df0dc78-b42b-44c7-6f49-2a006f0c9ccd@oracle.com> approved for backport to 8u-dev Cheers, -Buck On 2016/10/24 19:31, Aleks Efimov wrote: > Hi, > > I'd like to backport JDK-8167179 fix to JDK8u-dev. The source changes > are identical and were applied cleanly after reshuffling. > Minor changes was made to regression test: It was moved to jdk repo and > the execution mode with security manager was removed from jtreg header - > it was added to JDK9/jaxp repo only. > Test functionality has not been changed. Build with these changes was > tested with JPRT and JCK tests: no failures observed > > JBS: > https://bugs.openjdk.java.net/browse/JDK-8167179 > > JDK9 Changeset: > http://hg.openjdk.java.net/jdk9/dev/jaxp/rev/bdafa0cc34a9 > > JDK9 Review Thread: > http://mail.openjdk.java.net/pipermail/core-libs-dev/2016-October/044239.html > > > Webrev with JDK8 changes: > http://cr.openjdk.java.net/~aefimov/8167179/8/00 > > With Best Regards, > Aleksej > From ivan.gerasimov at oracle.com Mon Oct 24 17:25:49 2016 From: ivan.gerasimov at oracle.com (Ivan Gerasimov) Date: Mon, 24 Oct 2016 20:25:49 +0300 Subject: [8u-dev] Request for Approval to backport: 8149521: automatic discovery of LDAP servers with Kerberos authentication Message-ID: Hello! We've got a request to backport this fix to jdk8-dev. The unshuffled patch applies cleanly. With the fix in place, the corresponding regression test runs well on all tested platforms. Bug: https://bugs.openjdk.java.net/browse/JDK-8149521 Jdk9 change: http://hg.openjdk.java.net/jdk9/jdk9/jdk/rev/0e78d0a7a517 Jdk9 review: http://mail.openjdk.java.net/pipermail/security-dev/2016-May/013842.html Would you please approve the backport? With kind regards, Ivan From rob.mckenna at oracle.com Mon Oct 24 21:24:36 2016 From: rob.mckenna at oracle.com (Rob McKenna) Date: Mon, 24 Oct 2016 22:24:36 +0100 Subject: [8u] Backport Request: 7172652: With JDK 1.7 text field does not obtain focus when using mnemonic Alt/Key combin In-Reply-To: <580A6231.7030805@oracle.com> References: <580A6231.7030805@oracle.com> Message-ID: <20161024212436.GA5046@tecra> Approved -Rob On 21/10/16 11:45, Vivi An wrote: > Hello, > > Could you please approve straight backport for below bug fix to 8udev ? > > Description: > This is a bug in Label UI's key release event processing routine for key > mnemonics: only one release event is captured instead of two(Alt release and > the mnemonic key release). The Alt release event goes up on hierarchy and is > captured by the internal frame menu for which Alt release means selection > change event. > / > /Bug: https://bugs.openjdk.java.net/browse/JDK-7172652 > JDK9 review thread: http://mail.openjdk.java.net/pipermail/jdk9-client-changes/2015-May/001219.html > JDK9 changeset: http://hg.openjdk.java.net/jdk9/client/jdk/rev/f4fb05e323a0 > > Thank you > > Regards, > > ~ Vivi From rob.mckenna at oracle.com Mon Oct 24 21:24:56 2016 From: rob.mckenna at oracle.com (Rob McKenna) Date: Mon, 24 Oct 2016 22:24:56 +0100 Subject: [8u-dev] Request for Approval to backport: 8149521: automatic discovery of LDAP servers with Kerberos authentication In-Reply-To: References: Message-ID: <20161024212456.GB5046@tecra> Approved -Rob On 24/10/16 08:25, Ivan Gerasimov wrote: > Hello! > > We've got a request to backport this fix to jdk8-dev. > The unshuffled patch applies cleanly. > With the fix in place, the corresponding regression test runs well on all > tested platforms. > > Bug: https://bugs.openjdk.java.net/browse/JDK-8149521 > Jdk9 change: http://hg.openjdk.java.net/jdk9/jdk9/jdk/rev/0e78d0a7a517 > Jdk9 review: > http://mail.openjdk.java.net/pipermail/security-dev/2016-May/013842.html > > Would you please approve the backport? > > > With kind regards, > Ivan > From sven.ebenfeld at vaillant-group.com Tue Oct 25 11:32:27 2016 From: sven.ebenfeld at vaillant-group.com (Ebenfeld, Sven) Date: Tue, 25 Oct 2016 11:32:27 +0000 Subject: Hotspot in Mixed mode on ARM Message-ID: Hello, I've tried using the current version of jdk8u on an ARM Cortex-A9 with ThumbEE enabled. While OpenJDK 7 Hotspot uses the ThumbEE unit and goes into mixed mode, OpenJDK 8 does not. As far as I can see, the support for mixed mode on ARM has been removed in the last year. Can you tell me, why mixed mode is currently not supported on ARM? And is it planned to integrate the mixed mode back into the ARM hotspot any time soon? Regards, Sven Ebenfeld From aph at redhat.com Tue Oct 25 11:35:34 2016 From: aph at redhat.com (Andrew Haley) Date: Tue, 25 Oct 2016 12:35:34 +0100 Subject: Hotspot in Mixed mode on ARM In-Reply-To: References: Message-ID: <0573f0e0-7c12-d5dc-c394-5a87eefca073@redhat.com> On 25/10/16 12:32, Ebenfeld, Sven wrote: > I've tried using the current version of jdk8u on an ARM Cortex-A9 with > ThumbEE enabled. > While OpenJDK 7 Hotspot uses the ThumbEE unit and goes into mixed mode, > OpenJDK 8 does not. > > As far as I can see, the support for mixed mode on ARM has been removed in > the last year. > Can you tell me, why mixed mode is currently not supported on ARM? And is it > planned to integrate the mixed mode back into the ARM hotspot any time soon? It's hard to follow what you're saying. Please tell us exactly what "java -version" says, and where you got it. Please From philip.race at oracle.com Tue Oct 25 18:02:40 2016 From: philip.race at oracle.com (Phil Race) Date: Tue, 25 Oct 2016 11:02:40 -0700 Subject: [8u] Request for approval to backport 8168364: [macosx] Delete unused class NSPrintinfo Message-ID: <62792f63-31d7-68ba-30ff-0c2f562d3b1b@oracle.com> Hello, Please approve the backport of 8168364 to 8u-dev. Bug: https://bugs.openjdk.java.net/browse/JDK-8168364 9 webrev : http://cr.openjdk.java.net/~prr/8168364/ 9 patch was applied to backport to 8. Builds cleanly. -phil. From philip.race at oracle.com Tue Oct 25 18:03:52 2016 From: philip.race at oracle.com (Phil Race) Date: Tue, 25 Oct 2016 11:03:52 -0700 Subject: [8u] Request for approval to backport 8168364: [macosx] Delete unused class NSPrintinfo In-Reply-To: <62792f63-31d7-68ba-30ff-0c2f562d3b1b@oracle.com> References: <62792f63-31d7-68ba-30ff-0c2f562d3b1b@oracle.com> Message-ID: <236bf6ff-dfda-298d-5e6e-e421b1004d36@oracle.com> PS .. sorry forgot to reference the 9 review thread :- http://mail.openjdk.java.net/pipermail/2d-dev/2016-October/007824.html -phil. On 10/25/2016 11:02 AM, Phil Race wrote: > Hello, > > Please approve the backport of 8168364 to 8u-dev. > Bug: https://bugs.openjdk.java.net/browse/JDK-8168364 > 9 webrev : http://cr.openjdk.java.net/~prr/8168364/ > > 9 patch was applied to backport to 8. Builds cleanly. > > -phil. From rob.mckenna at oracle.com Tue Oct 25 18:18:01 2016 From: rob.mckenna at oracle.com (Rob McKenna) Date: Tue, 25 Oct 2016 19:18:01 +0100 Subject: [8u] Request for approval to backport 8168364: [macosx] Delete unused class NSPrintinfo In-Reply-To: <236bf6ff-dfda-298d-5e6e-e421b1004d36@oracle.com> References: <62792f63-31d7-68ba-30ff-0c2f562d3b1b@oracle.com> <236bf6ff-dfda-298d-5e6e-e421b1004d36@oracle.com> Message-ID: <20161025181801.GB4317@tecra> Approved -Rob On 25/10/16 11:03, Phil Race wrote: > PS .. sorry forgot to reference the 9 review thread :- > http://mail.openjdk.java.net/pipermail/2d-dev/2016-October/007824.html > > -phil. > > On 10/25/2016 11:02 AM, Phil Race wrote: > >Hello, > > > >Please approve the backport of 8168364 to 8u-dev. > >Bug: https://bugs.openjdk.java.net/browse/JDK-8168364 > >9 webrev : http://cr.openjdk.java.net/~prr/8168364/ > > > >9 patch was applied to backport to 8. Builds cleanly. > > > >-phil. > From robbin.ehn at oracle.com Thu Oct 27 13:02:02 2016 From: robbin.ehn at oracle.com (Robbin Ehn) Date: Thu, 27 Oct 2016 15:02:02 +0200 Subject: [8u] RFA: 8164501: Uninitialised memory in byteArrayToPacket of SharedMemoryConnection.c Message-ID: Hi all, please consider this for backport: "This patch makes sure pktHeader is not used uninitialized and that total_length is at least 11. The bug is confidential, but subject and patch says alot." Changeset applied cleanly on jdk8u-dev. (file had been moved) Passes jdk_jdi tests in 8. JDK9 commit: http://hg.openjdk.java.net/jdk9/hs/jdk/rev/11932b04e6a0 RFR thread: http://mail.openjdk.java.net/pipermail/serviceability-dev/2016-October/020612.html JDK8 (conf) bug: https://bugs.openjdk.java.net/browse/JDK-8168152 JDK9 (conf) bug: https://bugs.openjdk.java.net/browse/JDK-8164501 Thanks! /Robbin From rob.mckenna at oracle.com Thu Oct 27 19:07:45 2016 From: rob.mckenna at oracle.com (Rob McKenna) Date: Thu, 27 Oct 2016 20:07:45 +0100 Subject: [8u-communication] JDK 8u112 is now available! Message-ID: <20161027190745.GA4625@tecra> JDK 8u112 was released [0] on October 18. Thanks to all those who have contributed towards it. OpenJDK 8u112 source code is available from the jdk8u master forest via use of the 'jdk8u112-b16' 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 the next release. Note that 8u122 has now entered rampdown phase 2 and early access binaries from Oracle are available for testing. [2] Thanks, -Rob [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 From rob.mckenna at oracle.com Fri Oct 28 05:54:17 2016 From: rob.mckenna at oracle.com (Rob McKenna) Date: Fri, 28 Oct 2016 06:54:17 +0100 Subject: [8u] RFA: 8164501: Uninitialised memory in byteArrayToPacket of SharedMemoryConnection.c In-Reply-To: References: Message-ID: <20161028055417.GB27035@tecra> Approved. Please add an appropriate noreg label to the bug. -Rob On 27/10/16 03:02, Robbin Ehn wrote: > > Hi all, please consider this for backport: > > "This patch makes sure pktHeader is not used uninitialized and that total_length is at least 11. > The bug is confidential, but subject and patch says alot." > > Changeset applied cleanly on jdk8u-dev. (file had been moved) > > Passes jdk_jdi tests in 8. > > JDK9 commit: http://hg.openjdk.java.net/jdk9/hs/jdk/rev/11932b04e6a0 > RFR thread: http://mail.openjdk.java.net/pipermail/serviceability-dev/2016-October/020612.html > > JDK8 (conf) bug: https://bugs.openjdk.java.net/browse/JDK-8168152 > JDK9 (conf) bug: https://bugs.openjdk.java.net/browse/JDK-8164501 > > Thanks! > > /Robbin From volker.simonis at gmail.com Fri Oct 28 08:37:25 2016 From: volker.simonis at gmail.com (Volker Simonis) Date: Fri, 28 Oct 2016 10:37:25 +0200 Subject: CFV: New jdk8u Committer: Christoph Langer Message-ID: I hereby nominate Christoph Langer to jdk8u Committer. Christoph is a long-term member of the SAP JVM support team at SAP. He is a jdk9 committer with a broad knowledge of the entire JDK as can be seen from his record of twenty changes which he has contributed to the class library [1], jaxp [2] as well as to the hotspot repository [3]. Recently he has contributed 12 non-trivial changes to jdk8u (see below for details): JDK === 8134505: Cleanup of "TimeZone_md.c" http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/e647bfb114da 8163181: Further improvements for Unix NetworkInterface native implementation http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/8bc3c39ab16a 8162598: XSLTC transformer swallows empty namespace declaration which is needed to undeclare default namespace http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/ffa5c441e033 8165320: Small flaw when integrating 8160174 to JDK8 http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/f4bb6973e2ce 8160174: java.net.NetworkInterface - fixes and improvements for network interface listing http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/3dc438e0c8e1 8156521: Minor fixes and cleanups in NetworkInterface.c http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/c5c0728e113d 8153781: Issue in XMLScanner: EXPECTED_SQUARE_BRACKET_TO_CLOSE_INTERNAL_SUBSET when skipping large DOCTYPE section with CRLF at wrong place http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/39f7a1ab2b08 8139436: sun.security.mscapi.KeyStore might load incomplete data http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/44a6aa58f7b2 JAXP ==== 8153781: Issue in XMLScanner: EXPECTED_SQUARE_BRACKET_TO_CLOSE_INTERNAL_SUBSET when skipping large DOCTYPE section with CRLF at wrong place http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jaxp/rev/b2b2a7b4a160 8162598: XSLTC transformer swallows empty namespace declaration which is needed to undeclare default namespace http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jaxp/rev/cbb8efe13927 HOTSPOT ======= 8140244: Port fix of JDK-8075773 to AIX and possibly MacOSX http://hg.openjdk.java.net/jdk8u/jdk8u-dev/hotspot/rev/2d23269a45a0 8130910: hsperfdata file is created in wrong directory and not cleaned up if /tmp/hsperfdata_ has wrong permissions http://hg.openjdk.java.net/jdk8u/jdk8u-dev/hotspot/rev/9f8038f83a6e Votes are due by 20:00 CET, November 11th, 2016. Only current jdk8u Committers [4] are eligible to vote on this nomination. Votes must be cast in the open by replying to this mailing list. For Lazy Consensus voting instructions, see [5]. Thanks, Volker [1] http://hg.openjdk.java.net/jdk9/dev/jdk/log?rev=%28author%28%22clanger%22%29+or+desc%28%22Contributed-by%3A+christoph.langer at sap.com%22%29%29+and+not+merge%28%29 [2] http://hg.openjdk.java.net/jdk9/dev/jaxp/log?rev=%28author%28%22clanger%22%29+or+desc%28%22Contributed-by%3A+christoph.langer at sap.com%22%29%29+and+not+merge%28%29 [3] http://hg.openjdk.java.net/jdk9/dev/hotspot/log?rev=%28author%28%22clanger%22%29+or+desc%28%22Contributed-by%3A+christoph.langer at sap.com%22%29%29+and+not+merge%28%29 [4] http://openjdk.java.net/census#jdk8u [5] http://openjdk.java.net/projects/#committer-vote From jesper.wilhelmsson at oracle.com Fri Oct 28 10:57:17 2016 From: jesper.wilhelmsson at oracle.com (Jesper Wilhelmsson) Date: Fri, 28 Oct 2016 12:57:17 +0200 Subject: CFV: New jdk8u Committer: Christoph Langer In-Reply-To: References: Message-ID: <6c19d578-ad1b-5d97-49e6-2b27f00d3d65@oracle.com> Vote: Yes /Jesper Den 28/10/16 kl. 10:37, skrev Volker Simonis: > I hereby nominate Christoph Langer to jdk8u Committer. > > Christoph is a long-term member of the SAP JVM support team at SAP. > He is a jdk9 committer with a broad knowledge of the entire JDK as can > be seen from his record of twenty changes which he has contributed to > the class library [1], jaxp [2] as well as to the hotspot repository > [3]. > > Recently he has contributed 12 non-trivial changes to jdk8u (see below > for details): > > JDK > === > 8134505: Cleanup of "TimeZone_md.c" > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/e647bfb114da > > 8163181: Further improvements for Unix NetworkInterface native implementation > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/8bc3c39ab16a > > > 8162598: XSLTC transformer swallows empty namespace declaration which > is needed to undeclare default namespace > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/ffa5c441e033 > > 8165320: Small flaw when integrating 8160174 to JDK8 > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/f4bb6973e2ce > > > 8160174: java.net.NetworkInterface - fixes and improvements for > network interface listing > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/3dc438e0c8e1 > > 8156521: Minor fixes and cleanups in NetworkInterface.c > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/c5c0728e113d > > > 8153781: Issue in XMLScanner: > EXPECTED_SQUARE_BRACKET_TO_CLOSE_INTERNAL_SUBSET when skipping large > DOCTYPE section with CRLF at wrong place > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/39f7a1ab2b08 > > 8139436: sun.security.mscapi.KeyStore might load incomplete data > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/44a6aa58f7b2 > > JAXP > ==== > 8153781: Issue in XMLScanner: > EXPECTED_SQUARE_BRACKET_TO_CLOSE_INTERNAL_SUBSET when skipping large > DOCTYPE section with CRLF at wrong place > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jaxp/rev/b2b2a7b4a160 > > 8162598: XSLTC transformer swallows empty namespace declaration which > is needed to undeclare default namespace > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jaxp/rev/cbb8efe13927 > > HOTSPOT > ======= > 8140244: Port fix of JDK-8075773 to AIX and possibly MacOSX > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/hotspot/rev/2d23269a45a0 > > 8130910: hsperfdata file is created in wrong directory and not cleaned > up if /tmp/hsperfdata_ has wrong permissions > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/hotspot/rev/9f8038f83a6e > > Votes are due by 20:00 CET, November 11th, 2016. > > Only current jdk8u Committers [4] are eligible to vote on this > nomination. Votes must be cast in the open by replying to this mailing > list. > > For Lazy Consensus voting instructions, see [5]. > > Thanks, > Volker > > [1] http://hg.openjdk.java.net/jdk9/dev/jdk/log?rev=%28author%28%22clanger%22%29+or+desc%28%22Contributed-by%3A+christoph.langer at sap.com%22%29%29+and+not+merge%28%29 > [2] http://hg.openjdk.java.net/jdk9/dev/jaxp/log?rev=%28author%28%22clanger%22%29+or+desc%28%22Contributed-by%3A+christoph.langer at sap.com%22%29%29+and+not+merge%28%29 > [3] http://hg.openjdk.java.net/jdk9/dev/hotspot/log?rev=%28author%28%22clanger%22%29+or+desc%28%22Contributed-by%3A+christoph.langer at sap.com%22%29%29+and+not+merge%28%29 > [4] http://openjdk.java.net/census#jdk8u > [5] http://openjdk.java.net/projects/#committer-vote > From volker.simonis at gmail.com Fri Oct 28 12:20:50 2016 From: volker.simonis at gmail.com (Volker Simonis) Date: Fri, 28 Oct 2016 14:20:50 +0200 Subject: CFV: New jdk8u Committer: Christoph Langer In-Reply-To: References: Message-ID: Vote: yes On Fri, Oct 28, 2016 at 10:37 AM, Volker Simonis wrote: > I hereby nominate Christoph Langer to jdk8u Committer. > > Christoph is a long-term member of the SAP JVM support team at SAP. > He is a jdk9 committer with a broad knowledge of the entire JDK as can > be seen from his record of twenty changes which he has contributed to > the class library [1], jaxp [2] as well as to the hotspot repository > [3]. > > Recently he has contributed 12 non-trivial changes to jdk8u (see below > for details): > > JDK > === > 8134505: Cleanup of "TimeZone_md.c" > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/e647bfb114da > > 8163181: Further improvements for Unix NetworkInterface native implementation > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/8bc3c39ab16a > > > 8162598: XSLTC transformer swallows empty namespace declaration which > is needed to undeclare default namespace > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/ffa5c441e033 > > 8165320: Small flaw when integrating 8160174 to JDK8 > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/f4bb6973e2ce > > > 8160174: java.net.NetworkInterface - fixes and improvements for > network interface listing > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/3dc438e0c8e1 > > 8156521: Minor fixes and cleanups in NetworkInterface.c > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/c5c0728e113d > > > 8153781: Issue in XMLScanner: > EXPECTED_SQUARE_BRACKET_TO_CLOSE_INTERNAL_SUBSET when skipping large > DOCTYPE section with CRLF at wrong place > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/39f7a1ab2b08 > > 8139436: sun.security.mscapi.KeyStore might load incomplete data > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/44a6aa58f7b2 > > JAXP > ==== > 8153781: Issue in XMLScanner: > EXPECTED_SQUARE_BRACKET_TO_CLOSE_INTERNAL_SUBSET when skipping large > DOCTYPE section with CRLF at wrong place > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jaxp/rev/b2b2a7b4a160 > > 8162598: XSLTC transformer swallows empty namespace declaration which > is needed to undeclare default namespace > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jaxp/rev/cbb8efe13927 > > HOTSPOT > ======= > 8140244: Port fix of JDK-8075773 to AIX and possibly MacOSX > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/hotspot/rev/2d23269a45a0 > > 8130910: hsperfdata file is created in wrong directory and not cleaned > up if /tmp/hsperfdata_ has wrong permissions > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/hotspot/rev/9f8038f83a6e > > Votes are due by 20:00 CET, November 11th, 2016. > > Only current jdk8u Committers [4] are eligible to vote on this > nomination. Votes must be cast in the open by replying to this mailing > list. > > For Lazy Consensus voting instructions, see [5]. > > Thanks, > Volker > > [1] http://hg.openjdk.java.net/jdk9/dev/jdk/log?rev=%28author%28%22clanger%22%29+or+desc%28%22Contributed-by%3A+christoph.langer at sap.com%22%29%29+and+not+merge%28%29 > [2] http://hg.openjdk.java.net/jdk9/dev/jaxp/log?rev=%28author%28%22clanger%22%29+or+desc%28%22Contributed-by%3A+christoph.langer at sap.com%22%29%29+and+not+merge%28%29 > [3] http://hg.openjdk.java.net/jdk9/dev/hotspot/log?rev=%28author%28%22clanger%22%29+or+desc%28%22Contributed-by%3A+christoph.langer at sap.com%22%29%29+and+not+merge%28%29 > [4] http://openjdk.java.net/census#jdk8u > [5] http://openjdk.java.net/projects/#committer-vote From chris.hegarty at oracle.com Fri Oct 28 12:37:44 2016 From: chris.hegarty at oracle.com (Chris Hegarty) Date: Fri, 28 Oct 2016 13:37:44 +0100 Subject: CFV: New jdk8u Committer: Christoph Langer In-Reply-To: References: Message-ID: <76bc197f-2695-1a74-2006-6846c001a288@oracle.com> Vote: YES -Chris. On 28/10/16 09:37, Volker Simonis wrote: > I hereby nominate Christoph Langer to jdk8u Committer. > > Christoph is a long-term member of the SAP JVM support team at SAP. > He is a jdk9 committer with a broad knowledge of the entire JDK as can > be seen from his record of twenty changes which he has contributed to > the class library [1], jaxp [2] as well as to the hotspot repository > [3]. > > Recently he has contributed 12 non-trivial changes to jdk8u (see below > for details): > > JDK > === > 8134505: Cleanup of "TimeZone_md.c" > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/e647bfb114da > > 8163181: Further improvements for Unix NetworkInterface native implementation > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/8bc3c39ab16a > > > 8162598: XSLTC transformer swallows empty namespace declaration which > is needed to undeclare default namespace > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/ffa5c441e033 > > 8165320: Small flaw when integrating 8160174 to JDK8 > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/f4bb6973e2ce > > > 8160174: java.net.NetworkInterface - fixes and improvements for > network interface listing > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/3dc438e0c8e1 > > 8156521: Minor fixes and cleanups in NetworkInterface.c > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/c5c0728e113d > > > 8153781: Issue in XMLScanner: > EXPECTED_SQUARE_BRACKET_TO_CLOSE_INTERNAL_SUBSET when skipping large > DOCTYPE section with CRLF at wrong place > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/39f7a1ab2b08 > > 8139436: sun.security.mscapi.KeyStore might load incomplete data > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/44a6aa58f7b2 > > JAXP > ==== > 8153781: Issue in XMLScanner: > EXPECTED_SQUARE_BRACKET_TO_CLOSE_INTERNAL_SUBSET when skipping large > DOCTYPE section with CRLF at wrong place > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jaxp/rev/b2b2a7b4a160 > > 8162598: XSLTC transformer swallows empty namespace declaration which > is needed to undeclare default namespace > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jaxp/rev/cbb8efe13927 > > HOTSPOT > ======= > 8140244: Port fix of JDK-8075773 to AIX and possibly MacOSX > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/hotspot/rev/2d23269a45a0 > > 8130910: hsperfdata file is created in wrong directory and not cleaned > up if /tmp/hsperfdata_ has wrong permissions > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/hotspot/rev/9f8038f83a6e > > Votes are due by 20:00 CET, November 11th, 2016. > > Only current jdk8u Committers [4] are eligible to vote on this > nomination. Votes must be cast in the open by replying to this mailing > list. > > For Lazy Consensus voting instructions, see [5]. > > Thanks, > Volker > > [1] http://hg.openjdk.java.net/jdk9/dev/jdk/log?rev=%28author%28%22clanger%22%29+or+desc%28%22Contributed-by%3A+christoph.langer at sap.com%22%29%29+and+not+merge%28%29 > [2] http://hg.openjdk.java.net/jdk9/dev/jaxp/log?rev=%28author%28%22clanger%22%29+or+desc%28%22Contributed-by%3A+christoph.langer at sap.com%22%29%29+and+not+merge%28%29 > [3] http://hg.openjdk.java.net/jdk9/dev/hotspot/log?rev=%28author%28%22clanger%22%29+or+desc%28%22Contributed-by%3A+christoph.langer at sap.com%22%29%29+and+not+merge%28%29 > [4] http://openjdk.java.net/census#jdk8u > [5] http://openjdk.java.net/projects/#committer-vote > From daniel.fuchs at oracle.com Fri Oct 28 12:42:45 2016 From: daniel.fuchs at oracle.com (Daniel Fuchs) Date: Fri, 28 Oct 2016 13:42:45 +0100 Subject: CFV: New jdk8u Committer: Christoph Langer In-Reply-To: References: Message-ID: <31cbf63a-4cec-649d-6bec-962f3b75629a@oracle.com> Vote: yes -- daniel On 28/10/16 09:37, Volker Simonis wrote: > I hereby nominate Christoph Langer to jdk8u Committer. > > Christoph is a long-term member of the SAP JVM support team at SAP. > He is a jdk9 committer with a broad knowledge of the entire JDK as can > be seen from his record of twenty changes which he has contributed to > the class library [1], jaxp [2] as well as to the hotspot repository > [3]. From poonam.bajaj at oracle.com Fri Oct 28 13:37:57 2016 From: poonam.bajaj at oracle.com (Poonam Bajaj Parhar) Date: Fri, 28 Oct 2016 06:37:57 -0700 Subject: [8u] Request for approval to backport JDK-8038348: Instance field load is replaced by wrong data Phi Message-ID: <565350d9-f690-a677-6b75-fbcd49323c83@oracle.com> Hello, Please approve the backport of the fix for JDK-8038348 to 8u. JDK-8038348 : Instance field load is replaced by wrong data Phi JDK 9 changeset: http://hg.openjdk.java.net/jdk9/jdk9/hotspot/rev/2629b100a7e3 Webrev:http://cr.openjdk.java.net/~poonam/8038348/webrev.8u.00/ Review Thread: http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/2016-October/024825.html Testing: Customer verification and JPRT Thanks, Poonam From poonam.bajaj at oracle.com Fri Oct 28 13:50:13 2016 From: poonam.bajaj at oracle.com (Poonam Bajaj Parhar) Date: Fri, 28 Oct 2016 06:50:13 -0700 Subject: [8u] Request for approval to backport JDK-8140584: nmethod::oops_do_marking_epilogue always runs verification code Message-ID: <635cb9d4-55f3-3f6b-30c3-8acbf146033e@oracle.com> Hello, Please approve the backport of the fix for JDK-8140584 to 8u: JDK-8140584 : nmethod::oops_do_marking_epilogue always runs verification code webrev: http://cr.openjdk.java.net/~poonam/8140584/webrev.8u.00/ JDK 9 Changeset: http://hg.openjdk.java.net/jdk9/jdk9/hotspot/rev/11486a585ba2 8u review thread: http://mail.openjdk.java.net/pipermail/hotspot-gc-dev/2016-October/019188.html Build and Test: JPRT Thanks, Poonam From sean.coffey at oracle.com Fri Oct 28 14:28:17 2016 From: sean.coffey at oracle.com (=?UTF-8?Q?Se=c3=a1n_Coffey?=) Date: Fri, 28 Oct 2016 07:28:17 -0700 Subject: CFV: New jdk8u Committer: Christoph Langer In-Reply-To: References: Message-ID: Vote: yes regards, Sean. On 28/10/2016 01:37, Volker Simonis wrote: > I hereby nominate Christoph Langer to jdk8u Committer. > > Christoph is a long-term member of the SAP JVM support team at SAP. > He is a jdk9 committer with a broad knowledge of the entire JDK as can > be seen from his record of twenty changes which he has contributed to > the class library [1], jaxp [2] as well as to the hotspot repository > [3]. > > Recently he has contributed 12 non-trivial changes to jdk8u (see below > for details): > > JDK > === > 8134505: Cleanup of "TimeZone_md.c" > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/e647bfb114da > > 8163181: Further improvements for Unix NetworkInterface native implementation > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/8bc3c39ab16a > > > 8162598: XSLTC transformer swallows empty namespace declaration which > is needed to undeclare default namespace > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/ffa5c441e033 > > 8165320: Small flaw when integrating 8160174 to JDK8 > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/f4bb6973e2ce > > > 8160174: java.net.NetworkInterface - fixes and improvements for > network interface listing > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/3dc438e0c8e1 > > 8156521: Minor fixes and cleanups in NetworkInterface.c > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/c5c0728e113d > > > 8153781: Issue in XMLScanner: > EXPECTED_SQUARE_BRACKET_TO_CLOSE_INTERNAL_SUBSET when skipping large > DOCTYPE section with CRLF at wrong place > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/39f7a1ab2b08 > > 8139436: sun.security.mscapi.KeyStore might load incomplete data > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/44a6aa58f7b2 > > JAXP > ==== > 8153781: Issue in XMLScanner: > EXPECTED_SQUARE_BRACKET_TO_CLOSE_INTERNAL_SUBSET when skipping large > DOCTYPE section with CRLF at wrong place > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jaxp/rev/b2b2a7b4a160 > > 8162598: XSLTC transformer swallows empty namespace declaration which > is needed to undeclare default namespace > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jaxp/rev/cbb8efe13927 > > HOTSPOT > ======= > 8140244: Port fix of JDK-8075773 to AIX and possibly MacOSX > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/hotspot/rev/2d23269a45a0 > > 8130910: hsperfdata file is created in wrong directory and not cleaned > up if /tmp/hsperfdata_ has wrong permissions > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/hotspot/rev/9f8038f83a6e > > Votes are due by 20:00 CET, November 11th, 2016. > > Only current jdk8u Committers [4] are eligible to vote on this > nomination. Votes must be cast in the open by replying to this mailing > list. > > For Lazy Consensus voting instructions, see [5]. > > Thanks, > Volker > > [1] http://hg.openjdk.java.net/jdk9/dev/jdk/log?rev=%28author%28%22clanger%22%29+or+desc%28%22Contributed-by%3A+christoph.langer at sap.com%22%29%29+and+not+merge%28%29 > [2] http://hg.openjdk.java.net/jdk9/dev/jaxp/log?rev=%28author%28%22clanger%22%29+or+desc%28%22Contributed-by%3A+christoph.langer at sap.com%22%29%29+and+not+merge%28%29 > [3] http://hg.openjdk.java.net/jdk9/dev/hotspot/log?rev=%28author%28%22clanger%22%29+or+desc%28%22Contributed-by%3A+christoph.langer at sap.com%22%29%29+and+not+merge%28%29 > [4] http://openjdk.java.net/census#jdk8u > [5] http://openjdk.java.net/projects/#committer-vote From david.buck at oracle.com Fri Oct 28 14:54:51 2016 From: david.buck at oracle.com (david buck) Date: Fri, 28 Oct 2016 23:54:51 +0900 Subject: [8u] Request for approval to backport JDK-8140584: nmethod::oops_do_marking_epilogue always runs verification code In-Reply-To: <635cb9d4-55f3-3f6b-30c3-8acbf146033e@oracle.com> References: <635cb9d4-55f3-3f6b-30c3-8acbf146033e@oracle.com> Message-ID: <18a8e804-b2d4-861a-09e6-37c2e629e4e9@oracle.com> approved for push to 8u-dev Cheers, -Buck On 2016/10/28 22:50, Poonam Bajaj Parhar wrote: > Hello, > > Please approve the backport of the fix for JDK-8140584 to 8u: > > JDK-8140584 : > nmethod::oops_do_marking_epilogue always runs verification code > webrev: http://cr.openjdk.java.net/~poonam/8140584/webrev.8u.00/ > JDK 9 Changeset: > http://hg.openjdk.java.net/jdk9/jdk9/hotspot/rev/11486a585ba2 > 8u review thread: > http://mail.openjdk.java.net/pipermail/hotspot-gc-dev/2016-October/019188.html > > Build and Test: JPRT > > Thanks, > Poonam > From david.buck at oracle.com Fri Oct 28 14:56:49 2016 From: david.buck at oracle.com (david buck) Date: Fri, 28 Oct 2016 23:56:49 +0900 Subject: [8u] Request for approval to backport JDK-8038348: Instance field load is replaced by wrong data Phi In-Reply-To: <565350d9-f690-a677-6b75-fbcd49323c83@oracle.com> References: <565350d9-f690-a677-6b75-fbcd49323c83@oracle.com> Message-ID: approved for push to 8u-dev Cheers, -Buck On 2016/10/28 22:37, Poonam Bajaj Parhar wrote: > Hello, > > Please approve the backport of the fix for JDK-8038348 to 8u. > > JDK-8038348 : Instance > field load is replaced by wrong data Phi > JDK 9 changeset: > http://hg.openjdk.java.net/jdk9/jdk9/hotspot/rev/2629b100a7e3 > Webrev:http://cr.openjdk.java.net/~poonam/8038348/webrev.8u.00/ > Review Thread: > http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/2016-October/024825.html > > Testing: Customer verification and JPRT > > Thanks, > Poonam > > From aleksej.efimov at oracle.com Fri Oct 28 14:59:53 2016 From: aleksej.efimov at oracle.com (Aleks Efimov) Date: Fri, 28 Oct 2016 17:59:53 +0300 Subject: CFV: New jdk8u Committer: Christoph Langer In-Reply-To: References: Message-ID: <3192b8a4-76fd-7776-8593-781439ee5c2a@oracle.com> Vote: yes On 28/10/16 11:37, Volker Simonis wrote: > I hereby nominate Christoph Langer to jdk8u Committer. > > Christoph is a long-term member of the SAP JVM support team at SAP. > He is a jdk9 committer with a broad knowledge of the entire JDK as can > be seen from his record of twenty changes which he has contributed to > the class library [1], jaxp [2] as well as to the hotspot repository > [3]. > > Recently he has contributed 12 non-trivial changes to jdk8u (see below > for details): > > JDK > === > 8134505: Cleanup of "TimeZone_md.c" > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/e647bfb114da > > 8163181: Further improvements for Unix NetworkInterface native implementation > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/8bc3c39ab16a > > > 8162598: XSLTC transformer swallows empty namespace declaration which > is needed to undeclare default namespace > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/ffa5c441e033 > > 8165320: Small flaw when integrating 8160174 to JDK8 > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/f4bb6973e2ce > > > 8160174: java.net.NetworkInterface - fixes and improvements for > network interface listing > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/3dc438e0c8e1 > > 8156521: Minor fixes and cleanups in NetworkInterface.c > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/c5c0728e113d > > > 8153781: Issue in XMLScanner: > EXPECTED_SQUARE_BRACKET_TO_CLOSE_INTERNAL_SUBSET when skipping large > DOCTYPE section with CRLF at wrong place > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/39f7a1ab2b08 > > 8139436: sun.security.mscapi.KeyStore might load incomplete data > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/44a6aa58f7b2 > > JAXP > ==== > 8153781: Issue in XMLScanner: > EXPECTED_SQUARE_BRACKET_TO_CLOSE_INTERNAL_SUBSET when skipping large > DOCTYPE section with CRLF at wrong place > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jaxp/rev/b2b2a7b4a160 > > 8162598: XSLTC transformer swallows empty namespace declaration which > is needed to undeclare default namespace > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jaxp/rev/cbb8efe13927 > > HOTSPOT > ======= > 8140244: Port fix of JDK-8075773 to AIX and possibly MacOSX > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/hotspot/rev/2d23269a45a0 > > 8130910: hsperfdata file is created in wrong directory and not cleaned > up if /tmp/hsperfdata_ has wrong permissions > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/hotspot/rev/9f8038f83a6e > > Votes are due by 20:00 CET, November 11th, 2016. > > Only current jdk8u Committers [4] are eligible to vote on this > nomination. Votes must be cast in the open by replying to this mailing > list. > > For Lazy Consensus voting instructions, see [5]. > > Thanks, > Volker > > [1] http://hg.openjdk.java.net/jdk9/dev/jdk/log?rev=%28author%28%22clanger%22%29+or+desc%28%22Contributed-by%3A+christoph.langer at sap.com%22%29%29+and+not+merge%28%29 > [2] http://hg.openjdk.java.net/jdk9/dev/jaxp/log?rev=%28author%28%22clanger%22%29+or+desc%28%22Contributed-by%3A+christoph.langer at sap.com%22%29%29+and+not+merge%28%29 > [3] http://hg.openjdk.java.net/jdk9/dev/hotspot/log?rev=%28author%28%22clanger%22%29+or+desc%28%22Contributed-by%3A+christoph.langer at sap.com%22%29%29+and+not+merge%28%29 > [4] http://openjdk.java.net/census#jdk8u > [5] http://openjdk.java.net/projects/#committer-vote From vladimir.x.ivanov at oracle.com Fri Oct 28 15:04:39 2016 From: vladimir.x.ivanov at oracle.com (Vladimir Ivanov) Date: Fri, 28 Oct 2016 18:04:39 +0300 Subject: CFV: New jdk8u Committer: Christoph Langer In-Reply-To: References: Message-ID: <8a8e5b0f-0938-ee4f-6acd-b6224d285ff8@oracle.com> Vote: yes Best regards, Vladimir Ivanov On 10/28/16 11:37 AM, Volker Simonis wrote: > I hereby nominate Christoph Langer to jdk8u Committer. From vladimir.kozlov at oracle.com Fri Oct 28 15:37:18 2016 From: vladimir.kozlov at oracle.com (Vladimir Kozlov) Date: Fri, 28 Oct 2016 08:37:18 -0700 Subject: CFV: New jdk8u Committer: Christoph Langer In-Reply-To: References: Message-ID: <581370AE.7070804@oracle.com> Vote: yes On 10/28/16 1:37 AM, Volker Simonis wrote: > I hereby nominate Christoph Langer to jdk8u Committer. > > Christoph is a long-term member of the SAP JVM support team at SAP. > He is a jdk9 committer with a broad knowledge of the entire JDK as can > be seen from his record of twenty changes which he has contributed to > the class library [1], jaxp [2] as well as to the hotspot repository > [3]. > > Recently he has contributed 12 non-trivial changes to jdk8u (see below > for details): > > JDK > === > 8134505: Cleanup of "TimeZone_md.c" > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/e647bfb114da > > 8163181: Further improvements for Unix NetworkInterface native implementation > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/8bc3c39ab16a > > > 8162598: XSLTC transformer swallows empty namespace declaration which > is needed to undeclare default namespace > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/ffa5c441e033 > > 8165320: Small flaw when integrating 8160174 to JDK8 > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/f4bb6973e2ce > > > 8160174: java.net.NetworkInterface - fixes and improvements for > network interface listing > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/3dc438e0c8e1 > > 8156521: Minor fixes and cleanups in NetworkInterface.c > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/c5c0728e113d > > > 8153781: Issue in XMLScanner: > EXPECTED_SQUARE_BRACKET_TO_CLOSE_INTERNAL_SUBSET when skipping large > DOCTYPE section with CRLF at wrong place > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/39f7a1ab2b08 > > 8139436: sun.security.mscapi.KeyStore might load incomplete data > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/44a6aa58f7b2 > > JAXP > ==== > 8153781: Issue in XMLScanner: > EXPECTED_SQUARE_BRACKET_TO_CLOSE_INTERNAL_SUBSET when skipping large > DOCTYPE section with CRLF at wrong place > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jaxp/rev/b2b2a7b4a160 > > 8162598: XSLTC transformer swallows empty namespace declaration which > is needed to undeclare default namespace > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jaxp/rev/cbb8efe13927 > > HOTSPOT > ======= > 8140244: Port fix of JDK-8075773 to AIX and possibly MacOSX > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/hotspot/rev/2d23269a45a0 > > 8130910: hsperfdata file is created in wrong directory and not cleaned > up if /tmp/hsperfdata_ has wrong permissions > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/hotspot/rev/9f8038f83a6e > > Votes are due by 20:00 CET, November 11th, 2016. > > Only current jdk8u Committers [4] are eligible to vote on this > nomination. Votes must be cast in the open by replying to this mailing > list. > > For Lazy Consensus voting instructions, see [5]. > > Thanks, > Volker > > [1] http://hg.openjdk.java.net/jdk9/dev/jdk/log?rev=%28author%28%22clanger%22%29+or+desc%28%22Contributed-by%3A+christoph.langer at sap.com%22%29%29+and+not+merge%28%29 > [2] http://hg.openjdk.java.net/jdk9/dev/jaxp/log?rev=%28author%28%22clanger%22%29+or+desc%28%22Contributed-by%3A+christoph.langer at sap.com%22%29%29+and+not+merge%28%29 > [3] http://hg.openjdk.java.net/jdk9/dev/hotspot/log?rev=%28author%28%22clanger%22%29+or+desc%28%22Contributed-by%3A+christoph.langer at sap.com%22%29%29+and+not+merge%28%29 > [4] http://openjdk.java.net/census#jdk8u > [5] http://openjdk.java.net/projects/#committer-vote > From david.holmes at oracle.com Fri Oct 28 17:25:35 2016 From: david.holmes at oracle.com (David Holmes) Date: Sat, 29 Oct 2016 03:25:35 +1000 Subject: CFV: New jdk8u Committer: Christoph Langer In-Reply-To: References: Message-ID: Vote: yes David On 28/10/2016 6:37 PM, Volker Simonis wrote: > I hereby nominate Christoph Langer to jdk8u Committer. > > Christoph is a long-term member of the SAP JVM support team at SAP. > He is a jdk9 committer with a broad knowledge of the entire JDK as can > be seen from his record of twenty changes which he has contributed to > the class library [1], jaxp [2] as well as to the hotspot repository > [3]. > > Recently he has contributed 12 non-trivial changes to jdk8u (see below > for details): > > JDK > === > 8134505: Cleanup of "TimeZone_md.c" > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/e647bfb114da > > 8163181: Further improvements for Unix NetworkInterface native implementation > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/8bc3c39ab16a > > > 8162598: XSLTC transformer swallows empty namespace declaration which > is needed to undeclare default namespace > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/ffa5c441e033 > > 8165320: Small flaw when integrating 8160174 to JDK8 > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/f4bb6973e2ce > > > 8160174: java.net.NetworkInterface - fixes and improvements for > network interface listing > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/3dc438e0c8e1 > > 8156521: Minor fixes and cleanups in NetworkInterface.c > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/c5c0728e113d > > > 8153781: Issue in XMLScanner: > EXPECTED_SQUARE_BRACKET_TO_CLOSE_INTERNAL_SUBSET when skipping large > DOCTYPE section with CRLF at wrong place > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/39f7a1ab2b08 > > 8139436: sun.security.mscapi.KeyStore might load incomplete data > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/44a6aa58f7b2 > > JAXP > ==== > 8153781: Issue in XMLScanner: > EXPECTED_SQUARE_BRACKET_TO_CLOSE_INTERNAL_SUBSET when skipping large > DOCTYPE section with CRLF at wrong place > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jaxp/rev/b2b2a7b4a160 > > 8162598: XSLTC transformer swallows empty namespace declaration which > is needed to undeclare default namespace > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jaxp/rev/cbb8efe13927 > > HOTSPOT > ======= > 8140244: Port fix of JDK-8075773 to AIX and possibly MacOSX > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/hotspot/rev/2d23269a45a0 > > 8130910: hsperfdata file is created in wrong directory and not cleaned > up if /tmp/hsperfdata_ has wrong permissions > http://hg.openjdk.java.net/jdk8u/jdk8u-dev/hotspot/rev/9f8038f83a6e > > Votes are due by 20:00 CET, November 11th, 2016. > > Only current jdk8u Committers [4] are eligible to vote on this > nomination. Votes must be cast in the open by replying to this mailing > list. > > For Lazy Consensus voting instructions, see [5]. > > Thanks, > Volker > > [1] http://hg.openjdk.java.net/jdk9/dev/jdk/log?rev=%28author%28%22clanger%22%29+or+desc%28%22Contributed-by%3A+christoph.langer at sap.com%22%29%29+and+not+merge%28%29 > [2] http://hg.openjdk.java.net/jdk9/dev/jaxp/log?rev=%28author%28%22clanger%22%29+or+desc%28%22Contributed-by%3A+christoph.langer at sap.com%22%29%29+and+not+merge%28%29 > [3] http://hg.openjdk.java.net/jdk9/dev/hotspot/log?rev=%28author%28%22clanger%22%29+or+desc%28%22Contributed-by%3A+christoph.langer at sap.com%22%29%29+and+not+merge%28%29 > [4] http://openjdk.java.net/census#jdk8u > [5] http://openjdk.java.net/projects/#committer-vote > From serguei.spitsyn at oracle.com Fri Oct 28 19:33:42 2016 From: serguei.spitsyn at oracle.com (serguei.spitsyn at oracle.com) Date: Fri, 28 Oct 2016 12:33:42 -0700 Subject: CFV: New jdk8u Committer: Christoph Langer In-Reply-To: References: Message-ID: Vote: yes From omajid at redhat.com Mon Oct 31 19:43:18 2016 From: omajid at redhat.com (Omair Majid) Date: Mon, 31 Oct 2016 15:43:18 -0400 Subject: the Minimum set when install jdk8u In-Reply-To: References: Message-ID: <20161031194317.GA25511@redhat.com> * Diaz Soho [2016-10-20 09:29]: > when I finish build jdk8u, and try to install to a local directory, > I find the folder /usr/local/jvm/ include a lots of files , the size > is very large. > > is there any minimun set that I can select to install to my embedded platform? > for example: > if I would like to run "java -jar " on my embedded > platform, then how to figure out the minimum set that should be > installed ? I can't help with the produce-minimal-jre-for-my-app problem, but here are some ideas for trimming your JDK image if you are still having trouble with this: - If you don't care about debugging the JVM itself, remove all debugging information. Look for .debuginfo or .diz files and delete them. - If you don't care about using a debugger with java to debug the JDK itself, you can get rid of src.zip. - You probably can live without docs, right? demo/, man/ and samples/ can be removed. - If your application doesn't use the JDK bits (doesn't compile code and doesn't need to use tools.jar), then you can try using just the JRE, not the JDK (j2re-image instead of j2sdk-image). In my tests, I end up with a 99M JRE image rather than a 300M JDK image. Thanks, Omair -- PGP Key: 66484681 (http://pgp.mit.edu/) Fingerprint = F072 555B 0A17 3957 4E95 0056 F286 F14F 6648 4681 From sybersnake at gmail.com Mon Oct 31 20:20:00 2016 From: sybersnake at gmail.com (Jon V.) Date: Mon, 31 Oct 2016 16:20:00 -0400 Subject: NIO excessive Integer allocation Message-ID: Hi everyone, EpollSelectorImpl.java #updateSelectedKeys() has a major object creation problem. It creates Integers in order to get the key from the map. Changing Integer.valueOf() to new Integer() might allow escape-analysis to remove the object. Otherwise, fdToKey could be change to a Map and a new mutable Integer class could be created so only a single object would need to be created per select() instead of every Key. /** * Update the keys whose fd's have been selected by the epoll. * Add the ready keys to the ready queue. */ private int updateSelectedKeys() { int entries = pollWrapper.updated; int numKeysUpdated = 0; for (int i=0; i References: Message-ID: Best to being this to nio-dev. On 31/10/2016 20:20, Jon V. wrote: > Hi everyone, > > EpollSelectorImpl.java #updateSelectedKeys() has a major object creation > problem. It creates Integers in order to get the key from the map. > > Changing Integer.valueOf() to new Integer() might allow escape-analysis to > remove the object. > > Otherwise, fdToKey could be change to a Map and a > new mutable Integer class could be created so only a single object would > need to be created per select() instead of every Key. > > /** > * Update the keys whose fd's have been selected by the epoll. > * Add the ready keys to the ready queue. > */ > private int updateSelectedKeys() { > int entries = pollWrapper.updated; > int numKeysUpdated = 0; > for (int i=0; i int nextFD = pollWrapper.getDescriptor(i); > SelectionKeyImpl ski = fdToKey.get(Integer.valueOf(nextFD)); > // ski is null in the case of an interrupt > if (ski != null) { > int rOps = pollWrapper.getEventOps(i); > if (selectedKeys.contains(ski)) { > if (ski.channel.translateAndSetReadyOps(rOps, ski)) { > numKeysUpdated++; > } > } else { > ski.channel.translateAndSetReadyOps(rOps, ski); > if ((ski.nioReadyOps() & ski.nioInterestOps()) != 0) { > selectedKeys.add(ski); > numKeysUpdated++; > } > } > } > } > return numKeysUpdated; > }