RFR: 8377950: Refactor ClassUnloader to provide explicit waiting API [v2]
Albert Mingkun Yang
ayang at openjdk.org
Tue Feb 17 07:33:13 UTC 2026
On Tue, 17 Feb 2026 04:12:07 GMT, David Holmes <dholmes at openjdk.org> wrote:
>> Albert Mingkun Yang has updated the pull request incrementally with two additional commits since the last revision:
>>
>> - copyright
>> - review
>
> test/hotspot/jtreg/vmTestbase/nsk/jvmti/CompiledMethodUnload/compmethunload001.java line 98:
>
>> 96: // BackgroundCompilation is on by default so wait for compiler threads
>> 97: // to drop references to the to-be-unload class.
>> 98: if (!clsUnLoader.unloadClassAndWait(10_000)) {
>
> Why 10_000 when the implicit timeout was 10 * 100 ?
The previous timeout was probably insufficient -- [Test vmTestbase/nsk/jvmti/CompiledMethodUnload/compmethunload001/TestDescription.java is still failing](https://bugs.openjdk.org/browse/JDK-8376547).
Now that the timeout is specific to this test, we can use a somewhat larger value, without affecting other tests using `ClassUnloader`. (The value 10s was suggested by Chris Plummer. Can revise if there are better alternatives.)
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/29735#discussion_r2815397768
More information about the serviceability-dev
mailing list