On Sat, 17 Jan 2026 18:15:50 GMT, Chris Plummer <[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/share/ClassUnloader.java line 262:
>
>> 260: if (isClassLoaderReclaimed()) {
>> 261: System.out.println("ClassUnloader: class loader has been
>> reclaimed.");
>> 262: Runtime.getRuntime().gc();
>
> It's not clear to me why this gc() is even needed after the WhiteBox gc()
> calls. What does it do that we haven't already accomplished with WhiteBox
> gc()?
This gc() seems meaningless but is harmless. Should we remove this gc() call.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/29285#discussion_r2703487032
