Re: RFR: 8313274: Failure building java.base-jmod target

2023-08-01 Thread David Holmes
On Tue, 1 Aug 2023 10:31:13 GMT, Jaikiran Pai wrote: >> Can I please get a review of this change which proposes to address the build >> failure noted in https://bugs.openjdk.org/browse/JDK-8313274? >> >> The build failure is consistently reproducible with `--with-jobs=1`. Martin, >> in that

Re: [jdk21] RFR: 8313023: Return value corrupted when using CCS + isTrivial (mainline)

2023-08-01 Thread Tobias Hartmann
On Mon, 31 Jul 2023 08:13:55 GMT, Jorn Vernee wrote: > Hi all, > > This pull request contains a backport of commit > [6fca2898](https://github.com/openjdk/jdk/commit/6fca28988794b52a6aa974bed1ed6f4f07e0994b) > from the [openjdk/jdk](https://git.openjdk.org/jdk) repository. > > The commit

Re: RFR: 8306582: Remove MetaspaceShared::exit_after_static_dump() [v4]

2023-08-01 Thread David Holmes
On Tue, 1 Aug 2023 21:02:36 GMT, Matias Saavedra Silva wrote: >> src/hotspot/share/oops/constantPool.cpp line 308: >> >>> 306: assert(java_lang_String::is_instance(obj), "must be"); >>> 307: if (!ArchiveHeapWriter::is_string_too_large_to_archive(obj)) >>> { >>> 308:

Re: RFR: 8313406: nep_invoker_blob can be simplified more

2023-08-01 Thread Jorn Vernee
On Mon, 31 Jul 2023 12:22:00 GMT, Yasumasa Suenaga wrote: > In FFM, native function would be called via `nep_invoker_blob`. If the > function has two arguments, it would be following: > > > Decoding RuntimeStub - nep_invoker_blob 0x7fcae394cd10 >

Integrated: 8310991: missing @since tags in java.xml

2023-08-01 Thread Joe Wang
On Tue, 1 Aug 2023 19:32:40 GMT, Joe Wang wrote: > Please review a patch that adds the @since tag to DOM package. This pull request has now been integrated. Changeset: 528596fa Author:Joe Wang URL: https://git.openjdk.org/jdk/commit/528596fa937f8589f71fa06929fbb75f62142a4f Stats:

Re: RFR: 8310991: missing @since tags in java.xml

2023-08-01 Thread Joe Wang
On Tue, 1 Aug 2023 19:32:40 GMT, Joe Wang wrote: > Please review a patch that adds the @since tag to DOM package. Thanks all! - PR Comment: https://git.openjdk.org/jdk/pull/15109#issuecomment-1661344704

Integrated: 8312572: JDK 21 RDP2 L10n resource files update

2023-08-01 Thread Justin Lu
On Wed, 26 Jul 2023 20:41:36 GMT, Justin Lu wrote: > Please review this PR which contains the translations for updates to > localized resources in the JDK since RDP1. > > Included in this change are improved translations for certain values, which > also includes global updates for

[jdk21] Integrated: 8039165: [Doc] MessageFormat null locale generates NullPointerException

2023-08-01 Thread Justin Lu
On Mon, 31 Jul 2023 21:13:16 GMT, Justin Lu wrote: > Please review this change which contains a backport of commit > [c6396dce](https://github.com/openjdk/jdk/commit/c6396dceb9a64578d5b335af27ad1d968190a1fa) > from the [openjdk/jdk](https://git.openjdk.org/jdk) repository. > > This PR is a

[jdk21] RFR: 8311822: AIX : test/jdk/java/foreign/TestLayouts.java fails because of different output - expected [[i4](struct)] but found [[I4](struct)]

2023-08-01 Thread Christoph Langer
Hi all, This pull request contains a backport of [JDK-8311822](https://bugs.openjdk.org/browse/JDK-8311822), commit [d1cc2782](https://github.com/openjdk/jdk/commit/d1cc2782606e8a3cfead9055aa845e48e851edd4) from the [openjdk/jdk](https://git.openjdk.org/jdk) repository. The commit being

Re: RFR: 8306582: Remove MetaspaceShared::exit_after_static_dump() [v5]

2023-08-01 Thread Calvin Cheung
On Tue, 1 Aug 2023 21:08:05 GMT, Matias Saavedra Silva wrote: >> Currently we exit the VM after static dumping with >> `MetaspaceShared::exit_after_static_dump()`. >> >> >> // We have finished dumping the static archive. At this point, there may be >> pending VM >> // operations. We have

Re: RFR: 8306582: Remove MetaspaceShared::exit_after_static_dump() [v5]

2023-08-01 Thread Matias Saavedra Silva
> Currently we exit the VM after static dumping with > `MetaspaceShared::exit_after_static_dump()`. > > > // We have finished dumping the static archive. At this point, there may be > pending VM > // operations. We have changed some global states (such as > vmClasses::_klasses) that > //

Re: RFR: 8306582: Remove MetaspaceShared::exit_after_static_dump() [v4]

2023-08-01 Thread Matias Saavedra Silva
On Tue, 1 Aug 2023 17:02:46 GMT, Calvin Cheung wrote: >> Matias Saavedra Silva has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Ioi comments > > src/hotspot/share/oops/constantPool.cpp line 308: > >> 306:

Re: RFR: 8309622: Re-examine the cache mechanism in BaseLocale [v5]

2023-08-01 Thread Naoto Sato
> This is stemming from the PR: https://github.com/openjdk/jdk/pull/14211 where > aggressive GC can cause NPE in `BaseLocale$Key` class. I refactored the > in-house cache with WeakHashMap, and removed the Key class as it is no longer > needed (thus the original NPE will no longer be possible).

Re: RFR: 8313274: Failure building java.base-jmod target

2023-08-01 Thread phuchau1989
On Tue, 1 Aug 2023 09:01:49 GMT, Jaikiran Pai wrote: > Can I please get a review of this change which proposes to address the build > failure noted in https://bugs.openjdk.org/browse/JDK-8313274? > > The build failure is consistently reproducible with `--with-jobs=1`. Martin, > in that JBS

Re: RFR: 8310991: missing @since tags in java.xml

2023-08-01 Thread Lance Andersen
On Tue, 1 Aug 2023 19:32:40 GMT, Joe Wang wrote: > Please review a patch that adds the @since tag to DOM package. Marked as reviewed by lancea (Reviewer). - PR Review: https://git.openjdk.org/jdk/pull/15109#pullrequestreview-1557752047

Re: RFR: 8310991: missing @since tags in java.xml

2023-08-01 Thread Naoto Sato
On Tue, 1 Aug 2023 19:32:40 GMT, Joe Wang wrote: > Please review a patch that adds the @since tag to DOM package. Marked as reviewed by naoto (Reviewer). - PR Review: https://git.openjdk.org/jdk/pull/15109#pullrequestreview-1557745601

Re: RFR: 8310991: missing @since tags in java.xml

2023-08-01 Thread Iris Clark
On Tue, 1 Aug 2023 19:32:40 GMT, Joe Wang wrote: > Please review a patch that adds the @since tag to DOM package. Added @since tags follow existing convention for XML APIs in java.xml and jdk.xml.dom modules. - Marked as reviewed by iris (Reviewer). PR Review:

RFR: 8310991: missing @since tags in java.xml

2023-08-01 Thread Joe Wang
Please review a patch that adds the @since tag to DOM package. - Commit messages: - 8310991: missing @since tags in java.xml Changes: https://git.openjdk.org/jdk/pull/15109/files Webrev: https://webrevs.openjdk.org/?repo=jdk=15109=00 Issue:

Re: RFR: 8306582: Remove MetaspaceShared::exit_after_static_dump() [v4]

2023-08-01 Thread Calvin Cheung
On Thu, 27 Jul 2023 19:09:00 GMT, Matias Saavedra Silva wrote: >> Currently we exit the VM after static dumping with >> `MetaspaceShared::exit_after_static_dump()`. >> >> >> // We have finished dumping the static archive. At this point, there may be >> pending VM >> // operations. We

Re: RFR: 8312522: Implementation of Foreign Function & Memory API

2023-08-01 Thread Jorn Vernee
On Tue, 1 Aug 2023 11:17:57 GMT, ExE Boss wrote: >> This patch contains the implementation of the foreign linker & memory API >> JEP for Java 22. The initial patch is composed of commits brought over >> directly from the [panama-foreign >> repo](https://github.com/openjdk/panama-foreign). The

Re: RFR: 8312522: Implementation of Foreign Function & Memory API

2023-08-01 Thread ExE Boss
On Tue, 1 Aug 2023 10:29:06 GMT, Jorn Vernee wrote: > This patch contains the implementation of the foreign linker & memory API JEP > for Java 22. The initial patch is composed of commits brought over directly > from the [panama-foreign repo](https://github.com/openjdk/panama-foreign). > The

Re: RFR: 8312522: Implementation of Foreign Function & Memory API

2023-08-01 Thread Jorn Vernee
On Tue, 1 Aug 2023 10:29:06 GMT, Jorn Vernee wrote: > This patch contains the implementation of the foreign linker & memory API JEP > for Java 22. The initial patch is composed of commits brought over directly > from the [panama-foreign repo](https://github.com/openjdk/panama-foreign). > The

RFR: 8312522: Implementation of Foreign Function & Memory API

2023-08-01 Thread Jorn Vernee
This patch contains the implementation of the foreign linker & memory API JEP for Java 22. The initial patch is composed of commits brought over directly from the [panama-foreign repo](https://github.com/openjdk/panama-foreign). The main changes found in this patch come from the following PRs:

[jdk21] Integrated: 8310033: Clarify return value of Java Time compareTo methods

2023-08-01 Thread Roger Riggs
On Mon, 31 Jul 2023 19:48:45 GMT, Roger Riggs wrote: > A doc-only change to improve javadoc in java.time. This pull request has now been integrated. Changeset: a228185e Author:Roger Riggs URL: https://git.openjdk.org/jdk21/commit/a228185ebbda9b2004262b494f648ab62078fcde Stats:

Re: RFR: 8313406: nep_invoker_blob can be simplified more

2023-08-01 Thread Jorn Vernee
On Mon, 31 Jul 2023 12:22:00 GMT, Yasumasa Suenaga wrote: > In FFM, native function would be called via `nep_invoker_blob`. If the > function has two arguments, it would be following: > > > Decoding RuntimeStub - nep_invoker_blob 0x7fcae394cd10 >

Withdrawn: 8313274: Failure building java.base-jmod target

2023-08-01 Thread Jaikiran Pai
On Tue, 1 Aug 2023 09:01:49 GMT, Jaikiran Pai wrote: > Can I please get a review of this change which proposes to address the build > failure noted in https://bugs.openjdk.org/browse/JDK-8313274? > > The build failure is consistently reproducible with `--with-jobs=1`. Martin, > in that JBS

Re: RFR: 8313274: Failure building java.base-jmod target

2023-08-01 Thread Jaikiran Pai
On Tue, 1 Aug 2023 10:30:18 GMT, Alan Bateman wrote: > It also opens the door to accidental mix 'n match of modules from different > JDK modules. So I don't think this the right change for this issue. - PR Comment: https://git.openjdk.org/jdk/pull/15102#issuecomment-1660035694

Re: RFR: 8313274: Failure building java.base-jmod target

2023-08-01 Thread Alan Bateman
On Tue, 1 Aug 2023 09:01:49 GMT, Jaikiran Pai wrote: > Can I please get a review of this change which proposes to address the build > failure noted in https://bugs.openjdk.org/browse/JDK-8313274? > > The build failure is consistently reproducible with `--with-jobs=1`. Martin, > in that JBS

Re: RFR: 8313274: Failure building java.base-jmod target

2023-08-01 Thread Jaikiran Pai
On Tue, 1 Aug 2023 09:01:49 GMT, Jaikiran Pai wrote: > Can I please get a review of this change which proposes to address the build > failure noted in https://bugs.openjdk.org/browse/JDK-8313274? > > The build failure is consistently reproducible with `--with-jobs=1`. Martin, > in that JBS

Re: RFR: 8311630: [s390] Implementation of Foreign Function & Memory API (Preview) [v3]

2023-08-01 Thread sid8606
On Mon, 31 Jul 2023 14:27:38 GMT, Amit Kumar wrote: >> src/hotspot/cpu/s390/frame_s390.cpp line 228: >> >>> 226: >>> 227: bool frame::upcall_stub_frame_is_first() const { >>> 228: assert(is_upcall_stub_frame(), "must be optimzed entry frame"); >> >> typo: s/optimzed/optimized > > @sid8606

Re: RFR: 8311630: [s390] Implementation of Foreign Function & Memory API (Preview) [v10]

2023-08-01 Thread sid8606
> Implementation of "Foreign Function & Memory API" for s390x (Big Endian). sid8606 has updated the pull request incrementally with one additional commit since the last revision: Fix typo Signed-off-by: Sidraya - Changes: - all: