RFR: 8364343: Virtual Thread transition management needs to be independent of JVM TI [v5]
Patricio Chilano Mateo
pchilanomate at openjdk.org
Tue Nov 25 19:58:43 UTC 2025
On Thu, 20 Nov 2025 22:26:26 GMT, David Holmes <dholmes at openjdk.org> wrote:
>> Patricio Chilano Mateo has updated the pull request incrementally with one additional commit since the last revision:
>>
>> David's comments
>
> src/hotspot/share/prims/jvm.cpp line 3668:
>
>> 3666: if (!DoJVMTIVirtualThreadTransitions) {
>> 3667: assert(!JvmtiExport::can_support_virtual_threads(), "sanity check");
>> 3668: return;
>
> Does this not still need checking somewhere?
The check for `DoJVMTIVirtualThreadTransitions` was moved to the `JVMTIStartTransition\JVMTIEndTransition` classes, but I guess you refer to the assert: I missed to move it. Added now too.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/28361#discussion_r2561201839
More information about the graal-dev
mailing list