Re: RFR: 8308804: Improve UUID.randomUUID performance with bulk/scalable PRNG access

2023-05-25 Thread Hannes Greule
On Thu, 25 May 2023 23:54:14 GMT, Andrei Pangin wrote: >> UUID is the very important class that is used to track identities of objects >> in large scale systems. On some of our systems, `UUID.randomUUID` takes >1% >> of total CPU time, and is frequently a scalability bottleneck due to >> `Secu

Re: [External] : Re: Classes used in method body are loaded lazily or eagerly depending on method return type

2023-05-25 Thread David Holmes
On 25/05/2023 7:21 pm, Raffaello Giulietti wrote: Yes, ChildClass.class is removed from the filesystem. And here's, the relevant info when running with -Xlog:class+init, showing that verification succeeds for both TestLoading$ObjectReturner and TestLoading$BaseClassReturner: loading: TestL

Re: RFR: JDK-8306112 Implementation of JEP 445: Unnamed Classes and Instance Main Methods (Preview) [v20]

2023-05-25 Thread Rémi Forax
On Fri, 26 May 2023 06:07:05 GMT, Joe Darcy wrote: >> Jim Laskey has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Improving error recovery in presence of less important syntactic errors in >> top-level methods and fields. >> >> Auth

Re: RFR: JDK-8306112 Implementation of JEP 445: Unnamed Classes and Instance Main Methods (Preview) [v20]

2023-05-25 Thread Joe Darcy
On Thu, 25 May 2023 14:32:44 GMT, Jim Laskey wrote: >> Add flexible main methods and anonymous main classes to the Java language. > > Jim Laskey has updated the pull request incrementally with one additional > commit since the last revision: > > Improving error recovery in presence of less im

Re: RFR: 8308804: Improve UUID.randomUUID performance with bulk/scalable PRNG access

2023-05-25 Thread Brett Okken
On Wed, 24 May 2023 19:36:44 GMT, Aleksey Shipilev wrote: > UUID is the very important class that is used to track identities of objects > in large scale systems. On some of our systems, `UUID.randomUUID` takes >1% > of total CPU time, and is frequently a scalability bottleneck due to > `Secur

Re: RFR: 8308804: Improve UUID.randomUUID performance with bulk/scalable PRNG access

2023-05-25 Thread Andrei Pangin
On Wed, 24 May 2023 19:36:44 GMT, Aleksey Shipilev wrote: > UUID is the very important class that is used to track identities of objects > in large scale systems. On some of our systems, `UUID.randomUUID` takes >1% > of total CPU time, and is frequently a scalability bottleneck due to > `Secur

Re: RFR: 8308293: A linker should expose the layouts it supports [v6]

2023-05-25 Thread Paul Sandoz
On Wed, 24 May 2023 09:36:34 GMT, Maurizio Cimadamore wrote: >> This patch adds an instance method on `Linker`, namely >> `Linker::canonicalLayouts` which returns all the layouts known by the linker >> as implementing some ABI type. For instance, if I call this on my machine >> (Linux/x64) I

Re: RFR: 8308842: Consolidate exceptions thrown from Class-File API

2023-05-25 Thread ExE Boss
On Thu, 25 May 2023 08:29:37 GMT, Adam Sotona wrote: > Class-File API actually throws wide variety of exceptions based on the actual > situation. Complete error handling code must cover > `IndexOutOfBoundsException`, `IllegalStateException` and > `IllegalArgumentException`. > > Based on prev

Re: RFR: 8308316: Default decomposition mode in Collator [v3]

2023-05-25 Thread Naoto Sato
> Amending the description about the default decomposition mode in > `Collator.NO_DECOMPOSITION` javadoc. Naoto Sato has updated the pull request incrementally with one additional commit since the last revision: use present tense - Changes: - all: https://git.openjdk.org/jdk/p

Re: RFR: 8307944: ClassFileDumper should only load java.nio.file.Path if enabled

2023-05-25 Thread Roger Riggs
On Thu, 25 May 2023 20:23:27 GMT, Mandy Chung wrote: > Change `ClassFileDumper` constructor to take a `String` rather than `Path` to > avoid the file system implementation classes loaded during startup. In > addition, `FilePermission` also causes the default file system to be > initialized.

Integrated: 8159023: Engineering notation of DecimalFormat does not work as documented

2023-05-25 Thread Justin Lu
On Sat, 20 May 2023 00:00:42 GMT, Justin Lu wrote: > Please review this PR which updates the Scientific Notation section of > Decimal Format. It aims to resolve > [JDK-8159023](https://bugs.openjdk.org/browse/JDK-8159023) as well as > [JDK-6282188](https://bugs.openjdk.org/browse/JDK-6282188).

Re: RFR: 8308316: Default decomposition mode in Collator [v2]

2023-05-25 Thread Naoto Sato
> Amending the description about the default decomposition mode in > `Collator.NO_DECOMPOSITION` javadoc. Naoto Sato 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 req

Re: RFR: 8303530: Redefine JAXP Configuration File [v13]

2023-05-25 Thread Joe Wang
> Add a system property, jdk.xml.config.file, to return the path to a custom > JAXP configuration file. The current configuration file, jaxp.properties, > that the JDK supports will become the default configuration file. > > CSR: https://bugs.openjdk.org/browse/JDK-8303531 > > Tests: XML SQE an

Re: RFR: 8303530: Redefine JAXP Configuration File [v9]

2023-05-25 Thread Joe Wang
On Fri, 5 May 2023 08:12:01 GMT, Alan Bateman wrote: >> Changed the opening statement and also the naming of config file as above. > > I've looked through the updated proposal but I think the intro is going to > take a few iterations to get to something that is easy to read. I had hoped > the i

Re: RFR: 8303530: Redefine JAXP Configuration File [v11]

2023-05-25 Thread Joe Wang
On Fri, 5 May 2023 08:16:14 GMT, Alan Bateman wrote: >> Joe Wang has updated the pull request incrementally with one additional >> commit since the last revision: >> >> address review comments > > src/java.xml/share/conf/jaxp.properties line 2: > >> 1: >> ###

Re: RFR: 8303530: Redefine JAXP Configuration File [v12]

2023-05-25 Thread Joe Wang
> Add a system property, jdk.xml.config.file, to return the path to a custom > JAXP configuration file. The current configuration file, jaxp.properties, > that the JDK supports will become the default configuration file. > > CSR: https://bugs.openjdk.org/browse/JDK-8303531 > > Tests: XML SQE an

Re: RFR: 8306647: Implementation of Structured Concurrency (Preview) [v2]

2023-05-25 Thread Paul Sandoz
On Thu, 25 May 2023 08:21:39 GMT, Alan Bateman wrote: >> This is the implementation of: >> >> - JEP 453: Structured Concurrency (Preview) >> - JEP 446: Scoped Values (Preview) >> >> For the most part, this is just moving code and tests. StructuredTaskScope >> moves to j.u.concurrent as a prev

RFR: 8307944: ClassFileDumper should only load java.nio.file.Path if enabled

2023-05-25 Thread Mandy Chung
Change `ClassFileDumper` constructor to take a `String` rather than `Path` to avoid the file system implementation classes loaded during startup. In addition, `FilePermission` also causes the default file system to be initialized. This patch changes `ClassFileDumper` to validate the given pat

Re: RFR: 8306647: Implementation of Structured Concurrency (Preview) [v2]

2023-05-25 Thread Paul Sandoz
On Thu, 25 May 2023 08:21:39 GMT, Alan Bateman wrote: >> This is the implementation of: >> >> - JEP 453: Structured Concurrency (Preview) >> - JEP 446: Scoped Values (Preview) >> >> For the most part, this is just moving code and tests. StructuredTaskScope >> moves to j.u.concurrent as a prev

Re: RFR: 8306647: Implementation of Structured Concurrency (Preview) [v2]

2023-05-25 Thread Paul Sandoz
On Thu, 25 May 2023 08:21:39 GMT, Alan Bateman wrote: >> This is the implementation of: >> >> - JEP 453: Structured Concurrency (Preview) >> - JEP 446: Scoped Values (Preview) >> >> For the most part, this is just moving code and tests. StructuredTaskScope >> moves to j.u.concurrent as a prev

Re: EnumSet doesn't implement SequencedSet

2023-05-25 Thread David Alayachew
Hello folks, Since you are asking, let me chime in and say that I use enum's very frequently. However, my most common use case for them isn't for boolean flags -- it's actually the strategy pattern. Meaning, attaching a piece of logic to each value. And more importantly, I order the strategies via

Re: RFR: JDK-8308288: Fix xlc17 clang warnings in shared code

2023-05-25 Thread Martin Doerr
On Thu, 25 May 2023 15:04:32 GMT, Kim Barrett wrote: >> When using the new xlc17 compiler (based on a recent clang) to build OpenJDk >> on AIX , we run into various "warnings as errors". >> Some of those are in shared codebase and could be addressed by small >> adjustments. >> A lot of those ch

Integrated: 8308108: Support Unicode extension for collation settings

2023-05-25 Thread Naoto Sato
On Wed, 17 May 2023 19:40:04 GMT, Naoto Sato wrote: > This change intends to interpret the BCP47 U extension wrt collation settings > in the given `Locale`, then applies them to the created instances in the > 1-arg factory method in `Collator`. A corresponding CSR has also been drafted. This p

Re: RFR: JDK-8266431: Dual-Pivot Quicksort improvements (Radix sort) [v6]

2023-05-25 Thread Laurent Bourgès
On Sun, 23 Apr 2023 17:33:38 GMT, Laurent Bourgès wrote: >> * improved mixed insertion sort (makes whole sorting faster) >> * introduced Radix which sort shows several times boost of performance and >> has linear complexity instead of n*ln(n) >> * improved merging sort for almost sorted data >>

Re: RFR: JDK-8308288: Fix xlc17 clang warnings in shared code

2023-05-25 Thread JoKern65
On Thu, 25 May 2023 15:18:41 GMT, Kim Barrett wrote: >> When using the new xlc17 compiler (based on a recent clang) to build OpenJDk >> on AIX , we run into various "warnings as errors". >> Some of those are in shared codebase and could be addressed by small >> adjustments. >> A lot of those ch

Re: RFR: 8308645: Javadoc of FFM API needs to be refreshed [v2]

2023-05-25 Thread Maurizio Cimadamore
> As the FFM API evolved over time, some parts of the javadoc went out of sync. > Now that most of the API is stable, it is a good time to look again at the > javadoc as a whole, and bring some more consistency. > > While most of the changes in this PR are stylistic, I'd like to call out few >

Re: RFR: 8307990: jspawnhelper must close its writing side of a pipe before reading from it [v7]

2023-05-25 Thread Volker Simonis
On Thu, 25 May 2023 15:25:40 GMT, Volker Simonis wrote: >> Since JDK13, executing commands in a sub-process defaults to the so called >> `POSIX_SPAWN` launching mechanism (i.e. >> `-Djdk.lang.Process.launchMechanism=POSIX_SPAWN`) on Linux. This works by >> using `posix_spawn(3)` to firstly sta

Re: RFR: 8307990: jspawnhelper must close its writing side of a pipe before reading from it [v7]

2023-05-25 Thread Volker Simonis
> Since JDK13, executing commands in a sub-process defaults to the so called > `POSIX_SPAWN` launching mechanism (i.e. > `-Djdk.lang.Process.launchMechanism=POSIX_SPAWN`) on Linux. This works by > using `posix_spawn(3)` to firstly start a tiny helper program called > `jspawnhelper` in a subproc

Re: RFR: JDK-8308288: Fix xlc17 clang warnings in shared code

2023-05-25 Thread Kim Barrett
On Thu, 25 May 2023 09:14:14 GMT, JoKern65 wrote: > When using the new xlc17 compiler (based on a recent clang) to build OpenJDk > on AIX , we run into various "warnings as errors". > Some of those are in shared codebase and could be addressed by small > adjustments. > A lot of those changes ar

Integrated: 8300491: SymbolLookup::libraryLookup accepts strings with terminators

2023-05-25 Thread Maurizio Cimadamore
On Wed, 24 May 2023 15:22:15 GMT, Maurizio Cimadamore wrote: > There is a difference in behavior between `System::loadLibrary` and > `SymbolLookup::libraryLookup(String)`. While the former catches library names > containing NULL chars (because, internally it uses Path/File logic, which > reje

Integrated: 8300038: Make new version of JNU_GetStringPlatformChars which checks for null characters

2023-05-25 Thread Michael McMahon
On Mon, 22 May 2023 13:19:04 GMT, Michael McMahon wrote: > This PR creates a new version of the JNI utility function > JNU_GetStringPlatformChars called JNU_GetStringPlatformCharsStrict, which > performs additional validation of the returned string, namely that it does > not contain any embedd

Re: RFR: JDK-8306112 Implementation of JEP 445: Unnamed Classes and Instance Main Methods (Preview) [v20]

2023-05-25 Thread Jim Laskey
> Add flexible main methods and anonymous main classes to the Java language. Jim Laskey has updated the pull request incrementally with one additional commit since the last revision: Improving error recovery in presence of less important syntactic errors in top-level methods and fields.

Re: RFR: 8300038: Make new version of JNU_GetStringPlatformChars which checks for null characters [v4]

2023-05-25 Thread Alan Bateman
On Thu, 25 May 2023 14:21:34 GMT, Michael McMahon wrote: > I'll integrate this today unless there are further comments. Thanks for addressing my comments. - PR Comment: https://git.openjdk.org/jdk/pull/14083#issuecomment-1563002770

Re: RFR: 8300038: Make new version of JNU_GetStringPlatformChars which checks for null characters [v4]

2023-05-25 Thread Michael McMahon
On Wed, 24 May 2023 20:46:23 GMT, Michael McMahon wrote: >> This PR creates a new version of the JNI utility function >> JNU_GetStringPlatformChars called JNU_GetStringPlatformCharsStrict, which >> performs additional validation of the returned string, namely that it does >> not contain any em

Re: RFR: 8308804: Improve UUID.randomUUID performance with bulk/scalable PRNG access

2023-05-25 Thread Aleksey Shipilev
On Thu, 25 May 2023 14:01:43 GMT, Alan Bateman wrote: >> UUID is the very important class that is used to track identities of objects >> in large scale systems. On some of our systems, `UUID.randomUUID` takes >1% >> of total CPU time, and is frequently a scalability bottleneck due to >> `Secur

Re: RFR: 8308804: Improve UUID.randomUUID performance with bulk/scalable PRNG access

2023-05-25 Thread Alan Bateman
On Wed, 24 May 2023 19:36:44 GMT, Aleksey Shipilev wrote: > UUID is the very important class that is used to track identities of objects > in large scale systems. On some of our systems, `UUID.randomUUID` takes >1% > of total CPU time, and is frequently a scalability bottleneck due to > `Secur

Re: RFR: JDK-8298066: java/util/concurrent/locks/Lock/OOMEInAQS.java timed out [v3]

2023-05-25 Thread Viktor Klang
On Thu, 25 May 2023 07:35:39 GMT, David Holmes wrote: >> @AlanBateman Sounds good, thanks for confirming. I've updated both the >> ProblemLists, but it needed yet another rebase, so yet another forced push. > >> it needed yet another rebase, so yet another forced push. > > You don't need to reb

Re: RFR: JDK-8308288: Fix xlc17 clang warnings in shared code

2023-05-25 Thread Matthias Baesken
On Thu, 25 May 2023 09:14:14 GMT, JoKern65 wrote: > When using the new xlc17 compiler (based on a recent clang) to build OpenJDk > on AIX , we run into various "warnings as errors". > Some of those are in shared codebase and could be addressed by small > adjustments. > A lot of those changes ar

RFR: 8308856: jdk.internal.classfile.impl.EntryMap::nextPowerOfTwo math problem

2023-05-25 Thread Adam Sotona
Fix of jdk.internal.classfile.impl.EntryMap::nextPowerOfTwo returning correct zero power of two. Please review. Thanks, Adam - Commit messages: - 8308856: jdk.internal.classfile.impl.EntryMap::nextPowerOfTwo math problem Changes: https://git.openjdk.org/jdk/pull/14148/files Webr

Re: RFR: 8308804: Improve UUID.randomUUID performance with bulk/scalable PRNG access

2023-05-25 Thread Alan Bateman
On Wed, 24 May 2023 19:36:44 GMT, Aleksey Shipilev wrote: > UUID is the very important class that is used to track identities of objects > in large scale systems. On some of our systems, `UUID.randomUUID` takes >1% > of total CPU time, and is frequently a scalability bottleneck due to > `Secur

RFR: 8308804: Improve UUID.randomUUID performance with bulk/scalable PRNG access

2023-05-25 Thread Aleksey Shipilev
UUID is the very important class that is used to track identities of objects in large scale systems. On some of our systems, `UUID.randomUUID` takes >1% of total CPU time, and is frequently a scalability bottleneck due to `SecureRandom` synchronization. The major issue with UUID code itself is

Re: RFR: 8301492: Modernize equals() method of ResourceBundle.CacheKey and Bundles.CacheKey [v3]

2023-05-25 Thread Sergey Tsypanov
On Wed, 1 Feb 2023 10:36:12 GMT, Sergey Tsypanov wrote: >> `ResourceBundle.CacheKey.equals()` and `Bundles.CacheKey.equals()` are quire >> outdated. This simple clean-up modernizes them. > > Sergey Tsypanov has updated the pull request incrementally with one > additional commit since the last r

Integrated: 8308735: Typos in parameter names

2023-05-25 Thread Pavel Rappo
On Wed, 24 May 2023 23:32:18 GMT, Pavel Rappo wrote: > Please review this simple fix. This pull request has now been integrated. Changeset: 38367d3c Author:Pavel Rappo URL: https://git.openjdk.org/jdk/commit/38367d3c3ad9292b7c581917c89e9f07fac3dd31 Stats: 9 lines in 3 files chan

Re: RFR: 8308842: Consolidate exceptions thrown from Class-File API

2023-05-25 Thread Chen Liang
On Thu, 25 May 2023 08:29:37 GMT, Adam Sotona wrote: > Class-File API actually throws wide variety of exceptions based on the actual > situation. Complete error handling code must cover > `IndexOutOfBoundsException`, `IllegalStateException` and > `IllegalArgumentException`. > > Based on prev

Re: RFR: 8308842: Consolidate exceptions thrown from Class-File API

2023-05-25 Thread Adam Sotona
On Thu, 25 May 2023 09:12:58 GMT, Chen Liang wrote: > What other types of exceptions does javap distinguish, and should Classfile > API distinguish them as well? Invalid CP index or content are cases where javap tries to recover, replace the value with '???' and print the rest. This PR is extr

RFR: JDK-8308288: Fix xlc17 clang warnings in shared code

2023-05-25 Thread JoKern65
When using the new xlc17 compiler (based on a recent clang) to build OpenJDk on AIX , we run into various "warnings as errors". Some of those are in shared codebase and could be addressed by small adjustments. A lot of those changes are in hotspot, some might be somewhere else in the OpenJDK C/C

Integrated: 8308475: Make the thread dump files generated by jcmd Thread.dump_to_file jtreg failure handler action easily accessible

2023-05-25 Thread Jaikiran Pai
On Sun, 21 May 2023 09:19:47 GMT, Jaikiran Pai wrote: > Can I please get a review of this change which proposes to improve the > accessibility of the thread dump files that are generated by the `jcmd > Thread.dump_to_file` command configured in the failure handler > configurations? This addres

Re: RFR: 8308475: Make the thread dump files generated by jcmd Thread.dump_to_file jtreg failure handler action easily accessible

2023-05-25 Thread Jaikiran Pai
On Sun, 21 May 2023 09:19:47 GMT, Jaikiran Pai wrote: > Can I please get a review of this change which proposes to improve the > accessibility of the thread dump files that are generated by the `jcmd > Thread.dump_to_file` command configured in the failure handler > configurations? This addres

Re: RFR: 8308780: Fix the Java Integer types on Windows

2023-05-25 Thread Julian Waters
On Wed, 24 May 2023 13:56:05 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 > rel

Re: [External] : Re: Classes used in method body are loaded lazily or eagerly depending on method return type

2023-05-25 Thread Raffaello Giulietti
Yes, ChildClass.class is removed from the filesystem. And here's, the relevant info when running with -Xlog:class+init, showing that verification succeeds for both TestLoading$ObjectReturner and TestLoading$BaseClassReturner: loading: TestLoading$ObjectReturner... [0.039s][info][class,init

Re: RFR: 8308842: Consolidate exceptions thrown from Class-File API

2023-05-25 Thread Chen Liang
On Thu, 25 May 2023 08:29:37 GMT, Adam Sotona wrote: > Class-File API actually throws wide variety of exceptions based on the actual > situation. Complete error handling code must cover > `IndexOutOfBoundsException`, `IllegalStateException` and > `IllegalArgumentException`. > > Based on prev

Re: RFR: 8307795: AArch64: Optimize VectorMask.truecount() on Neon [v3]

2023-05-25 Thread Andrew Haley
On Thu, 18 May 2023 09:50:13 GMT, Chang Peng wrote: >> In Vector API Java level, vector mask is represented as a boolean array with >> 0x00/0x01 (8 bits of each element) as values, aka in-memory format. When it >> is loaded into vector register, e.g. Neon, the in-memory format will be >> conve

Re: RFR: 8308780: Fix the Java Integer types on Windows

2023-05-25 Thread Julian Waters
On Wed, 24 May 2023 13:56:05 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 > rel

RFR: 8308842: Consolidate exceptions thrown from Class-File API

2023-05-25 Thread Adam Sotona
Class-File API actually throws wide variety of exceptions based on the actual situation. Complete error handling code must cover `IndexOutOfBoundsException`, `IllegalStateException` and `IllegalArgumentException`. Based on previous discussions we decided to consolidate on `IllegalArgumentExcep

Re: RFR: 8308475: Make the thread dump files generated by jcmd Thread.dump_to_file jtreg failure handler action easily accessible

2023-05-25 Thread Alan Bateman
On Thu, 25 May 2023 04:46:04 GMT, Jaikiran Pai wrote: >> Can I please get a review of this change which proposes to improve the >> accessibility of the thread dump files that are generated by the `jcmd >> Thread.dump_to_file` command configured in the failure handler >> configurations? This ad

Re: RFR: 8306647: Implementation of Structured Concurrency (Preview) [v2]

2023-05-25 Thread Alan Bateman
> This is the implementation of: > > - JEP 453: Structured Concurrency (Preview) > - JEP 446: Scoped Values (Preview) > > For the most part, this is just moving code and tests. StructuredTaskScope > moves to j.u.concurrent as a preview API, ScopedValue moves to j.lang as a > preview API, and m

Re: RFR: 8294969: Convert jdk.jdeps javap to use the Classfile API [v3]

2023-05-25 Thread Adam Sotona
> javap uses proprietary com.sun.tools.classfile library to parse class files. > > This patch converts javap to use Classfile API. > > Please review. > > Thanks, > Adam Adam Sotona has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains 217

Re: RFR: JDK-8298066: java/util/concurrent/locks/Lock/OOMEInAQS.java timed out [v3]

2023-05-25 Thread David Holmes
On Wed, 24 May 2023 08:30:21 GMT, Viktor Klang wrote: > it needed yet another rebase, so yet another forced push. You don't need to rebase (and force-push) you just merge with the master branch and skara will take care of flattening the commits. - PR Comment: https://git.openjdk.o

Re: RFR: 8308780: Fix the Java Integer types on Windows

2023-05-25 Thread David Holmes
On Thu, 25 May 2023 07:22:50 GMT, Julian Waters wrote: > All the changes from long were done since there was conversion from or to a > jint somewhere down the line, Okay I see that now. It is a messy situation - at some point the incoming jint's need to be "converted" to a native type. --

Re: RFR: 8308780: Fix the Java Integer types on Windows

2023-05-25 Thread Julian Waters
On Wed, 24 May 2023 13:56:05 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 > rel

Re: RFR: 8308780: Fix the Java Integer types on Windows

2023-05-25 Thread Julian Waters
On Thu, 25 May 2023 01:34:15 GMT, David Holmes wrote: > I think the JNI type definition change is okay. > > However many of the other changes appear to me to not involve Java variables > and so don't need to be Java types i.e they should be `int` rather than > `jint` - though as these are nati

Re: RFR: 8308475: Make the thread dump files generated by jcmd Thread.dump_to_file jtreg failure handler action easily accessible

2023-05-25 Thread David Holmes
On Thu, 25 May 2023 04:46:04 GMT, Jaikiran Pai wrote: >> Can I please get a review of this change which proposes to improve the >> accessibility of the thread dump files that are generated by the `jcmd >> Thread.dump_to_file` command configured in the failure handler >> configurations? This ad

Integrated: 8308549: Classfile API should fail to generate over-sized Code attribute

2023-05-25 Thread Adam Sotona
On Tue, 23 May 2023 12:54:20 GMT, Adam Sotona wrote: > Classfile API allowed to generate Code attribute exceeding the 65k limit. No > exception has been thrown during class generation and the class failed > verification later during class loading. > This patch adds Code size limit check throwin