Re: RFR: JDK-8310502 : Optimization for j.l.Long.fastUUID() [v7]

2023-06-21 Thread Alan Bateman
On Thu, 22 Jun 2023 00:13:06 GMT, 温绍锦 wrote: >> By optimizing the implementation of java.lang.Long#fastUUID, the performance >> of the java.util.UUID#toString method can be significantly improved. >> >> The following are the test results of JMH: >> >> Benchmark Mode Cnt

Integrated: 8309853: StructuredTaskScope.join description improvements

2023-06-21 Thread Alan Bateman
On Mon, 12 Jun 2023 14:32:07 GMT, Alan Bateman wrote: > StructuredTaskScope's class description introduces the join method as waiting > for all subtasks to finish but the API docs for join/joinUntil are phrased in > terms of waiting for all threads to finish. ShutdownOnXXX join/joinUntil >

Re: RFR: JDK-8310502 : Optimization for j.l.Long.fastUUID() [v7]

2023-06-21 Thread Chen Liang
On Thu, 22 Jun 2023 00:13:06 GMT, 温绍锦 wrote: >> By optimizing the implementation of java.lang.Long#fastUUID, the performance >> of the java.util.UUID#toString method can be significantly improved. >> >> The following are the test results of JMH: >> >> Benchmark Mode Cnt

Re: RFR: 8308780: Fix the Java Integer types on Windows [v6]

2023-06-21 Thread Julian Waters
> On Windows, the basic Java Integer types are defined as long and __int64 > respectively. In particular, the former is rather problematic since it breaks > compilation as the Visual C++ becomes stricter and more compliant with every > release, which means the way Windows code treats long as a

Re: RFR: 8308780: Fix the Java Integer types on Windows [v5]

2023-06-21 Thread Julian Waters
> On Windows, the basic Java Integer types are defined as long and __int64 > respectively. In particular, the former is rather problematic since it breaks > compilation as the Visual C++ becomes stricter and more compliant with every > release, which means the way Windows code treats long as a

Re: RFR: 8310242: Clarify the name parameter to Class::forName [v7]

2023-06-21 Thread David Holmes
On Thu, 22 Jun 2023 00:33:33 GMT, Mandy Chung wrote: >> This PR clarifies the spec of the 3-arg Class::forName regarding the format >> of the name for an array type which is of the form: one or more of "[" + >> binary name of the element type + ";'. > > Mandy Chung has updated the pull

Re: RFR: 8310242: Clarify the name parameter to Class::forName [v6]

2023-06-21 Thread Mandy Chung
On Wed, 21 Jun 2023 23:06:23 GMT, Chen Liang wrote: >> Mandy Chung has updated the pull request with a new target base due to a >> merge or a rebase. The incremental webrev excludes the unrelated changes >> brought in by the merge/rebase. The pull request contains eight additional >> commits

Re: RFR: JDK-8310502 : Optimization for j.l.Long.fastUUID() [v7]

2023-06-21 Thread Chen Liang
On Thu, 22 Jun 2023 00:13:06 GMT, 温绍锦 wrote: >> By optimizing the implementation of java.lang.Long#fastUUID, the performance >> of the java.util.UUID#toString method can be significantly improved. >> >> The following are the test results of JMH: >> >> Benchmark Mode Cnt

Re: RFR: 8309819: Clarify API note in Class::getName and MethodType::toMethodDescriptorString [v7]

2023-06-21 Thread Chen Liang
> The API specification for descriptorString not being a strict inverse of > Class::forName and MethodType::fromDescriptorString are not entirely correct. > > 1. Class::descriptorString was never an inverse of Class::forName, which > takes a binary name instead. Class::getName was a partial

Re: RFR: 8309819: Clarify API note in Class::getName and MethodType::toMethodDescriptorString [v5]

2023-06-21 Thread Chen Liang
On Thu, 15 Jun 2023 10:01:31 GMT, Alan Bateman wrote: >> Chen Liang has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Update the note for getName > > src/java.base/share/classes/java/lang/invoke/MethodType.java line 1233: > >> 1231:

Re: RFR: JDK-8310502 : Optimization for j.l.Long.fastUUID() [v7]

2023-06-21 Thread Roger Riggs
On Thu, 22 Jun 2023 00:13:06 GMT, 温绍锦 wrote: >> By optimizing the implementation of java.lang.Long#fastUUID, the performance >> of the java.util.UUID#toString method can be significantly improved. >> >> The following are the test results of JMH: >> >> Benchmark Mode Cnt

RFR: 8310234: Refactor Locale tests to use JUnit

2023-06-21 Thread Justin Lu
Please review this PR as apart of [JDK-8307843](https://bugs.openjdk.org/browse/JDK-8307843) which refactors some tests in Locale to use JUnit. Other cleanup and small changes are included as well. More refactoring in Locale tests will be done in separate PRs. If the test had a bugN.java

Re: RFR: JDK-8310502 : Optimization for j.l.Long.fastUUID() [v7]

2023-06-21 Thread 温绍锦
On Thu, 22 Jun 2023 01:03:11 GMT, Roger Riggs wrote: > fyi, the title of this PR need to match exactly the title of the bug > [JDK-8310502](https://bugs.openjdk.org/browse/JDK-8310502). The mismatch is > an Integration blocker. (See the comment in the description). i have made the changes. is

Re: RFR: JDK-8310502 : Optimization for j.l.Long.fastUUID() [v7]

2023-06-21 Thread Roger Riggs
On Thu, 22 Jun 2023 00:13:06 GMT, 温绍锦 wrote: >> By optimizing the implementation of java.lang.Long#fastUUID, the performance >> of the java.util.UUID#toString method can be significantly improved. >> >> The following are the test results of JMH: >> >> Benchmark Mode Cnt

Re: RFR: JDK-8310571: Use inline @return tag on java.util.Objects

2023-06-21 Thread Stuart Marks
On Thu, 22 Jun 2023 00:19:03 GMT, Joe Darcy wrote: > Small cleanup, minor differences in the wording of portions of > toString(Object, String), nonNull(Object), requireNonNullElse, and > requireNonNullElseGet. Looks good. I had to refresh my understanding of the exact behavior of the inline

Re: RFR: 8310242: Clarify the name parameter to Class::forName [v6]

2023-06-21 Thread Chen Liang
On Wed, 21 Jun 2023 22:02:16 GMT, Mandy Chung wrote: >> This PR clarifies the spec of the 3-arg Class::forName regarding the format >> of the name for an array type which is of the form: one or more of "[" + >> binary name of the element type + ";'. > > Mandy Chung has updated the pull

Re: RFR: 8310242: Clarify the name parameter to Class::forName [v7]

2023-06-21 Thread Chen Liang
On Thu, 22 Jun 2023 00:33:33 GMT, Mandy Chung wrote: >> This PR clarifies the spec of the 3-arg Class::forName regarding the format >> of the name for an array type which is of the form: one or more of "[" + >> binary name of the element type + ";'. > > Mandy Chung has updated the pull

Re: RFR: 8309819: Clarify API note in Class::getName and MethodType::toMethodDescriptorString [v6]

2023-06-21 Thread Chen Liang
> The API specification for descriptorString not being a strict inverse of > Class::forName and MethodType::fromDescriptorString are not entirely correct. > > 1. Class::descriptorString was never an inverse of Class::forName, which > takes a binary name instead. Class::getName was a partial

Re: RFR: 8310242: Clarify the name parameter to Class::forName [v7]

2023-06-21 Thread Mandy Chung
> This PR clarifies the spec of the 3-arg Class::forName regarding the format > of the name for an array type which is of the form: one or more of "[" + > binary name of the element type + ";'. Mandy Chung has updated the pull request incrementally with one additional commit since the last

Re: RFR: 8310502 : optimization for UUID#toString [v3]

2023-06-21 Thread 温绍锦
On Wed, 21 Jun 2023 13:32:32 GMT, 温绍锦 wrote: >> 温绍锦 has updated the pull request incrementally with one additional commit >> since the last revision: >> >> 8310502 : hex literal > > * benchmark code >

RFR: JDK-8310571: Use inline @return tag on java.util.Objects

2023-06-21 Thread Joe Darcy
Small cleanup, minor differences in the wording of portions of toString(Object, String), nonNull(Object), requireNonNullElse, and requireNonNullElseGet. - Commit messages: - JDK-8310571: Use inline @return tag on java.util.Objects Changes:

Re: RFR: 8310242: Clarify the name parameter to Class::forName [v6]

2023-06-21 Thread David Holmes
On Wed, 21 Jun 2023 22:02:16 GMT, Mandy Chung wrote: >> This PR clarifies the spec of the 3-arg Class::forName regarding the format >> of the name for an array type which is of the form: one or more of "[" + >> binary name of the element type + ";'. > > Mandy Chung has updated the pull

Re: RFR: 8310502 : optimization for UUID#toString [v7]

2023-06-21 Thread 温绍锦
On Wed, 21 Jun 2023 15:21:45 GMT, Roger Riggs wrote: > CDS can initialize arrays efficiently. The Long class already uses CDS to > initialize the cache of Long values. See the LongCache code for an example of > initializing from CDS with a fallback to direct initialization. You could > move

Re: RFR: 8310502 : optimization for UUID#toString [v7]

2023-06-21 Thread 温绍锦
> By optimizing the implementation of java.lang.Long#fastUUID, the performance > of the java.util.UUID#toString method can be significantly improved. > > The following are the test results of JMH: > > Benchmark Mode Cnt Score Error Units >

Re: RFR: 8310405: Linker.Option.firstVariadicArg should specify which index values are valid [v2]

2023-06-21 Thread Jorn Vernee
On Wed, 21 Jun 2023 07:34:20 GMT, Per Minborg wrote: >> Possible suggestion/thing to try: use a bullet list to spell out all cases >> for `index`. E.g. we know there's index == 0 (all variadic). Then we know >> there's index = N (no variadic). Then there's index == m, 0 < m < N - which >>

Re: RFR: 8310405: Linker.Option.firstVariadicArg should specify which index values are valid [v2]

2023-06-21 Thread Jorn Vernee
> Improve the specification of `Linker.Option.firstVariadicArg`, by specifying > more clearly which index values are valid. Jorn Vernee has updated the pull request incrementally with one additional commit since the last revision: polish doc, review comments - Changes: - all:

Re: RFR: 8308995: Update Network IO JFR events to be static mirror events

2023-06-21 Thread Alan Bateman
On Wed, 21 Jun 2023 16:59:22 GMT, Stuart Marks wrote: > Are we using a convention of `implRead` or `readImpl`? Either is ok with me, > but I think we had been using `readImpl` and similar elsewhere. This code is already using implXXX so it's just be consistent. - PR Review

Re: RFR: 8308995: Update Network IO JFR events to be static mirror events

2023-06-21 Thread Stuart Marks
On Wed, 21 Jun 2023 09:46:35 GMT, Alan Bateman wrote: >> The socket read/write JFR events currently use instrumentation of java.base >> code using templates in the jdk.jfr modules. This results in some java.base >> code residing in the jdk.jfr module which is undesirable. >> >> JDK19 added

RFR: 8308995: Update Network IO JFR events to be static mirror events

2023-06-21 Thread Tim Prinzing
The socket read/write JFR events currently use instrumentation of java.base code using templates in the jdk.jfr modules. This results in some java.base code residing in the jdk.jfr module which is undesirable. JDK19 added static support for event classes. The old instrumentor classes should be

Re: RFR: 8308995: Update Network IO JFR events to be static mirror events

2023-06-21 Thread Alan Bateman
On Tue, 6 Jun 2023 19:39:31 GMT, Tim Prinzing wrote: > The socket read/write JFR events currently use instrumentation of java.base > code using templates in the jdk.jfr modules. This results in some java.base > code residing in the jdk.jfr module which is undesirable. > > JDK19 added static

Re: RFR: 8310242: Clarify the name parameter to Class::forName [v6]

2023-06-21 Thread Mandy Chung
On Wed, 21 Jun 2023 22:02:16 GMT, Mandy Chung wrote: >> This PR clarifies the spec of the 3-arg Class::forName regarding the format >> of the name for an array type which is of the form: one or more of "[" + >> binary name of the element type + ";'. > > Mandy Chung has updated the pull

Re: RFR: 8310242: Clarify the name parameter to Class::forName [v6]

2023-06-21 Thread Mandy Chung
> This PR clarifies the spec of the 3-arg Class::forName regarding the format > of the name for an array type which is of the form: one or more of "[" + > binary name of the element type + ";'. Mandy Chung has updated the pull request with a new target base due to a merge or a rebase. The

Re: RFR: JDK-8310550: Adjust references to rt.jar

2023-06-21 Thread David Holmes
On Wed, 21 Jun 2023 15:18:19 GMT, Matthias Baesken wrote: > There are a few references to rt.jar in comments and in the codebase itself. > Some of them might be removed or adjusted. Mostly seems okay - a couple of things need further adjusting I think. Thanks.

Re: Integrated: 8310586: ProblemList java/lang/ScopedValue/StressStackOverflow.java#default with virtual threads on linux-all

2023-06-21 Thread Daniel D . Daugherty
On Wed, 21 Jun 2023 21:05:11 GMT, Mikael Vidstedt wrote: >> A trivial fix to ProblemList >> java/lang/ScopedValue/StressStackOverflow.java#default with virtual threads >> on linux-all > > Marked as reviewed by mikael (Reviewer). @vidmik - Thanks for your review also. - PR

Re: Integrated: 8310586: ProblemList java/lang/ScopedValue/StressStackOverflow.java#default with virtual threads on linux-all

2023-06-21 Thread Daniel D . Daugherty
On Wed, 21 Jun 2023 21:02:58 GMT, David Holmes wrote: >> A trivial fix to ProblemList >> java/lang/ScopedValue/StressStackOverflow.java#default with virtual threads >> on linux-all > > Marked as reviewed by dholmes (Reviewer). @dholmes-ora - Thanks for the lightning fast review!

Integrated: 8310586: ProblemList java/lang/ScopedValue/StressStackOverflow.java#default with virtual threads on linux-all

2023-06-21 Thread Daniel D . Daugherty
On Wed, 21 Jun 2023 20:59:43 GMT, Daniel D. Daugherty wrote: > A trivial fix to ProblemList > java/lang/ScopedValue/StressStackOverflow.java#default with virtual threads > on linux-all This pull request has now been integrated. Changeset: ac44ef19 Author:Daniel D. Daugherty URL:

Re: Integrated: 8310586: ProblemList java/lang/ScopedValue/StressStackOverflow.java#default with virtual threads on linux-all

2023-06-21 Thread Mikael Vidstedt
On Wed, 21 Jun 2023 20:59:43 GMT, Daniel D. Daugherty wrote: > A trivial fix to ProblemList > java/lang/ScopedValue/StressStackOverflow.java#default with virtual threads > on linux-all Marked as reviewed by mikael (Reviewer). - PR Review:

Re: Integrated: 8310586: ProblemList java/lang/ScopedValue/StressStackOverflow.java#default with virtual threads on linux-all

2023-06-21 Thread David Holmes
On Wed, 21 Jun 2023 20:59:43 GMT, Daniel D. Daugherty wrote: > A trivial fix to ProblemList > java/lang/ScopedValue/StressStackOverflow.java#default with virtual threads > on linux-all Marked as reviewed by dholmes (Reviewer). - PR Review:

Integrated: 8310586: ProblemList java/lang/ScopedValue/StressStackOverflow.java#default with virtual threads on linux-all

2023-06-21 Thread Daniel D . Daugherty
A trivial fix to ProblemList java/lang/ScopedValue/StressStackOverflow.java#default with virtual threads on linux-all - Commit messages: - 8310586: ProblemList java/lang/ScopedValue/StressStackOverflow.java#default with virtual threads on linux-all Changes:

Re: RFR: JDK-8310380: Handle problems in core-related tests on macOS when codesign tool does not work

2023-06-21 Thread Chris Plummer
On Tue, 20 Jun 2023 13:23:16 GMT, Matthias Baesken wrote: > Currently, a number of tests fail on macOS because they miss the core file > (e.g. serviceability/sa/TestJmapCore.java). > The reason is that configure detects on some setups that codesign does not > work ("checking if debug mode

Re: RFR: JDK-8310380: Handle problems in core-related tests on macOS when codesign tool does not work

2023-06-21 Thread Chris Plummer
On Tue, 20 Jun 2023 13:23:16 GMT, Matthias Baesken wrote: > Currently, a number of tests fail on macOS because they miss the core file > (e.g. serviceability/sa/TestJmapCore.java). > The reason is that configure detects on some setups that codesign does not > work ("checking if debug mode

Re: RFR: 8310502 : optimization for UUID#toString [v4]

2023-06-21 Thread 温绍锦
On Wed, 21 Jun 2023 19:02:59 GMT, ExE Boss wrote: >> src/java.base/share/classes/java/lang/Long.java line 563: >> >>> 561: StringUTF16.putChar(buf, 7, (byte) i3); >>> 562: StringUTF16.putChar(buf, 8, '-'); >>> 563: StringUTF16.putChar(buf, 9, (byte) (i4 >>

Re: RFR: 8310502 : optimization for UUID#toString [v6]

2023-06-21 Thread 温绍锦
> By optimizing the implementation of java.lang.Long#fastUUID, the performance > of the java.util.UUID#toString method can be significantly improved. > > The following are the test results of JMH: > > Benchmark Mode Cnt Score Error Units >

Re: RFR: 8310502 : optimization for UUID#toString [v5]

2023-06-21 Thread 温绍锦
> By optimizing the implementation of java.lang.Long#fastUUID, the performance > of the java.util.UUID#toString method can be significantly improved. > > The following are the test results of JMH: > > Benchmark Mode Cnt Score Error Units >

Re: RFR: JDK-8310380: Handle problems in core-related tests on macOS when codesign tool does not work

2023-06-21 Thread Chris Plummer
On Tue, 20 Jun 2023 13:23:16 GMT, Matthias Baesken wrote: > Currently, a number of tests fail on macOS because they miss the core file > (e.g. serviceability/sa/TestJmapCore.java). > The reason is that configure detects on some setups that codesign does not > work ("checking if debug mode

Re: RFR: 8310502 : optimization for UUID#toString [v4]

2023-06-21 Thread Glavo
On Wed, 21 Jun 2023 14:39:18 GMT, 温绍锦 wrote: >> By optimizing the implementation of java.lang.Long#fastUUID, the performance >> of the java.util.UUID#toString method can be significantly improved. >> >> The following are the test results of JMH: >> >> Benchmark Mode Cnt

Re: RFR: 8310502 : optimization for UUID#toString [v4]

2023-06-21 Thread ExE Boss
On Wed, 21 Jun 2023 14:53:22 GMT, Roger Riggs wrote: >> 温绍锦 has updated the pull request incrementally with one additional commit >> since the last revision: >> >> add annotation Stable > > src/java.base/share/classes/java/lang/Long.java line 563: > >> 561:

Re: RFR: JDK-8310453: Update javadoc of java.lang.Object

2023-06-21 Thread Joe Darcy
On Wed, 21 Jun 2023 13:00:38 GMT, Alan Bateman wrote: >> That is, a clear connection to what's being described by that `@implSpec`. > > This method has an existing apiNote and implSpec. I suspect Joe meant to add > this sentence to the apiNote, not the implSpec. I did mean to add the

Re: [jdk21] RFR: 8307858: [REDO] JDK-8307194 Add make target for optionally building a complete set of all JDK and hotspot libjvm static libraries

2023-06-21 Thread Jiangli Zhou
On Wed, 21 Jun 2023 17:09:58 GMT, Kevin Rushforth wrote: > Since this Enhancement was rejected for JDK 21, this PR should be closed. Closing without integration accordingly, thanks. - PR Comment: https://git.openjdk.org/jdk21/pull/26#issuecomment-1601273074

[jdk21] Withdrawn: 8307858: [REDO] JDK-8307194 Add make target for optionally building a complete set of all JDK and hotspot libjvm static libraries

2023-06-21 Thread Jiangli Zhou
On Fri, 16 Jun 2023 20:36:07 GMT, Jiangli Zhou wrote: > 8307858: [REDO] JDK-8307194 Add make target for optionally building a > complete set of all JDK and hotspot libjvm static libraries This pull request has been closed without being integrated. - PR:

Re: [jdk21] RFR: 8307858: [REDO] JDK-8307194 Add make target for optionally building a complete set of all JDK and hotspot libjvm static libraries

2023-06-21 Thread Kevin Rushforth
On Fri, 16 Jun 2023 20:36:07 GMT, Jiangli Zhou wrote: > 8307858: [REDO] JDK-8307194 Add make target for optionally building a > complete set of all JDK and hotspot libjvm static libraries Since this Enhancement was rejected for JDK 21, this PR should be closed. - PR Comment:

Re: RFR: JDK-8310550: Adjust references to rt.jar

2023-06-21 Thread Erik Joelsson
On Wed, 21 Jun 2023 15:18:19 GMT, Matthias Baesken wrote: > There are a few references to rt.jar in comments and in the codebase itself. > Some of them might be removed or adjusted. The update to Java.gmk is good. - Marked as reviewed by erikj (Reviewer). PR Review:

Re: RFR: 8309853: StructuredTaskScope.join description improvements

2023-06-21 Thread Joe Darcy
On Mon, 12 Jun 2023 14:32:07 GMT, Alan Bateman wrote: > StructuredTaskScope's class description introduces the join method as waiting > for all subtasks to finish but the API docs for join/joinUntil are phrased in > terms of waiting for all threads to finish. ShutdownOnXXX join/joinUntil >

Re: RFR: 8310033: Clarify return value of Java Time compareTo methods [v3]

2023-06-21 Thread Roger Riggs
> In java.time packages, clarify timeline order javadoc to mention "before" and > "after" in the value of the `compareTo` method return values. > Add javadoc @see tags to isBefore and isAfter methods > > Replace use of "negative" and positive with "less than zero" and "greater > than zero" in

Re: RFR: 8310033: Clarify return value of Java Time compareTo methods [v2]

2023-06-21 Thread Roger Riggs
On Sun, 18 Jun 2023 20:24:07 GMT, Stephen Colebourne wrote: >> Roger Riggs has updated the pull request with a new target base due to a >> merge or a rebase. The incremental webrev excludes the unrelated changes >> brought in by the merge/rebase. The pull request contains five additional >>

Re: RFR: JDK-8310380: Handle problems in core-related tests on macOS when codesign tool does not work

2023-06-21 Thread Christoph Langer
On Tue, 20 Jun 2023 13:23:16 GMT, Matthias Baesken wrote: > Currently, a number of tests fail on macOS because they miss the core file > (e.g. serviceability/sa/TestJmapCore.java). > The reason is that configure detects on some setups that codesign does not > work ("checking if debug mode

RFR: JDK-8310550: Adjust references to rt.jar

2023-06-21 Thread Matthias Baesken
There are a few references to rt.jar in comments and in the codebase itself. Some of them might be removed or adjusted. - Commit messages: - JDK-8310550 Changes: https://git.openjdk.org/jdk/pull/14593/files Webrev: https://webrevs.openjdk.org/?repo=jdk=14593=00 Issue:

Re: RFR: 8310502 : optimization for UUID#toString [v4]

2023-06-21 Thread Roger Riggs
On Wed, 21 Jun 2023 14:31:20 GMT, Chen Liang wrote: >> Not sure if this can be applied but some months ago, I optimized Bits to use >> VarHandles rather than explicitly shifting bits around. This gave us a >> significant performance increase: >> >>

Re: RFR: JDK-8310380: Handle problems in core-related tests on macOS when codesign tool does not work

2023-06-21 Thread Lutz Schmidt
On Tue, 20 Jun 2023 13:23:16 GMT, Matthias Baesken wrote: > Currently, a number of tests fail on macOS because they miss the core file > (e.g. serviceability/sa/TestJmapCore.java). > The reason is that configure detects on some setups that codesign does not > work ("checking if debug mode

Re: RFR: 8310530: PipedOutputStream.flush() accesses sink racily [v2]

2023-06-21 Thread Daniel Fuchs
On Wed, 21 Jun 2023 14:27:26 GMT, Chen Liang wrote: >> src/java.base/share/classes/java/io/PipedOutputStream.java line 166: >> >>> 164: @Override >>> 165: public synchronized void flush() throws IOException { >>> 166: PipedInputStream sink = this.sink; >> >> Suggestion: >> >>

Re: RFR: 8310530: PipedOutputStream.flush() accesses sink racily [v2]

2023-06-21 Thread Sergey Tsypanov
> Just a tiny clean-up to remove racy read within synchronized method Sergey Tsypanov has updated the pull request incrementally with one additional commit since the last revision: Update src/java.base/share/classes/java/io/PipedOutputStream.java Co-authored-by: liach

[jdk21] Integrated: 8310053: VarHandle and slice handle derived from layout are lacking alignment check

2023-06-21 Thread Jorn Vernee
On Wed, 21 Jun 2023 00:08:03 GMT, Jorn Vernee wrote: > Hi all, > > This pull request contains a backport of commit > [e022e876](https://github.com/openjdk/jdk/commit/e022e876543b65b531027662326f35b497861f33) > from the [openjdk/jdk](https://git.openjdk.org/jdk) repository. > > The commit

Re: RFR: 8310502 : optimization for UUID#toString [v4]

2023-06-21 Thread Roger Riggs
On Wed, 21 Jun 2023 14:39:18 GMT, 温绍锦 wrote: >> By optimizing the implementation of java.lang.Long#fastUUID, the performance >> of the java.util.UUID#toString method can be significantly improved. >> >> The following are the test results of JMH: >> >> Benchmark Mode Cnt

Re: RFR: 8310502 : optimization for UUID#toString [v4]

2023-06-21 Thread 温绍锦
> By optimizing the implementation of java.lang.Long#fastUUID, the performance > of the java.util.UUID#toString method can be significantly improved. > > The following are the test results of JMH: > > Benchmark Mode Cnt Score Error Units >

Re: RFR: 8310502 : optimization for UUID#toString [v4]

2023-06-21 Thread Chen Liang
On Wed, 21 Jun 2023 14:13:34 GMT, Per Minborg wrote: >>> > Another thing to try is moving fastUUID out of Long - moving to an array >>> > of precomputed hex values means it is not tied to Long internals anymore. >>> >>> A note about `@Stable`: `Integer.digits` and `HEX256` are not, and they

Re: RFR: 8310530: PipedOutputStream.flush() accesses sink racily

2023-06-21 Thread Chen Liang
On Wed, 21 Jun 2023 14:03:23 GMT, Chen Liang wrote: >> Just a tiny clean-up to remove racy read within synchronized method > > src/java.base/share/classes/java/io/PipedOutputStream.java line 166: > >> 164: @Override >> 165: public synchronized void flush() throws IOException { >> 166:

Re: RFR: 8310502 : optimization for UUID#toString [v3]

2023-06-21 Thread Per Minborg
On Wed, 21 Jun 2023 09:48:54 GMT, 温绍锦 wrote: >>> Another thing to try is moving fastUUID out of Long - moving to an array of >>> precomputed hex values means it is not tied to Long internals anymore. >> >> A note about `@Stable`: `Integer.digits` and `HEX256` are not, and they >> might see

Re: RFR: JDK-8310453: Update javadoc of java.lang.Object

2023-06-21 Thread Chen Liang
On Tue, 20 Jun 2023 17:13:26 GMT, Joe Darcy wrote: > I had occasion to read over the javadoc sources in java.lang.Object recently > and noticed a few items that could be updated. > > There are some new or expanded API notes referring to methods in > java.util.Objects. I added these references

Re: [jdk21] RFR: 8310053: VarHandle and slice handle derived from layout are lacking alignment check

2023-06-21 Thread Maurizio Cimadamore
On Wed, 21 Jun 2023 00:08:03 GMT, Jorn Vernee wrote: > Hi all, > > This pull request contains a backport of commit > [e022e876](https://github.com/openjdk/jdk/commit/e022e876543b65b531027662326f35b497861f33) > from the [openjdk/jdk](https://git.openjdk.org/jdk) repository. > > The commit

Re: RFR: 8310530: PipedOutputStream.flush() accesses sink racily

2023-06-21 Thread Chen Liang
On Wed, 21 Jun 2023 14:01:33 GMT, Sergey Tsypanov wrote: > Just a tiny clean-up to remove racy read within synchronized method src/java.base/share/classes/java/io/PipedOutputStream.java line 166: > 164: @Override > 165: public synchronized void flush() throws IOException { > 166:

RFR: 8310530: PipedOutputStream.flush() accesses sink racily

2023-06-21 Thread Sergey Tsypanov
Just a tiny clean-up to remove racy read within synchronized method - Commit messages: - 8310530: PipedOutputStream.flush() accesses sink racily Changes: https://git.openjdk.org/jdk/pull/14589/files Webrev: https://webrevs.openjdk.org/?repo=jdk=14589=00 Issue:

Re: RFR: 8310502 : optimization for UUID#toString [v3]

2023-06-21 Thread 温绍锦
On Wed, 21 Jun 2023 11:06:08 GMT, 温绍锦 wrote: >> By optimizing the implementation of java.lang.Long#fastUUID, the performance >> of the java.util.UUID#toString method can be significantly improved. >> >> The following are the test results of JMH: >> >> Benchmark Mode Cnt

Re: RFR: JDK-8310453: Update javadoc of java.lang.Object

2023-06-21 Thread Alan Bateman
On Wed, 21 Jun 2023 11:49:54 GMT, Pavel Rappo wrote: >> src/java.base/share/classes/java/lang/Object.java line 260: >> >>> 258: * } >>> 259: * The {@link java.util.Objects#toIdentityString(Object) >>> 260: * Objects.toIdentityString} method returns the string for an >> >> That

Re: RFR: JDK-8310267: Javadoc for Class#isPrimitive() is incorrect regarding Class objects for primitives

2023-06-21 Thread Alan Bateman
On Wed, 21 Jun 2023 00:00:54 GMT, Joe Darcy wrote: > Correct misstatement that the Class object for a primitive type can only be > be access via fields like java.lang.Integer.TYPE. src/java.base/share/classes/java/lang/Class.java line 823: > 821: * > 822: * @apiNote > 823: *

Re: RFR: 8308780: Fix the Java Integer types on Windows [v4]

2023-06-21 Thread Julian Waters
On Thu, 1 Jun 2023 11:49:24 GMT, Julian Waters wrote: >> On Windows, the basic Java Integer types are defined as long and __int64 >> respectively. In particular, the former is rather problematic since it >> breaks compilation as the Visual C++ becomes stricter and more compliant >> with every

Re: RFR: 8308780: Fix the Java Integer types on Windows [v4]

2023-06-21 Thread Daniel Jeliński
On Thu, 1 Jun 2023 11:49:24 GMT, Julian Waters wrote: >> On Windows, the basic Java Integer types are defined as long and __int64 >> respectively. In particular, the former is rather problematic since it >> breaks compilation as the Visual C++ becomes stricter and more compliant >> with every

Re: RFR: JDK-8310453: Update javadoc of java.lang.Object

2023-06-21 Thread Pavel Rappo
On Tue, 20 Jun 2023 17:13:26 GMT, Joe Darcy wrote: > I had occasion to read over the javadoc sources in java.lang.Object recently > and noticed a few items that could be updated. > > There are some new or expanded API notes referring to methods in > java.util.Objects. I added these references

Re: RFR: JDK-8310453: Update javadoc of java.lang.Object

2023-06-21 Thread Pavel Rappo
On Wed, 21 Jun 2023 11:39:43 GMT, Pavel Rappo wrote: >> I had occasion to read over the javadoc sources in java.lang.Object recently >> and noticed a few items that could be updated. >> >> There are some new or expanded API notes referring to methods in >> java.util.Objects. I added these

Re: RFR: JDK-8310453: Update javadoc of java.lang.Object

2023-06-21 Thread Pavel Rappo
On Tue, 20 Jun 2023 17:13:26 GMT, Joe Darcy wrote: > I had occasion to read over the javadoc sources in java.lang.Object recently > and noticed a few items that could be updated. > > There are some new or expanded API notes referring to methods in > java.util.Objects. I added these references

Re: RFR: JDK-8310453: Update javadoc of java.lang.Object

2023-06-21 Thread Pavel Rappo
On Tue, 20 Jun 2023 17:13:26 GMT, Joe Darcy wrote: > I had occasion to read over the javadoc sources in java.lang.Object recently > and noticed a few items that could be updated. > > There are some new or expanded API notes referring to methods in > java.util.Objects. I added these references

Re: RFR: 8310502 : optimization for UUID#toString [v3]

2023-06-21 Thread 温绍锦
> By optimizing the implementation of java.lang.Long#fastUUID, the performance > of the java.util.UUID#toString method can be significantly improved. > > The following are the test results of JMH: > > Benchmark Mode Cnt Score Error Units >

Re: RFR: 8310502 : optimization for UUID#toString [v2]

2023-06-21 Thread Raffaello Giulietti
On Wed, 21 Jun 2023 10:11:09 GMT, 温绍锦 wrote: >> By optimizing the implementation of java.lang.Long#fastUUID, the performance >> of the java.util.UUID#toString method can be significantly improved. >> >> The following are the test results of JMH: >> >> Benchmark Mode Cnt

Re: RFR: 8310502 : optimization for UUID#toString [v2]

2023-06-21 Thread 温绍锦
> By optimizing the implementation of java.lang.Long#fastUUID, the performance > of the java.util.UUID#toString method can be significantly improved. > > The following are the test results of JMH: > > Benchmark Mode Cnt Score Error Units >

Re: RFR: 8310242: Clarify the name parameter to Class::forName [v4]

2023-06-21 Thread Chen Liang
On Wed, 21 Jun 2023 09:54:53 GMT, Alan Bateman wrote: >> Binary name is a long-standing behavior. It's a spec bug. > >> Binary name is a long-standing behavior. It's a spec bug. > > Yes, I view this PR as just specifying long standing behavior. The name used > for array class may be

Re: RFR: 8310242: Clarify the name parameter to Class::forName [v4]

2023-06-21 Thread Alan Bateman
On Tue, 20 Jun 2023 17:45:03 GMT, Mandy Chung wrote: > Binary name is a long-standing behavior. It's a spec bug. Yes, I view this PR as just specifying long standing behavior. The name used for array class may be surprising but I don't think it can be changed. - PR Review

Re: RFR: 8310502 : optimization for UUID#toString

2023-06-21 Thread 温绍锦
On Wed, 21 Jun 2023 08:58:53 GMT, Chen Liang wrote: > > Another thing to try is moving fastUUID out of Long - moving to an array of > > precomputed hex values means it is not tied to Long internals anymore. > > A note about `@Stable`: `Integer.digits` and `HEX256` are not, and they might >

Re: RFR: 8308899: Introduce Classfile.Context and improve Classfile.Option(s) [v22]

2023-06-21 Thread Adam Sotona
> Classfile context object and multi-state options have been discussed at > https://mail.openjdk.org/pipermail/classfile-api-dev/2023-May/000321.html > This patch implements the proposed changes in Classfile API and fixes all > affected code across JDK sources and tests. > > Please review. > >

Re: RFR: 8310502 : optimization for UUID#toString

2023-06-21 Thread Chen Liang
On Wed, 21 Jun 2023 08:40:43 GMT, 温绍锦 wrote: >> src/java.base/share/classes/java/lang/Long.java line 97: >> >>> 95: + (lo < 10 ? '0' + lo : 'a' + lo - 10)); >>> 96: } >>> 97: } >> >> Are you checking the impact on startup? I guess I would have expected to see

Re: RFR: 8310502 : optimization for UUID#toString

2023-06-21 Thread 温绍锦
On Wed, 21 Jun 2023 07:58:11 GMT, Alan Bateman wrote: >> By optimizing the implementation of java.lang.Long#fastUUID, the performance >> of the java.util.UUID#toString method can be significantly improved. >> >> The following are the test results of JMH: >> >> Benchmark

[jdk21] RFR: 8310191: com/sun/tools/attach/warnings/DynamicLoadWarningTest.java second failure on AIX

2023-06-21 Thread Matthias Baesken
8310191: com/sun/tools/attach/warnings/DynamicLoadWarningTest.java second failure on AIX - Commit messages: - Backport 6a63badd8ea3e79cd9fc3cb33aff499fc9a6d3f1 Changes: https://git.openjdk.org/jdk21/pull/45/files Webrev: https://webrevs.openjdk.org/?repo=jdk21=45=00 Issue:

Re: RFR: 8308780: Fix the Java Integer types on Windows [v4]

2023-06-21 Thread Julian Waters
On Thu, 1 Jun 2023 11:49:24 GMT, Julian Waters wrote: >> On Windows, the basic Java Integer types are defined as long and __int64 >> respectively. In particular, the former is rather problematic since it >> breaks compilation as the Visual C++ becomes stricter and more compliant >> with every

[jdk21] Integrated: 8309937: Add @sealedGraph for some Panama FFM interfaces

2023-06-21 Thread Per Minborg
On Mon, 19 Jun 2023 14:58:13 GMT, Per Minborg wrote: > Hi all, > > This pull request contains a backport of commit > [b412fc79](https://github.com/openjdk/jdk/commit/b412fc79c3c2548df10918090beedaf6b2d08d96) > from the [openjdk/jdk](https://git.openjdk.org/jdk) repository. > > The commit

Re: RFR: 8308899: Introduce Classfile.Context and improve Classfile.Option(s) [v21]

2023-06-21 Thread Chen Liang
On Thu, 15 Jun 2023 17:22:32 GMT, Adam Sotona wrote: >> Classfile context object and multi-state options have been discussed at >> https://mail.openjdk.org/pipermail/classfile-api-dev/2023-May/000321.html >> This patch implements the proposed changes in Classfile API and fixes all >> affected

Re: [jdk21] RFR: 8309937: Add @sealedGraph for some Panama FFM interfaces

2023-06-21 Thread Maurizio Cimadamore
On Mon, 19 Jun 2023 14:58:13 GMT, Per Minborg wrote: > Hi all, > > This pull request contains a backport of commit > [b412fc79](https://github.com/openjdk/jdk/commit/b412fc79c3c2548df10918090beedaf6b2d08d96) > from the [openjdk/jdk](https://git.openjdk.org/jdk) repository. > > The commit

Re: RFR: 8310502 : optimization for UUID#toString

2023-06-21 Thread Alan Bateman
On Wed, 21 Jun 2023 07:28:54 GMT, 温绍锦 wrote: > By optimizing the implementation of java.lang.Long#fastUUID, the performance > of the java.util.UUID#toString method can be significantly improved. > > The following are the test results of JMH: > > Benchmark Mode Cnt

Re: RFR: JDK-8310265: (process) jspawnhelper should not use argv[0] [v2]

2023-06-21 Thread Thomas Stuefe
On Wed, 21 Jun 2023 07:55:31 GMT, David Holmes wrote: >> I'm not aware of any other uses either; its not intended to be used outside >> of ProcessImpl especially since the addition of the new protocol to >> communicate parameters and confirm launching of the child. > > This curiosity has been

Re: RFR: JDK-8310265: (process) jspawnhelper should not use argv[0] [v2]

2023-06-21 Thread David Holmes
On Tue, 20 Jun 2023 16:27:38 GMT, Roger Riggs wrote: >> I wondered about this, it looked so deliberate. So I wondered whether >> jspawnhelper is used outside of the posix_spawn context, but cannot find >> anything. Maybe historic? > > I'm not aware of any other uses either; its not intended to

Re: RFR: 8308286 Fix clang warnings in linux code [v5]

2023-06-21 Thread Daniel Jeliński
On Sun, 11 Jun 2023 16:38:31 GMT, Artem Semenov wrote: >> When using the clang compiler to build OpenJDk on Linux, we encounter >> various "warnings as errors". >> They can be fixed with small changes. > > Artem Semenov has updated the pull request with a new target base due to a > merge or a

Re: RFR: 8310405: Linker.Option.firstVariadicArg should specify which index values are valid

2023-06-21 Thread Per Minborg
On Wed, 21 Jun 2023 00:07:54 GMT, Maurizio Cimadamore wrote: > Possible suggestion/thing to try: use a bullet list to spell out all cases > for `index`. E.g. we know there's index == 0 (all variadic). Then we know > there's index = N (no variadic). Then there's index == m, 0 < m < N - which

RFR: 8310502 : optimization for UUID#toString

2023-06-21 Thread 温绍锦
By optimizing the implementation of java.lang.Long#fastUUID, the performance of the java.util.UUID#toString method can be significantly improved. The following are the test results of JMH: Benchmark Mode Cnt Score Error Units UUIDUtilsBenchmark.new thrpt

  1   2   >