Re: [jdk19] RFR: 8287847: Fatal Error when suspending virtual thread after it has terminated [v2]

2022-06-29 Thread Alan Bateman
On Wed, 29 Jun 2022 23:24:28 GMT, Chris Plummer wrote: >> test/jdk/com/sun/jdi/SuspendAfterDeath.java line 62: >> >>> 60: public class SuspendAfterDeath extends TestScaffold { >>> 61: private volatile ThreadReference thread; >>> 62: private volatile boolean breakpointReached = false; >>

Re: [jdk19] RFR: 8289278: Suspend/ResumeAllVirtualThreads need both can_suspend and can_support_virtual_threads

2022-06-29 Thread David Holmes
On Tue, 28 Jun 2022 09:02:59 GMT, Alan Bateman wrote: > This is spec only change to the JVM TI spec. The `SuspendAllVirtualThreads` > and `ResumeAllVirtualThreads` functions added in Java 19 currently specify > that they require one of the capabilities `can_suspend` or > `can_support_virtual_t

Re: RFR: 8256811: Delayed/missed jdwp class unloading events [v5]

2022-06-29 Thread Zhengyu Gu
On Wed, 29 Jun 2022 22:33:32 GMT, Chris Plummer wrote: > > > I didn't see any response to my suggested improvements to the test. Can > > > you please comment on them and update the test if you agree? Thanks. > > > > > > I am still learning JDI and not quite sure how it works. But I think it is

Re: [jdk19] RFR: 8287847: Fatal Error when suspending virtual thread after it has terminated [v2]

2022-06-29 Thread Chris Plummer
On Wed, 29 Jun 2022 08:55:22 GMT, Alan Bateman wrote: >> Chris Plummer has updated the pull request incrementally with one additional >> commit since the last revision: >> >> fix indentation > > src/jdk.jdwp.agent/share/native/libjdwp/threadControl.c line 418: > >> 416: // Thre

Re: [jdk19] RFR: 8287847: Fatal Error when suspending virtual thread after it has terminated [v2]

2022-06-29 Thread Chris Plummer
> This fixes a bug in the debug agent when there is a request to suspend a > virtual thread that has already terminated. The issue was that unless the > debug agent was currently under a "suspend all", it would not properly put > the virtual thread on the `otherThreads` list, and instead added i

Re: RFR: 8256811: Delayed/missed jdwp class unloading events [v9]

2022-06-29 Thread Chris Plummer
On Wed, 29 Jun 2022 19:14:39 GMT, Zhengyu Gu wrote: >> Currently, jdi only check and process class unloading event when it detects >> a new GC cycle. >> >> After [JDK-8212879](https://bugs.openjdk.org/browse/JDK-8212879), posting >> class events can overlap with GC finish event, that results,

Re: RFR: 8256811: Delayed/missed jdwp class unloading events [v5]

2022-06-29 Thread Chris Plummer
On Wed, 29 Jun 2022 12:40:42 GMT, Zhengyu Gu wrote: > > I didn't see any response to my suggested improvements to the test. Can you > > please comment on them and update the test if you agree? Thanks. > > I am still learning JDI and not quite sure how it works. But I think it is > unrelated to

Re: RFR: 8289440: Remove vmTestbase/nsk/monitoring/MemoryPoolMBean/isCollectionUsageThresholdExceeded/isexceeded003 from ProblemList.txt

2022-06-29 Thread Alex Menkov
On Wed, 29 Jun 2022 09:57:29 GMT, Kevin Walls wrote: > More context in the bug, but there's no evidence that this test should still > be problemlisted. > > Adding enableVerbose in the test itself should still keep this trivial: > seeing what actually happened in a test that was once labelled a

Re: RFR: 8256811: Delayed/missed jdwp class unloading events [v9]

2022-06-29 Thread Zhengyu Gu
> Currently, jdi only check and process class unloading event when it detects a > new GC cycle. > > After [JDK-8212879](https://bugs.openjdk.org/browse/JDK-8212879), posting > class events can overlap with GC finish event, that results, sometimes, it > only captures partial or even empty unload

Withdrawn: 8289231: Move java_lang_VirtualThread to continuationJavaClasses.hpp

2022-06-29 Thread Ioi Lam
On Mon, 27 Jun 2022 19:13:40 GMT, Ioi Lam wrote: > This is a follow up to > [JDK-8288623](https://bugs.openjdk.org/browse/JDK-8288623). > `java_lang_VirtualThread` is part of loom and should be move to > continuationJavaClasses.hpp, which contains the accessors for the other java > classes us

Re: RFR: 8289231: Move java_lang_VirtualThread to continuationJavaClasses.hpp

2022-06-29 Thread Ioi Lam
On Mon, 27 Jun 2022 19:13:40 GMT, Ioi Lam wrote: > This is a follow up to > [JDK-8288623](https://bugs.openjdk.org/browse/JDK-8288623). > `java_lang_VirtualThread` is part of loom and should be move to > continuationJavaClasses.hpp, which contains the accessors for the other java > classes us

Re: RFR: 8256811: Delayed/missed jdwp class unloading events [v8]

2022-06-29 Thread Zhengyu Gu
> Currently, jdi only check and process class unloading event when it detects a > new GC cycle. > > After [JDK-8212879](https://bugs.openjdk.org/browse/JDK-8212879), posting > class events can overlap with GC finish event, that results, sometimes, it > only captures partial or even empty unload

Re: [jdk19] RFR: 8289278: Suspend/ResumeAllVirtualThreads need both can_suspend and can_support_virtual_threads

2022-06-29 Thread Serguei Spitsyn
On Tue, 28 Jun 2022 09:02:59 GMT, Alan Bateman wrote: > This is spec only change to the JVM TI spec. The `SuspendAllVirtualThreads` > and `ResumeAllVirtualThreads` functions added in Java 19 currently specify > that they require one of the capabilities `can_suspend` or > `can_support_virtual_t

Re: [jdk19] RFR: 8289278: Suspend/ResumeAllVirtualThreads need both can_suspend and can_support_virtual_threads

2022-06-29 Thread Daniel D . Daugherty
On Tue, 28 Jun 2022 09:02:59 GMT, Alan Bateman wrote: > This is spec only change to the JVM TI spec. The `SuspendAllVirtualThreads` > and `ResumeAllVirtualThreads` functions added in Java 19 currently specify > that they require one of the capabilities `can_suspend` or > `can_support_virtual_t

Re: RFR: 8256811: Delayed/missed jdwp class unloading events [v5]

2022-06-29 Thread Zhengyu Gu
On Tue, 28 Jun 2022 23:51:20 GMT, Chris Plummer wrote: > It looks like TestClassUnloadEvents is failing on Windows-x64: > https://github.com/openjdk/jdk/pull/9168/checks?check_run_id=7102500258 Thanks for pointing out. I could not reproduce locally so far, but continue debugging it. -

Re: RFR: 8256811: Delayed/missed jdwp class unloading events [v7]

2022-06-29 Thread Zhengyu Gu
> Currently, jdi only check and process class unloading event when it detects a > new GC cycle. > > After [JDK-8212879](https://bugs.openjdk.org/browse/JDK-8212879), posting > class events can overlap with GC finish event, that results, sometimes, it > only captures partial or even empty unload

Re: RFR: 8286397: Address possibly lossy conversions in jdk.hotspot.agent [v3]

2022-06-29 Thread Chris Hegarty
On Sat, 25 Jun 2022 13:08:30 GMT, Ryan Ernst wrote: >> Applied required casts in jdk.hotspot.agent for the upcoming warning. >> Verified by cherry-picking @asotona's patch. > > Ryan Ernst has updated the pull request with a new target base due to a merge > or a rebase. The incremental webrev exc

Re: [jdk19] RFR: 8289278: Suspend/ResumeAllVirtualThreads need both can_suspend and can_support_virtual_threads

2022-06-29 Thread Serguei Spitsyn
On Tue, 28 Jun 2022 09:02:59 GMT, Alan Bateman wrote: > This is spec only change to the JVM TI spec. The `SuspendAllVirtualThreads` > and `ResumeAllVirtualThreads` functions added in Java 19 currently specify > that they require one of the capabilities `can_suspend` or > `can_support_virtual_t

[jdk19] RFR: 8289278: Suspend/ResumeAllVirtualThreads need both can_suspend and can_support_virtual_threads

2022-06-29 Thread Alan Bateman
This is spec only change to the JVM TI spec. The `SuspendAllVirtualThreads` and `ResumeAllVirtualThreads` functions added in Java 19 currently specify that they require one of the capabilities `can_suspend` or `can_support_virtual_threads`. This is not correct as both capabilities are required.

Integrated: 8285364: Remove REF_ enum for java.lang.ref.Reference

2022-06-29 Thread Albert Mingkun Yang
On Thu, 21 Apr 2022 10:48:06 GMT, Albert Mingkun Yang wrote: > Simple rename and some comments update. > > Test: build This pull request has now been integrated. Changeset: 2961b7ee Author:Albert Mingkun Yang URL: https://git.openjdk.org/jdk/commit/2961b7eede7205f8d67427bdf020de796

Re: RFR: 8285364: Remove REF_ enum for java.lang.ref.Reference [v10]

2022-06-29 Thread Albert Mingkun Yang
On Tue, 28 Jun 2022 19:43:33 GMT, Albert Mingkun Yang wrote: >> Simple rename and some comments update. >> >> Test: build > > Albert Mingkun Yang has updated the pull request incrementally with four > additional commits since the last revision: > > - arg rename > - default arg > - revert >

Re: RFR: 8256811: Delayed/missed jdwp class unloading events [v6]

2022-06-29 Thread Zhengyu Gu
> Currently, jdi only check and process class unloading event when it detects a > new GC cycle. > > After [JDK-8212879](https://bugs.openjdk.org/browse/JDK-8212879), posting > class events can overlap with GC finish event, that results, sometimes, it > only captures partial or even empty unload

Re: RFR: 8256811: Delayed/missed jdwp class unloading events [v5]

2022-06-29 Thread Zhengyu Gu
On Tue, 28 Jun 2022 23:54:20 GMT, Chris Plummer wrote: > I didn't see any response to my suggested improvements to the test. Can you > please comment on them and update the test if you agree? Thanks. I am still learning JDI and not quite sure how it works. But I think it is unrelated to this C

Re: RFR: 8285364: Remove REF_ enum for java.lang.ref.Reference [v10]

2022-06-29 Thread Stefan Karlsson
On Tue, 28 Jun 2022 19:43:33 GMT, Albert Mingkun Yang wrote: >> Simple rename and some comments update. >> >> Test: build > > Albert Mingkun Yang has updated the pull request incrementally with four > additional commits since the last revision: > > - arg rename > - default arg > - revert >

RFR: 8289440: Remove vmTestbase/nsk/monitoring/MemoryPoolMBean/isCollectionUsageThresholdExceeded/isexceeded003 from ProblemList.txt

2022-06-29 Thread Kevin Walls
More context in the bug, but there's no evidence that this test should still be problemlisted. Adding enableVerbose in the test itself should still keep this trivial: seeing what actually happened in a test that was once labelled as failing is really important! - Commit messages:

Re: [jdk19] RFR: 8287847: Fatal Error when suspending virtual thread after it has terminated

2022-06-29 Thread Alan Bateman
On Tue, 28 Jun 2022 22:33:07 GMT, Chris Plummer wrote: > This fixes a bug in the debug agent when there is a request to suspend a > virtual thread that has already terminated. The issue was that unless the > debug agent was currently under a "suspend all", it would not properly put > the virtu

RFR: 8289436: Make the redefine timer statistics more accurate

2022-06-29 Thread Tongbao Zhang
Make the redefine timer statistics more accurate - Commit messages: - 8289436: Make the redefine timer statistics more accurate Changes: https://git.openjdk.org/jdk/pull/9322/files Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=9322&range=00 Issue: https://bugs.openjdk.org/brow

Re: RFR: 8198668: MemoryPoolMBean/isUsageThresholdExceeded/isexceeded001/TestDescription.java still failing [v2]

2022-06-29 Thread Kevin Walls
On Tue, 28 Jun 2022 11:43:42 GMT, Kevin Walls wrote: >> Test has been problemlisted for a long time due to intermittent failures. >> >> This is a difficult test as it tries to monitor usage thresholds on Memory >> Pools which are outside its control. >> Not just Java heap pools, where the alloc