On Sun, 18 Jan 2026 05:59:58 GMT, Leonid Mesnik <[email protected]> wrote:

>> SendaoYan has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   1. Add small delay after check thrd.isAlive; 2. Add small delay between 
>> classloader reclaimed retries.
>
> test/hotspot/jtreg/vmTestbase/nsk/jvmti/scenarios/events/EM02/em02t003.java 
> line 109:
> 
>> 107: 
>> 108:             while (thrd.isAlive()) {
>> 109:                 logger.display("Thread state: " + thrd.getState()
> 
> Please add some sleep to don't produce huge amount of output.

Fixed.

> test/hotspot/jtreg/vmTestbase/nsk/share/ClassUnloader.java line 256:
> 
>> 254:         while (count++ < MAX_UNLOAD_ATTEMPS && 
>> !isClassLoaderReclaimed()) {
>> 255:             System.out.println("ClassUnloader: waiting for class loader 
>> reclaiming... " + count);
>> 256:             WhiteBox.getWhiteBox().fullGC();
> 
> Can you add some small delay (100ms) between attempts. To give more chances 
> to process ojbects inside VM like jvmti deferred queue.

Thanks for the suggestion. Thread.sleep(100) has been added after WB.fullgc().

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/29285#discussion_r2703461267
PR Review Comment: https://git.openjdk.org/jdk/pull/29285#discussion_r2703463604

Reply via email to