RFR: 8299414: JVMTI FollowReferences should support references from VirtualThread stack [v19]
Kevin Walls
kevinw at openjdk.org
Mon May 22 19:30:09 UTC 2023
On Wed, 10 May 2023 23:41:07 GMT, Alex Menkov <amenkov at openjdk.org> wrote:
>> The fix updates JVMTI FollowReferences implementation to report references from virtual threads:
>> - unmounted vthreads are detected, their stack references for JVMTI_HEAP_REFERENCE_STACK_LOCAL/JVMTI_HEAP_REFERENCE_JNI_LOCAL;
>> - stacks of mounted vthreads are splitted into 2 parts (virtual thread stack and carrier thread stack), references are reported with correct thread id/class tag/object tags/frame depth;
>> - common code to handle stack frames are moved into separate class;
>>
>> Threads are reported as:
>> - platform threads: JVMTI_HEAP_REFERENCE_THREAD (as before);
>> - mounted vthreads (synthetic references, consider them as heap roots because carrier threads are roots): JVMTI_HEAP_REFERENCE_OTHER;
>> - unmounted vthreads: not reported as heap roots.
>
> Alex Menkov has updated the pull request incrementally with one additional commit since the last revision:
>
> some refactoring
>
> added StackRefCollector::process_frames;
> used single RegisterMap instance;
> used RegisterMap::WalkContinuation::include for RegisterMap;
I spent some time looking through this and follow enough to say I think it looks good.
-------------
Marked as reviewed by kevinw (Committer).
PR Review: https://git.openjdk.org/jdk/pull/13254#pullrequestreview-1437321344
More information about the hotspot-dev
mailing list