Re: RFR: 8332923: ObjectMonitorUsage.java failed with unexpected waiter_count [v2]

2024-05-27 Thread SendaoYan
> Hi all, > ObjectMonitorUsage.java failed with `unexpected waiter_count` after > [JDK-8328083](https://bugs.openjdk.org/browse/JDK-8328083) on linux x86_32. > It should be predicated with `@requires vm.continuations` to be skipped. > > Additional testing: > - [x] linux x86_32, test has been s

RFR: 8333013: Update vmTestbase/nsk/share/LocalProcess.java to don't use finalization

2024-05-27 Thread Leonid Mesnik
The vmTestbase/nsk/share/LocalProcess.java is a wrapper for debuggee process. It extends FinalizableObject to kill the debuggee process. The debuggee process is used by nsk.jdb tests only, see runTest(...) in vmTestbase/nsk/share/jdb/JdbTest.java: https://github.com/openjdk/jdk/blob/master/test/

Re: RFR: 8332923: ObjectMonitorUsage.java failed with unexpected waiter_count

2024-05-27 Thread Jiawei Tang
On Sun, 26 May 2024 17:02:33 GMT, SendaoYan wrote: > > > That would mean it's not tested. I suspect the > > > java_lang_VirtualThread::is_instance checks will need to be changed to > > > test with is_a(vmClasses::BaseVirtualThread_klass()) to allow for the > > > alternative implementation. > >

Re: RFR: 8332070: Convert package.html files in `java.management` to package-info.java [v2]

2024-05-27 Thread Nizar Benalla
> This is a simple noreg cleanup. The motivation was that I noticed javac > doesn't recognise package.html files well. > > Some of the contents of the `package.html` files (and code in the package) > may be outdated, but I think it is out of scope for this PR. > > I have also changed three `{@l