Re: RFR: 8275721: Name of UTC timezone in a locale changes depending on previous code [v2]

2021-12-03 Thread Naoto Sato
> Fixing time zone name provider for CLDR. In some cases, COMPAT's `UTC` > display names were incorrectly substituted for CLDR. The reason it worked > fine after `zh-Hant-HK` was that by loading names for `zh-Hant-HK`, the names > for `zh-Hant` were cached and hit for the following `zh-MO` name

Re: RFR: 8003417: WeakHashMap$HashIterator removes wrong entry

2021-12-03 Thread Stuart Marks
On Sat, 20 Nov 2021 10:08:41 GMT, Jaikiran Pai wrote: > Can I please get a review for this change which proposes to fix the issue > reported in https://bugs.openjdk.java.net/browse/JDK-8003417? > > The issue notes that this is applicable for `WeakHashMap` which have `null` > keys. However, the

Re: RFR: 8003417: WeakHashMap$HashIterator removes wrong entry

2021-12-03 Thread Stuart Marks
On Sat, 4 Dec 2021 00:02:04 GMT, Brent Christian wrote: >> Can I please get a review for this change which proposes to fix the issue >> reported in https://bugs.openjdk.java.net/browse/JDK-8003417? >> >> The issue notes that this is applicable for `WeakHashMap` which have `null` >> keys. Howev

RFR: 8275721: Name of UTC timezone in a locale changes depending on previous code

2021-12-03 Thread Naoto Sato
Fixing time zone name provider for CLDR. In some cases, COMPAT's `UTC` display names were incorrectly substituted for CLDR. The reason it worked fine after `zh-Hant-HK` was that by loading names for `zh-Hant-HK`, the names for `zh-Hant` were cached and hit for the following `zh-MO` name retrieva

Re: RFR: 8003417: WeakHashMap$HashIterator removes wrong entry

2021-12-03 Thread Brent Christian
On Sat, 20 Nov 2021 10:08:41 GMT, Jaikiran Pai wrote: > Can I please get a review for this change which proposes to fix the issue > reported in https://bugs.openjdk.java.net/browse/JDK-8003417? > > The issue notes that this is applicable for `WeakHashMap` which have `null` > keys. However, the

Re: RFR: 8278080: Add --with-cacerts-src='user cacerts folder' to enable deterministic cacerts generation [v2]

2021-12-03 Thread Rajan Halade
On Thu, 2 Dec 2021 12:13:03 GMT, Andrew Leonard wrote: >> Addition of a configure option --with-cacerts-src='user cacerts folder' to >> allow developers to specify their own cacerts PEM folder for generation of >> the cacerts store using the deterministic openjdk GenerateCacerts tool. >> >> Si

Re: RFR: 8278080: Add --with-cacerts-src='user cacerts folder' to enable deterministic cacerts generation

2021-12-03 Thread Sergey Bylokhov
On Thu, 2 Dec 2021 10:55:57 GMT, Andrew Leonard wrote: > This is the case at Adoptium for example, which uses the Mozilla trusted CA > certs. But they didn't think skipping this test was too strong a step? For example validation of the certs expiration is quite useful. I tried to update the te

Integrated: 8275821: Optimize random number generators developed in JDK-8248862 using Math.unsignedMultiplyHigh()

2021-12-03 Thread Vamsi Parasa
On Tue, 2 Nov 2021 03:02:42 GMT, Vamsi Parasa wrote: > This change optimizes random number generators using > Math.unsignedMultiplyHigh() This pull request has now been integrated. Changeset: 38f525e9 Author:vamsi-parasa Committer: Sandhya Viswanathan URL: https://git.openjdk.java

Re: RFR: 8276766: Enable jar and jmod to produce deterministic timestamped content [v17]

2021-12-03 Thread Andrew Leonard
On Fri, 3 Dec 2021 10:05:43 GMT, Andrew Leonard wrote: >> Add a new --source-date (epoch seconds) option to jar and jmod >> to allow specification of time to use for created/updated jar/jmod entries. >> This then allows the ability to make the content deterministic. >> >> Signed-off-by: Andre

Re: RFR: 8275821: Optimize random number generators developed in JDK-8248862 using Math.unsignedMultiplyHigh() [v4]

2021-12-03 Thread Vamsi Parasa
On Fri, 3 Dec 2021 17:19:17 GMT, Paul Sandoz wrote: >> @JimLaskey Could you please review this PR which optimizes three of the >> Pseudo-Random Number Generators you implemented in >> https://bugs.openjdk.java.net/browse/JDK-8248862 ? > > @vamsi-parasa you can now integrate this and then I or s

Re: RFR: 8276766: Enable jar and jmod to produce deterministic timestamped content [v14]

2021-12-03 Thread John Neffenger
On Thu, 2 Dec 2021 01:53:47 GMT, John Neffenger wrote: >>> I'm testing it now. So far, it's working great! >>> >>> I found one boundary problem. The magic instant of `1980-01-01T00:00:00Z` >>> triggers an extended header with the wrong time: >>> >>> ``` >>> file last modified on (DOS date/ti

Re: RFR: 8275821: Optimize random number generators developed in JDK-8248862 using Math.unsignedMultiplyHigh() [v4]

2021-12-03 Thread Paul Sandoz
On Thu, 2 Dec 2021 20:56:46 GMT, Vamsi Parasa wrote: >> Vamsi Parasa has updated the pull request incrementally with one additional >> commit since the last revision: >> >> add seeds for the random generators to eliminate run-to-run variance > > @JimLaskey Could you please review this PR whic

Integrated: 8278171: [vectorapi] Mask incorrectly computed for zero extending cast

2021-12-03 Thread Mai Đặng Quân Anh
On Wed, 1 Dec 2021 12:03:15 GMT, Mai Đặng Quân Anh wrote: > When doing an unsigned upcast, we perform a signed cast followed by a bitwise > and operation to clip the extended signed bit. The sign clip mask is > currently calculated incorrectly, the correct mask should have the number of > lea

Re: RFR: 8278171: [vectorapi] Mask incorrectly computed for zero extending cast

2021-12-03 Thread Paul Sandoz
On Fri, 3 Dec 2021 03:44:09 GMT, Mai Đặng Quân Anh wrote: > Thanks a lot for your support and review. Do I need another review here? No need for this one, for next PR for the intrinsics will likely need two HotSpot reviewers. - PR: https://git.openjdk.java.net/jdk/pull/6634

Re: RFR: 8003417: WeakHashMap$HashIterator removes wrong entry

2021-12-03 Thread Roger Riggs
On Sat, 20 Nov 2021 10:08:41 GMT, Jaikiran Pai wrote: > Can I please get a review for this change which proposes to fix the issue > reported in https://bugs.openjdk.java.net/browse/JDK-8003417? > > The issue notes that this is applicable for `WeakHashMap` which have `null` > keys. However, the

RFR: 8278044: ObjectInputStream methods invoking the OIF.CFG.getSerialFilterFactory() silent about error cases.

2021-12-03 Thread Roger Riggs
The specification of ObjectInputStream constructors that invoke `ObjectInputFilter.Config.getSerialFilterFactory()` do not mention exceptions that may be thrown by the apply() method. In both constructors, add the following to the paragraph the describes invoking the factory: * When the f

Re: RFR: 8278087: Deserialization filter and filter factory property error reporting under specified

2021-12-03 Thread Roger Riggs
On Wed, 1 Dec 2021 18:19:05 GMT, Roger Riggs wrote: > The effects of invalid values of `jdk.serialFilter` and > `jdk.serialFilterFactory` properties are > incompletely specified. The behavior for invalid values of the properties is > different and > use an unconventional exception type, `Excep

Re: RFR: 8276766: Enable jar and jmod to produce deterministic timestamped content [v17]

2021-12-03 Thread Alan Bateman
On Mon, 29 Nov 2021 17:07:52 GMT, Alan Bateman wrote: >> @andrew-m-leonard I see you've add several APIs to ZipEntry in this PR. I >> think this part will require discussion as it's not immediately clear that >> we should over burden the ZipEntry API as proposed. > >> @AlanBateman yes, see abov

Re: RFR: 8276766: Enable jar and jmod to produce deterministic timestamped content [v14]

2021-12-03 Thread Andrew Leonard
On Thu, 2 Dec 2021 01:53:47 GMT, John Neffenger wrote: >>> I'm testing it now. So far, it's working great! >>> >>> I found one boundary problem. The magic instant of `1980-01-01T00:00:00Z` >>> triggers an extended header with the wrong time: >>> >>> ``` >>> file last modified on (DOS date/ti

Integrated: 8278144: Javadoc for MemorySegment::set/MemorySegment::setAtIndex is missing throws tag

2021-12-03 Thread Maurizio Cimadamore
On Thu, 2 Dec 2021 12:26:33 GMT, Maurizio Cimadamore wrote: > This patch adds missing `@throws` tags to the javadoc of `MemorySegment::set` > and `MemorySegment::setAtIndex`. > These methods can fail with `UnsupportedOperationException` if the segment is > read-only. This pull request has now

Integrated: 8278205: jlink plugins should dump .class file in debug mode

2021-12-03 Thread Athijegannathan Sundararajan
On Fri, 3 Dec 2021 10:03:30 GMT, Athijegannathan Sundararajan wrote: > jlink plugins dump .class files in debug mode. jpackage tests already set > jlink.debug property to true. This pull request has now been integrated. Changeset: ba2a8e5a Author:Athijegannathan Sundararajan URL:

Re: RFR: 8275821: Optimize random number generators developed in JDK-8248862 using Math.unsignedMultiplyHigh() [v6]

2021-12-03 Thread Jim Laskey
On Fri, 3 Dec 2021 02:14:51 GMT, Vamsi Parasa wrote: >> This change optimizes random number generators using >> Math.unsignedMultiplyHigh() > > Vamsi Parasa has updated the pull request incrementally with one additional > commit since the last revision: > > minor changes to the benchmark LG

Re: RFR: 8278205: jlink plugins should dump .class file in debug mode

2021-12-03 Thread Jim Laskey
On Fri, 3 Dec 2021 10:03:30 GMT, Athijegannathan Sundararajan wrote: > jlink plugins dump .class files in debug mode. jpackage tests already set > jlink.debug property to true. LGTM - Marked as reviewed by jlaskey (Reviewer). PR: https://git.openjdk.java.net/jdk/pull/6696

Re: RFR: 8278185: Custom JRE cannot find non-ASCII named module inside

2021-12-03 Thread Jim Laskey
On Fri, 3 Dec 2021 07:29:17 GMT, Toshio Nakamura wrote: > Could you review this fix? > > Problem: > Custom JRE generated by jlink cannot find non-ASCII named modules included > inside the JRE. > > Cause and fix: > If module or package name was composed by ASCII then non-ASCII characters, > Im

RFR: 8278205: jlink plugins should dump .class file in debug mode

2021-12-03 Thread Athijegannathan Sundararajan
jlink plugins dump .class files in debug mode. jpackage tests already set jlink.debug property to true. - Commit messages: - 8278205: jlink plugins should dump .class file in debug mode Changes: https://git.openjdk.java.net/jdk/pull/6696/files Webrev: https://webrevs.openjdk.java.

Re: RFR: 8276766: Enable jar and jmod to produce deterministic timestamped content [v17]

2021-12-03 Thread Andrew Leonard
> Add a new --source-date (epoch seconds) option to jar and jmod to > allow specification of time to use for created/updated jar/jmod entries. This > then allows the ability to make the content deterministic. > > Signed-off-by: Andrew Leonard Andrew Leonard has updated the pull request with a

Re: RFR: 8277992: Add fast jdk_svc subtests to jdk:tier3

2021-12-03 Thread Alan Bateman
On Thu, 2 Dec 2021 12:15:51 GMT, David Holmes wrote: > I've solicited feedback from core-libs folk as this affects them the most. At > present we, Oracle, run the jdk_svc tests as part of hotspot testing, but > this change will suddenly cause jdk testing to include them. That is probably > not

Integrated: 8278163: --with-cacerts-src variable resolved after GenerateCacerts recipe setup

2021-12-03 Thread Andrew Leonard
On Thu, 2 Dec 2021 21:07:30 GMT, Andrew Leonard wrote: > PR https://github.com/openjdk/jdk/pull/6647 resolved the GENDATA_CACERTS_SRC > fir --with-cacerts-src after the recipe, which meant the dependencies were > wrong. > This PR moves it before the recipe. > > Signed-off-by: Andrew Leonard